Re: [PATCH v3 0/2] fw_devlink overlay fix

Hi Saravana,

On Tue, 23 Apr 2024 10:39:24 +0200
Herve Codina [off-list ref] wrote:

Hi Saravana,

On Thu, 11 Apr 2024 16:56:20 -0700
Saravana Kannan [off-list ref] wrote:

quoted

Overlays don't work correctly with fw_devlink. This patch series fixes
it. This series is now ready for review and merging once Geert and Herve
give they Tested-by.

Geert and Herve,

This patch series should hopefully fix both of your use cases [1][2][3].
Can you please check to make sure the device links created to/from the
overlay devices are to/from the right ones?

Thanks,
Saravana
  

I tested the series.

On my Microchip use case (i.e. DT overlay on a PCIe device), I observed that
some driver removal were done in a wrong order. For instance, the onboard
PCIe device interrupt controller (oic@e00c0120) was removed before its
consumers.

I enabled debug traces in core.c and observed that many links were dropped.
These links are related to pinctrl, clock, reset, interrupts, ...
I have the feeling that these links should not be dropped.

Have you made any progress on this topic ?

I haven't seen any updates.
Maybe I missed something.

Best regards,
Hervé