* [PATCH v2 1/1] dt-bindings: ata: ahci-fsl-qoriq: add fsl,ls1046a-ahci and fsl,ls1012a-ahci @ 2024-07-12 14:29 Frank Li 2024-07-12 15:19 ` Niklas Cassel 2024-07-12 17:16 ` Rob Herring 0 siblings, 2 replies; 5+ messages in thread From: Frank Li @ 2024-07-12 14:29 UTC (permalink / raw) To: Damien Le Moal, Niklas Cassel, Rob Herring, Krzysztof Kozlowski, Conor Dooley, open list:LIBATA SUBSYSTEM (Serial and Parallel ATA drivers), open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, open list Cc: imx Add missing documented compatible strings 'fsl,ls1046a-ahci' and 'fsl,ls1012a-ahci'. Allow 'fsl,ls1012a-ahci' to fallback to 'fsl,ls1043a-ahci'. Fix below CHECK_DTB warnings arch/arm64/boot/dts/freescale/fsl-ls1012a-frwy.dtb: /soc/sata@3200000: failed to match any schema with compatible: ['fsl,ls1012a-ahci', 'fsl,ls1043a-ahci'] Signed-off-by: Frank Li <Frank.Li@nxp.com> --- Change from v1 to v2 - rework commit message to show fix CHECK_DTB warning. --- .../devicetree/bindings/ata/fsl,ahci.yaml | 19 ++++++++++++------- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/Documentation/devicetree/bindings/ata/fsl,ahci.yaml b/Documentation/devicetree/bindings/ata/fsl,ahci.yaml index 162b3bb5427ed..a244bc603549d 100644 --- a/Documentation/devicetree/bindings/ata/fsl,ahci.yaml +++ b/Documentation/devicetree/bindings/ata/fsl,ahci.yaml @@ -11,13 +11,18 @@ maintainers: properties: compatible: - enum: - - fsl,ls1021a-ahci - - fsl,ls1043a-ahci - - fsl,ls1028a-ahci - - fsl,ls1088a-ahci - - fsl,ls2080a-ahci - - fsl,lx2160a-ahci + oneOf: + - items: + - const: fsl,ls1012a-ahci + - const: fsl,ls1043a-ahci + - enum: + - fsl,ls1021a-ahci + - fsl,ls1043a-ahci + - fsl,ls1046a-ahci + - fsl,ls1028a-ahci + - fsl,ls1088a-ahci + - fsl,ls2080a-ahci + - fsl,lx2160a-ahci reg: minItems: 1 -- 2.34.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v2 1/1] dt-bindings: ata: ahci-fsl-qoriq: add fsl,ls1046a-ahci and fsl,ls1012a-ahci 2024-07-12 14:29 [PATCH v2 1/1] dt-bindings: ata: ahci-fsl-qoriq: add fsl,ls1046a-ahci and fsl,ls1012a-ahci Frank Li @ 2024-07-12 15:19 ` Niklas Cassel 2024-07-12 16:04 ` Frank Li 2024-07-12 17:16 ` Rob Herring 1 sibling, 1 reply; 5+ messages in thread From: Niklas Cassel @ 2024-07-12 15:19 UTC (permalink / raw) To: Frank Li Cc: Damien Le Moal, Rob Herring, Krzysztof Kozlowski, Conor Dooley, open list:LIBATA SUBSYSTEM (Serial and Parallel ATA drivers), open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, open list, imx On Fri, Jul 12, 2024 at 10:29:22AM -0400, Frank Li wrote: > Add missing documented compatible strings 'fsl,ls1046a-ahci' and > 'fsl,ls1012a-ahci'. Allow 'fsl,ls1012a-ahci' to fallback to > 'fsl,ls1043a-ahci'. > > Fix below CHECK_DTB warnings > arch/arm64/boot/dts/freescale/fsl-ls1012a-frwy.dtb: /soc/sata@3200000: failed to match any schema with compatible: ['fsl,ls1012a-ahci', 'fsl,ls1043a-ahci'] > > Signed-off-by: Frank Li <Frank.Li@nxp.com> > --- > Change from v1 to v2 > - rework commit message to show fix CHECK_DTB warning. > --- > .../devicetree/bindings/ata/fsl,ahci.yaml | 19 ++++++++++++------- > 1 file changed, 12 insertions(+), 7 deletions(-) > > diff --git a/Documentation/devicetree/bindings/ata/fsl,ahci.yaml b/Documentation/devicetree/bindings/ata/fsl,ahci.yaml > index 162b3bb5427ed..a244bc603549d 100644 > --- a/Documentation/devicetree/bindings/ata/fsl,ahci.yaml > +++ b/Documentation/devicetree/bindings/ata/fsl,ahci.yaml > @@ -11,13 +11,18 @@ maintainers: > > properties: > compatible: > - enum: > - - fsl,ls1021a-ahci > - - fsl,ls1043a-ahci > - - fsl,ls1028a-ahci > - - fsl,ls1088a-ahci > - - fsl,ls2080a-ahci > - - fsl,lx2160a-ahci > + oneOf: > + - items: > + - const: fsl,ls1012a-ahci > + - const: fsl,ls1043a-ahci > + - enum: > + - fsl,ls1021a-ahci > + - fsl,ls1043a-ahci > + - fsl,ls1046a-ahci > + - fsl,ls1028a-ahci > + - fsl,ls1088a-ahci > + - fsl,ls2080a-ahci > + - fsl,lx2160a-ahci I think that you should add the following Fixes-tag: Fixes: e58e12c5c34c ("dt-bindings: ata: ahci-fsl-qoriq: convert to yaml format") Considering that the commit that your are fixing is only in libata for-6.11, and has thus never been in a released kernel version, perhaps the following patch would be better (if it also solves the warnings): diff --git a/Documentation/devicetree/bindings/ata/fsl,ahci.yaml b/Documentation/devicetree/bindings/ata/fsl,ahci.yaml index 162b3bb5427e..8953b1847305 100644 --- a/Documentation/devicetree/bindings/ata/fsl,ahci.yaml +++ b/Documentation/devicetree/bindings/ata/fsl,ahci.yaml @@ -12,8 +12,10 @@ maintainers: properties: compatible: enum: + - fsl,ls1012a-ahci - fsl,ls1021a-ahci - fsl,ls1043a-ahci + - fsl,ls1046a-ahci - fsl,ls1028a-ahci - fsl,ls1088a-ahci - fsl,ls2080a-ahci This assumes that we can get the patch included before 6.11 final is released, but considering that the merge window hasn't even opened yet, that should be doable. Thoughts from DT maintainers? Kind regards, Niklas ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v2 1/1] dt-bindings: ata: ahci-fsl-qoriq: add fsl,ls1046a-ahci and fsl,ls1012a-ahci 2024-07-12 15:19 ` Niklas Cassel @ 2024-07-12 16:04 ` Frank Li 2024-07-12 17:16 ` Niklas Cassel 0 siblings, 1 reply; 5+ messages in thread From: Frank Li @ 2024-07-12 16:04 UTC (permalink / raw) To: Niklas Cassel Cc: Damien Le Moal, Rob Herring, Krzysztof Kozlowski, Conor Dooley, open list:LIBATA SUBSYSTEM (Serial and Parallel ATA drivers), open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, open list, imx On Fri, Jul 12, 2024 at 05:19:34PM +0200, Niklas Cassel wrote: > On Fri, Jul 12, 2024 at 10:29:22AM -0400, Frank Li wrote: > > Add missing documented compatible strings 'fsl,ls1046a-ahci' and > > 'fsl,ls1012a-ahci'. Allow 'fsl,ls1012a-ahci' to fallback to > > 'fsl,ls1043a-ahci'. > > > > Fix below CHECK_DTB warnings > > arch/arm64/boot/dts/freescale/fsl-ls1012a-frwy.dtb: /soc/sata@3200000: failed to match any schema with compatible: ['fsl,ls1012a-ahci', 'fsl,ls1043a-ahci'] > > > > Signed-off-by: Frank Li <Frank.Li@nxp.com> > > --- > > Change from v1 to v2 > > - rework commit message to show fix CHECK_DTB warning. > > --- > > .../devicetree/bindings/ata/fsl,ahci.yaml | 19 ++++++++++++------- > > 1 file changed, 12 insertions(+), 7 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/ata/fsl,ahci.yaml b/Documentation/devicetree/bindings/ata/fsl,ahci.yaml > > index 162b3bb5427ed..a244bc603549d 100644 > > --- a/Documentation/devicetree/bindings/ata/fsl,ahci.yaml > > +++ b/Documentation/devicetree/bindings/ata/fsl,ahci.yaml > > @@ -11,13 +11,18 @@ maintainers: > > > > properties: > > compatible: > > - enum: > > - - fsl,ls1021a-ahci > > - - fsl,ls1043a-ahci > > - - fsl,ls1028a-ahci > > - - fsl,ls1088a-ahci > > - - fsl,ls2080a-ahci > > - - fsl,lx2160a-ahci > > + oneOf: > > + - items: > > + - const: fsl,ls1012a-ahci > > + - const: fsl,ls1043a-ahci > > + - enum: > > + - fsl,ls1021a-ahci > > + - fsl,ls1043a-ahci > > + - fsl,ls1046a-ahci > > + - fsl,ls1028a-ahci > > + - fsl,ls1088a-ahci > > + - fsl,ls2080a-ahci > > + - fsl,lx2160a-ahci > > I think that you should add the following Fixes-tag: > Fixes: e58e12c5c34c ("dt-bindings: ata: ahci-fsl-qoriq: convert to yaml format") I am not sure if need it because e58e12c5c34c still not release yet. Needn't backport. You may squash into e58e12c5c34c if you like. > > Considering that the commit that your are fixing is only in libata for-6.11, > and has thus never been in a released kernel version, perhaps the following > patch would be better (if it also solves the warnings): > > diff --git a/Documentation/devicetree/bindings/ata/fsl,ahci.yaml b/Documentation/devicetree/bindings/ata/fsl,ahci.yaml > index 162b3bb5427e..8953b1847305 100644 > --- a/Documentation/devicetree/bindings/ata/fsl,ahci.yaml > +++ b/Documentation/devicetree/bindings/ata/fsl,ahci.yaml > @@ -12,8 +12,10 @@ maintainers: > properties: > compatible: > enum: > + - fsl,ls1012a-ahci > - fsl,ls1021a-ahci > - fsl,ls1043a-ahci > + - fsl,ls1046a-ahci > - fsl,ls1028a-ahci > - fsl,ls1088a-ahci > - fsl,ls2080a-ahci driver have not support "fsl,ls1012a-ahci", which have to fall back to "fsl,ls1043a-ahci". and DTS already use compatible = "fsl,ls1012a-ahci", "fsl,ls1046a-ahci". It can't fix 1012's warning. Only fix 1046's warning. DT team also don't want to remove fsl,ls1012a-ahci in dts file. > > > This assumes that we can get the patch included before 6.11 final is released, > but considering that the merge window hasn't even opened yet, that should be > doable. > > > Thoughts from DT maintainers? Supposed through ata tree. > > > > Kind regards, > Niklas ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 1/1] dt-bindings: ata: ahci-fsl-qoriq: add fsl,ls1046a-ahci and fsl,ls1012a-ahci 2024-07-12 16:04 ` Frank Li @ 2024-07-12 17:16 ` Niklas Cassel 0 siblings, 0 replies; 5+ messages in thread From: Niklas Cassel @ 2024-07-12 17:16 UTC (permalink / raw) To: Frank Li Cc: Damien Le Moal, Rob Herring, Krzysztof Kozlowski, Conor Dooley, open list:LIBATA SUBSYSTEM (Serial and Parallel ATA drivers), open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, open list, imx On Fri, Jul 12, 2024 at 12:04:02PM -0400, Frank Li wrote: > On Fri, Jul 12, 2024 at 05:19:34PM +0200, Niklas Cassel wrote: > > On Fri, Jul 12, 2024 at 10:29:22AM -0400, Frank Li wrote: > > > Add missing documented compatible strings 'fsl,ls1046a-ahci' and > > > 'fsl,ls1012a-ahci'. Allow 'fsl,ls1012a-ahci' to fallback to > > > 'fsl,ls1043a-ahci'. > > > > > > Fix below CHECK_DTB warnings > > > arch/arm64/boot/dts/freescale/fsl-ls1012a-frwy.dtb: /soc/sata@3200000: failed to match any schema with compatible: ['fsl,ls1012a-ahci', 'fsl,ls1043a-ahci'] > > > > > > Signed-off-by: Frank Li <Frank.Li@nxp.com> > > > --- > > > Change from v1 to v2 > > > - rework commit message to show fix CHECK_DTB warning. > > > --- > > > .../devicetree/bindings/ata/fsl,ahci.yaml | 19 ++++++++++++------- > > > 1 file changed, 12 insertions(+), 7 deletions(-) > > > > > > diff --git a/Documentation/devicetree/bindings/ata/fsl,ahci.yaml b/Documentation/devicetree/bindings/ata/fsl,ahci.yaml > > > index 162b3bb5427ed..a244bc603549d 100644 > > > --- a/Documentation/devicetree/bindings/ata/fsl,ahci.yaml > > > +++ b/Documentation/devicetree/bindings/ata/fsl,ahci.yaml > > > @@ -11,13 +11,18 @@ maintainers: > > > > > > properties: > > > compatible: > > > - enum: > > > - - fsl,ls1021a-ahci > > > - - fsl,ls1043a-ahci > > > - - fsl,ls1028a-ahci > > > - - fsl,ls1088a-ahci > > > - - fsl,ls2080a-ahci > > > - - fsl,lx2160a-ahci > > > + oneOf: > > > + - items: > > > + - const: fsl,ls1012a-ahci > > > + - const: fsl,ls1043a-ahci > > > + - enum: > > > + - fsl,ls1021a-ahci > > > + - fsl,ls1043a-ahci > > > + - fsl,ls1046a-ahci > > > + - fsl,ls1028a-ahci > > > + - fsl,ls1088a-ahci > > > + - fsl,ls2080a-ahci > > > + - fsl,lx2160a-ahci > > > > I think that you should add the following Fixes-tag: > > Fixes: e58e12c5c34c ("dt-bindings: ata: ahci-fsl-qoriq: convert to yaml format") > > I am not sure if need it because e58e12c5c34c still not release yet. > Needn't backport. You may squash into e58e12c5c34c if you like. Fixes tags are used to indicate that the commit Fixes another commit that introduced a bug. e58e12c5c34c introduced a bug that wasn't there before, namely a CHECK_DTB warning. Cc: stable@vger.kernel.org is used to indicate that the patch should be backported. In your case, I would use Fixes:, but without Cc: stable. Anyway, I could add the Fixes tag if we get a R-b tag from a DT maintainer. > > > > > Considering that the commit that your are fixing is only in libata for-6.11, > > and has thus never been in a released kernel version, perhaps the following > > patch would be better (if it also solves the warnings): > > > > diff --git a/Documentation/devicetree/bindings/ata/fsl,ahci.yaml b/Documentation/devicetree/bindings/ata/fsl,ahci.yaml > > index 162b3bb5427e..8953b1847305 100644 > > --- a/Documentation/devicetree/bindings/ata/fsl,ahci.yaml > > +++ b/Documentation/devicetree/bindings/ata/fsl,ahci.yaml > > @@ -12,8 +12,10 @@ maintainers: > > properties: > > compatible: > > enum: > > + - fsl,ls1012a-ahci > > - fsl,ls1021a-ahci > > - fsl,ls1043a-ahci > > + - fsl,ls1046a-ahci > > - fsl,ls1028a-ahci > > - fsl,ls1088a-ahci > > - fsl,ls2080a-ahci > > driver have not support "fsl,ls1012a-ahci", which have to fall back to > "fsl,ls1043a-ahci". and DTS already use > compatible = "fsl,ls1012a-ahci", "fsl,ls1046a-ahci". > > It can't fix 1012's warning. Only fix 1046's warning. > DT team also don't want to remove fsl,ls1012a-ahci in dts file. That is not fully true. Before my proposed patch: $ ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- make CHECK_DTBS=y freescale/fsl-ls1012a-qds.dtb arch/arm64/boot/dts/freescale/fsl-ls1012a-qds.dtb: sata@3200000: compatible:0: 'fsl,ls1012a-ahci' is not one of ['fsl,ls1021a-ahci', 'fsl,ls1043a-ahci', 'fsl,ls1028a-ahci', 'fsl,ls1088a-ahci', 'fsl,ls2080a-ahci', 'fsl,lx2160a-ahci'] from schema $id: http://devicetree.org/schemas/ata/fsl,ahci.yaml# arch/arm64/boot/dts/freescale/fsl-ls1012a-qds.dtb: sata@3200000: compatible: ['fsl,ls1012a-ahci', 'fsl,ls1043a-ahci'] is too long from schema $id: http://devicetree.org/schemas/ata/fsl,ahci.yaml# After my proposed patch: $ ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- make CHECK_DTBS=y freescale/fsl-ls1012a-qds.dtb arch/arm64/boot/dts/freescale/fsl-ls1012a-qds.dtb: sata@3200000: compatible: ['fsl,ls1012a-ahci', 'fsl,ls1043a-ahci'] is too long from schema $id: http://devicetree.org/schemas/ata/fsl,ahci.yaml# So it would only fix one of the two warnings for fsl-ls1012a platforms. I see. Perhaps you should add this additional warning to your commit message. So the patch has to look like your suggested patch. You could possible add '- description: PCIe controller in ls1012a' in front of: + - items: + - const: fsl,ls1012a-ahci + - const: fsl,ls1043a-ahci Too make it even clearer that this ugly item-list is only needed for ls1012a, see e.g.: https://github.com/torvalds/linux/blob/v6.10-rc7/Documentation/devicetree/bindings/pci/ti%2Cj721e-pci-host.yaml#L18-L21 Kind regards, Niklas ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 1/1] dt-bindings: ata: ahci-fsl-qoriq: add fsl,ls1046a-ahci and fsl,ls1012a-ahci 2024-07-12 14:29 [PATCH v2 1/1] dt-bindings: ata: ahci-fsl-qoriq: add fsl,ls1046a-ahci and fsl,ls1012a-ahci Frank Li 2024-07-12 15:19 ` Niklas Cassel @ 2024-07-12 17:16 ` Rob Herring 1 sibling, 0 replies; 5+ messages in thread From: Rob Herring @ 2024-07-12 17:16 UTC (permalink / raw) To: Frank Li Cc: Damien Le Moal, Niklas Cassel, Krzysztof Kozlowski, Conor Dooley, open list:LIBATA SUBSYSTEM (Serial and Parallel ATA drivers), open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, open list, imx On Fri, Jul 12, 2024 at 8:29 AM Frank Li <Frank.Li@nxp.com> wrote: > > Add missing documented compatible strings 'fsl,ls1046a-ahci' and > 'fsl,ls1012a-ahci'. Allow 'fsl,ls1012a-ahci' to fallback to > 'fsl,ls1043a-ahci'. > > Fix below CHECK_DTB warnings > arch/arm64/boot/dts/freescale/fsl-ls1012a-frwy.dtb: /soc/sata@3200000: failed to match any schema with compatible: ['fsl,ls1012a-ahci', 'fsl,ls1043a-ahci'] > > Signed-off-by: Frank Li <Frank.Li@nxp.com> > --- > Change from v1 to v2 > - rework commit message to show fix CHECK_DTB warning. > --- > .../devicetree/bindings/ata/fsl,ahci.yaml | 19 ++++++++++++------- > 1 file changed, 12 insertions(+), 7 deletions(-) > > diff --git a/Documentation/devicetree/bindings/ata/fsl,ahci.yaml b/Documentation/devicetree/bindings/ata/fsl,ahci.yaml > index 162b3bb5427ed..a244bc603549d 100644 > --- a/Documentation/devicetree/bindings/ata/fsl,ahci.yaml > +++ b/Documentation/devicetree/bindings/ata/fsl,ahci.yaml > @@ -11,13 +11,18 @@ maintainers: > > properties: > compatible: > - enum: > - - fsl,ls1021a-ahci > - - fsl,ls1043a-ahci > - - fsl,ls1028a-ahci > - - fsl,ls1088a-ahci > - - fsl,ls2080a-ahci > - - fsl,lx2160a-ahci > + oneOf: > + - items: > + - const: fsl,ls1012a-ahci > + - const: fsl,ls1043a-ahci > + - enum: > + - fsl,ls1021a-ahci > + - fsl,ls1043a-ahci > + - fsl,ls1046a-ahci > + - fsl,ls1028a-ahci Fix the alpha-numeric order while you're changing it. > + - fsl,ls1088a-ahci > + - fsl,ls2080a-ahci > + - fsl,lx2160a-ahci Reviewed-by: Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2024-07-12 17:16 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-07-12 14:29 [PATCH v2 1/1] dt-bindings: ata: ahci-fsl-qoriq: add fsl,ls1046a-ahci and fsl,ls1012a-ahci Frank Li 2024-07-12 15:19 ` Niklas Cassel 2024-07-12 16:04 ` Frank Li 2024-07-12 17:16 ` Niklas Cassel 2024-07-12 17:16 ` Rob Herring
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).