RE: [PATCH v2 5/5] spi: tegra210-quad: combined sequence mode

From: Krishna Yarlagadda
Date: Wed Feb 23 2022 - 01:34:51 EST


> -----Original Message-----
> From: Mark Brown <broonie@xxxxxxxxxx>
> Sent: 23 February 2022 00:12
> To: Krishna Yarlagadda <kyarlagadda@xxxxxxxxxx>
> Cc: thierry.reding@xxxxxxxxx; Jonathan Hunter
> <jonathanh@xxxxxxxxxx>; linux-spi@xxxxxxxxxxxxxxx; linux-
> tegra@xxxxxxxxxxxxxxx; Sowjanya Komatineni
> <skomatineni@xxxxxxxxxx>; Laxman Dewangan
> <ldewangan@xxxxxxxxxx>; robh+dt@xxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
> p.zabel@xxxxxxxxxxxxxx
> Subject: Re: [PATCH v2 5/5] spi: tegra210-quad: combined sequence
> mode
>
> On Tue, Feb 22, 2022 at 11:26:11PM +0530, Krishna Yarlagadda wrote:
>
> > + val = tegra_qspi_readl(tqspi, QSPI_GLOBAL_CONFIG);
> > + val |= QSPI_CMB_SEQ_EN;
> > + tegra_qspi_writel(tqspi, val, QSPI_GLOBAL_CONFIG);
>
> I notice that nothing seems to clear QSPI_CMB_SEQ_EN - is that self
> clearing or something?
Need a change in non-combined sequence code to reset this.
Will add the change in v3