Re: [U-Boot] [PATCH 3/5] Removed unused define, CONFIG_ARMV7.

Dear Christopher Harvey,

In message 20110704184659.GG3016@harvey-pc.matrox.com you wrote:

...

Need the empty line that wasn't there before? Same Q for every file in this patch.

I generated this patch with sed actually. I could easily re-run it and tell it to remove the newlines. Lets see if it gets considered/accepted first.

Please fix this (and the other comments), and it goes in.

...

...

#define CONFIG_SYS_TEXT_BASE 0x60800000

/* High Level Configuration Options */ -#define CONFIG_ARMV7 1

If that was the only remaining high level config option, should the comment be removed?

I figured maintainers would get to it eventually.

Such "eventually" never works. Please go through this manually and clean up. Thanks.

Best regards,

Wolfgang Denk