| Message ID | 20240215134856.1313239-1-quic_mdalam@quicinc.com |
|---|---|
| Headers | show |
| Series | Add QPIC SPI NAND driver | expand |
On Thu, Feb 15, 2024 at 07:18:51PM +0530, Md Sadre Alam wrote: > This series of patches will add initial supports > for QPIC SPI NAND driver. > > Currently this driver support following commands > > -- RESET > -- READ ID > -- BLOCK ERASE > -- PAGE READ > -- PAGE WRITE > -- GET FEATURE > -- SET FEATURE > -- BAD BLOCK CHECK > > This driver has been tested with dd command with read/write page > with multiple file size 1MiB, 10MiB,40MiB etc. > Also tested with "mtd" command like mtd erase, mtd write, mtd verify etc. > This is not the first version isn't it? Where is the changelog describing what has changed since then? - Mani > Need help to test these all patches on SDX65 and SDX75 platform. > > Md Sadre Alam (5): > spi: dt-bindings: add binding doc for spi-qpic-snand > drivers: mtd: nand: Add qpic_common API file > spi: spi-qpic: Add qpic spi nand driver support > arm64: dts: qcom: ipq9574: Add SPI nand support > arm64: dts: qcom: ipq9574: Disable eMMC node > > .../bindings/spi/qcom,spi-qpic-snand.yaml | 82 ++ > .../boot/dts/qcom/ipq9574-rdp-common.dtsi | 43 + > arch/arm64/boot/dts/qcom/ipq9574-rdp433.dts | 2 +- > arch/arm64/boot/dts/qcom/ipq9574.dtsi | 27 + > drivers/mtd/nand/Makefile | 1 + > drivers/mtd/nand/qpic_common.c | 794 +++++++++++ > drivers/mtd/nand/raw/qcom_nandc.c | 1226 +---------------- > drivers/spi/Kconfig | 9 + > drivers/spi/Makefile | 1 + > drivers/spi/spi-qpic-snand.c | 1025 ++++++++++++++ > include/linux/mtd/nand-qpic-common.h | 548 ++++++++ > 11 files changed, 2547 insertions(+), 1211 deletions(-) > create mode 100644 Documentation/devicetree/bindings/spi/qcom,spi-qpic-snand.yaml > create mode 100644 drivers/mtd/nand/qpic_common.c > create mode 100644 drivers/spi/spi-qpic-snand.c > create mode 100644 include/linux/mtd/nand-qpic-common.h > > -- > 2.34.1 >
On 2/19/2024 6:34 PM, Manivannan Sadhasivam wrote: > On Thu, Feb 15, 2024 at 07:18:51PM +0530, Md Sadre Alam wrote: >> This series of patches will add initial supports >> for QPIC SPI NAND driver. >> >> Currently this driver support following commands >> >> -- RESET >> -- READ ID >> -- BLOCK ERASE >> -- PAGE READ >> -- PAGE WRITE >> -- GET FEATURE >> -- SET FEATURE >> -- BAD BLOCK CHECK >> >> This driver has been tested with dd command with read/write page >> with multiple file size 1MiB, 10MiB,40MiB etc. >> Also tested with "mtd" command like mtd erase, mtd write, mtd verify etc. >> > > This is not the first version isn't it? Where is the changelog describing what > has changed since then? The earlier patch was the RFC for design review only. > > - Mani > >> Need help to test these all patches on SDX65 and SDX75 platform. >> >> Md Sadre Alam (5): >> spi: dt-bindings: add binding doc for spi-qpic-snand >> drivers: mtd: nand: Add qpic_common API file >> spi: spi-qpic: Add qpic spi nand driver support >> arm64: dts: qcom: ipq9574: Add SPI nand support >> arm64: dts: qcom: ipq9574: Disable eMMC node >> >> .../bindings/spi/qcom,spi-qpic-snand.yaml | 82 ++ >> .../boot/dts/qcom/ipq9574-rdp-common.dtsi | 43 + >> arch/arm64/boot/dts/qcom/ipq9574-rdp433.dts | 2 +- >> arch/arm64/boot/dts/qcom/ipq9574.dtsi | 27 + >> drivers/mtd/nand/Makefile | 1 + >> drivers/mtd/nand/qpic_common.c | 794 +++++++++++ >> drivers/mtd/nand/raw/qcom_nandc.c | 1226 +---------------- >> drivers/spi/Kconfig | 9 + >> drivers/spi/Makefile | 1 + >> drivers/spi/spi-qpic-snand.c | 1025 ++++++++++++++ >> include/linux/mtd/nand-qpic-common.h | 548 ++++++++ >> 11 files changed, 2547 insertions(+), 1211 deletions(-) >> create mode 100644 Documentation/devicetree/bindings/spi/qcom,spi-qpic-snand.yaml >> create mode 100644 drivers/mtd/nand/qpic_common.c >> create mode 100644 drivers/spi/spi-qpic-snand.c >> create mode 100644 include/linux/mtd/nand-qpic-common.h >> >> -- >> 2.34.1 >> >
On 20/02/2024 12:32, Md Sadre Alam wrote: > > > On 2/19/2024 6:34 PM, Manivannan Sadhasivam wrote: >> On Thu, Feb 15, 2024 at 07:18:51PM +0530, Md Sadre Alam wrote: >>> This series of patches will add initial supports >>> for QPIC SPI NAND driver. >>> >>> Currently this driver support following commands >>> >>> -- RESET >>> -- READ ID >>> -- BLOCK ERASE >>> -- PAGE READ >>> -- PAGE WRITE >>> -- GET FEATURE >>> -- SET FEATURE >>> -- BAD BLOCK CHECK >>> >>> This driver has been tested with dd command with read/write page >>> with multiple file size 1MiB, 10MiB,40MiB etc. >>> Also tested with "mtd" command like mtd erase, mtd write, mtd verify etc. >>> >> >> This is not the first version isn't it? Where is the changelog describing what >> has changed since then? > > The earlier patch was the RFC for design review only. RFC is state of patch, not version. This is v2 then. These RFC postings are really becoming mess. Some people make multiple RFCs and then post v1 hiding entire previous history... And why even bother with calling it RFC? Best regards, Krzysztof
On 2/20/2024 5:06 PM, Krzysztof Kozlowski wrote: > On 20/02/2024 12:32, Md Sadre Alam wrote: >> >> >> On 2/19/2024 6:34 PM, Manivannan Sadhasivam wrote: >>> On Thu, Feb 15, 2024 at 07:18:51PM +0530, Md Sadre Alam wrote: >>>> This series of patches will add initial supports >>>> for QPIC SPI NAND driver. >>>> >>>> Currently this driver support following commands >>>> >>>> -- RESET >>>> -- READ ID >>>> -- BLOCK ERASE >>>> -- PAGE READ >>>> -- PAGE WRITE >>>> -- GET FEATURE >>>> -- SET FEATURE >>>> -- BAD BLOCK CHECK >>>> >>>> This driver has been tested with dd command with read/write page >>>> with multiple file size 1MiB, 10MiB,40MiB etc. >>>> Also tested with "mtd" command like mtd erase, mtd write, mtd verify etc. >>>> >>> >>> This is not the first version isn't it? Where is the changelog describing what >>> has changed since then? >> >> The earlier patch was the RFC for design review only. > > RFC is state of patch, not version. This is v2 then. > > These RFC postings are really becoming mess. Some people make multiple > RFCs and then post v1 hiding entire previous history... And why even > bother with calling it RFC? Sorry, I was not aware of this. Shall I post the next one as V3 and add references to the RFC patch and this patch in the cover letter of V3? > > Best regards, > Krzysztof >
On 21/02/2024 11:34, Md Sadre Alam wrote: > > > On 2/20/2024 5:06 PM, Krzysztof Kozlowski wrote: >> On 20/02/2024 12:32, Md Sadre Alam wrote: >>> >>> >>> On 2/19/2024 6:34 PM, Manivannan Sadhasivam wrote: >>>> On Thu, Feb 15, 2024 at 07:18:51PM +0530, Md Sadre Alam wrote: >>>>> This series of patches will add initial supports >>>>> for QPIC SPI NAND driver. >>>>> >>>>> Currently this driver support following commands >>>>> >>>>> -- RESET >>>>> -- READ ID >>>>> -- BLOCK ERASE >>>>> -- PAGE READ >>>>> -- PAGE WRITE >>>>> -- GET FEATURE >>>>> -- SET FEATURE >>>>> -- BAD BLOCK CHECK >>>>> >>>>> This driver has been tested with dd command with read/write page >>>>> with multiple file size 1MiB, 10MiB,40MiB etc. >>>>> Also tested with "mtd" command like mtd erase, mtd write, mtd verify etc. >>>>> >>>> >>>> This is not the first version isn't it? Where is the changelog describing what >>>> has changed since then? >>> >>> The earlier patch was the RFC for design review only. >> >> RFC is state of patch, not version. This is v2 then. >> >> These RFC postings are really becoming mess. Some people make multiple >> RFCs and then post v1 hiding entire previous history... And why even >> bother with calling it RFC? > > Sorry, I was not aware of this. Shall I post the next one as V3 > and add references to the RFC patch and this patch in the cover > letter of V3? Yes, like with every posting. Best regards, Krzysztof
On 2/21/2024 4:31 PM, Krzysztof Kozlowski wrote: > On 21/02/2024 11:34, Md Sadre Alam wrote: >> >> >> On 2/20/2024 5:06 PM, Krzysztof Kozlowski wrote: >>> On 20/02/2024 12:32, Md Sadre Alam wrote: >>>> >>>> >>>> On 2/19/2024 6:34 PM, Manivannan Sadhasivam wrote: >>>>> On Thu, Feb 15, 2024 at 07:18:51PM +0530, Md Sadre Alam wrote: >>>>>> This series of patches will add initial supports >>>>>> for QPIC SPI NAND driver. >>>>>> >>>>>> Currently this driver support following commands >>>>>> >>>>>> -- RESET >>>>>> -- READ ID >>>>>> -- BLOCK ERASE >>>>>> -- PAGE READ >>>>>> -- PAGE WRITE >>>>>> -- GET FEATURE >>>>>> -- SET FEATURE >>>>>> -- BAD BLOCK CHECK >>>>>> >>>>>> This driver has been tested with dd command with read/write page >>>>>> with multiple file size 1MiB, 10MiB,40MiB etc. >>>>>> Also tested with "mtd" command like mtd erase, mtd write, mtd verify etc. >>>>>> >>>>> >>>>> This is not the first version isn't it? Where is the changelog describing what >>>>> has changed since then? >>>> >>>> The earlier patch was the RFC for design review only. >>> >>> RFC is state of patch, not version. This is v2 then. >>> >>> These RFC postings are really becoming mess. Some people make multiple >>> RFCs and then post v1 hiding entire previous history... And why even >>> bother with calling it RFC? >> >> Sorry, I was not aware of this. Shall I post the next one as V3 >> and add references to the RFC patch and this patch in the cover >> letter of V3? > > Yes, like with every posting. Thank you. > > Best regards, > Krzysztof >