| Message ID | 20241116172934.1829676-2-niklas.soderlund+renesas@ragnatech.se |
|---|---|
| State | New |
| Headers | show |
| Series | thermal: rcar_gen3: Improve reading calibration fuses | expand |
Hi Niklas, On Sat, Nov 16, 2024 at 6:30 PM Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> wrote: > The style of the driver is to use lowercase hex constants, correct the > few outlines. > > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> Thanks for your patch! > --- a/drivers/thermal/renesas/rcar_gen3_thermal.c > +++ b/drivers/thermal/renesas/rcar_gen3_thermal.c > @@ -57,11 +57,11 @@ > /* THSCP bits */ > #define THSCP_COR_PARA_VLD (BIT(15) | BIT(14)) > > -#define CTEMP_MASK 0xFFF > +#define CTEMP_MASK 0xfff > > #define MCELSIUS(temp) ((temp) * 1000) > -#define GEN3_FUSE_MASK 0xFFF > -#define GEN4_FUSE_MASK 0xFFF > +#define GEN3_FUSE_MASK 0xfff > +#define GEN4_FUSE_MASK 0xfff > > #define TSC_MAX_NUM 5 LGTM, but you missed a few: $ git grep "\<0x.*[A-Z]" -- drivers/thermal/renesas/rcar_gen3_thermal.c drivers/thermal/renesas/rcar_gen3_thermal.c:#define REG_GEN3_IRQCTL 0x0C drivers/thermal/renesas/rcar_gen3_thermal.c:#define REG_GEN3_IRQTEMP3 0x1C drivers/thermal/renesas/rcar_gen3_thermal.c:#define REG_GEN4_THSFMON15 0x1BC drivers/thermal/renesas/rcar_gen3_thermal.c:#define REG_GEN4_THSFMON16 0x1C0 drivers/thermal/renesas/rcar_gen3_thermal.c:#define REG_GEN4_THSFMON17 0x1C4 Gr{oetje,eeting}s, Geert
diff --git a/drivers/thermal/renesas/rcar_gen3_thermal.c b/drivers/thermal/renesas/rcar_gen3_thermal.c index 810f86677461..35b8ecb8877e 100644 --- a/drivers/thermal/renesas/rcar_gen3_thermal.c +++ b/drivers/thermal/renesas/rcar_gen3_thermal.c @@ -57,11 +57,11 @@ /* THSCP bits */ #define THSCP_COR_PARA_VLD (BIT(15) | BIT(14)) -#define CTEMP_MASK 0xFFF +#define CTEMP_MASK 0xfff #define MCELSIUS(temp) ((temp) * 1000) -#define GEN3_FUSE_MASK 0xFFF -#define GEN4_FUSE_MASK 0xFFF +#define GEN3_FUSE_MASK 0xfff +#define GEN4_FUSE_MASK 0xfff #define TSC_MAX_NUM 5
The style of the driver is to use lowercase hex constants, correct the few outlines. Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> --- drivers/thermal/renesas/rcar_gen3_thermal.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)