Andrey, On Wed, Dec 20, 2017 at 9:45 PM, Andrey Smirnov <andrew.smirnov@xxxxxxxxx> wrote: > Everyone: > > This patch series is v16 of the driver for supervisory processor found > on RAVE series of devices from ZII. Supervisory processor is a PIC > microcontroller connected to various electrical subsystems on RAVE > devices whose firmware implements protocol to command/qery them. > > NOTE: > > * This driver dependends on crc_ccitt_false(), added by > 2da9378d531f8cc6670c7497f20d936b706ab80b in 'linux-next', the patch > was pulled in by Andrew Morton and is currently avaiting users, so > this series might have to go in through Andrew's tree > > Changes since [v15]: > > - Adopted SPDX tags for licensing information per Philippe's > request Thank you for using the SPDX tags: you have my cheerful ack for the SPDX tags for the whole patch set There is one minor problem though: your comment style for the SPDX tag lines. Check Thomas doc patches and Linus comments on this topic: you should use // C++ style commnent in .c files and /* */ C-style comments in .h files. e.g. do not use this for a .c file: > +++ b/drivers/watchdog/rave-sp-wdt.c > @@ -0,0 +1,337 @@ > +/* SPDX-License-Identifier: GPL-2.0+ */ But this instead: > +++ b/drivers/watchdog/rave-sp-wdt.c > @@ -0,0 +1,337 @@ > +// SPDX-License-Identifier: GPL-2.0+ Acked-by: Philippe Ombredanne <pombredanne@xxxxxxxx> -- Cordially Philippe -- 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
