Re: [PATCH 2/2] watchdog: pm8916_wdt: fix missing include

From: Guenter Roeck
Date: Sun Sep 08 2019 - 18:54:19 EST

On 9/6/19 1:30 PM, Jorge Ramirez-Ortiz wrote:
As per Documentation/process/submit-checklist.rst, when using a
facility #include the file that defines/declares that facility.

Don't depend on other header files pulling in ones that you use.

Correct, but then also don't include header files you don't use.
In this case, the include of linux/bitops.h is no longer necessary
if linux/bits.h is included since the driver doesn't really use bit
operations, only bit masks.


Signed-off-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@xxxxxxxxxx>
drivers/watchdog/pm8916_wdt.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/drivers/watchdog/pm8916_wdt.c b/drivers/watchdog/pm8916_wdt.c
index cb5304c26ac3..b8d9df0f96f7 100644
--- a/drivers/watchdog/pm8916_wdt.c
+++ b/drivers/watchdog/pm8916_wdt.c
@@ -1,5 +1,6 @@
// SPDX-License-Identifier: GPL-2.0
#include <linux/bitops.h>
+#include <linux/bits.h>
#include <linux/interrupt.h>
#include <linux/kernel.h>
#include <linux/module.h>