linux-kernel - Re: [PATCH v3 0/2] NanoPi NEO Plus2: Fix regulators and assign them

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: <20240905235847.3b4b1374@minigeek.lan>
Date: Thu, 5 Sep 2024 23:58:47 +0100
From: Andre Przywara <andre.przywara@....com>
To: Kryštof Černý <cleverline1mc@...il.com>
Cc: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
 Conor Dooley <conor+dt@...nel.org>, Chen-Yu Tsai <wens@...e.org>, Jernej
 Skrabec <jernej.skrabec@...il.com>, Samuel Holland <samuel@...lland.org>,
 devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
 linux-sunxi@...ts.linux.dev, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 0/2] NanoPi NEO Plus2: Fix regulators and assign them

On Thu, 05 Sep 2024 19:37:09 +0200
Kryštof Černý <cleverline1mc@...il.com> wrote:

Hi Kryštof,

thanks for the follow-up!

> Just minor changes based on feedback.
> 
> Signed-off-by: Kryštof Černý <cleverline1mc@...il.com>

There is no need for a Signed-off-by: in the cover letter.
But you are supposed to add any tags, like my "Reviewed-by", to a new
revision.
I guess unfortunately you missed the v6.12 merge window by an hour or
so, but in any case I hope Chen-Yu can copy in the tags when applying.

> ---
> Changes in v3:
> - Renamed patches - added prefix.
> - Remove the patch that added regulator to emmc.
> - Link to v2: https://lore.kernel.org/r/20240827-nanopi-neo-plus2-regfix-v2-0-497684ec82c7@gmail.com

Thanks for the changelog, that's helpful. Ideally you would stack this
up with the changes from previous revisions, so listing the changes
from v1 .. v2 here as well.

Cheers,
Andre

> 
> ---
> Kryštof Černý (2):
>       arm64: dts: allwinner: H5: NanoPi Neo Plus2: Fix regulators
>       arm64: dts: allwinner: H5: NanoPi NEO Plus2: Use regulators for pio
> 
>  .../dts/allwinner/sun50i-h5-nanopi-neo-plus2.dts   | 37 ++++++++++++++++++++++
>  1 file changed, 37 insertions(+)
> ---
> base-commit: 6f923748057a4f6aa187e0d5b22990d633a48d12
> change-id: 20240827-nanopi-neo-plus2-regfix-b60116ce12ca
> 
> Best regards,