mbox series

[platform,0/3] platform/x86: Add fixes and amendments for init()/exit() flows

Message ID 20231005075616.42777-1-vadimp@nvidia.com (mailing list archive)
Headers show
Series platform/x86: Add fixes and amendments for init()/exit() flows | expand

Message

Vadim Pasternak Oct. 5, 2023, 7:56 a.m. UTC 
The patchset:
- Fixes memory leak in error handling flow.
- Add cosmetic changes - misspelling fix, better naming/

Patches #1: Fixes memory leak issue.
Patch #2: Fix misspelling.
Patches #3: Renames few routines for better naming convention.

Vadim Pasternak (3):
  platform: mellanox: Fix a resource leak in an error handling path in
    probing flow
  platform: mellanox: Fix misspelling error in routine name
  platform: mellanox: Rename some init()/exit() functions for consistent
    naming

 drivers/platform/x86/mlx-platform.c | 29 +++++++++++++++--------------
 1 file changed, 15 insertions(+), 14 deletions(-)

Comments

Ilpo Järvinen Oct. 6, 2023, 2:55 p.m. UTC | #1 
On Thu, 5 Oct 2023, Vadim Pasternak wrote:

> The patchset:
> - Fixes memory leak in error handling flow.
> - Add cosmetic changes - misspelling fix, better naming/
> 
> Patches #1: Fixes memory leak issue.
> Patch #2: Fix misspelling.
> Patches #3: Renames few routines for better naming convention.
> 
> Vadim Pasternak (3):
>   platform: mellanox: Fix a resource leak in an error handling path in
>     probing flow
>   platform: mellanox: Fix misspelling error in routine name
>   platform: mellanox: Rename some init()/exit() functions for consistent
>     naming

Thanks for the patches. I've applied these into 
platform-drivers-x86-mellanox-init branch and merged that into 
review-ilpo. (I decided to reorder the patches 2 and 3 though as it felt 
more logical ordering.)