All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Mark A. Greer" <mgreer@mvista.com>
To: Andrew Morton <akpm@osdl.org>
Cc: linuxppc-embedded@ozlabs.org
Subject: Re: [PATCH 2.6.12] ppc32: Fix mv64x60 internal SRAM size
Date: Fri, 18 Mar 2005 15:03:59 -0700	[thread overview]
Message-ID: <423B504F.50208@mvista.com> (raw)
In-Reply-To: <20050318134429.7dae3962.akpm@osdl.org>

Yes, it looks correct to me.  Sorry for the bad patch.

Mark
--

Andrew Morton wrote:

>"Mark A. Greer" <mgreer@mvista.com> wrote:
>  
>
>> ppc32: Fix wrong size for mv64[34]60's internal SRAM.
>>    
>>
>
>This got a reject against other ppc32 patches which I had (they've all just
>been flushed to Linus).
>
>The reject was in arch/ppc/Kconfig.  Please check that I fixed it up
>correctly.
>
>
>From: "Mark A. Greer" <mgreer@mvista.com>
>
>ppc32: Fix wrong size for mv64[34]60's internal SRAM.
>
>- Fix incorrect SRAM size
>- Minor Kconfig cleanups for mv64x60 platforms
>
>Signed-off-by: Mark A. Greer <mgreer@mvista.com>
>Signed-off-by: Andrew Morton <akpm@osdl.org>
>---
>
> 25-akpm/arch/ppc/Kconfig               |   10 ++--------
> 25-akpm/arch/ppc/platforms/chestnut.h  |    4 ++--
> 25-akpm/arch/ppc/platforms/katana.h    |    2 +-
> 25-akpm/include/asm-ppc/mv64x60_defs.h |    2 +-
> 4 files changed, 6 insertions(+), 12 deletions(-)
>
>diff -puN arch/ppc/Kconfig~ppc32-fix-mv64x60-internal-sram-size arch/ppc/Kconfig
>--- 25/arch/ppc/Kconfig~ppc32-fix-mv64x60-internal-sram-size	2005-03-18 13:41:27.000000000 -0800
>+++ 25-akpm/arch/ppc/Kconfig	2005-03-18 13:42:57.000000000 -0800
>@@ -577,7 +577,6 @@ config SANDPOINT
> 
> config RADSTONE_PPC7D
> 	bool "Radstone Technology PPC7D board"
>-	select MV64360
> 
> config ADIR
> 	bool "SBS-Adirondack"
>@@ -753,16 +752,11 @@ config GT64260
> 	depends on EV64260 || CPCI690
> 	default y
> 
>-config MV64360
>+config MV64360		# Really MV64360 & MV64460
> 	bool
>-	depends on KATANA || RADSTONE_PPC7D || HDPU
>+	depends on CHESTNUT || KATANA || RADSTONE_PPC7D || HDPU
> 	default y
> 
>-config MV64360
>-	bool
>-	depends on CHESTNUT
>-  	default y
>-
> config MV64X60
> 	bool
> 	depends on (GT64260 || MV64360)
>diff -puN arch/ppc/platforms/chestnut.h~ppc32-fix-mv64x60-internal-sram-size arch/ppc/platforms/chestnut.h
>--- 25/arch/ppc/platforms/chestnut.h~ppc32-fix-mv64x60-internal-sram-size	2005-03-18 13:41:27.000000000 -0800
>+++ 25-akpm/arch/ppc/platforms/chestnut.h	2005-03-18 13:41:27.000000000 -0800
>@@ -28,8 +28,8 @@
>  *    0xffd00000-0xffd00004      - CPLD
>  *    0xffc00000-0xffc0000f      - UART
>  *    0xffb00000-0xffb07fff      - FRAM
>- *    0xffa00000-0xffafffff      - *** HOLE ***
>- *    0xff800000-0xff9fffff      - MV64460 Integrated SRAM
>+ *    0xff840000-0xffafffff      - *** HOLE ***
>+ *    0xff800000-0xff83ffff      - MV64460 Integrated SRAM
>  *    0xfe000000-0xff8fffff      - *** HOLE ***
>  *    0xfc000000-0xfdffffff      - 32bit Flash
>  *    0xf1010000-0xfbffffff      - *** HOLE ***
>diff -puN arch/ppc/platforms/katana.h~ppc32-fix-mv64x60-internal-sram-size arch/ppc/platforms/katana.h
>--- 25/arch/ppc/platforms/katana.h~ppc32-fix-mv64x60-internal-sram-size	2005-03-18 13:41:27.000000000 -0800
>+++ 25-akpm/arch/ppc/platforms/katana.h	2005-03-18 13:41:27.000000000 -0800
>@@ -24,7 +24,7 @@
>  * on a boundary that is a multiple of the window size):
>  *
>  *    0xff800000-0xffffffff      - Boot window
>- *    0xf8400000-0xf85fffff      - Internal SRAM
>+ *    0xf8400000-0xf843ffff      - Internal SRAM
>  *    0xf8200000-0xf83fffff      - CPLD
>  *    0xf8100000-0xf810ffff      - MV64360 Registers (CONFIG_MV64X60_NEW_BASE)
>  *    0xf8000000-0xf80fffff      - Socketed FLASH
>diff -puN include/asm-ppc/mv64x60_defs.h~ppc32-fix-mv64x60-internal-sram-size include/asm-ppc/mv64x60_defs.h
>--- 25/include/asm-ppc/mv64x60_defs.h~ppc32-fix-mv64x60-internal-sram-size	2005-03-18 13:41:27.000000000 -0800
>+++ 25-akpm/include/asm-ppc/mv64x60_defs.h	2005-03-18 13:41:27.000000000 -0800
>@@ -347,7 +347,7 @@
> #define	MV64360_SRAM_ERR_DATA_HI		0x03a0
> #define	MV64360_SRAM_ERR_PARITY			0x03a8
> 
>-#define	MV64360_SRAM_SIZE			0x00200000 /* 2 MB of SRAM */
>+#define	MV64360_SRAM_SIZE			0x00040000 /* 2Mb/256KB SRAM */
> 
> /*
>  *****************************************************************************
>_
>
>
>  
>

      reply	other threads:[~2005-03-18 22:04 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-03-18 18:50 [PATCH 2.6.12] ppc32: Fix mv64x60 internal SRAM size Mark A. Greer
2005-03-18 21:44 ` Andrew Morton
2005-03-18 22:03   ` Mark A. Greer [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=423B504F.50208@mvista.com \
    --to=mgreer@mvista.com \
    --cc=akpm@osdl.org \
    --cc=linuxppc-embedded@ozlabs.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.