From: Uwe Kleine-König <uwe@xxxxxxxxxxxxxxxxx> Hello, compared to the goal (i.e. getting rid of some forward declarations) the change is quite big. Also the diffstat is positive because I spent some more empty lines than the original driver. Still I think its worthwhile as it prevents errors like https://lore.kernel.org/linux-i2c/20220916090802.261279-1-u.kleine-koenig@xxxxxxxxxxxxxx . Note that the 2nd patch isn't checkpatch-clean, but the problems are all "old", I didn't touch any function bodys, just shuffled them around. Uwe Kleine-König (2): hwmon: pc87360: Introduce a #define for the driver name and use it hwmon: pc87360: Reorder symbols to get rid of a few forward declarations drivers/hwmon/pc87360.c | 1535 ++++++++++++++++++++------------------- 1 file changed, 776 insertions(+), 759 deletions(-) base-commit: 568035b01cfb107af8d2e4bd2fb9aea22cf5b868 -- 2.37.2