[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <f71cef1c-f90b-ebb8-660d-29e6a174a079@roeck-us.net>
Date: Thu, 5 Nov 2020 19:08:12 -0800
From: Guenter Roeck <linux@...ck-us.net>
To: Brad Campbell <brad@...rfbargle.com>,
Andreas Kemnade <andreas@...nade.info>,
Jean Delvare <jdelvare@...e.com>
Cc: Arnd Bergmann <arnd@...db.de>, rydberg@...math.org,
linux-hwmon@...r.kernel.org,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
hns@...delico.com
Subject: Re: [PATCH] applesmc: Re-work SMC comms v2
On 11/5/20 4:02 PM, Brad Campbell wrote:
[ ... ]
>>> +/* Apple SMC status bits */
>>> +#define SMC_STATUS_AWAITING_DATA BIT(0) /* SMC has data waiting */
>>> +#define SMC_STATUS_IB_CLOSED BIT(1) /* Will ignore any input */
>>> +#define SMC_STATUS_BUSY BIT(2) /* Command in progress */
>>> +
>>
>> Hah, tricked you here ;-). Using "BIT()" requires
>>
>> #include <linux/bits.h>
>
> "requires" ??
> It compiles and tests without warning, but I'll certainly add it in.
>
Each driver should include the files with the declarations it needs, and
not depend on some indirect includes. Those indirect includes are not guaranteed
to exist and may be removed at some point in the future. "It compiles" is most
definitely not a valid argument.
Guenter
Powered by blists - more mailing lists