On Mon, May 11, 2015 at 04:23:59PM +0530, Vinod Koul wrote:
From: Jeeja KP jeeja.kp@intel.com
ASoC hda codec driver will use id_table for registration. id_table has info of codec vendor_id, revision_id and name. For ASoC device/driver matching, device vendor_id, rev_id will be matched to id_table to identify the device/driver.
I'm sorry but I still don't entirely understand what this is supposed to do. It *looks* like it's trying to create a bus for HDA with this:
+/* HDA device table */ +#define HDA_NAME_SIZE 20 +struct soc_hda_device_id {
- __u32 id;
- __u32 rev_id;
- char name[HDA_NAME_SIZE];
- unsigned long driver_data;
+};
+struct soc_hda_codec_driver {
- struct hdac_driver core;
- const struct soc_hda_device_id *id_table;
+};
but it's not actually defining a driver model bus type (though it does use driver_register()) and I'd expect the bus type to be provided by the generic HDA code. Looking (fairly quickly admittedly) at the HDA code it's not entirely obvious how it all fits together and the changelog for that just talks about moving code around.
We really need to make sure we don't end up with the same shambles we've got for AC'97 here (though we're unlikely to get similar touchscreen type things so it should be less bad).