Hi Geert,

On 18/02/2019 09:01, Geert Uytterhoeven wrote:
> Hi Kieran,
> 
> On Sun, Feb 17, 2019 at 11:13 PM Kieran Bingham
> <kieran.bing...@ideasonboard.com> wrote:
>> On 16/02/2019 23:58, Niklas Söderlund wrote:
>>> From: Takeshi Kihara <takeshi.kihara...@renesas.com>
>>>
>>> The register address used for the reg property of the adv7482 node in
>>> other Renesas device trees are decimal not hex, change this for Ebisu to
>>> align it with the others.
>>>
>>> Signed-off-by: Takeshi Kihara <takeshi.kihara...@renesas.com>
>>> [Niklas: rewrite commit message]
>>> Signed-off-by: Niklas Söderlund <niklas.soderlund+rene...@ragnatech.se>
>>> ---
>>>  arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts 
>>> b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
>>> index 62bdddcbbae7d9e9..23914c2b83965621 100644
>>> --- a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
>>> +++ b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
>>> @@ -383,7 +383,7 @@
>>>               };
>>>
>>>               port@a {
>>> -                     reg = <0xa>;
>>> +                     reg = <10>;
>>
>> From the discussions on this before, we converted the port number to be hex.
>>
>> I really dislike that we have port@a, but reg=<10>;
>>
>> However as long as the value is written by the DTC as 'ten' and
>> correctly interpreted as decimal this is still correct.
>>
>> I guess it doesn't hurt to have a decimal representation as that will
>> then match the bindings port descriptions. It's a shame this can't be
>> the 'port@10' for comprehension though.
>>
>> My only hesitation on providing either an RB/or AB tag here is whether
>> perhaps we should instead convert the salvator-common to use hex so that
>> they match that way instead?
> 
> (More rationale, which may be considered bikeshedding ;-)

All input here is helpful :)


> I believe we tend to use decimal for small numbers, and hex for large numbers.
> Exceptions are values where the meaning of the individual bits is important,
> and hex is thus more suitable (e.g. rohm,ddr-backup-power = <0xf>).


I believe without the 0x prefix in the reg <> the value is taken as
decimal, so keeping decimal will still function correctly.

I think my issue only stems from the fact that the @N value is 'hex'
without the prefix which causes the difference.

I'll stop fussing over this. The value is interpreted correctly, and as
I said above - having a place where the port number is listed in decimal
helps to match to the bindings table. It's just not in the place I'd
like it to be - but I have no control over that.

So, lets get this patch in and at least unify the current users ...

Acked-by: Kieran Bingham <kieran.bingham+rene...@ideasonboard.com>


> Gr{oetje,eeting}s,
> 
>                         Geert
> 

-- 
Regards
--
Kieran

Reply via email to