Re: [PATCH v5 02/10] iio: adc: ti-ads1015: sort headers alphabetically

[Joshua Crofts <joshua.crofts1@gmail.com>]

On Wed, 30 Sep 2026 22:41:56 +0530
Archit Anant <architanant5@gmail.com> wrote:

> Reorder include headers alphabetically to comply with the preferred
> coding style and improve maintainability.
> 
> No functional changes.
> 
> Signed-off-by: Archit Anant <architanant5@gmail.com>
> ---
>  drivers/iio/adc/ti-ads1015.c | 18 +++++++++---------
>  1 file changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/drivers/iio/adc/ti-ads1015.c b/drivers/iio/adc/ti-ads1015.c
> index 93a165c5416a..ef03b74bda3e 100644
> --- a/drivers/iio/adc/ti-ads1015.c
> +++ b/drivers/iio/adc/ti-ads1015.c
> @@ -11,24 +11,24 @@
>   *	* 0x4B - ADDR connected to SCL
>   */
>  
> -#include <linux/module.h>
>  #include <linux/cleanup.h>
> +#include <linux/delay.h>
> +#include <linux/i2c.h>
>  #include <linux/init.h>
>  #include <linux/irq.h>
> -#include <linux/i2c.h>
> +#include <linux/module.h>
> +#include <linux/mutex.h>
> +#include <linux/pm_runtime.h>
>  #include <linux/property.h>
>  #include <linux/regmap.h>
> -#include <linux/pm_runtime.h>
> -#include <linux/mutex.h>
> -#include <linux/delay.h>
>  
> +#include <linux/iio/buffer.h>
> +#include <linux/iio/events.h>
>  #include <linux/iio/iio.h>
> -#include <linux/iio/types.h>
>  #include <linux/iio/sysfs.h>
> -#include <linux/iio/events.h>
> -#include <linux/iio/buffer.h>
> -#include <linux/iio/triggered_buffer.h>
>  #include <linux/iio/trigger_consumer.h>
> +#include <linux/iio/triggered_buffer.h>
> +#include <linux/iio/types.h>

Small nit, but I'd drop iio/types.h as it is included by iio/iio.h
(No need to send a new version though, this can be addressed in the
future).

>  #define ADS1015_DRV_NAME "ads1015"
>  



-- 
Kind regards,
Joshua Crofts