On Fri, May 06, 2022 at 08:34:09PM +0200, Rafael J. Wysocki wrote: > On Mon, Apr 25, 2022 at 4:14 AM Ian Cowan <ian@xxxxxxxxxxxxxxxx> wrote: > > @@ -190,12 +190,14 @@ static int acpi_ac_battery_notify(struct notifier_block *nb, > > static int __init thinkpad_e530_quirk(const struct dmi_system_id *d) > > { > > ac_sleep_before_get_state_ms = 1000; > > + > > return 0; > > } > > > > static int __init ac_only_quirk(const struct dmi_system_id *d) > > { > > ac_only = 1; > > + > > return 0; > > } > > I don't really think that it is useful to add empty lines in the two > cases above. > > The rest of the patch is fine with me. > I just submitted v2 removing these 2 changes and leaving the others. Thanks! Ian