Re: [PATCH v4 4/4] HID: wiimote: wiimote_probe with scoped cleanup

["Rafael Passos" <rafael@rcpassos.me>]

Hi David,

On Sun Sep 20, 2026 at 5:21 AM -03, David Rheinsberg wrote:
> Hi
>
> On Mon, Aug 17, 2026, at 11:38 PM, Rafael Passos wrote:
>> Use the safer scoped cleanup with a single destroy function.
>> A new bitmask was introduced to track probing state.
>> This is needed because the hid_hw calls cannot be made with null.
>>
>> A few other functions are safe to call without checking.
>> These cases are annotated with comments above them.
>>
>> Also, a new debugfs entry was added tracking this new state (bitmask).
>>
>> Signed-off-by: Rafael Passos <rafael@rcpassos.me>
>
> I am really not sold on this. This does not make the code any simpler, does it? IMO, the goto-paths are much easier to read than tracking the state at runtime.
>
> Do you think this makes the code easier to understand? Am I off here?

It does not make the code simpler, indeed.
For me, it looks easier to understand the possible states
the driver can be in during startup.

Do you have any suggestions on how to make this simpler with scoped cleanup?
Otherwise, I have no problems dropping it. (It was still fun trying it out)
The rest of the patches are ready to be re-sent with your reviews.

Thanks,
Rafael Passos