Re: [U-Boot] [PATCH] driver/mtd: Add MICRON manufacturer id in spi framework
Hi Jagan,
> -Original Message-
> From: U-Boot [mailto:u-boot-boun...@lists.denx.de] On Behalf Of Ashish
> Kumar
> Sent: Wednesday, October 10, 2018 4:23 PM
> To: Jagan Teki
> Cc: U-Boot-Denx ; Jagan Teki ;
> Rajat Srivastava ; Suresh Gupta
>
> Subject: Re: [U-Boot] [PATCH] driver/mtd: Add MICRON manufacturer id in spi
> framework
>
> > -Original Message-
> > From: Jagan Teki
> > Sent: Wednesday, October 10, 2018 11:51 AM
> > To: Ashish Kumar
> > Cc: U-Boot-Denx ; Jagan Teki
> > ; Rajat Srivastava ;
> > Suresh Gupta
> > Subject: Re: [U-Boot] [PATCH] driver/mtd: Add MICRON manufacturer id
> > in spi framework
> >
> > On Tue, Sep 25, 2018 at 2:13 PM Ashish Kumar
> > wrote:
> > >
> > > Signed-off-by: Suresh Gupta
> > > Signed-off-by: Yogesh Gaur
> > > Signed-off-by: Ashish Kumar
> > > ---
> > > drivers/mtd/spi/sf_internal.h | 1 +
> > > drivers/mtd/spi/spi_flash.c | 2 ++
> > > 2 files changed, 3 insertions(+)
> > >
> > > diff --git a/drivers/mtd/spi/sf_internal.h
> > > b/drivers/mtd/spi/sf_internal.h index 4f63cacc64..26f5c7c995 100644
> > > --- a/drivers/mtd/spi/sf_internal.h
> > > +++ b/drivers/mtd/spi/sf_internal.h
> > > @@ -32,6 +32,7 @@ enum spi_nor_option_flags {
> > > /* CFI Manufacture ID's */
> > > #define SPI_FLASH_CFI_MFR_SPANSION 0x01
> > > #define SPI_FLASH_CFI_MFR_STMICRO 0x20
> > > +#define SPI_FLASH_CFI_MFR_MICRON 0x2C
> >
> > Why we need this? is it to support old micron chips?
>
> NOR flash name MT35X_QLKA and MT25Q_** used on NXP board has
> manufacturer id as 0x2C, which are rather for newer flashes after the split of
> Micron from ST.
>
Same is being discussed in Linux-mtd mailing list [1].
Public data sheet of MT35x flash can be checked at[2]
--
Regards
Yogesh Gaur.
[1] https://patchwork.ozlabs.org/patch/971439/
[2] https://www.micron.com/resource-details/0b74b806-bbf1-4c24-b07b-35e2799bb6ff
> Regards
> Ashish
> ___
> U-Boot mailing list
> U-Boot@lists.denx.de
> https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.de
> nx.de%2Flistinfo%2Fu-
> bootdata=02%7C01%7Cyogeshnarayan.gaur%40nxp.com%7Cd2c8d343b
> b8646a75c1b08d62e9eabc3%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7
> C0%7C636747656373729732sdata=58px6wLL96jaImJ6MGXC6rRHiXO3oe
> sj1EFfCrEmoGY%3Dreserved=0
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] driver: net: fsl-mc: updated copyright info
Patch rebased with top, v2 send. Yogesh Gaur -Original Message- From: York Sun Sent: Tuesday, May 8, 2018 9:19 PM To: Yogesh Narayan Gaur <yogeshnarayan.g...@nxp.com>; u-boot@lists.denx.de Subject: Re: [PATCH] driver: net: fsl-mc: updated copyright info On 03/15/2018 02:07 AM, Yogesh Narayan Gaur wrote: > Updated copyright info for the issues reported after running > check-legal test. > > Signed-off-by: Yogesh Gaur <yogeshnarayan.g...@nxp.com> > --- > Legally NXP and Freescale Semiconductor are same entity. Can you refresh this patch? It has conflict with Tom's recent change on SPDX. York ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] driver: fsl-mc: Perform fsl-mc fdt fixup for lazyapply dpl
> -Original Message-
> From: York Sun
> Sent: Monday, January 15, 2018 10:49 PM
> To: Yogesh Narayan Gaur <yogeshnarayan.g...@nxp.com>; u-
> b...@lists.denx.de
> Subject: Re: [PATCH] driver: fsl-mc: Perform fsl-mc fdt fixup for lazyapply
> dpl
>
> On 01/11/2018 09:19 PM, Yogesh Gaur wrote:
> > For for case of lazyapply method, API fdt_fixup_board_enet() gets
> > invoked before DPL being deployed.
> > This leads to an issue that fsl-mc fdt fixup status marked as fail and
> > dprc driver didn't get registered in linux boot.
> >
> > Fixes this issue by calling fdt_fixup_board_enet() for case when DPL
> > is deployed successfully in lazyapply method.
> >
> > Signed-off-by: Yogesh Gaur <yogeshnarayan.g...@nxp.com>
> > ---
> > drivers/net/fsl-mc/mc.c | 6 +-
> > include/fdt_support.h | 3 +++
> > 2 files changed, 8 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/net/fsl-mc/mc.c b/drivers/net/fsl-mc/mc.c index
> > f36fe06..ab2cce1 100644
> > --- a/drivers/net/fsl-mc/mc.c
> > +++ b/drivers/net/fsl-mc/mc.c
> > @@ -1415,7 +1415,11 @@ int fsl_mc_ldpaa_exit(bd_t *bd)
> > bool mc_boot_status = false;
> >
> > if (bd && mc_lazy_dpl_addr && !fsl_mc_ldpaa_exit(NULL)) {
> > - mc_apply_dpl(mc_lazy_dpl_addr);
> > + err = mc_apply_dpl(mc_lazy_dpl_addr); #ifdef
> CONFIG_FSL_MC_ENET
>
> Is this really needed? We have this in Makefile
> drivers/net/Makefile:72:obj-$(CONFIG_FSL_MC_ENET) += fsl-mc/
>
+#ifdef CONFIG_FSL_MC_ENET
Is not required, would send next version of this patch after removing #ifdef
CONFIG_FSL_MC_ENET
--
Regards
Yogesh Gaur
> York
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] mtd/spi: Add MT35XU512ABA1G12 NOR flash support
> -Original Message-
> From: Jagan Teki [mailto:jagannadh.t...@gmail.com]
> Sent: Tuesday, August 29, 2017 10:48 PM
> To: Yogesh Narayan Gaur <yogeshnarayan.g...@nxp.com>
> Cc: u-boot@lists.denx.de; York Sun <york@nxp.com>
> Subject: Re: [PATCH] mtd/spi: Add MT35XU512ABA1G12 NOR flash support
>
> On Tue, Aug 29, 2017 at 9:42 PM, Yogesh Narayan Gaur
> <yogeshnarayan.g...@nxp.com> wrote:
> >
> >
> >> -Original Message-
> >> From: Jagan Teki [mailto:jagannadh.t...@gmail.com]
> >> Sent: Friday, August 11, 2017 4:03 PM
> >> To: Yogesh Narayan Gaur <yogeshnarayan.g...@nxp.com>
> >> Cc: u-boot@lists.denx.de; York Sun <york@nxp.com>
> >> Subject: Re: [PATCH] mtd/spi: Add MT35XU512ABA1G12 NOR flash support
> >>
> >> On Fri, Aug 11, 2017 at 3:38 PM, Yogesh Narayan Gaur
> >> <yogeshnarayan.g...@nxp.com> wrote:
> >> >
> >> >
> >> >> -Original Message-
> >> >> From: Jagan Teki [mailto:jagannadh.t...@gmail.com]
> >> >> Sent: Friday, August 11, 2017 3:32 PM
> >> >> To: Yogesh Narayan Gaur <yogeshnarayan.g...@nxp.com>
> >> >> Cc: u-boot@lists.denx.de; York Sun <york@nxp.com>
> >> >> Subject: Re: [PATCH] mtd/spi: Add MT35XU512ABA1G12 NOR flash
> >> >> support
> >> >>
> >> >> On Fri, Aug 11, 2017 at 3:30 PM, Yogesh Narayan Gaur
> >> >> <yogeshnarayan.g...@nxp.com> wrote:
> >> >> >
> >> >> >
> >> >> >> -Original Message-
> >> >> >> From: Jagan Teki [mailto:jagannadh.t...@gmail.com]
> >> >> >> Sent: Friday, August 11, 2017 3:25 PM
> >> >> >> To: Yogesh Narayan Gaur <yogeshnarayan.g...@nxp.com>
> >> >> >> Cc: u-boot@lists.denx.de; York Sun <york@nxp.com>
> >> >> >> Subject: Re: [PATCH] mtd/spi: Add MT35XU512ABA1G12 NOR flash
> >> >> >> support
> >> >> >>
> >> >> >> On Tue, Aug 1, 2017 at 9:43 AM, Yogesh Gaur
> >> >> >> <yogeshnarayan.g...@nxp.com>
> >> >> >> wrote:
> >> >> >> > Add MT35XU512ABA1G12 parameters to NOR flash parameters
> array.
> >> >> >> > Since the manufactory ID is changed to 0x2C, add it for
> >> >> >> > micron and using it for relevant settings.
> >> >> >> >
> >> >> >> > The MT35XU512ABA1G12 only supports 1 bit mode and 8 bits. It
> >> >> >> > can't support dual and quad. Supports subsector erase with
> >> >> >> > 4KB granularity, have support of FSR(flag status register)
> >> >> >> > and flash size is
> >> 64MB.
> >> >> >> > ---
> >> >> >> > drivers/mtd/spi/sf_internal.h | 1 +
> >> >> >> > drivers/mtd/spi/spi_flash.c | 2 ++
> >> >> >> > drivers/mtd/spi/spi_flash_ids.c | 1 +
> >> >> >> > 3 files changed, 4 insertions(+)
> >> >> >> >
> >> >> >> > diff --git a/drivers/mtd/spi/sf_internal.h
> >> >> >> > b/drivers/mtd/spi/sf_internal.h index 839cdbe..f46cb3a 100644
> >> >> >> > --- a/drivers/mtd/spi/sf_internal.h
> >> >> >> > +++ b/drivers/mtd/spi/sf_internal.h
> >> >> >> > @@ -33,6 +33,7 @@ enum spi_nor_option_flags {
> >> >> >> > /* CFI Manufacture ID's */
> >> >> >> > #define SPI_FLASH_CFI_MFR_SPANSION 0x01
> >> >> >> > #define SPI_FLASH_CFI_MFR_STMICRO 0x20
> >> >> >> > +#define SPI_FLASH_CFI_MFR_MICRON 0x2C
> >> >> >>
> >> >> >> Does this new mfr id from micro? so 0x20 still exists?
> >> >> >>
> >> >> > On older Micron flash like MT25QU512ABB mfr id still is 0x20 but
> >> >> > in newer
> >> >> flash, MT35X, it's been changed to 0x2c.
> >> >>
> >> >> OK, so the flash features like quad ennoblements, opcodes remains same?
> >> >> except the mfr?
> >> >>
> >> > This flash support single bit and octal bit cmds no dual or quad support.
> >> > Opcode for read/write/erase and other register operation related
> >> > cmds are
> >> similar to already supported flashes like MT25QU512ABB(n25q512a in
> >> existing table).
> >> > Only mfr id has been modified.
> >>
> >> OK.
> >
> > Did this patch needs to be modified further or can be upstreamed?
>
> Yes, ie what I commented in previous thread.
>
Updated the patch and sent on mailing list again.
--
Thanks
> thanks!
> --
> Jagan Teki
> Free Software Engineer | www.openedev.com U-Boot, Linux | Upstream
> Maintainer Hyderabad, India.
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] mtd/spi: Add MT35XU512ABA1G12 NOR flash support
> -Original Message-
> From: Jagan Teki [mailto:jagannadh.t...@gmail.com]
> Sent: Friday, August 11, 2017 4:03 PM
> To: Yogesh Narayan Gaur <yogeshnarayan.g...@nxp.com>
> Cc: u-boot@lists.denx.de; York Sun <york@nxp.com>
> Subject: Re: [PATCH] mtd/spi: Add MT35XU512ABA1G12 NOR flash support
>
> On Fri, Aug 11, 2017 at 3:38 PM, Yogesh Narayan Gaur
> <yogeshnarayan.g...@nxp.com> wrote:
> >
> >
> >> -Original Message-
> >> From: Jagan Teki [mailto:jagannadh.t...@gmail.com]
> >> Sent: Friday, August 11, 2017 3:32 PM
> >> To: Yogesh Narayan Gaur <yogeshnarayan.g...@nxp.com>
> >> Cc: u-boot@lists.denx.de; York Sun <york@nxp.com>
> >> Subject: Re: [PATCH] mtd/spi: Add MT35XU512ABA1G12 NOR flash support
> >>
> >> On Fri, Aug 11, 2017 at 3:30 PM, Yogesh Narayan Gaur
> >> <yogeshnarayan.g...@nxp.com> wrote:
> >> >
> >> >
> >> >> -Original Message-
> >> >> From: Jagan Teki [mailto:jagannadh.t...@gmail.com]
> >> >> Sent: Friday, August 11, 2017 3:25 PM
> >> >> To: Yogesh Narayan Gaur <yogeshnarayan.g...@nxp.com>
> >> >> Cc: u-boot@lists.denx.de; York Sun <york@nxp.com>
> >> >> Subject: Re: [PATCH] mtd/spi: Add MT35XU512ABA1G12 NOR flash
> >> >> support
> >> >>
> >> >> On Tue, Aug 1, 2017 at 9:43 AM, Yogesh Gaur
> >> >> <yogeshnarayan.g...@nxp.com>
> >> >> wrote:
> >> >> > Add MT35XU512ABA1G12 parameters to NOR flash parameters array.
> >> >> > Since the manufactory ID is changed to 0x2C, add it for micron
> >> >> > and using it for relevant settings.
> >> >> >
> >> >> > The MT35XU512ABA1G12 only supports 1 bit mode and 8 bits. It
> >> >> > can't support dual and quad. Supports subsector erase with 4KB
> >> >> > granularity, have support of FSR(flag status register) and flash size
> >> >> > is
> 64MB.
> >> >> > ---
> >> >> > drivers/mtd/spi/sf_internal.h | 1 +
> >> >> > drivers/mtd/spi/spi_flash.c | 2 ++
> >> >> > drivers/mtd/spi/spi_flash_ids.c | 1 +
> >> >> > 3 files changed, 4 insertions(+)
> >> >> >
> >> >> > diff --git a/drivers/mtd/spi/sf_internal.h
> >> >> > b/drivers/mtd/spi/sf_internal.h index 839cdbe..f46cb3a 100644
> >> >> > --- a/drivers/mtd/spi/sf_internal.h
> >> >> > +++ b/drivers/mtd/spi/sf_internal.h
> >> >> > @@ -33,6 +33,7 @@ enum spi_nor_option_flags {
> >> >> > /* CFI Manufacture ID's */
> >> >> > #define SPI_FLASH_CFI_MFR_SPANSION 0x01
> >> >> > #define SPI_FLASH_CFI_MFR_STMICRO 0x20
> >> >> > +#define SPI_FLASH_CFI_MFR_MICRON 0x2C
> >> >>
> >> >> Does this new mfr id from micro? so 0x20 still exists?
> >> >>
> >> > On older Micron flash like MT25QU512ABB mfr id still is 0x20 but in
> >> > newer
> >> flash, MT35X, it's been changed to 0x2c.
> >>
> >> OK, so the flash features like quad ennoblements, opcodes remains same?
> >> except the mfr?
> >>
> > This flash support single bit and octal bit cmds no dual or quad support.
> > Opcode for read/write/erase and other register operation related cmds are
> similar to already supported flashes like MT25QU512ABB(n25q512a in existing
> table).
> > Only mfr id has been modified.
>
> OK.
Did this patch needs to be modified further or can be upstreamed?
>
> And better remove the new CONFIG_ macro since the both chips have equal
> functionalities and anyway changes should managed by id table.
> the chip you defined under STMICRO ifdef :)
>
> thanks!
> --
> Jagan Teki
> Free Software Engineer | www.openedev.com U-Boot, Linux | Upstream
> Maintainer Hyderabad, India.
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] mtd/spi: Add MT35XU512ABA1G12 NOR flash support
> -Original Message-
> From: Jagan Teki [mailto:jagannadh.t...@gmail.com]
> Sent: Friday, August 11, 2017 3:32 PM
> To: Yogesh Narayan Gaur <yogeshnarayan.g...@nxp.com>
> Cc: u-boot@lists.denx.de; York Sun <york@nxp.com>
> Subject: Re: [PATCH] mtd/spi: Add MT35XU512ABA1G12 NOR flash support
>
> On Fri, Aug 11, 2017 at 3:30 PM, Yogesh Narayan Gaur
> <yogeshnarayan.g...@nxp.com> wrote:
> >
> >
> >> -Original Message-
> >> From: Jagan Teki [mailto:jagannadh.t...@gmail.com]
> >> Sent: Friday, August 11, 2017 3:25 PM
> >> To: Yogesh Narayan Gaur <yogeshnarayan.g...@nxp.com>
> >> Cc: u-boot@lists.denx.de; York Sun <york@nxp.com>
> >> Subject: Re: [PATCH] mtd/spi: Add MT35XU512ABA1G12 NOR flash support
> >>
> >> On Tue, Aug 1, 2017 at 9:43 AM, Yogesh Gaur
> >> <yogeshnarayan.g...@nxp.com>
> >> wrote:
> >> > Add MT35XU512ABA1G12 parameters to NOR flash parameters array.
> >> > Since the manufactory ID is changed to 0x2C, add it for micron and
> >> > using it for relevant settings.
> >> >
> >> > The MT35XU512ABA1G12 only supports 1 bit mode and 8 bits. It can't
> >> > support dual and quad. Supports subsector erase with 4KB
> >> > granularity, have support of FSR(flag status register) and flash size is
> >> > 64MB.
> >> > ---
> >> > drivers/mtd/spi/sf_internal.h | 1 +
> >> > drivers/mtd/spi/spi_flash.c | 2 ++
> >> > drivers/mtd/spi/spi_flash_ids.c | 1 +
> >> > 3 files changed, 4 insertions(+)
> >> >
> >> > diff --git a/drivers/mtd/spi/sf_internal.h
> >> > b/drivers/mtd/spi/sf_internal.h index 839cdbe..f46cb3a 100644
> >> > --- a/drivers/mtd/spi/sf_internal.h
> >> > +++ b/drivers/mtd/spi/sf_internal.h
> >> > @@ -33,6 +33,7 @@ enum spi_nor_option_flags {
> >> > /* CFI Manufacture ID's */
> >> > #define SPI_FLASH_CFI_MFR_SPANSION 0x01
> >> > #define SPI_FLASH_CFI_MFR_STMICRO 0x20
> >> > +#define SPI_FLASH_CFI_MFR_MICRON 0x2C
> >>
> >> Does this new mfr id from micro? so 0x20 still exists?
> >>
> > On older Micron flash like MT25QU512ABB mfr id still is 0x20 but in newer
> flash, MT35X, it's been changed to 0x2c.
>
> OK, so the flash features like quad ennoblements, opcodes remains same?
> except the mfr?
>
This flash support single bit and octal bit cmds no dual or quad support.
Opcode for read/write/erase and other register operation related cmds are
similar to already supported flashes like MT25QU512ABB(n25q512a in existing
table).
Only mfr id has been modified.
--
Regards
Yogesh Gaur.
> thanks!
> --
> Jagan Teki
> Free Software Engineer | www.openedev.com U-Boot, Linux | Upstream
> Maintainer Hyderabad, India.
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] mtd/spi: Add MT35XU512ABA1G12 NOR flash support
> -Original Message-
> From: Jagan Teki [mailto:jagannadh.t...@gmail.com]
> Sent: Friday, August 11, 2017 3:25 PM
> To: Yogesh Narayan Gaur <yogeshnarayan.g...@nxp.com>
> Cc: u-boot@lists.denx.de; York Sun <york@nxp.com>
> Subject: Re: [PATCH] mtd/spi: Add MT35XU512ABA1G12 NOR flash support
>
> On Tue, Aug 1, 2017 at 9:43 AM, Yogesh Gaur <yogeshnarayan.g...@nxp.com>
> wrote:
> > Add MT35XU512ABA1G12 parameters to NOR flash parameters array. Since
> > the manufactory ID is changed to 0x2C, add it for micron and using it
> > for relevant settings.
> >
> > The MT35XU512ABA1G12 only supports 1 bit mode and 8 bits. It can't
> > support dual and quad. Supports subsector erase with 4KB granularity,
> > have support of FSR(flag status register) and flash size is 64MB.
> > ---
> > drivers/mtd/spi/sf_internal.h | 1 +
> > drivers/mtd/spi/spi_flash.c | 2 ++
> > drivers/mtd/spi/spi_flash_ids.c | 1 +
> > 3 files changed, 4 insertions(+)
> >
> > diff --git a/drivers/mtd/spi/sf_internal.h
> > b/drivers/mtd/spi/sf_internal.h index 839cdbe..f46cb3a 100644
> > --- a/drivers/mtd/spi/sf_internal.h
> > +++ b/drivers/mtd/spi/sf_internal.h
> > @@ -33,6 +33,7 @@ enum spi_nor_option_flags {
> > /* CFI Manufacture ID's */
> > #define SPI_FLASH_CFI_MFR_SPANSION 0x01
> > #define SPI_FLASH_CFI_MFR_STMICRO 0x20
> > +#define SPI_FLASH_CFI_MFR_MICRON 0x2C
>
> Does this new mfr id from micro? so 0x20 still exists?
>
On older Micron flash like MT25QU512ABB mfr id still is 0x20 but in newer
flash, MT35X, it's been changed to 0x2c.
--
Regards
Yogesh Gaur.
> thanks!
> --
> Jagan Teki
> Free Software Engineer | www.openedev.com U-Boot, Linux | Upstream
> Maintainer Hyderabad, India.
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH 1/1] driver: net: fsl-mc: fsl_mc_ldpaa_exit exit earlier if dpl applied
Ack-ed
--
Yogesh Gaur
> -Original Message-
> From: York Sun
> Sent: Tuesday, August 08, 2017 2:43 AM
> To: Yogesh Narayan Gaur <yogeshnarayan.g...@nxp.com>; Priyanka Jain
> <priyanka.j...@nxp.com>
> Cc: Santan Kumar <santan.ku...@nxp.com>; u-boot@lists.denx.de; Prabhakar
> Kushwaha <prabhakar.kushw...@nxp.com>
> Subject: Re: [PATCH 1/1] driver: net: fsl-mc: fsl_mc_ldpaa_exit exit earlier
> if dpl
> applied
>
> On 06/28/2017 10:47 PM, Santan Kumar wrote:
> > In fsl_mc_ldpaa_exit(), in case of mc is booted and dpl is applied, it
> > should return earlier without executing
> > dpbp_exit()
> >
> > Signed-off-by: Santan Kumar <santan.ku...@nxp.com>
> > ---
> > This piece of code is mistakenly removed in below patch.
> >
> > https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpat
> >
> chwork.ozlabs.org%2Fpatch%2F756038%2F=01%7C01%7Cyork.sun%40nx
> p.co
> >
> m%7C67ffe48248b14183512708d4beb2434e%7C686ea1d3bc2b4c6fa92cd99c5c
> 30163
> >
> 5%7C0=PvzNEpCqJ8MOLT9vRSNuNLTeH9hjxLxQi97ngEkrXf0%3D
> ed=0
> >
> > drivers/net/fsl-mc/mc.c | 11 +--
> > 1 file changed, 9 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/net/fsl-mc/mc.c b/drivers/net/fsl-mc/mc.c index
> > 8bf25c7..3a30c03 100644
> > --- a/drivers/net/fsl-mc/mc.c
> > +++ b/drivers/net/fsl-mc/mc.c
> > @@ -1336,14 +1336,18 @@ int fsl_mc_ldpaa_exit(bd_t *bd)
> > {
> > int err = 0;
> > bool is_dpl_apply_status = false;
> > + bool mc_boot_status = false;
> >
> > if (bd && mc_lazy_dpl_addr && !fsl_mc_ldpaa_exit(NULL)) {
> > mc_apply_dpl(mc_lazy_dpl_addr);
> > mc_lazy_dpl_addr = 0;
> > }
> >
> > + if (!get_mc_boot_status())
> > + mc_boot_status = true;
> > +
> > /* MC is not loaded intentionally, So return success. */
> > - if (bd && get_mc_boot_status() != 0)
> > + if (bd && !mc_boot_status)
> > return 0;
> >
> > /* If DPL is deployed, set is_dpl_apply_status as TRUE. */ @@
> > -1354,11 +1358,14 @@ int fsl_mc_ldpaa_exit(bd_t *bd)
> > * For case MC is loaded but DPL is not deployed, return success and
> > * print message on console. Else FDT fix-up code execution hanged.
> > */
> > - if (bd && !get_mc_boot_status() && !is_dpl_apply_status) {
> > + if (bd && mc_boot_status && !is_dpl_apply_status) {
> > printf("fsl-mc: DPL not deployed, DPAA2 ethernet not work\n");
> > return 0;
> > }
> >
> > + if (bd && mc_boot_status && is_dpl_apply_status)
> > + return 0;
> > +
> > err = dpbp_exit();
> > if (err < 0) {
> > printf("dpbp_exit() failed: %d\n", err);
> >
>
> Yogesh and Priyanka,
>
> Please review this patch. Thanks.
>
> York
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
