[U-Boot] [PATCH v5 3/3] h2200: Add USB CDC ethernet support
Łukasz Dałek
luk0104 at gmail.com
Wed Oct 3 11:50:58 CEST 2012
On 03.10.2012 03:04, Marek Vasut wrote:
> 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
Yes
Łukasz Dałek
More information about the U-Boot
mailing list