lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-Id: <20171220204517.28313-2-andrew.smirnov@gmail.com> Date: Wed, 20 Dec 2017 12:45:13 -0800 From: Andrey Smirnov <andrew.smirnov@...il.com> To: Lee Jones <lee.jones@...aro.org> Cc: Andrey Smirnov <andrew.smirnov@...il.com>, linux-kernel@...r.kernel.org, linux-serial@...r.kernel.org, Rob Herring <robh@...nel.org>, cphealy@...il.com, Guenter Roeck <linux@...ck-us.net>, Lucas Stach <l.stach@...gutronix.de>, Nikita Yushchenko <nikita.yoush@...entembedded.com>, Greg Kroah-Hartman <gregkh@...uxfoundation.org>, Pavel Machek <pavel@....cz>, Andy Shevchenko <andy.shevchenko@...il.com>, Johan Hovold <johan@...nel.org>, Sebastian Reichel <sebastian.reichel@...labora.co.uk>, Philippe Ombredanne <pombredanne@...b.com> Subject: [PATCH v16 1/5] serdev: Make .remove in struct serdev_device_driver optional Using devres infrastructure it is possible to write a serdev driver that doesn't have any code that needs to be called as a part of .remove. Add code to make .remove optional. Cc: linux-kernel@...r.kernel.org Cc: linux-serial@...r.kernel.org Cc: Rob Herring <robh@...nel.org> Cc: cphealy@...il.com Cc: Guenter Roeck <linux@...ck-us.net> Cc: Lucas Stach <l.stach@...gutronix.de> Cc: Nikita Yushchenko <nikita.yoush@...entembedded.com> Cc: Lee Jones <lee.jones@...aro.org> Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org> Cc: Pavel Machek <pavel@....cz> Cc: Andy Shevchenko <andy.shevchenko@...il.com> Cc: Johan Hovold <johan@...nel.org> Cc: Sebastian Reichel <sebastian.reichel@...labora.co.uk> Cc: Philippe Ombredanne <pombredanne@...b.com> Acked-by: Pavel Machek <pavel@....cz> Acked-by: Rob Herring <robh@...nel.org> Reviewed-by: Sebastian Reichel <sebastian.reichel@...labora.co.uk> Reviewed-by: Guenter Roeck <linux@...ck-us.net> Signed-off-by: Andrey Smirnov <andrew.smirnov@...il.com> --- drivers/tty/serdev/core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/tty/serdev/core.c b/drivers/tty/serdev/core.c index 1bef39828ca7..34050b439c1f 100644 --- a/drivers/tty/serdev/core.c +++ b/drivers/tty/serdev/core.c @@ -268,8 +268,8 @@ static int serdev_drv_probe(struct device *dev) static int serdev_drv_remove(struct device *dev) { const struct serdev_device_driver *sdrv = to_serdev_device_driver(dev->driver); - - sdrv->remove(to_serdev_device(dev)); + if (sdrv->remove) + sdrv->remove(to_serdev_device(dev)); return 0; } -- 2.14.3
Powered by blists - more mailing lists