Re: [GIT PULL] amlogic ARM64 DT updates for v7.2 take 1

Hi Linus,

On Mon, Jun 1, 2026 at 10:06 PM Linus Walleij <[email protected]> wrote:
>
> Hi Neil,
>
> thanks for your pull request!
>
> On Thu, May 21, 2026 at 3:19 PM Neil Armstrong
> <[email protected]> wrote:
>
> >        arm64: dts: amlogic: t7: khadas-vim4: Remove invalid property
>
> This commit has a problematic Fixes:
>
> commit 6f14ba3c3fa3d67b72e052f431f6776470eb2cbd
> Author: Ronald Claveau <[email protected]>
> Date:   Mon Mar 30 14:11:21 2026 +0200
>
>     arm64: dts: amlogic: t7: khadas-vim4: Remove invalid property
>
>     Fix introduced invalid property for Khadas VIM4 sdcard regulator.
>
>     arch/arm64/boot/dts/amlogic/amlogic-t7-a311d2-khadas-vim4.dtb:
> regulator-sdcard-3v3 (regulator-fixed): Unevaluated properties are not
> allowed ('enable-active-low' was unexpected)
>
>     Reported-by: kernel test robot <[email protected]>
>     Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/
>     Fixes: 60eff75ac67b ("arm64: dts: amlogic: t7: khadas-vim4: Add
> power regulators")
>     Reviewed-by: Krzysztof Kozlowski <[email protected]>
>     Signed-off-by: Ronald Claveau <[email protected]>
>     Link: https://patch.msgid.link/[email protected]
>     Signed-off-by: Neil Armstrong <[email protected]>
>
> Commit 60eff75ac67b does not exist in the SoC tree nor in Torvalds'
> tree.
That should be 8f4e4fbd5072 (which is part of this PR - as the
extraneous property was only found after applying).
It seems that Neil did rebase (to modify or drop a commit?) after this
fix was applied. But I can't fully recall the details.
Linus, what's the suggested fix in this situation?


Best regards,
Martin