[U-Boot] [PATCH] MAINTAINERS: Add a pointer to the custodians page.

2012-11-24 Thread Robert P. J. Day

Since there's no obvious mention, add a brief reference to the
custodians page at www.denx.de

Signed-off-by: Robert P. J. Day rpj...@crashcourse.ca

---

  remembering that i embarrassed myself by not understanding the
difference between a maintainer and a custodian, i think it's worth
adding a brief mention.  feel free to reword to taste.

diff --git a/MAINTAINERS b/MAINTAINERS
index c430574..55ee9b8 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -7,6 +7,10 @@
 # and Cc: the u-boot@lists.denx.de mailing list. #
 #  #
 # Note: lists sorted by Maintainer Name
#
+# Note: These are the maintainers for specific *boards*.  The  #
+#  custodians for general architectures and subsystems can #
+#  be found here -- http://www.denx.de/wiki/U-Boot/Custodians  #
+#  #
 #



rday
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v4 1/3] Add README for the Falcon mode

2012-11-24 Thread Stefano Babic
On 23/11/2012 19:10, Vikram Narayanan wrote:
 Hi Stefano,
 
 Sorry for bumping in at v4. Below are some of my comments.
 

No problems, all reviews are welcome ! I fix your comments in V5.

 
 If the offset is known at compile time, why should the end user use the
 above commands to write it? Can't it be automated? Just an idea.

No, it's not - Andreas have already given links to previous threads
where this issue was discussed.

Best regards,
Stefano

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] mx28: Fix typo in POWER_DCLIMITS_NEGLIMIT_OFFSET

2012-11-24 Thread Stefano Babic
On 22/11/2012 04:02, Marek Vasut wrote:
 The POWER_DCLIMITS_NEGLIMIT_OFFSET bit in mx28 power supply block is
 not called POWER_DCLIMITS_NETLIMIT_OFFSET, but POWER_DCLIMITS_NEGLIMIT_OFFSET.
 Correct the name in the header file.
 
 Signed-off-by: Marek Vasut ma...@denx.de
 Cc: Stefano Babic sba...@denx.de
 ---

Applied to u-boot-imx, thanks.

Best regards,
Stefano Babic



-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] mx28: Fix typo in POWER_MINPWR_VBG_OFF

2012-11-24 Thread Stefano Babic
On 22/11/2012 03:50, Marek Vasut wrote:
 The POWER_MINPWR_VBG_OFF bit in mx28 power supply block is not called
 POWER_MINPWR_FBG_OFF, but POWER_MINPWR_VBG_OFF. Correct the name in the
 header file.
 
 Signed-off-by: Marek Vasut ma...@denx.de
 Cc: Stefano Babic sba...@denx.de
 ---


Applied to u-boot-imx, thanks.

Best regards,
Stefano Babic


-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PULL] : Please pull u-boot-imx

2012-11-24 Thread Stefano Babic
Hi Albert,

please pull from u-boot-imx, thanks.

The following changes since commit 34275d70fec6cc369a931090ebb686bc213bb80d:

  arch-mx6: add mx6dl_pins.h (2012-11-10 08:15:40 +0100)

are available in the git repository at:

  git://www.denx.de/git/u-boot-imx.git master

for you to fetch changes up to 1d4fd0dc3b45de7b7a706285845ef5b527422014:

  mx28: Fix typo in POWER_DCLIMITS_NEGLIMIT_OFFSET (2012-11-24 16:21:37
+0100)


Benoît Thébaudeau (16):
  mx31: Move EHCI definitions to ehci-fsl.h
  ehci-mxc: Clean up
  ehci-mx5: Clean up
  ehci-mx5: Fix OC_DIS usage
  ehci-mx5: Fix OPM usage
  ehci-mx5: Fix *PM usage for i.MX53
  ehci-mx5: Add missing OC_DIS for i.MX53
  ehci-mxc: Make EHCI power/oc polarities configurable
  ehci-mxc: Make i.MX25 EHCI configurable
  ehci-mxc: Define host offsets
  ehci-mxc: Add support for i.MX35
  mx35pdk: Add support for OTG
  ehci-mx5/6: Make board_ehci_hcd_init() optional
  ehci-mxc: Fix host power mask bit for i.MX35
  ehci-mxc: Fix host power mask bit for i.MX25
  mx5: Mark lowlevel_init board-specific code

Fabio Estevam (13):
  mx5: Align SPI CS naming with i.MX53 reference manual
  mx5: Print CSPI clock in 'clock' command
  spi: mxc_spi: Fix handling of chip select
  spi: mxc_spi: Fix spi clock glitch durant reset
  mx6: clock: Only show CSPI clock if CSPI is enabled
  mx28evk: Configure CONFIG_BOOTDELAY to one second
  mx53loco: Configure CONFIG_BOOTDELAY to one second
  mx6qsabrelite: Configure CONFIG_BOOTDELAY to one second
  mx6qsabre_common: Configure CONFIG_BOOTDELAY to one second
  mx51evk: Configure CONFIG_BOOTDELAY to one second
  mx25pdk: Configure CONFIG_BOOTDELAY to one second
  mx31pdk: Configure CONFIG_BOOTDELAY to one second
  mx35pdk: Configure CONFIG_BOOTDELAY to one second

Marek Vasut (2):
  mx28: Fix typo in POWER_MINPWR_VBG_OFF
  mx28: Fix typo in POWER_DCLIMITS_NEGLIMIT_OFFSET

 arch/arm/cpu/armv7/mx5/clock.c |4 +-
 arch/arm/cpu/armv7/mx5/lowlevel_init.S |2 +-
 arch/arm/cpu/armv7/mx6/clock.c |2 +
 arch/arm/include/asm/arch-mx25/imx-regs.h  |1 +
 arch/arm/include/asm/arch-mx31/imx-regs.h  |   27 +---
 arch/arm/include/asm/arch-mx35/imx-regs.h  |2 +
 arch/arm/include/asm/arch-mx5/mx5x_pins.h  |6 +-
 arch/arm/include/asm/arch-mxs/regs-power.h |4 +-
 board/freescale/mx35pdk/lowlevel_init.S|4 +
 board/freescale/mx35pdk/mx35pdk.c  |   21 +++
 doc/README.imx5|5 +
 drivers/spi/mxc_spi.c  |9 +-
 drivers/usb/host/ehci-mx5.c|  146 ++
 drivers/usb/host/ehci-mx6.c|5 +
 drivers/usb/host/ehci-mxc.c|  225
+---
 include/configs/mx25pdk.h  |2 +-
 include/configs/mx28evk.h  |2 +-
 include/configs/mx31pdk.h  |2 +-
 include/configs/mx35pdk.h  |   16 +-
 include/configs/mx51_efikamx.h |1 +
 include/configs/mx51evk.h  |3 +-
 include/configs/mx53loco.h |2 +-
 include/configs/mx6qsabre_common.h |2 +-
 include/configs/mx6qsabrelite.h|2 +-
 include/configs/vision2.h  |1 +
 include/configs/zmx25.h|6 +-
 include/usb/ehci-fsl.h |   28 +++-
 27 files changed, 401 insertions(+), 129 deletions(-)

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v4 6/7] gpt: Support for new gpt command

2012-11-24 Thread Stephen Warren
On 11/21/2012 04:22 AM, Piotr Wilczek wrote:
 Dear Stephen,
 
 Stephen Warren wrote at Monday, November 19, 2012 10:35 PM:
 On 11/09/2012 03:48 AM, Piotr Wilczek wrote:
 New command - gpt is supported. It restores the GPT partition table.
 It looks into the partitions environment variable for partitions 
 definition.
 It can be enabled at target configuration file with CONFIG_CMD_GPT.
 Simple UUID generator has been implemented. It uses the the
 gd-start_addr_sp for entrophy pool. Moreover the pool address is used as 
 crc32 seed.

 diff --git a/common/cmd_gpt.c b/common/cmd_gpt.c

 +U_BOOT_CMD(gpt, CONFIG_SYS_MAXARGS, 1, do_gpt,
 +   GUID Partition Table,
 +   interface dev partions list\n
 +partions list is in format: name=..,size=..,uuid=..;...\n
 +and can be passed as env or string ex.:\n
 +   gpt mmc 0 partitions\n

 I don't think that form makes sense. The user should just pass
 ${partitions} instead. The command can't know for certain whether the
 user actually intended to pass the text partitions and made a
 mistake, or whether they passed an environment variable. If you really
 want to be able to pass an environment variable name, an explicit
 command-line option such as:

 gpt mmc 0 name=...  # definition on cmd-line
 gpt mmc 0 --from-environment partitions # definition in environment

 seems best.

 The intention was that the command automatically figures out whether user
 passed environmental variable or directly partitions as text. Then the
 command splits this string, checks if it is a valid partitions list and if
 so the table is written to mmc. That is how the code is supposed to work.
 
 The question here is, if it should work like that. If it is desired that
 user explicitly states that the partitions list is passed from
 environmental, then I should change the code.

I personally prefer things to be explicit; that way, there can't be any
corner-case that isn't covered by the automatic mode.

 +/**
 + * extract_env(): Convert string from '{env_name}' to 'env_name'

 s//$/

 It's doing more than that; it locates that syntax within an arbitrary
 string and ignores anything before ${ or after }. Is that
 intentional?

 Yes, it was. The u-boot's shell expands to one only, so it allow to pass any
 partition parameter as env when the partitions list itself is passed as env.

OK. The issue here is that the comment doesn't exactly describe what the
code is doing.

Also, what if the user wrote foo${var}bar; I can't recall if the code
handles that correct; is the result of that just ${var}, or do foo
and bar actually make it into the result string?

 +static int extract_env(char *p)

 +   p1 = strstr(p, ${);
 +   p2 = strstr(p, });
 +
 +   if (p1  p2) {
 +   *p2 = '\0';
 +   memmove(p, p+2, p2-p1-1);

 s/-1/-2/ I think, since the length of ${ is 2 not 1.

 p2-p1-1 gives length of the env name + trailing zero.
 p2-p1-2 would give only the env's length and the trailing zero wouldn't be
 moved.

Ah right. I tend to write things like that as:

p2-p1-2+1 /* strlen(${)==2, length '\0'==1

to make it obvious what's going on

 +   tok = strsep(p, ;);
 +   if (tok == NULL)
 +   break;
 +
 +   if (extract_val(tok, name, i, name)) {
 +   ret = -1;
 +   goto err;
 +   }
 +
 +   if (extract_val(tok, ps, i, size)) {
 +   ret = -1;
 +   free(name[i]);
 +   goto err;
 +   }

 I think that requires the parameters to be passed in order
 name=foo,size=5,uuid=xxx. That seems inflexible. The syntax may as well
 just be value,value,value rather than key=value,key=value,key=value in
 that case (although the keys are useful in order to understand the
 data, so I'd prefer parsing flexibility rather than removing key=).

 I would say that the key=value is flexible. The 'extract_env' function
 tells you if the requested key was provided or not. Also, the order of keys
 is not important.

The order of the keys shouldn't be important, but doesn't the code above
expect to find key name first, then key size, etc., in tha specific
order, as it walks through the string?
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v4 5/7] gpt: Support for GPT (GUID Partition Table) restoration

2012-11-24 Thread Stephen Warren
On 11/21/2012 06:18 AM, Piotr Wilczek wrote:
 Dear Stephen,
 
 Stephen Warren wrote at Monday, November 19, 2012 9:17 PM:
 On 11/09/2012 03:48 AM, Piotr Wilczek wrote:
 The restoration of GPT table (both primary and secondary) is now possible.
 Simple GUID generation is supported.
...
 +   for (i = 0; i  parts; i++) {
 +   /* partition starting lba */
 +   start = partitions[i]-start;
 +   if (start  (offset = start))
 +   gpt_e[i].starting_lba = cpu_to_le32(start);
 +   else
 +   gpt_e[i].starting_lba = cpu_to_le32(offset);

 That seems a little odd. The else branch seems fine when !start, but
 what about when (start  (offset  start))? Shouldn't that be an
 error, rather than just ignoring the requested start value?

 The idea is that if the user provided start address and the partitions does
 not overlap, the partition is located at the start address. Otherwise
 partitions are located next to each other.

But if the user provided a start address, shouldn't it always be honored
exactly, or any error generated; silently ignoring a start address when
there's an overlap seems wrong. Also, the overlap checking seems a
little simplistic; it should really sort the partition array and then
walk through checking for overlaps, rather than maintaining a single
highest used offset, since there's no requirement for the physical
order of partitions to match the order in the partition table, hence why
I asked:

 Why can't partitions be out of order? IIRC, the GPT spec allows it.


 Oh, so is set_gpt_table() an internal-only function? If so, shouldn't
 it be static and not in the header file?

 It could be used by some other code in future.

Perhaps. It can always be made non-static at that time.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] MX6 solo support

2012-11-24 Thread Fabio Estevam
Hi,

Just wondering if someone has already started adding mx6solo support
into mainline U-boot?

Carolyn,

You posted some questions about mx6solo in this list. Were you using
mainline U-boot or the FSL version?

In case you were using mainline U-boot, and would like to share some
mx6solo related patches, I would be glad to to receive them and submit
them.

Regards,

Fabio Estevam
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] MX6 solo support

2012-11-24 Thread Eric Nelson

On 11/24/2012 11:26 AM, Fabio Estevam wrote:

Hi,

Just wondering if someone has already started adding mx6solo support
into mainline U-boot?

Carolyn,

You posted some questions about mx6solo in this list. Were you using
mainline U-boot or the FSL version?

In case you were using mainline U-boot, and would like to share some
mx6solo related patches, I would be glad to to receive them and submit
them.

Regards,

Fabio Estevam


Hi Fabio,

Troy's been working diligently to push Solo support, but it's taking
a while to work through the bits. The first attempt, which allowed a
single binary to be used on Solo-Quad required the use of i.MX
plugins and was more or less rejected with pushback to use SPL instead.

We have some Solo boards out in the field using plugins from this branch:
https://github.com/boundarydevices/u-boot-imx6/tree/solo-dl-staging

Based on the feedback, our plan was (is) to create separate board
file for the processor variants, but don't want to duplicate everything,
so we planned on these steps:

- Add preprocessor support to imximage.

This is stalled at the moment, waiting for Stefano
and Tom to resolve differences

- Push nitrogen6q board file for Quad/Dual

- Create a new board for nitrogen6s (Solo/Dual Lite)

We've been at the first stage for a while though.

Regards,


Eric
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [Patch 1/2] MIPS: fix a latent bug on initialize $gp

2012-11-24 Thread Zhi-zhou Zhang
If bal is 8 bytes aligned, the _gp will not be 8 bytes aligned.
then the following ld insntrustion generates a Adel exception.
So here make _gp be always aligned in 8 bytes.

Signed-off-by: Zhi-zhou Zhang zhizhou...@gmail.com
---
 arch/mips/cpu/mips64/start.S |5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/arch/mips/cpu/mips64/start.S b/arch/mips/cpu/mips64/start.S
index 4112de7..8e8cc33 100644
--- a/arch/mips/cpu/mips64/start.S
+++ b/arch/mips/cpu/mips64/start.S
@@ -108,7 +108,10 @@ reset:
mtc0t0, CP0_CONFIG
 #endif
 
-   /* Initialize $gp */
+   /* Initialize $gp, _gp must be 8 bytes algined. */
+   .align 3
+   nop
+   nop
bal 1f
 nop
.dword  _gp
-- 
1.7.9.5

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [Patch 2/2] MIPS: do not modify variable before relocate_code

2012-11-24 Thread Zhi-zhou Zhang
Because timestamp is declared as `static', we needn't initialize
it by writing it a zero. If we do it before relocate_code, we
will write into a flash address(0xbfc0).

Signed-off-by: Zhi-zhou Zhang zhizhou...@gmail.com
---
 arch/mips/cpu/mips32/time.c |1 -
 arch/mips/cpu/mips64/time.c |1 -
 2 files changed, 2 deletions(-)

diff --git a/arch/mips/cpu/mips32/time.c b/arch/mips/cpu/mips32/time.c
index 350896a..09fc842 100644
--- a/arch/mips/cpu/mips32/time.c
+++ b/arch/mips/cpu/mips32/time.c
@@ -36,7 +36,6 @@ static unsigned long timestamp;
 int timer_init(void)
 {
/* Set up the timer for the first expiration. */
-   timestamp = 0;
write_c0_compare(read_c0_count() + CYCLES_PER_JIFFY);
 
return 0;
diff --git a/arch/mips/cpu/mips64/time.c b/arch/mips/cpu/mips64/time.c
index 5154280..720f7b7 100644
--- a/arch/mips/cpu/mips64/time.c
+++ b/arch/mips/cpu/mips64/time.c
@@ -37,7 +37,6 @@ static unsigned long timestamp;
 int timer_init(void)
 {
/* Set up the timer for the first expiration. */
-   timestamp = 0;
write_c0_compare(read_c0_count() + CYCLES_PER_JIFFY);
 
return 0;
-- 
1.7.9.5

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] mx28: Remove SET, CLR, TOG ops from PLLxCTRL1 registers

2012-11-24 Thread Marek Vasut
These registers don't have _SET, _CLR and _TOG at the respective offsets
available, these registers has to be toggled via R-M-W if needed. Thus do
not export these offsets anymore.

Signed-off-by: Marek Vasut ma...@denx.de
Cc: Stefano Babic sba...@denx.de
---
 arch/arm/include/asm/arch-mxs/regs-clkctrl-mx28.h |6 --
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/arch/arm/include/asm/arch-mxs/regs-clkctrl-mx28.h 
b/arch/arm/include/asm/arch-mxs/regs-clkctrl-mx28.h
index b662fbe..23e9adc 100644
--- a/arch/arm/include/asm/arch-mxs/regs-clkctrl-mx28.h
+++ b/arch/arm/include/asm/arch-mxs/regs-clkctrl-mx28.h
@@ -31,9 +31,11 @@
 #ifndef__ASSEMBLY__
 struct mxs_clkctrl_regs {
mxs_reg_32(hw_clkctrl_pll0ctrl0)/* 0x00 */
-   mxs_reg_32(hw_clkctrl_pll0ctrl1)/* 0x10 */
+   uint32_thw_clkctrl_pll0ctrl1;   /* 0x10 */
+   uint32_treserved_pll0ctrl1[3];  /* 0x14-0x1c */
mxs_reg_32(hw_clkctrl_pll1ctrl0)/* 0x20 */
-   mxs_reg_32(hw_clkctrl_pll1ctrl1)/* 0x30 */
+   uint32_thw_clkctrl_pll1ctrl1;   /* 0x30 */
+   uint32_treserved_pll1ctrl1[3];  /* 0x34-0x3c */
mxs_reg_32(hw_clkctrl_pll2ctrl0)/* 0x40 */
mxs_reg_32(hw_clkctrl_cpu)  /* 0x50 */
mxs_reg_32(hw_clkctrl_hbus) /* 0x60 */
-- 
1.7.10.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] mx28: Rename regs-power.h to regs-power-mx28.h

2012-11-24 Thread Marek Vasut
The i.MX23 has different register layout and bit placement in the
power supply. Thus, in order to be able to add support for MX23,
rename the MX28's regs-power.h to regs-power-mx28.h . Moreover,
add ifdef around inclusion of regs-*-mx28.h in imx-regs.h so the
MX23 boards will include proper set of registers.

Signed-off-by: Marek Vasut ma...@denx.de
Cc: Stefano Babic sba...@denx.de
---
 arch/arm/include/asm/arch-mxs/imx-regs.h|7 +--
 .../include/asm/arch-mxs/{regs-power.h = regs-power-mx28.h}|0
 2 files changed, 5 insertions(+), 2 deletions(-)
 rename arch/arm/include/asm/arch-mxs/{regs-power.h = regs-power-mx28.h} (100%)

diff --git a/arch/arm/include/asm/arch-mxs/imx-regs.h 
b/arch/arm/include/asm/arch-mxs/imx-regs.h
index 5e1901e..9764041 100644
--- a/arch/arm/include/asm/arch-mxs/imx-regs.h
+++ b/arch/arm/include/asm/arch-mxs/imx-regs.h
@@ -26,7 +26,6 @@
 #include asm/arch/regs-apbh.h
 #include asm/arch/regs-base.h
 #include asm/arch/regs-bch.h
-#include asm/arch/regs-clkctrl-mx28.h
 #include asm/arch/regs-digctl.h
 #include asm/arch/regs-gpmi.h
 #include asm/arch/regs-i2c.h
@@ -34,9 +33,13 @@
 #include asm/arch/regs-lradc.h
 #include asm/arch/regs-ocotp.h
 #include asm/arch/regs-pinctrl.h
-#include asm/arch/regs-power.h
 #include asm/arch/regs-rtc.h
 #include asm/arch/regs-ssp.h
 #include asm/arch/regs-timrot.h
 
+#ifdef CONFIG_MX28
+#include asm/arch/regs-clkctrl-mx28.h
+#include asm/arch/regs-power-mx28.h
+#endif
+
 #endif /* __IMX_REGS_H__ */
diff --git a/arch/arm/include/asm/arch-mxs/regs-power.h 
b/arch/arm/include/asm/arch-mxs/regs-power-mx28.h
similarity index 100%
rename from arch/arm/include/asm/arch-mxs/regs-power.h
rename to arch/arm/include/asm/arch-mxs/regs-power-mx28.h
-- 
1.7.10.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] mx23: Add POWER and CLKCTRL register definitions

2012-11-24 Thread Marek Vasut
Add register definitions for the i.MX23 power control block and
clock control block. These are essential for the basic bootstrap
of the i.MX23. Also, properly include them in imx-regs.h .

Signed-off-by: Marek Vasut ma...@denx.de
Cc: Stefano Babic sba...@denx.de
---
 arch/arm/include/asm/arch-mxs/imx-regs.h  |5 +
 arch/arm/include/asm/arch-mxs/regs-clkctrl-mx23.h |  221 +
 arch/arm/include/asm/arch-mxs/regs-power-mx23.h   |  357 +
 3 files changed, 583 insertions(+)
 create mode 100644 arch/arm/include/asm/arch-mxs/regs-clkctrl-mx23.h
 create mode 100644 arch/arm/include/asm/arch-mxs/regs-power-mx23.h

diff --git a/arch/arm/include/asm/arch-mxs/imx-regs.h 
b/arch/arm/include/asm/arch-mxs/imx-regs.h
index 9764041..563e4a3 100644
--- a/arch/arm/include/asm/arch-mxs/imx-regs.h
+++ b/arch/arm/include/asm/arch-mxs/imx-regs.h
@@ -37,6 +37,11 @@
 #include asm/arch/regs-ssp.h
 #include asm/arch/regs-timrot.h
 
+#ifdef CONFIG_MX23
+#include asm/arch/regs-clkctrl-mx23.h
+#include asm/arch/regs-power-mx23.h
+#endif
+
 #ifdef CONFIG_MX28
 #include asm/arch/regs-clkctrl-mx28.h
 #include asm/arch/regs-power-mx28.h
diff --git a/arch/arm/include/asm/arch-mxs/regs-clkctrl-mx23.h 
b/arch/arm/include/asm/arch-mxs/regs-clkctrl-mx23.h
new file mode 100644
index 000..ba2dae2
--- /dev/null
+++ b/arch/arm/include/asm/arch-mxs/regs-clkctrl-mx23.h
@@ -0,0 +1,221 @@
+/*
+ * Freescale i.MX23 CLKCTRL Register Definitions
+ *
+ * Copyright (C) 2012 Marek Vasut marek.va...@gmail.com
+ * on behalf of DENX Software Engineering GmbH
+ *
+ * Based on code from LTIB:
+ * Copyright 2008-2010 Freescale Semiconductor, Inc. All Rights Reserved.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
+ *
+ */
+
+#ifndef __MX23_REGS_CLKCTRL_H__
+#define __MX23_REGS_CLKCTRL_H__
+
+#include asm/arch/regs-common.h
+
+#ifndef__ASSEMBLY__
+struct mxs_clkctrl_regs {
+   mxs_reg_32(hw_clkctrl_pll0ctrl0)/* 0x00 */
+   uint32_thw_clkctrl_pll0ctrl1;   /* 0x10 */
+   uint32_treserved_pll0ctrl1[3];  /* 0x14-0x1c */
+   mxs_reg_32(hw_clkctrl_cpu)  /* 0x20 */
+   mxs_reg_32(hw_clkctrl_hbus) /* 0x30 */
+   mxs_reg_32(hw_clkctrl_xbus) /* 0x40 */
+   mxs_reg_32(hw_clkctrl_xtal) /* 0x50 */
+   mxs_reg_32(hw_clkctrl_pix)  /* 0x60 */
+   mxs_reg_32(hw_clkctrl_ssp0) /* 0x70 */
+   mxs_reg_32(hw_clkctrl_gpmi) /* 0x80 */
+   mxs_reg_32(hw_clkctrl_spdif)/* 0x90 */
+   mxs_reg_32(hw_clkctrl_emi)  /* 0xa0 */
+
+   uint32_treserved1[4];
+
+   mxs_reg_32(hw_clkctrl_saif0)/* 0xc0 */
+   mxs_reg_32(hw_clkctrl_tv)   /* 0xd0 */
+   mxs_reg_32(hw_clkctrl_etm)  /* 0xe0 */
+   mxs_reg_8(hw_clkctrl_frac0) /* 0xf0 */
+   mxs_reg_8(hw_clkctrl_frac1) /* 0x100 */
+   mxs_reg_32(hw_clkctrl_clkseq)   /* 0x110 */
+   mxs_reg_32(hw_clkctrl_reset)/* 0x120 */
+   mxs_reg_32(hw_clkctrl_status)   /* 0x130 */
+   mxs_reg_32(hw_clkctrl_version)  /* 0x140 */
+};
+#endif
+
+#defineCLKCTRL_PLL0CTRL0_LFR_SEL_MASK  (0x3  28)
+#defineCLKCTRL_PLL0CTRL0_LFR_SEL_OFFSET28
+#defineCLKCTRL_PLL0CTRL0_LFR_SEL_DEFAULT   (0x0  28)
+#defineCLKCTRL_PLL0CTRL0_LFR_SEL_TIMES_2   (0x1  28)
+#defineCLKCTRL_PLL0CTRL0_LFR_SEL_TIMES_05  (0x2  28)
+#defineCLKCTRL_PLL0CTRL0_LFR_SEL_UNDEFINED (0x3  28)
+#defineCLKCTRL_PLL0CTRL0_CP_SEL_MASK   (0x3  24)
+#defineCLKCTRL_PLL0CTRL0_CP_SEL_OFFSET 24
+#defineCLKCTRL_PLL0CTRL0_CP_SEL_DEFAULT(0x0  24)
+#defineCLKCTRL_PLL0CTRL0_CP_SEL_TIMES_2(0x1  24)
+#defineCLKCTRL_PLL0CTRL0_CP_SEL_TIMES_05   (0x2  24)
+#defineCLKCTRL_PLL0CTRL0_CP_SEL_UNDEFINED  (0x3  24)
+#defineCLKCTRL_PLL0CTRL0_DIV_SEL_MASK  (0x3  20)
+#defineCLKCTRL_PLL0CTRL0_DIV_SEL_OFFSET20
+#defineCLKCTRL_PLL0CTRL0_DIV_SEL_DEFAULT   (0x0  20)
+#defineCLKCTRL_PLL0CTRL0_DIV_SEL_LOWER (0x1  20)
+#defineCLKCTRL_PLL0CTRL0_DIV_SEL_LOWEST(0x2  20)
+#define 

Re: [U-Boot] Reg Bootstrapping u-boot on x86-64 for tizen

2012-11-24 Thread Graeme Russ
Hi Manohar,

On 11/24/2012 06:08 PM, manohar.bet...@smartplayin.com wrote:
 
 Hi,
  
 I am getting the following error  after running make all for coreboot 
 target,from /usr/bin/ld.bfd.real: skipping incompatible 
 /home/manoharb/Kernel_Tizen/intel_tizen/latest/u-boot/arch/x86/lib/libgcc.a 
 when searching for -lgcc
 /usr/bin/ld.bfd.real: cannot find -lgcc
 make: *** [u-boot] Error 1

What toolchain (and version) are you using? I've never seen ld.bfd.real before

Regards,

Graeme
  
 Please help me.
  
 Best Regards,
 Manohar
  
 
 
 -Original Message-
 From: Graeme Russ graeme.r...@gmail.com
 Sent: Friday, November 23, 2012 4:23pm
 To: manohar.bet...@smartplayin.com
 Cc: Marek Vasut ma...@denx.de, u-boot@lists.denx.de, Simon Glass 
 s...@chromium.org
 Subject: Re: [U-Boot] Reg Bootstrapping u-boot on x86-64 for tizen
 
 
 
 Hi Manohar,
 
 On 11/23/2012 09:10 PM, manohar.bet...@smartplayin.com wrote:

 Dear Graeme,
  
 Thank you!
  
 I downloaded the u-boot-x86 on Ubuntu 11.10 loaded(Linux smart-OptiPlex-390 
 3.0.0-26-generic  x86_64 GNU/Linux) intel i5 M1H61R-MB montherboard from the 
 given below link.
  
 http://git.denx.de/?p=u-boot/u-boot-x86.git;a=summary
 
 Depending on when you downloaded it, you may not have the latest mainline
 which is at:
 
 http://git.denx.de/?p=u-boot.git;a=summary
 
 Nevertheless, you will have the latest x86 patches
 
  
 and yes I am planning to run u-boot on x86_64 machine and my target is also 
 x86_64 machine are the same .
 cd u-boot-x86
 
 Hmm, I don't think U-Boot is what your after. There are a few problems you
 will be faced with:
  - There is no ACPI support (no power management)
  - No System Management Mode (SMM) support
  - No chipset support (No RAM initialisation)
  - No Cache-As-RAM init code
 
 Unless you have at least these covered, you will brick your board if you
 try to flash U-Boot. You might instead take a look at coreboot
 
 opened the boards.config file and appended my board details in the config 
 file .
  
 MIH61R-MBx86 x86MIH61R-MB
 GenuineIntel   -
 # Target ARCHCPU Board name  
 VendorSoC Options
  
  
 and then run the below commands and i didnot modify any code other than that 
 above line.
 .
  
 manoharb@smart-OptiPlex-390:~/Kernel_Tizen/intel_tizen/u-boot-x86$ vim 
 boards.cfg 
 manoharb@smart-OptiPlex-390:~/Kernel_Tizen/intel_tizen/u-boot-x86$ make 
 distclean
 /bin/bash: i386-linux-gcc: command not found
 /bin/bash: i386-linux-gcc: command not found
 
 [snip]
 
 Ah, I now see that the patch I thought had been applied has not :(
 
 You will need this patch:
 
 http://patchwork.ozlabs.org/patch/155727/
 
 Regards,
 
 Graeme
 
 ...
  
 Please help me what are the files i need to modify and provide me the 
 documentation.
  
 Thank you!
 Best Regards,
 Manohar
 8790215215
  
  
 -Original Message-
 From: Graeme Russ graeme.r...@gmail.com
 Sent: Friday, November 23, 2012 12:12pm
 To: manohar.bet...@smartplayin.com
 Cc: Marek Vasut ma...@denx.de, u-boot@lists.denx.de
 Subject: Re: [U-Boot] Reg Bootstrapping u-boot on x86-64 for tizen



 Hi Manohar,

 On 11/23/2012 04:46 PM, manohar.bet...@smartplayin.com wrote:

 Dear Marek,
  
 Thank you for the reply.
  
 Can you please send me the steps/procedure to build/support u-boot on 
 x86_64 bit (Intel(R) Core(TM) i5-2400 CPU @ 3.10GHz.

 I build the x86 U-Boot target on an x86_64 i7 Linux (Fedora) system using
 the standard GNU gcc package. There was an issue with the Makefile and
 there was a need to manually create some toolchain symlinks, but the was
 rectified some time ago. Are you using the latest mainline U-Boot from
 git.denx.de? If so, what errors are you getting - please provide a copy of
 your build output.

 We need help from you is there any documentation to enable x86_64 bit pc or 
 links to post my query.
  
 What are the config/code i need to change to make my pc as host  target 
 are the same.

 Are you planning on running U-Boot on an x86_64 machine? If so, you will
 need to create code to support that - Current U-Boot only supports 32-bit
 x86 targets. The sandbox target (designed for testing on a Linux host, not
 for downloading onto a board) should run on an x86_64 host.

  
 Please help us to build the u-boot on x86_64.

 Please provide more detail - what have you tried? What was the result? The
 build scripts should work as is on a properly configured x86_64 build machine

 

Re: [U-Boot] [PATCH] mx23: Add POWER and CLKCTRL register definitions

2012-11-24 Thread Otavio Salvador
On Sat, Nov 24, 2012 at 10:21 PM, Marek Vasut ma...@denx.de wrote:
 Add register definitions for the i.MX23 power control block and
 clock control block. These are essential for the basic bootstrap
 of the i.MX23. Also, properly include them in imx-regs.h .

 Signed-off-by: Marek Vasut ma...@denx.de
 Cc: Stefano Babic sba...@denx.de

Please don't merge it now as this has been already done in my branch with mx23.

-- 
Otavio Salvador O.S. Systems
E-mail: ota...@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854  http://projetos.ossystems.com.br
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] mx23: Add POWER and CLKCTRL register definitions

2012-11-24 Thread Marek Vasut
Dear Otavio Salvador,

 On Sat, Nov 24, 2012 at 10:21 PM, Marek Vasut ma...@denx.de wrote:
  Add register definitions for the i.MX23 power control block and
  clock control block. These are essential for the basic bootstrap
  of the i.MX23. Also, properly include them in imx-regs.h .
  
  Signed-off-by: Marek Vasut ma...@denx.de
  Cc: Stefano Babic sba...@denx.de
 
 Please don't merge it now as this has been already done in my branch with
 mx23.

I'm attaching a proper diff against your branch. There are multiple wrong and 
missing bits in your branch. This patch is done by processing the FSL-provided 
header file and cross-checking with the datasheet.

So, this patch is right, yours is wrong. Sorry.

--- arch/arm/include/asm/arch-mxs/regs-clkctrl-mx23.h   2012-10-01 
02:46:18.0 +0200
+++ ../u-boot-imx/arch/arm/include/asm/arch-mxs/regs-clkctrl-mx23.h 
2012-11-25 00:55:43.299072391 +0100
@@ -1,8 +1,11 @@
 /*
  * Freescale i.MX23 CLKCTRL Register Definitions
  *
- * Copyright (C) 2012 Otavio Salvador ota...@ossystems.com.br
- * on behalf of O.S. Systems Software LTDA.
+ * Copyright (C) 2012 Marek Vasut marek.va...@gmail.com
+ * on behalf of DENX Software Engineering GmbH
+ *
+ * Based on code from LTIB:
+ * Copyright 2008-2010 Freescale Semiconductor, Inc. All Rights Reserved.
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -28,7 +31,8 @@
 #ifndef__ASSEMBLY__
 struct mxs_clkctrl_regs {
mxs_reg_32(hw_clkctrl_pll0ctrl0)/* 0x00 */
-   mxs_reg_32(hw_clkctrl_pll0ctrl1)/* 0x10 */
+   uint32_thw_clkctrl_pll0ctrl1;   /* 0x10 */
+   uint32_treserved_pll0ctrl1[3];  /* 0x14-0x1c */
mxs_reg_32(hw_clkctrl_cpu)  /* 0x20 */
mxs_reg_32(hw_clkctrl_hbus) /* 0x30 */
mxs_reg_32(hw_clkctrl_xbus) /* 0x40 */
@@ -39,15 +43,13 @@
mxs_reg_32(hw_clkctrl_spdif)/* 0x90 */
mxs_reg_32(hw_clkctrl_emi)  /* 0xa0 */
 
-   uint32_t reserved[4];
-
-   mxs_reg_32(hw_clkctrl_saif) /* 0x0c0 */
-   mxs_reg_32(hw_clkctrl_tv)   /* 0x0d0 */
-   mxs_reg_32(hw_clkctrl_etm)  /* 0x0e0 */
-
-   mxs_reg_8(hw_clkctrl_frac0) /* 0x0f0 */
-   mxs_reg_8(hw_clkctrl_frac1) /* 0x100 */
+   uint32_treserved1[4];
 
+   mxs_reg_32(hw_clkctrl_saif0)/* 0xc0 */
+   mxs_reg_32(hw_clkctrl_tv)   /* 0xd0 */
+   mxs_reg_32(hw_clkctrl_etm)  /* 0xe0 */
+   mxs_reg_8(hw_clkctrl_frac0) /* 0xf0 */
+   mxs_reg_8(hw_clkctrl_frac1) /* 0x100 */
mxs_reg_32(hw_clkctrl_clkseq)   /* 0x110 */
mxs_reg_32(hw_clkctrl_reset)/* 0x120 */
mxs_reg_32(hw_clkctrl_status)   /* 0x130 */
@@ -87,11 +89,11 @@
 #defineCLKCTRL_CPU_DIV_XTAL_MASK   (0x3ff  16)
 #defineCLKCTRL_CPU_DIV_XTAL_OFFSET 16
 #defineCLKCTRL_CPU_INTERRUPT_WAIT  (1  12)
-#defineCLKCTRL_CPU_DIV_CPU_FRAC_EN (1  5)
+#defineCLKCTRL_CPU_DIV_CPU_FRAC_EN (1  10)
 #defineCLKCTRL_CPU_DIV_CPU_MASK0x3f
 #defineCLKCTRL_CPU_DIV_CPU_OFFSET  0
 
-#defineCLKCTRL_HBUS_ASM_BUSY   (1  29)
+#defineCLKCTRL_HBUS_BUSY   (1  29)
 #defineCLKCTRL_HBUS_DCP_AS_ENABLE  (1  28)
 #defineCLKCTRL_HBUS_PXP_AS_ENABLE  (1  27)
 #defineCLKCTRL_HBUS_APBHDMA_AS_ENABLE  (1  26)
@@ -101,7 +103,7 @@
 #defineCLKCTRL_HBUS_CPU_DATA_AS_ENABLE (1  22)
 #defineCLKCTRL_HBUS_CPU_INSTR_AS_ENABLE(1  21)
 #defineCLKCTRL_HBUS_AUTO_SLOW_MODE (1  20)
-#defineCLKCTRL_HBUS_SLOW_DIV_MASK  (0x7  18)
+#defineCLKCTRL_HBUS_SLOW_DIV_MASK  (0x7  16)
 #defineCLKCTRL_HBUS_SLOW_DIV_OFFSET16
 #defineCLKCTRL_HBUS_SLOW_DIV_BY1   (0x0  16)
 #defineCLKCTRL_HBUS_SLOW_DIV_BY2   (0x1  16)
@@ -114,16 +116,25 @@
 #defineCLKCTRL_HBUS_DIV_OFFSET 0
 
 #defineCLKCTRL_XBUS_BUSY   (1  31)
-#defineCLKCTRL_XBUS_DIV_FRAC_EN(1  9)
+#defineCLKCTRL_XBUS_DIV_FRAC_EN(1  10)
 #defineCLKCTRL_XBUS_DIV_MASK   0x3ff
 #defineCLKCTRL_XBUS_DIV_OFFSET 0
 
 #defineCLKCTRL_XTAL_UART_CLK_GATE  (1  31)
+#defineCLKCTRL_XTAL_FILT_CLK24M_GATE   (1  30)
 #defineCLKCTRL_XTAL_PWM_CLK24M_GATE(1  29)
+#defineCLKCTRL_XTAL_DRI_CLK24M_GATE(1  28)
+#defineCLKCTRL_XTAL_DIGCTRL_CLK1M_GATE (1  27)
 #define

Re: [U-Boot] [PATCH] mx23: Add POWER and CLKCTRL register definitions

2012-11-24 Thread Otavio Salvador
On Sun, Nov 25, 2012 at 12:54 AM, Marek Vasut ma...@denx.de wrote:
 Dear Otavio Salvador,

 On Sat, Nov 24, 2012 at 10:21 PM, Marek Vasut ma...@denx.de wrote:
  Add register definitions for the i.MX23 power control block and
  clock control block. These are essential for the basic bootstrap
  of the i.MX23. Also, properly include them in imx-regs.h .
 
  Signed-off-by: Marek Vasut ma...@denx.de
  Cc: Stefano Babic sba...@denx.de

 Please don't merge it now as this has been already done in my branch with
 mx23.

 I'm attaching a proper diff against your branch. There are multiple wrong and
 missing bits in your branch. This patch is done by processing the FSL-provided
 header file and cross-checking with the datasheet.

 So, this patch is right, yours is wrong. Sorry.

I am not saying mine is perfect, far from it. And we can fix it in the
branch but please don't push it until we finish mx23 bootstrap.

Let's work in same code, not duplicate work...

-- 
Otavio Salvador O.S. Systems
E-mail: ota...@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854  http://projetos.ossystems.com.br
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] mx23: Add POWER and CLKCTRL register definitions

2012-11-24 Thread Marek Vasut
Dear Otavio Salvador,

 On Sun, Nov 25, 2012 at 12:54 AM, Marek Vasut ma...@denx.de wrote:
  Dear Otavio Salvador,
  
  On Sat, Nov 24, 2012 at 10:21 PM, Marek Vasut ma...@denx.de wrote:
   Add register definitions for the i.MX23 power control block and
   clock control block. These are essential for the basic bootstrap
   of the i.MX23. Also, properly include them in imx-regs.h .
   
   Signed-off-by: Marek Vasut ma...@denx.de
   Cc: Stefano Babic sba...@denx.de
  
  Please don't merge it now as this has been already done in my branch
  with mx23.
  
  I'm attaching a proper diff against your branch. There are multiple wrong
  and missing bits in your branch. This patch is done by processing the
  FSL-provided header file and cross-checking with the datasheet.
  
  So, this patch is right, yours is wrong. Sorry.
 
 I am not saying mine is perfect, far from it. And we can fix it in the
 branch but please don't push it until we finish mx23 bootstrap.
 
 Let's work in same code, not duplicate work...

I still don't see any argument against applying this patch -- other than the 
fact it will break some out-of-tree port.

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v4 4/4] ARM: lacie_kw: add support for WIRELESS_SPACE

2012-11-24 Thread Albert ARIBAUD
Hi Prafulla,

On Fri, 9 Nov 2012 23:09:50 -0800, Prafulla Wadaskar
prafu...@marvell.com wrote:

 
 
  -Original Message-
  From: Albert ARIBAUD [mailto:albert.u.b...@aribaud.net]
  Sent: 05 November 2012 05:02
  To: U-Boot
  Cc: Prafulla Wadaskar; Simon Guinot; Albert ARIBAUD
  Subject: [PATCH v4 4/4] ARM: lacie_kw: add support for WIRELESS_SPACE
 
  Signed-off-by: Albert ARIBAUD albert.u.b...@aribaud.net
  ---
  Changes in v4:
  - switched from netspace_V2-based to standalone board
  - added support for mv88e61xx switch
  - corrected some kwbimage.cfg values
 
  Changes in v3:
  - fix broken support for NETSPACE_(MINI|LITE)_V2
 
  Changes in v2:
  - split the patch in two: mvgbe phy/port changes and WS support.
  - removed spurious DEBUG define
  - fixed various checkpatch errors/warnings/typos
 
   board/LaCie/wireless_space/Makefile |   46 +++
   board/LaCie/wireless_space/kwbimage.cfg |  190
  +++
   board/LaCie/wireless_space/wireless_space.c |  165
  +++
   boards.cfg  |1 +
   include/configs/wireless_space.h|  183
  ++
   5 files changed, 585 insertions(+)
   create mode 100644 board/LaCie/wireless_space/Makefile
   create mode 100644 board/LaCie/wireless_space/kwbimage.cfg
   create mode 100644 board/LaCie/wireless_space/wireless_space.c
   create mode 100644 include/configs/wireless_space.h
 
  diff --git a/board/LaCie/wireless_space/Makefile
  b/board/LaCie/wireless_space/Makefile
  new file mode 100644
  index 000..b43c3d3
  --- /dev/null
  +++ b/board/LaCie/wireless_space/Makefile
  @@ -0,0 +1,46 @@
  +#
  +# Copyright (C) 2011 Simon Guinot sgui...@lacie.com
  +#
  +# Based on Kirkwood support:
  +# (C) Copyright 2009
  +# Marvell Semiconductor www.marvell.com
  +# Written-by: Prafulla Wadaskar prafu...@marvell.com
  +#
  +# See file CREDITS for list of people who contributed to this
  +# project.
  +#
  +# This program is free software; you can redistribute it and/or
  +# modify it under the terms of the GNU General Public License as
  +# published by the Free Software Foundation; either version 2 of
  +# the License, or (at your option) any later version.
  +#
  +# This program is distributed in the hope that it will be useful,
  +# but WITHOUT ANY WARRANTY; without even the implied warranty of
  +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.See the
  +# GNU General Public License for more details.
  +#
  +
  +include $(TOPDIR)/config.mk
  +ifneq ($(OBJTREE),$(SRCTREE))
  +$(shell mkdir -p $(obj)../common)
  +endif
  +
  +LIB  = $(obj)lib$(BOARD).o
  +
  +COBJS:= $(BOARD).o ../common/common.o
  +
  +SRCS := $(SOBJS:.o=.S) $(COBJS:.o=.c)
  +OBJS := $(addprefix $(obj),$(COBJS))
  +SOBJS:= $(addprefix $(obj),$(SOBJS))
  +
  +$(LIB):  $(obj).depend $(OBJS) $(SOBJS)
  + $(call cmd_link_o_target, $(OBJS) $(SOBJS))
  +
  +#
  
  +
  +# defines $(obj).depend target
  +include $(SRCTREE)/rules.mk
  +
  +sinclude $(obj).depend
  +
  +#
  
  diff --git a/board/LaCie/wireless_space/kwbimage.cfg
  b/board/LaCie/wireless_space/kwbimage.cfg
  new file mode 100644
  index 000..a5b200f
  --- /dev/null
  +++ b/board/LaCie/wireless_space/kwbimage.cfg
  @@ -0,0 +1,190 @@
  +#
  +# Copyright (C) 2012 Albert ARIBAUD albert.u.b...@aribaud.net
  +#
  +# Based on netspace_v2 kwbimage.cfg:
  +# Copyright (C) 2011 Simon Guinot sgui...@lacie.com
  +#
  +# Based on Kirkwood support:
  +# (C) Copyright 2009
  +# Marvell Semiconductor www.marvell.com
  +# Written-by: Prafulla Wadaskar prafu...@marvell.com
  +#
  +# See file CREDITS for list of people who contributed to this
  +# project.
  +#
  +# This program is free software; you can redistribute it and/or
  +# modify it under the terms of the GNU General Public License as
  +# published by the Free Software Foundation; either version 2 of
  +# the License, or (at your option) any later version.
  +#
  +# This program is distributed in the hope that it will be useful,
  +# but WITHOUT ANY WARRANTY; without even the implied warranty of
  +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
  +# GNU General Public License for more details.
  +#
  +# Refer docs/README.kwimage for more details about how-to configure
  +# and create kirkwood boot image
  +#
  +
  +# Boot Media configurations
  +BOOT_FROMnand# Boot from NAND flash
  +
  +# SOC registers configuration using bootrom header extension
  +# Maximum KWBIMAGE_MAX_CONFIG configurations allowed
  +
  +# Configure RGMII-0 interface pad voltage to 1.8V
  +DATA 0xFFD100e0 0x1B1B1B9B
  +
  +#Dram initalization for SINGLE x16 CL=5 @ 400MHz
  +DATA 0xFFD01400 0x43000618   # DDR Configuration register
  +# bit13-0:  0xa00 (2560 DDR2 clks refresh rate)
  +# bit23-14: zero
  +# 

Re: [U-Boot] [PATCH v4 1/3] Add README for the Falcon mode

2012-11-24 Thread Vikram Narayanan

Hello Andreas,

On 11/24/2012 3:29 AM, Andreas Bießmann wrote:

Dear Vikram Narayanan,

On 23.11.12 19:10, Vikram Narayanan wrote:

snip


+The parameters generated with this step can be saved into NAND at the
offset
+0x80 (value for twister for CONFIG_CMD_SPL_NAND_OFS)
+
+nand erase.part bootparms
+nand write 0x8100 bootparms 0x4000


If the offset is known at compile time, why should the end user use the
above commands to write it? Can't it be automated? Just an idea.


No, please read
http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/102326
or
http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/147205



Thanks. Clarified.

Regards,
Vikram
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v4 4/4] ARM: lacie_kw: add support for WIRELESS_SPACE

2012-11-24 Thread Prafulla Wadaskar


 -Original Message-
 From: Albert ARIBAUD [mailto:albert.u.b...@aribaud.net]
 Sent: 25 November 2012 08:51
 To: Prafulla Wadaskar
 Cc: U-Boot; Simon Guinot
 Subject: Re: [PATCH v4 4/4] ARM: lacie_kw: add support for
 WIRELESS_SPACE

 Hi Prafulla,

 On Fri, 9 Nov 2012 23:09:50 -0800, Prafulla Wadaskar
 prafu...@marvell.com wrote:

 
 
   -Original Message-
   From: Albert ARIBAUD [mailto:albert.u.b...@aribaud.net]
   Sent: 05 November 2012 05:02
   To: U-Boot
   Cc: Prafulla Wadaskar; Simon Guinot; Albert ARIBAUD
   Subject: [PATCH v4 4/4] ARM: lacie_kw: add support for
 WIRELESS_SPACE
  
   Signed-off-by: Albert ARIBAUD albert.u.b...@aribaud.net
   ---
   Changes in v4:
   - switched from netspace_V2-based to standalone board
   - added support for mv88e61xx switch
   - corrected some kwbimage.cfg values
  
   Changes in v3:
   - fix broken support for NETSPACE_(MINI|LITE)_V2
  
   Changes in v2:
   - split the patch in two: mvgbe phy/port changes and WS support.
   - removed spurious DEBUG define
   - fixed various checkpatch errors/warnings/typos
  
board/LaCie/wireless_space/Makefile |   46 +++
board/LaCie/wireless_space/kwbimage.cfg |  190
   +++
board/LaCie/wireless_space/wireless_space.c |  165
   +++
boards.cfg  |1 +
include/configs/wireless_space.h|  183
   ++
5 files changed, 585 insertions(+)
create mode 100644 board/LaCie/wireless_space/Makefile
create mode 100644 board/LaCie/wireless_space/kwbimage.cfg
create mode 100644 board/LaCie/wireless_space/wireless_space.c
create mode 100644 include/configs/wireless_space.h
  
   diff --git a/board/LaCie/wireless_space/Makefile
   b/board/LaCie/wireless_space/Makefile
   new file mode 100644
   index 000..b43c3d3
   --- /dev/null
   +++ b/board/LaCie/wireless_space/Makefile
   @@ -0,0 +1,46 @@
   +#
   +# Copyright (C) 2011 Simon Guinot sgui...@lacie.com
   +#
   +# Based on Kirkwood support:
   +# (C) Copyright 2009
   +# Marvell Semiconductor www.marvell.com
   +# Written-by: Prafulla Wadaskar prafu...@marvell.com
   +#
   +# See file CREDITS for list of people who contributed to this
   +# project.
   +#
   +# This program is free software; you can redistribute it and/or
   +# modify it under the terms of the GNU General Public License as
   +# published by the Free Software Foundation; either version 2 of
   +# the License, or (at your option) any later version.
   +#
   +# This program is distributed in the hope that it will be useful,
   +# but WITHOUT ANY WARRANTY; without even the implied warranty of
   +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.See
 the
   +# GNU General Public License for more details.
   +#
   +
   +include $(TOPDIR)/config.mk
   +ifneq ($(OBJTREE),$(SRCTREE))
   +$(shell mkdir -p $(obj)../common)
   +endif
   +
   +LIB  = $(obj)lib$(BOARD).o
   +
   +COBJS:= $(BOARD).o ../common/common.o
   +
   +SRCS := $(SOBJS:.o=.S) $(COBJS:.o=.c)
   +OBJS := $(addprefix $(obj),$(COBJS))
   +SOBJS:= $(addprefix $(obj),$(SOBJS))
   +
   +$(LIB):  $(obj).depend $(OBJS) $(SOBJS)
   + $(call cmd_link_o_target, $(OBJS) $(SOBJS))
   +
  
 +#
   
   +
   +# defines $(obj).depend target
   +include $(SRCTREE)/rules.mk
   +
   +sinclude $(obj).depend
   +
  
 +#
   
   diff --git a/board/LaCie/wireless_space/kwbimage.cfg
   b/board/LaCie/wireless_space/kwbimage.cfg
   new file mode 100644
   index 000..a5b200f
   --- /dev/null
   +++ b/board/LaCie/wireless_space/kwbimage.cfg
   @@ -0,0 +1,190 @@
   +#
   +# Copyright (C) 2012 Albert ARIBAUD albert.u.b...@aribaud.net
   +#
   +# Based on netspace_v2 kwbimage.cfg:
   +# Copyright (C) 2011 Simon Guinot sgui...@lacie.com
   +#
   +# Based on Kirkwood support:
   +# (C) Copyright 2009
   +# Marvell Semiconductor www.marvell.com
   +# Written-by: Prafulla Wadaskar prafu...@marvell.com
   +#
   +# See file CREDITS for list of people who contributed to this
   +# project.
   +#
   +# This program is free software; you can redistribute it and/or
   +# modify it under the terms of the GNU General Public License as
   +# published by the Free Software Foundation; either version 2 of
   +# the License, or (at your option) any later version.
   +#
   +# This program is distributed in the hope that it will be useful,
   +# but WITHOUT ANY WARRANTY; without even the implied warranty of
   +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
   +# GNU General Public License for more details.
   +#
   +# Refer docs/README.kwimage for more details about how-to
 configure
   +# and create kirkwood boot image
   +#
   +
   +# Boot Media configurations
   +BOOT_FROMnand# Boot from NAND flash
   +
   +# SOC registers