[PATCH 1/2] firmware: ti_sci: Include linux/err.h in ti_sci_protocol.h

[Nishanth Menon]

On 18:24-20210727, Suman Anna wrote:
> The common TI SCI header file uses some macros from err.h and these
> get exercised when CONFIG_TI_SCI_PROTOCOL is not defined. Include
> the linux/err.h header file in this header file directly rather
> than relying on source files to include it to eliminate any
> potential build errors.
> 
> While at this, reorder the existing header file include to the
> beginning of the file.
> 
> Signed-off-by: Suman Anna <s-anna at ti.com>
> ---
>  include/linux/soc/ti/ti_sci_protocol.h | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/include/linux/soc/ti/ti_sci_protocol.h b/include/linux/soc/ti/ti_sci_protocol.h
> index 794737923cf1..7f9941894cdf 100644
> --- a/include/linux/soc/ti/ti_sci_protocol.h
> +++ b/include/linux/soc/ti/ti_sci_protocol.h
> @@ -11,6 +11,9 @@
>  #ifndef __TISCI_PROTOCOL_H
>  #define __TISCI_PROTOCOL_H
>  
> +#include <linux/bitops.h>
> +#include <linux/err.h>
> +
>  /**
>   * struct ti_sci_version_info - version information structure
>   * @abi_major:	Major ABI version. Change here implies risk of backward
> @@ -20,7 +23,6 @@
>   * @firmware_revision:	Firmware revision (not usually used).
>   * @firmware_description: Firmware description (not usually used).
>   */
> -#include <linux/bitops.h>
>  struct ti_sci_version_info {
>  	u8 abi_major;
>  	u8 abi_minor;

Reviewed-by: Nishanth Menon <nm at ti.com>
-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D