[PATCH 0/2] spi: clps711x: SPI driver refactor

[PATCH 0/2] spi: clps711x: SPI driver refactor

[Alexander Shiyan]

This is a complex patch for refactoring CLPS711X SPI driver
as a next step to adding DT support for this platform.
Major changes:
- Eliminate <mach/hardware.h> usage.
- Devicetree support.

Since we do not have any platform users of this driver,
non-DT support is removed. DT users will be introduced later
once DT support for this platform will be completed.
"ACK" from arm-soc maintainers will be useful here.

Alexander Shiyan (2):
  spi: clps711x: Driver refactor
  spi: clps711x: dts: Add bindings documentation for the CLPS711X SPI

 .../devicetree/bindings/spi/spi-clps711x.txt       |  22 ++
 drivers/spi/Kconfig                                |   2 +-
 drivers/spi/spi-clps711x.c                         | 233 ++++++++-------------
 include/linux/platform_data/spi-clps711x.h         |  21 --
 4 files changed, 106 insertions(+), 172 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/spi/spi-clps711x.txt
 delete mode 100644 include/linux/platform_data/spi-clps711x.h

-- 
1.8.3.2

[PATCH 0/2] spi: clps711x: SPI driver refactor

[Mark Brown]

On Wed, Jan 01, 2014 at 10:08:08AM +0400, Alexander Shiyan wrote:

> Since we do not have any platform users of this driver,
> non-DT support is removed. DT users will be introduced later
> once DT support for this platform will be completed.
> "ACK" from arm-soc maintainers will be useful here.

That's not likely to happen given that you didn't CC them, though it
shouldn't be need anyway - there's no reason for the arm-soc guys to
have to review driver changes.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140101/2ba37789/attachment.sig>

Re: [PATCH 0/2] spi: clps711x: SPI driver refactor

[Alexander Shiyan]

Hello.

> On Wed, Jan 01, 2014 at 10:08:08AM +0400, Alexander Shiyan wrote:
> 
> > Since we do not have any platform users of this driver,
> > non-DT support is removed. DT users will be introduced later
> > once DT support for this platform will be completed.
> > "ACK" from arm-soc maintainers will be useful here.
> 
> That's not likely to happen given that you didn't CC them, 
...

AFAIK, CC is not need if "arm at kenrel.org" is in the CC, since it's an alias
for all arm-soc maintainers.
Thanks.

---

[PATCH 0/2] spi: clps711x: SPI driver refactor

[Russell King - ARM Linux]

On Wed, Jan 01, 2014 at 06:02:32PM +0400, Alexander Shiyan wrote:
> AFAIK, CC is not need if "arm at kenrel.org" is in the CC, since it's an alias
> for all arm-soc maintainers.

but only if you spell it correctly.  (removed the broken address from the
Cc line)

Re: [PATCH 0/2] spi: clps711x: SPI driver refactor

[Alexander Shiyan]

> On Wed, Jan 01, 2014 at 06:02:32PM +0400, Alexander Shiyan wrote:
> > AFAIK, CC is not need if "arm at kenrel.org" is in the CC, since it's an alias
> > for all arm-soc maintainers.
> 
> but only if you spell it correctly.  (removed the broken address from the
> Cc line)

Ahh, yes! My bug. Sorry.

---