Messages in this thread | | | From | Andy Shevchenko <> | Date | Tue, 2 Nov 2021 16:12:54 +0200 | Subject | Re: [PATCH v5 07/11] platform/x86: int3472: Split into 2 drivers |
| |
On Tue, Nov 2, 2021 at 11:49 AM Hans de Goede <hdegoede@redhat.com> wrote: > > The intel_skl_int3472.ko module contains 2 separate drivers, > the int3472_discrete platform driver and the int3472_tps68470 > I2C-driver. > > These 2 drivers contain very little shared code, only > skl_int3472_get_acpi_buffer() and skl_int3472_fill_cldb() are > shared. > > Split the module into 2 drivers, linking the little shared code > directly into both. > > This will allow us to add soft-module dependencies for the > tps68470 clk, gpio and regulator drivers to the new > intel_skl_int3472_tps68470.ko to help with probe ordering issues > without causing these modules to get loaded on boards which only > use the int3472_discrete platform driver. > > While at it also rename the .c and .h files to remove the > cumbersome intel_skl_int3472_ prefix. > > Signed-off-by: Hans de Goede <hdegoede@redhat.com> > --- > Note git rename detection is failing for the new common.c but this is > just the old intel_skl_int3472_common.c with the driver registering > bits removed.
-M accepts a parameter (in percents) to set the threshold. Default is something like ~70 (?). You may play with it and see when it starts detecting the rename.
-- With Best Regards, Andy Shevchenko
| |