Hello Chen-Yu, +Rob On Thu, 27 Jun 2024 15:19:03 +0800 Chen-Yu Tsai <wenst@xxxxxxxxxxxx> wrote: > Add OF notifier handler needed for creating/destroying MIPI DSI devices > according to dynamic runtime changes in the DT live tree. This code is > enabled when CONFIG_OF_DYNAMIC is selected. > > This is based on existing code for I2C and SPI subsystems. > > Signed-off-by: Chen-Yu Tsai <wenst@xxxxxxxxxxxx> Thanks for copying me on this patch. Could be useful for my hotplug-bridge work, however I'm aiming at writing code that works also for non-DSI so we'll see. The code looks pretty fine however. My concern however is about the usage of an OF reconfig notifier. A few days ago Rob Herring wrote: > a notifier is never a great design so > maybe we can come up with something better (https://lore.kernel.org/all/CAL_Jsq+=mGEJXsjq1UZFMJtHko_z+doiFMXnx9K7exDuznymSA@xxxxxxxxxxxxxx) So maybe this is something you can clarify with him. Luca -- Luca Ceresoli, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
