Re: [PATCH] 1wire: family module autoload fails because of upper/lower case mismatch.

From: Evgeniy Polyakov
Date: Tue May 01 2018 - 15:02:09 EST


Hi Ingo

Sorry for late reply and because of forcing you to resend the patch, your changes are correct

01.05.2018, 17:10, "Ingo Flaschberger" <ingo.flaschberger@xxxxxxxxx>:
> 1wire family module autoload fails because of upper/lower
> ÂÂcase mismatch.
>
> Signed-off-by: Ingo Flaschberger <ingo.flaschberger@xxxxxxxxx>

Greg, please pull this patch into your tree.
Should it be stable material?

Acked-by: Evgeniy Polyakov <zbr@xxxxxxxxxxx>

> ---
> ÂÂdrivers/w1/w1.c | 2 +-
> ÂÂ1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/w1/w1.c b/drivers/w1/w1.c
> index ab0931e..aa458f2 100644
> --- a/drivers/w1/w1.c
> +++ b/drivers/w1/w1.c
> @@ -741,7 +741,7 @@ int w1_attach_slave_device(struct w1_master *dev,
> struct w1_reg_num *rn)
>
> ÂÂÂÂÂÂÂÂ /* slave modules need to be loaded in a context with unlocked
> mutex */
> ÂÂÂÂÂÂÂÂ mutex_unlock(&dev->mutex);
> -ÂÂÂÂÂÂ request_module("w1-family-0x%02x", rn->family);
> +ÂÂÂÂÂÂ request_module("w1-family-0x%02X", rn->family);
> ÂÂÂÂÂÂÂÂ mutex_lock(&dev->mutex);
>
> ÂÂÂÂÂÂÂÂ spin_lock(&w1_flock);
> --
> 2.7.4