[U-Boot] [PATCH v5 3/3] h2200: Add USB CDC ethernet support

Marek Vasut marex at denx.de
Wed Oct 3 03:04:22 CEST 2012


Dear Łukasz Dałek,

> On 03.10.2012 02:25, Marek Vasut wrote:
> > Dear Łukasz Dałek,
> > 
> >> On 03.10.2012 02:18, Marek Vasut wrote:
> >>> Dear Łukasz Dałek,
> >>> 
> >>>> On 03.10.2012 02:06, Marek Vasut wrote:
> >>>>> Dear Lukasz Dalek,
> >>>>> 
> >>>>>> Add support for ethernet over USB which can be used for e.g. booting
> >>>>>> process. It works with tftp and dhcp clients code.
> >>>>>> 
> >>>>>> Signed-off-by: Lukasz Dalek<luk0104 at gmail.com>
> >>>>>> ---
> >>>>>> 
> >>>>>> Changes for v5:
> >>>>>> 	- Surrendered #include<usb.h>    with CONFIGs
> >>>>>> 	
> >>>>>>     board/h2200/h2200.c     |   17 +++++++++++++++++
> >>>>>>     boards.cfg              |    1 +
> >>>>>>     include/configs/h2200.h |   27 +++++++++++++++++++++++++++
> >>>>>>     3 files changed, 45 insertions(+), 0 deletions(-)
> >>>>> 
> >>>>> [...]
> >>>>> 
> >>>>>> diff --git a/boards.cfg b/boards.cfg
> >>>>>> index ab10938..a03a469 100644
> >>>>>> --- a/boards.cfg
> >>>>>> +++ b/boards.cfg
> >>>>>> @@ -283,6 +283,7 @@ pdnb3                        arm         ixp
> >>>>>> pdnb3               prodriv scpu                         arm
> >>>>>> ixp
> >>>>>> 
> >>>>>>          pdnb3               prodrive       -           pdnb3:SCPU
> >>>>>>          balloon3
> >>>>>>          
> >>>>>>                      arm         pxa
> >>>>>>     
> >>>>>>     h2200                        arm         pxa
> >>>>>> 
> >>>>>> +h2200_usbeth                 arm         pxa         h2200
> >>>>>> 
> >>>>>>    -
> >>>>> 
> >>>>> I told you to enable the USB ethernet by default, so why do you add
> >>>>> this board?
> >>>> 
> >>>> I don't understand.
> >>>> We agreed to split h2200 support into two parts. First one adding
> >>>> basic support for device, second one adding support for usb ethernet
> >>>> for h2200.
> >>> 
> >>> Yes, but I fail to understand why you need TWO board entries for that.
> >>> Adding the USB ethernet support is just a feature, you don't need
> >>> additional board entry for that ... so what is the idea behind adding
> >>> it?
> >> 
> >> Because easier is to type make h2200_usbeth_config than modifiy config.
> > 
> > Why don't you enable the USB ethernet by default for "h2200" target then?
> > 
> > Best regards,
> > Marek Vasut
> 
> I will enable. This two entries really look weird.

Ok, so we're clear there'll be one entry with ethernet enabled called "h2200", 
yes?

Best regards,
Marek Vasut


More information about the U-Boot mailing list