From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Thu, 25 Mar 2021 18:41:43 +0100 Subject: [PATCH V2] spi: Update speed/mode on change In-Reply-To: <20210226142124.56305-1-marex@denx.de> References: <20210226142124.56305-1-marex@denx.de> Message-ID: <6e384b07-816f-12aa-b206-af1784875d45@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 2/26/21 3:21 PM, Marek Vasut wrote: > The spi_get_bus_and_cs() may be called on the same bus and chipselect > with different frequency or mode. This is valid usecase, but the code > fails to notify the controller of such a configuration change. Call > spi_set_speed_mode() in case bus frequency or bus mode changed to let > the controller update the configuration. > > The problem can easily be triggered using the sspi command: > => sspi 0:0 at 1000 > => sspi 0:0 at 2000 > Without this patch, both transfers happen at 1000 Hz. With this patch, > the later transfer happens correctly at 2000 Hz. > > Signed-off-by: Marek Vasut > Cc: Jagan Teki > Cc: Patrick Delaunay This bugfix is still missing from upstream.