[PATCH v2 2/2] spi: npcm-fiu: add NPCM FIU controller driver

Boris Brezillon boris.brezillon at collabora.com
Fri Aug 9 01:32:32 AEST 2019


On Thu,  8 Aug 2019 16:14:48 +0300
Tomer Maimon <tmaimon77 at gmail.com> wrote:


> +
> +static const struct spi_controller_mem_ops npcm_fiu_mem_ops = {
> +	.exec_op = npcm_fiu_exec_op,

No npcm_supports_op()? That's suspicious, especially after looking at
the npcm_fiu_exec_op() (and the functions called from there) where the
requested ->buswidth seems to be completely ignored...

> +	.dirmap_create = npcm_fiu_dirmap_create,
> +	.dirmap_read = npcm_fiu_direct_read,
> +	.dirmap_write = npcm_fiu_direct_write,
> +};
> +


More information about the openbmc mailing list