[PATCH v1 1/1] spi: pxa2xx: Move PXA SSP bindings to the correct folder

[PATCH v1 1/1] spi: pxa2xx: Move PXA SSP bindings to the correct folder

[Andy Shevchenko]

SSP stands for Serial Synchronous Protocol and has nothing to do with
UART, also known as USART, where 'A' stands for Asynchronous.

Move the SSP bindings to where it belongs.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 .../devicetree/bindings/{serial => spi}/mrvl,pxa-ssp.txt          | 0
 1 file changed, 0 insertions(+), 0 deletions(-)
 rename Documentation/devicetree/bindings/{serial => spi}/mrvl,pxa-ssp.txt (100%)

diff --git a/Documentation/devicetree/bindings/serial/mrvl,pxa-ssp.txt b/Documentation/devicetree/bindings/spi/mrvl,pxa-ssp.txt
similarity index 100%
rename from Documentation/devicetree/bindings/serial/mrvl,pxa-ssp.txt
rename to Documentation/devicetree/bindings/spi/mrvl,pxa-ssp.txt
-- 
2.43.0.rc1.1336.g36b5255a03ac

Re: [PATCH v1 1/1] spi: pxa2xx: Move PXA SSP bindings to the correct folder

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 473 bytes --]

On Fri, May 17, 2024 at 08:11:03PM +0300, Andy Shevchenko wrote:
> SSP stands for Serial Synchronous Protocol and has nothing to do with
> UART, also known as USART, where 'A' stands for Asynchronous.
> 
> Move the SSP bindings to where it belongs.

It's a serial device which is also used for other applications (the
other one upstream being audio) so I can see where the current binding
comes from and it's not super obvious that spi is especially better
here.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: [PATCH v1 1/1] spi: pxa2xx: Move PXA SSP bindings to the correct folder

[Andy Shevchenko]

On Fri, May 17, 2024 at 06:24:37PM +0100, Mark Brown wrote:
> On Fri, May 17, 2024 at 08:11:03PM +0300, Andy Shevchenko wrote:
> > SSP stands for Serial Synchronous Protocol and has nothing to do with
> > UART, also known as USART, where 'A' stands for Asynchronous.
> > 
> > Move the SSP bindings to where it belongs.
> 
> It's a serial device which is also used for other applications (the
> other one upstream being audio) so I can see where the current binding
> comes from and it's not super obvious that spi is especially better
> here.

Hmm... okay. Then it's question to DT people. Consider this as a report.
Because UART (aka serial) is definitely not the place for SPI/SSP bindings
either.

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH v1 1/1] spi: pxa2xx: Move PXA SSP bindings to the correct folder

[Rob Herring]

On Fri, May 17, 2024 at 2:58 PM Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
>
> On Fri, May 17, 2024 at 06:24:37PM +0100, Mark Brown wrote:
> > On Fri, May 17, 2024 at 08:11:03PM +0300, Andy Shevchenko wrote:
> > > SSP stands for Serial Synchronous Protocol and has nothing to do with
> > > UART, also known as USART, where 'A' stands for Asynchronous.
> > >
> > > Move the SSP bindings to where it belongs.
> >
> > It's a serial device which is also used for other applications (the
> > other one upstream being audio) so I can see where the current binding
> > comes from and it's not super obvious that spi is especially better
> > here.
>
> Hmm... okay. Then it's question to DT people. Consider this as a report.
> Because UART (aka serial) is definitely not the place for SPI/SSP bindings
> either.

Move it when it is converted. Until then, I don't care too much. SPI
seems better than serial at least. The sound part is its own
binding/node (something we wouldn't do today).

Rob

Re: [PATCH v1 1/1] spi: pxa2xx: Move PXA SSP bindings to the correct folder

[Andy Shevchenko]

On Fri, May 17, 2024 at 03:19:51PM -0500, Rob Herring wrote:
> On Fri, May 17, 2024 at 2:58 PM Andy Shevchenko
> <andriy.shevchenko@linux.intel.com> wrote:
> > On Fri, May 17, 2024 at 06:24:37PM +0100, Mark Brown wrote:
> > > On Fri, May 17, 2024 at 08:11:03PM +0300, Andy Shevchenko wrote:

...

> > > > SSP stands for Serial Synchronous Protocol and has nothing to do with
> > > > UART, also known as USART, where 'A' stands for Asynchronous.
> > > >
> > > > Move the SSP bindings to where it belongs.
> > >
> > > It's a serial device which is also used for other applications (the
> > > other one upstream being audio) so I can see where the current binding
> > > comes from and it's not super obvious that spi is especially better
> > > here.
> >
> > Hmm... okay. Then it's question to DT people. Consider this as a report.
> > Because UART (aka serial) is definitely not the place for SPI/SSP bindings
> > either.
> 
> Move it when it is converted.

The problem is that somebody added a binding (in YAML) for SPI PXA2xx
in the spi/ folder while this one kept unconverted.

If it dangles more, it might be that we will have two asynchronous bindings
for the co-existed drivers.

> Until then, I don't care too much. SPI seems better than serial at least.
> The sound part is its own binding/node (something we wouldn't do today).

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH v1 1/1] spi: pxa2xx: Move PXA SSP bindings to the correct folder

[Rob Herring]

On Mon, May 20, 2024 at 01:47:26PM +0300, Andy Shevchenko wrote:
> On Fri, May 17, 2024 at 03:19:51PM -0500, Rob Herring wrote:
> > On Fri, May 17, 2024 at 2:58 PM Andy Shevchenko
> > <andriy.shevchenko@linux.intel.com> wrote:
> > > On Fri, May 17, 2024 at 06:24:37PM +0100, Mark Brown wrote:
> > > > On Fri, May 17, 2024 at 08:11:03PM +0300, Andy Shevchenko wrote:
> 
> ...
> 
> > > > > SSP stands for Serial Synchronous Protocol and has nothing to do with
> > > > > UART, also known as USART, where 'A' stands for Asynchronous.
> > > > >
> > > > > Move the SSP bindings to where it belongs.
> > > >
> > > > It's a serial device which is also used for other applications (the
> > > > other one upstream being audio) so I can see where the current binding
> > > > comes from and it's not super obvious that spi is especially better
> > > > here.
> > >
> > > Hmm... okay. Then it's question to DT people. Consider this as a report.
> > > Because UART (aka serial) is definitely not the place for SPI/SSP bindings
> > > either.
> > 
> > Move it when it is converted.
> 
> The problem is that somebody added a binding (in YAML) for SPI PXA2xx
> in the spi/ folder while this one kept unconverted.

Ah, well that detail was missed.

> 
> If it dangles more, it might be that we will have two asynchronous bindings
> for the co-existed drivers.

Looks like all that is needed is adding the compatible strings and 
'dmas' property to spi/marvell,mmp2-ssp.yaml. The examples in the old 
binding have other stuff, but looks like that's garbage.

Rob

Re: [PATCH v1 1/1] spi: pxa2xx: Move PXA SSP bindings to the correct folder

[Andy Shevchenko]

On Mon, May 20, 2024 at 03:36:00PM -0500, Rob Herring wrote:
> On Mon, May 20, 2024 at 01:47:26PM +0300, Andy Shevchenko wrote:
> > On Fri, May 17, 2024 at 03:19:51PM -0500, Rob Herring wrote:
> > > On Fri, May 17, 2024 at 2:58 PM Andy Shevchenko
> > > <andriy.shevchenko@linux.intel.com> wrote:
> > > > On Fri, May 17, 2024 at 06:24:37PM +0100, Mark Brown wrote:
> > > > > On Fri, May 17, 2024 at 08:11:03PM +0300, Andy Shevchenko wrote:

...

> > > > > > SSP stands for Serial Synchronous Protocol and has nothing to do with
> > > > > > UART, also known as USART, where 'A' stands for Asynchronous.
> > > > > >
> > > > > > Move the SSP bindings to where it belongs.
> > > > >
> > > > > It's a serial device which is also used for other applications (the
> > > > > other one upstream being audio) so I can see where the current binding
> > > > > comes from and it's not super obvious that spi is especially better
> > > > > here.
> > > >
> > > > Hmm... okay. Then it's question to DT people. Consider this as a report.
> > > > Because UART (aka serial) is definitely not the place for SPI/SSP bindings
> > > > either.
> > > 
> > > Move it when it is converted.
> > 
> > The problem is that somebody added a binding (in YAML) for SPI PXA2xx
> > in the spi/ folder while this one kept unconverted.
> 
> Ah, well that detail was missed.
> 
> > 
> > If it dangles more, it might be that we will have two asynchronous bindings
> > for the co-existed drivers.
> 
> Looks like all that is needed is adding the compatible strings and 
> 'dmas' property to spi/marvell,mmp2-ssp.yaml. The examples in the old 
> binding have other stuff, but looks like that's garbage.

I'm not an expert in DT, anybody to join them in a nicest possible way?

-- 
With Best Regards,
Andy Shevchenko