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

Groups > linux.kernel > #1643155

Re: [PATCH 1/2] spi: spidev: introduce SPI_IOC_WR_DEFAULT_MAX_SPEED_HZ command

Show all headers | View raw

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

On Mon, May 15, 2017 at 10:17:11AM +0200, Seraphime Kirkovski wrote:
> On Sun, May 14, 2017 at 06:27:16PM +0900, Mark Brown wrote:

> > Do we have any evidence that such users exist?  

> I can't guarantee for other such users, but this change did disturb our 
> workflow. We were using this feature to prototype and test 
> hardware/firmware at different speeds.

> I was thinking this morning that maybe a sysfs interface will be better 
> for setting global settings. Do you prefer ?

I just don't see this as something that should be being varied at
runtime; it is supposed to be the maximum speed that the bus can be run
at, that's not something that should vary at runtime.  Possibly a sysfs
thing but honestly if you're prototyping via spidev I'd just set the
speed in your application each time it starts.

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

Thread

[PATCH 1/2] spi: spidev: introduce SPI_IOC_WR_DEFAULT_MAX_SPEED_HZ  command Seraphime Kirkovski <kirkseraph@gmail.com> - 2017-05-15 10:30 +0200
  Re: [PATCH 1/2] spi: spidev: introduce  SPI_IOC_WR_DEFAULT_MAX_SPEED_HZ command Mark Brown <broonie@kernel.org> - 2017-05-17 11:10 +0200

csiph-web