Messages in this thread | | | Date | Mon, 12 Aug 2019 22:40:24 +0200 | From | Lukas Wunner <> | Subject | Re: [PATCH v2 1/4] PCI: pciehp: Add pciehp_set_indicators() to jointly set LED indicators |
| |
On Mon, Aug 12, 2019 at 11:49:23AM -0700, sathyanarayanan kuppuswamy wrote: > > On 8/11/19 12:59 PM, Denis Efremov wrote: > > > + if ((!PWR_LED(ctrl) || pwr == PWR_NONE) && > > > + (!ATTN_LED(ctrl) || attn == ATTN_NONE)) > > > + return; > > Also I think this condition needs to expand to handle the case whether pwr > != PWR_NONE and !PWR_LED(ctrl) is true. > > you need to return for case, pwr = PWR_ON, !PWR_LED(ctrl)=true , > !ATTN_LED(ctrl)=false, attn=on
Why should we return in that case? We need to update the Attention Indicator Control to On.
Thanks,
Lukas
| |