Hi Patrice,
On 1/3/23 14:35, Patrice CHOTARD wrote:
Hi Tom
This patch is landing in the mailing list since a while.
Do you expect to merge it in master or in next branch soon ?
This patch will be useful for STM32MP SoC in order to clean according machine
code.
The main issue here is that Simon would like a test case for the first
patch in the series and I wasn't able to add tests to the sandbox to
guarantee there's no regression in the future, hence why it was kinda
parked on the ML without much work on it.
From my U-Boot IRC archive:
2022-09-26 15:22:58 ^[[94m^[[0m^[[1m^[[38;5;7m^[[49mqschulz
^[[0msjg1: it seems I'm unable to write a unit test for this dm patch
2022-09-26 15:24:33 ^[[94m^[[0m^[[1m^[[38;5;7m^[[49mqschulz
^[[0msjg1: https://paste.ack.tf/dc8598 is what I came up with, but
impossible to actually run this test
2022-09-26 15:25:23 ^[[94m^[[0m^[[1m^[[38;5;7m^[[49mqschulz ^[[0mway
too many changes to be able to even compile this with sandbox_spl_defconfig
2022-09-26 15:25:47 ^[[94m^[[0m^[[1m^[[38;5;7m^[[49mqschulz ^[[0mand
with sandbox_defconfig, ./u-boot -T -c "ut dm fdt" does not run it either
2022-09-26 15:27:18 ^[[94m^[[0m^[[1m^[[38;5;7m^[[49mqschulz ^[[0mso
i'm not against some guidance on how to write a unit test for that patch
2022-09-26 15:28:52 ^[[94m^[[0m^[[1m^[[38;5;7m^[[49mqschulz
^[[0mwhat we want to test is that in pre-reloc, a driver with
DM_FLAG_PROBE_AFTER_BIND flag but not DM_FLAG_PRE_RELOC gets probed if
it is bound to a device tree node where u-boot,dm-pre-reloc is set
Cheers,
Quentin
