02.02.2021 19:24, Jon Hunter пишет:
On 02/02/2021 15:25, Dmitry Osipenko wrote:
02.02.2021 16:22, Jon Hunter пишет:
So this is very similar to tegra_rt5677, is it not possible to support both codecs with the same machine driver?
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?
I recall that some downstream drivers were doing some special programming of codecs. This is not relevant to the current upstream drivers, but potentially it may become needed and then that single driver could become unmanageable.
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 :-)
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.
Of course we could try, but I suggest that it should be done separately from this series. Certainly it will take a lot of extra effort and this series isn't about improving older drivers, it's about enabling h/w support for the RT5631 codec.
It shouldn't be a problem to switch to the common machine driver later on if this driver will turn out to be feasible.