[PATCH 1/2] spi: clps711x: Driver refactor

Mark Brown broonie at kernel.org
Thu Jan 2 13:25:49 EST 2014


On Thu, Jan 02, 2014 at 06:46:23PM +0100, Arnd Bergmann wrote:

> I definitely agree with what you say, but please note that Alexander is
> the only person who ever sends clps711x patches, so there is not much
> hope of collaboration. I also suspect he's the only one who would

There's stuff been going in from the people who do generic cleanups and
updates.

> suffer from accidental breakage, especially since the existing driver
> is not used anywhere upstream (no instance of platform data, and no
> DT support).

Right.  On the other hand I really don't want to get into setting bad
precedents that people then use when they complain that I actually want
to review their code, that's just going to be more miserable in the long
run.

There were also some things in there that I really did want to review
properly but I wasn't entirely sure what they were actually supposed to
be doing.  Getting things split up with better changelogs will make that
review much more tractable, it might even be immediately obvious what is
going on.
-------------- 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/20140102/0da08763/attachment-0001.sig>


More information about the linux-arm-kernel mailing list