Re: [PATCH 11/30] spi: dw: Add DWC SSI capability

From: Mark Brown
Date: Wed Sep 30 2020 - 11:42:54 EST


On Wed, Sep 30, 2020 at 06:03:12PM +0300, Serge Semin wrote:
> On Wed, Sep 30, 2020 at 01:17:37AM +0300, Serge Semin wrote:

> > > > - /*
> > > > - * SPI mode (SCPOL|SCPH)
> > > > - * CTRLR0[ 8] Serial Clock Phase
> > > > - * CTRLR0[ 9] Serial Clock Polarity
> > > > - */
> > > > - cr0 |= ((spi->mode & SPI_CPOL) ? 1 : 0) << DWC_SSI_CTRLR0_SCPOL_OFFSET;

> anyway. If you are agree with me having that done here, then please, accept the
> patch the way it is. If you disagree, or have any other though, please give me
> your answer, why.

Those comments did seem to help mitigate the wall of acronym soup issue
that the code has, it seems a shame to drop them.

Attachment: signature.asc
Description: PGP signature