On Fri, Mar 17, 2017 at 4:52 PM, Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx> wrote: > Record how many TSCs are found in struct rcar_gen3_thermal_priv, this is > needed to be able to add hardware interrupts for trip points later. Also > add a check to make sure at least one TSC is found. > > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx> Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > --- > drivers/thermal/rcar_gen3_thermal.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/drivers/thermal/rcar_gen3_thermal.c b/drivers/thermal/rcar_gen3_thermal.c > index 9b6bc03dd142a8dd..3b6e85abfadc61a3 100644 > --- a/drivers/thermal/rcar_gen3_thermal.c > +++ b/drivers/thermal/rcar_gen3_thermal.c > @@ -74,6 +74,7 @@ struct rcar_gen3_thermal_tsc { > }; > > struct rcar_gen3_thermal_priv { > + unsigned int num_tscs; I would insert this 32-bit number after the pointers below (which are 64-bit), because of alignment rules and implicit gaps. > struct rcar_gen3_thermal_tsc *tscs[TSC_MAX_NUM]; > }; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds