Re: [PATCH v3 0/7] Add support for GE SUNH hot-pluggable connector

Luca Ceresoli
2024-08-14T15:59:58Z

Hello Rob,

On Fri, 09 Aug 2024 17:34:48 +0200
Luca Ceresoli <luca.ceresoli@bootlin.com> wrote:

...

> However a few new rough edges emerged that are not yet solved in this
> v3. Discussion would help in finding the right direction:
>
> * Describing the NVMEM cell addition still requires adding two properties
> to a node in the base tree. Not sure the current NVMEM cell bindings
> allow to do better.

Do you have any thoughts about how to describe the NVMEM cell in DT
without adding properties?

As of now...

> 2: the "base" overlay
>
> The "base" overlay describes the common components that are required to
> read the model ID. These are identical for all add-on models, thus only one
> "base" overlay is needed:
>
> /dts-v1/;
> /plugin/;
>
> / {
> fragment@0 {
> target-path = "";
>
> __overlay__ {
> nvmem-cells = <&addon_id>;
> nvmem-cell-names = "id";

...the nvmem-cell* properties are the only ones that for lack of a
good solution get added to a node in the base tree, causing the memory
leak issue to be still present.

Luca

--
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering