[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <fc0d7a20-f5ef-e8b4-6be8-ac9879399023@gmail.com>
Date: Tue, 2 Feb 2021 19:57:41 +0300
From: Dmitry Osipenko <digetx@...il.com>
To: Jon Hunter <jonathanh@...dia.com>, Ion Agorria <AG0RRIA@...oo.com>,
Thierry Reding <thierry.reding@...il.com>,
Stephen Warren <swarren@...dia.com>,
Liam Girdwood <lgirdwood@...il.com>,
Mark Brown <broonie@...nel.org>, Takashi Iwai <tiwai@...e.com>,
Jaroslav Kysela <perex@...ex.cz>,
Rob Herring <robh+dt@...nel.org>,
Svyatoslav Ryhel <clamor95@...il.com>,
Ion Agorria <ion@...rria.com>
Cc: alsa-devel@...a-project.org, devicetree@...r.kernel.org,
linux-tegra@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v1 2/2] ASoC: tegra: Add RT5631 machine driver
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.
Powered by blists - more mailing lists