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

[Neil Armstrong <neil.armstrong@linaro.org>]

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
> <neil.armstrong@linaro.org> wrote:
> 
>>         arm64: dts: amlogic: t7: khadas-vim4: Remove invalid property
> 
> This commit has a problematic Fixes:
> 
> commit 6f14ba3c3fa3d67b72e052f431f6776470eb2cbd
> Author: Ronald Claveau <linux-kernel-dev@aliel.fr>
> 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 <lkp@intel.com>
>      Closes: https://lore.kernel.org/oe-kbuild-all/202603290828.5gt393t6-lkp@intel.com/
>      Fixes: 60eff75ac67b ("arm64: dts: amlogic: t7: khadas-vim4: Add
> power regulators")
>      Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
>      Signed-off-by: Ronald Claveau <linux-kernel-dev@aliel.fr>
>      Link: https://patch.msgid.link/20260330-fix-invalid-property-v2-1-228c51c8de93@aliel.fr
>      Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
> 
> 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