[U-Boot] [PATCH v5 3/3] h2200: Add USB CDC ethernet support
Marek Vasut
marex at denx.de
Wed Oct 3 02:25:02 CEST 2012
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
More information about the U-Boot
mailing list