Re: [PATCH] Fix Atmel soc audio boards Kconfig dependency

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2010-11-10 at 17:09 -0600, Geoffrey Wossum wrote:
> On Wednesday 10 November 2010 3:54:08 pm Ryan Mallon wrote:
> > On 11/11/2010 10:04 AM, Liam Girdwood wrote:
> > > On Thu, 2010-11-11 at 09:02 +1300, Ryan Mallon wrote:
> > >> Add Kconfig dependency on AT91_PROGRAMMABLE_CLOCKS for the Atmel SoC
> > >> audio SAM9G20-EK and PlayPaq boards. Fixes link errors on missing
> > >> clk_set_parent and clk_set_rate when building without
> > >> AT91_PROGRAMMABLE_CLOCKS.
> > >> 
> > >> Signed-off-by: Ryan Mallon <ryan@xxxxxxxxxxxxxxxx>
> > > 
> > > Acked-by: Liam Girdwood <lrg@xxxxxxxxxxxxxxx>
> > 
> > Just realised that the PlayPaq board only appears in the sound/soc/atmel
> > directory. I cannot find a board file for it under either
> > arch/arm/mach-at91 or arch/avr32/. The updated patch below only changes
> > the dependency for the SAM9G20-EK board since I think the PlayPaq is
> > actually AVR32, and therefore should not depend on
> > AT91_PROGRAMMABLE_CLOCKS. This patch replaces my previous one.
> 
> PlayPaq is an AT32AP7000 board, so the new patch is correct.
> 
> > ---
> > Add Kconfig dependency on AT91_PROGRAMMABLE_CLOCKS for the Atmel SoC
> > audio  SAM9G20-EK board. Fixes link errors on missing clk_set_parent and
> > clk_set_rate when building without AT91_PROGRAMMABLE_CLOCKS.
> > 
> > Signed-off-by: Ryan Mallon <ryan@xxxxxxxxxxxxxxxx>
> 
> Acked-by: Geoffrey Wossum <gwossum@xxxxxxx>

Acked-by: Liam Girdwood <lrg@xxxxxxxxxxxxxxx>
-- 
Freelance Developer, SlimLogic Ltd
ASoC and Voltage Regulator Maintainer.
http://www.slimlogic.co.uk

_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel


[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux