Re: [RFC PATCH 3/5] w1: report errors returned by w1_family_notify

From: Mathieu Poirier
Date: Wed Oct 26 2016 - 12:39:32 EST


On 26 October 2016 at 08:57, Antoine Tenart
<antoine.tenart@xxxxxxxxxxxxxxxxxx> wrote:

GKH won't accept an empty commit log.

> Signed-off-by: Antoine Tenart <antoine.tenart@xxxxxxxxxxxxxxxxxx>
> ---
> drivers/w1/w1.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/w1/w1.c b/drivers/w1/w1.c
> index bb34362e930a..80d0cc4e6e7f 100644
> --- a/drivers/w1/w1.c
> +++ b/drivers/w1/w1.c
> @@ -702,7 +702,9 @@ static int __w1_attach_slave_device(struct w1_slave *sl)
> dev_name(&sl->dev), err);
> return err;
> }
> - w1_family_notify(BUS_NOTIFY_ADD_DEVICE, sl);
> + err = w1_family_notify(BUS_NOTIFY_ADD_DEVICE, sl);
> + if (err)
> + return err;
>
> dev_set_uevent_suppress(&sl->dev, false);
> kobject_uevent(&sl->dev.kobj, KOBJ_ADD);
> --
> 2.10.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html