* [PATCH 1/1] dt-bindings: soc: imx: add fsl,aipi-bus and fsl,emi-bus
@ 2026-02-11 22:19 Frank Li
2026-02-12 11:59 ` Krzysztof Kozlowski
0 siblings, 1 reply; 4+ messages in thread
From: Frank Li @ 2026-02-11 22:19 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
open list
Cc: imx
Add fsl,aipi-bus and fsl,emi-bus for i.MX1, i.MX2*.
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml b/Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml
index 00bbde203f598..4808065fc9115 100644
--- a/Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml
+++ b/Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml
@@ -26,8 +26,10 @@ select:
compatible:
contains:
enum:
+ - fsl,aipi-bus
- fsl,aips
- fsl,emi
+ - fsl,emi-bus
- fsl,spba-bus
required:
- compatible
@@ -39,8 +41,10 @@ properties:
compatible:
items:
- enum:
+ - fsl,aipi-bus
- fsl,aips
- fsl,emi
+ - fsl,emi-bus
- fsl,spba-bus
- const: simple-bus
--
2.43.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 1/1] dt-bindings: soc: imx: add fsl,aipi-bus and fsl,emi-bus
2026-02-11 22:19 [PATCH 1/1] dt-bindings: soc: imx: add fsl,aipi-bus and fsl,emi-bus Frank Li
@ 2026-02-12 11:59 ` Krzysztof Kozlowski
2026-02-12 15:20 ` Frank Li
0 siblings, 1 reply; 4+ messages in thread
From: Krzysztof Kozlowski @ 2026-02-12 11:59 UTC (permalink / raw)
To: Frank Li
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
open list, imx
On Wed, Feb 11, 2026 at 05:19:35PM -0500, Frank Li wrote:
> Add fsl,aipi-bus and fsl,emi-bus for i.MX1, i.MX2*.
>
> Signed-off-by: Frank Li <Frank.Li@nxp.com>
> ---
> Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml b/Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml
> index 00bbde203f598..4808065fc9115 100644
> --- a/Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml
> +++ b/Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml
> @@ -26,8 +26,10 @@ select:
> compatible:
> contains:
> enum:
> + - fsl,aipi-bus
> - fsl,aips
> - fsl,emi
> + - fsl,emi-bus
> - fsl,spba-bus
> required:
> - compatible
> @@ -39,8 +41,10 @@ properties:
> compatible:
> items:
> - enum:
> + - fsl,aipi-bus
> - fsl,aips
> - fsl,emi
> + - fsl,emi-bus
Why exactly? There is already emi, although not in my next, so don't do
it per one compatible.
https://lore.kernel.org/all/20260201011913.2419626-1-Frank.Li@nxp.com/
> - fsl,spba-bus
> - const: simple-bus
>
> --
> 2.43.0
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 1/1] dt-bindings: soc: imx: add fsl,aipi-bus and fsl,emi-bus
2026-02-12 11:59 ` Krzysztof Kozlowski
@ 2026-02-12 15:20 ` Frank Li
2026-02-13 14:54 ` Krzysztof Kozlowski
0 siblings, 1 reply; 4+ messages in thread
From: Frank Li @ 2026-02-12 15:20 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
open list, imx
On Thu, Feb 12, 2026 at 12:59:45PM +0100, Krzysztof Kozlowski wrote:
> On Wed, Feb 11, 2026 at 05:19:35PM -0500, Frank Li wrote:
> > Add fsl,aipi-bus and fsl,emi-bus for i.MX1, i.MX2*.
> >
> > Signed-off-by: Frank Li <Frank.Li@nxp.com>
> > ---
> > Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml b/Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml
> > index 00bbde203f598..4808065fc9115 100644
> > --- a/Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml
> > +++ b/Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml
> > @@ -26,8 +26,10 @@ select:
> > compatible:
> > contains:
> > enum:
> > + - fsl,aipi-bus
> > - fsl,aips
> > - fsl,emi
> > + - fsl,emi-bus
> > - fsl,spba-bus
> > required:
> > - compatible
> > @@ -39,8 +41,10 @@ properties:
> > compatible:
> > items:
> > - enum:
> > + - fsl,aipi-bus
> > - fsl,aips
> > - fsl,emi
> > + - fsl,emi-bus
>
> Why exactly? There is already emi, although not in my next, so don't do
> it per one compatible.
>
> https://lore.kernel.org/all/20260201011913.2419626-1-Frank.Li@nxp.com/
It is really old chips. some mx3/5 use fsl,emi. but imx2 use fsl,emi-bus.
it is not used in current kernel source. Consider it is really old, over
20 years, Just allow it to avoid CHECK_DTBS warning.
Frank
>
> > - fsl,spba-bus
> > - const: simple-bus
> >
> > --
> > 2.43.0
> >
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 1/1] dt-bindings: soc: imx: add fsl,aipi-bus and fsl,emi-bus
2026-02-12 15:20 ` Frank Li
@ 2026-02-13 14:54 ` Krzysztof Kozlowski
0 siblings, 0 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2026-02-13 14:54 UTC (permalink / raw)
To: Frank Li
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
open list, imx
On 12/02/2026 16:20, Frank Li wrote:
> On Thu, Feb 12, 2026 at 12:59:45PM +0100, Krzysztof Kozlowski wrote:
>> On Wed, Feb 11, 2026 at 05:19:35PM -0500, Frank Li wrote:
>>> Add fsl,aipi-bus and fsl,emi-bus for i.MX1, i.MX2*.
>>>
>>> Signed-off-by: Frank Li <Frank.Li@nxp.com>
>>> ---
>>> Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml | 4 ++++
>>> 1 file changed, 4 insertions(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml b/Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml
>>> index 00bbde203f598..4808065fc9115 100644
>>> --- a/Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml
>>> +++ b/Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml
>>> @@ -26,8 +26,10 @@ select:
>>> compatible:
>>> contains:
>>> enum:
>>> + - fsl,aipi-bus
>>> - fsl,aips
>>> - fsl,emi
>>> + - fsl,emi-bus
>>> - fsl,spba-bus
>>> required:
>>> - compatible
>>> @@ -39,8 +41,10 @@ properties:
>>> compatible:
>>> items:
>>> - enum:
>>> + - fsl,aipi-bus
>>> - fsl,aips
>>> - fsl,emi
>>> + - fsl,emi-bus
>>
>> Why exactly? There is already emi, although not in my next, so don't do
>> it per one compatible.
>>
>> https://lore.kernel.org/all/20260201011913.2419626-1-Frank.Li@nxp.com/
>
> It is really old chips. some mx3/5 use fsl,emi. but imx2 use fsl,emi-bus.
> it is not used in current kernel source. Consider it is really old, over
> 20 years, Just allow it to avoid CHECK_DTBS warning.
So mention in the commit msg that this is for existing old DTS. I
understand the list will not grow and compatibles will not be used in
any new designs, so it's fine.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2026-02-13 14:54 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-02-11 22:19 [PATCH 1/1] dt-bindings: soc: imx: add fsl,aipi-bus and fsl,emi-bus Frank Li
2026-02-12 11:59 ` Krzysztof Kozlowski
2026-02-12 15:20 ` Frank Li
2026-02-13 14:54 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox