[PATCH] arm: omap2: rx51-peripherals: fix build warning
Tony Lindgren
tony at atomide.com
Thu Dec 4 08:39:31 PST 2014
* Felipe Balbi <balbi at ti.com> [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 at kernel.org>
> > > > Cc: Tony Lindgren <tony at atomide.com>
> > > > Cc: Hans Verkuil <hans.verkuil at cisco.com>
> > > > Cc: Mauro Carvalho Chehab <mchehab at osg.samsung.com>
> > > > Signed-off-by: Felipe Balbi <balbi at ti.com>
> > >
> > > a gentle reminder on this one.
> > >
> >
> > Let me add my
> >
> > Acked-by: Arnd Bergmann <arnd at arndb.de>
> >
> > 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 at atomide.com>
More information about the linux-arm-kernel
mailing list