Hello Rob, On Fri, 09 Aug 2024 17:34:48 +0200 Luca Ceresoli <luca.ceresoli@xxxxxxxxxxx> 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 https://bootlin.com