On Mon, Jan 13, 2025 at 3:25 PM <victor.duicu@xxxxxxxxxxxxx> wrote: > > From: Victor Duicu <victor.duicu@xxxxxxxxxxxxx> > > This patch moves ACPI_FREE in pac1921_match_acpi_device ACPI_FREE() pac1921_match_acpi_device() > in order to cover all branches. Do we need a Fixes tag? ... > + ACPI_FREE(status); > + Not sure if we need this blank line. > if (!label) > return -ENOMEM; ... Other than above, LGTM, thanks for addressing it. -- With Best Regards, Andy Shevchenko