Thread (15 messages) 15 messages, 6 authors, 2014-07-11

[PATCH 1/3 v5] spi: s3c64xx: fix broken "cs_gpios" usage in the driver

flat view

From: Naveen Krishna Ch <hidden>
Date: 2014-06-21 17:23:14
Also in: linux-devicetree, linux-samsung-soc

Hello Mark,

On 21 June 2014 15:21, Mark Brown [off-list ref] wrote:
On Fri, Jun 20, 2014 at 02:18:08PM -0700, Doug Anderson wrote:
quoted
Is one of you two planning to apply (parts 1 and 2)?  I know that
Kukjin needs to handle the part 3 (the .dts files)...
quoted
Ideally it seems like it could go to 3.16 since it is a bugfix...
Don't nag, it just means yet more mail to look at which makes things
worse not better.  Given all the problems the DT code for this driver
has I need to review carefully, and the fact that this has got to v5
before I had a chance to look at it sets off alarm bells too.
The revisions were pretty quick
v1 to v2 : Updated DT Documentation for the change
v2 to v3 : use spi->cs_gpio in the driver
v3 to v4 : use gpio_is_valid() instead of using sdd->cs_gpio (local variable,
              also remove sdd->cs_gpio)
v4 to v5 : Added Acked-by and Revewied-by from Javier and Rob Herring

Major discussion in this thread was about keeping the old DT bindings
to support legacy or move on with the new bindings.

Tomasz Figa, Javier Martinez Canillas, Doug Anderson and
Rob Herring agreed with the latter.

Kindly review and let us know your comments.

Thanks
There's a couple of months to get bug fixes in yet.


-- 
Shine bright,
(: Nav :)
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help