Re: [PATCH] arm: omap2: rx51-peripherals: fix build warning

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

 



* Felipe Balbi <balbi@xxxxxx> [141204 07:50]:
> Hi,
> 
> On Thu, Dec 04, 2014 at 04:41:13PM +0100, Arnd Bergmann wrote:
> > On Thursday 04 December 2014 09:24:11 Felipe Balbi wrote:
> > > On Wed, Nov 26, 2014 at 02:27:35PM -0600, Felipe Balbi wrote:
> > > > commit 68a3c04 ([media] ARM: OMAP2: RX-51: update
> > > > si4713 platform data) updated board-rx51-peripherals.c
> > > > so that si4713 could be easily used on DT boot, but
> > > > it ended up introducing a build warning whenever
> > > > si4713 isn't enabled.
> > > > 
> > > > This patches fixes that warning:
> > > > 
> > > > arch/arm/mach-omap2/board-rx51-peripherals.c:1000:36: warning: \
> > > > 	‘rx51_si4713_platform_data’ defined but not used [-Wunused-variable]
> > > >  static struct si4713_platform_data rx51_si4713_platform_data = {
> > > > 
> > > > Cc: Sebastian Reichel <sre@xxxxxxxxxx>
> > > > Cc: Tony Lindgren <tony@xxxxxxxxxxx>
> > > > Cc: Hans Verkuil <hans.verkuil@xxxxxxxxx>
> > > > Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxx>
> > > > Signed-off-by: Felipe Balbi <balbi@xxxxxx>
> > > 
> > > a gentle reminder on this one.
> > > 
> > 
> > Let me add my
> > 
> > Acked-by: Arnd Bergmann <arnd@xxxxxxxx>
> > 
> > You didn't say who you expect to pick up the patch. I assume Mauro
> 
> patch author now decides who takes the patch ? That's new :-)
> 
> > should take it because he took the patch that caused the problem,
> > but he might not be aware that he should look at this now.
> 
> He is in Cc, let's ask him :-)

Best that this one goes in along with the other si4713 patches
to avoid dependencies between trees:

Acked-by: Tony Lindgren <tony@xxxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux