linux-kernel - Re: [PATCH v2 0/4] nvmem: layouts: Create a macro to register drivers

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]

Date:   Thu, 23 Mar 2023 08:43:17 +0000
From:   Srinivas Kandagatla <srinivas.kandagatla@...aro.org>
To:     Miquel Raynal <miquel.raynal@...tlin.com>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        linux-kernel@...r.kernel.org
Cc:     Robert Marko <robert.marko@...tura.hr>,
        Luka Perkov <luka.perkov@...tura.hr>,
        Thomas Petazzoni <thomas.petazzoni@...tlin.com>,
        Michael Walle <michael@...le.cc>,
        Rafał Miłecki <rafal@...ecki.pl>
Subject: Re: [PATCH v2 0/4] nvmem: layouts: Create a macro to register drivers



On 22/03/2023 16:58, Miquel Raynal wrote:
> Hello Srinivas,
> 
> Following your former review, here are a few changes creating and using
> a macro to simplify nvmem layout driver registration.
> 
> While doing that I figured out the MODULE_ALIAS() macro in the Onie TLV
> driver was useless and wrong, so there is a patch to drop it, but if you
> wish you can also squash it with:
> "nvmem: layouts: onie-tlv: Add new layout driver".
> 
> Thanks!
> Miquèl
> 
> Changes in v2:
> * First patch now includes driver.h as reported by Rafal.
> 
> Miquel Raynal (4):
>    nvmem: Add macro to register nvmem layout drivers
>    nvmem: layouts: sl28vpd: Use module_nvmem_layout_driver()
>    nvmem: layouts: onie-tlv: Use module_nvmem_layout_driver()
>    nvmem: layouts: onie-tlv: Drop wrong module alias

Applied thanks,

--srini

> 
>   drivers/nvmem/layouts/onie-tlv.c | 15 +--------------
>   drivers/nvmem/layouts/sl28vpd.c  | 14 +-------------
>   include/linux/nvmem-provider.h   |  6 ++++++
>   3 files changed, 8 insertions(+), 27 deletions(-)
>