public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot-Users] Cleanup in AT91RM9200.h ?
@ 2005-03-02  9:25 Steven Scholz
  2005-03-02 10:12 ` Wolfgang Denk
  0 siblings, 1 reply; 4+ messages in thread
From: Steven Scholz @ 2005-03-02  9:25 UTC (permalink / raw)
  To: u-boot

Dear Wolfgang,

the formating/indenting in include/asm-arm/arch-at91rm9200/AT91RM9200.h looks 
terrible.

Would use except a patch that only does whitspace fixing in this file?

--
Steven

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot-Users] Cleanup in AT91RM9200.h ?
  2005-03-02  9:25 [U-Boot-Users] Cleanup in AT91RM9200.h ? Steven Scholz
@ 2005-03-02 10:12 ` Wolfgang Denk
  2005-03-02 12:35   ` Steven Scholz
  0 siblings, 1 reply; 4+ messages in thread
From: Wolfgang Denk @ 2005-03-02 10:12 UTC (permalink / raw)
  To: u-boot

In message <422586A2.3060705@imc-berlin.de> you wrote:
> 
> the formating/indenting in include/asm-arm/arch-at91rm9200/AT91RM9200.h looks 
> terrible.

It does, but I think  I  can  detect  some  kind  of  logic  in  this
formatting.

> Would use except a patch that only does whitspace fixing in this file?

If  it  really  improves  the  readbility  while  still  keeping  all
structural information, sure.

Best regards,

Wolfgang Denk

-- 
Software Engineering:  Embedded and Realtime Systems,  Embedded Linux
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
There are bugs and then there are bugs.  And then there are bugs.
                                                    - Karl Lehenbauer

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot-Users] Cleanup in AT91RM9200.h ?
  2005-03-02 10:12 ` Wolfgang Denk
@ 2005-03-02 12:35   ` Steven Scholz
  2005-09-25 15:15     ` Wolfgang Denk
  0 siblings, 1 reply; 4+ messages in thread
From: Steven Scholz @ 2005-03-02 12:35 UTC (permalink / raw)
  To: u-boot

Hi,

>>the formating/indenting in include/asm-arm/arch-at91rm9200/AT91RM9200.h looks 
>>terrible.
> 
> It does, but I think  I  can  detect  some  kind  of  logic  in  this
> formatting.
;-)

>>Would use except a patch that only does whitspace fixing in this file?
> 
> If  it  really  improves  the  readbility  while  still  keeping  all
> structural information, sure.

Ok. Here's the first attempt. Tested with 'MAKEALL ARM9'.
I send the complete new AT91RM9200.h. Diff makes no sense.

* Patch by Steven Scholz, 02 March 2005:
   Fix formating in include/asm-arm/arch-at91rm9200/AT91RM9200.h

--
Steven

-------------- next part --------------
A non-text attachment was scrubbed...
Name: AT91RM9200.h.bz2
Type: application/octet-stream
Size: 7333 bytes
Desc: not available
Url : http://lists.denx.de/pipermail/u-boot/attachments/20050302/e4edc24a/attachment.obj 

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot-Users] Cleanup in AT91RM9200.h ?
  2005-03-02 12:35   ` Steven Scholz
@ 2005-09-25 15:15     ` Wolfgang Denk
  0 siblings, 0 replies; 4+ messages in thread
From: Wolfgang Denk @ 2005-09-25 15:15 UTC (permalink / raw)
  To: u-boot

In message <4225B304.4060409@imc-berlin.de> you wrote:
>
> * Patch by Steven Scholz, 02 March 2005:
>    Fix formating in include/asm-arm/arch-at91rm9200/AT91RM9200.h

Added, thanks.

Best regards,

Wolfgang Denk

-- 
Software Engineering:  Embedded and Realtime Systems,  Embedded Linux
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Roses are red
Violets are blue
Some poems rhyme

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2005-09-25 15:15 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-03-02  9:25 [U-Boot-Users] Cleanup in AT91RM9200.h ? Steven Scholz
2005-03-02 10:12 ` Wolfgang Denk
2005-03-02 12:35   ` Steven Scholz
2005-09-25 15:15     ` Wolfgang Denk

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox