Re: [U-Boot] [RFC PATCH 0/4] usb: net: Migrate USB Ethernet to Kconfig

2017-08-28 Thread Ɓukasz Majewski

Hi Chris,


I ran into a few problems running moveconfig.py mainly due to boards
that include a common .h file. I'm not confident I've captured all the
defconfigs that need CONFIG_USB_HOST_ETHER or one of the specific
drivers. Is there a more fool-proof way of catching these cases?
Needless to say this probably isn't ready for 2017.09 but hopefully I
can polish it enough for the next release.


Those patches look OK for me.

I've tried to apply them on top of u-boot-usb/master branch [1].

I've succeed with two of them (1,2), but merging third failed.

Could you rebase your patch set on top of [1] ?

Your patches will be probably merged in the next merge window.

[1] http://git.denx.de/?p=u-boot/u-boot-usb.git;a=summary



Chris Packham (4):
  Kconfig: drop CONFIG_USB_ETHER_DM9601
  Kconfig: drop CONFIG_USB_ETHER_RNDIS
  usb: net: migrate CONFIG_USB_HOST_ETHER to Kconfig
  usb: net: migrate USB Ethernet adapters to Kconfig

 configs/apalis_imx6_defconfig  |  1 +
 configs/apalis_imx6_nospl_com_defconfig|  1 +
 configs/apalis_imx6_nospl_it_defconfig |  1 +
 configs/arndale_defconfig  |  5 +++
 configs/bayleybay_defconfig|  3 ++
 configs/beaver_defconfig   |  2 +
 configs/cardhu_defconfig   |  2 +
 configs/cei-tk1-som_defconfig  |  2 +
 configs/cgtqmx6eval_defconfig  |  2 +
 configs/chromebook_link64_defconfig|  3 ++
 configs/chromebook_link_defconfig  |  3 ++
 configs/chromebook_samus_defconfig |  3 ++
 configs/chromebox_panther_defconfig|  3 ++
 configs/cl-som-am57x_defconfig |  4 ++
 configs/cm_t54_defconfig   |  4 ++
 configs/colibri_imx6_defconfig |  1 +
 configs/colibri_imx6_nospl_defconfig   |  1 +
 configs/colibri_t20_defconfig  |  2 +
 configs/colibri_t30_defconfig  |  2 +
 ...conga-qeval20-qa3-e3845-internal-uart_defconfig |  3 ++
 configs/conga-qeval20-qa3-e3845_defconfig  |  3 ++
 configs/coreboot-x86_defconfig |  3 ++
 configs/cougarcanyon2_defconfig|  3 ++
 configs/crownbay_defconfig |  3 ++
 configs/dalmore_defconfig  |  2 +
 configs/dfi-bt700-q7x-151_defconfig|  5 +++
 configs/dragonboard410c_defconfig  |  5 +++
 configs/e2220-1170_defconfig   |  2 +
 configs/evb-rk3399_defconfig   |  6 +++
 configs/fennec-rk3288_defconfig|  3 ++
 configs/firefly-rk3288_defconfig   |  3 ++
 configs/firefly-rk3399_defconfig   |  6 +++
 configs/galileo_defconfig  |  3 ++
 configs/gwventana_emmc_defconfig   |  3 ++
 configs/gwventana_gw5904_defconfig |  3 ++
 configs/gwventana_nand_defconfig   |  3 ++
 configs/harmony_defconfig  |  4 ++
 configs/hikey_defconfig|  3 ++
 configs/jetson-tk1_defconfig   |  2 +
 configs/kylin-rk3036_defconfig |  3 ++
 configs/m53evk_defconfig   |  4 ++
 configs/marsboard_defconfig|  2 +
 configs/mcx_defconfig  |  3 ++
 configs/medcom-wide_defconfig  |  2 +
 configs/minnowmax_defconfig|  3 ++
 configs/miqi-rk3288_defconfig  |  3 ++
 configs/mvebu_db-88f3720_defconfig |  5 +++
 configs/mvebu_db_armada8k_defconfig|  5 +++
 configs/mvebu_espressobin-88f3720_defconfig|  5 +++
 configs/mvebu_mcbin-88f8040_defconfig  |  5 +++
 configs/mx23_olinuxino_defconfig   |  2 +
 configs/mx28evk_auart_console_defconfig|  3 ++
 configs/mx28evk_defconfig  |  3 ++
 configs/mx28evk_nand_defconfig |  3 ++
 configs/mx28evk_spi_defconfig  |  3 ++
 configs/mx51evk_defconfig  |  3 ++
 configs/mx53loco_defconfig |  4 ++
 configs/mx6dlarm2_defconfig|  2 +
 configs/mx6dlarm2_lpddr2_defconfig |  2 +
 configs/mx6qarm2_defconfig |  2 +
 configs/mx6qarm2_lpddr2_defconfig  |  2 +
 configs/mx6qsabrelite_defconfig|  4 ++
 configs/mx6sabreauto_defconfig |  2 +
 configs/mx6sabresd_defconfig   |  2 +
 configs/mx6slevk_defconfig |  2 +
 configs/mx6slevk_spinor_defconfig  |  2 +
 configs/mx6slevk_spl_defconfig |  2 +
 configs/mx6sllevk_defconfig|  3 

[U-Boot] [RFC PATCH 0/4] usb: net: Migrate USB Ethernet to Kconfig

2017-08-20 Thread Chris Packham
I ran into a few problems running moveconfig.py mainly due to boards
that include a common .h file. I'm not confident I've captured all the
defconfigs that need CONFIG_USB_HOST_ETHER or one of the specific
drivers. Is there a more fool-proof way of catching these cases?
Needless to say this probably isn't ready for 2017.09 but hopefully I
can polish it enough for the next release.

Chris Packham (4):
  Kconfig: drop CONFIG_USB_ETHER_DM9601
  Kconfig: drop CONFIG_USB_ETHER_RNDIS
  usb: net: migrate CONFIG_USB_HOST_ETHER to Kconfig
  usb: net: migrate USB Ethernet adapters to Kconfig

 configs/apalis_imx6_defconfig  |  1 +
 configs/apalis_imx6_nospl_com_defconfig|  1 +
 configs/apalis_imx6_nospl_it_defconfig |  1 +
 configs/arndale_defconfig  |  5 +++
 configs/bayleybay_defconfig|  3 ++
 configs/beaver_defconfig   |  2 +
 configs/cardhu_defconfig   |  2 +
 configs/cei-tk1-som_defconfig  |  2 +
 configs/cgtqmx6eval_defconfig  |  2 +
 configs/chromebook_link64_defconfig|  3 ++
 configs/chromebook_link_defconfig  |  3 ++
 configs/chromebook_samus_defconfig |  3 ++
 configs/chromebox_panther_defconfig|  3 ++
 configs/cl-som-am57x_defconfig |  4 ++
 configs/cm_t54_defconfig   |  4 ++
 configs/colibri_imx6_defconfig |  1 +
 configs/colibri_imx6_nospl_defconfig   |  1 +
 configs/colibri_t20_defconfig  |  2 +
 configs/colibri_t30_defconfig  |  2 +
 ...conga-qeval20-qa3-e3845-internal-uart_defconfig |  3 ++
 configs/conga-qeval20-qa3-e3845_defconfig  |  3 ++
 configs/coreboot-x86_defconfig |  3 ++
 configs/cougarcanyon2_defconfig|  3 ++
 configs/crownbay_defconfig |  3 ++
 configs/dalmore_defconfig  |  2 +
 configs/dfi-bt700-q7x-151_defconfig|  5 +++
 configs/dragonboard410c_defconfig  |  5 +++
 configs/e2220-1170_defconfig   |  2 +
 configs/evb-rk3399_defconfig   |  6 +++
 configs/fennec-rk3288_defconfig|  3 ++
 configs/firefly-rk3288_defconfig   |  3 ++
 configs/firefly-rk3399_defconfig   |  6 +++
 configs/galileo_defconfig  |  3 ++
 configs/gwventana_emmc_defconfig   |  3 ++
 configs/gwventana_gw5904_defconfig |  3 ++
 configs/gwventana_nand_defconfig   |  3 ++
 configs/harmony_defconfig  |  4 ++
 configs/hikey_defconfig|  3 ++
 configs/jetson-tk1_defconfig   |  2 +
 configs/kylin-rk3036_defconfig |  3 ++
 configs/m53evk_defconfig   |  4 ++
 configs/marsboard_defconfig|  2 +
 configs/mcx_defconfig  |  3 ++
 configs/medcom-wide_defconfig  |  2 +
 configs/minnowmax_defconfig|  3 ++
 configs/miqi-rk3288_defconfig  |  3 ++
 configs/mvebu_db-88f3720_defconfig |  5 +++
 configs/mvebu_db_armada8k_defconfig|  5 +++
 configs/mvebu_espressobin-88f3720_defconfig|  5 +++
 configs/mvebu_mcbin-88f8040_defconfig  |  5 +++
 configs/mx23_olinuxino_defconfig   |  2 +
 configs/mx28evk_auart_console_defconfig|  3 ++
 configs/mx28evk_defconfig  |  3 ++
 configs/mx28evk_nand_defconfig |  3 ++
 configs/mx28evk_spi_defconfig  |  3 ++
 configs/mx51evk_defconfig  |  3 ++
 configs/mx53loco_defconfig |  4 ++
 configs/mx6dlarm2_defconfig|  2 +
 configs/mx6dlarm2_lpddr2_defconfig |  2 +
 configs/mx6qarm2_defconfig |  2 +
 configs/mx6qarm2_lpddr2_defconfig  |  2 +
 configs/mx6qsabrelite_defconfig|  4 ++
 configs/mx6sabreauto_defconfig |  2 +
 configs/mx6sabresd_defconfig   |  2 +
 configs/mx6slevk_defconfig |  2 +
 configs/mx6slevk_spinor_defconfig  |  2 +
 configs/mx6slevk_spl_defconfig |  2 +
 configs/mx6sllevk_defconfig|  3 ++
 configs/mx6sllevk_plugin_defconfig |  3 ++
 configs/mx6sxsabreauto_defconfig   |  2 +
 configs/mx6sxsabresd_defconfig |  2 +
 configs/mx6sxsabresd_spl_defconfig |  2 +
 configs/mx7dsabresd_defconfig  |  2 +
 configs/mx7dsabresd_secure_defconfig