All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH 1/3] imx: remove board specific boot order from spl
@ 2019-08-23 16:35 Parthiban Nallathambi
  2019-08-23 16:35 ` [U-Boot] [PATCH 2/3] imx: initialize fec only when enabled Parthiban Nallathambi
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Parthiban Nallathambi @ 2019-08-23 16:35 UTC (permalink / raw)
  To: u-boot

boot order was added to handle both SD and eMMC. But commit
14d319b1 introduced to handle both eMMC and SD globally.

Signed-off-by: Parthiban Nallathambi <pn@denx.de>
---
 board/phytec/pcl063/spl.c | 23 -----------------------
 1 file changed, 23 deletions(-)

diff --git a/board/phytec/pcl063/spl.c b/board/phytec/pcl063/spl.c
index 6d4c827918..18fc251890 100644
--- a/board/phytec/pcl063/spl.c
+++ b/board/phytec/pcl063/spl.c
@@ -182,29 +182,6 @@ int board_mmc_init(bd_t *bis)
 
 	return 0;
 }
-
-void board_boot_order(u32 *spl_boot_list)
-{
-	u32 bmode = imx6_src_get_boot_mode();
-	u8 boot_dev = BOOT_DEVICE_MMC1;
-
-	switch ((bmode & IMX6_BMODE_MASK) >> IMX6_BMODE_SHIFT) {
-	case IMX6_BMODE_SD:
-	case IMX6_BMODE_ESD:
-		boot_dev = BOOT_DEVICE_MMC1;
-		break;
-	case IMX6_BMODE_MMC:
-	case IMX6_BMODE_EMMC:
-		boot_dev = BOOT_DEVICE_MMC2;
-		break;
-	default:
-		/* Default - BOOT_DEVICE_MMC1 */
-		printf("Wrong board boot order\n");
-		break;
-	}
-
-	spl_boot_list[0] = boot_dev;
-}
 #endif /* CONFIG_FSL_ESDHC_IMX */
 
 void board_init_f(ulong dummy)
-- 
2.21.0

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [U-Boot] [PATCH 2/3] imx: initialize fec only when enabled
  2019-08-23 16:35 [U-Boot] [PATCH 1/3] imx: remove board specific boot order from spl Parthiban Nallathambi
@ 2019-08-23 16:35 ` Parthiban Nallathambi
  2019-08-26  8:26   ` Peng Fan
  2019-08-23 16:35 ` [U-Boot] [PATCH 3/3] imx: nandbcb: include long help " Parthiban Nallathambi
  2019-08-26  8:24 ` [U-Boot] [PATCH 1/3] imx: remove board specific boot order from spl Peng Fan
  2 siblings, 1 reply; 6+ messages in thread
From: Parthiban Nallathambi @ 2019-08-23 16:35 UTC (permalink / raw)
  To: u-boot

board early initialize fec ethernet controller pinmux
only when FEC is enabled

Signed-off-by: Parthiban Nallathambi <pn@denx.de>
---
 board/phytec/pcl063/pcl063.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/board/phytec/pcl063/pcl063.c b/board/phytec/pcl063/pcl063.c
index f8cbd1c11e..96dd9e38f3 100644
--- a/board/phytec/pcl063/pcl063.c
+++ b/board/phytec/pcl063/pcl063.c
@@ -178,7 +178,9 @@ int board_phy_config(struct phy_device *phydev)
 int board_early_init_f(void)
 {
 	setup_iomux_uart();
+#ifdef CONFIG_FEC_MXC
 	setup_iomux_fec();
+#endif
 
 	return 0;
 }
-- 
2.21.0

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [U-Boot] [PATCH 3/3] imx: nandbcb: include long help only when enabled
  2019-08-23 16:35 [U-Boot] [PATCH 1/3] imx: remove board specific boot order from spl Parthiban Nallathambi
  2019-08-23 16:35 ` [U-Boot] [PATCH 2/3] imx: initialize fec only when enabled Parthiban Nallathambi
@ 2019-08-23 16:35 ` Parthiban Nallathambi
  2019-08-26  8:26   ` Peng Fan
  2019-08-26  8:24 ` [U-Boot] [PATCH 1/3] imx: remove board specific boot order from spl Peng Fan
  2 siblings, 1 reply; 6+ messages in thread
From: Parthiban Nallathambi @ 2019-08-23 16:35 UTC (permalink / raw)
  To: u-boot

conditionally include long help text when enabled

Signed-off-by: Parthiban Nallathambi <pn@denx.de>
---
 arch/arm/mach-imx/cmd_nandbcb.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm/mach-imx/cmd_nandbcb.c b/arch/arm/mach-imx/cmd_nandbcb.c
index 065b814b2e..7811c61d22 100644
--- a/arch/arm/mach-imx/cmd_nandbcb.c
+++ b/arch/arm/mach-imx/cmd_nandbcb.c
@@ -359,9 +359,11 @@ usage:
 	return CMD_RET_USAGE;
 }
 
+#ifdef CONFIG_SYS_LONGHELP
 static char nandbcb_help_text[] =
 	"update addr off|partition len	- update 'len' bytes starting at\n"
 	"	'off|part' to memory address 'addr', skipping  bad blocks";
+#endif
 
 U_BOOT_CMD(nandbcb, 5, 1, do_nandbcb,
 	   "i.MX6 Nand BCB",
-- 
2.21.0

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [U-Boot] [PATCH 1/3] imx: remove board specific boot order from spl
  2019-08-23 16:35 [U-Boot] [PATCH 1/3] imx: remove board specific boot order from spl Parthiban Nallathambi
  2019-08-23 16:35 ` [U-Boot] [PATCH 2/3] imx: initialize fec only when enabled Parthiban Nallathambi
  2019-08-23 16:35 ` [U-Boot] [PATCH 3/3] imx: nandbcb: include long help " Parthiban Nallathambi
@ 2019-08-26  8:24 ` Peng Fan
  2 siblings, 0 replies; 6+ messages in thread
From: Peng Fan @ 2019-08-26  8:24 UTC (permalink / raw)
  To: u-boot

> Subject: [PATCH 1/3] imx: remove board specific boot order from spl
> 
> boot order was added to handle both SD and eMMC. But commit
> 14d319b1 introduced to handle both eMMC and SD globally.
> 
> Signed-off-by: Parthiban Nallathambi <pn@denx.de>
> ---
>  board/phytec/pcl063/spl.c | 23 -----------------------
>  1 file changed, 23 deletions(-)
> 
> diff --git a/board/phytec/pcl063/spl.c b/board/phytec/pcl063/spl.c index
> 6d4c827918..18fc251890 100644
> --- a/board/phytec/pcl063/spl.c
> +++ b/board/phytec/pcl063/spl.c
> @@ -182,29 +182,6 @@ int board_mmc_init(bd_t *bis)
> 
>  	return 0;
>  }
> -
> -void board_boot_order(u32 *spl_boot_list) -{
> -	u32 bmode = imx6_src_get_boot_mode();
> -	u8 boot_dev = BOOT_DEVICE_MMC1;
> -
> -	switch ((bmode & IMX6_BMODE_MASK) >> IMX6_BMODE_SHIFT) {
> -	case IMX6_BMODE_SD:
> -	case IMX6_BMODE_ESD:
> -		boot_dev = BOOT_DEVICE_MMC1;
> -		break;
> -	case IMX6_BMODE_MMC:
> -	case IMX6_BMODE_EMMC:
> -		boot_dev = BOOT_DEVICE_MMC2;
> -		break;
> -	default:
> -		/* Default - BOOT_DEVICE_MMC1 */
> -		printf("Wrong board boot order\n");
> -		break;
> -	}
> -
> -	spl_boot_list[0] = boot_dev;
> -}
>  #endif /* CONFIG_FSL_ESDHC_IMX */
> 
>  void board_init_f(ulong dummy)

Reviewed-by: Peng Fan <peng.fan@nxp.com>

> --
> 2.21.0

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [U-Boot] [PATCH 2/3] imx: initialize fec only when enabled
  2019-08-23 16:35 ` [U-Boot] [PATCH 2/3] imx: initialize fec only when enabled Parthiban Nallathambi
@ 2019-08-26  8:26   ` Peng Fan
  0 siblings, 0 replies; 6+ messages in thread
From: Peng Fan @ 2019-08-26  8:26 UTC (permalink / raw)
  To: u-boot

> Subject: [PATCH 2/3] imx: initialize fec only when enabled
> 
> board early initialize fec ethernet controller pinmux only when FEC is enabled
> 
> Signed-off-by: Parthiban Nallathambi <pn@denx.de>
> ---
>  board/phytec/pcl063/pcl063.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/board/phytec/pcl063/pcl063.c b/board/phytec/pcl063/pcl063.c
> index f8cbd1c11e..96dd9e38f3 100644
> --- a/board/phytec/pcl063/pcl063.c
> +++ b/board/phytec/pcl063/pcl063.c
> @@ -178,7 +178,9 @@ int board_phy_config(struct phy_device *phydev)
> int board_early_init_f(void)  {
>  	setup_iomux_uart();
> +#ifdef CONFIG_FEC_MXC
>  	setup_iomux_fec();
> +#endif
> 

Reviewed-by: Peng Fan <peng.fan@nxp.com>

It would be better if dm pinctrl used.

Regards,
Peng.

>  	return 0;
>  }
> --
> 2.21.0

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [U-Boot] [PATCH 3/3] imx: nandbcb: include long help only when enabled
  2019-08-23 16:35 ` [U-Boot] [PATCH 3/3] imx: nandbcb: include long help " Parthiban Nallathambi
@ 2019-08-26  8:26   ` Peng Fan
  0 siblings, 0 replies; 6+ messages in thread
From: Peng Fan @ 2019-08-26  8:26 UTC (permalink / raw)
  To: u-boot

> Subject: [PATCH 3/3] imx: nandbcb: include long help only when enabled
> 
> conditionally include long help text when enabled
> 
> Signed-off-by: Parthiban Nallathambi <pn@denx.de>
> ---
>  arch/arm/mach-imx/cmd_nandbcb.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/arch/arm/mach-imx/cmd_nandbcb.c
> b/arch/arm/mach-imx/cmd_nandbcb.c index 065b814b2e..7811c61d22
> 100644
> --- a/arch/arm/mach-imx/cmd_nandbcb.c
> +++ b/arch/arm/mach-imx/cmd_nandbcb.c
> @@ -359,9 +359,11 @@ usage:
>  	return CMD_RET_USAGE;
>  }
> 
> +#ifdef CONFIG_SYS_LONGHELP
>  static char nandbcb_help_text[] =
>  	"update addr off|partition len	- update 'len' bytes starting at\n"
>  	"	'off|part' to memory address 'addr', skipping  bad blocks";
> +#endif
> 
>  U_BOOT_CMD(nandbcb, 5, 1, do_nandbcb,
>  	   "i.MX6 Nand BCB",
> --

Reviewed-by: Peng Fan <peng.fan@nxp.com>

> 2.21.0

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2019-08-26  8:26 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-23 16:35 [U-Boot] [PATCH 1/3] imx: remove board specific boot order from spl Parthiban Nallathambi
2019-08-23 16:35 ` [U-Boot] [PATCH 2/3] imx: initialize fec only when enabled Parthiban Nallathambi
2019-08-26  8:26   ` Peng Fan
2019-08-23 16:35 ` [U-Boot] [PATCH 3/3] imx: nandbcb: include long help " Parthiban Nallathambi
2019-08-26  8:26   ` Peng Fan
2019-08-26  8:24 ` [U-Boot] [PATCH 1/3] imx: remove board specific boot order from spl Peng Fan

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.