[PATCH v3 2/2] gpio: Get rid of gpio_hog_probe_all()

Quentin Schulz quentin.schulz at theobroma-systems.com
Tue Jan 3 14:50:19 CET 2023


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


More information about the U-Boot mailing list