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

[Jacky Huang]

On 2023/4/14 下午 03:46, Krzysztof Kozlowski wrote:
> 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

Dear Krzysztof,


Got it, I will make sure to add review tags for this patch in all 
subsequent versions.
Of course, other patches that have received review tags should also be 
included.


Best regards,
Jacky Huang