Hi, On Fri, Aug 10, 2018 at 9:43 AM, Mark Brown <broonie@xxxxxxxxxx> wrote: >> IMO the line marked "/* UNNEEDED */" below should be removed: > >> ... >> spi-max-frequency = <50000000>; /* UNNEEDED */ > > This is a line in the device tree (which I agree shouldn't be there), > not code in the SPI driver? My whole point was that the line above shouldn't be there. If you agree to that then we're done. Everything else was my suggestions of ways the driver would deal with that line not being there. -Doug