Hello, On Sat, Jun 01, 2024 at 05:00:29PM +0200, Christophe JAILLET wrote: > 'struct power_supply_maintenance_charge_table' is not modified in this > driver. > > Constifying this structure moves some data to a read-only section, so > increase overall security. > > In order to do it, some code also needs to be adjusted to this new const > qualifier. > > On a x86_64, with allmodconfig: > Before: > ====== > $ size drivers/power/supply/samsung-sdi-battery.o > text data bss dec hex filename > 4055 4584 0 8639 21bf drivers/power/supply/samsung-sdi-battery.o > > After: > ===== > $ size drivers/power/supply/samsung-sdi-battery.o > text data bss dec hex filename > 4087 4552 0 8639 21bf drivers/power/supply/samsung-sdi-battery.o > > Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx> > --- > Compile tested-only > --- Looks like that compile testing did not include ab8500_chargalg :) Fixing the error reported by the Intel bot should be trivial (i.e. add another const). When you send a new version, please also Cc Linus Walleij <linus.walleij@xxxxxxxxxx>. Thanks for your patches and greetings, -- Sebastian
Attachment:
signature.asc
Description: PGP signature