All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 6/6] MX35: add support for woodburn board
Date: Sat, 8 Dec 2012 08:58:43 -0500	[thread overview]
Message-ID: <50C34793.5070304@ti.com> (raw)
In-Reply-To: <50C32351.9090705@denx.de>

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 12/08/12 06:24, Stefano Babic wrote:
> On 08/12/2012 10:31, Albert ARIBAUD wrote:
>> Hi Stefano,
>> 
> 
> Hi Albert,
> 
>> On Thu, 11 Oct 2012 09:11:46 +0200, Stefano Babic
>> <sbabic@denx.de> wrote:
>> 
>>> The woodburn board is based on the MX35 SOC. Support for both
>>> external (NOR) and internal (SD Card) boot mode are added. It
>>> uses the generic SPL framework to implement the internal boot 
>>> mode.
>>> 
>>> The following peripherals are supported: - Ethernet (FEC) - SD
>>> Card - NAND (512 MB) - NOR Flash
>>> 
>>> In the internal boot mode, a simple imximage header is
>>> generated to set the address in internal RAM where the SOC must
>>> copy the SPL code. The initial setup is then demanded to the
>>> SPL itself.
>>> 
>>> Signed-off-by: Stefano Babic <sbabic@denx.de>
>> 
>> It seems this board uses the old PMIC framework, so it does not
>> merge properly with u-boot/master which has switched to power
>> in-between. Can you look into this?
>> 
>> I must say I'm not entirely sure how to handle this, but here is
>> my proposal: AFAIU this patch was applied initially in your imx
>> tree, so I suggest that you merge u-boot/master into your tree
>> (with a comment to the effect that you need this to resolve PMIC
>> vs POWER conflicts) or into an ad hoc branch, and resolve the
>> conflicts there, then send a pull request to me for your resolved
>> tree or branch.
> 
> There are conflicts with three boards : mx35pdk, woodburn and
> mx53loco.
> 
> It seems that a couple of patches were lost in the transition from
> pmic to power, too:
> 
> commit 787f4f30823a15b1e45c0a0b901cfa5bd415e251 Author: Fabio
> Estevam <fabio.estevam@freescale.com> Date:   Tue Oct 23 06:36:18
> 2012 +0000
> 
> pmic: Add support for mc34704
> 
> Add the register layout for the MC34704 PMIC from Freescale.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> 
> commit 4cfc6c4fa7611e79e6434e97db56b6072ad4d05c Author: Fabio
> Estevam <fabio.estevam@freescale.com> Date:   Tue Oct 23 06:34:50
> 2012 +0000
> 
> pmic_fsl: Introduce FSL_PMIC_I2C_LENGTH
> 
> I solved the conflicts in the way you suggested and I pushed a
> temporary branch "master_merge" on u-boot-imx.
> 
> I am unsure, too, if this is the best way to solve these conflicts,
> but if Tom agrees on our way to proceed, I will post a new pull
> request.

I'm OK with this method now, so long as at the end of the day, git
diff looks good.

- -- 
Tom

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://www.enigmail.net/

iQIcBAEBAgAGBQJQw0eSAAoJENk4IS6UOR1W/7AP/R5Jmr+oQVJ6hSpw4tPD7iTO
O/ofmOAKFJ4+4fgnzIY75f06aTjiGMX0sC6sttvQDwKGGJHUJmphf6Le39WwxwRR
/pVQCshsCu4s4IWnQq4lNq4I1SvinLdduu5kR+QddieB8GxsHDBB1a1SBtKzS0/2
ERkbKfI6n4lXJT2mrR+pe7ckJMSemLYO+sk8sa9JaAhhM3Ny4NhaEa8iIdmULSXx
lgG8dKAZz+SrTIej5NJwPcLIyq/03E8PxGB/R8vl6GBKp4OPWJL4Wz8UXADyK0Mi
sUrqfFIsSTVdmNa1NeGP1Ymj+VeQZEnk2+eXaZE+u+4k4BMgAmFINZzw6bYLc8Ar
u47GEJlXFrwjsOUITkjBS3W/BH0/0EYMn8eG/gyCKNpK0HY84o3tOvt15BW9IPa8
DPPmXjUm6tf96QgitvTwCk9UHpQiIw24tBJ9ZmnmxMrYxOM0ZLnJixWuxTJXBccm
HueU+ECBUPpBZMPyo3W8xcNUI7+iV7/s9s1AB+CMQMcnwm6UhMeYnrV+HRL65WKU
qZ6Ynoy7GDwKK1ZAsfqHnd2q2eyGnMYGbB2ryGcSrQxnyvSqTrL/8AU3cevmoW8t
jNSEW8+MPHAF+LK7SgY+QbHVdflkfTDwdD6f3+lJQLlqNq0FzJkdo8VEL4gEfyY9
ucuEsCUNIuggiKJJwxhp
=3Y25
-----END PGP SIGNATURE-----

      reply	other threads:[~2012-12-08 13:58 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-11  7:11 [U-Boot] [PATCH v2 0/6] Added SPL for MX35 boards Stefano Babic
2012-10-11  7:11 ` [U-Boot] [PATCH v2 1/6] ARM: Fix start.S when used with SPL in arm1136 Stefano Babic
2012-10-26 14:48   ` [U-Boot] [PATCH V3 " Stefano Babic
2012-10-26 16:23     ` Fabio Estevam
2012-10-26 16:44       ` Stefano Babic
2012-10-11  7:11 ` [U-Boot] [PATCH v2 2/6] MX35: add LOW_LEVEL_SRAM_STACK to use SPL_FRAMEWORK Stefano Babic
2012-10-11  7:11 ` [U-Boot] [PATCH v2 3/6] MX35: Add soc_boot_mode and soc_boot_device to MX35 Stefano Babic
2012-10-11  7:11 ` [U-Boot] [PATCH v2 4/6] SPL: Added SPL target for mx35 SOC to SPL Makefile Stefano Babic
2012-10-11  7:11 ` [U-Boot] [PATCH v2 5/6] ARM: Add SPL target to arm1136 Stefano Babic
2012-10-14  9:17   ` Albert ARIBAUD
2012-10-15  9:43     ` Stefano Babic
2012-10-17 16:04   ` [U-Boot] [PATCH v3 " Stefano Babic
2012-10-11  7:11 ` [U-Boot] [PATCH v2 6/6] MX35: add support for woodburn board Stefano Babic
2012-12-08  9:31   ` Albert ARIBAUD
2012-12-08 11:24     ` Stefano Babic
2012-12-08 13:58       ` Tom Rini [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=50C34793.5070304@ti.com \
    --to=trini@ti.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.