Re: [PATCH v4 1/2] dt-bindings: clock: add video clock indices for Amlogic S4 SoC

[Chuan Liu]

Hi Krzysztof & Conor:


On 9/9/2025 3:40 PM, Krzysztof Kozlowski wrote:
> [ EXTERNAL EMAIL ]
>
> On 09/09/2025 09:29, Chuan Liu via B4 Relay wrote:
> > From: Chuan Liu <chuan.liu@xxxxxxxxxxx>
> >
> > Add indices for video encoder, demodulator and CVBS clocks.
> >
> > Acked-by: Rob Herring (Arm) <robh@xxxxxxxxxx>
> > Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> What?
>
> You received one tag at v1, you ignored it.
> Then you received another at v2, you ignored it.
>
> You added these now... but:
> Then third maintainer acked it THIRD DAMN TIME and you ignored it.
>
> Three maintainers had to look at this little patch and yet you still
> cannot collect the tags. You waste our time.


My apologies for the unnecessary trouble this has caused. This was
indeed a rather basic oversight...

A few days ago, I used the command 'b4 trailers -u' which automatically
added your and Rob's tags. I then failed to notice that Conor had also
provided an 'Acked-by'.


Hi Conor:
        My apologies for missing your 'Acked-by' tag again. Do you think
it's necessary for me to send another revision to add it?  I'm concerned
about taking up more of yours time for what seems like a minor issue,
especially since the previous versions have already taken up yours time.


I'm truly sorry for this mistake.


> Best regards,
> Krzysztof