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

Hi,

On 6/1/26 22:06, Linus Walleij 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.
> 
> What's up here, it looks incorrect?

Yeah sorry it was probably based on an earlier apply for the previous rejected PR

The right commit is 8f4e4fbd5072 arm64: dts: amlogic: t7: khadas-vim4: Add power regulators

But since it's in the same PR I will just squash it...

@arnd I'll post a v2 of this PR with this issue fixed.

Thanks,
Neil

> 
> Can you look into it?
> 
> Yours,
> Linus Walleij