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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20240814175952.6b22b7ad@booty>
Date: Wed, 14 Aug 2024 17:59:52 +0200
From: Luca Ceresoli <luca.ceresoli@...tlin.com>
To: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
 Conor Dooley <conor+dt@...nel.org>
Cc: Andrzej Hajda <andrzej.hajda@...el.com>, Neil Armstrong
 <neil.armstrong@...aro.org>, Robert Foss <rfoss@...nel.org>, Laurent
 Pinchart <Laurent.pinchart@...asonboard.com>, Jonas Karlman
 <jonas@...boo.se>, Jernej Skrabec <jernej.skrabec@...il.com>, Maarten
 Lankhorst <maarten.lankhorst@...ux.intel.com>, Maxime Ripard
 <mripard@...nel.org>, Thomas Zimmermann <tzimmermann@...e.de>, David Airlie
 <airlied@...il.com>, Daniel Vetter <daniel@...ll.ch>, Derek Kiernan
 <derek.kiernan@....com>, Dragan Cvetic <dragan.cvetic@....com>, Arnd
 Bergmann <arnd@...db.de>, Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
 Saravana Kannan <saravanak@...gle.com>, Wolfram Sang
 <wsa+renesas@...g-engineering.com>, "Rafael J. Wysocki"
 <rafael@...nel.org>, Paul Kocialkowski <contact@...lk.fr>, Hervé Codina <herve.codina@...tlin.com>, Thomas Petazzoni
 <thomas.petazzoni@...tlin.com>, devicetree@...r.kernel.org,
 linux-kernel@...r.kernel.org, dri-devel@...ts.freedesktop.org,
 linux-i2c@...r.kernel.org, Paul Kocialkowski
 <paul.kocialkowski@...tlin.com>
Subject: Re: [PATCH v3 0/7] Add support for GE SUNH hot-pluggable connector

Hello Rob,

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