[PATCH 2/3] thermal/drivers/imx91: Drop extra spaces

Frank Li Frank.li at nxp.com
Fri Dec 12 09:29:41 PST 2025


On Fri, Dec 12, 2025 at 03:51:15PM +0800, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan at nxp.com>
>
> Unify the code style, drop extra spaces for the macros.
>
> Signed-off-by: Peng Fan <peng.fan at nxp.com>
> ---
>  drivers/thermal/imx91_thermal.c | 22 +++++++++++-----------
>  1 file changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/thermal/imx91_thermal.c b/drivers/thermal/imx91_thermal.c
> index 77e8e6a921c6af308b830c36721293c007256ca6..768c5acc384eca1c2203098bd4749628d9ffb8e6 100644
> --- a/drivers/thermal/imx91_thermal.c
> +++ b/drivers/thermal/imx91_thermal.c
> @@ -22,17 +22,17 @@
>  #define REG_TOG					0xc
>
>  #define IMX91_TMU_CTRL0				0x0
> -#define   IMX91_TMU_CTRL0_THR1_IE		BIT(9)
> -#define   IMX91_TMU_CTRL0_THR1_MASK		GENMASK(3, 2)
> -#define   IMX91_TMU_CTRL0_CLR_FLT1		BIT(21)
> +#define IMX91_TMU_CTRL0_THR1_IE			BIT(9)
> +#define IMX91_TMU_CTRL0_THR1_MASK		GENMASK(3, 2)
> +#define IMX91_TMU_CTRL0_CLR_FLT1		BIT(21)

It is used for register field. Many new driver use this style.

#define REGISTER_NAME
#define   REGISTER_FIELD

So register defination looks like tree. Reader can quick search field and
register name.

Frank


>
>  #define IMX91_TMU_THR_MODE_LE			0
>  #define IMX91_TMU_THR_MODE_GE			1
>
>  #define IMX91_TMU_STAT0				0x10
> -#define   IMX91_TMU_STAT0_THR1_IF		BIT(9)
> -#define   IMX91_TMU_STAT0_THR1_STAT		BIT(13)
> -#define   IMX91_TMU_STAT0_DRDY0_IF_MASK		BIT(16)
> +#define IMX91_TMU_STAT0_THR1_IF			BIT(9)
> +#define IMX91_TMU_STAT0_THR1_STAT		BIT(13)
> +#define IMX91_TMU_STAT0_DRDY0_IF_MASK		BIT(16)
>
>  #define IMX91_TMU_DATA0				0x20
>
> @@ -42,12 +42,12 @@
>  #define IMX91_TMU_CTRL1_STOP			BIT(29)
>  #define IMX91_TMU_CTRL1_RES_MASK		GENMASK(19, 18)
>  #define IMX91_TMU_CTRL1_MEAS_MODE_MASK		GENMASK(25, 24)
> -#define   IMX91_TMU_CTRL1_MEAS_MODE_SINGLE	0
> -#define   IMX91_TMU_CTRL1_MEAS_MODE_CONTINUES	1
> -#define   IMX91_TMU_CTRL1_MEAS_MODE_PERIODIC	2
> +#define IMX91_TMU_CTRL1_MEAS_MODE_SINGLE	0
> +#define IMX91_TMU_CTRL1_MEAS_MODE_CONTINUES	1
> +#define IMX91_TMU_CTRL1_MEAS_MODE_PERIODIC	2
>
>  #define IMX91_TMU_THR_CTRL01			0x30
> -#define   IMX91_TMU_THR_CTRL01_THR1_MASK	GENMASK(31, 16)
> +#define IMX91_TMU_THR_CTRL01_THR1_MASK		GENMASK(31, 16)
>
>  #define IMX91_TMU_REF_DIV			0x280
>  #define IMX91_TMU_DIV_EN			BIT(31)
> @@ -67,7 +67,7 @@
>  #define IMX91_TMU_DEFAULT_TRIM2_CONFIG		0x65d4
>
>  #define IMX91_TMU_PERIOD_CTRL			0x270
> -#define   IMX91_TMU_PERIOD_CTRL_MEAS_MASK	GENMASK(23, 0)
> +#define IMX91_TMU_PERIOD_CTRL_MEAS_MASK		GENMASK(23, 0)
>
>  #define IMX91_TMP_FRAC				64
>
>
> --
> 2.37.1
>



More information about the linux-arm-kernel mailing list