* Re: [PATCH V8 1/4] dt-bindings: clock: document Amlogic S4 SoC PLL clock controller [not found] ` <8c6ad0a9-7820-c7a2-bd3b-1eee87d96728@amlogic.com> @ 2023-05-15 10:04 ` Krzysztof Kozlowski [not found] ` <33f5ff59-db7b-7576-64cb-972c0dfb0f7b@amlogic.com> 1 sibling, 0 replies; 3+ messages in thread From: Krzysztof Kozlowski @ 2023-05-15 10:04 UTC (permalink / raw) To: Yu Tu, linux-clk, linux-arm-kernel, linux-amlogic, linux-kernel, devicetree, Rob Herring, Neil Armstrong, Jerome Brunet, Kevin Hilman, Michael Turquette, Stephen Boyd, Krzysztof Kozlowski, Conor Dooley, Martin Blumenstingl Cc: kelvin.zhang, qi.duan On 15/05/2023 09:35, Yu Tu wrote: > Hi Krzysztof, > Thank you for your prompt reply. > > On 2023/5/15 14:32, Krzysztof Kozlowski wrote: >> On 15/05/2023 05:15, Yu Tu wrote: >>> Add the S4 PLL clock controller dt-bindings in the s4 SoC family. >>> >>> Signed-off-by: Yu Tu <yu.tu@amlogic.com> >> >> This is a friendly reminder during the review process. >> >> It looks like you received a tag and forgot to add it. >> >> 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. 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 >> >> If a tag was not added on purpose, please state why and what changed. > > Yes. I don't know the process. So I need to add Reviewed-by: Rob Herring > <robh@kernel.org>. And resend V8? Yes. I cannot add it top your emails. Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 3+ messages in thread
[parent not found: <33f5ff59-db7b-7576-64cb-972c0dfb0f7b@amlogic.com>]
* Re: [PATCH V8 1/4] dt-bindings: clock: document Amlogic S4 SoC PLL clock controller [not found] ` <33f5ff59-db7b-7576-64cb-972c0dfb0f7b@amlogic.com> @ 2023-05-15 10:06 ` Krzysztof Kozlowski 2023-05-15 10:12 ` Yu Tu 0 siblings, 1 reply; 3+ messages in thread From: Krzysztof Kozlowski @ 2023-05-15 10:06 UTC (permalink / raw) To: Yu Tu, linux-clk, linux-arm-kernel, linux-amlogic, linux-kernel, devicetree, Rob Herring, Neil Armstrong, Jerome Brunet, Kevin Hilman, Michael Turquette, Stephen Boyd, Krzysztof Kozlowski, Conor Dooley, Martin Blumenstingl Cc: kelvin.zhang, qi.duan On 15/05/2023 09:57, Yu Tu wrote: > > > On 2023/5/15 15:35, Yu Tu wrote: >> Hi Krzysztof, >> Thank you for your prompt reply. >> >> On 2023/5/15 14:32, Krzysztof Kozlowski wrote: >>> On 15/05/2023 05:15, Yu Tu wrote: >>>> Add the S4 PLL clock controller dt-bindings in the s4 SoC family. >>>> >>>> Signed-off-by: Yu Tu <yu.tu@amlogic.com> >>> >>> This is a friendly reminder during the review process. >>> >>> It looks like you received a tag and forgot to add it. >>> >>> 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. 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 >>> >>> If a tag was not added on purpose, please state why and what changed. >> >> Yes. I don't know the process. So I need to add Reviewed-by: Rob Herring >> <robh@kernel.org>. And resend V8? >> > > I would like to ask you again by the way. I'm not sure if I can just add > the TAG. Because I actually changed the V8. Your changelog in cover letter does not describe it. It only mentions vaguely "change patch series". Describe exactly what changed. Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH V8 1/4] dt-bindings: clock: document Amlogic S4 SoC PLL clock controller 2023-05-15 10:06 ` Krzysztof Kozlowski @ 2023-05-15 10:12 ` Yu Tu 0 siblings, 0 replies; 3+ messages in thread From: Yu Tu @ 2023-05-15 10:12 UTC (permalink / raw) To: Krzysztof Kozlowski, linux-clk, linux-arm-kernel, linux-amlogic, linux-kernel, devicetree, Rob Herring, Neil Armstrong, Jerome Brunet, Kevin Hilman, Michael Turquette, Stephen Boyd, Krzysztof Kozlowski, Conor Dooley, Martin Blumenstingl Cc: kelvin.zhang, qi.duan On 2023/5/15 18:06, Krzysztof Kozlowski wrote: > [ EXTERNAL EMAIL ] > > On 15/05/2023 09:57, Yu Tu wrote: >> >> >> On 2023/5/15 15:35, Yu Tu wrote: >>> Hi Krzysztof, >>> Thank you for your prompt reply. >>> >>> On 2023/5/15 14:32, Krzysztof Kozlowski wrote: >>>> On 15/05/2023 05:15, Yu Tu wrote: >>>>> Add the S4 PLL clock controller dt-bindings in the s4 SoC family. >>>>> >>>>> Signed-off-by: Yu Tu <yu.tu@amlogic.com> >>>> >>>> This is a friendly reminder during the review process. >>>> >>>> It looks like you received a tag and forgot to add it. >>>> >>>> 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. 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 >>>> >>>> If a tag was not added on purpose, please state why and what changed. >>> >>> Yes. I don't know the process. So I need to add Reviewed-by: Rob Herring >>> <robh@kernel.org>. And resend V8? >>> >> >> I would like to ask you again by the way. I'm not sure if I can just add >> the TAG. Because I actually changed the V8. > > Your changelog in cover letter does not describe it. It only mentions > vaguely "change patch series". Describe exactly what changed. Okay. I will correct and resend this patch. > > Best regards, > Krzysztof > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-05-15 10:14 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20230515031557.31143-1-yu.tu@amlogic.com>
[not found] ` <20230515031557.31143-2-yu.tu@amlogic.com>
[not found] ` <590560c9-4da6-bbd4-6aac-de57ab5403ba@linaro.org>
[not found] ` <8c6ad0a9-7820-c7a2-bd3b-1eee87d96728@amlogic.com>
2023-05-15 10:04 ` [PATCH V8 1/4] dt-bindings: clock: document Amlogic S4 SoC PLL clock controller Krzysztof Kozlowski
[not found] ` <33f5ff59-db7b-7576-64cb-972c0dfb0f7b@amlogic.com>
2023-05-15 10:06 ` Krzysztof Kozlowski
2023-05-15 10:12 ` Yu Tu
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).