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. 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; > > > > > >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#11355): https://lists.yoctoproject.org/g/linux-yocto/message/11355 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]] -=-=-=-=-=-=-=-=-=-=-=-
