On 1/29/2021 9:18 PM, Mark Brown wrote:

On Fri, Jan 29, 2021 at 09:02:52PM +0530, Sameer Pujar wrote:

quoted

On 1/29/2021 6:14 PM, Mark Brown wrote:

quoted

This is a generic device, not something specific to Jetson, why force
on a driver that may not be required on other boards using this driver?

Yes specific boards using 'SND_SOC_TEGRA_AUDIO_GRAPH_CARD' may require
'SND_SOC_RT5659'. But there is no platform specific config. Is there a
better way to enable config 'SND_SOC_RT5659' conditionally?

If the user wants a given CODEC driver then they should enable that
driver.

Above card driver is intended to be used on multiple platforms. DT has 
already a way for user to describe the CODEC connection required for 
specific platform. So idea is enable these CODECs from driver point of 
view for this card driver and platform DT can use what is required. Also 
the CODEC driver will be a loadable module here.

If above does not seem fine, alternatively can I just enable CODEC 
config independently from defconfig?