Re: [PATCH] USB: core: add 'shutdown' callback to usb_driver

From: Aditya Garg
Date: Sat Jul 06 2024 - 05:04:26 EST




> On 6 Jul 2024, at 2:32 PM, gregkh@xxxxxxxxxxxxxxxxxxx wrote:
>
> On Sat, Jul 06, 2024 at 08:56:05AM +0000, Aditya Garg wrote:
>>
>> Hi Greg
>>
>>>> On 6 Jul 2024, at 1:43 PM, gregkh@xxxxxxxxxxxxxxxxxxx wrote:
>>>
>>> On Fri, Jul 05, 2024 at 11:21:06AM +0000, Aditya Garg wrote:
>>>> From: Kerem Karabay <kekrby@xxxxxxxxx>
>>>>
>>>> This simplifies running code on shutdown for USB drivers.
>>>>
>>>
>>> Sorry, but this does not explain why this is needed at all :(
>>>
>>> Where did this change come from? What problem does it solve? Why
>>> should we take it?
>>>
>>
>> Currently there is no standardized method for USB drivers to handle
>> shutdown events. This patch simplifies running code on shutdown for USB
>> devices by adding a shutdown callback to usb_driver. It also implements the
>> new method to existing "USB Attached SCSI” driver that required shutdown
>>
>> Is this good enough?
>
> It's a good start, yes.
>
> But as you say "also" that means you should split this up into at least
> 2 changes, right?

Patch 1 with changes to hid core
Patch 2 implementing the change to uas

Right?
>
>>> I think I know the answers to these questions, but you need to document
>>> it here as to why it is needed (please read the kernel documentation for
>>> how to write a good changelog text and subject line.)
>>
>> The subject looks fine to me. If you think it can be improved, suggestions shall be appreciated.
>
> When you split the patch up, the subject lines will get better.
>
> thanks,
>
> greg k-h