| Message ID | 20210323014320.136473-1-wanjiabing@vivo.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | drivers: watchdog: Remove duplicate include of kernel.h | expand |
On 3/22/21 6:43 PM, Wan Jiabing wrote: > linux/kernel.h has been included at line 17, so remove the > duplicate one at line 22. > > Signed-off-by: Wan Jiabing <wanjiabing@vivo.com> > --- > drivers/watchdog/dw_wdt.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/watchdog/dw_wdt.c b/drivers/watchdog/dw_wdt.c > index 32d0e1781e63..b1642e2d9175 100644 > --- a/drivers/watchdog/dw_wdt.c > +++ b/drivers/watchdog/dw_wdt.c > @@ -19,7 +19,6 @@ > #include <linux/delay.h> > #include <linux/err.h> > #include <linux/io.h> > -#include <linux/kernel.h> > #include <linux/module.h> > #include <linux/moduleparam.h> > #include <linux/interrupt.h> > The reason for the duplicate include is the non-alphabetic order of include files. I find that worse than the duplicate include. If it is too much effort to reorder include files to be in alphabetic order, please at least don't make it worse and remove the other include. Guenter
diff --git a/drivers/watchdog/dw_wdt.c b/drivers/watchdog/dw_wdt.c index 32d0e1781e63..b1642e2d9175 100644 --- a/drivers/watchdog/dw_wdt.c +++ b/drivers/watchdog/dw_wdt.c @@ -19,7 +19,6 @@ #include <linux/delay.h> #include <linux/err.h> #include <linux/io.h> -#include <linux/kernel.h> #include <linux/module.h> #include <linux/moduleparam.h> #include <linux/interrupt.h>
linux/kernel.h has been included at line 17, so remove the duplicate one at line 22. Signed-off-by: Wan Jiabing <wanjiabing@vivo.com> --- drivers/watchdog/dw_wdt.c | 1 - 1 file changed, 1 deletion(-)