On Tue, May 31, 2022 at 4:13 PM Bruce Ashfield via lists.yoctoproject.org <[email protected]> wrote: > > In message: Re: [linux-yocto][v5.10/standard/preempt-rt/base][PATCH] ACPI: > thermal: drop an always true check > on 31/05/2022 Quanyang Wang wrote: > > > Hi Bruce, > > > > This patch is for v5.15, I wrote a wrong kernel version in Subject. > > No problem. > > But also, why would this just be on preempt-rt, shouldn't it be everywhere ? > > I see the same checks on standard/base as well.
Never mind, I see that you covered that in the patch itself! Bruce > > Bruce > > > > > Thanks, > > > > Quanyang > > > > On 5/31/22 15:49, quanyang.wang wrote: > > > From: Adam Borowski <[email protected]> > > > > > > commit e5b5d25444e9ee3ae439720e62769517d331fa39 upstream. > > > > > > Address of a field inside a struct can't possibly be null; gcc-12 warns > > > about this. > > > > > > Signed-off-by: Adam Borowski <[email protected]> > > > Signed-off-by: Rafael J. Wysocki <[email protected]> > > > --- > > > Hi Bruce, > > > Would you please help merge this patch to the branches: > > > v5.15/standard/preempt-rt/base > > > v5.15/standard/base > > > Thanks, > > > Quanyang > > > --- > > > drivers/acpi/thermal.c | 2 -- > > > 1 file changed, 2 deletions(-) > > > > > > diff --git a/drivers/acpi/thermal.c b/drivers/acpi/thermal.c > > > index 95105db642b9..155bbabcc6f5 100644 > > > --- a/drivers/acpi/thermal.c > > > +++ b/drivers/acpi/thermal.c > > > @@ -1098,8 +1098,6 @@ static int acpi_thermal_resume(struct device *dev) > > > return -EINVAL; > > > for (i = 0; i < ACPI_THERMAL_MAX_ACTIVE; i++) { > > > - if (!(&tz->trips.active[i])) > > > - break; > > > if (!tz->trips.active[i].flags.valid) > > > break; > > > tz->trips.active[i].flags.enabled = 1; > > > > > > > > > > > > -- - Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end - "Use the force Harry" - Gandalf, Star Trek II
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#11356): https://lists.yoctoproject.org/g/linux-yocto/message/11356 Mute This Topic: https://lists.yoctoproject.org/mt/91447016/21656 Group Owner: [email protected] Unsubscribe: https://lists.yoctoproject.org/g/linux-yocto/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
