Re: [PATCH 2/2] HID: wacom: do not call hid_set_drvdata(hdev, NULL)
On Sat, Aug 17, 2019 at 12:04 AM Jason Gerecke wrote: > > On Mon, Aug 12, 2019 at 9:29 AM Benjamin Tissoires > wrote: > > > > This is a common pattern in the HID drivers to reset the drvdata. > > However, this is actually already handled by driver core, so there > > is no need to do it manually. > > > > Signed-off-by: Benjamin Tissoires > > Acked-by: Jason Gerecke Thanks Applied to for-5.4/wacom Cheers, Benjamin > > Jason > --- > Now instead of four in the eights place / > you’ve got three, ‘Cause you added one / > (That is to say, eight) to the two, / > But you can’t take seven from three,/ > So you look at the sixty-fours > > > > --- > > drivers/hid/wacom_sys.c | 18 +- > > 1 file changed, 5 insertions(+), 13 deletions(-) > > > > diff --git a/drivers/hid/wacom_sys.c b/drivers/hid/wacom_sys.c > > index 53bddb50aeba..69ccfdd51a6f 100644 > > --- a/drivers/hid/wacom_sys.c > > +++ b/drivers/hid/wacom_sys.c > > @@ -2718,14 +2718,12 @@ static int wacom_probe(struct hid_device *hdev, > > wacom_wac->features = *((struct wacom_features *)id->driver_data); > > features = _wac->features; > > > > - if (features->check_for_hid_type && features->hid_type != > > hdev->type) { > > - error = -ENODEV; > > - goto fail; > > - } > > + if (features->check_for_hid_type && features->hid_type != > > hdev->type) > > + return -ENODEV; > > > > error = kfifo_alloc(_wac->pen_fifo, WACOM_PKGLEN_MAX, > > GFP_KERNEL); > > if (error) > > - goto fail; > > + return error; > > > > wacom_wac->hid_data.inputmode = -1; > > wacom_wac->mode_report = -1; > > @@ -2743,12 +2741,12 @@ static int wacom_probe(struct hid_device *hdev, > > error = hid_parse(hdev); > > if (error) { > > hid_err(hdev, "parse failed\n"); > > - goto fail; > > + return error; > > } > > > > error = wacom_parse_and_register(wacom, false); > > if (error) > > - goto fail; > > + return error; > > > > if (hdev->bus == BUS_BLUETOOTH) { > > error = device_create_file(>dev, _attr_speed); > > @@ -2759,10 +2757,6 @@ static int wacom_probe(struct hid_device *hdev, > > } > > > > return 0; > > - > > -fail: > > - hid_set_drvdata(hdev, NULL); > > - return error; > > } > > > > static void wacom_remove(struct hid_device *hdev) > > @@ -2791,8 +2785,6 @@ static void wacom_remove(struct hid_device *hdev) > > wacom_release_resources(wacom); > > > > kfifo_free(_wac->pen_fifo); > > - > > - hid_set_drvdata(hdev, NULL); > > } > > > > #ifdef CONFIG_PM > > -- > > 2.19.2 > >
Re: [PATCH 2/2] HID: wacom: do not call hid_set_drvdata(hdev, NULL)
On Mon, Aug 12, 2019 at 9:29 AM Benjamin Tissoires wrote: > > This is a common pattern in the HID drivers to reset the drvdata. > However, this is actually already handled by driver core, so there > is no need to do it manually. > > Signed-off-by: Benjamin Tissoires Acked-by: Jason Gerecke Jason --- Now instead of four in the eights place / you’ve got three, ‘Cause you added one / (That is to say, eight) to the two, / But you can’t take seven from three,/ So you look at the sixty-fours > --- > drivers/hid/wacom_sys.c | 18 +- > 1 file changed, 5 insertions(+), 13 deletions(-) > > diff --git a/drivers/hid/wacom_sys.c b/drivers/hid/wacom_sys.c > index 53bddb50aeba..69ccfdd51a6f 100644 > --- a/drivers/hid/wacom_sys.c > +++ b/drivers/hid/wacom_sys.c > @@ -2718,14 +2718,12 @@ static int wacom_probe(struct hid_device *hdev, > wacom_wac->features = *((struct wacom_features *)id->driver_data); > features = _wac->features; > > - if (features->check_for_hid_type && features->hid_type != hdev->type) > { > - error = -ENODEV; > - goto fail; > - } > + if (features->check_for_hid_type && features->hid_type != hdev->type) > + return -ENODEV; > > error = kfifo_alloc(_wac->pen_fifo, WACOM_PKGLEN_MAX, > GFP_KERNEL); > if (error) > - goto fail; > + return error; > > wacom_wac->hid_data.inputmode = -1; > wacom_wac->mode_report = -1; > @@ -2743,12 +2741,12 @@ static int wacom_probe(struct hid_device *hdev, > error = hid_parse(hdev); > if (error) { > hid_err(hdev, "parse failed\n"); > - goto fail; > + return error; > } > > error = wacom_parse_and_register(wacom, false); > if (error) > - goto fail; > + return error; > > if (hdev->bus == BUS_BLUETOOTH) { > error = device_create_file(>dev, _attr_speed); > @@ -2759,10 +2757,6 @@ static int wacom_probe(struct hid_device *hdev, > } > > return 0; > - > -fail: > - hid_set_drvdata(hdev, NULL); > - return error; > } > > static void wacom_remove(struct hid_device *hdev) > @@ -2791,8 +2785,6 @@ static void wacom_remove(struct hid_device *hdev) > wacom_release_resources(wacom); > > kfifo_free(_wac->pen_fifo); > - > - hid_set_drvdata(hdev, NULL); > } > > #ifdef CONFIG_PM > -- > 2.19.2 >
[PATCH 2/2] HID: wacom: do not call hid_set_drvdata(hdev, NULL)
This is a common pattern in the HID drivers to reset the drvdata. However, this is actually already handled by driver core, so there is no need to do it manually. Signed-off-by: Benjamin Tissoires --- drivers/hid/wacom_sys.c | 18 +- 1 file changed, 5 insertions(+), 13 deletions(-) diff --git a/drivers/hid/wacom_sys.c b/drivers/hid/wacom_sys.c index 53bddb50aeba..69ccfdd51a6f 100644 --- a/drivers/hid/wacom_sys.c +++ b/drivers/hid/wacom_sys.c @@ -2718,14 +2718,12 @@ static int wacom_probe(struct hid_device *hdev, wacom_wac->features = *((struct wacom_features *)id->driver_data); features = _wac->features; - if (features->check_for_hid_type && features->hid_type != hdev->type) { - error = -ENODEV; - goto fail; - } + if (features->check_for_hid_type && features->hid_type != hdev->type) + return -ENODEV; error = kfifo_alloc(_wac->pen_fifo, WACOM_PKGLEN_MAX, GFP_KERNEL); if (error) - goto fail; + return error; wacom_wac->hid_data.inputmode = -1; wacom_wac->mode_report = -1; @@ -2743,12 +2741,12 @@ static int wacom_probe(struct hid_device *hdev, error = hid_parse(hdev); if (error) { hid_err(hdev, "parse failed\n"); - goto fail; + return error; } error = wacom_parse_and_register(wacom, false); if (error) - goto fail; + return error; if (hdev->bus == BUS_BLUETOOTH) { error = device_create_file(>dev, _attr_speed); @@ -2759,10 +2757,6 @@ static int wacom_probe(struct hid_device *hdev, } return 0; - -fail: - hid_set_drvdata(hdev, NULL); - return error; } static void wacom_remove(struct hid_device *hdev) @@ -2791,8 +2785,6 @@ static void wacom_remove(struct hid_device *hdev) wacom_release_resources(wacom); kfifo_free(_wac->pen_fifo); - - hid_set_drvdata(hdev, NULL); } #ifdef CONFIG_PM -- 2.19.2