On 04/08/2018 09:34 AM, Marek Vasut wrote: > On 04/08/2018 05:27 PM, Randy Dunlap wrote: >> On 04/08/2018 06:09 AM, Marek Vasut wrote: >>> This patch replaces the (1 << n) with BIT(n) and cleans up whitespace, >>> no functional change. >>> >>> Signed-off-by: Marek Vasut <[email protected]> >>> Cc: Geert Uytterhoeven <[email protected]> >>> Cc: Phil Edworthy <[email protected]> >>> Cc: Simon Horman <[email protected]> >>> Cc: Wolfram Sang <[email protected]> >>> Cc: [email protected] >>> --- >>> V2: Reword the commit message >>> --- >>> drivers/pci/host/pcie-rcar.c | 52 >>> ++++++++++++++++++++++---------------------- >>> 1 file changed, 26 insertions(+), 26 deletions(-) >>> >>> diff --git a/drivers/pci/host/pcie-rcar.c b/drivers/pci/host/pcie-rcar.c >>> index 25f68305322c..5ab7bf6a8de0 100644 >>> --- a/drivers/pci/host/pcie-rcar.c >>> +++ b/drivers/pci/host/pcie-rcar.c >>> @@ -30,9 +30,9 @@ >> >> missing this: >> >> #include <linux/bitops.h> > > This compiles just fine without it though, which means it's probably > pulled in through some of the other includes already present. >
I suspected that. However, please see rule #1 in Documentation/process/submit-checklist.rst: 1) If you use a facility then #include the file that defines/declares that facility. Don't depend on other header files pulling in ones that you use. -- ~Randy
