Hi Koguchi-san,

> -----Original Message-----
> From: cip-dev@lists.cip-project.org [mailto:cip-dev@lists.cip-project.org] On Behalf Of Takuo Koguchi
> Sent: Monday, March 15, 2021 1:33 PM
> To: cip-dev@lists.cip-project.org
> Subject: Re: [cip-dev] Is cadence-quadspi working on cyclone V?
> 
> Hi Pavel,
> 
> > >> Git bisect has told me the first bad commit is
> > >> 31fb632b5d43 spi: Move cadence-quadspi driver to drivers/spi/
> > >
> > > That's strange.
> > >
> > >> Does this change requires device tree modification?
> > >> Any suggestions or comments would be appreciated.
> > >
> > > ...because that particular change does not change any C code. But it
> > > touches Kconfig bits; can you check that CONFIG_SPI_CADENCE_QUADSPI is
> > > enabled in the "bad" configuration?
> > >
> > > Do you have any local changes in
> > > drivers/mtd/spi-nor/controllers/cadence-quadspi.c or
> > > drivers/spi/spi-cadence-quadspi.c ?
> > I have no local changes to those files.
> > Maybe I lost CONFIG_SPI_CADENCE_QUADSPI accidentally while bisecting.
> > I will try bisect and post the result again.
> >
> 
> I redo the bisect and get the following result;
> # first bad commit: [a314f6367787ee1d767df9a2120f17e4511144d0] mtd: spi-nor: Convert cadence-quadspi to use spi-mem
> framework
> 
> It contains significant amount of changes;
> 
> cadence-quadspi.c |  476 +++++++++++++++++++++---------------------------------
>  1 file changed, 191 insertions(+), 285 deletions(-)
> 
> With this change, cpspi_probe return 0 without any error messages, but it does not detect NOR flash on a custom cyclone
> V board.
> 
> Any information will be appreciated.

I checked on a Sodia board with the same SoC.
  https://git.kernel.org/pub/scm/linux/kernel/git/cip/linux-cip.git/tree/arch/arm/boot/dts/socfpga_cyclone5_sodia.dts?h=linux-5.10.y-cip

It recognizes NOR flash in 5.10.y. How about checking DTS?
Sodia board uses n25q512a and has the following DT:
```
&qspi {
	status = "okay";

	flash0: n25q512a@0 {
		#address-cells = <1>;
		#size-cells = <1>;
		compatible = "n25q512a";
		reg = <0>;
		spi-max-frequency = <100000000>;

		m25p,fast-read;
		cdns,page-size = <256>;
		cdns,block-size = <16>;
		cdns,read-delay = <4>;
		cdns,tshsl-ns = <50>;
		cdns,tsd2d-ns = <50>;
		cdns,tchsh-ns = <4>;
		cdns,tslch-ns = <4>;
	};
};
```
> 
> Thanks,
> 
> Takuo Koguchi
> 

Best regards,
  Nobuhiro


> > -----Original Message-----
> > From: 小口琢夫 / KOGUCHI，TAKUO
> > Sent: Sunday, March 14, 2021 4:37 PM
> > To: cip-dev@lists.cip-project.org
> > Subject: Re: [!]Re: [cip-dev] Is cadence-quadspi working on cyclone V?
> >
> > Hi Pavel,
> > Thank you for the response.
> >
> > >> Git bisect has told me the first bad commit is
> > >> 31fb632b5d43 spi: Move cadence-quadspi driver to drivers/spi
> > >
> > > That's strange.
> > >
> > >> Does this change requires device tree modification?
> > >> Any suggestions or comments would be appreciated.
> > >
> > > ...because that particular change does not change any C code. But it
> > > touches Kconfig bits; can you check that CONFIG_SPI_CADENCE_QUADSPI is
> > > enabled in the "bad" configuration?
> > >
> > > Do you have any local changes in
> > > drivers/mtd/spi-nor/controllers/cadence-quadspi.c or
> > > drivers/spi/spi-cadence-quadspi.c ?
> > I have no local changes to those files.
> > Maybe I lost CONFIG_SPI_CADENCE_QUADSPI accidentally while bisecting.
> > I will try bisect and post the result again.
> >
> > Regards,
> >
> > Takuo Koguchi
> >
> > > 2021/03/13 8:07、Pavel Machek <pavel@denx.de>のメール:
> > >
> > > Hi!
> > >
> > >> Git bisect has told me the first bad commit is
> > >> 31fb632b5d43 spi: Move cadence-quadspi driver to drivers/spi/
> > >
> > > That's strange.
> > >
> > >> Does this change requires device tree modification?
> > >> Any suggestions or comments would be appreciated.
> > >
> > > ...because that particular change does not change any C code. But it
> > > touches Kconfig bits; can you check that CONFIG_SPI_CADENCE_QUADSPI is
> > > enabled in the "bad" configuration?
> > >
> > > Do you have any local changes in
> > > drivers/mtd/spi-nor/controllers/cadence-quadspi.c or
> > > drivers/spi/spi-cadence-quadspi.c ?
> > >
> > > Best regards,
> > >                                Pavel
> > > --
> > > DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
> > > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> > >
> > >
> > >