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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxx>
>>> ---
>>
>> 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).

> 
> 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.

Best regards,
Krzysztof




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux