diff mbox

[U-Boot,v4,08/10] config: Remove usage of CONFIG_STORAGE_EMMC

Message ID 20160830190629.13956-9-afd@ti.com
State Accepted
Delegated to: Tom Rini
Headers show

Commit Message

Andrew Davis Aug. 30, 2016, 7:06 p.m. UTC
This config option seems to be unused and is probably vestigial.
Remove it.

Signed-off-by: Andrew F. Davis <afd@ti.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
---
 include/configs/am335x_evm.h  | 2 --
 include/configs/am335x_shc.h  | 2 --
 include/configs/am335x_sl50.h | 2 --
 include/configs/bav335x.h     | 2 --
 4 files changed, 8 deletions(-)

Comments

Lokesh Vutla Aug. 31, 2016, 3:28 a.m. UTC | #1
On Wednesday 31 August 2016 12:36 AM, Andrew F. Davis wrote:
> This config option seems to be unused and is probably vestigial.
> Remove it.
> 
> Signed-off-by: Andrew F. Davis <afd@ti.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>

Acked-by: Lokesh Vutla <lokeshvutla@ti.com>

Thanks and regards,
Lokesh

> ---
>  include/configs/am335x_evm.h  | 2 --
>  include/configs/am335x_shc.h  | 2 --
>  include/configs/am335x_sl50.h | 2 --
>  include/configs/bav335x.h     | 2 --
>  4 files changed, 8 deletions(-)
> 
> diff --git a/include/configs/am335x_evm.h b/include/configs/am335x_evm.h
> index ba4c215..23ee75c 100644
> --- a/include/configs/am335x_evm.h
> +++ b/include/configs/am335x_evm.h
> @@ -301,8 +301,6 @@
>  #define CONFIG_FASTBOOT_BUF_ADDR	CONFIG_SYS_LOAD_ADDR
>  #define CONFIG_FASTBOOT_BUF_SIZE	0x07000000
>  
> -/* To support eMMC booting */
> -#define CONFIG_STORAGE_EMMC
>  #define CONFIG_FASTBOOT_FLASH_MMC_DEV   1
>  #endif
>  
> diff --git a/include/configs/am335x_shc.h b/include/configs/am335x_shc.h
> index f2484cb..8708c91 100644
> --- a/include/configs/am335x_shc.h
> +++ b/include/configs/am335x_shc.h
> @@ -287,8 +287,6 @@
>  #define CONFIG_SPL_LDSCRIPT		"$(CPUDIR)/am33xx/u-boot-spl.lds"
>  
>  #ifndef CONFIG_SPL_USBETH_SUPPORT
> -/* To support eMMC booting */
> -#define CONFIG_STORAGE_EMMC
>  #define CONFIG_FASTBOOT_FLASH_MMC_DEV   1
>  #endif
>  
> diff --git a/include/configs/am335x_sl50.h b/include/configs/am335x_sl50.h
> index 8454872..06ce4e9 100644
> --- a/include/configs/am335x_sl50.h
> +++ b/include/configs/am335x_sl50.h
> @@ -91,8 +91,6 @@
>  #define CONFIG_SPL_LDSCRIPT		"$(CPUDIR)/am33xx/u-boot-spl.lds"
>  
>  #ifndef CONFIG_SPL_USBETH_SUPPORT
> -/* To support eMMC booting */
> -#define CONFIG_STORAGE_EMMC
>  #define CONFIG_FASTBOOT_FLASH_MMC_DEV   1
>  #endif
>  
> diff --git a/include/configs/bav335x.h b/include/configs/bav335x.h
> index e391c91..05fc6d9 100644
> --- a/include/configs/bav335x.h
> +++ b/include/configs/bav335x.h
> @@ -451,8 +451,6 @@ DEFAULT_LINUX_BOOT_ENV \
>  #define CONFIG_FASTBOOT_BUF_ADDR	CONFIG_SYS_LOAD_ADDR
>  #define CONFIG_FASTBOOT_BUF_SIZE	0x07000000
>  
> -/* To support eMMC booting */
> -#define CONFIG_STORAGE_EMMC
>  #define CONFIG_FASTBOOT_FLASH_MMC_DEV   1
>  #endif
>  
>
Tom Rini Sept. 2, 2016, 2:54 p.m. UTC | #2
On Tue, Aug 30, 2016 at 02:06:27PM -0500, Andrew F. Davis wrote:

> This config option seems to be unused and is probably vestigial.
> Remove it.
> 
> Signed-off-by: Andrew F. Davis <afd@ti.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>

Reviewed-by: Tom Rini <trini@konsulko.com>
Tom Rini Oct. 3, 2016, 1:38 p.m. UTC | #3
On Tue, Aug 30, 2016 at 02:06:27PM -0500, Andrew F. Davis wrote:

> This config option seems to be unused and is probably vestigial.
> Remove it.
> 
> Signed-off-by: Andrew F. Davis <afd@ti.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>
> Acked-by: Lokesh Vutla <lokeshvutla@ti.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>

Applied to u-boot/master, thanks!
diff mbox

Patch

diff --git a/include/configs/am335x_evm.h b/include/configs/am335x_evm.h
index ba4c215..23ee75c 100644
--- a/include/configs/am335x_evm.h
+++ b/include/configs/am335x_evm.h
@@ -301,8 +301,6 @@ 
 #define CONFIG_FASTBOOT_BUF_ADDR	CONFIG_SYS_LOAD_ADDR
 #define CONFIG_FASTBOOT_BUF_SIZE	0x07000000
 
-/* To support eMMC booting */
-#define CONFIG_STORAGE_EMMC
 #define CONFIG_FASTBOOT_FLASH_MMC_DEV   1
 #endif
 
diff --git a/include/configs/am335x_shc.h b/include/configs/am335x_shc.h
index f2484cb..8708c91 100644
--- a/include/configs/am335x_shc.h
+++ b/include/configs/am335x_shc.h
@@ -287,8 +287,6 @@ 
 #define CONFIG_SPL_LDSCRIPT		"$(CPUDIR)/am33xx/u-boot-spl.lds"
 
 #ifndef CONFIG_SPL_USBETH_SUPPORT
-/* To support eMMC booting */
-#define CONFIG_STORAGE_EMMC
 #define CONFIG_FASTBOOT_FLASH_MMC_DEV   1
 #endif
 
diff --git a/include/configs/am335x_sl50.h b/include/configs/am335x_sl50.h
index 8454872..06ce4e9 100644
--- a/include/configs/am335x_sl50.h
+++ b/include/configs/am335x_sl50.h
@@ -91,8 +91,6 @@ 
 #define CONFIG_SPL_LDSCRIPT		"$(CPUDIR)/am33xx/u-boot-spl.lds"
 
 #ifndef CONFIG_SPL_USBETH_SUPPORT
-/* To support eMMC booting */
-#define CONFIG_STORAGE_EMMC
 #define CONFIG_FASTBOOT_FLASH_MMC_DEV   1
 #endif
 
diff --git a/include/configs/bav335x.h b/include/configs/bav335x.h
index e391c91..05fc6d9 100644
--- a/include/configs/bav335x.h
+++ b/include/configs/bav335x.h
@@ -451,8 +451,6 @@  DEFAULT_LINUX_BOOT_ENV \
 #define CONFIG_FASTBOOT_BUF_ADDR	CONFIG_SYS_LOAD_ADDR
 #define CONFIG_FASTBOOT_BUF_SIZE	0x07000000
 
-/* To support eMMC booting */
-#define CONFIG_STORAGE_EMMC
 #define CONFIG_FASTBOOT_FLASH_MMC_DEV   1
 #endif