On Thu, 05 Sep 2024 19:37:09 +0200 Kryštof Černý <cleverline1mc@xxxxxxxxx> wrote: Hi Kryštof, thanks for the follow-up! > Just minor changes based on feedback. > > Signed-off-by: Kryštof Černý <cleverline1mc@xxxxxxxxx> 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@xxxxxxxxx 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,