devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2 1/1] dt-bindings: misc: fsl,qoriq-mc: remove ref for msi-parent
@ 2024-08-28 16:34 Frank Li
  2024-08-29  6:24 ` Krzysztof Kozlowski
  0 siblings, 1 reply; 4+ messages in thread
From: Frank Li @ 2024-08-28 16:34 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	open list
  Cc: imx

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(-)

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
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH v2 1/1] dt-bindings: misc: fsl,qoriq-mc: remove ref for msi-parent
  2024-08-28 16:34 [PATCH v2 1/1] dt-bindings: misc: fsl,qoriq-mc: remove ref for msi-parent Frank Li
@ 2024-08-29  6:24 ` Krzysztof Kozlowski
  2024-10-04 20:17   ` Frank Li
  0 siblings, 1 reply; 4+ messages in thread
From: Krzysztof Kozlowski @ 2024-08-29  6:24 UTC (permalink / raw)
  To: Frank Li
  Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	open list, imx

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


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v2 1/1] dt-bindings: misc: fsl,qoriq-mc: remove ref for msi-parent
  2024-08-29  6:24 ` Krzysztof Kozlowski
@ 2024-10-04 20:17   ` Frank Li
  2024-10-05  8:55     ` Krzysztof Kozlowski
  0 siblings, 1 reply; 4+ messages in thread
From: Frank Li @ 2024-10-04 20:17 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	open list, imx

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
>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v2 1/1] dt-bindings: misc: fsl,qoriq-mc: remove ref for msi-parent
  2024-10-04 20:17   ` Frank Li
@ 2024-10-05  8:55     ` Krzysztof Kozlowski
  0 siblings, 0 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2024-10-05  8:55 UTC (permalink / raw)
  To: Frank Li
  Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	open list, imx

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


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2024-10-05  8:55 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-08-28 16:34 [PATCH v2 1/1] dt-bindings: misc: fsl,qoriq-mc: remove ref for msi-parent Frank Li
2024-08-29  6:24 ` Krzysztof Kozlowski
2024-10-04 20:17   ` Frank Li
2024-10-05  8:55     ` Krzysztof Kozlowski

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).