[PATCH 02/24] spi: spi-mem: Add a new controller capability
Miquel Raynal
miquel.raynal at bootlin.com
Fri Dec 13 03:00:35 PST 2024
Hi Tudor,
>> Among all the spi-memory controllers, I believe all are capable of
>
> nit: SPI memory? or spi memory? but no "-" I think
...
>> changing the spi frequency on the fly. Some of the drivers do not make
>> any frequency setup though. And some others will derive a per-chip
>
> nit: s/per-chip/per chip?
...
>> pre-scaler value which will be used forever.
>
> nit: s/pre-scaler/prescaler?
...
>> + * @per_op_freq: Supports per-operation frequency switching
>
> nit: s/per-operation/per operation?
>
> If you fix the bug that you identified you can add my R-b tag,
> regardless if you address these nits or not:
>
> Reviewed-by: Tudor Ambarus <tudor.ambarus at linaro.org>
No idea why I like '-' so much. I removed them all as advised :-)
Thanks!
Miquèl
More information about the linux-mtd
mailing list