[alsa-devel] [RFC v3 06/11] ASoC: hdac_hda: add ASoC based HDA codec driver

[Ughreja, Rakesh A]

>-----Original Message-----
>From: Takashi Iwai [mailto:tiwai at suse.de]
>Sent: Tuesday, December 19, 2017 4:58 PM
>To: Ughreja, Rakesh A <rakesh.a.ughreja at intel.com>
>Cc: alsa-devel at alsa-project.org; Koul, Vinod <vinod.koul at intel.com>; pierre-
>louis.bossart at linux.intel.com; liam.r.girdwood at linux.intel.com; Patches Audio
><patches.audio at intel.com>; broonie at kernel.org
>Subject: Re: [alsa-devel] [RFC v3 06/11] ASoC: hdac_hda: add ASoC based HDA
>codec driver
>
>>
>> Hi Takashi,
>>
>> I worked on the concept that you proposed and here is the patch with main
>> code change. Basically I wrote common driver handlers for HDA Driver
>> snd_hdac_drv_probe, snd_hdac_drv_remove, snd_hdac_drv_shutdown,
>> snd_hdac_drv_match, snd_hdac_drv_unsol_event etc.
>>
>> Once the common driver is probed it checks what kind of bus it is enumerated
>> on by calling the bus API. If it is ASOC bus type it calls the ASOC driver
>> registered callbacks and if it is LEGACY bus type it calls the Legacy driver
>> registered callbacks.
>>
>> ASoC based platform drivers would create ASOC bus type while the legacy
>> controller drivers would create LEGACY bus type. I added the bus_type as
>> a part of hdac_bus, which gets set during snd_hdac_bus_init or
>> snd_hdac_ext_bus_init. hdac_device->type and hdac_driver->type are
>> set the HDA_DEV_CORE during registrations and enumeration.
>>
>> Also I have kept these routines as part of codec library, so that all the other
>> drivers can use it without duplicating the code.
>>
>> Let me know if you are okay, I can include these changes as part of my
>> next series.
>
>I need to think more deeply, but after a quick look, I find it too
>overhead.

Are you referring to the calling overhead because of the wrapper involved ?

The way I see is we have two options.

1. Single driver option. - There is going to be a common wrapper here,
which needs to come into picture before it re-directs it to the appropriate
driver. This is what is done in the following patch.

2. Separate driver for ASoC and Legacy. - This requires ID tables, Can we move
id tables into separate header file ? then it can solve the problem involved in
option 1. This also solves the problem related to wrappers as well as the
problem related to accessing the ID tables via extern symbols, that you
mentioned in the previous series.

>
>May we start from a "big picture" to describe the whole driver
>bindings?
>

This patch registers the hdac_driver callback at the root level agnostic to
asoc and legacy and then selects the appropriate legacy or asoc callbacks,
based on the bus which enumerated the driver.

I cannot think of any other approach if we want to go with a single driver
approach. You will have to give some more hints :-)

Thank you for taking time and giving feedback.

Regards,
Rakesh