Hi Daniel, On Sun, Oct 15, 2023 at 6:54 PM Daniel Lezcano <daniel.lezcano@xxxxxxxxxx> wrote: > > Are you happy with the v10 series? > > Yes, I think they are fine except one thing. > > The include/linux/reboot.h is changed along with thermal*.c file. IMO it > is preferable to have separate patch, I mean all reboot.h changes folded > in a single patch before the thermal_*.c changes. It is actually I tried to follow your suggestion of putting all reboot.h changes folded in a single patch before the thermal_*.c changes, but I don't think I can do this split and maintain a logic patch separation and bisectability. > orphaned and we should ask Matti Vaittinen <mazziesaccount@xxxxxxxxx> > its acked-by as he is the author of the code you are changing. > Otherwise, he will have to ack the patches which contain also thermal > code which is not its area. The reason I haven't added Matti on Cc on the series is that get_maintainer did not list him. Added him on Cc now. Maybe Matti could help acking patches 2/4 and 3/4? Full series: https://lore.kernel.org/linux-pm/20231006180453.2903342-1-festevam@xxxxxxxxx/ https://lore.kernel.org/linux-pm/20231006180453.2903342-2-festevam@xxxxxxxxx/ https://lore.kernel.org/linux-pm/20231006180453.2903342-3-festevam@xxxxxxxxx/ https://lore.kernel.org/linux-pm/20231006180453.2903342-4-festevam@xxxxxxxxx/
