This is the mail archive of the ecos-patches@sourceware.org mailing list for the eCos project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

RE: PHY update


Hi John,

Please find attached updated patch.

Regards,
Christophe

-----Original Message-----
From: John Dallaway [mailto:john@dallaway.org.uk] 
Sent: 9. juni 2010 18:45
To: Christophe Coutand
Cc: Sachin Sushil Chaddha; Edgar Grimberg;
ecos-patches@ecos.sourceware.org; chadhas@ymail.com
Subject: Re: PHY update

Hi Christophe

Christophe Coutand wrote:

> John Dallaway wrote:
>
>> Is there a reason why you have placed the PHY register definitions
>> into an external header file? Might they be required by other eCos
>> packages?
>>
>> If there _is_ a good reason, then we should add the standard
>> copyright banner to these header files and adhere to eCos file naming
>> conventions. Otherwise, I would suggest placing these definitions in
>> the relevant .c source files directly similar to the support for
>> other PHY parts (INLXT972.c and KSZ8041.c for example).
>
> The reason for having external definitions was to keep all registers
> definition in the same place. Some HW may have to configure special
PHY
> registers (to adjust clock delay etc..) to be able to work properly.
> This work is left to the user since it is HW specific.
> 
> Currently no package rely on these header files. Let me know what you
> prefer and I can submit a new patch.

OK. I think we should put the header files into the src/ directory for
now and give them names IP101A.h, VSC8244.h and VSC8641.h (matching the
.c source file naming). If they are needed by other packages in the
future, we can easily relocate them to the include/ directory. Can you
also add the standard eCos license/copyright banner at the top of each
header file please?

Thank you

John Dallaway
eCos maintainer

Attachment: eth_phy
Description: eth_phy


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]