Hi Simon, On Fri, Nov 30, 2018 at 1:23 PM Simon Horman <horms@xxxxxxxxxxxx> wrote: > On Fri, Nov 30, 2018 at 12:04:57PM +0000, Chris Brandt wrote: > > On Friday, November 30, 2018, Simon Horman wrote: > > > > + ostm0: timer@e803b000 { > > > > + compatible = "renesas,r7s9210-ostm", "renesas,ostm"; > > > > + reg = <0xe803b000 0x30>; > > > > > > > > > Its not clear to me why the size of the register range is 0x30. > > > > Because when reviewing the bindings docs, I was putting in 'exact' > > values, but Geert said "those are ugly, just round up to make it look > > nicer". > > So instead of 0x21, I put 0x30. > > > > You know...the standard answer: Throw Geert under the bus ;) > > Lets leave 0x30 :) Pfew... BTW, I'd be surprised the hardware address decoder would route all addresses in the range 0xe803b000..0xe803b02f to the OSTM. 0xe803b000..0xe803b03f sounds more logical to me, as it requires less gates. But hey, who said hardware designers are logical? ;-) 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