Xie Shaohui-B21989 wrote:
> [S.H] I thought one patch doing one thing even it's simple.
> I'm fine with the merge if you insist.
Well, it is a matter of opinion. I don't insist, but I think it would be
easier for everyone if you did merge these patches. Each one is very
short, and they are all
On Thu, Jan 17, 2013 at 9:26 AM, Roy Zang wrote:
> Signed-off-by: Roy Zang
> ---
Why? We don't need this for P5040 support.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/l
On Mon, Jan 14, 2013 at 10:26 PM, Prabhakar Kushwaha
wrote:
> #define SVR_9130 0x860001
> #define SVR_9131 0x86
> +#define SVR_9132 0x861000
> +#define SVR_9132_E 0x861800
> +#define SVR_9232 0x861400
Remove SVR_9132_E. See commit 48f6a5c348453fc3ab33aaa91e5e
On Thu, Jan 10, 2013 at 5:49 AM, Vakul Garg wrote:
> If property 'fsl,sec-era' is already present, it is updated.
> This property is required so that applications can ascertain which
> descriptor commands are supported on a particular CAAM version.
>
> Signed-off-by: Vakul Garg
> Cc: Andy Fleming
On Mon, Jan 7, 2013 at 4:39 PM, Wolfgang Denk wrote:
>
> This seems broken to me. Can we rather try8 and get rid of all this
> "bool" stuff instead? It's just obfuscating the code...
Like Scott said, we sometimes copy code from Linux that uses 'bool',
so it's simpler if we just retain this comm
On Wed, Nov 28, 2012 at 1:39 AM, Andy Fleming wrote:
>
> Timur Tabi (5):
> powerpc/85xx/p5040: add CONFIG_SYS_PPC64, del
> CONFIG_SYS_FSL_ELBC_MULTIBIT_ECC
> powerpc/85xx: add support for the Freescale P5040DS Superhydra
> reference board
> powerpc/85xx: implement check for err
On Thu, Nov 1, 2012 at 11:20 AM, Tabi Timur-B04825 wrote:
>
>> To solve this problem, move CONFIG_FSL_SATA_V2 to board
>> configuration header file.
>
> http://patchwork.ozlabs.org/patch/126958/
To be clear, I think you should say in the patch description that your
patch eff
On Thu, Nov 1, 2012 at 3:13 AM, Roy Zang wrote:
> board configuration file is included before asm/config_mpc85xx.h.
> however, CONFIG_FSL_SATA_V2 is defined in asm/config_mpc85xx.h.
> it will never take effective in the board configuration file for
> this kind of code :
>
> #ifdef CONFIG_FSL_SATA
On Mon, Oct 29, 2012 at 9:05 PM, Scott Wood wrote:
> diff --git a/include/configs/p1_p2_rdb_pc.h b/include/configs/p1_p2_rdb_pc.h
> index ba7a364..f02ba2b 100644
> --- a/include/configs/p1_p2_rdb_pc.h
> +++ b/include/configs/p1_p2_rdb_pc.h
> @@ -38,6 +38,7 @@
> #define __SW_BOOT_MASK 0x0
On Mon, Oct 22, 2012 at 5:36 PM, Andy Fleming wrote:
> Timur Tabi (4):
> powerpc/mpc85xx: fix Unicode characters in release.S
> powerpc/85xx: define SRIO LIODN functions only if SRIO is defined
> powerpc/85xx: move SRIO configuration out of corenet_ds.h
> powerpc/85xx: Add
On Fri, Oct 12, 2012 at 1:25 AM, Shaohui Xie wrote:
> From: shaohui xie
>
> TBI PHY address (TBIPA) register is set in general frame manager
> phy init funciton dtsec_init_phy() in drivers/net/fm/eth.c, and
> it is supposed to set TBIPA on FM1@DTSEC1 in case of FM1@DTSEC1
> isn't used directly, w
Robert Thorhuus wrote:
> I really see the advantage of keeping this map you have. It is of
> course more compatible with different OSEs and you do not need to do
> anything special with 32/64 bit cores. But at the same time you never
> take advantage of a 64 bit core with this approach.
U-Boot is
judge.pack...@gmail.com wrote:
> .../SBx81CFC960/fsl_fman_ucode_P2041_101_8.bin | Bin 0 -> 7240 bytes
> .../SBx81CFC960/fsl_fman_ucode_P2041_101_8.c | 607
>
Including the firmware itself in the patch is a bad idea. I do not want
any Freescale firmware to be
On Thu, Sep 20, 2012 at 3:24 PM, Robert Thorhuus
wrote:
> I'm using 64 bit mpc85xx cores.
U-Boot runs in 32-bit mode even on a 64-bit code.
> Mapping the whole memory from the beginning was of course just an initial
> thought.
> But for me it looks like the code is just not able to deal with m
On Sat, Sep 15, 2012 at 9:36 AM, Jerry Van Baren wrote:
> Queued up 2 of 2 patches to the u-boot-fdt "next" branch. I'll issue a
> pull request when the merge window opens.
I thought libfdt patches were suppose to go to dtc.git?
--
Timur Tabi
Linux kernel developer at Freescale
__
Kim Phillips wrote:
>> Cryptography acceleration (SEC 5.0) at up to 40 Gbps SerDes
>
> it's a SEC 5.2, but...
Ok.
>> Dual four channel
>> Support for hardware virtualization and partitioning enforcement
>> Extra privileged level for hypervisor support
>> QorIQ Trust Architecture 1
On Fri, Aug 17, 2012 at 1:20 PM, York Sun wrote:
>
> + * 8-19 CHIP_ID,2’b00 - SoC 1
> + * all others - reserved
> + * 20-24 CLUSTER_ID 5’b0 - CCM 1
> + * all others - reserved
> + * 25-26 CORE_CLUSTER_ID 2’b00 - cluster 1
> + *
On Thu, Aug 23, 2012 at 12:24 PM, Andy Fleming wrote:
> I'm guessing this change increased the amount of inlining. Sadly, the
> subsequent patches, which were intended to shrink the SPL build, were
> not enough to fix this problem.
Maybe there are places where we can use raw I/O instead of acces
Or Yoram-R56270 wrote:
> 1. Correct the v1 documentation to specify 23 MAC addresses and placing the
> CRC at 0xCC.
> 2. Create a new code and documentation for v2, specifying 31 MAC addresses
> and placing the CRC at 0xFC.
Both of these are unnecessary. My patch fixes the code to match the spe
sun york-R58495 wrote:
> I agree it was a broken design. Now we are using all available space
> and put CRC to the very end. It is not perfect but should work.
*sigh*
The design was never broken, the code was just wrong. The CRC has always
supposed to have been at the end.
--
Timur Tabi
Linu
Wolfgang Denk wrote:
>> >My patch provides transparent updates to handle it. It will read broken
>> >EEPROMs and verify the CRC in the old location, and if you have re-save
>> >the EEPROM, it will put the CRC in the right place.
> It will work by chance, accessing random data. This is crap.
It
Wolfgang Denk wrote:
> This is a totally broken design then, when you have a growing data
> structure where vital information fields get shifted. In such case,
> the CRC should have been at the beginning, so it never changes
> location. Or even better, you should not have used a binary data
> stru
Nobuhiro Iwamatsu wrote:
>> diff --git a/drivers/net/fec_mxc.c b/drivers/net/fec_mxc.c
>> index eee41d7..7a3d16b 100644
>> --- a/drivers/net/fec_mxc.c
>> +++ b/drivers/net/fec_mxc.c
>> @@ -510,7 +510,12 @@ static int fec_open(struct eth_device *edev)
>> fec_eth_phy_config(edev);
>
On Fri, Jun 22, 2012 at 2:41 PM, Troy Kisky
wrote:
>
> Correct and correct. If you need i2c working before relocation, you cannot
> enable MULTI_BUS.
That sounds like a really strong limitation of MULTI_BUS. i2c is
necessary for SPD, which is needed to initialize DDR.
--
Timur Tabi
Linux kerne
On Jun 12, 2012, at 7:54 PM, "Wood Scott-B07421" wrote:
>
> You mean to replace this tool? There are other PBL things that may need
> to be done, such as reconfiguring the SPI speed so it doesn't take
> forever to read out the rest of the PBL image.
My tool builds and RCW image. I could chang
On Jun 8, 2012, at 6:09 PM, "Wood Scott-B07421" wrote:
>>
>> I have two questions:
>> 1. Is there a plan that we submit Timur' tool to open source?
>
> I hope so. If not, the RCW will have to be provided directly as a
> hexdump/binary/etc.
Where do you think my compiler should go?
>
>> 2.
Scott Wood wrote:
>> > Well, as with most of our boards, NOR is the default configuration. Also,
>> > there's no NAND support upstream yet.
> What isn't upstream, besides the muxing hack? Does it need the 4K page
> hack?
There's no NAND support at all. However, I just tried the two SDK patche
On Thu, Apr 26, 2012 at 8:57 AM, Dirk Eibach wrote:
> Signed-off-by: Dirk Eibach
> Cc: Timur Tabi
> ---
> Changes in v2:
> - add Cc
> - split up original patch series
>
> board/freescale/p1022ds/p1022ds.c | 4 ++
> board/freescale/p1022ds/tlb.c | 10 ++
> boards.cfg
On Sat, Mar 17, 2012 at 6:19 PM, Wolfgang Denk wrote:
> Doubling it is kind of aggressive strategy. You know exactly how much
> free room needs to be guaranteed, so why don't you auto-adjust the
> size?
>
>> -#define CONFIG_SYS_MALLOC_LEN (128 * 1024) /* Reserved for malloc
>> */
>> +
On Fri, Mar 9, 2012 at 1:54 AM, wrote:
> -#define CONFIG_SATA_SIL3114
> +#define CONFIG_SATA_SIL
This is not "enabling" SATA. This is choosing the correct driver.
You can only use the word "enable" if the feature was intentionally
disabled before, e.g. because the driver did not exist yet.
You
On Mon, Feb 6, 2012 at 11:17 PM, Ramneek Mehresh
wrote:
>
> + start_offset, compat_mph);
> if (node_offset < 0) {
> - printf("WARNING: could not find compatible node %s: %s.\n",
> - compat, fdt_strerror(node_offset));
> -
On Tue, Jan 17, 2012 at 1:12 AM, Simon Glass wrote:
> From: Heiko Schocher
>
> This Patch introduce the new i2c_core file, which holds
> the I2C core functions, for the rework of the multibus/
> multiadapter support.
> Also adds changes in i2c.h for the new I2C multibus/multiadapter
> support. Th
On Thu, Jan 12, 2012 at 5:49 PM, Tabi Timur-B04825 wrote:
> Are you going to make another pull request before the window closes?
The reason I ask is that you're missing some of my patches.
http://patchwork.ozlabs.org/project/uboot/list/?submitter=timur
--
Timur Tabi
Linux kernel deve
Liu Gang-B34182 wrote:
> Where is CONFIG_SYS_QE_FMAN_FW_IN_REMOTE documented? And where is the code
> that uses it?
> [Liu Gang-B34182] Sorry I documented this definition in "[PATCH 6/8]
> powerpc/boot: Slave uploads ucode when boot from SRIO",
You need to update the README. That's where all t
On Tue, Jan 10, 2012 at 5:42 AM, Liu Gang wrote:
> When boot from SRIO, slave's ucode can be stored in master's memory space,
> then slave can fetch the ucode image through SRIO interface.
What kind of ucode is this? Fman or QE?
> +
> + /* configure inbound window for slave's ucode */
> +
On Tue, Jan 10, 2012 at 5:42 AM, Liu Gang wrote:
>
> +#define CONFIG_SYS_QE_FMAN_FW_IN_REMOTE
> +#define CONFIG_SYS_QE_FMAN_FW_ADDR 0
Where is CONFIG_SYS_QE_FMAN_FW_IN_REMOTE documented? And where is the
code that uses it?
--
Timur Tabi
Linux kernel developer at Freescale
_
On Tue, Jan 10, 2012 at 5:42 AM, Liu Gang wrote:
>
> There will be a confict if defined CONFIG_ENV_IS_NOWHERE and
> CONFIG_ENV_IS_IN_FLASH.
This doesn't make any sense. How can the environment be nowhere *and*
also in flash, at the same time?
--
Timur Tabi
Linux kernel developer at Freescale
_
On Wed, Dec 7, 2011 at 8:42 AM, Kumar Gala wrote:
> + if (flag & BOOTM_STATE_OS_PREP) {
> + boot_prep_linux(images);
> return 0;
> + }
>
> if (flag & BOOTM_STATE_OS_GO) {
> boot_jump_linux(images);
> return 0;
>
On Tue, Nov 22, 2011 at 2:51 AM, Shengzhou Liu
wrote:
>
> +#ifdef CONFIG_PHYS_64BIT
> + puts("36-bit Addressing\n");
> +#endif
We're not doing this any more. The 'bdinfo' command will be updated
to display this information instead.
Since Kumar already applied this patch, please post a fol
Kumar Gala wrote:
>> > #ifdef CONFIG_QE
>> > /* QE microcode/firmware address */
>> > -#define CONFIG_SYS_QE_FW_ADDR0xefec
>> > -#define CONFIG_SYS_QE_FW_LENGTH 0x1
>> > +#define CONFIG_SYS_QE_FMAN_FW_IN_NOR
>> > +#define CONFIG_SYS_QE_FMAN_FW_ADDR 0xefec000
On Wed, Oct 5, 2011 at 6:14 PM, Joe Hershberger wrote:
> Signed-off-by: Joe Hershberger
> Cc: Joe Hershberger
> Cc: Kim Phillips
> ---
Where's the patch description? You need to explain how this patch works.
--
Timur Tabi
Linux kernel developer at Freescale
_
On Tue, Nov 1, 2011 at 9:08 PM, Huang Changming-R66093
wrote:
> Kim,
> Since 2008, the boards shipped to customer are the new boards with 33.33MHz
> oscillator and 512MB DDR.
> How can I deal with the new board and old board?
Check the board revision at runtime.
--
Timur Tabi
Linux kernel deve
On Tue, Oct 18, 2011 at 4:05 PM, Kyle Moffett wrote:
> A followup patch will be adding a configurable feature to enable
> programming of E1000 EEPROMs from the command line or via the generic
> U-Boot SPI interface.
>
> In order for it to work it needs access to certain E1000-internal
> functions,
On Sun, Oct 30, 2011 at 7:54 PM, Mike Frysinger wrote:
>
>
> -static void print_num(const char *, ulong);
> +__maybe_unused
> +static void print_num(const char *name, ulong value)
> +{
> + printf("%-12s= 0x%08lX\n", name, value);
> +}
>
Will the linker remove the functions from the binary i
On Mon, Oct 31, 2011 at 9:46 AM, Andy Fleming wrote:
> We want to move everything to phylib, and we definitely don't want
> new drivers using the miiphy infrastructure.
How about using gcc's deprecated function feature? Or is that too aggressive?
http://gcc.gnu.org/onlinedocs/gcc/Function-Attri
On Mon, Oct 31, 2011 at 1:19 AM, wrote:
> From: Jerry Huang
>
> When the resolution is set to 800x600 and 1024x768,
> but, the driver will use 1280x1024 resolution to set the DIU register
>
> Signed-off-by: Jerry Huang
Acked-by: Timur Tabi
Thanks, Jerry. I'm amazed that I didn't notice this
On Fri, Oct 14, 2011 at 4:32 PM, Wolfgang Denk wrote:
> These changes silently change the behaviour of the command. So far,
> an argument line "24" would be parsed as hex number (i. e. result in
> decimal value 36); with this change it suddenly becomes devcimal 24,
> which is quite unexpected.
On Fri, Oct 14, 2011 at 2:34 AM, Kumar Gala wrote:
> The following changes since commit d8fffa057c9430fd0c5104ab6ff7db4cdb03db51:
>
> Merge branch 'master' of git://git.denx.de/u-boot-mips (2011-10-12 22:47:15
> +0200)
>
> are available in the git repository at:
>
> git://git.denx.de/u-boot-mpc
Kumar Gala wrote:
> For the FSL boards, lets add 'bdinfo' to 'preboot' and enable CONFIG_PREBOOT
I don't think that's a good idea. bdinfo display about a dozen lines of
text, most of it is not generally useful. Plus, this would make preboot
unavailable for everyone else.
--
Timur Tabi
Linux
Kumar Gala wrote:
> Rather than looking at tn2020_driver.uid lets just do something like:
>
> if ((phydev->drv->uid& phydev->drv->mask) == TN2020_UID)
>
> And add a #define in phy.h:
>
> #define TN2020_UID0x00a19410
I guess that makes more sense. I thought my original version was cooler,
th
On Thu, Oct 6, 2011 at 4:23 PM, Wolfgang Denk wrote:
> Checkpatch says:
>
> WARNING: static const char * array should probably be static const
> char * const
> #1511: FILE: board/freescale/p3060qds/p3060qds.c:84:
> + static const char *freq[] = {"100", "125", "Reserved",
>
>
> Pleas
On Mon, Oct 10, 2011 at 4:35 AM, Stefano Babic wrote:
> New default, weak i2c_set_bus_num() function.
>
> Signed-off-by: Stefano Babic
> Cc: Heiko Schocher
I would prefer to see a new set of I2C functions that take a bus
number as a parameter, so that we can eliminate i2c_set_bus_num() (and
the
Kumar Gala wrote:
> * The only things not explained is the env address change. I can see
> it matches our other boards MPC8572DS, etc. So not exactly critical to
> me.
There's no explanation for increasing the malloc size.
> * If Wolfgang wants to remove CONFIG_PCI_SCAN_SHOW so be it, but as
>
On Sun, Oct 9, 2011 at 5:55 PM, Kumar Gala wrote:
>
> - Increase the size of malloc space.
> - Enable e1000 network card.
> - Show pci devices on startup.
> - Change the location of env address.
> - Use hwconfig to turn off ECC by default.
This patch is missing all of the explanations that were p
On Sat, Oct 8, 2011 at 2:33 AM, Zhao Chenhui wrote:
>
> - Fix config according to system address map in the manual.
> - Rework tlb and law tables.
> - Remove unnecessary macros.
You're making a lot of changes here. The device tree needs to match
the U-Boot configuration. Are you changing U-Boot
Kumar Gala wrote:
>> > Note: I'm not sure of my change to struct fm_mdio. It works, but it
>> > doesn't
>> > smell right.
> What exactly is the concern?
Nothing specific, just a gut feeling.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-
On Mon, Oct 3, 2011 at 9:44 AM, Kumar Gala wrote:
> Timur Tabi (4):
> powerpc/85xx: introduce and document CONFIG_SYS_CCSRBAR macros
> powerpc/85xx: relocate CCSR before creating the initial RAM area
> powerpc/mp: add support for discontiguous cores
> powerpc/hydra: Add ethern
On Mon, Oct 3, 2011 at 6:59 PM, Becky Bruce wrote:
> -#ifdef DEBUG
> const char *step_string_tbl[] = {
> "STEP_GET_SPD",
> "STEP_COMPUTE_DIMM_PARMS",
> @@ -153,7 +152,6 @@ const char * step_to_string(unsigned int step) {
>
> return step_string_tbl[s];
> }
> -#endif
How big
On Tue, Sep 13, 2011 at 2:15 AM, Zhao Chenhui
wrote:
> @@ -222,8 +222,8 @@ int checkboard (void)
>
> printf("PCI1: %d bit, %s MHz, %s\n",
> (pci1_32) ? 32 : 64,
> - (pci1_speed == 3300) ? "33" :
> - (pci1_speed == 6600) ? "66" : "unknown",
On Tue, Sep 13, 2011 at 2:15 AM, Zhao Chenhui
wrote:
> From: chenhui zhao
>
> -Increase the size of malloc space.
> -Enable e1000 network card.
> -Change the location of env address.
> -Use hwconfig to turn off ECC.
Please repost this patch with a detailed explanation as to why these
changes are
Kumar Gala wrote:
> 1. We reduce the amount printed in the "default" case
> 2. First choice should always be to have a command the print status info
> 3. Allow a board port to makes its own decision if it wants to do something
> like enable CONFIG_PCI_SCAN_SHOW
>
> Any concerns w/that proposal?
A
On Thu, Sep 1, 2011 at 6:40 AM, Dave Aldridge wrote:
> #if !defined(CONFIG_SYS_NS16550_REG_SIZE) || (CONFIG_SYS_NS16550_REG_SIZE ==
> 0)
> #error "Please define NS16550 registers size."
> +#elif defined(CONFIG_SYS_NS16550_MEM32)
> +#define UART_REG(x) unsigned int x;
Shouldn't this be
#defin
Kumar Gala wrote:
>
> On Aug 31, 2011, at 12:14 PM, Tabi Timur-B04825 wrote:
>
>> On Wed, Aug 31, 2011 at 9:16 AM, Kumar Gala
>> wrote:
>>> Signed-off-by: Kumar Gala
>>> ---
>>
>> Why are we enabling this?
>
> because its a useful command
On Wed, Aug 31, 2011 at 9:16 AM, Kumar Gala wrote:
> Signed-off-by: Kumar Gala
> ---
Why are we enabling this?
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
On Mon, Aug 29, 2011 at 2:15 AM, Chunhe Lan wrote:
> In the p1023rds, accessing exclusively nor flash or nand flash device by
> BR0/OR0.
This is not an English sentence. I do not understand what you're saying.
--
Timur Tabi
Linux kernel developer at Freescale
__
On Thu, Aug 25, 2011 at 9:51 AM, Mike Frysinger wrote:
> i feel like some (many?) #ifdef's in the tree could be done without ifdefs (by
> relying on gcc's DCE) thus improving overall code quality
What's DCE?
--
Timur Tabi
Linux kernel developer at Freescale
On Mon, Aug 22, 2011 at 4:45 PM, Timur Tabi wrote:
> The Freescale QE and FMAN devices support uploading of firmware, and U-Boot
> can locate that firmware on different types of storage media. However,
> the current macros used for this code are inconsistent and poorly documented.
>
> Note that s
On Thu, Aug 11, 2011 at 11:18 PM, Chunhe Lan wrote:
> +/*
> + * Sets the specified node's status to the value contained in "status".
> + * If the first character of the specified path is "/" then we use
> + * alias as a path. Otherwise, we look for an alias of that name.
> + */
> +void fdt_set_no
I see two calls to simple_stroul, and both have 16 in them.
--
Timur Tabi
Linux kernel developer at Freescale
From: sun york-R58495
Sent: Thursday, August 11, 2011 10:03 AM
To: Tabi Timur-B04825
Cc: u-boot@lists.denx.de
Subject: Re: [u-boot-release
York Sun wrote:
> - e.mac_count = simple_strtoul(argv[2], NULL, 16);
> + e.mac_count = simple_strtoul(argv[2], NULL, 10);
You forgot the MAC address index. There's another use of strtoul in the
code that needs to be fixed.
Also, if you use "0" instead of "10", then it sh
York Sun wrote:
> +/* Option parameter Structures */
> +typedef struct {
> + const char *option_name;
> + size_t offset;
> + unsigned int size;
> + const char printHex;
> +} options_strings_t;
Does it make sense for only printHex (which should be "print_hex") and
option_name to be
York Sun wrote:
> Change the help message to be more helpful. Print argument format.
> Fix MAX_NUM_PORTS to comply with v1 NXID format.
>
> Signed-off-by: York Sun
Could you also fix the commands so that they take a number in decimal
instead of hex?
e.mac[index][i] = simple_strtoul(p,
Kumar Gala wrote:
> ok. Sounds fine to leave as it, just wondering.
Also, I want people to specifically set the physical address in the board
header file if they want to relocate CCSR.
Anyway, I'll post a new version that reduces the merge conflicts.
--
Timur Tabi
Linux kernel developer at Fre
Kumar Gala wrote:
> Why didn't we do in include/mpc85xx.h:
>
> #ifndef CONFIG_SYS_CCSRBAR_PHYS_LOW
> #define CONFIG_SYS_CCSRBAR_PHYS_LOW CONFIG_SYS_CCSRBAR
> #endif
>
> Thus removing that line from every board file?
That might make more sense, but that will break other boards where virtual
!= p
Kumar Gala wrote:
> What changed in v2?
I fixed the problem on several boards where I wasn't setting the relocated
physical address properly. For instance, on the P2020DS, the original
code moved CCSR from ff7 to ffe, but my v1 patch left it at ff7 because I
didn't notice that the device tree
On Mon, Aug 1, 2011 at 10:26 AM, Kumar Gala wrote:
> Just wondering which checkpatch you are running.
>
> I get:
>
> total: 1 errors, 16 warnings, 3172 lines checked
>
> Mine's from linux kernel tree HEAD as of a day or so ago.
I believe that checkpatch has become a more forgiving with the
80-ch
On Fri, Jul 22, 2011 at 2:55 AM, Kumar Gala wrote:
> The following changes since commit 74fac70084bb040342fafc3b0d2ef50fbe35646f:
>
> Merge branch 'master' of git://git.denx.de/u-boot-mmc (2011-07-19 22:27:07
> +0200)
>
> are available in the git repository at:
>
> git://git.denx.de/u-boot-mpc8
On Tue, Jul 26, 2011 at 9:52 PM, Kumar Gala wrote:
> +/* CCSRBAR PHYSICAL Address */
> +/* IN case of NAND bootloader relocate CCSRBAR in RAMboot code not in the 4k
> + SPL code*/
> +#if defined(CONFIG_NAND_U_BOOT) && defined(CONFIG_NAND_SPL)
> +#define CONFIG_SYS_CCSRBAR_PHYS CONFIG
The following URL doesn't show anything:
http://git.denx.de/?p=u-boot.git&a=search&h=HEAD&st=author&s=Timur+Tabi
No matter what I put after the "s=", I get nothing. I think it should
work, because this URL, which has the same search string, does work:
http://git.kernel.org/?p=linux/kernel/git/t
Kumar Gala wrote:
> Ok, Can you send a commit message for when I merge them or repost a merged
> patch.
powerpc/85xx: remove SERDES4 soft-reset work-around
Some P4080 rev1 errata work-arounds, notably erratum SERDES4, required a
bank soft-reset after the bank was configured and enabled, even tho
Kumar Gala wrote:
> Erratum SERDES-8 says that the clocks for bank 3 needs to be enabled if
> bank 2 is enabled, but this was not being done for SERDES protocols 0xF
> and 0x10. The bank reset that was being done for erratum SERDES4 (a
> left-over work-around that was removed in "powerpc/85xx: rem
Jerry,
Now that the merge window is open, could you please update libfdt?
The version in U-Boot hasn't been updated in over a year, and there
are new features that I need to have. Specifically, this commit:
http://git.jdl.com/gitweb/?p=dtc.git;a=commit;h=d75b33af676d0beac8398651a7f09037555a550b
York Sun wrote:
> From: york
You should fix this so that it includes your full name.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
On Mon, May 2, 2011 at 8:51 PM, York Sun wrote:
> - printf("The DIMM max tCKmin is %d ps,"
> + printf("The combined minimum tCKmin is %d ps,"
> "doesn't support the MCLK cycle %d ps\n",
That still needs some work, IMHO. I think you might need t
On Mon, May 2, 2011 at 10:00 AM, Kumar Gala wrote:
> I thought that was less of an issue because of our use of patchworks.
How? Patchwork uses the same subject line as the email. There's
still the problem of trying to correlate the actual commit to the
entry in patchwork.
On the other hand, I
Mike Frysinger wrote:
> so implement this in your board file in misc_init_r or board_eth_init. have
> the code do something like:
> uchar enetaddr[6];
> if (!eth_getenv_enetaddr("ethaddr", enetaddr)) {
> /* ... read current MAC out of the driver's registers ... */
>
Wolfgang Denk wrote:
> This gets lost in the code.
That's why I added the comments:
/* MPC, where == 4-digit number */
> If you keep the if / else, you have to add braces for the multiline
> statements.
Ok.
--
Timur Tabi
Linux kernel developer at Freescale
___
Wolfgang Denk wrote:
> Why do we need this "if" at all? tolower() on a digit is a nop, so you
> can omit the first branch.
Because cpu->name looks like one of two ways:
8578
or
P4080
In the case of "8578", we want to convert that to "mpc8578". In the case of
"P4080",
On Fri, Apr 29, 2011 at 10:34 AM, Kumar Gala wrote:
> Timur Tabi (7):
> powerpc/86xx: remove empty board_early_init_f()
> powerpc/fsl: add 'pixis_reset dump' command
> powerpc/85xx: Implement work-around for P4080 erratum SERDES-A005
> powerpc/85xx: Display SERDES 8 erratum war
On Thu, Apr 28, 2011 at 9:55 PM, Kumar Gala wrote:
> CONFIG_SYS_BOOTMAPSZ has been 64M on these boards for some time so we
> should also allow the kernel image to be up to 64M decompressed. This
> also matches what we pass to the OS based on the ePAPR specification.
Is there any e500-based board
Anatolij Gustschin wrote:
> it turns out that your patches broke DIU support on mpc5121. I need
> to fix it first before sending a pull request. I'm planning to work
> on fixes and hope to resolve the issue this week.
Anything I can do to help, please just let me know. We don't have any 5121s
he
On Wed, Apr 27, 2011 at 8:39 AM, Simon Schwarz
wrote:
> So here is my question: Where do you see the most potential to optimize
> u-boot?
I'm sure many of the timeouts could be optimized.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot m
Anatolij,
Are you planning on making a pull request for u-boot-video soon? The
merge window closed last week, but I have some pending video patches
that I would like to see included.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mail
On Sun, Apr 24, 2011 at 6:50 PM, Wolfgang Denk wrote:
> And please add documentation for the new CONFIG_PRESERVE_LOCAL_MAC to
> the README.
We have something similar already on Freescale parts, but it does
sometimes cause problems. If the environment ethaddr is already set,
it is left alone. O
On Fri, Apr 22, 2011 at 8:26 AM, Kumar Gala wrote:
>> Shouldn't there also be a README update to document this option?
>
> I think its self evident what it is.
I have to disagree. Without reading the patch description, it's hard
to know what value to choose for this macro. Besides, Wolfgang is
Kumar Gala wrote:
> The
> work-around is to enable and then disable bank two after bank three is
> disabled.
That should say, "after bank three is enabled".
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
On Thu, Apr 21, 2011 at 1:59 PM, Kumar Gala wrote:
> Introduce new CONFIG_SYS_FSL_TBCLK_DIV on 85xx platforms because
> different SoCs have different divisor amounts. All the PQ3 parts are
> /8, the P4080/P4080 is /16, and P2040/P3041/P5020 are /32.
Shouldn't there also be a README update to doc
"David Müller (ELSOFT AG)" wrote:
>> > Just out of curiosity -- do you have an example of a device tree with
>> > an external UART?
> Yes.
May I see it?
--
Timur Tabi
Linux kernel developer
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.d
Kumar Gala wrote:
> what about this patch?
I missed this one. Unfortunately, it conflicts with my patch. I'll have to
rewrite this one to make it work.
Plus, there were problems with it. For instance, I'm not crazy about the
setenv() calls (I NAK'd an earlier internal version of this patch t
Kumar Gala wrote:
> I'll deal with merge issues, can you ack or provide feedback on this patch
My patch, "[v3] powerpc: user 'video-mode' environment variable to configure
DIU" ("user" should be "use") should be applied instead.
--
Timur Tabi
Linux kernel developer
_
1 - 100 of 113 matches
Mail list logo