Re: [PATCH] spi: atmel: Exposing effective spi speed

[<Tudor.Ambarus@microchip.com>]

Hi, Thomas,

On 9/18/20 1:37 PM, Thomas Kopp wrote:
> This patch implements the reporting of the effectively used speed_hz for the
> transfer by setting xfer->effective_speed_hz.
> 
> See the following patch, which adds this feature to the SPI core for more
> information:
> 
>     5d7e2b5ed585 spi: core: allow reporting the effectivly used speed_hz for a transfer

s/effectivly/effectively

./scripts/checkpatch.pl --strict raises an error here, the commit should be
referred as commit 5d7e2b5ed585 ("spi: core: allow reporting the effectivly used speed_hz for a transfer")

> 
> Signed-off-by: Thomas Kopp <thomas.kopp@microchip.com>
> ---
>  drivers/spi/spi-atmel.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/spi/spi-atmel.c b/drivers/spi/spi-atmel.c
> index 2cfe6253a784..86857c8408b1 100644
> --- a/drivers/spi/spi-atmel.c
> +++ b/drivers/spi/spi-atmel.c
> @@ -859,6 +859,7 @@ static int atmel_spi_set_xfer_speed(struct atmel_spi *as,
>  	csr = spi_readl(as, CSR0 + 4 * chip_select);
>  	csr = SPI_BFINS(SCBR, scbr, csr);
>  	spi_writel(as, CSR0 + 4 * chip_select, csr);
> +	xfer->effective_speed_hz = bus_hz/scbr;

spaces preferred around that '/'

with these addressed one can add:
Reviewed-by: Tudor Ambarus <tudor.ambarus@microchip.com>

Cheers,
ta