[PATCH resend 3/5] mpc8xxx_spi: put max_cs to use
trini at konsulko.com
Tue Mar 31 21:10:02 CEST 2020
On Tue, Feb 11, 2020 at 03:20:24PM +0000, Rasmus Villemoes wrote:
> Currently, max_cs is write-only; it's just set in
> mpc8xxx_spi_ofdata_to_platdata and not otherwise used.
> My mpc8309 was always resetting during an "sf probe 0". It turns out
> dm_gpio_set_dir_flags() was being called with garbage, since nothing
> had initialized priv->gpios - our device tree used "cs-gpios"
> rather than "gpios", so gpio_request_list_by_name() had returned 0.
> That would have been a lot easier to figure out if the chip select
> index was sanity checked, so rename max_cs to cs_count, and reject a
> xfer with a too large cs index.
> Signed-off-by: Rasmus Villemoes <rasmus.villemoes at prevas.dk>
Applied to u-boot/master, thanks!
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 659 bytes
Desc: not available
More information about the U-Boot