2015-11-08 11:13 GMT+01:00 Marcus Weseloh <[email protected]>: > I did notice however, that the SPI system doesn't validate the > speed_hz value. So if I leave out the above quoted sanity checking, > then tfr->speed_hz could also contain values like -400, which trips up > the spi system in such a way that I could only repair with a reboot. I > was testing this with spidev... so maybe spidev should be patched to > check the speed value for validity?
I just had a look through the spi core and noticed that it does indeed validate the maximum and minium speed if set correctly on the spi master. So I guess it's just a matter of a missing minimum transfer speed in the DT. Sorry, should have done that earlier. Ok, I will prepare a new patch version that simply uses tfr->speed_hz. And I will also change sun6i-spi in the same patch, as sun4i and sun6i are using the same transfer logic. Cheers, Marcus -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
