[PATCH v2 1/3] dt-bindings: nvmem: syscon: Add syscon backed nvmem bindings

[Marek Vasut]

On 5/31/23 21:40, Krzysztof Kozlowski wrote:
> On 24/05/2023 05:29, Marek Vasut wrote:
>> On 5/18/23 16:30, Krzysztof Kozlowski wrote:
>>> On 17/05/2023 17:25, Marek Vasut wrote:
>>>> Add trivial bindings for driver which permits exposing syscon backed
>>>> register to userspace. This is useful e.g. to expose U-Boot boot
>>>> counter on various platforms where the boot counter is stored in
>>>> random volatile register, like STM32MP15xx TAMP_BKPxR register.
>>>>
>>>> Signed-off-by: Marek Vasut <marex at denx.de>
>>>> ---
>>>
>>> Let me also leave a note here - cross linking for all parties:
>>>
>>> Please propose a solution solving also mediatek,boottrap, probably
>>> extendable to range of registers. Other solution is what Rob mentioned -
>>> this might not be suitable for generic binding and device.
>>
>>   From what I can tell, shouldn't the mediatek 1g MAC driver have a
>> nvmem-cells phandle to this OTP/fuses/whatever area and shouldn't the
>> driver read out and decode its settings within the kernel ?
> 
> Maybe, but since you both implement the same driver and similar
> bindings, it's a NAK for having both. It looks like solution matching
> both or boottrap is not really nvmem syscon (as you said).

The later.

>> That doesn't seem really related to this particular issue I'm trying to
>> solve I'm afraid.
> 
> It's similar in implementation, not necessarily in hardware.

It seems to me what the mediatek thing is doing is already a solved 
problem, look e.g. at what imx8mp is doing, they read MAC address from 
OTP via nvmem . That's however unrelated to this work .