[v3,00/13] Port manufacturer specific initialization
mbox series

Message ID 20220721064435.2456601-1-dario.binacchi@amarulasolutions.com
Headers show
Series
  • Port manufacturer specific initialization
Related show

Message

Dario Binacchi July 21, 2022, 6:44 a.m. UTC
In preparation of re-sync of mtd stack, we opt to move the current stack
slowly in order to have a more easy sync and test. We would like to
prepare uboot to support no-jedec and no-onfi compliant nand so we need
to clean up a bit the code we have now and upstream some of the support.
In this series we expect no functional change

Tested on:
 - imx6ull Micron   MT29F2G08ABAGAH4
 - imx8mn  Macronix MX30LF4G18AC

Changes in v3:
- Use commit sha1 with 13 digits.
- Use commit sha1 with 13 digits.
- Fix code style warnings raised by patman.
- Use commit sha1 with 13 digits.
- Fix code style warnings raised by patman.
- Fix code style warnings raised by patman.
- Use commit sha1 with 13 digits.
- Fix code style warnings raised by patman.
- Add the SPDX-License-Identifier tag.
- Use commit sha1 with 13 digits.
- Add the SPDX-License-Identifier tag.
- Fix code style warnings raised by patman.
- Use commit sha1 with 13 digits.
- Add the SPDX-License-Identifier tag.
- Fix code style warnings raised by patman.
- Use commit sha1 with 13 digits.
- Add the SPDX-License-Identifier tag.
- Fix code style warnings raised by patman.
- Use commit sha1 with 13 digits.
- Add the SPDX-License-Identifier tag.
- Fix code style warnings raised by patman.
- Use commit sha1 with 13 digits.
- Add the SPDX-License-Identifier tag.
- Fix code style warnings raised by patman.
- Use commit sha1 with 13 digits.
- Use commit sha1 with 13 digits.
- Wrap commit description to a maximum of 75 chars.

Changes in v2:
- Use short-commit form.
- Remove linux info. Uboot seems that backport without add this
  extra information.
- Use short-commit form.
- Remove linux info. Uboot seems that backport without add this
  extra information.
- Use short-commit form.
- Remove linux info. Uboot seems that backport without add this
  extra information.
- Use short-commit form.
- Remove linux info. Uboot seems that backport without add this
  extra information.
- Adjust the include file in nand_samsung.
- Use short-commit form
- Remove linux info. Uboot seems that backport without add this
  extra information.
- Adjust the include file in nand_hynix.
- Use short-commit form
- Remove linux info. Uboot seems that backport without add this
  extra information.
- Adjust the include file in nand_toshiba.
- Use short-commit form
- Remove linux info. Uboot seems that backport without add this
  extra information.
- Adjust the include file in nand_micron.
- Use short-commit form
- Remove linux info. Uboot seems that backport without add this
  extra information.
- Adjust the include file in nand_amd.
- Use short-commit form
- Remove linux info. Uboot seems that backport without add this
  extra information.
- Adjust the include file in nand_macronix.
- Use short-commit form
- Remove linux info. Uboot seems that backport without add this
  extra information.
- Use short-commit form.
- Remove linux info. Uboot seems that backport without add this
  extra information.

Michael Trimarchi (13):
  mtd: nand: Get rid of busw parameter
  mtd: nand: Store nand ID in struct nand_chip
  mtd: nand: Add manufacturer specific initialization/detection steps
  mtd: nand: Get rid of mtd variable in function calls
  mtd: nand: Export symbol nand_decode_ext_id
  mtd: nand: Move Samsung specific init/detection logic in
    nand_samsung.c
  mtd: nand: Move Hynix specific init/detection logic in nand_hynix.c
  mtd: nand: Move Toshiba specific init/detection logic in
    nand_toshiba.c
  mtd: nand: Move Micron specific init logic in nand_micron.c
  mtd: nand: Move AMD/Spansion specific init/detection logic in
    nand_amd.c
  mtd: nand: Move Macronix specific initialization in nand_macronix.c
  mtd: nand: toshiba: Retrieve ECC requirements from extended ID
  mtd: decommission the NAND museum

 drivers/mtd/nand/raw/Makefile        |  10 +-
 drivers/mtd/nand/raw/nand_amd.c      |  52 ++++
 drivers/mtd/nand/raw/nand_base.c     | 370 ++++++++++-----------------
 drivers/mtd/nand/raw/nand_hynix.c    |  85 ++++++
 drivers/mtd/nand/raw/nand_ids.c      |  24 +-
 drivers/mtd/nand/raw/nand_macronix.c |  31 +++
 drivers/mtd/nand/raw/nand_micron.c   |  87 +++++++
 drivers/mtd/nand/raw/nand_samsung.c  |  90 +++++++
 drivers/mtd/nand/raw/nand_toshiba.c  |  79 ++++++
 include/linux/mtd/rawnand.h          |  78 ++++--
 10 files changed, 625 insertions(+), 281 deletions(-)
 create mode 100644 drivers/mtd/nand/raw/nand_amd.c
 create mode 100644 drivers/mtd/nand/raw/nand_hynix.c
 create mode 100644 drivers/mtd/nand/raw/nand_macronix.c
 create mode 100644 drivers/mtd/nand/raw/nand_micron.c
 create mode 100644 drivers/mtd/nand/raw/nand_samsung.c
 create mode 100644 drivers/mtd/nand/raw/nand_toshiba.c

Comments

Michael Trimarchi July 21, 2022, 11:15 a.m. UTC | #1
Hi Dario

On Thu, Jul 21, 2022 at 8:44 AM Dario Binacchi
<dario.binacchi@amarulasolutions.com> wrote:
>
> In preparation of re-sync of mtd stack, we opt to move the current stack
> slowly in order to have a more easy sync and test. We would like to
> prepare uboot to support no-jedec and no-onfi compliant nand so we need
> to clean up a bit the code we have now and upstream some of the support.
> In this series we expect no functional change
>
> Tested on:
>  - imx6ull Micron   MT29F2G08ABAGAH4
>  - imx8mn  Macronix MX30LF4G18AC
>

Thank you for V3 and clean up

> Changes in v3:
> - Use commit sha1 with 13 digits.
> - Use commit sha1 with 13 digits.
> - Fix code style warnings raised by patman.
> - Use commit sha1 with 13 digits.
> - Fix code style warnings raised by patman.
> - Fix code style warnings raised by patman.
> - Use commit sha1 with 13 digits.
> - Fix code style warnings raised by patman.
> - Add the SPDX-License-Identifier tag.
> - Use commit sha1 with 13 digits.
> - Add the SPDX-License-Identifier tag.
> - Fix code style warnings raised by patman.
> - Use commit sha1 with 13 digits.
> - Add the SPDX-License-Identifier tag.
> - Fix code style warnings raised by patman.
> - Use commit sha1 with 13 digits.
> - Add the SPDX-License-Identifier tag.
> - Fix code style warnings raised by patman.
> - Use commit sha1 with 13 digits.
> - Add the SPDX-License-Identifier tag.
> - Fix code style warnings raised by patman.
> - Use commit sha1 with 13 digits.
> - Add the SPDX-License-Identifier tag.
> - Fix code style warnings raised by patman.
> - Use commit sha1 with 13 digits.
> - Use commit sha1 with 13 digits.
> - Wrap commit description to a maximum of 75 chars.
>
> Changes in v2:
> - Use short-commit form.
> - Remove linux info. Uboot seems that backport without add this
>   extra information.
> - Use short-commit form.
> - Remove linux info. Uboot seems that backport without add this
>   extra information.
> - Use short-commit form.
> - Remove linux info. Uboot seems that backport without add this
>   extra information.
> - Use short-commit form.
> - Remove linux info. Uboot seems that backport without add this
>   extra information.
> - Adjust the include file in nand_samsung.
> - Use short-commit form
> - Remove linux info. Uboot seems that backport without add this
>   extra information.
> - Adjust the include file in nand_hynix.
> - Use short-commit form
> - Remove linux info. Uboot seems that backport without add this
>   extra information.
> - Adjust the include file in nand_toshiba.
> - Use short-commit form
> - Remove linux info. Uboot seems that backport without add this
>   extra information.
> - Adjust the include file in nand_micron.
> - Use short-commit form
> - Remove linux info. Uboot seems that backport without add this
>   extra information.
> - Adjust the include file in nand_amd.
> - Use short-commit form
> - Remove linux info. Uboot seems that backport without add this
>   extra information.
> - Adjust the include file in nand_macronix.
> - Use short-commit form
> - Remove linux info. Uboot seems that backport without add this
>   extra information.
> - Use short-commit form.
> - Remove linux info. Uboot seems that backport without add this
>   extra information.
>
> Michael Trimarchi (13):
>   mtd: nand: Get rid of busw parameter
>   mtd: nand: Store nand ID in struct nand_chip
>   mtd: nand: Add manufacturer specific initialization/detection steps
>   mtd: nand: Get rid of mtd variable in function calls
>   mtd: nand: Export symbol nand_decode_ext_id
>   mtd: nand: Move Samsung specific init/detection logic in
>     nand_samsung.c
>   mtd: nand: Move Hynix specific init/detection logic in nand_hynix.c
>   mtd: nand: Move Toshiba specific init/detection logic in
>     nand_toshiba.c
>   mtd: nand: Move Micron specific init logic in nand_micron.c
>   mtd: nand: Move AMD/Spansion specific init/detection logic in
>     nand_amd.c
>   mtd: nand: Move Macronix specific initialization in nand_macronix.c
>   mtd: nand: toshiba: Retrieve ECC requirements from extended ID
>   mtd: decommission the NAND museum
>
>  drivers/mtd/nand/raw/Makefile        |  10 +-
>  drivers/mtd/nand/raw/nand_amd.c      |  52 ++++
>  drivers/mtd/nand/raw/nand_base.c     | 370 ++++++++++-----------------
>  drivers/mtd/nand/raw/nand_hynix.c    |  85 ++++++
>  drivers/mtd/nand/raw/nand_ids.c      |  24 +-
>  drivers/mtd/nand/raw/nand_macronix.c |  31 +++
>  drivers/mtd/nand/raw/nand_micron.c   |  87 +++++++
>  drivers/mtd/nand/raw/nand_samsung.c  |  90 +++++++
>  drivers/mtd/nand/raw/nand_toshiba.c  |  79 ++++++
>  include/linux/mtd/rawnand.h          |  78 ++++--
>  10 files changed, 625 insertions(+), 281 deletions(-)
>  create mode 100644 drivers/mtd/nand/raw/nand_amd.c
>  create mode 100644 drivers/mtd/nand/raw/nand_hynix.c
>  create mode 100644 drivers/mtd/nand/raw/nand_macronix.c
>  create mode 100644 drivers/mtd/nand/raw/nand_micron.c
>  create mode 100644 drivers/mtd/nand/raw/nand_samsung.c
>  create mode 100644 drivers/mtd/nand/raw/nand_toshiba.c
>
> --
> 2.32.0
>

Applied, thanks.



--
Michael Nazzareno Trimarchi
Co-Founder & Chief Executive Officer
M. +39 347 913 2170
michael@amarulasolutions.com