From mboxrd@z Thu Jan 1 00:00:00 1970 From: Timur Tabi Date: Fri, 2 Sep 2011 08:41:00 -0500 Subject: [U-Boot] [PATCH 2/2] powerpc/85xx: standardize display of address map size (32-bit vs. 36-bit) In-Reply-To: <25912EDC-884B-47F0-B638-FA3F55E7DE3F@kernel.crashing.org> References: <1314828916-17931-1-git-send-email-timur@freescale.com> <1314828916-17931-2-git-send-email-timur@freescale.com> <20110901142237.E2D85166C886@gemini.denx.de> <4E5F9C8D.3080503@freescale.com> <20110901215931.D4D0B13B8A29@gemini.denx.de> <6A841E78-5D44-4190-9899-1976234EF955@freescale.com> <4E60BFD6.5020304@freescale.com> <25912EDC-884B-47F0-B638-FA3F55E7DE3F@kernel.crashing.org> Message-ID: <4E60DCEC.9090205@freescale.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Kumar Gala wrote: > Both. I'm think for your patch we'd add some general config option for extra print info. So you want to see this instead: /* * Display whether this is a 32-bit build or a 36-bit build. */ #ifdef CONFIG_DISPLAY_ADDR_SIZE #ifdef CONFIG_PHYS_64BIT puts("ADDR: 36-bit address map\n"); #else puts("ADDR: 32-bit address map\n"); #endif #endif I still like my way better. It eliminates the need to think about another CONFIG option. I think adding another CONFIG option is worse than adding another line of text. It think it's silly to complain about adding one line of text that is only displayed on e500 systems that support 36-bit addressing, especially since we display this information on most of our boards anyway. Surely we can find some other line of text that we can shorten or eliminate to make up for it. For instance, we can combine these two lines into one: CPU0: P1022E, Version: 1.0, (0x80ee0010) Core: E500, Version: 5.0, (0x80211050) or these two lines: L1: D-cache 32 kB enabled I-cache 32 kB enabled -- Timur Tabi Linux kernel developer at Freescale