Re: [PATCH V10 1/4] dt-bindings: clock: document Amlogic S4 SoC PLL clock controller

From: Krzysztof Kozlowski
Date: Wed Aug 23 2023 - 01:44:46 EST


On 23/08/2023 04:24, Yu Tu wrote:
>
>
> On 2023/8/23 0:32, Krzysztof Kozlowski wrote:
>> [ EXTERNAL EMAIL ]
>>> On 22/08/2023 10:27, Yu Tu wrote:
>>> Add the S4 PLL clock controller dt-bindings in the S4 SoC family.
>>>
>>> Signed-off-by: Yu Tu <yu.tu@xxxxxxxxxxx>
>>
>> Lovely. I sent youa friendly reminder at v8 which turns our you
>> ignored. You keep ignoring, I will start ignoring as well from now on.
>
> Hi Krzysztof,
>
> Sorry. I did not forget the friendly reminder in v8, I consulted you for
> this at the time, so I re-sent V9 after adding the tag. Because it was
> just "meson" that was removed. But V10 is based on Neil's patch, which I
> think is a bit of a change. So I didn't dare add it. Instead of
> forgetting your reminder.
>
> So what should I do, I'll follow your advice exactly.

I don't think there was much difference between the version, which
received the review, and the current patch. Otherwise your changelog
should say that you drop Rb tag. Rebasing of something somewhere is not
the reason, so please describe what are the differences in the patch?

Best regards,
Krzysztof