[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