2022-07-26 21:30
lanefu changed the topic of #armbian-rockchip to: Armbian - Linux for ARM development boards | Rockchip SoC |
www.armbian.com | This channel is relayed to the equivalent Discord channel | this channel is logged
12:04
archetech has joined #armbian-rockchip
20:05
<
DC-IRC >
[Discord] <rpardini> yeah, it's tricky, as BOOTCONFIG will be as `make ... $BOOTCONFIG` - always
20:05
<
DC-IRC >
[Discord] <rpardini> Then as you have another defconfig in each target, it will then
_change_ the config, in theory, per-target
20:05
<
DC-IRC >
[Discord] <rpardini> I've had similar problem in another board and had to do a stupid hack, let me find
20:07
<
DC-IRC >
[Discord] <rpardini> we
_must_ come up with a better way than UBOOT_TARGET_MAP, really. it's hell
21:06
<
DC-IRC >
[Discord] <efectn> Yeah it makes the things more and more complex
21:07
<
DC-IRC >
[Discord] <efectn> was it odroid hc4?
21:07
<
DC-IRC >
[Discord] <efectn> I guess i got why you changes bootconfig with hooks now
21:07
<
DC-IRC >
[Discord] <efectn> I guess i got why you changed bootconfig with hooks now
21:12
<
DC-IRC >
[Discord] <rpardini> yeah indeed hc4. but that's only because I'm changing configs in hooks, not separate defconfig files
21:12
<
DC-IRC >
[Discord] <rpardini> that forced me into that `armbian_target=sd` bs
21:14
<
DC-IRC >
[Discord] <efectn> Yeah good idea
21:14
<
DC-IRC >
[Discord] <rpardini> in this case is vendor rk uboot no?
21:15
<
DC-IRC >
[Discord] <efectn> Yes it is vendor uboot
21:17
<
DC-IRC >
[Discord] <rpardini> oh maybe you've a mixup with `rkspi_loader.img` ?
21:22
<
DC-IRC >
[Discord] <efectn> I don't use the uboot map generated by rockchip64 common
21:22
<
DC-IRC >
[Discord] <efectn> I set it to UBOOT_TARGET_MAP="nanopi-m6-rk3588s_defconfig BL31=$RKBIN_DIR/$BL31_BLOB spl/u-boot-spl.bin u-boot.dtb u-boot.itb;;idbloader.img u-boot.itb
21:22
<
DC-IRC >
[Discord] <efectn> nanopi-m6-spi-rk3588s_defconfig BL31=$RKBIN_DIR/$BL31_BLOB;; rkspi_loader.img" using a hook
21:23
<
DC-IRC >
[Discord] <efectn> But it seems like i have to use the way hc4 uses
21:23
<
DC-IRC >
[Discord] <efectn> I set it to `UBOOT_TARGET_MAP="nanopi-m6-rk3588s_defconfig BL31=$RKBIN_DIR/$BL31_BLOB spl/u-boot-spl.bin u-boot.dtb u-boot.itb;;idbloader.img u-boot.itb
21:23
<
DC-IRC >
[Discord] <efectn> nanopi-m6-spi-rk3588s_defconfig BL31=$RKBIN_DIR/$BL31_BLOB;; rkspi_loader.img"` using a hook
21:30
<
DC-IRC >
[Discord] <rpardini> looks to me:
21:30
<
DC-IRC >
[Discord] <rpardini> 1) you're not specifying the targets for the spi defconfig
21:30
<
DC-IRC >
[Discord] <rpardini> 2) framework has a bug and uses `rkspi_loader.img` from previous target
21:31
<
DC-IRC >
[Discord] <rpardini> for which I'm very, very sorry if that's the case
21:33
<
DC-IRC >
[Discord] <rpardini> have you had to short SPI pins? 😄
21:33
<
DC-IRC >
[Discord] <efectn> SPI is unattachable
21:34
<
DC-IRC >
[Discord] <rpardini> wtf produces `rkspi_loader.img` lol
21:35
<
DC-IRC >
[Discord] <efectn> I think it uses wrong defconfig
21:36
<
DC-IRC >
[Discord] <rpardini> nope, all it does is the defconfig (only)
21:36
<
DC-IRC >
[Discord] <rpardini> and does not build u-boot.itb and idbloader.img that is what compose rkspi_loader.img
21:37
<
DC-IRC >
[Discord] <efectn> If i create idblosder and itb for spi defconfig, then it will override the default one's while packaging right?
21:38
<
DC-IRC >
[Discord] <efectn> If i create idbloader and itb for spi defconfig, then it will override the default one's while packaging right?
21:38
<
DC-IRC >
[Discord] <rpardini> you can try
21:38
<
DC-IRC >
[Discord] <rpardini> ```bash
21:38
<
DC-IRC >
[Discord] <rpardini> UBOOT_TARGET_MAP="
21:38
<
DC-IRC >
[Discord] <rpardini> nanopi-m6-rk3588s_defconfig BL31=$RKBIN_DIR/$BL31_BLOB spl/u-boot-spl.bin u-boot.dtb u-boot.itb;;idbloader.img u-boot.itb
21:38
<
DC-IRC >
[Discord] <rpardini> nanopi-m6-spi-rk3588s_defconfig BL31=$RKBIN_DIR/$BL31_BLOB spl/u-boot-spl.bin u-boot.dtb u-boot.itb;;rkspi_loader.img
21:38
<
DC-IRC >
[Discord] <rpardini> "
21:38
<
DC-IRC >
[Discord] <rpardini> ```
21:41
<
DC-IRC >
[Discord] <rpardini> I need to add proper cleaning before building a target
21:41
<
DC-IRC >
[Discord] <rpardini> so it does not inherit binaries from previous targets, also I think here maybe even across-boards/builds is interferring
21:53
<
DC-IRC >
[Discord] <rpardini> yeah proper cleaning (`git clean -xfd`) demonstrates the bug clearly
21:53
<
DC-IRC >
[Discord] <rpardini> but also does cause every build to be a full rebuild
21:54
<
DC-IRC >
[Discord] <rpardini> ccache helps, but still
22:15
<
DC-IRC >
[Discord] <rpardini> goddamn it
22:15
<
DC-IRC >
[Discord] <rpardini> it's still using the old `rkspi_loader.img`
22:16
<
DC-IRC >
[Discord] <rpardini> you
_need_ `BOOT_SUPPORT_SPI="yes"` and `BOOT_SPI_RKSPI_LOADER="yes"`
22:16
<
DC-IRC >
[Discord] <rpardini> otherwise `rkspi_loader.img` is not created by the code in rk64_common 😐
22:19
<
DC-IRC >
[Discord] <rpardini> but if you do that, then it touches your `UBOOT_TARGET_MAP` 😿
22:20
<
DC-IRC >
[Discord] <rpardini> this is absolute hell, very sorry.
22:21
<
DC-IRC >
[Discord] <efectn> This uboot target map stuff is exactly awful 😅😅
22:21
<
DC-IRC >
[Discord] <rpardini> it's the combo of target_map being a bad idea with the postprocessor function
22:31
<
DC-IRC >
[Discord] <rpardini> nah in your case you're fine as you're setting `UBOOT_TARGET_MAP` in a post_family_config hook
22:32
<
DC-IRC >
[Discord] <rpardini> try this
22:32
<
DC-IRC >
[Discord] <rpardini> ```bash
22:32
<
DC-IRC >
[Discord] <rpardini> BOOT_SUPPORT_SPI="yes"
22:32
<
DC-IRC >
[Discord] <rpardini> BOOT_SPI_RKSPI_LOADER="yes"
22:32
<
DC-IRC >
[Discord] <rpardini> function post_family_config__nanopi_m6_support_spi_boot() {
22:32
<
DC-IRC >
[Discord] <rpardini> UBOOT_TARGET_MAP="nanopi-m6-rk3588s_defconfig BL31=$RKBIN_DIR/$BL31_BLOB spl/u-boot-spl.bin u-boot.dtb u-boot.itb;;idbloader.img u-boot.itb
22:32
<
DC-IRC >
[Discord] <rpardini> nanopi-m6-spi-rk3588s_defconfig BL31=$RKBIN_DIR/$BL31_BLOB spl/u-boot-spl.bin u-boot.dtb u-boot.itb;;rkspi_loader.img"
22:32
<
DC-IRC >
[Discord] <rpardini> }
22:32
<
DC-IRC >
[Discord] <rpardini> ```
22:32
<
DC-IRC >
[Discord] <rpardini> the whitespace is significant in this
_crap_ `UBOOT_TARGET_MAP` string
22:33
<
DC-IRC >
[Discord] <efectn> Ok will try it tomorrow
22:33
<
DC-IRC >
[Discord] <rpardini> terribly sorry Efe, I'll fix a bit 😉
22:33
<
DC-IRC >
[Discord] <rpardini> 🫡
22:42
<
DC-IRC >
[Discord] <efectn> No problem 😄