Re: [U-Boot] commit 83e359adf9f578a58f20daf2e4425a754defac7b breaks mmc

2014-09-10 Thread Peter A. Bigot
On 09/09/2014 12:27 PM, Felipe Balbi wrote: Hi, On Tue, Sep 09, 2014 at 12:09:02PM -0500, Felipe Balbi wrote: Hi, commit 83e359a (am335x_evm: Enable CONFIG_SPL_ENV_SUPPORT on EMMC_BOOT) breaks MMC on some boards. I believe it only breaks if EMMC has no partition on it, so that would point out

Re: [U-Boot] uboot env in mmc partition

2014-09-03 Thread Peter A. Bigot
On 09/03/2014 07:54 AM, Naitik Amin wrote: Hi Hannes, So I did as you had indicated and made some progress, so now, my #defines look as below. #define CONFIG_SYS_MMC_ENV_DEV 0 /* device 0 */ #define CONFIG_ENV_OFFSET 0x2190 I calculated the env offset from the

Re: [U-Boot] uboot env in mmc partition

2014-09-03 Thread Peter A. Bigot
On 09/03/2014 09:14 AM, Hannes Petermaier wrote: On 09/03/2014 07:54 AM, Naitik Amin wrote: Hi Hannes, So I did as you had indicated and made some progress, so now, my #defines look as below. #define CONFIG_SYS_MMC_ENV_DEV 0 /* device 0 */ #define CONFIG_ENV_OFFSET

Re: [U-Boot] [PATCH 2/2] mmc: restore capacity when switching to partition 0

2014-09-03 Thread Peter A. Bigot
On 09/03/2014 10:48 AM, Stephen Warren wrote: On 09/02/2014 05:31 PM, Peter A. Bigot wrote: The capacity and lba for an MMC device with part_num 0 reflects the whole device. When mmc_switch_part() successfully switches to a partition, the capacity is changed to that partition. As partition 0

Re: [U-Boot] [PATCH 1/2] env_mmc: remove condition on call to mmc_switch_part

2014-09-03 Thread Peter A. Bigot
On 09/03/2014 10:46 AM, Stephen Warren wrote: On 09/02/2014 05:31 PM, Peter A. Bigot wrote: Though it might be expected to do so, mmc_switch_part() does not change the part_num field of the device on which the partition has been changed. As such, checking to see whether the partition

[U-Boot] [PATCH v2] env_mmc: correct fini partition to match init partition

2014-09-03 Thread Peter A. Bigot
The code to set the MMC partition uses an weak function to obtain the correct partition number. Use that instead of the compile-time default when deciding whether it needs to switch back. Signed-off-by: Peter A. Bigot p...@pabigot.com --- V2: * Preserve desired behavior of avoiding diagnostic

Re: [U-Boot] [PATCH 2/2] mmc: restore capacity when switching to partition 0

2014-09-03 Thread Peter A. Bigot
On 09/03/2014 11:05 AM, Stephen Warren wrote: On 09/03/2014 09:59 AM, Peter A. Bigot wrote: On 09/03/2014 10:48 AM, Stephen Warren wrote: On 09/02/2014 05:31 PM, Peter A. Bigot wrote: The capacity and lba for an MMC device with part_num 0 reflects the whole device. When mmc_switch_part

[U-Boot] [PATCH v3] env_mmc: correct fini partition to match init partition

2014-09-03 Thread Peter A. Bigot
The code to set the MMC partition uses an weak function to obtain the correct partition number. Use that instead of the compile-time default when deciding whether it needs to switch back. Fixes: 6e7b7df4df43574 (env_mmc: support env partition setup in runtime) Signed-off-by: Peter A. Bigot p

Re: [U-Boot] [PATCH v2] env_mmc: correct fini partition to match init partition

2014-09-03 Thread Peter A. Bigot
On 09/03/2014 11:52 AM, Stephen Warren wrote: On 09/03/2014 10:32 AM, Peter A. Bigot wrote: The code to set the MMC partition uses an weak function to obtain the correct partition number. Use that instead of the compile-time default when deciding whether it needs to switch back. Yes

Re: [U-Boot] [PATCH v2] env_mmc: correct fini partition to match init partition

2014-09-03 Thread Peter A. Bigot
On 09/03/2014 12:46 PM, Stephen Warren wrote: On 09/03/2014 11:30 AM, Peter A. Bigot wrote: On 09/03/2014 11:52 AM, Stephen Warren wrote: On 09/03/2014 10:32 AM, Peter A. Bigot wrote: The code to set the MMC partition uses an weak function to obtain the correct partition number. Use

[U-Boot] [PATCH 0/2] fix issue with mmc partition management

2014-09-02 Thread Peter A. Bigot
patch is relevant to 2014.07; the first is not. Peter A. Bigot (2): env_mmc: remove condition on call to mmc_switch_part mmc: restore capacity when switching to partition 0 common/env_mmc.c | 11 --- drivers/mmc/mmc.c | 11 --- 2 files changed, 12 insertions(+), 10 deletions

[U-Boot] [PATCH 2/2] mmc: restore capacity when switching to partition 0

2014-09-02 Thread Peter A. Bigot
, but the capacity setting for the whole device must still be restored to match the partition. Signed-off-by: Peter A. Bigot p...@pabigot.com --- drivers/mmc/mmc.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c index a26f3ce

[U-Boot] [PATCH 1/2] env_mmc: remove condition on call to mmc_switch_part

2014-09-02 Thread Peter A. Bigot
env_mmc which rely on this behavior to avoid having to preserve the pre-switch partition number outside the device structure. Signed-off-by: Peter A. Bigot p...@pabigot.com --- common/env_mmc.c | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/common/env_mmc.c b

Re: [U-Boot] ARM: omap3: Implement dpll5 (HSUSB clk) workaround for OMAP36xx/AM/DM37xx according to errata sprz318e.

2013-08-16 Thread Peter A. Bigot
On 08/16/2013 08:38 AM, Tom Rini wrote: On Wed, Aug 14, 2013 at 09:53:16PM -0500, Peter A. Bigot wrote: On 07/09/2013 02:43 AM, Naumann Andreas wrote: In chapter 'Advisory 2.1 USB Host Clock Drift Causes USB Spec Non-compliance in Certain Configurations' of the TI Errata it is recommended

Re: [U-Boot] ARM: omap3: Implement dpll5 (HSUSB clk) workaround for OMAP36xx/AM/DM37xx according to errata sprz318e.

2013-08-14 Thread Peter A. Bigot
On 07/09/2013 02:43 AM, Naumann Andreas wrote: In chapter 'Advisory 2.1 USB Host Clock Drift Causes USB Spec Non-compliance in Certain Configurations' of the TI Errata it is recommended to use certain div/mult values for the DPLL5 clock setup. So far u-boot used the old 34xx values, so I added