On Mon, 13 Nov 2023 12:07:37 -0800, Harshit Mogalapalli wrote: > All cases in switch-case have a same goto on error, move the return > check out of the switch. This is a cleanup. > > 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/4] platform/x86: hp-bioscfg: Simplify return check in hp_add_other_attributes() commit: c5dbf04160005e07e8ca7232a7faa77ab1547ae0 [2/4] platform/x86: hp-bioscfg: move mutex_lock() down in hp_add_other_attributes() commit: 5736aa9537c9b8927dec32d3d47c8c31fe560f62 [3/4] platform/x86: hp-bioscfg: Fix error handling in hp_add_other_attributes() commit: f40f939917b2b4cbf18450096c0ce1c58ed59fae [4/4] platform/x86: hp-bioscfg: Remove unused obj in hp_add_other_attributes() commit: 92c47597db7d8fb500a4b04ebd457ec7360279cc -- i.