Hello Mark, Thanks for the review.
On 2 July 2014 22:26, Mark Brown <[email protected]> wrote: > On Fri, Jun 13, 2014 at 09:29:50AM +0530, Naveen Krishna Chatradhi wrote: > >> Hence, spi-s3c64xx.c is broken since "Jun 21 11:26:12 2013" and >> considering the time with no compliants about the breakage. > > I'm not clear what the breakage is? Some boards are broken but what's > the driver issue? ToT was broken for few boards exynos4412-trats2.dts, exynos4210-smdkv310.dts and exynos5250-smdk5250.dts With some DTS changes SPI works well, spi-s3c64xx.c driver had no issues. > >> if (!spi_get_ctldata(spi)) { >> /* Request gpio only if cs line is asserted by gpio pins */ >> - if (sdd->cs_gpio) { >> - err = gpio_request_one(cs->line, GPIOF_OUT_INIT_HIGH, >> - dev_name(&spi->dev)); >> + if (gpio_is_valid(spi->cs_gpio)) { >> + err = gpio_request_one(spi->cs_gpio, >> + GPIOF_OUT_INIT_HIGH, >> + dev_name(&spi->dev)); >> if (err) { >> dev_err(&spi->dev, >> "Failed to get /CS gpio [%d]: %d\n", >> - cs->line, err); >> + spi->cs_gpio, err); >> goto err_gpio_req; >> } >> - >> - spi->cs_gpio = cs->line; >> + } else { >> + dev_err(&spi->dev, "chip select gpio is invalid\n"); >> + return -EINVAL; >> } > > This appears to be making it mandatory to specify a GPIO when previously > it was not mandatory which I believe breaks some SoCs - the native chip > select has to be used on some devices as there is no pinmux option to > make it a GPIO (this was why the native chip select support was added). I read the commit "3146beec21b64f4551fcf0ac148381d54dc41b1b" "spi: s3c64xx: Added provision for dedicated cs pin" I understand "cs_gpio" shouldn't be mandatory. Will remove the else part. > > Also I'd need to check but are you sure that GPIO 0 is not valid? gpio_is_valid() returns true for "number >= 0 && number < ARCH_NR_GPIOS" -- Shine bright, (: Nav :) ------------------------------------------------------------------------------ Open source business process management suite built on Java and Eclipse Turn processes into business applications with Bonita BPM Community Edition Quickly connect people, data, and systems into organized workflows Winner of BOSSIE, CODIE, OW2 and Gartner awards http://p.sf.net/sfu/Bonitasoft _______________________________________________ spi-devel-general mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/spi-devel-general
