[U-Boot] [PATCH v2 03/14] usb: gadget: Move USBNET_HOST_ADDR to Kconfig

Łukasz Majewski lukma at denx.de
Wed Sep 13 10:38:38 UTC 2017 

On 09/12/2017 09:01 PM, Maxime Ripard wrote:
> While the USB Ethernet device address is already defined in Kconfig, the
> host address isn't. Convert it.
> 
> Signed-off-by: Maxime Ripard <maxime.ripard at free-electrons.com>
> ---
>   configs/am335x_baltos_defconfig        | 1 +
>   configs/am335x_boneblack_defconfig     | 1 +
>   configs/am335x_evm_norboot_defconfig   | 1 +
>   configs/pxm2_defconfig                 | 1 +
>   configs/rastaban_defconfig             | 1 +
>   configs/warp7_defconfig                | 1 +
>   configs/warp7_secure_defconfig         | 1 +
>   drivers/usb/gadget/Kconfig             | 7 +++++++
>   include/configs/am335x_evm.h           | 1 -
>   include/configs/baltos.h               | 1 -
>   include/configs/h2200.h                | 1 -
>   include/configs/siemens-am33x-common.h | 1 -
>   include/configs/warp7.h                | 1 -
>   scripts/config_whitelist.txt           | 1 -
>   14 files changed, 14 insertions(+), 6 deletions(-)
> 
> diff --git a/configs/am335x_baltos_defconfig b/configs/am335x_baltos_defconfig
> index eb8338f0f10d..be057acf8634 100644
> --- a/configs/am335x_baltos_defconfig
> +++ b/configs/am335x_baltos_defconfig
> @@ -63,5 +63,6 @@ CONFIG_USB_GADGET_DOWNLOAD=y
>   CONFIG_G_DNL_MANUFACTURER="Texas Instruments"
>   CONFIG_G_DNL_VENDOR_NUM=0x0403
>   CONFIG_G_DNL_PRODUCT_NUM=0xbd00
> +CONFIG_USBNET_HOST_ADDR="de:ad:be:af:00:00"
>   CONFIG_FAT_WRITE=y
>   CONFIG_OF_LIBFDT=y
> diff --git a/configs/am335x_boneblack_defconfig b/configs/am335x_boneblack_defconfig
> index b8526fc15ed3..719d16b145eb 100644
> --- a/configs/am335x_boneblack_defconfig
> +++ b/configs/am335x_boneblack_defconfig
> @@ -42,6 +42,7 @@ CONFIG_USB_GADGET_DOWNLOAD=y
>   CONFIG_G_DNL_MANUFACTURER="Texas Instruments"
>   CONFIG_G_DNL_VENDOR_NUM=0x0451
>   CONFIG_G_DNL_PRODUCT_NUM=0xd022
> +CONFIG_USBNET_HOST_ADDR="de:ad:be:af:00:00"
>   CONFIG_LZO=y
>   CONFIG_OF_LIBFDT=y
>   CONFIG_OF_LIBFDT_OVERLAY=y
> diff --git a/configs/am335x_evm_norboot_defconfig b/configs/am335x_evm_norboot_defconfig
> index c5e7425a7797..c9eab68c8299 100644
> --- a/configs/am335x_evm_norboot_defconfig
> +++ b/configs/am335x_evm_norboot_defconfig
> @@ -35,6 +35,7 @@ CONFIG_USB_GADGET_DOWNLOAD=y
>   CONFIG_G_DNL_MANUFACTURER="Texas Instruments"
>   CONFIG_G_DNL_VENDOR_NUM=0x0451
>   CONFIG_G_DNL_PRODUCT_NUM=0xd022
> +CONFIG_USBNET_HOST_ADDR="de:ad:be:af:00:00"
>   CONFIG_LZO=y
>   CONFIG_OF_LIBFDT=y
>   CONFIG_OF_LIBFDT_OVERLAY=y
> diff --git a/configs/pxm2_defconfig b/configs/pxm2_defconfig
> index 38370ee8d8f5..b1da71d4c7ed 100644
> --- a/configs/pxm2_defconfig
> +++ b/configs/pxm2_defconfig
> @@ -74,6 +74,7 @@ CONFIG_USB_GADGET_DOWNLOAD=y
>   CONFIG_G_DNL_MANUFACTURER="Siemens AG"
>   CONFIG_G_DNL_VENDOR_NUM=0x0908
>   CONFIG_G_DNL_PRODUCT_NUM=0x02d2
> +CONFIG_USBNET_HOST_ADDR="de:ad:be:af:00:00"
>   # CONFIG_VIDEO_SW_CURSOR is not set
>   CONFIG_SYS_CONSOLE_BG_COL=0xff
>   CONFIG_SYS_CONSOLE_FG_COL=0x00
> diff --git a/configs/rastaban_defconfig b/configs/rastaban_defconfig
> index 5acfe22b0a6d..940c4144d406 100644
> --- a/configs/rastaban_defconfig
> +++ b/configs/rastaban_defconfig
> @@ -70,3 +70,4 @@ CONFIG_USB_GADGET_DOWNLOAD=y
>   CONFIG_G_DNL_MANUFACTURER="Siemens AG"
>   CONFIG_G_DNL_VENDOR_NUM=0x0908
>   CONFIG_G_DNL_PRODUCT_NUM=0x02d2
> +CONFIG_USBNET_HOST_ADDR="de:ad:be:af:00:00"
> diff --git a/configs/warp7_defconfig b/configs/warp7_defconfig
> index 99fe800317c2..f430f30bbbe5 100644
> --- a/configs/warp7_defconfig
> +++ b/configs/warp7_defconfig
> @@ -38,4 +38,5 @@ CONFIG_USB_GADGET_DOWNLOAD=y
>   CONFIG_G_DNL_MANUFACTURER="FSL"
>   CONFIG_G_DNL_VENDOR_NUM=0x0525
>   CONFIG_G_DNL_PRODUCT_NUM=0xa4a5
> +CONFIG_USBNET_HOST_ADDR="de:ad:be:af:00:00"
>   CONFIG_OF_LIBFDT=y
> diff --git a/configs/warp7_secure_defconfig b/configs/warp7_secure_defconfig
> index 8beda72cd882..96e2c9e5cfde 100644
> --- a/configs/warp7_secure_defconfig
> +++ b/configs/warp7_secure_defconfig
> @@ -36,4 +36,5 @@ CONFIG_USB_GADGET_DOWNLOAD=y
>   CONFIG_G_DNL_MANUFACTURER="FSL"
>   CONFIG_G_DNL_VENDOR_NUM=0x0525
>   CONFIG_G_DNL_PRODUCT_NUM=0xa4a5
> +CONFIG_USBNET_HOST_ADDR="de:ad:be:af:00:00"
>   CONFIG_OF_LIBFDT=y
> diff --git a/drivers/usb/gadget/Kconfig b/drivers/usb/gadget/Kconfig
> index 6dc9d177f5ad..510efd67b9a4 100644
> --- a/drivers/usb/gadget/Kconfig
> +++ b/drivers/usb/gadget/Kconfig
> @@ -128,4 +128,11 @@ config USBNET_DEVADDR
>   	  Ethernet MAC address of the device-side (ie. local board's) MAC
>   	  address of the usb_ether interface
>   
> +config USBNET_HOST_ADDR
> +	string "USB Gadget Ethernet host mac address"
> +	default "de:ad:be:ef:00:00"
> +	help
> +	  Ethernet MAC address of the host-side (ie. remote device's) MAC
> +	  address of the usb_ether interface
> +
>   endif # USB_GADGET
> diff --git a/include/configs/am335x_evm.h b/include/configs/am335x_evm.h
> index 415ce46e0d8d..7c025c7e46c8 100644
> --- a/include/configs/am335x_evm.h
> +++ b/include/configs/am335x_evm.h
> @@ -266,7 +266,6 @@
>   #ifdef CONFIG_USB_MUSB_GADGET
>   #define CONFIG_USB_ETHER
>   #define CONFIG_USB_ETH_RNDIS
> -#define CONFIG_USBNET_HOST_ADDR	"de:ad:be:af:00:00"
>   #endif /* CONFIG_USB_MUSB_GADGET */
>   
>   /*
> diff --git a/include/configs/baltos.h b/include/configs/baltos.h
> index 185c749d7868..535fdd4373b0 100644
> --- a/include/configs/baltos.h
> +++ b/include/configs/baltos.h
> @@ -286,7 +286,6 @@
>   #ifdef CONFIG_USB_MUSB_GADGET
>   #define CONFIG_USB_ETHER
>   #define CONFIG_USB_ETH_RNDIS
> -#define CONFIG_USBNET_HOST_ADDR	"de:ad:be:af:00:00"
>   #endif /* CONFIG_USB_MUSB_GADGET */
>   
>   #if defined(CONFIG_SPL_BUILD) && defined(CONFIG_SPL_USBETH_SUPPORT)
> diff --git a/include/configs/h2200.h b/include/configs/h2200.h
> index 870014ddf475..e956e89a89d5 100644
> --- a/include/configs/h2200.h
> +++ b/include/configs/h2200.h
> @@ -127,7 +127,6 @@
>   #define CONFIG_USB_ETH_SUBSET
>   
>   #define CONFIG_USBNET_DEV_ADDR		"de:ad:be:ef:00:01"
> -#define CONFIG_USBNET_HOST_ADDR	"de:ad:be:ef:00:02"
>   #define CONFIG_EXTRA_ENV_SETTINGS \
>   	"stdin=serial\0" \
>   	"stdout=serial\0" \
> diff --git a/include/configs/siemens-am33x-common.h b/include/configs/siemens-am33x-common.h
> index 2314a2d2ef35..2bcd77e9efc8 100644
> --- a/include/configs/siemens-am33x-common.h
> +++ b/include/configs/siemens-am33x-common.h
> @@ -181,7 +181,6 @@
>   #ifdef CONFIG_USB_MUSB_GADGET
>   #define CONFIG_USB_ETHER
>   #define CONFIG_USB_ETH_RNDIS
> -#define CONFIG_USBNET_HOST_ADDR	"de:ad:be:af:00:00"
>   #endif /* CONFIG_USB_MUSB_GADGET */
>   
>   /* USB DRACO ID as default */
> diff --git a/include/configs/warp7.h b/include/configs/warp7.h
> index 75ae8a3e3338..9ce425156677 100644
> --- a/include/configs/warp7.h
> +++ b/include/configs/warp7.h
> @@ -139,7 +139,6 @@
>   #define CONFIG_USB_ETHER
>   #define CONFIG_USB_ETH_CDC
>   #define CONFIG_USB_ETH_RNDIS
> -#define CONFIG_USBNET_HOST_ADDR		"de:ad:be:af:00:00"
>   #define CONFIG_USBNET_DEV_ADDR		"de:ad:be:af:00:01"
>   
>   #endif
> diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt
> index 9ce0c3f039ff..784e83c7c8a1 100644
> --- a/scripts/config_whitelist.txt
> +++ b/scripts/config_whitelist.txt
> @@ -4983,7 +4983,6 @@ CONFIG_USBD_SERIAL_OUT_PKTSIZE
>   CONFIG_USBD_VENDORID
>   CONFIG_USBID_ADDR
>   CONFIG_USBNET_DEV_ADDR
> -CONFIG_USBNET_HOST_ADDR
>   CONFIG_USBNET_MANUFACTURER
>   CONFIG_USBTTY
>   CONFIG_USB_AM35X
> 

Reviewed-by: Łukasz Majewski <lukma at denx.de>

-- 
Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de

More information about the U-Boot mailing list