On Wed, Mar 25, 2015 at 08:10:57AM +0100, Martin Sperl wrote:

> +static inline void bcm2835_set_cs(struct spi_device *spi, bool enable)
> +{
> +     if (spi->mode & SPI_CS_HIGH)
> +             enable = !enable;
> +
> +     if (gpio_is_valid(spi->cs_gpio))
> +             gpio_set_value(spi->cs_gpio, !enable);
> +}

This appears to be open coding the core spi_set_cs() support for GPIOs,
why are we not just setting cs_gpio and letting the core take care of
things?

Attachment: signature.asc
Description: Digital signature

Reply via email to