* Re: [cip-dev] Is cadence-quadspi working on cyclone V? @ 2021-03-15 4:32 Takuo Koguchi 2021-03-15 5:25 ` Nobuhiro Iwamatsu 0 siblings, 1 reply; 8+ messages in thread From: Takuo Koguchi @ 2021-03-15 4:32 UTC (permalink / raw) To: cip-dev [-- Attachment #1: Type: text/plain, Size: 3361 bytes --] 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. Thanks, Takuo Koguchi > -----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 > > > > > > [-- Attachment #2: Type: text/plain, Size: 428 bytes --] -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#6281): https://lists.cip-project.org/g/cip-dev/message/6281 Mute This Topic: https://lists.cip-project.org/mt/81272003/4520388 Group Owner: cip-dev+owner@lists.cip-project.org Unsubscribe: https://lists.cip-project.org/g/cip-dev/leave/8129055/4520388/727948398/xyzzy [cip-dev@archiver.kernel.org] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [cip-dev] Is cadence-quadspi working on cyclone V? 2021-03-15 4:32 [cip-dev] Is cadence-quadspi working on cyclone V? Takuo Koguchi @ 2021-03-15 5:25 ` Nobuhiro Iwamatsu 2021-03-15 5:31 ` Takuo Koguchi 0 siblings, 1 reply; 8+ messages in thread From: Nobuhiro Iwamatsu @ 2021-03-15 5:25 UTC (permalink / raw) To: cip-dev [-- Attachment #1: Type: text/plain, Size: 4588 bytes --] 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 > > > > > > > > > [-- Attachment #2: Type: text/plain, Size: 428 bytes --] -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#6282): https://lists.cip-project.org/g/cip-dev/message/6282 Mute This Topic: https://lists.cip-project.org/mt/81272003/4520388 Group Owner: cip-dev+owner@lists.cip-project.org Unsubscribe: https://lists.cip-project.org/g/cip-dev/leave/8129055/4520388/727948398/xyzzy [cip-dev@archiver.kernel.org] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [cip-dev] Is cadence-quadspi working on cyclone V? 2021-03-15 5:25 ` Nobuhiro Iwamatsu @ 2021-03-15 5:31 ` Takuo Koguchi 2021-03-16 5:54 ` Takuo Koguchi 0 siblings, 1 reply; 8+ messages in thread From: Takuo Koguchi @ 2021-03-15 5:31 UTC (permalink / raw) To: cip-dev [-- Attachment #1: Type: text/plain, Size: 6257 bytes --] Iwamatsu-san, Thank you for the information. > 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>; > }; > }; > ``` I am going to look into the difference. Takuo Koguchi > -----Original Message----- > From: cip-dev@lists.cip-project.org <cip-dev@lists.cip-project.org> On Behalf Of Nobuhiro Iwamatsu > Sent: Monday, March 15, 2021 2:25 PM > To: cip-dev@lists.cip-project.org > Subject: [!]Re: [cip-dev] Is cadence-quadspi working on cyclone V? > > 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://secure-web.cisco.com/1ChOMrKnSBR5QZeejvrNs3Pf4x8__M_QVUHErnB9slV-TAWP3NUwe0LEzsirvM7rr-Oqv > RKrgxgI3iQo5ITdUehFyDppd2GljkEzx7izv16pERniX0DeDmzwE5h9pUYqtIxO3WAFTDuH_EG-RNhNFqK4LQakZ90sESF6 > DMvs_jovN55Xj5u9kNOC_ew76pLtAMAOdN3BZSV_fR_n6tcWlhvqJ6OlUEfrpz9xvy-gu1VHrgcgj1BICaBid9bVUza1-T7rBc > I2EjECtd0qCMcsvLYZPxvzfZ9-Hu6rFlaCQ0ruhxZVfHuVWEB37eQmm0HLHYJG2_R_ousVeVDk8bJa2iA/https%3A%2F%2 > Fgit.kernel.org%2Fpub%2Fscm%2Flinux%2Fkernel%2Fgit%2Fcip%2Flinux-cip.git%2Ftree%2Farch%2Farm%2Fboot%2Fd > ts%2Fsocfpga_cyclone5_sodia.dts%3Fh%3Dlinux-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 > > > > > > > > > > > > [-- Attachment #2: Type: text/plain, Size: 428 bytes --] -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#6283): https://lists.cip-project.org/g/cip-dev/message/6283 Mute This Topic: https://lists.cip-project.org/mt/81272003/4520388 Group Owner: cip-dev+owner@lists.cip-project.org Unsubscribe: https://lists.cip-project.org/g/cip-dev/leave/8129055/4520388/727948398/xyzzy [cip-dev@archiver.kernel.org] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [cip-dev] Is cadence-quadspi working on cyclone V? 2021-03-15 5:31 ` Takuo Koguchi @ 2021-03-16 5:54 ` Takuo Koguchi 2021-03-16 6:54 ` Nobuhiro Iwamatsu 0 siblings, 1 reply; 8+ messages in thread From: Takuo Koguchi @ 2021-03-16 5:54 UTC (permalink / raw) To: cip-dev [-- Attachment #1: Type: text/plain, Size: 8253 bytes --] Iwamatsu-san, > > 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>; > > };t > > }; > > ``` > I am going to look into the difference. I confirmed that "n25q512a" in the node name and the compatible string are the only difference. Interestingly, if I used compatible = "n25q512a" instead of compatible = "s25fl128s1" or "s25fl128s", which is the current setting for linux-4.19.177-cip, the kernel prints the following message and is able to detect spi-nor flash as expected. [ 2.701294] spi-nor spi0.0: found s25fl128s1, expected n25q512a [ 2.711093] spi-nor spi0.0: s25fl128s1 (16384 Kbytes) Probably the detect mechanism has been changed by the commit. Apparently setting compatible = "n25q512a" for my board does not make sense. I'd like to find out why compatible = "s25fl128s1" does not work. Regards, Takuo Koguchi > -----Original Message----- > From: 小口琢夫 / KOGUCHI，TAKUO > Sent: Monday, March 15, 2021 2:31 PM > To: cip-dev@lists.cip-project.org > Subject: RE: [cip-dev] Is cadence-quadspi working on cyclone V? > > Iwamatsu-san, > > Thank you for the information. > > 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>; > > }; > > }; > > ``` > I am going to look into the difference. > > Takuo Koguchi > > > -----Original Message----- > > From: cip-dev@lists.cip-project.org <cip-dev@lists.cip-project.org> On > > Behalf Of Nobuhiro Iwamatsu > > Sent: Monday, March 15, 2021 2:25 PM > > To: cip-dev@lists.cip-project.org > > Subject: [!]Re: [cip-dev] Is cadence-quadspi working on cyclone V? > > > > 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://secure-web.cisco.com/1ChOMrKnSBR5QZeejvrNs3Pf4x8__M_QVUHErnB9s > > lV-TAWP3NUwe0LEzsirvM7rr-Oqv > > RKrgxgI3iQo5ITdUehFyDppd2GljkEzx7izv16pERniX0DeDmzwE5h9pUYqtIxO3WAFTDu > > H_EG-RNhNFqK4LQakZ90sESF6 > > DMvs_jovN55Xj5u9kNOC_ew76pLtAMAOdN3BZSV_fR_n6tcWlhvqJ6OlUEfrpz9xvy-gu1 > > VHrgcgj1BICaBid9bVUza1-T7rBc > > I2EjECtd0qCMcsvLYZPxvzfZ9-Hu6rFlaCQ0ruhxZVfHuVWEB37eQmm0HLHYJG2_R_ousV > > eVDk8bJa2iA/https%3A%2F%2 > > Fgit.kernel.org%2Fpub%2Fscm%2Flinux%2Fkernel%2Fgit%2Fcip%2Flinux-cip.g > > it%2Ftree%2Farch%2Farm%2Fboot%2Fd > > ts%2Fsocfpga_cyclone5_sodia.dts%3Fh%3Dlinux-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 > > > > > > > > > > > > > > > [-- Attachment #2: Type: text/plain, Size: 428 bytes --] -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#6286): https://lists.cip-project.org/g/cip-dev/message/6286 Mute This Topic: https://lists.cip-project.org/mt/81272003/4520388 Group Owner: cip-dev+owner@lists.cip-project.org Unsubscribe: https://lists.cip-project.org/g/cip-dev/leave/8129055/4520388/727948398/xyzzy [cip-dev@archiver.kernel.org] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [cip-dev] Is cadence-quadspi working on cyclone V? 2021-03-16 5:54 ` Takuo Koguchi @ 2021-03-16 6:54 ` Nobuhiro Iwamatsu 0 siblings, 0 replies; 8+ messages in thread From: Nobuhiro Iwamatsu @ 2021-03-16 6:54 UTC (permalink / raw) To: cip-dev [-- Attachment #1: Type: text/plain, Size: 9251 bytes --] Hi, > -----Original Message----- > From: cip-dev@lists.cip-project.org [mailto:cip-dev@lists.cip-project.org] On Behalf Of Takuo Koguchi > Sent: Tuesday, March 16, 2021 2:54 PM > To: cip-dev@lists.cip-project.org > Subject: Re: [cip-dev] Is cadence-quadspi working on cyclone V? > > Iwamatsu-san, > > > > 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>; > > > };t > > > }; > > > ``` > > I am going to look into the difference. > > I confirmed that "n25q512a" in the node name and the compatible string are the only difference. > Interestingly, if I used compatible = "n25q512a" instead of compatible = "s25fl128s1" or "s25fl128s", > which is the current setting for linux-4.19.177-cip, > the kernel prints the following message and is able to detect spi-nor flash as expected. > > [ 2.701294] spi-nor spi0.0: found s25fl128s1, expected n25q512a > [ 2.711093] spi-nor spi0.0: s25fl128s1 (16384 Kbytes) > > Probably the detect mechanism has been changed by the commit. > Apparently setting compatible = "n25q512a" for my board does not make sense. I'd like to find out why > compatible = "s25fl128s1" does not work. Probably because they are not registered as compatible strings in the driver. They seem to be used as chip names rather than compatible strings. Currently, the implementation is to specify "jedec,spi-nor" for the compatible string. Best regards, Nobuhiro > > -----Original Message----- > > From: 小口琢夫 / KOGUCHI，TAKUO > > Sent: Monday, March 15, 2021 2:31 PM > > To: cip-dev@lists.cip-project.org > > Subject: RE: [cip-dev] Is cadence-quadspi working on cyclone V? > > > > Iwamatsu-san, > > > > Thank you for the information. > > > 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>; > > > }; > > > }; > > > ``` > > I am going to look into the difference. > > > > Takuo Koguchi > > > > > -----Original Message----- > > > From: cip-dev@lists.cip-project.org <cip-dev@lists.cip-project.org> On > > > Behalf Of Nobuhiro Iwamatsu > > > Sent: Monday, March 15, 2021 2:25 PM > > > To: cip-dev@lists.cip-project.org > > > Subject: [!]Re: [cip-dev] Is cadence-quadspi working on cyclone V? > > > > > > 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://secure-web.cisco.com/1ChOMrKnSBR5QZeejvrNs3Pf4x8__M_QVUHErnB9s > > > lV-TAWP3NUwe0LEzsirvM7rr-Oqv > > > RKrgxgI3iQo5ITdUehFyDppd2GljkEzx7izv16pERniX0DeDmzwE5h9pUYqtIxO3WAFTDu > > > H_EG-RNhNFqK4LQakZ90sESF6 > > > DMvs_jovN55Xj5u9kNOC_ew76pLtAMAOdN3BZSV_fR_n6tcWlhvqJ6OlUEfrpz9xvy-gu1 > > > VHrgcgj1BICaBid9bVUza1-T7rBc > > > I2EjECtd0qCMcsvLYZPxvzfZ9-Hu6rFlaCQ0ruhxZVfHuVWEB37eQmm0HLHYJG2_R_ousV > > > eVDk8bJa2iA/https%3A%2F%2 > > > Fgit.kernel.org%2Fpub%2Fscm%2Flinux%2Fkernel%2Fgit%2Fcip%2Flinux-cip.g > > > it%2Ftree%2Farch%2Farm%2Fboot%2Fd > > > ts%2Fsocfpga_cyclone5_sodia.dts%3Fh%3Dlinux-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 > > > > > > > > > > > > > > > > > > [-- Attachment #2: Type: text/plain, Size: 428 bytes --] -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#6287): https://lists.cip-project.org/g/cip-dev/message/6287 Mute This Topic: https://lists.cip-project.org/mt/81272003/4520388 Group Owner: cip-dev+owner@lists.cip-project.org Unsubscribe: https://lists.cip-project.org/g/cip-dev/leave/8129055/4520388/727948398/xyzzy [cip-dev@archiver.kernel.org] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 8+ messages in thread
* [cip-dev] Is cadence-quadspi working on cyclone V? @ 2021-03-12 3:50 Takuo Koguchi 2021-03-12 23:06 ` Pavel Machek 0 siblings, 1 reply; 8+ messages in thread From: Takuo Koguchi @ 2021-03-12 3:50 UTC (permalink / raw) To: 'cip-dev@lists.cip-project.org' [-- Attachment #1.1: Type: text/plain, Size: 486 bytes --] Hi CIP developers, Can anyone confirm that spi-cadence-quadspi works fine with SPI-NOR on cyclone V? I recently moved to 5.10.y-cip from 4.19.y-cip, then my custom board with cyclone V failed to detect SPI-NOR Git bisect has told me the first bad commit is 31fb632b5d43 spi: Move cadence-quadspi driver to drivers/spi/ Does this change requires device tree modification? Any suggestions or comments would be appreciated. Takuo Koguchi V [-- Attachment #1.2: Type: text/html, Size: 4850 bytes --] [-- Attachment #2: Type: text/plain, Size: 428 bytes --] -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#6277): https://lists.cip-project.org/g/cip-dev/message/6277 Mute This Topic: https://lists.cip-project.org/mt/81272003/4520388 Group Owner: cip-dev+owner@lists.cip-project.org Unsubscribe: https://lists.cip-project.org/g/cip-dev/leave/8129055/4520388/727948398/xyzzy [cip-dev@archiver.kernel.org] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [cip-dev] Is cadence-quadspi working on cyclone V? 2021-03-12 3:50 Takuo Koguchi @ 2021-03-12 23:06 ` Pavel Machek 2021-03-14 7:37 ` Takuo Koguchi 0 siblings, 1 reply; 8+ messages in thread From: Pavel Machek @ 2021-03-12 23:06 UTC (permalink / raw) To: cip-dev [-- Attachment #1.1: Type: text/plain, Size: 738 bytes --] 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 [-- Attachment #1.2: Digital signature --] [-- Type: application/pgp-signature, Size: 181 bytes --] [-- Attachment #2: Type: text/plain, Size: 428 bytes --] -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#6278): https://lists.cip-project.org/g/cip-dev/message/6278 Mute This Topic: https://lists.cip-project.org/mt/81272003/4520388 Group Owner: cip-dev+owner@lists.cip-project.org Unsubscribe: https://lists.cip-project.org/g/cip-dev/leave/8129055/4520388/727948398/xyzzy [cip-dev@archiver.kernel.org] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [cip-dev] Is cadence-quadspi working on cyclone V? 2021-03-12 23:06 ` Pavel Machek @ 2021-03-14 7:37 ` Takuo Koguchi 0 siblings, 0 replies; 8+ messages in thread From: Takuo Koguchi @ 2021-03-14 7:37 UTC (permalink / raw) To: cip-dev [-- Attachment #1: Type: text/plain, Size: 1694 bytes --] 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 > > > [-- Attachment #2: Type: text/plain, Size: 428 bytes --] -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#6279): https://lists.cip-project.org/g/cip-dev/message/6279 Mute This Topic: https://lists.cip-project.org/mt/81272003/4520388 Group Owner: cip-dev+owner@lists.cip-project.org Unsubscribe: https://lists.cip-project.org/g/cip-dev/leave/8129055/4520388/727948398/xyzzy [cip-dev@archiver.kernel.org] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2021-03-16 6:54 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-03-15 4:32 [cip-dev] Is cadence-quadspi working on cyclone V? Takuo Koguchi 2021-03-15 5:25 ` Nobuhiro Iwamatsu 2021-03-15 5:31 ` Takuo Koguchi 2021-03-16 5:54 ` Takuo Koguchi 2021-03-16 6:54 ` Nobuhiro Iwamatsu -- strict thread matches above, loose matches on Subject: below -- 2021-03-12 3:50 Takuo Koguchi 2021-03-12 23:06 ` Pavel Machek 2021-03-14 7:37 ` Takuo Koguchi
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).