mbox series

[v3,0/4] Add support for Qcom SMEM based NAND parser

Message ID 20201119071308.9292-1-manivannan.sadhasivam@linaro.org
Headers show
Series Add support for Qcom SMEM based NAND parser | expand

Message

Manivannan Sadhasivam Nov. 19, 2020, 7:13 a.m. UTC
Hello,

This series adds support for parsing the partitions defined in Shared
Memory (SMEM) of the Qualcomm platforms supporting NAND interface.
Current parser only supports V3 and V4 of the partition tables.

This series has been tested on SDX55 MTP board which has an onboard NAND
device.

Thanks,
Mani

Changes in v3:

* Handled le32 parameters in parser, collected review from Linus W.

Changes in v2:

* Added additionalProperties: false and removed unit address in binding

Manivannan Sadhasivam (4):
  dt-bindings: mtd: partitions: Add binding for Qcom SMEM parser
  mtd: parsers: Add Qcom SMEM parser
  mtd: rawnand: qcom: Add support for Qcom SMEM parser
  mtd: parsers: afs: Fix freeing the part name memory in failure

 .../mtd/partitions/qcom,smem-part.yaml        |  33 ++++
 drivers/mtd/nand/raw/qcom_nandc.c             |   4 +-
 drivers/mtd/parsers/Kconfig                   |   8 +
 drivers/mtd/parsers/Makefile                  |   1 +
 drivers/mtd/parsers/afs.c                     |   4 +-
 drivers/mtd/parsers/qcomsmempart.c            | 170 ++++++++++++++++++
 6 files changed, 216 insertions(+), 4 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/mtd/partitions/qcom,smem-part.yaml
 create mode 100644 drivers/mtd/parsers/qcomsmempart.c

Comments

Manivannan Sadhasivam Dec. 10, 2020, 4:09 a.m. UTC | #1
Hi,

On Thu, Nov 19, 2020 at 12:43:04PM +0530, Manivannan Sadhasivam wrote:
> Hello,
> 
> This series adds support for parsing the partitions defined in Shared
> Memory (SMEM) of the Qualcomm platforms supporting NAND interface.
> Current parser only supports V3 and V4 of the partition tables.
> 
> This series has been tested on SDX55 MTP board which has an onboard NAND
> device.
> 

Any update on this series?

Thanks,
Mani

> Thanks,
> Mani
> 
> Changes in v3:
> 
> * Handled le32 parameters in parser, collected review from Linus W.
> 
> Changes in v2:
> 
> * Added additionalProperties: false and removed unit address in binding
> 
> Manivannan Sadhasivam (4):
>   dt-bindings: mtd: partitions: Add binding for Qcom SMEM parser
>   mtd: parsers: Add Qcom SMEM parser
>   mtd: rawnand: qcom: Add support for Qcom SMEM parser
>   mtd: parsers: afs: Fix freeing the part name memory in failure
> 
>  .../mtd/partitions/qcom,smem-part.yaml        |  33 ++++
>  drivers/mtd/nand/raw/qcom_nandc.c             |   4 +-
>  drivers/mtd/parsers/Kconfig                   |   8 +
>  drivers/mtd/parsers/Makefile                  |   1 +
>  drivers/mtd/parsers/afs.c                     |   4 +-
>  drivers/mtd/parsers/qcomsmempart.c            | 170 ++++++++++++++++++
>  6 files changed, 216 insertions(+), 4 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/mtd/partitions/qcom,smem-part.yaml
>  create mode 100644 drivers/mtd/parsers/qcomsmempart.c
> 
> -- 
> 2.17.1
>
Miquel Raynal Dec. 10, 2020, 9:24 p.m. UTC | #2
Hi Manivannan,

Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> wrote on Thu,
10 Dec 2020 09:39:42 +0530:

> Hi,
> 
> On Thu, Nov 19, 2020 at 12:43:04PM +0530, Manivannan Sadhasivam wrote:
> > Hello,
> > 
> > This series adds support for parsing the partitions defined in Shared
> > Memory (SMEM) of the Qualcomm platforms supporting NAND interface.
> > Current parser only supports V3 and V4 of the partition tables.
> > 
> > This series has been tested on SDX55 MTP board which has an onboard NAND
> > device.
> >   
> 
> Any update on this series?

Patches lgtm, I think Rob's Ack is still missing, let's wait for his
review.


Thanks,
Miquèl
Rob Herring (Arm) Dec. 11, 2020, 3:10 a.m. UTC | #3
On Thu, Dec 10, 2020 at 10:24:47PM +0100, Miquel Raynal wrote:
> Hi Manivannan,
> 
> Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> wrote on Thu,
> 10 Dec 2020 09:39:42 +0530:
> 
> > Hi,
> > 
> > On Thu, Nov 19, 2020 at 12:43:04PM +0530, Manivannan Sadhasivam wrote:
> > > Hello,
> > > 
> > > This series adds support for parsing the partitions defined in Shared
> > > Memory (SMEM) of the Qualcomm platforms supporting NAND interface.
> > > Current parser only supports V3 and V4 of the partition tables.
> > > 
> > > This series has been tested on SDX55 MTP board which has an onboard NAND
> > > device.
> > >   
> > 
> > Any update on this series?
> 
> Patches lgtm, I think Rob's Ack is still missing, let's wait for his
> review.

Perhaps answer my reply on Dec 7.

Rob
Manivannan Sadhasivam Dec. 11, 2020, 3:36 a.m. UTC | #4
On Thu, Dec 10, 2020 at 09:10:48PM -0600, Rob Herring wrote:
> On Thu, Dec 10, 2020 at 10:24:47PM +0100, Miquel Raynal wrote:
> > Hi Manivannan,
> > 
> > Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> wrote on Thu,
> > 10 Dec 2020 09:39:42 +0530:
> > 
> > > Hi,
> > > 
> > > On Thu, Nov 19, 2020 at 12:43:04PM +0530, Manivannan Sadhasivam wrote:
> > > > Hello,
> > > > 
> > > > This series adds support for parsing the partitions defined in Shared
> > > > Memory (SMEM) of the Qualcomm platforms supporting NAND interface.
> > > > Current parser only supports V3 and V4 of the partition tables.
> > > > 
> > > > This series has been tested on SDX55 MTP board which has an onboard NAND
> > > > device.
> > > >   
> > > 
> > > Any update on this series?
> > 
> > Patches lgtm, I think Rob's Ack is still missing, let's wait for his
> > review.
> 
> Perhaps answer my reply on Dec 7.
> 

Sorry missed that... Just replied.

Thanks,
Mani

> Rob