On Thu, 07 Nov 2024 11:35:43 +0000, Colin Ian King wrote: > The if statement performs the same action if the strcmp result is > true or false since there is identical code on both branches. The > if statement is redundant and can be replaced with just one call > to sysfs_remove_group. > > Thank you for your contribution, it has been applied to my local review-ilpo branch. Note it will show up in the public platform-drivers-x86/review-ilpo branch only once I've pushed my local branch there, which might take a while. The list of commits applied: [1/1] platform/x86: hp: hp-bioscfg: remove redundant if statement commit: 4ceb681f1822819f78b747ddc189479fead43be2 -- i.