On Tue, Feb 02, 2021 at 04:24:31PM +0000, Jon Hunter wrote:

On 02/02/2021 15:25, Dmitry Osipenko wrote:

quoted

These codecs require individual configurations and those
"../codecs/rt5631.h" and  "../codecs/rt5677.h" aren't compatible at a
quick glance.

Right but not all of that is needed. What is actually needed from the
header files?

Right, and if it's just a case of having a different hw_params() or
something then the majority of the driver could be shared with just a
few bits being handled differently.

quoted

The tegra_rt5677 also uses outdated GPIO API and etc. Hence the new
driver should be a better base anyways.

Sounds like a good time to update it :-)

Yeah.

quoted

Overall it shouldn't worth time and effort trying to squeeze these
drivers into a single one, IMO.

Not sure I agree when these drivers appear to be 90% the same.

It's certainly worth considering - given that it's the same silicon
vendor working with the same SoC vendor's reference designs it seems
likely that things will look pretty similar at the system integration
level.  It's possible that it's more trouble than it's worth but it'd be
good to have a more concrete understanding of why.