Messages in this thread | | | Date | Sat, 9 Mar 2024 13:11:10 +0100 | Subject | Re: [PATCH 01/11] dt-bindings: qspi: cdns,qspi-nor: add mobileye,eyeq5-ospi compatible | From | Krzysztof Kozlowski <> |
| |
On 08/03/2024 18:18, Théo Lebrun wrote: > Add Mobileye EyeQ5 compatible. > FIFO depth shall not be passed; hardware can detect it. >
Please use subject prefixes matching the subsystem. You can get them for example with `git log --oneline -- DIRECTORY_OR_FILE` on the directory your patch is touching. There is no subsystem as "qspi". Also order of tags is incorrect.
> Signed-off-by: Théo Lebrun <theo.lebrun@bootlin.com> > --- > Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml | 13 ++++++++++++- > 1 file changed, 12 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml b/Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml > index cca81f89e252..cf7abf48673a 100644 > --- a/Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml > +++ b/Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml > @@ -61,6 +61,17 @@ allOf: > cdns,fifo-depth: > enum: [ 128, 256 ] > default: 128 > + - if: > + properties: > + compatible: > + contains: > + const: mobileye,eyeq5-ospi > + then: > + properties: > + cdns,fifo-depth: false > + else: > + required: > + - cdns,fifo-depth > > properties: > compatible: > @@ -74,6 +85,7 @@ properties: > - xlnx,versal-ospi-1.0 > - intel,socfpga-qspi > - starfive,jh7110-qspi > + - mobileye,eyeq5-ospi
So everyone adds at the end...
Please put it after amd. With subject fixed and above:
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards, Krzysztof
| |