Re: [PATCH] w1: support auto-load of w1_bq27000 module.

From: NeilBrown
Date: Sun Nov 09 2014 - 19:32:35 EST


On Mon, 10 Nov 2014 03:11:29 +0300 ÐÐÐÐÐÐÐ ÐÐÐÑÐÐÐ <zbr@xxxxxxxxxxx> wrote:

> Hi
>
> 08.11.2014, 02:33, "NeilBrown" <neilb@xxxxxxx>:
> > 1/ change request_module call to zero-pad single digit
> > ÂÂÂfamily numbers. ÂThis appears to be the intention of
> > ÂÂÂthe code, but not what it actually does.
> >
> > ÂÂÂThis means that the alias created for W1_FAMILY_SMEM_01
> > ÂÂÂmight actually be useful.
> >
> > 2/ Define a family name for the BQ27000 battery charge monitor.
> > ÂÂÂUnfortunately this is the same number as W1_FAMILY_SMEM_01
> > ÂÂÂso if both a compiled on a system, one module might need to
> > ÂÂÂbe blacklisted.
> >
> > 3/ Add a MODULE_ALIAS for the bq27000.
> >
> > Signed-off-by: NeilBrown <neilb@xxxxxxx>
>
> Looks good to me.
> Acked-by: Evgeniy Polyakov <zbr@xxxxxxxxxxx>
>
> Which tree should this patch use to get into the kernel?

MAINTAINERS lists you as the maintainer, so I though you would know.

When is doubt: send to akpm .. or gregkh.
It looks like greg has more Signed-off-by's in drivers/w1 than akpm,
so I'll add your acked-by and send to him.

Thanks,
NeilBrown

Attachment: pgpj9kI5CUPs2.pgp
Description: OpenPGP digital signature