* [U-Boot] [PATCH] configs: spi: Miss the SPI_FLASH_BAR for ESPI
@ 2019-10-21 9:02 Xiaowei Bao
2019-10-22 9:45 ` Priyanka Jain
0 siblings, 1 reply; 5+ messages in thread
From: Xiaowei Bao @ 2019-10-21 9:02 UTC (permalink / raw)
To: u-boot
Miss the SPI_FLASH_BAR for the ESPI controller of FSL.
Signed-off-by: Xiaowei Bao <xiaowei.bao@nxp.com>
---
drivers/spi/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
index b8ca2bd..7003569 100644
--- a/drivers/spi/Kconfig
+++ b/drivers/spi/Kconfig
@@ -367,6 +367,7 @@ config CF_SPI
config FSL_ESPI
bool "Freescale eSPI driver"
+ imply SPI_FLASH_BAR
help
Enable the Freescale eSPI driver. This driver can be used to
access the SPI interface and SPI NOR flash on platforms embedding
--
2.9.5
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] configs: spi: Miss the SPI_FLASH_BAR for ESPI
2019-10-21 9:02 [U-Boot] [PATCH] configs: spi: Miss the SPI_FLASH_BAR for ESPI Xiaowei Bao
@ 2019-10-22 9:45 ` Priyanka Jain
2019-10-22 9:51 ` Xiaowei Bao
0 siblings, 1 reply; 5+ messages in thread
From: Priyanka Jain @ 2019-10-22 9:45 UTC (permalink / raw)
To: u-boot
>-----Original Message-----
>From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Xiaowei Bao
>Sent: Monday, October 21, 2019 2:32 PM
>To: Jagdish Gediya <jagdish.gediya@nxp.com>; jagan at amarulasolutions.com;
>u-boot at lists.denx.de
>Cc: Xiaowei Bao <xiaowei.bao@nxp.com>
>Subject: [U-Boot] [PATCH] configs: spi: Miss the SPI_FLASH_BAR for ESPI
configs: spi: -> spi: Kconfig
I understand you are adding config, because you have missed it earlier.
But better would be something like Miss->Add
ESPI -> FSL_ESPI
>
>Miss the SPI_FLASH_BAR for the ESPI controller of FSL.
>
>Signed-off-by: Xiaowei Bao <xiaowei.bao@nxp.com>
>---
> drivers/spi/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
>diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig index b8ca2bd..7003569
>100644
>--- a/drivers/spi/Kconfig
>+++ b/drivers/spi/Kconfig
>@@ -367,6 +367,7 @@ config CF_SPI
>
> config FSL_ESPI
> bool "Freescale eSPI driver"
>+ imply SPI_FLASH_BAR
> help
> Enable the Freescale eSPI driver. This driver can be used to
> access the SPI interface and SPI NOR flash on platforms embedding
>--
--priyankajain
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] configs: spi: Miss the SPI_FLASH_BAR for ESPI
2019-10-22 9:45 ` Priyanka Jain
@ 2019-10-22 9:51 ` Xiaowei Bao
2019-10-23 19:39 ` Jagan Teki
0 siblings, 1 reply; 5+ messages in thread
From: Xiaowei Bao @ 2019-10-22 9:51 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Priyanka Jain <priyanka.jain@nxp.com>
> Sent: 2019年10月22日 17:45
> To: Xiaowei Bao <xiaowei.bao@nxp.com>; Jagdish Gediya
> <jagdish.gediya@nxp.com>; jagan at amarulasolutions.com;
> u-boot at lists.denx.de
> Cc: Xiaowei Bao <xiaowei.bao@nxp.com>
> Subject: RE: [U-Boot] [PATCH] configs: spi: Miss the SPI_FLASH_BAR for ESPI
>
>
>
> >-----Original Message-----
> >From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Xiaowei Bao
> >Sent: Monday, October 21, 2019 2:32 PM
> >To: Jagdish Gediya <jagdish.gediya@nxp.com>;
> >jagan at amarulasolutions.com; u-boot at lists.denx.de
> >Cc: Xiaowei Bao <xiaowei.bao@nxp.com>
> >Subject: [U-Boot] [PATCH] configs: spi: Miss the SPI_FLASH_BAR for ESPI
> configs: spi: -> spi: Kconfig
>
> I understand you are adding config, because you have missed it earlier.
> But better would be something like Miss->Add
Thanks for your comments, this CONFIG option is missed by Jagan Teki, so I add it
in Kconfig file, otherwise, the ESPI will not work in some PPC board.
Thanks
Xiaowei
>
> ESPI -> FSL_ESPI
> >
> >Miss the SPI_FLASH_BAR for the ESPI controller of FSL.
> >
> >Signed-off-by: Xiaowei Bao <xiaowei.bao@nxp.com>
> >---
> > drivers/spi/Kconfig | 1 +
> > 1 file changed, 1 insertion(+)
> >
> >diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig index
> >b8ca2bd..7003569
> >100644
> >--- a/drivers/spi/Kconfig
> >+++ b/drivers/spi/Kconfig
> >@@ -367,6 +367,7 @@ config CF_SPI
> >
> > config FSL_ESPI
> > bool "Freescale eSPI driver"
> >+ imply SPI_FLASH_BAR
> > help
> > Enable the Freescale eSPI driver. This driver can be used to
> > access the SPI interface and SPI NOR flash on platforms embedding
> >--
>
> --priyankajain
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] configs: spi: Miss the SPI_FLASH_BAR for ESPI
2019-10-22 9:51 ` Xiaowei Bao
@ 2019-10-23 19:39 ` Jagan Teki
2019-10-25 2:42 ` Xiaowei Bao
0 siblings, 1 reply; 5+ messages in thread
From: Jagan Teki @ 2019-10-23 19:39 UTC (permalink / raw)
To: u-boot
On Tue, Oct 22, 2019 at 3:21 PM Xiaowei Bao <xiaowei.bao@nxp.com> wrote:
>
>
>
> > -----Original Message-----
> > From: Priyanka Jain <priyanka.jain@nxp.com>
> > Sent: 2019年10月22日 17:45
> > To: Xiaowei Bao <xiaowei.bao@nxp.com>; Jagdish Gediya
> > <jagdish.gediya@nxp.com>; jagan at amarulasolutions.com;
> > u-boot at lists.denx.de
> > Cc: Xiaowei Bao <xiaowei.bao@nxp.com>
> > Subject: RE: [U-Boot] [PATCH] configs: spi: Miss the SPI_FLASH_BAR for ESPI
> >
> >
> >
> > >-----Original Message-----
> > >From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Xiaowei Bao
> > >Sent: Monday, October 21, 2019 2:32 PM
> > >To: Jagdish Gediya <jagdish.gediya@nxp.com>;
> > >jagan at amarulasolutions.com; u-boot at lists.denx.de
> > >Cc: Xiaowei Bao <xiaowei.bao@nxp.com>
> > >Subject: [U-Boot] [PATCH] configs: spi: Miss the SPI_FLASH_BAR for ESPI
> > configs: spi: -> spi: Kconfig
> >
> > I understand you are adding config, because you have missed it earlier.
> > But better would be something like Miss->Add
>
> Thanks for your comments, this CONFIG option is missed by Jagan Teki, so I add it
> in Kconfig file, otherwise, the ESPI will not work in some PPC board.
Did I miss while moving around? please mark the commit where it missed
it would help to track where the issue comes from.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] configs: spi: Miss the SPI_FLASH_BAR for ESPI
2019-10-23 19:39 ` Jagan Teki
@ 2019-10-25 2:42 ` Xiaowei Bao
0 siblings, 0 replies; 5+ messages in thread
From: Xiaowei Bao @ 2019-10-25 2:42 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Jagan Teki <jagan@amarulasolutions.com>
> Sent: 2019年10月24日 3:40
> To: Xiaowei Bao <xiaowei.bao@nxp.com>
> Cc: Priyanka Jain <priyanka.jain@nxp.com>; Jagdish Gediya
> <jagdish.gediya@nxp.com>; u-boot at lists.denx.de
> Subject: Re: [U-Boot] [PATCH] configs: spi: Miss the SPI_FLASH_BAR for ESPI
>
> On Tue, Oct 22, 2019 at 3:21 PM Xiaowei Bao <xiaowei.bao@nxp.com>
> wrote:
> >
> >
> >
> > > -----Original Message-----
> > > From: Priyanka Jain <priyanka.jain@nxp.com>
> > > Sent: 2019年10月22日 17:45
> > > To: Xiaowei Bao <xiaowei.bao@nxp.com>; Jagdish Gediya
> > > <jagdish.gediya@nxp.com>; jagan at amarulasolutions.com;
> > > u-boot at lists.denx.de
> > > Cc: Xiaowei Bao <xiaowei.bao@nxp.com>
> > > Subject: RE: [U-Boot] [PATCH] configs: spi: Miss the SPI_FLASH_BAR
> > > for ESPI
> > >
> > >
> > >
> > > >-----Original Message-----
> > > >From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Xiaowei
> > > >Bao
> > > >Sent: Monday, October 21, 2019 2:32 PM
> > > >To: Jagdish Gediya <jagdish.gediya@nxp.com>;
> > > >jagan at amarulasolutions.com; u-boot at lists.denx.de
> > > >Cc: Xiaowei Bao <xiaowei.bao@nxp.com>
> > > >Subject: [U-Boot] [PATCH] configs: spi: Miss the SPI_FLASH_BAR for
> > > >ESPI
> > > configs: spi: -> spi: Kconfig
> > >
> > > I understand you are adding config, because you have missed it earlier.
> > > But better would be something like Miss->Add
> >
> > Thanks for your comments, this CONFIG option is missed by Jagan Teki,
> > so I add it in Kconfig file, otherwise, the ESPI will not work in some PPC
> board.
>
> Did I miss while moving around? please mark the commit where it missed it
> would help to track where the issue comes from.
Oh, sorry, it is missed by Vignesh R, the commit is 6d82517836418f984b7b4c05cf1427d7b49b1169
Thanks
Xiaowei
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-10-25 2:42 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-21 9:02 [U-Boot] [PATCH] configs: spi: Miss the SPI_FLASH_BAR for ESPI Xiaowei Bao
2019-10-22 9:45 ` Priyanka Jain
2019-10-22 9:51 ` Xiaowei Bao
2019-10-23 19:39 ` Jagan Teki
2019-10-25 2:42 ` Xiaowei Bao
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.