soundwire: intel_ace2x: use DOAIS and DODS settings from firmware
authorPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Mon, 29 Apr 2024 00:43:19 +0000 (00:43 +0000)
committerVinod Koul <vkoul@kernel.org>
Sat, 4 May 2024 12:56:50 +0000 (18:26 +0530)
Starting with LNL, the recommendation is to use settings read from DSD
properties instead of hard-coding the values.

The DOAIS and DODS values are completely-specific to Intel and are
stored in a vendor-specific property structure.

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Link: https://lore.kernel.org/r/20240429004321.2399754-3-yung-chuan.liao@linux.intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/soundwire/intel.h
drivers/soundwire/intel_ace2x.c
drivers/soundwire/intel_auxdevice.c
include/linux/soundwire/sdw.h

index 511932c55216c77da4f34d54fcd40501b0b1e282..b36d46319f0f351080e419c0f911a85011bacc55 100644 (file)
@@ -58,6 +58,11 @@ struct sdw_intel {
 #endif
 };
 
+struct sdw_intel_prop {
+       u16 doais;
+       u16 dods;
+};
+
 enum intel_pdi_type {
        INTEL_PDI_IN = 0,
        INTEL_PDI_OUT = 1,
index 32b538cd6d66f3ff8e5de535a6ac4cd2491c1025..917cc79d4d853f6353e1ce568aeb2e5c3cc70f93 100644 (file)
 static void intel_shim_vs_init(struct sdw_intel *sdw)
 {
        void __iomem *shim_vs = sdw->link_res->shim_vs;
+       struct sdw_bus *bus = &sdw->cdns.bus;
+       struct sdw_intel_prop *intel_prop;
        u16 doais;
        u16 dods;
        u16 act;
 
-       doais = 0x3;
-       dods = 0x1;
+       intel_prop = bus->vendor_specific_prop;
+       doais = intel_prop->doais;
+       dods = intel_prop->dods;
 
        act = intel_readw(shim_vs, SDW_SHIM2_INTEL_VS_ACTMCTL);
        u16p_replace_bits(&act, doais, SDW_SHIM2_INTEL_VS_ACTMCTL_DOAIS);
index 296a470ce1e072c5d412051e5e0fe188c50c2b9d..697d640707940f9505cfedac95eb9848deaa3a32 100644 (file)
@@ -122,6 +122,7 @@ static void generic_new_peripheral_assigned(struct sdw_bus *bus,
 static int sdw_master_read_intel_prop(struct sdw_bus *bus)
 {
        struct sdw_master_prop *prop = &bus->prop;
+       struct sdw_intel_prop *intel_prop;
        struct fwnode_handle *link;
        char name[32];
        u32 quirk_mask;
@@ -153,6 +154,26 @@ static int sdw_master_read_intel_prop(struct sdw_bus *bus)
        prop->quirks = SDW_MASTER_QUIRKS_CLEAR_INITIAL_CLASH |
                SDW_MASTER_QUIRKS_CLEAR_INITIAL_PARITY;
 
+       intel_prop = devm_kzalloc(bus->dev, sizeof(*intel_prop), GFP_KERNEL);
+       if (!intel_prop)
+               return -ENOMEM;
+
+       /* initialize with hardware defaults, in case the properties are not found */
+       intel_prop->doais = 0x3;
+       intel_prop->dods  = 0x1;
+
+       fwnode_property_read_u16(link,
+                                "intel-sdw-doais",
+                                &intel_prop->doais);
+       fwnode_property_read_u16(link,
+                                "intel-sdw-dods",
+                                &intel_prop->dods);
+       bus->vendor_specific_prop = intel_prop;
+
+       dev_dbg(bus->dev, "doais %#x dods %#x\n",
+               intel_prop->doais,
+               intel_prop->dods);
+
        return 0;
 }
 
index 7bb9119f30699b09250b2c5db4af3e5516bc9ee4..13e96d8b7423cb459850136600e13022f22ab4ca 100644 (file)
@@ -886,6 +886,7 @@ struct sdw_master_ops {
  * @port_ops: Master port callback ops
  * @params: Current bus parameters
  * @prop: Master properties
+ * @vendor_specific_prop: pointer to non-standard properties
  * @m_rt_list: List of Master instance of all stream(s) running on Bus. This
  * is used to compute and program bus bandwidth, clock, frame shape,
  * transport and port parameters
@@ -920,6 +921,7 @@ struct sdw_bus {
        const struct sdw_master_port_ops *port_ops;
        struct sdw_bus_params params;
        struct sdw_master_prop prop;
+       void *vendor_specific_prop;
        struct list_head m_rt_list;
 #ifdef CONFIG_DEBUG_FS
        struct dentry *debugfs;