On Fri, 15 Aug 2025 16:46:04 +0000
Ben Collins <bcollins@watter.com> wrote:
> Purely to align tabs for #defines.
Hi Ben,
Mention the changes to two digits for register addresses to.
Good change to roll in here given what you are touching anyway so
just make sure to call it out in the patch description.
Jonathan
>
> Signed-off-by: Ben Collins <bcollins@watter.com>
> ---
> drivers/iio/temperature/mcp9600.c | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/iio/temperature/mcp9600.c b/drivers/iio/temperature/mcp9600.c
> index 6e9108d5cf75f..40906bb200ec9 100644
> --- a/drivers/iio/temperature/mcp9600.c
> +++ b/drivers/iio/temperature/mcp9600.c
> @@ -23,25 +23,25 @@
> #include <linux/iio/iio.h>
>
> /* MCP9600 registers */
> -#define MCP9600_HOT_JUNCTION 0x0
> -#define MCP9600_COLD_JUNCTION 0x2
> -#define MCP9600_STATUS 0x4
> +#define MCP9600_HOT_JUNCTION 0x00
> +#define MCP9600_COLD_JUNCTION 0x02
> +#define MCP9600_STATUS 0x04
> #define MCP9600_STATUS_ALERT(x) BIT(x)
> -#define MCP9600_ALERT_CFG1 0x8
> +#define MCP9600_ALERT_CFG1 0x08
> #define MCP9600_ALERT_CFG(x) (MCP9600_ALERT_CFG1 + (x - 1))
> #define MCP9600_ALERT_CFG_ENABLE BIT(0)
> #define MCP9600_ALERT_CFG_ACTIVE_HIGH BIT(2)
> #define MCP9600_ALERT_CFG_FALLING BIT(3)
> #define MCP9600_ALERT_CFG_COLD_JUNCTION BIT(4)
> -#define MCP9600_ALERT_HYSTERESIS1 0xc
> +#define MCP9600_ALERT_HYSTERESIS1 0x0c
> #define MCP9600_ALERT_HYSTERESIS(x) (MCP9600_ALERT_HYSTERESIS1 + (x - 1))
> #define MCP9600_ALERT_LIMIT1 0x10
> #define MCP9600_ALERT_LIMIT(x) (MCP9600_ALERT_LIMIT1 + (x - 1))
> #define MCP9600_ALERT_LIMIT_MASK GENMASK(15, 2)
> -#define MCP9600_DEVICE_ID 0x20
> +#define MCP9600_DEVICE_ID 0x20
>
> /* MCP9600 device id value */
> -#define MCP9600_DEVICE_ID_MCP9600 0x40
> +#define MCP9600_DEVICE_ID_MCP9600 0x40
>
> #define MCP9600_ALERT_COUNT 4
>