All of lore.kernel.org
 help / color / mirror / Atom feed
From: Samuel Egli <samuel.egli@siemens.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/5] siemens: cosmetic: remove unused and rename defines
Date: Thu, 24 Apr 2014 17:57:52 +0200	[thread overview]
Message-ID: <1398355077-6661-2-git-send-email-samuel.egli@siemens.com> (raw)
In-Reply-To: <1398355077-6661-1-git-send-email-samuel.egli@siemens.com>

For dxr2 board DXR2_IOCTRL_VAL is set by data in EEPROM. In pxm2
board it does not make sense to have dxr2 as prefix. Replace it with
more meaningful DDR prefix.

Signed-off-by: Samuel Egli <samuel.egli@siemens.com>
Cc: Pascal Bach <pascal.bach@siemens.com>
Cc: Roger Meier <r.meier@siemens.com>
Cc: Heiko Schocher <hs@denx.de>
Cc: Wolfgang Denk <wd@denx.de>
---
 board/siemens/pxm2/board.c |   10 +++++-----
 include/configs/dxr2.h     |    1 -
 include/configs/pxm2.h     |    2 +-
 3 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/board/siemens/pxm2/board.c b/board/siemens/pxm2/board.c
index 98083d5..64e69dc 100644
--- a/board/siemens/pxm2/board.c
+++ b/board/siemens/pxm2/board.c
@@ -70,11 +70,11 @@ struct cmd_control pxm2_ddr3_cmd_ctrl_data = {
 };
 
 const struct ctrl_ioregs ioregs = {
-	.cm0ioctl		= DXR2_IOCTRL_VAL,
-	.cm1ioctl		= DXR2_IOCTRL_VAL,
-	.cm2ioctl		= DXR2_IOCTRL_VAL,
-	.dt0ioctl		= DXR2_IOCTRL_VAL,
-	.dt1ioctl		= DXR2_IOCTRL_VAL,
+	.cm0ioctl		= DDR_IOCTRL_VAL,
+	.cm1ioctl		= DDR_IOCTRL_VAL,
+	.cm2ioctl		= DDR_IOCTRL_VAL,
+	.dt0ioctl		= DDR_IOCTRL_VAL,
+	.dt1ioctl		= DDR_IOCTRL_VAL,
 };
 
 	config_ddr(DDR_PLL_FREQ, &ioregs, &pxm2_ddr3_data,
diff --git a/include/configs/dxr2.h b/include/configs/dxr2.h
index 75f7812..4612f85 100644
--- a/include/configs/dxr2.h
+++ b/include/configs/dxr2.h
@@ -20,7 +20,6 @@
 #include "siemens-am33x-common.h"
 
 #define CONFIG_SYS_MPUCLK	275
-#define DXR2_IOCTRL_VAL	0x18b
 #define DDR_PLL_FREQ	303
 #undef CONFIG_SPL_AM33XX_ENABLE_RTC32K_OSC
 
diff --git a/include/configs/pxm2.h b/include/configs/pxm2.h
index 6276d43..d75d562 100644
--- a/include/configs/pxm2.h
+++ b/include/configs/pxm2.h
@@ -21,7 +21,7 @@
 #include "siemens-am33x-common.h"
 
 #define CONFIG_SYS_MPUCLK	720
-#define DXR2_IOCTRL_VAL		0x18b
+#define DDR_IOCTRL_VAL		0x18b
 #define DDR_PLL_FREQ		266
 
 #define BOARD_DFU_BUTTON_GPIO	59
-- 
1.7.10.4

  reply	other threads:[~2014-04-24 15:57 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-24 15:57 [U-Boot] [PATCH 0/5] siemens: update and cleanup code Samuel Egli
2014-04-24 15:57 ` Samuel Egli [this message]
2014-05-14  1:35   ` [U-Boot] [U-Boot, 1/5] siemens: cosmetic: remove unused and rename defines Tom Rini
2014-04-24 15:57 ` [U-Boot] [PATCH 2/5] siemens: update DDR3 parameters for dxr2 Samuel Egli
2014-05-14  1:35   ` [U-Boot] [U-Boot,2/5] " Tom Rini
2014-04-24 15:57 ` [U-Boot] [PATCH 3/5] siemens: add led cmd for flexible LED control Samuel Egli
2014-05-14  1:35   ` [U-Boot] [U-Boot, " Tom Rini
2014-04-24 15:57 ` [U-Boot] [PATCH 4/5] siemens: change LED indication in DFU mode Samuel Egli
2014-05-14  1:35   ` [U-Boot] [U-Boot, " Tom Rini
2014-04-24 15:57 ` [U-Boot] [PATCH 5/5] siemens: cosmetic: rename project_dir Samuel Egli
2014-05-14  1:35   ` [U-Boot] [U-Boot,5/5] " Tom Rini

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=1398355077-6661-2-git-send-email-samuel.egli@siemens.com \
    --to=samuel.egli@siemens.com \
    --cc=u-boot@lists.denx.de \
    /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.