Hi Balaji, On 24/07/2025 11:00, Balaji Selvanathan wrote: > This patch series improves the reliability of EFI capsule updates by > fixing issues in device path matching and partition name filtering. > > The first patch updates the logic in find_handle() to correctly match > device paths when the EFI System Partition (ESP) is a sub-partition > within a larger boot device. Previously, the function only compared > paths when the candidate path was shorter or equal in length to the > input path. > This caused failures when the ESP path was longer, as is > common when the ESP resides inside a UFS LUN. The new logic allows > matching when the input path is a prefix of the candidate path, > ensuring the correct EFI handle is returned for accessing the ESP > during capsule updates. > > The second patch fixes a bug in 'capsule_update.c' where the logic > was incorrectly matching partitions like 'uefisecapps_a' when it > intended to match only 'uefi_a' or 'uefi_b'. This prevents flashing > to unintended partitions. > > Together, these changes improve the correctness and safety of the > capsule update process on Snapdragon platforms.
Same deal as in patch 2, this isn't an essay. Please refer to https://www.kernel.org/doc/html/v5.10/process/submitting-patches.html#describe-your-changes Specifically "Describe your changes in imperative mood", you shouldn't need to use the words "patch" or "patch series". Compare to [1] for example. Save the technical details for the commit messages and use the cover letter to provide a brief overview of what the series does to the codebase. [1]: https://lore.kernel.org/u-boot/20250618-sm6350-bringup-v1-0-3cc22aab7...@fairphone.com Kind regards, > > Balaji Selvanathan (2): > efi_loader: improve device path matching for partition lookup > capsule_update: fix partition match to exclude uefisecapps > > arch/arm/mach-snapdragon/capsule_update.c | 2 +- > lib/efi_loader/efi_device_path.c | 62 ++++++++++++++++------- > lib/efi_loader/efi_disk.c | 2 +- > 3 files changed, 46 insertions(+), 20 deletions(-) > -- // Casey (she/her)
