diff mbox series

[v2,1/1] dt-bindings: misc: fsl,qoriq-mc: remove ref for msi-parent

Message ID 20240828163417.1130252-1-Frank.Li@nxp.com (mailing list archive)
State Superseded
Headers show
Series [v2,1/1] dt-bindings: misc: fsl,qoriq-mc: remove ref for msi-parent | expand

Commit Message

Frank Li Aug. 28, 2024, 4:34 p.m. UTC
msi-parent is standard property. Needn't ref to phandle. Add maxItems: 1
for it.

Fix below warning:
  arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dtb: fsl-mc@80c000000: msi-parent:0: [16, 0] is too long

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Change from v1 to v2
- add maxItems: 1 for msi-parent
---
 Documentation/devicetree/bindings/misc/fsl,qoriq-mc.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Krzysztof Kozlowski Aug. 29, 2024, 6:24 a.m. UTC | #1
On Wed, Aug 28, 2024 at 12:34:13PM -0400, Frank Li wrote:
> msi-parent is standard property. Needn't ref to phandle. Add maxItems: 1
> for it.
> 
> Fix below warning:
>   arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dtb: fsl-mc@80c000000: msi-parent:0: [16, 0] is too long
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>
> ---
> Change from v1 to v2
> - add maxItems: 1 for msi-parent

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof
Frank Li Oct. 4, 2024, 8:17 p.m. UTC | #2
On Thu, Aug 29, 2024 at 08:24:26AM +0200, Krzysztof Kozlowski wrote:
> On Wed, Aug 28, 2024 at 12:34:13PM -0400, Frank Li wrote:
> > msi-parent is standard property. Needn't ref to phandle. Add maxItems: 1
> > for it.
> >
> > Fix below warning:
> >   arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dtb: fsl-mc@80c000000: msi-parent:0: [16, 0] is too long
> >
> > Signed-off-by: Frank Li <Frank.Li@nxp.com>
> > ---
> > Change from v1 to v2
> > - add maxItems: 1 for msi-parent
>
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Anyone pick this patch?

Frank
>
> Best regards,
> Krzysztof
>
Krzysztof Kozlowski Oct. 5, 2024, 8:55 a.m. UTC | #3
On 04/10/2024 22:17, Frank Li wrote:
> On Thu, Aug 29, 2024 at 08:24:26AM +0200, Krzysztof Kozlowski wrote:
>> On Wed, Aug 28, 2024 at 12:34:13PM -0400, Frank Li wrote:
>>> msi-parent is standard property. Needn't ref to phandle. Add maxItems: 1
>>> for it.
>>>
>>> Fix below warning:
>>>   arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dtb: fsl-mc@80c000000: msi-parent:0: [16, 0] is too long
>>>
>>> Signed-off-by: Frank Li <Frank.Li@nxp.com>
>>> ---
>>> Change from v1 to v2
>>> - add maxItems: 1 for msi-parent
>>
>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> Anyone pick this patch?

Your maintainer, if you cared to Cc him?

If you do not send patch to people, how they are supposed to pick it up?

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/misc/fsl,qoriq-mc.yaml b/Documentation/devicetree/bindings/misc/fsl,qoriq-mc.yaml
index 01b00d89a9921..df45ff56d4445 100644
--- a/Documentation/devicetree/bindings/misc/fsl,qoriq-mc.yaml
+++ b/Documentation/devicetree/bindings/misc/fsl,qoriq-mc.yaml
@@ -113,7 +113,7 @@  properties:
 
   msi-parent:
     deprecated: true
-    $ref: /schemas/types.yaml#/definitions/phandle
+    maxItems: 1
     description:
       Describes the MSI controller node handling message
       interrupts for the MC. When there is no translation