Re: [U-Boot] [PATCH ATMEL REWORK][V2] update arm/at91rm9200 work with rework rework110202

2011-02-19 Thread Jens Scharsig
Am 11.02.2011 21:13, schrieb Wolfgang Denk:
 Dear Reinhard Meyer,
 
 In message 4d5597a8.9050...@emk-elektronik.de you wrote:
 Dear Wolfgang Denk,
 Dear Jens Scharsig,

 In message4d5573ca.9000...@scharsoft.de  you wrote:
   * convert at91rm9200ek and eb_cpux9k2 board to ATMEL_xxx name scheme
   * Fix: timer.c compile error io.h not found with arm/at91rm9200
   * update arm920t/at91 to ATMEL_xxx name scheme
   * update arm920t/at91 soc lib
   * update at91_emac driver
 ...
   /* Port B */
 -#define AT91_PMX_BA_ERXCK 0x0008
 -#define AT91_PMX_BA_ECOL  0x0004
 -#define AT91_PMX_BA_ERXDV 0x0002
 -#define AT91_PMX_BA_ERX3  0x0001
 -#define AT91_PMX_BA_ERX2  0x8000
 ^^^

 Please, no space before tabs
 Those lines that are being removed by the patch.
 
 Oops. I marked the wrong line.
 
 I meant this one:
 
 - checkpatch ~/Mail/U-Boot/3377
 WARNING: please, no space before tabs
 #379: FILE: arch/arm/include/asm/arch-at91/at91rm9200.h:112:
 +#define ATMEL_PMX_BA_ERX2 ^I0x8000$
 
 
 
 Best regards,
 
 Wolfgang Denk
 
I don't know, why my checkpatch version does not report this.
But I have fix this in V3 of the Patch.

Best regards,

Jens Scharsig



___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH ATMEL REWORK][V2] update arm/at91rm9200 work with rework rework110202

2011-02-19 Thread Wolfgang Denk
Dear Jens Scharsig,

In message 4d5fee3d.2000...@scharsoft.de you wrote:

 I don't know, why my checkpatch version does not report this.

Try and update against curent Linux kernel version.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
In an organization, each person rises to the level of his own  incom-
petency - The Peter Principle
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH ATMEL REWORK][V2] update arm/at91rm9200 work with rework rework110202

2011-02-11 Thread Wolfgang Denk
Dear Jens Scharsig,

In message 4d5573ca.9000...@scharsoft.de you wrote:
  * convert at91rm9200ek and eb_cpux9k2 board to ATMEL_xxx name scheme
  * Fix: timer.c compile error io.h not found with arm/at91rm9200
  * update arm920t/at91 to ATMEL_xxx name scheme
  * update arm920t/at91 soc lib
  * update at91_emac driver
...
  /* Port B */
 -#define AT91_PMX_BA_ERXCK0x0008
 -#define AT91_PMX_BA_ECOL 0x0004
 -#define AT91_PMX_BA_ERXDV0x0002
 -#define AT91_PMX_BA_ERX3 0x0001
 -#define AT91_PMX_BA_ERX2 0x8000
^^^

Please, no space before tabs


Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Gewöhnlich glaubt der Mensch,  wenn er nur Worte hört,  es müsse sich
dabei doch auch was denken lassen. -- Goethe, Faust I
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH ATMEL REWORK][V2] update arm/at91rm9200 work with rework rework110202

2011-02-11 Thread Reinhard Meyer
Dear Wolfgang Denk,
 Dear Jens Scharsig,

 In message4d5573ca.9000...@scharsoft.de  you wrote:
   * convert at91rm9200ek and eb_cpux9k2 board to ATMEL_xxx name scheme
   * Fix: timer.c compile error io.h not found with arm/at91rm9200
   * update arm920t/at91 to ATMEL_xxx name scheme
   * update arm920t/at91 soc lib
   * update at91_emac driver
 ...
   /* Port B */
 -#define AT91_PMX_BA_ERXCK   0x0008
 -#define AT91_PMX_BA_ECOL0x0004
 -#define AT91_PMX_BA_ERXDV   0x0002
 -#define AT91_PMX_BA_ERX30x0001
 -#define AT91_PMX_BA_ERX20x8000
 ^^^

 Please, no space before tabs
Those lines that are being removed by the patch.
Best Regards,
Reinhard
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH ATMEL REWORK][V2] update arm/at91rm9200 work with rework rework110202

2011-02-11 Thread Wolfgang Denk
Dear Reinhard Meyer,

In message 4d5597a8.9050...@emk-elektronik.de you wrote:
 Dear Wolfgang Denk,
  Dear Jens Scharsig,
 
  In message4d5573ca.9000...@scharsoft.de  you wrote:
* convert at91rm9200ek and eb_cpux9k2 board to ATMEL_xxx name scheme
* Fix: timer.c compile error io.h not found with arm/at91rm9200
* update arm920t/at91 to ATMEL_xxx name scheme
* update arm920t/at91 soc lib
* update at91_emac driver
  ...
/* Port B */
  -#define AT91_PMX_BA_ERXCK 0x0008
  -#define AT91_PMX_BA_ECOL  0x0004
  -#define AT91_PMX_BA_ERXDV 0x0002
  -#define AT91_PMX_BA_ERX3  0x0001
  -#define AT91_PMX_BA_ERX2  0x8000
  ^^^
 
  Please, no space before tabs
 Those lines that are being removed by the patch.

Oops. I marked the wrong line.

I meant this one:

- checkpatch ~/Mail/U-Boot/3377
WARNING: please, no space before tabs
#379: FILE: arch/arm/include/asm/arch-at91/at91rm9200.h:112:
+#define ATMEL_PMX_BA_ERX2 ^I0x8000$



Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
EMACS belongs in sys/errno.h: Editor too big!
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot