Message ID | 20170309093651.7824-1-kernel@kempniu.pl (mailing list archive) |
---|---|
State | Accepted, archived |
Delegated to: | Andy Shevchenko |
Headers | show |
On 2017-03-09 05:36 PM, Michał Kępień wrote: > devm_input_allocate_device() already causes the supplied struct device > to be set as the parent of the input device, so doing it again is > redundant. > > Signed-off-by: Michał Kępień <kernel@kempniu.pl> > --- > This patch needs my recent intel-hid cleanup series to apply cleanly. > In other words, it should apply on top of testing, but not for-next. > > drivers/platform/x86/intel-hid.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/platform/x86/intel-hid.c b/drivers/platform/x86/intel-hid.c > index 5eab31659cba..b40059aba856 100644 > --- a/drivers/platform/x86/intel-hid.c > +++ b/drivers/platform/x86/intel-hid.c > @@ -152,7 +152,6 @@ static int intel_hid_input_setup(struct platform_device *device) > if (ret) > return ret; > > - priv->input_dev->dev.parent = &device->dev; > priv->input_dev->name = "Intel HID events"; > priv->input_dev->id.bustype = BUS_HOST; > > @@ -173,7 +172,6 @@ static int intel_button_array_input_setup(struct platform_device *device) > if (ret) > return ret; > > - priv->array->dev.parent = &device->dev; > priv->array->name = "Intel HID 5 button array"; > priv->array->id.bustype = BUS_HOST; > > The patch, on top of previous patches, is tested on Latitude 7480 Reviewed-and-tested-by: Alex Hung <alex.hung@canonical.com>
On Mon, Mar 13, 2017 at 12:37 PM, Alex Hung <alex.hung@canonical.com> wrote: > On 2017-03-09 05:36 PM, Michał Kępień wrote: >> >> devm_input_allocate_device() already causes the supplied struct device >> to be set as the parent of the input device, so doing it again is >> redundant. >> >> Signed-off-by: Michał Kępień <kernel@kempniu.pl> Pushed to testing, thanks. > > The patch, on top of previous patches, is tested on Latitude 7480 > > Reviewed-and-tested-by: Alex Hung <alex.hung@canonical.com> Alex, this tag is not recognized by patchwork :-(, It would be nice to use separate tags. I would join them if needed (usually when they sent against cover letter).
diff --git a/drivers/platform/x86/intel-hid.c b/drivers/platform/x86/intel-hid.c index 5eab31659cba..b40059aba856 100644 --- a/drivers/platform/x86/intel-hid.c +++ b/drivers/platform/x86/intel-hid.c @@ -152,7 +152,6 @@ static int intel_hid_input_setup(struct platform_device *device) if (ret) return ret; - priv->input_dev->dev.parent = &device->dev; priv->input_dev->name = "Intel HID events"; priv->input_dev->id.bustype = BUS_HOST; @@ -173,7 +172,6 @@ static int intel_button_array_input_setup(struct platform_device *device) if (ret) return ret; - priv->array->dev.parent = &device->dev; priv->array->name = "Intel HID 5 button array"; priv->array->id.bustype = BUS_HOST;
devm_input_allocate_device() already causes the supplied struct device to be set as the parent of the input device, so doing it again is redundant. Signed-off-by: Michał Kępień <kernel@kempniu.pl> --- This patch needs my recent intel-hid cleanup series to apply cleanly. In other words, it should apply on top of testing, but not for-next. drivers/platform/x86/intel-hid.c | 2 -- 1 file changed, 2 deletions(-)