Re: [PATCH v3 0/7] support overlays to the barebox live tree

[Michael Riesch <michael.riesch@wolfvision.net>]

Hi Sascha,

On 3/10/23 10:42, Sascha Hauer wrote:
> Support for applying a device tree overlay to the barebox live tree
> is already present. This series adds a few missing pieces:
> 
> - add support for applying overlays to the live tree from the command
>   line
> - add support for adding new I2C/SPI devices via overlays
> - add Makefile magic to build dtbo files and dtbo.o files to include
>   into the barebox binary from dtso files
> 
> Changes since v2:
> - Don't build compressed dtbo files
> - Do not build overlays for PBL
> - rename symbols to __dtbo_ to avoid possible name clashes
> 
> Changes since v1:
> - add a device_rescan() static inline wrapper
> - add missing call to of_i2c_register_devices() in i2c_add_numbered_adapter()
> - replace device_detect with device_rescan in of_device_create_on_demand()
> 
> Sascha Hauer (7):
>   kbuild: Add target to build dtb overlay files
>   driver: Add rescan hook to struct device
>   i2c: implement rescan
>   spi: Directly register SPI device
>   spi: reduce scope of 'chip'
>   spi: implement rescan
>   of_overlay: Add option to apply overlay to live tree
> 
>  commands/of_overlay.c  | 38 ++++++++++++++++++++++++++++++++++----
>  drivers/i2c/i2c.c      | 24 +++++++++---------------
>  drivers/of/platform.c  | 12 +++++-------
>  drivers/spi/spi.c      | 29 ++++++++++++++++++++++++-----
>  include/driver.h       |  6 ++++++
>  scripts/Makefile.build |  2 ++
>  scripts/Makefile.lib   | 10 +++++++++-
>  scripts/gen-dtbo-s     | 14 ++++++++++++++
>  8 files changed, 103 insertions(+), 32 deletions(-)
>  create mode 100755 scripts/gen-dtbo-s
> 

With this series I can apply a device tree overlay, but a node with a
"fixed-clock" does not lead to a corresponding clock showing up in clk_dump.

Any idea what went wrong here?

Thanks and best regards,
Michael