[EXT] Re: Boot failed after patch "mtd: rawnand: Support for sequential cache reads"

[Alexander Shiyan]

Hello.

For now I just changed rawnand_check_cont_read_support() function to
turn off supported_op.cont_read variable.
I haven't tried anything new.

пн, 26 июн. 2023 г. в 12:22, Bean Huo <beanhuo at micron.com>:
>
> Hi Alexander,
>
> Do you have any update on this issue?
>
>
> Kind regards,
> Bean
>
> > -----Original Message-----
> > From: Bean Huo
> > Sent: Monday, June 5, 2023 5:46 PM
> > To: Miquel Raynal <miquel.raynal at bootlin.com>; Alexander Shiyan
> > <eagle.alexander923 at gmail.com>
> > Cc: JaimeLiao <jaimeliao.tw at gmail.com>; linux-mtd at lists.infradead.org
> > Subject: RE: [EXT] Re: Boot failed after patch "mtd: rawnand: Support for sequential
> > cache reads"
> >
> > Hi Miquel
> >
> > I only can say, we have not experienced this kind issue yet.  My suggestion as yours,
> > please firstly Let Alex add debug print in mtd, especially in NAND driver. To check
> > what exactly error causes this SquashFS complain, is it IO error,  or CRC,  or ECC
> > error, or NAND device no response and timeout?
> >
> > Kind regards,
> > Bean
> >
> > > -----Original Message-----
> > > From: Miquel Raynal <miquel.raynal at bootlin.com>
> > > Sent: Monday, June 5, 2023 10:40 AM
> > > To: Alexander Shiyan <eagle.alexander923 at gmail.com>
> > > Cc: Bean Huo <beanhuo at micron.com>; JaimeLiao <jaimeliao.tw at gmail.com>;
> > > linux-mtd at lists.infradead.org
> > > Subject: Re: [EXT] Re: Boot failed after patch "mtd: rawnand: Support
> > > for sequential cache reads"
> > >
> > > CAUTION: EXTERNAL EMAIL. Do not click links or open attachments unless
> > > you recognize the sender and were expecting this message.
> > >
> > >
> > > Hi Alexander,
> > >
> > > eagle.alexander923 at gmail.com wrote on Fri, 2 Jun 2023 12:52:08 +0300:
> > >
> > > > Hello.
> > > >
> > > > > As you mentioned no ECC error.  And SquashFS complains: Unable to
> > > > > read data
> > > cache.
> > > > > We want to see I/Ox, RE#, WE# and R/B#, to check if command input
> > > > > and data
> > > output properly.
> > > > > It is better to capture the command 31h, and its following data.
> > > >
> > > > I only have a two-channel oscilloscope :) In any case, in order to
> > > > capture the right moment, I need to somehow loop command 0x31 when
> > > > an error occurs in order to take a picture. Right?
> > > >
> > > > Please, tell me where I can patch the nand_base to get such an
> > > > infinity loop on error?
> > >
> > > Anywhere in the core you could just make your own exec_op sequence and
> > > call it in a loop I guess?
> > >
> > > Bean, what are you trying to picture precisely? Have you ever had any
> > > issues with these commands? Can Alexander try to add a surgical delay somewhere?
> > >
> > > Thanks,
> > > Miquèl