Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1694638

Re: [PATCH] spi: pxa2xx: Only claim CS GPIOs when the slave device is created

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

On 2017-07-24 12:44, Andy Shevchenko wrote:
> +Cc: Mika
> 
> On Sat, Jul 8, 2017 at 11:41 AM, Jan Kiszka <jan.kiszka@web.de> wrote:
>> From: Jan Kiszka <jan.kiszka@siemens.com>
>>
>> Avoid hogging chip select GPIOs just because they are listed for the
>> master. They might be mulitplexed and, if no slave device is attached,
>> used for different purposes. Moreover, this strategy avoids having to
>> allocate a cs_gpiods structure.
>>
>> Tested on the IOT2000 where the second SPI bus is connected to an
>> Arduino-compatible connector and multiplexed between SPI, GPIO and PWM
>> usage.
>>
> 
> This breaks all systems which are using _DSD.

Err, can you elaborate? Worked fine here with _DSD on the IOT2000.

> 
> While I'm looking for fix, I get feeling that the approach itself is not right,
> 
> So, for now I would vote for immediate revert and then rethink what we
> can do here.

I'm fine with reverting because the patch wasn't clean anyway (mixed old
and new GPIO API) - aside from whatever you found in addition. I had an
update pending but, as you are looking into this anyway, I'm sure your
patches will be more holistic.

Thanks,
Jan

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH] spi: pxa2xx: Only claim CS GPIOs when the slave device is created Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-07-24 12:50 +0200
  Re: [PATCH] spi: pxa2xx: Only claim CS GPIOs when the slave device is  created Jan Kiszka <jan.kiszka@web.de> - 2017-07-24 13:00 +0200
    Re: [PATCH] spi: pxa2xx: Only claim CS GPIOs when the slave device is  created Jan Kiszka <jan.kiszka@web.de> - 2017-07-24 13:10 +0200
      Re: [PATCH] spi: pxa2xx: Only claim CS GPIOs when the slave device is created Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-07-24 13:20 +0200
    Re: [PATCH] spi: pxa2xx: Only claim CS GPIOs when the slave device is created Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-07-24 13:10 +0200

csiph-web