linux-mips
[Top] [All Lists]

Re: [PATCH] IP22: Add platform device for Indy volume buttons

To: Geert Uytterhoeven <geert@linux-m68k.org>
Subject: Re: [PATCH] IP22: Add platform device for Indy volume buttons
From: tsbogend@alpha.franken.de (Thomas Bogendoerfer)
Date: Fri, 11 Jul 2008 22:56:28 +0200
Cc: linux-mips@linux-mips.org, ralf@linux-mips.org
In-reply-to: <Pine.LNX.4.64.0807112107360.7822@anakin>
Original-recipient: rfc822;linux-mips@linux-mips.org
References: <20080711183432.1CEEDC2EB7@solo.franken.de> <Pine.LNX.4.64.0807112107360.7822@anakin>
Sender: linux-mips-bounce@linux-mips.org
User-agent: Mutt/1.5.13 (2006-08-11)
On Fri, Jul 11, 2008 at 09:09:31PM +0200, Geert Uytterhoeven wrote:
> On Fri, 11 Jul 2008, Thomas Bogendoerfer wrote:
> > --- a/arch/mips/sgi-ip22/ip22-platform.c
> > +++ b/arch/mips/sgi-ip22/ip22-platform.c
> > @@ -182,3 +182,14 @@ static int __init sgi_hal2_devinit(void)
> >  }
> >  
> >  device_initcall(sgi_hal2_devinit);
> > +
> > +static int __init sgi_button_devinit(void)
> > +{
> > +   if (ip22_is_fullhouse())
> > +           return 0; /* full house has no volume buttons */
> > +
> > +   return IS_ERR(platform_device_register_simple("sgiindybtns",
> > +                                                 0, NULL, 0));
>                                                       ^
> Shouldn't the instance id be -1, as there can be only one?
> (cfr. Documentation/driver-model/platform.txt)

thanks. I'll send an updated patch asap.

Thomas.

-- 
Crap can work. Given enough thrust pigs will fly, but it's not necessary a
good idea.                                                [ RFC1925, 2.3 ]

<Prev in Thread] Current Thread [Next in Thread>