[AMD Official Use Only - AMD Internal Distribution Only] Hi Prasanth,
> -----Original Message----- > From: Neha Malcom Francis <[email protected]> > Sent: Wednesday, July 16, 2025 1:11 PM > To: Prasanth Babu Mantena <[email protected]>; [email protected]; > [email protected]; [email protected]; Simek, Michal > <[email protected]> > Cc: [email protected]; Abbarapu, Venkatesh <[email protected]>; > [email protected]; [email protected]; [email protected]; [email protected] > Subject: Re: [PATCH] Revert "spi: cadence_qspi: Fix odd byte write issue in > STIG > mode" > > Hi Prasanth > > On 16/07/25 12:34, Prasanth Babu Mantena wrote: > > The buffer that is being used to write into the flash needs to be > > handled properly with padding of 0xFF. The place that this is done can > > be at a more generic place like spi-nor core. > > > > This reverts commit cd9123507003e07b13e61d72e14e493bb338e827. > > > > Signed-off-by: Prasanth Babu Mantena <[email protected]> > > --- > > drivers/spi/cadence_qspi_apb.c | 3 --- > > 1 file changed, 3 deletions(-) > > > > diff --git a/drivers/spi/cadence_qspi_apb.c > > b/drivers/spi/cadence_qspi_apb.c index 6f89d3add5d..4696c09f754 100644 > > --- a/drivers/spi/cadence_qspi_apb.c > > +++ b/drivers/spi/cadence_qspi_apb.c > > @@ -558,9 +558,6 @@ int cadence_qspi_apb_command_write(struct > cadence_spi_priv *priv, > > void *reg_base = priv->regbase; > > u8 opcode; > > > > - if (priv->dtr) > > - txlen += txlen & 1; > > - > > if (priv->dtr) > > opcode = op->cmd.opcode >> 8; > > else > > After reading [0], makes sense. > > Reviewed-by: Neha Malcom Francis <[email protected]> > > [0] > https://lore.kernel.org/u-boot/[email protected]/ Ok...Will check and handle this in spi-nor core framework itself. Reviewed-by: Venkatesh Yadav Abbarapu <[email protected]> Thanks Venkatesh > > -- > Thanking You > Neha Malcom Francis
