[U-Boot] [PATCH v2 1/6] h2200: Add support for iPAQ h2200 palmtop

Łukasz Dałek luk0104 at gmail.com
Sun Sep 23 22:24:28 CEST 2012


On 23.09.2012 22:05, Marek Vasut wrote:
> Dear Łukasz Dałek,
>
>> On 23.09.2012 21:31, Marek Vasut wrote:
>>> Dear Łukasz Dałek,
>>>
>>>> On 23.09.2012 17:43, Marek Vasut wrote:
>>>>>> +	gd->ram_size = CONFIG_SYS_SDRAM_SIZE;
>>>>>> +	return 0;
>>>>>> +}
>>>>>> diff --git a/board/h2200/h2200.h b/board/h2200/h2200.h
>>>>>> new file mode 100644
>>>>> Do you need this file at all? Can the udc_disconnect() not be put
>>>>> somewhere into usb.h ?
>>>> Can I just move this declaration into h2200.c?
>>> extern ... ? No, checkpatch will scream and extern is prohibited for a
>>> good reason.
>> So where should I put it?
> Isn't it a general call? some include/usb.h ?
>
For old layer maybe it is.

I'm not sure but include/usb.h is for device implementing usb host but
pxa can only be device. Including usb.h generate error.

I'm going to add sd card support and other functions, maybe leaving it
would be better idea?

Łukasz Dałek


More information about the U-Boot mailing list