Dear Sachin Kamat, > Commit 0998d06310 (device-core: Ensure drvdata = NULL when no > driver is bound) removes the need to set driver data field to > NULL. > > Signed-off-by: Sachin Kamat <[email protected]> > Cc: Marek Vasut <[email protected]>
It does make sense. btw. if you're up to cleaning this up, you can also introduce some devm_input_device_allocate() and switch kzalloc() to devm_kzalloc etc. > --- > drivers/input/keyboard/matrix_keypad.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/input/keyboard/matrix_keypad.c > b/drivers/input/keyboard/matrix_keypad.c index 71d7719..90ff73a 100644 > --- a/drivers/input/keyboard/matrix_keypad.c > +++ b/drivers/input/keyboard/matrix_keypad.c > @@ -549,8 +549,6 @@ static int matrix_keypad_remove(struct platform_device > *pdev) input_unregister_device(keypad->input_dev); > kfree(keypad); > > - platform_set_drvdata(pdev, NULL); > - > return 0; > } Best regards, Marek Vasut -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
