Re: [PATCH 5/9] ns558: adjust pnp_register_driver signature

[Bjorn Helgaas <bjorn.helgaas@hp.com>]

On Friday 03 March 2006 05:55, Vojtech Pavlik wrote:
> On Thu, Mar 02, 2006 at 04:09:37PM -0700, Bjorn Helgaas wrote:
> > Remove the assumption that pnp_register_driver() returns the number of
> > devices claimed.
> > 
> > Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com>
> 
> Wouldn't a diff like
> 
> --- a/drivers/input/gameport/ns558.c	2006-03-02 12:40:45.000000000 -0700
> +++ b/drivers/input/gameport/ns558.c	2006-03-02 12:43:58.000000000 -0700
> @@ -258,7 +256,7 @@
>  {
>  	int i = 0;
>  
> -	if (pnp_register_driver(&ns558_pnp_driver) >= 0)
> +	if (!pnp_register_driver(&ns558_pnp_driver))
>  		pnp_registered = 1;
>   
>   /*
> 
> be enough? The err variable isn't used anywhere else.

Yup, that'd be fine, and in fact my first version of the patch
did just that.  I introduced "err" because most of the other
drivers I had to touch used "err," and it seemed a bit clearer to me
because it gives an extra hint that pnp_register_driver() returns
only an error code.

But I don't care either way, so say the word, and I'll send
Andrew your patch to replace mine.

Bjorn

> > Index: work-mm4/drivers/input/gameport/ns558.c
> > ===================================================================
> > --- work-mm4.orig/drivers/input/gameport/ns558.c	2006-03-02 12:40:45.000000000 -0700
> > +++ work-mm4/drivers/input/gameport/ns558.c	2006-03-02 12:43:58.000000000 -0700
> > @@ -256,9 +256,10 @@
> >  
> >  static int __init ns558_init(void)
> >  {
> > -	int i = 0;
> > +	int i = 0, err;
> >  
> > -	if (pnp_register_driver(&ns558_pnp_driver) >= 0)
> > +	err = pnp_register_driver(&ns558_pnp_driver);
> > +	if (!err)
> >  		pnp_registered = 1;
> >  
> >  /*
> > 
> > 
>