Hello Naveen,
On 06/12/2014 03:13 PM, Naveen Krishna Chatradhi wrote:
> Use controller_data structure only for the Non Device tree platforms.
> For Device tree platforms, always derive the chipselect info from
> DT node.
>
> Signed-off-by: Naveen Krishna Chatradhi <ch.nav...@samsung.com>
> Cc: Javier Martinez Canillas <javier.marti...@collabora.co.uk>
> Cc: Doug Anderson <diand...@chromium.org>
> Cc: Tomasz Figa <t.f...@samsung.com>
> ---
> drivers/spi/spi-s3c64xx.c | 11 ++++++-----
> 1 file changed, 6 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/spi/spi-s3c64xx.c b/drivers/spi/spi-s3c64xx.c
> index b888c66..f27e15d 100644
> --- a/drivers/spi/spi-s3c64xx.c
> +++ b/drivers/spi/spi-s3c64xx.c
> @@ -795,14 +795,15 @@ static int s3c64xx_spi_setup(struct spi_device *spi)
> int err;
>
> sdd = spi_master_get_devdata(spi->master);
> - if (!cs && spi->dev.of_node) {
> + if (spi->dev.of_node) {
> cs = s3c64xx_get_slave_ctrldata(spi);
> spi->controller_data = cs;
> - }
> -
> - /* For the non-DT platforms derive chip selects from controller data */
> - if (!spi->dev.of_node)
> + } else {
> + /* For the non-DT platforms derive chip
> + * selects from controller data
> + */
> spi->cs_gpio = cs->line;
> + }
>
> if (IS_ERR_OR_NULL(cs)) {
> dev_err(&spi->dev, "No CS for SPI(%d)\n", spi->chip_select);
>
Personally I wouldn't have this change as a separate patch since it's too
related to what you changed in Patch 1. But it's just a nitpick.
Reviewed-by: Javier Martinez Canillas <javier.marti...@collabora.co.uk>
Best regards,
Javier
------------------------------------------------------------------------------
HPCC Systems Open Source Big Data Platform from LexisNexis Risk Solutions
Find What Matters Most in Your Big Data with HPCC Systems
Open Source. Fast. Scalable. Simple. Ideal for Dirty Data.
Leverages Graph Analysis for Fast Processing & Easy Data Exploration
http://p.sf.net/sfu/hpccsystems
_______________________________________________
spi-devel-general mailing list
spi-devel-general@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/spi-devel-general
