[U-Boot] [PATCH v3] h2200: Add support for iPAQ h2200 palmtop

Marek Vasut marex at denx.de
Thu Sep 27 01:58:53 CEST 2012


Dear Łukasz Dałek,

> On 27.09.2012 01:32, Marek Vasut wrote:
> > Dear Łukasz Dałek,
> > 
> >> On 26.09.2012 18:13, Marek Vasut wrote:
> >>> Dear Łukasz Dałek,
> >>> 
> >>>> On 26.09.2012 17:29, Marek Vasut wrote:
> >>>>> Dear Lukasz Dalek,
> >>>>> 
> >>>>>> Basic support of HP iPAQ h2200 palmtop. Support includes also USB
> >>>>>> ethernet gadget.
> >>>>> 
> >>>>> Two minor nits ... can you extend the patch description?
> >>>>> 
> >>>>>> Signed-off-by: Lukasz Dalek<luk0104 at gmail.com>
> >>>>>> [...]
> >>>>>> +h2200                        arm         pxa
> >>>>>> +h2200_usbeth                 arm         pxa         h2200
> >>>>>> 
> >>>>>>    -
> >>>>> 
> >>>>> Can you not enable USB ethernet by default?
> >>>>> 
> >>>>> [...]
> >>>> 
> >>>> I don't understand.
> >>>> 
> >>>> make h2200_config creates config without usb ethernet
> >>>> make h2200_usbeth_config creates config with usb ethernet
> >>> 
> >>> Why not just make config with usb ethernet then?
> >> 
> >> Sorry but I don't understand. You've asked me that can I not
> >> enable USB ethernet by default.
> > 
> > I think you misunderstood. I was never against enabling the USB ethernet.
> > I was against enabling it before the USB ethernet code is in, so it
> > won't break bisectability.
> > 
> > If I did really say so, I'm sorry for the confusion.
> 
> So you want from me to split h2200 patches? I mean to prepare 2 patches:
> 1st one only adding support for device and 2nd one adding support for
> ethernet?

Oh, that would work nicely, good idea.

> Łukasz Dałek

Best regards,
Marek Vasut


More information about the U-Boot mailing list