On Wed, Nov 16, 2022 at 04:17:28PM +0200, Andy Shevchenko wrote: > Some of the functions are using struct fwnode_handle, some struct device > pointer. In the GPIO library the firmware node of the GPIO device is the > same as GPIO node of the GPIO chip. Due to this fact we may use former > to access properties everywhere in the code. Citing myself from another thread to the topic: Nevertheless, for of_gpiochip_add()/of_gpiochip_remove() and of_mm_gpiochip_add_data() I still left use of fwnode, because it feels the right thing to do: we are taking reference on the input data in such cases. -- With Best Regards, Andy Shevchenko