[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <nycvar.YFH.7.76.2310042058000.3534@cbobk.fhfr.pm>
Date: Wed, 4 Oct 2023 20:58:31 +0200 (CEST)
From: Jiri Kosina <jikos@...nel.org>
To: Rahul Rameshbabu <rrameshbabu@...dia.com>
cc: Benjamin Tissoires <benjamin.tissoires@...hat.com>,
Christophe JAILLET <christophe.jaillet@...adoo.fr>,
kernel-janitors@...r.kernel.org, linux-input@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 0/3] HID: nvidia-shield: Fix the error handling path
of shield_probe()
On Mon, 18 Sep 2023, Rahul Rameshbabu wrote:
> This series fixes some missing clean-up function calls in the error handling of
> the probe.
>
> Patch 1 and 2 fix some similar issues introduced in 2 different commits (hence 2
> patches)
>
> Patch 3 is an enhancement that creates a common function for cleaning up
> thunderstrike instances.
>
> Changes:
>
> v1->v2:
> - Add the LED_RETAIN_AT_SHUTDOWN flag to prevent
> led_classdev_unregister from trying to set the LED to off before a
> successful call to hid_hw_start.
> - Rename err_haptics label to err_ts_create to make the label name more
> accurate.
> - Re-order operations in thunderstrike_destroy to be in LIFO order with
> regards to the operations in thunderstrike_create.
> v2->v3:
> - Refactor thunderstrike_destroy to take a thunderstrike instance
> pointer as a parameter and prevent a variable from being unused
> in shield_probe.
>
> Link: https://lore.kernel.org/linux-input/cover.1693070958.git.christophe.jaillet@wanadoo.fr/
> Link: https://lore.kernel.org/linux-input/20230918041345.59859-1-rrameshbabu@nvidia.com/
>
> Notes from Rahul:
> - Thank you so much Christophe for these patches.
> - Sent v2 without accounting for the fact that thunderstrike_destroy in v1
> makes the thunderstrike instance in shield_probe unused. Tested v3 with W=1.
>
> Christophe JAILLET (3):
> HID: nvidia-shield: Fix a missing led_classdev_unregister() in the
> probe error handling path
> HID: nvidia-shield: Fix some missing function calls() in the probe
> error handling path
> HID: nvidia-shield: Introduce thunderstrike_destroy()
I have applied 1/3 and 2/3 to for-6.6/upstream-fixes and 3/3 to
for-6.7/nvidia-shield.
Thanks,
--
Jiri Kosina
SUSE Labs
Powered by blists - more mailing lists