On Tue, Aug 01, 2023 at 09:52:05AM -0500, Jorge Lopez wrote: > I will submit a new patch replacing 'value_len' for 'size' in line 267 > as indicated. > 'value_len' is utilized earlier in the code so we cannot remove it > completely from the function. > After replacing size then it looks like this. $ grep value_len drivers/platform/x86/hp/hp-bioscfg/order-list-attributes.c int value_len = 0; &str_value, &value_len); &str_value, &value_len); It's a write only variable. regards, dan carpenter