linux-kernel - Re: [PATCH V2 0/3] nvmem: support post read for U-Boot's "ethaddr"

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <7e9ee8e2-988e-1f51-e7ed-19f9e3181d9e@linaro.org>
Date:   Thu, 23 Mar 2023 08:23:40 +0000
From:   Srinivas Kandagatla <srinivas.kandagatla@...aro.org>
To:     Rafał Miłecki <zajec5@...il.com>,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>
Cc:     Shawn Guo <shawnguo@...nel.org>,
        Sascha Hauer <s.hauer@...gutronix.de>,
        Pengutronix Kernel Team <kernel@...gutronix.de>,
        Fabio Estevam <festevam@...il.com>,
        NXP Linux Team <linux-imx@....com>,
        Miquel Raynal <miquel.raynal@...tlin.com>,
        Michael Walle <michael@...le.cc>, gregkh@...uxfoundation.org,
        devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        linux-kernel@...r.kernel.org, u-boot@...ts.denx.de,
        Rafał Miłecki <rafal@...ecki.pl>
Subject: Re: [PATCH V2 0/3] nvmem: support post read for U-Boot's "ethaddr"



On 18/03/2023 17:13, Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal@...ecki.pl>
> 
> This change is required for NVMEM consumers to get expected MAC address
> from U-Boot env data variable "ethaddr".
> 
> To address some previous concerns:
> 1. Yes, U-Boot env binding & driver should be converted to NVMEM layout
> 2. My priority is to get working MAC rather than fancy DT syntax
> 3. I AM going to convert U-Boot env into layout later
> 4. This work DOESN'T conflict with layout migration
> 5. This code WILL stay during / after layout migration
> 
> My brain limitations don't allow me to handle everything at once, please
> let's review what I already have here.


Thanks for your patience.

Applied now,

--srini

> 
> Rafał Miłecki (3):
>    dt-bindings: nvmem: u-boot,env: add MAC's #nvmem-cell-cells
>    nvmem: core: support specifying both: cell raw data & post read
>      lengths
>    nvmem: u-boot-env: post-process "ethaddr" env variable
> 
>   .../devicetree/bindings/nvmem/u-boot,env.yaml |  7 ++++-
>   drivers/nvmem/Kconfig                         |  1 +
>   drivers/nvmem/core.c                          | 11 +++++---
>   drivers/nvmem/u-boot-env.c                    | 26 +++++++++++++++++++
>   include/linux/nvmem-provider.h                |  2 ++
>   5 files changed, 42 insertions(+), 5 deletions(-)
>