Hi John,
On 1/17/23 16:20, John Keeping wrote:
Hi Johan,
On Tue, Jan 17, 2023 at 03:44:07PM +0100, Johan Jonker wrote:
On 1/17/23 10:46, Quentin Schulz wrote:
On 1/16/23 20:45, Johan Jonker wrote:
Sync rk3066/rk3188 DT files from Linux. This is the state as of linux-next v6.2-rc4. New nfc node for MK808 rk3066a. CRU nodes now have a clock property. To prefend dtoc errors a fixed clock must also be included for tpl/spl in the rk3xxx-u-boot.dtsi file.
Signed-off-by: Johan Jonker jbx6244@gmail.com
[..]
@@ -223,7 +224,7 @@ #size-cells = <1>; ranges;
- gpio0: gpio0@2000a000 { + gpio0: gpio@2000a000 { compatible = "rockchip,rk3188-gpio-bank0"; reg = <0x2000a000 0x100>; interrupts = <GIC_SPI 54 IRQ_TYPE_LEVEL_HIGH>; @@ -236,7 +237,7 @@ #interrupt-cells = <2>; };
- gpio1: gpio1@2003c000 {
+ gpio1: gpio@2003c000 {
Hi,
LOL: I made that binding change on request from Linux DT maintainers. Node names should generic.
===
My full u-boot is able to boot a Linux kernel for rk3066a. Only when I give the command below it crashes:
gpio status -a
Could you confirm what other parts are effected?
If it's boots then it's good enough for me and move forward, so please merge.(Kever)
Driver fixes for u-boot depending on Linux DT changes is already very time consuming enough!
This reminded me of a patch that I never followed up with:
https://urldefense.com/v3/__https://lore.kernel.org/u-boot/20220726162509.13...
Can you test if that fixes `gpio status -a` for you?
That's not enough I believe. You need CONFIG_DM_SEQ_ALIAS enabled, which isn't a given and we shouldn't rely on it. Also, there are currently no aliases for gpio controllers. And we shouldn't rely on u-boot.dtsi for those, because upstream might decide to have an alias gpio2 for gpio0 for a board.
Cheers, Quentin