This is the mail archive of the
ecos-patches@sources.redhat.com
mailing list for the eCos project.
Re: New port for ARM Industrial Modules AIM 711 - Checked by AntiVir DE
- From: Roland Caßebohm <roland dot cassebohm at VisionSystems dot de>
- To: Andrew Lunn <andrew at lunn dot ch>
- Cc: eCos Patches <ecos-patches at ecos dot sourceware dot org>
- Date: Tue, 13 Apr 2004 10:11:32 +0200
- Subject: Re: New port for ARM Industrial Modules AIM 711 - Checked by AntiVir DE
- References: <20040406162819.GA29940@lunn.ch>
Hi Andrew,
sorry I wasn't there for a while.
You are right, I will make this changes.
I have a general question about copyright header, what exactly
do I have to write in them? In most cases I have just copied
files from other packages, made my changes and added rcassebohm
to the line contributors. Is this right?
Roland
On Dienstag, 6. April 2004 18:28, Andrew Lunn wrote:
> Hi Roland
>
> I took a quick look at another file as well.
>
> rtl8201.c:
>
> It needs a copyright header.
>
> The DEBUG_PRINTF macro is not used. Delete it.
>
> There is more code inside #if 0 which forces the PHY to different
> rates and duplex's. Looks like it should be controlled from CDL. I
> would turn cdl_option CYGPKG_DEVS_ETH_ARM_KS32C5000_PHY_RTL8201 into a
> cdl_package and put the options inside it.
>
> do_autonegotation seems to be an unused variable.
>
> Andrew
--
___________________________________________________
VS Vision Systems GmbH, Industrial Image Processing
Dipl.-Ing. Roland Caßebohm
Aspelohe 27A, D-22848 Norderstedt, Germany
Mail: roland.cassebohm@visionsystems.de
http://www.visionsystems.de
___________________________________________________