Re: [v6] usb-storage: alauda: Check whether the media is initialized

From: Markus Elfring
Date: Sun May 26 2024 - 05:20:52 EST


>> …
>>> Fixes: e80b0fade09e ("[PATCH] USB Storage: add alauda support")
>>
>> How do you think about to omit the text “[PATCH] ” from the tag summary?
>
> Then it would be incorrect.

I find this view interesting.


> Again, Markus, please STOP sending review comments that are obviously wrong.

The mentioned tag needs an “one line summary”.
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.9#n145

Do you find it required to repeat a questionable commit subject completely?


> New developers do not know who to ignore and you are telling
> them things that are wrong and not helpful at all.

Additional development views can occasionally become helpful.


> Please stop reviewing patches,

Patch review is one of the usual software development activities.


> this is not ok

I suggest to reconsider such a view once more.


> is actually harmful to our community.

Possible improvements are varying between affected software components.

Regards,
Markus