Re: [U-Boot] [PATCH 5/8] warp7: defconfig: Switch on IMX7 pinctrl for both ports

2019-01-09 Thread Bryan O'Donoghue



On 08/01/2019 13:26, Peng Fan wrote:






-Original Message-
From: Bryan O'Donoghue [mailto:bryan.odonog...@linaro.org]
Sent: 2019年1月3日 9:45
To: u-boot@lists.denx.de
Cc: Fabio Estevam ; Peng Fan
; Bryan O'Donoghue ;
Stefano Babic 
Subject: [PATCH 5/8] warp7: defconfig: Switch on IMX7 pinctrl for both
ports

Switches on the IMX7 pinctrl driver for the warp7 and warp7_bl33
ports, necessary to convert over to DM for this board.

Signed-off-by: Bryan O'Donoghue 
Cc: Peng Fan 
Cc: Fabio Estevam 
Cc: Stefano Babic 
---
  configs/warp7_bl33_defconfig | 2 ++
  configs/warp7_defconfig  | 2 ++
  2 files changed, 4 insertions(+)

diff --git a/configs/warp7_bl33_defconfig
b/configs/warp7_bl33_defconfig index b2f943c775..a2c2555315 100644
--- a/configs/warp7_bl33_defconfig
+++ b/configs/warp7_bl33_defconfig
@@ -41,3 +41,5 @@ CONFIG_USB_ETH_CDC=y
CONFIG_USBNET_HOST_ADDR="de:ad:be:af:00:00"
  CONFIG_OPTEE_TZDRAM_SIZE=0x200
  CONFIG_DEFAULT_DEVICE_TREE="imx7s-warp"
+CONFIG_PINCTRL=y
+CONFIG_PINCTRL_IMX7=y
diff --git a/configs/warp7_defconfig b/configs/warp7_defconfig index
65d801ffe2..8691d00491 100644
--- a/configs/warp7_defconfig
+++ b/configs/warp7_defconfig
@@ -54,3 +54,5 @@ CONFIG_OPTEE_TZDRAM_SIZE=0x300
  CONFIG_OPTEE_TZDRAM_BASE=0x9d00
  CONFIG_BOOTM_OPTEE=y
  CONFIG_DEFAULT_DEVICE_TREE="imx7s-warp"
+CONFIG_PINCTRL=y
+CONFIG_PINCTRL_IMX7=y


Reviewed-by: Peng Fan 


Just have a follow up question,
in patch 4/8, DM MMC is enabled, but in 5/8 pinctrl DM is enabled.
This might be not correct, the DM MMC/PINCTRL/GPIO needs to be in one patch.
The fsl_esdhc DM support does not support DM MMC with legacy iomux code.


Hmm.

OK, then I'll squash these.
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


Re: [U-Boot] [PATCH 5/8] warp7: defconfig: Switch on IMX7 pinctrl for both ports

2019-01-08 Thread Peng Fan

> 
> 
> > -Original Message-
> > From: Bryan O'Donoghue [mailto:bryan.odonog...@linaro.org]
> > Sent: 2019年1月3日 9:45
> > To: u-boot@lists.denx.de
> > Cc: Fabio Estevam ; Peng Fan
> > ; Bryan O'Donoghue ;
> > Stefano Babic 
> > Subject: [PATCH 5/8] warp7: defconfig: Switch on IMX7 pinctrl for both
> > ports
> >
> > Switches on the IMX7 pinctrl driver for the warp7 and warp7_bl33
> > ports, necessary to convert over to DM for this board.
> >
> > Signed-off-by: Bryan O'Donoghue 
> > Cc: Peng Fan 
> > Cc: Fabio Estevam 
> > Cc: Stefano Babic 
> > ---
> >  configs/warp7_bl33_defconfig | 2 ++
> >  configs/warp7_defconfig  | 2 ++
> >  2 files changed, 4 insertions(+)
> >
> > diff --git a/configs/warp7_bl33_defconfig
> > b/configs/warp7_bl33_defconfig index b2f943c775..a2c2555315 100644
> > --- a/configs/warp7_bl33_defconfig
> > +++ b/configs/warp7_bl33_defconfig
> > @@ -41,3 +41,5 @@ CONFIG_USB_ETH_CDC=y
> > CONFIG_USBNET_HOST_ADDR="de:ad:be:af:00:00"
> >  CONFIG_OPTEE_TZDRAM_SIZE=0x200
> >  CONFIG_DEFAULT_DEVICE_TREE="imx7s-warp"
> > +CONFIG_PINCTRL=y
> > +CONFIG_PINCTRL_IMX7=y
> > diff --git a/configs/warp7_defconfig b/configs/warp7_defconfig index
> > 65d801ffe2..8691d00491 100644
> > --- a/configs/warp7_defconfig
> > +++ b/configs/warp7_defconfig
> > @@ -54,3 +54,5 @@ CONFIG_OPTEE_TZDRAM_SIZE=0x300
> >  CONFIG_OPTEE_TZDRAM_BASE=0x9d00
> >  CONFIG_BOOTM_OPTEE=y
> >  CONFIG_DEFAULT_DEVICE_TREE="imx7s-warp"
> > +CONFIG_PINCTRL=y
> > +CONFIG_PINCTRL_IMX7=y
> 
> Reviewed-by: Peng Fan 

Just have a follow up question,
in patch 4/8, DM MMC is enabled, but in 5/8 pinctrl DM is enabled.
This might be not correct, the DM MMC/PINCTRL/GPIO needs to be in one patch.
The fsl_esdhc DM support does not support DM MMC with legacy iomux code.

Regards,
Peng.
> 
> > --
> > 2.20.0
> 
> ___
> U-Boot mailing list
> U-Boot@lists.denx.de
> https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.
> denx.de%2Flistinfo%2Fu-bootdata=02%7C01%7CPeng.Fan%40nxp.co
> m%7Cf6fbfc5331eb4328683308d6756c7489%7C686ea1d3bc2b4c6fa92cd99c
> 5c301635%7C0%7C0%7C636825506006750873sdata=iXlXGGewCrW%
> 2B4srkP4d%2BWnZ7UX2AyCK8Tgx93g3SpF8%3Dreserved=0
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


Re: [U-Boot] [PATCH 5/8] warp7: defconfig: Switch on IMX7 pinctrl for both ports

2019-01-08 Thread Peng Fan


> -Original Message-
> From: Bryan O'Donoghue [mailto:bryan.odonog...@linaro.org]
> Sent: 2019年1月3日 9:45
> To: u-boot@lists.denx.de
> Cc: Fabio Estevam ; Peng Fan
> ; Bryan O'Donoghue ;
> Stefano Babic 
> Subject: [PATCH 5/8] warp7: defconfig: Switch on IMX7 pinctrl for both ports
> 
> Switches on the IMX7 pinctrl driver for the warp7 and warp7_bl33 ports,
> necessary to convert over to DM for this board.
> 
> Signed-off-by: Bryan O'Donoghue 
> Cc: Peng Fan 
> Cc: Fabio Estevam 
> Cc: Stefano Babic 
> ---
>  configs/warp7_bl33_defconfig | 2 ++
>  configs/warp7_defconfig  | 2 ++
>  2 files changed, 4 insertions(+)
> 
> diff --git a/configs/warp7_bl33_defconfig b/configs/warp7_bl33_defconfig
> index b2f943c775..a2c2555315 100644
> --- a/configs/warp7_bl33_defconfig
> +++ b/configs/warp7_bl33_defconfig
> @@ -41,3 +41,5 @@ CONFIG_USB_ETH_CDC=y
>  CONFIG_USBNET_HOST_ADDR="de:ad:be:af:00:00"
>  CONFIG_OPTEE_TZDRAM_SIZE=0x200
>  CONFIG_DEFAULT_DEVICE_TREE="imx7s-warp"
> +CONFIG_PINCTRL=y
> +CONFIG_PINCTRL_IMX7=y
> diff --git a/configs/warp7_defconfig b/configs/warp7_defconfig index
> 65d801ffe2..8691d00491 100644
> --- a/configs/warp7_defconfig
> +++ b/configs/warp7_defconfig
> @@ -54,3 +54,5 @@ CONFIG_OPTEE_TZDRAM_SIZE=0x300
>  CONFIG_OPTEE_TZDRAM_BASE=0x9d00
>  CONFIG_BOOTM_OPTEE=y
>  CONFIG_DEFAULT_DEVICE_TREE="imx7s-warp"
> +CONFIG_PINCTRL=y
> +CONFIG_PINCTRL_IMX7=y

Reviewed-by: Peng Fan 

> --
> 2.20.0

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


[U-Boot] [PATCH 5/8] warp7: defconfig: Switch on IMX7 pinctrl for both ports

2019-01-02 Thread Bryan O'Donoghue
Switches on the IMX7 pinctrl driver for the warp7 and warp7_bl33 ports,
necessary to convert over to DM for this board.

Signed-off-by: Bryan O'Donoghue 
Cc: Peng Fan 
Cc: Fabio Estevam 
Cc: Stefano Babic 
---
 configs/warp7_bl33_defconfig | 2 ++
 configs/warp7_defconfig  | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/configs/warp7_bl33_defconfig b/configs/warp7_bl33_defconfig
index b2f943c775..a2c2555315 100644
--- a/configs/warp7_bl33_defconfig
+++ b/configs/warp7_bl33_defconfig
@@ -41,3 +41,5 @@ CONFIG_USB_ETH_CDC=y
 CONFIG_USBNET_HOST_ADDR="de:ad:be:af:00:00"
 CONFIG_OPTEE_TZDRAM_SIZE=0x200
 CONFIG_DEFAULT_DEVICE_TREE="imx7s-warp"
+CONFIG_PINCTRL=y
+CONFIG_PINCTRL_IMX7=y
diff --git a/configs/warp7_defconfig b/configs/warp7_defconfig
index 65d801ffe2..8691d00491 100644
--- a/configs/warp7_defconfig
+++ b/configs/warp7_defconfig
@@ -54,3 +54,5 @@ CONFIG_OPTEE_TZDRAM_SIZE=0x300
 CONFIG_OPTEE_TZDRAM_BASE=0x9d00
 CONFIG_BOOTM_OPTEE=y
 CONFIG_DEFAULT_DEVICE_TREE="imx7s-warp"
+CONFIG_PINCTRL=y
+CONFIG_PINCTRL_IMX7=y
-- 
2.20.0

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot