| Message ID | 20240417032642.3178669-3-joy.zou@nxp.com |
|---|---|
| State | Not Applicable |
| Headers | show |
| Series | clean up unused "fsl,imx8qm-adma" compatible string | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/patch-applied | fail | build log |
On Wed, Apr 17, 2024 at 11:26:42AM +0800, Joy Zou wrote: > The eDMA hardware issue only exist imx8QM A0. A0 never mass production. > The compatible string "fsl,imx8qm-adma" is unused. So remove the > workaround safely. > > Signed-off-by: Joy Zou <joy.zou@nxp.com> "So remove it safely." There are no 'workaround' in bindings doc. after fix it, Reviewed-by: Frank Li <Frank.Li@nxp.com> > --- > Changes for v4: > 1. adjust the subject to keep consistent with existing patches. > > Changes for v3: > 1. modify the commit message. > 2. remove the unused compatible string "fsl,imx8qm-adma" from allOf property. > --- > Documentation/devicetree/bindings/dma/fsl,edma.yaml | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/dma/fsl,edma.yaml b/Documentation/devicetree/bindings/dma/fsl,edma.yaml > index 825f4715499e..cf97ea86a7a2 100644 > --- a/Documentation/devicetree/bindings/dma/fsl,edma.yaml > +++ b/Documentation/devicetree/bindings/dma/fsl,edma.yaml > @@ -21,7 +21,6 @@ properties: > - enum: > - fsl,vf610-edma > - fsl,imx7ulp-edma > - - fsl,imx8qm-adma > - fsl,imx8qm-edma > - fsl,imx8ulp-edma > - fsl,imx93-edma3 > @@ -92,7 +91,6 @@ allOf: > compatible: > contains: > enum: > - - fsl,imx8qm-adma > - fsl,imx8qm-edma > - fsl,imx93-edma3 > - fsl,imx93-edma4 > -- > 2.37.1 >
On Wed, 17 Apr 2024 11:26:42 +0800, Joy Zou wrote: > The eDMA hardware issue only exist imx8QM A0. A0 never mass production. > The compatible string "fsl,imx8qm-adma" is unused. So remove the > workaround safely. > > Signed-off-by: Joy Zou <joy.zou@nxp.com> > --- > Changes for v4: > 1. adjust the subject to keep consistent with existing patches. > > Changes for v3: > 1. modify the commit message. > 2. remove the unused compatible string "fsl,imx8qm-adma" from allOf property. > --- > Documentation/devicetree/bindings/dma/fsl,edma.yaml | 2 -- > 1 file changed, 2 deletions(-) > Reviewed-by: Rob Herring (Arm) <robh@kernel.org>
> -----Original Message----- > From: Frank Li <frank.li@nxp.com> > Sent: 2024年4月17日 22:45 > To: Joy Zou <joy.zou@nxp.com> > Cc: Peng Fan <peng.fan@nxp.com>; vkoul@kernel.org; robh@kernel.org; > krzk+dt@kernel.org; conor+dt@kernel.org; imx@lists.linux.dev; > dmaengine@vger.kernel.org; linux-kernel@vger.kernel.org; > devicetree@vger.kernel.org > Subject: Re: [PATCH v4 2/2] dt-bindings: fsl-dma: fsl-edma: clean up unused > "fsl,imx8qm-adma" compatible string > > On Wed, Apr 17, 2024 at 11:26:42AM +0800, Joy Zou wrote: > > The eDMA hardware issue only exist imx8QM A0. A0 never mass > production. > > The compatible string "fsl,imx8qm-adma" is unused. So remove the > > workaround safely. > > > > Signed-off-by: Joy Zou <joy.zou@nxp.com> > > "So remove it safely." > > There are no 'workaround' in bindings doc. after fix it, Thanks for your comments! Will change the commit message in next version. BR Joy Zou > > Reviewed-by: Frank Li <Frank.Li@nxp.com> > > > --- > > Changes for v4: > > 1. adjust the subject to keep consistent with existing patches. > > > > Changes for v3: > > 1. modify the commit message. > > 2. remove the unused compatible string "fsl,imx8qm-adma" from allOf > property. > > --- > > Documentation/devicetree/bindings/dma/fsl,edma.yaml | 2 -- > > 1 file changed, 2 deletions(-) > >
diff --git a/Documentation/devicetree/bindings/dma/fsl,edma.yaml b/Documentation/devicetree/bindings/dma/fsl,edma.yaml index 825f4715499e..cf97ea86a7a2 100644 --- a/Documentation/devicetree/bindings/dma/fsl,edma.yaml +++ b/Documentation/devicetree/bindings/dma/fsl,edma.yaml @@ -21,7 +21,6 @@ properties: - enum: - fsl,vf610-edma - fsl,imx7ulp-edma - - fsl,imx8qm-adma - fsl,imx8qm-edma - fsl,imx8ulp-edma - fsl,imx93-edma3 @@ -92,7 +91,6 @@ allOf: compatible: contains: enum: - - fsl,imx8qm-adma - fsl,imx8qm-edma - fsl,imx93-edma3 - fsl,imx93-edma4
The eDMA hardware issue only exist imx8QM A0. A0 never mass production. The compatible string "fsl,imx8qm-adma" is unused. So remove the workaround safely. Signed-off-by: Joy Zou <joy.zou@nxp.com> --- Changes for v4: 1. adjust the subject to keep consistent with existing patches. Changes for v3: 1. modify the commit message. 2. remove the unused compatible string "fsl,imx8qm-adma" from allOf property. --- Documentation/devicetree/bindings/dma/fsl,edma.yaml | 2 -- 1 file changed, 2 deletions(-)