| Message ID | 20240327025126.229475-2-chunhau.tan@starfivetech.com |
|---|---|
| State | Not Applicable |
| Headers | show |
| Series | Add JH8100 support for snps,dw-axi-dmac | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/patch-applied | success | |
| robh/dtbs-check | warning | build log |
| robh/dt-meta-schema | success |
On 27/03/2024 03:51, Tan Chun Hau wrote: > Add support for StarFive JH8100 SoC in Sysnopsys Designware AXI DMA > controller. > > Both JH8100 and JH7110 require reset operation in device probe. > However, JH8100 doesn't need to apply different configuration on > CH_CFG registers. 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, under or above your Signed-off-by tag. Tag is "received", when provided in a message replied to you on the mailing list. 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 tags received on the version they apply. https://elixir.bootlin.com/linux/v6.5-rc3/source/Documentation/process/submitting-patches.rst#L577 If a tag was not added on purpose, please state why and what changed. Best regards, Krzysztof
On Tue, Mar 26, 2024 at 07:51:25PM -0700, Tan Chun Hau wrote: > Add support for StarFive JH8100 SoC in Sysnopsys Designware AXI DMA > controller. > > Both JH8100 and JH7110 require reset operation in device probe. > However, JH8100 doesn't need to apply different configuration on > CH_CFG registers. > > Signed-off-by: Tan Chun Hau <chunhau.tan@starfivetech.com> Acked-by: Conor Dooley <conor.dooley@microchip.com> Thanks, Conor.
> -----Original Message----- > From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > Sent: Wednesday, 27 March, 2024 3:50 PM > To: ChunHau Tan <chunhau.tan@starfivetech.com>; Eugeniy Paltsev > <Eugeniy.Paltsev@synopsys.com>; Vinod Koul <vkoul@kernel.org>; Rob Herring > <robh@kernel.org>; Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>; > Conor Dooley <conor+dt@kernel.org> > Cc: Leyfoon Tan <leyfoon.tan@starfivetech.com>; JeeHeng Sia > <jeeheng.sia@starfivetech.com>; dmaengine@vger.kernel.org; > devicetree@vger.kernel.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH v2 1/2] dt-bindings: dma: snps,dw-axi-dmac: Add JH8100 > support > > On 27/03/2024 03:51, Tan Chun Hau wrote: > > Add support for StarFive JH8100 SoC in Sysnopsys Designware AXI DMA > > controller. > > > > Both JH8100 and JH7110 require reset operation in device probe. > > However, JH8100 doesn't need to apply different configuration on > > CH_CFG registers. > > 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, > under or above your Signed-off-by tag. Tag is "received", when provided in a > message replied to you on the mailing list. 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 tags received on the version they apply. > > https://elixir.bootlin.com/linux/v6.5-rc3/source/Documentation/process/submitt > ing-patches.rst#L577 > > If a tag was not added on purpose, please state why and what changed. Hi Krzysztof, thank you very much for the feedback, Sorry I overlooked it. Do you prefer I resend V2 patch or send a V3 patch to include the Acked-by ? Thank you. > > Best regards, > Krzysztof
On Wed, Mar 27, 2024 at 08:24:01AM +0000, ChunHau Tan wrote: > > > > -----Original Message----- > > From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > Sent: Wednesday, 27 March, 2024 3:50 PM > > To: ChunHau Tan <chunhau.tan@starfivetech.com>; Eugeniy Paltsev > > <Eugeniy.Paltsev@synopsys.com>; Vinod Koul <vkoul@kernel.org>; Rob Herring > > <robh@kernel.org>; Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>; > > Conor Dooley <conor+dt@kernel.org> > > Cc: Leyfoon Tan <leyfoon.tan@starfivetech.com>; JeeHeng Sia > > <jeeheng.sia@starfivetech.com>; dmaengine@vger.kernel.org; > > devicetree@vger.kernel.org; linux-kernel@vger.kernel.org > > Subject: Re: [PATCH v2 1/2] dt-bindings: dma: snps,dw-axi-dmac: Add JH8100 > > support > > > > On 27/03/2024 03:51, Tan Chun Hau wrote: > > > Add support for StarFive JH8100 SoC in Sysnopsys Designware AXI DMA > > > controller. > > > > > > Both JH8100 and JH7110 require reset operation in device probe. > > > However, JH8100 doesn't need to apply different configuration on > > > CH_CFG registers. > > > > 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, > > under or above your Signed-off-by tag. Tag is "received", when provided in a > > message replied to you on the mailing list. 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 tags received on the version they apply. > > > > https://elixir.bootlin.com/linux/v6.5-rc3/source/Documentation/process/submitt > > ing-patches.rst#L577 > > > > If a tag was not added on purpose, please state why and what changed. > > Hi Krzysztof, thank you very much for the feedback, > Sorry I overlooked it. > Do you prefer I resend V2 patch or send a V3 patch to include the Acked-by ? I resent the tag, you do not need to do anything. Thanks, Conor.
diff --git a/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml b/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml index 363cf8bd150d..525f5f3932f5 100644 --- a/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml +++ b/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml @@ -21,6 +21,7 @@ properties: - snps,axi-dma-1.01a - intel,kmb-axi-dma - starfive,jh7110-axi-dma + - starfive,jh8100-axi-dma reg: minItems: 1
Add support for StarFive JH8100 SoC in Sysnopsys Designware AXI DMA controller. Both JH8100 and JH7110 require reset operation in device probe. However, JH8100 doesn't need to apply different configuration on CH_CFG registers. Signed-off-by: Tan Chun Hau <chunhau.tan@starfivetech.com> --- Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml | 1 + 1 file changed, 1 insertion(+)