Re: [PATCH v1] i2c: acpi: Move I2C bits from acpi.h to i2c.h

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wednesday, January 9, 2019 4:24:55 PM CET Andy Shevchenko wrote:
> As discussed previously the best location for certain bus related bits,
> e.g. I2C, is its own realm of the headers.
> 
> In order to uncontaminate acpi.h move the I2C bits to i2c.h.
> 
> There is no functional change intended.
> 
> Link: https://lkml.org/lkml/2018/11/28/744
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> ---
>  include/linux/acpi.h | 11 -----------
>  include/linux/i2c.h  | 10 ++++++++++
>  2 files changed, 10 insertions(+), 11 deletions(-)
> 
> diff --git a/include/linux/acpi.h b/include/linux/acpi.h
> index 83e55a704870..5d6caba28c00 100644
> --- a/include/linux/acpi.h
> +++ b/include/linux/acpi.h
> @@ -1023,17 +1023,6 @@ static inline int acpi_dev_gpio_irq_get(struct acpi_device *adev, int index)
>  }
>  #endif
>  
> -#if defined(CONFIG_ACPI) && IS_ENABLED(CONFIG_I2C)
> -bool i2c_acpi_get_i2c_resource(struct acpi_resource *ares,
> -			       struct acpi_resource_i2c_serialbus **i2c);
> -#else
> -static inline bool i2c_acpi_get_i2c_resource(struct acpi_resource *ares,
> -					     struct acpi_resource_i2c_serialbus **i2c)
> -{
> -	return false;
> -}
> -#endif
> -
>  /* Device properties */
>  
>  #ifdef CONFIG_ACPI
> diff --git a/include/linux/i2c.h b/include/linux/i2c.h
> index 65b4eaed1d96..10c8e021b35f 100644
> --- a/include/linux/i2c.h
> +++ b/include/linux/i2c.h
> @@ -933,11 +933,21 @@ static inline int of_i2c_get_board_info(struct device *dev,
>  
>  #endif /* CONFIG_OF */
>  
> +struct acpi_resource;
> +struct acpi_resource_i2c_serialbus;
> +
>  #if IS_ENABLED(CONFIG_ACPI)
> +bool i2c_acpi_get_i2c_resource(struct acpi_resource *ares,
> +			       struct acpi_resource_i2c_serialbus **i2c);
>  u32 i2c_acpi_find_bus_speed(struct device *dev);
>  struct i2c_client *i2c_acpi_new_device(struct device *dev, int index,
>  				       struct i2c_board_info *info);
>  #else
> +static inline bool i2c_acpi_get_i2c_resource(struct acpi_resource *ares,
> +					     struct acpi_resource_i2c_serialbus **i2c)
> +{
> +	return false;
> +}
>  static inline u32 i2c_acpi_find_bus_speed(struct device *dev)
>  {
>  	return 0;
> 

Acked-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>





[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux