[PATCH 5/6] dt-bindings: crypto: Add bindings for Starfive crypto driver

[JiaJie Ho]

> -----Original Message-----
> From: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>
> Sent: Tuesday, December 6, 2022 4:26 PM
> To: JiaJie Ho <jiajie.ho at starfivetech.com>; Rob Herring <robh at kernel.org>
> Cc: linux-crypto at vger.kernel.org; linux-riscv at lists.infradead.org; Rob
> Herring <robh+dt at kernel.org>; Herbert Xu
> <herbert at gondor.apana.org.au>; Krzysztof Kozlowski
> <krzysztof.kozlowski+dt at linaro.org>; linux-kernel at vger.kernel.org;
> devicetree at vger.kernel.org; David S . Miller <davem at davemloft.net>
> Subject: Re: [PATCH 5/6] dt-bindings: crypto: Add bindings for Starfive crypto
> driver
> 
> On 06/12/2022 04:48, JiaJie Ho wrote:
> >
> >
> > The #include in example have dependencies on the following patches:
> > https://patchwork.kernel.org/project/linux-riscv/cover/20221118010627.
> > 70576-1-hal.feng at starfivetech.com/
> > https://patchwork.kernel.org/project/linux-riscv/cover/20221118011714.
> > 70877-1-hal.feng at starfivetech.com/
> > I've noted them in the cover letter.
> > How do I add them in this patch?
> 
> You cannot. Testing bot does not take dependencies, so it did not have
> above clock IDs. This also should point your attention that if crypto
> maintainers pick up this patch, they also won't have the clock IDs so their tree
> will have such failure as well.
> 
> You can wait with your patch till dependency hits mainline or you can just
> replace clock IDs with numbers and drop the header (and later you can
> correct the example if needed... or leave it as is).
> 
I'll replace the IDs and drop the header then.
Thanks.

Regards,
Jia Jie