Hello, As part of a previous series, Rob suggested that keeping too much logic in of/device.c was backward and would benefit from a gradual cleanup with the hope some day to move the remaining helpers into inline functions wrapping the proper of_*() methods. Link: https://lore.kernel.org/lkml/CAL_JsqJE43qfYzHUuCJsbaPPBTbYX05Q7FFmPTjPFZ3Dmz_mXg@xxxxxxxxxxxxxx/ A few of these "conversions" happened in the series I was originally working on. At this time I actually wrote a few other changes, completely unrelated to my original series, but still following Rob's cleanup idea: here they are. Link: https://lore.kernel.org/lkml/20230307165359.225361-1-miquel.raynal@xxxxxxxxxxx/ The last step of this series is to actually remove a copy of one of these helpers which I think is not needed. This drivers/gpu/ patch depends on the previous changes. Thanks, Miquèl Miquel Raynal (5): of: module: Mutate of_device_modalias() into two helpers of: module: Mutate of_device_uevent() into two helpers of: module: Mutate of_device_uevent_modalias() into two helpers of: module: Export of_uevent() gpu: host1x: Stop open-coding of_device_uevent() drivers/gpu/host1x/bus.c | 31 +++----------- drivers/of/device.c | 90 --------------------------------------- drivers/of/module.c | 82 +++++++++++++++++++++++++++++++++++ include/linux/of.h | 21 +++++++++ include/linux/of_device.h | 39 ++++++++++++++--- 5 files changed, 141 insertions(+), 122 deletions(-) -- 2.34.1
