[alsa-devel] [PATCH 16/31] ASoC: tegra: use reset framework
swarren at wwwdotorg.org
Tue Nov 26 17:31:25 CET 2013
On 11/26/2013 06:14 AM, Mark Brown wrote:
> On Mon, Nov 25, 2013 at 02:56:23PM -0700, Stephen Warren wrote:
>> You didn't ack this one patch; I assume that was just an
> No, it was because it looked incorrect based on the lack of tie in
> between the description and the code.
Hmm. You had asked:
>> @@ -1,6 +1,8 @@ config SND_SOC_TEGRA tristate "SoC Audio for the
>> Tegra System-on-Chip" depends on (ARCH_TEGRA && TEGRA20_APB_DMA)
>> || COMPILE_TEST + depends on COMMON_CLK + depends on
> Do you depend on COMMON_CLK here? I only noticed reset controller
> API dependencies here but perhaps I missed this (or it's fixing a
> dependency that should be there already).
> It's fixing a dependency that should already be there, in the
> COMPILE_TEST case. In the (ARCH_TEGRA && TEGRA20_APB_DMA) case,
> COMMON_CLOCK is always selected.
> Do you want me to split this out into a separate patch? If so, I'd
> prefer not to apply that separate patch immediately to 3.13 as a
> fix, since then it'd delay applying this series until after -rc2 is
> out, unless you can get the fix into -rc1 quickly...
(although at this point in time, the DMA patches which this depend on
aren't likely to be ready soon enough for the delay to matter, so I
could send the addition of depends COMMON_CLK as a separate patch for
-rc2 if you want)
and you said:
> I don't really care, it was just that I was looking for something
> to do with clocks in the patch and couldn't find anything. Perhaps
> a note in the changelog if you need to respin so I don't forget and
> say the same thing again.
... so, I thought you were OK with that one issue. Were there other
issues you didn't mention before?
More information about the Alsa-devel