Re: [PATCH v7 04/12] dt-bindings: reset: nuvoton: Document ma35d1 reset control

From: Krzysztof Kozlowski
Date: Fri Apr 14 2023 - 03:47:01 EST


On 14/04/2023 02:55, Jacky Huang wrote:
> Dear Krzysztof,
>
>
> On 2023/4/14 上午 12:58, Krzysztof Kozlowski wrote:
>> On 12/04/2023 07:38, Jacky Huang wrote:
>>> From: Jacky Huang <ychuang3@xxxxxxxxxxx>
>>>
>>> Add the dt-bindings header for Nuvoton ma35d1, that gets shared
>>> between the reset controller and reset references in the dts.
>>> Add documentation to describe nuvoton ma35d1 reset driver.
>>>
>>> Signed-off-by: Jacky Huang <ychuang3@xxxxxxxxxxx>
>>
>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
>>
>>
>> This is an automated instruction, just in case, because many review tags
>> are being ignored. If you do not know the process, here is a short
>> explanation:
>>
>> Please add Acked-by/Reviewed-by/Tested-by tags when posting new
>> versions, under or above your Signed-off-by tag. Tools like b4 can help
>> here. However, there's no need to repost patches *only* to add the tags.
>> The upstream maintainer will do that for acks received on the version
>> they apply.
>>
>> https://elixir.bootlin.com/linux/v5.17/source/Documentation/process/submitting-patches.rst#L540
>>
>> Best regards,
>> Krzysztof
>>
>
> Thank you for your explanation. I was not aware of the rules for adding
> tags, and it was my mistake.
> I thought that it was necessary for the reviewer to proactively add the
> "reviewed-by" tag to the patch

I proactively added the tag last time. You expect me to review it every
time and every time add my tag? You know, we all have better things to do...

Best regards,
Krzysztof