[U-Boot] [PATCH] [v2] p1022ds: allow for board-specific ngPIXIS functions

2011-01-21 Thread Timur Tabi
the DIU is active. Signed-off-by: Timur Tabi ti...@freescale.com --- board/freescale/common/ngpixis.c | 56 board/freescale/common/ngpixis.h |8 - board/freescale/p1022ds/diu.c| 66 -- 3 files changed, 112

[U-Boot] powerpc-linux-gnu-ld: Warning: lib_powerpc/fpu/libpostpowerpcfpu.o uses hard float, libpost.o uses soft float

2011-01-20 Thread Timur Tabi
/libpostpowerpcfpu.o uses hard float, libpost.o uses soft float -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] powerpc-linux-gnu-ld: Warning: lib_powerpc/fpu/libpostpowerpcfpu.o uses hard float, libpost.o uses soft float

2011-01-20 Thread Timur Tabi
=master Assuming you're trying to build a current tree in a workspace that hasn't been used in a while, try make clean, or possbly git clean -f -x -d to be really sure. This is from a freshly cloned tree. I'm running git bisect now. 2010.12-rc1 works -- Timur Tabi Linux kernel developer

Re: [U-Boot] [PATCH][v1] Fix wrong CONFIG_SYS_MPC85xx_SERDES1_ADDR

2011-01-17 Thread Timur Tabi
...@freescale.com Please resubmit this patch using your full name in the signed-off-by line. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] p1022ds: allow for board-specific ngPIXIS functions

2011-01-17 Thread Timur Tabi
the DIU is active. Signed-off-by: Timur Tabi ti...@freescale.com --- board/freescale/common/ngpixis.c | 54 board/freescale/common/ngpixis.h |6 +++ board/freescale/p1022ds/diu.c| 64 - 3 files changed, 109 insertions

Re: [U-Boot] [PATCH] powerpc/85xx: Bump up the CONFIG_SYS_BOOTM_LEN to 16M on FSL 85xx boards

2011-01-11 Thread Timur Tabi
(+), 16 deletions(-) Wouldn't it be simpler to do this in arch/powerpc/include/asm/config.h? -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] powerpc: fix register usage in some inline assembly code

2010-12-15 Thread Timur Tabi
'next'. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] powerpc: fix register usage in some inline assembly code

2010-12-14 Thread Timur Tabi
In some usages of inline assembly, hard-coded registers were specified when a scratch register should have been used instead. Signed-off-by: Timur Tabi ti...@freescale.com --- I can't test the kgdb changes, but the rest appears to be okay. arch/powerpc/lib/kgdb.c | 25

[U-Boot] [PATCH] powerpc: fix implementation of out_8 to match the other out_XX functions

2010-12-03 Thread Timur Tabi
Signed-off-by: Timur Tabi ti...@freescale.com --- arch/powerpc/include/asm/io.h |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/arch/powerpc/include/asm/io.h b/arch/powerpc/include/asm/io.h index 4ddad26..56ac9fe 100644 --- a/arch/powerpc/include/asm/io.h +++ b/arch

[U-Boot] [PATCH] [v2] p1022ds: fix switching of DIU/LBC signals

2010-12-03 Thread Timur Tabi
. Also add a follow-up read after a write to NOR flash if we're going to mux back to DIU after the write, as described in the P1022 RM. Signed-off-by: Timur Tabi ti...@freescale.com --- I have no idea how this ever worked before, but without this fix, video on the P1022DS is completely broken

Re: [U-Boot] [PATCH] p1022ds: fix switching of DIU/LBC signals

2010-12-02 Thread Timur Tabi
Scott Wood wrote: If you were to immediately follow it with out_8 as currently defined, yes. But setbits_8 should be OK. I don't want my flash_writeX functions to make any assumptions about what set_mux_to_diu() does. Can I just replace the __raw_readb() with an in_8()? -- Timur Tabi Linux

Re: [U-Boot] [PATCH 08/13] SBC8560: #define CONFIG_SYS_LBC_NO_SDRAM_INIT

2010-12-02 Thread Timur Tabi
that require an sdram_init() function, and those that do not.  We could have defined an empty sdram_init() but I hate doing that. Would a weak function be a better way to do this? -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing

Re: [U-Boot] [PATCH] p1022ds: fix switching of DIU/LBC signals

2010-12-01 Thread Timur Tabi
, etc, it's because I'm trying to optimize this code. Unlike Dave's code, this stuff needs to be as fast as possible. Although, I wonder if an extra sync will make a difference considering the overhead of switching the muxes. -- Timur Tabi Linux kernel developer at Freescale

Re: [U-Boot] [PATCH] p1022ds: fix switching of DIU/LBC signals

2010-12-01 Thread Timur Tabi
the read? -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] hwconfig: Fix handling of env_hwconfig, board_hwconfig, and cpu_hwconfig

2010-11-30 Thread Timur Tabi
-requisite patch that I'm missing? -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] p1022ds: fix switching of DIU/LBC signals

2010-11-30 Thread Timur Tabi
. Also add a follow-up read after a write to NOR flash if we're going to mux back to DIU after the write, as described in the P1022 RM. Signed-off-by: Timur Tabi ti...@freescale.com --- I have no idea how this ever worked before. It's a complete mystery to me. board/freescale/p1022ds/diu.c | 45

Re: [U-Boot] [PATCH] 512x: Cleanup for partial linking and --gc-sections

2010-11-22 Thread Timur Tabi
visibility, like drivers/video/ ? I'm okay with that. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] 4 PiB left unmapped

2010-11-18 Thread Timur Tabi
On Thu, Nov 18, 2010 at 9:02 AM, DUNDA Matthias matthias.du...@thalesgroup.com wrote: Well, it's actually 2 Gig, just like u-boot recognizes above... There's something wrong with the parameter you're passing to print_size(). -- Timur Tabi Linux kernel developer at Freescale

[U-Boot] [PATCH] [v2] powerpc/85xx: introduce 'fdt verify' command

2010-11-17 Thread Timur Tabi
addresses. It's not safe or practical to have U-Boot update the addresses in the device tree, mostly because U-Boot is not aware of all the devices. We can, however, spot-check a few common devices to see if the addresses match, and then display a warning if they do not. Signed-off-by: Timur Tabi ti

Re: [U-Boot] [PATCH] [v2] powerpc/85xx: introduce 'fdt verify' command

2010-11-17 Thread Timur Tabi
if And the top half is correct. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] [v3] powerpc/85xx: introduce 'fdt verify' command

2010-11-17 Thread Timur Tabi
addresses. It's not safe or practical to have U-Boot update the addresses in the device tree, mostly because U-Boot is not aware of all the devices. We can, however, spot-check a few common devices to see if the addresses match, and then display a warning if they do not. Signed-off-by: Timur Tabi ti

Re: [U-Boot] [PATCH] powerpc/85xx: compare actual device addresses with the device tree

2010-11-16 Thread Timur Tabi
: Dereference aliases by omiting the leading '/', e.g. fdt print ethernet0. ); -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] powerpc/85xx: compare actual device addresses with the device tree

2010-11-16 Thread Timur Tabi
at all? -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] Please pull u-boot-mpc85xx.git (updated)

2010-11-12 Thread Timur Tabi
On Fri, Nov 12, 2010 at 9:49 AM, Kumar Gala ga...@kernel.crashing.org wrote:      powerpc/8xxx: Enable e1000 driver on some FSL boards I have an objection to this patch that I asked you about earlier, and you never answered my question. -- Timur Tabi Linux kernel developer at Freescale

Re: [U-Boot] MPC8377: USB breaks board

2010-11-11 Thread Timur Tabi
is too large. Try disabling some other large feature (like PCI and Ethernet support) and booting that image. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] MPC8377: USB breaks board

2010-11-11 Thread Timur Tabi
On Thu, Nov 11, 2010 at 2:11 PM, Schwarz,Andre andre.schw...@matrix-vision.de wrote: I'm at 300KiB at the moment with plenty of flash. Is there anything specific you're thinking about ? No, it's just wild speculation. -- Timur Tabi Linux kernel developer at Freescale

Re: [U-Boot] [PATCH] powerpc/8xxx: Enable e1000 driver on some FSL boards

2010-11-10 Thread Timur Tabi
Ethernet of the P1022DS, so why are you enabling the E1000 by default? -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] powerpc/85xx: compare actual device addresses with the device tree

2010-11-10 Thread Timur Tabi
only once, so please move the code and get rid of the goto. Ok. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] powerpc/85xx: compare actual device addresses with the device tree

2010-11-10 Thread Timur Tabi
Wolfgang Denk wrote: Dear Timur Tabi, In message 4cdafc37.40...@freescale.com you wrote: It's more than that. Any mismatch in the CCSR base address, serial device offsets, or PCI addresses will be caught. For instance, if you put the PCIE1 memory range at ff80 in the device tree

Re: [U-Boot] [PATCH] powerpc/85xx: compare actual device addresses with the device tree

2010-11-10 Thread Timur Tabi
Wolfgang Denk wrote: Definitely not. I'm afraid you'd never be sober again, and while this might deem a desirable state to you I would not want to see you submitting U-Boot patches then :-) I guess you don't know about the U-Boot patch submission drinking game? :-) -- Timur Tabi Linux

Re: [U-Boot] [PATCH] mpc83xx: Make it boot again

2010-11-04 Thread Timur Tabi
effect of twi? -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] mmc: new legacy MMC/SPI driver

2010-10-27 Thread Timur Tabi
On Wed, Oct 27, 2010 at 8:23 PM, Mike Frysinger vap...@gentoo.org wrote: + * Licensed under the GPL-2 or later. Could you include the normal GPL license text instead? We have in-house tools that scan for the strings in that text to determine the license of the file. -- Timur Tabi Linux

Re: [U-Boot] [PATCH] mmc: new legacy MMC/SPI driver

2010-10-27 Thread Timur Tabi
asking for a simple change that doesn't inconvenience you, and this is how you respond? Are you having a bad day or something? -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman

Re: [U-Boot] [PATCH V2 1/4] Add linux framebuffer header include/linux/fb.h

2010-10-22 Thread Timur Tabi
there is nothing in this header that is suitable only for Freescale's board, but it points to the fact that a general header is required. So would you expect fsl_diu.c to be modified to use the new header files? -- Timur Tabi Linux kernel developer

Re: [U-Boot] 36-bits U-Boot

2010-10-22 Thread Timur Tabi
by reply and then delete it from your system; you should not copy it or disclose its contents to anyone. You really should not include messages like these when you send email to a public mailing list. -- Timur Tabi Linux kernel developer at Freescale

Re: [U-Boot] [PATCH V2 1/4] Add linux framebuffer header include/linux/fb.h

2010-10-21 Thread Timur Tabi
message Why are we adding this header file? -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] MPC8377: SVR wrong ?

2010-10-18 Thread Timur Tabi
of the chip is not unusual. A dip switch setting on the P1022DS board will change the P1022 into a P1013, and the SVR will change accordingly. Perhaps your pin config is not set to the default for the chip you ordered, which is why it looks like a different chip? -- Timur Tabi Linux kernel

Re: [U-Boot] [PATCH] always relocate fdt into an lmb-allocated memory block

2010-10-18 Thread Timur Tabi
On Mon, May 24, 2010 at 3:10 PM, Timur Tabi ti...@freescale.com wrote: The device tree (fdt) must always exist in within the bootmap (usually the first 16MB of RAM).  If it doesn't, then boot_relocate_fdt() will allocate an LMB region in the bootmap and copy the fdt into that region

Re: [U-Boot] [PATCH] always relocate fdt into an lmb-allocated memory block

2010-10-11 Thread Timur Tabi
On Fri, Aug 20, 2010 at 2:12 PM, Timur Tabi ti...@freescale.com wrote: On Sat, Aug 7, 2010 at 6:36 PM, Wolfgang Denk w...@denx.de wrote: I would like to see any ACK or Tested-by: from actual users of this code. Kumar, who should ack this patch? This patch has been tested and acked by Ira

Re: [U-Boot] MPC8377 system bring up

2010-10-08 Thread Timur Tabi
André Schwarz wrote: Are you using HRCW from Nor on your MPC837x based systems or are you actually using Nand, SPI or I2C PROM ? I've only worked on boards that load the RCW from NOR flash, but some of our BSPs support loading the RCW from other sources, like NAND flash or I2C. -- Timur

Re: [U-Boot] [PATCH 01/17] XPedite5500 board support

2010-10-04 Thread Timur Tabi
-by: Peter Tyser pty...@xes-inc.com CC: Kumar Gala ga...@kernel.crashing.org 17-patch series: Tested-by: Timur Tabi ti...@freescale.com With these patches, the 8610 still compiles and boots. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot

[U-Boot] [PATCH] fsl: add support for NXID v1 EEPROM format

2010-09-30 Thread Timur Tabi
, this new version is called v1. Boards that are shipped with EEPROMs in the NXID v1 format should define CONFIG_SYS_I2C_EEPROM_NXID_1 instead of CONFIG_SYS_I2C_EEPROM_NXID. Signed-off-by: Timur Tabi ti...@freescale.com --- board/freescale/common/sys_eeprom.c | 26 +- 1 files

Re: [U-Boot] ppc44x - watchdog timers reboots during the image loading

2010-09-30 Thread Timur Tabi
. That's what you should debug. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] ppc44x - watchdog timers reboots during the image loading

2010-09-30 Thread Timur Tabi
ame wrote: Finally reset_4xx_watchdog does mtspr(tsr, 0xc000) where tsr is 0x150 IIRC. My guess is that this code doesn't actually reset the watchdog. Perhaps your value of TSR is wrong, or 0xc000 is wrong, or perhaps there's some other watchdog active that you're ignoring. -- Timur

Re: [U-Boot] [PATCH 15/17] 86xx: Create common linker script

2010-09-29 Thread Timur Tabi
/freescale/mpc8610hpcd/u-boot.lds             |  132 --- Have you actually tested this on an 8610? If not, I can test it for you. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http

Re: [U-Boot] [PATCH] fsl_diu_fb: further refactoring of FLS DIU code

2010-09-23 Thread Timur Tabi
/P1022DS.h index 8e0117f..523494d 100644 --- a/include/configs/P1022DS.h +++ b/include/configs/P1022DS.h @@ -178,7 +178,7 @@ #define CONFIG_SYS_PROMPT_HUSH_PS2 /* Video */ -#undef CONFIG_FSL_DIU_FB +#define CONFIG_FSL_DIU_FB don't make this change. -- Timur Tabi Linux kernel developer

Re: [U-Boot] [PATCH] p1022ds: add video support

2010-09-22 Thread Timur Tabi
the code. There may be a way to eliminate any such initialization. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] [v2] p1022ds: add video support

2010-09-22 Thread Timur Tabi
Add support for the DIU controller. If CONFIG_VIDEO is defined, then the console will appear on a DVI monitor instead of the serial port. Signed-off-by: Timur Tabi ti...@freescale.com --- board/freescale/p1022ds/Makefile |2 + board/freescale/p1022ds/diu.c| 148

Re: [U-Boot] [PATCH] p1022ds: add video support

2010-09-21 Thread Timur Tabi
. CONFIG_VIDEO then tells U-Boot to actually use the DIU. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] p1022ds: add video support

2010-09-21 Thread Timur Tabi
of devices that are initialized but not used in U-Boot already. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] p1022ds: use weak CFI flash accessors when DIU is enabled

2010-09-16 Thread Timur Tabi
' commands does not. Also, while flash is being written to, the video display will be blank. Signed-off-by: Timur Tabi ti...@freescale.com --- board/freescale/p1022ds/diu.c | 205 +++-- include/configs/P1022DS.h |6 + 2 files changed, 203 insertions(+), 8

[U-Boot] [PATCH] [v2] p1022ds: use weak CFI flash accessors when DIU is enabled

2010-09-16 Thread Timur Tabi
' commands does not. Also, while flash is being written to, the video display will be blank. Signed-off-by: Timur Tabi ti...@freescale.com --- v2: found another place to use the PMUXCR_ELBCDIU_xxx macros board/freescale/p1022ds/diu.c | 208 +++-- include

Re: [U-Boot] [PATCH] p1022ds: use weak CFI flash accessors when DIU is enabled

2010-09-16 Thread Timur Tabi
? CONFIG_CFI_FLASH_USE_WEAK_ACCESSORS is only defined if CONFIG_VIDEO is defined. It's just one block of macros. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] [v3] p1022ds: use weak CFI flash accessors when DIU is enabled

2010-09-16 Thread Timur Tabi
(with the 'md' and 'cp' commands) does not. Also, while flash is being written, the video display will be blank. Signed-off-by: Timur Tabi ti...@freescale.com --- v3: changes as requested board/freescale/p1022ds/diu.c | 208 +++-- include/configs/P1022DS.h

[U-Boot] Summary of my patches for v2010.12 merge window

2010-09-09 Thread Timur Tabi
maintainer: 1. video: cfb_console: add support for 4bpp bitmaps with GDF_32BIT_X888RGB Thanks. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] MPC8536 custom board without SPD EEPROM configured DDR

2010-09-02 Thread Timur Tabi
, the non-SPD code usually succumbs to bit rot. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] p1022ds: add video support

2010-09-02 Thread Timur Tabi
Add support for the DIU controller. If CONFIG_VIDEO is defined, then the console will appear on a DVI monitor instead of the serial port. Signed-off-by: Timur Tabi ti...@freescale.com --- This patch depends on: [v3] fsl: refactor MPC8610 and MPC5121 DIU code to use existing bitmap and logo

Re: [U-Boot] warning: dereferencing pointer '({anonymous})' does break strict-aliasing rules

2010-09-01 Thread Timur Tabi
On Wed, Sep 1, 2010 at 12:01 PM, Mike Frysinger vap...@gentoo.org wrote: On Tuesday, August 31, 2010 18:30:39 Timur Tabi wrote: I ran Software Update on my Fedora 13 system, and now whenever I build U-Boot, I get a bunch of warnings like this: what version of u-boot exactly ? Any recent

[U-Boot] [PATCH] [v2] mpc8610: refactor DIU initialization code to use existing bitmap and logo features

2010-08-31 Thread Timur Tabi
The Freescale MPC8610 DIU code had re-implement two features that already existed in U-Boot: bitmap drawing and top-of-screen logo (CONFIG_VIDEO_LOGO). So delete the 8610-specific code and use the built-in features instead. Signed-off-by: Timur Tabi ti...@freescale.com --- This patch depends

[U-Boot] warning: dereferencing pointer '({anonymous})' does break strict-aliasing rules

2010-08-31 Thread Timur Tabi
I ran Software Update on my Fedora 13 system, and now whenever I build U-Boot, I get a bunch of warnings like this: dlmalloc.c: In function 'malloc': dlmalloc.c:2252: warning: dereferencing pointer '({anonymous})' does break strict-aliasing rules dlmalloc.c:2252: note: initialized from here

Re: [U-Boot] Why does logo_plot() not use video_display_bitmap()?

2010-08-29 Thread Timur Tabi
. I'll look into merging the two functions together. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] video: cfb_console: fix definition and usage of CURSOR_xxx macros

2010-08-27 Thread Timur Tabi
this: if (console_col CONSOLE_COLS) console_row++; This is obviously not what was intended. Signed-off-by: Timur Tabi ti...@freescale.com --- This patch depends on video: cfb_console: add support for 4bpp bitmaps with GDF_32BIT_X888RGB drivers/video/cfb_console.c

Re: [U-Boot] [PATCH 2/2] cmp_bmp: allow negative X, Y values for display command

2010-08-27 Thread Timur Tabi
I'm rescinding this patch. I have some new code that doesn't need this hack. On Wed, Aug 25, 2010 at 4:36 PM, Timur Tabi ti...@freescale.com wrote: The 'bmp display' command accepts optional X and Y values for the position of the image to be displayed.  Change the code which parses the command

Re: [U-Boot] [PATCH 3/3] mpc8610: refactor DIU initialization code to use existing bitmap function

2010-08-27 Thread Timur Tabi
I'm rescinding this patch. I'll have v2 some time next week. On Thu, Aug 26, 2010 at 3:59 PM, Timur Tabi ti...@freescale.com wrote: The Freescale MPC8610 DIU code has its own bitmap-drawing function, fsl_diu_display_bmp(), which is redundant.  Instead, use the existing function

[U-Boot] Why does logo_plot() not use video_display_bitmap()?

2010-08-27 Thread Timur Tabi
logo_plot() has its own code for rendering a bitmap, but this code is much less capable than video_display_bitmap(). Why doesn't logo_plot() just call video_display_bitmap() to draw the actual bitmap? ___ U-Boot mailing list U-Boot@lists.denx.de

[U-Boot] [PATCH 3/3] mpc8610: refactor DIU initialization code to use existing bitmap function

2010-08-26 Thread Timur Tabi
-by: Timur Tabi ti...@freescale.com --- This patch requires this other patch: video: cfb_console: add support for 4bpp bitmaps with GDF_32BIT_X888RGB board/freescale/common/fsl_diu_fb.c | 155 +++- board/freescale/common/fsl_diu_fb.h |7 +- board

[U-Boot] [PATCH 2/2] cmp_bmp: allow negative X, Y values for display command

2010-08-25 Thread Timur Tabi
of the screen that's 100 pixels tall, then video_hw_init() should say that display is really 1280x924, and then the banner should be drawn at position (0, -100). Signed-off-by: Timur Tabi ti...@freescale.com --- common/cmd_bmp.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git

[U-Boot] [PATCH] video: cfb_console: add support for 4bpp bitmaps with GDF_32BIT_X888RGB

2010-08-23 Thread Timur Tabi
Add support for 4bpp bitmaps, but only when writing to a GDF_32BIT_X888RGB. Signed-off-by: Timur Tabi ti...@freescale.com --- I only added support for GDF_32BIT_X888RGB because that's the only mode that my video device supports in U-Boot. I wouldn't be able to test any other mode, and it took

Re: [U-Boot] [PATCH] always relocate fdt into an lmb-allocated memory block

2010-08-20 Thread Timur Tabi
On Sat, Aug 7, 2010 at 6:36 PM, Wolfgang Denk w...@denx.de wrote: I would like to see any ACK or Tested-by: from actual users of this code. Kumar, who should ack this patch? -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U

Re: [U-Boot] Please pull u-boot-mpc85xx.git

2010-08-02 Thread Timur Tabi
board/freescale/corenet_ds/tlb.c  create mode 100644 include/configs/P4080DS.h  create mode 100644 include/configs/corenet_ds.h ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot -- Timur Tabi Linux kernel

Re: [U-Boot] Please pull u-boot-mpc85xx.git

2010-08-02 Thread Timur Tabi
Wolfgang Denk wrote: Dear Timur Tabi, In message aanlktino-ay_v0homz6vgp-vyyfehpsd3irvjdsf1...@mail.gmail.com you wrote: I have a patch that enables audio on the 1022, can you pick up that one too ? This is not exactly a bug fix, right? Audio is broken in Linux without this patch. It's

Re: [U-Boot] [PATCH] always relocate fdt into an lmb-allocated memory block

2010-08-02 Thread Timur Tabi
Kumar, do you have any issues with this patch? I submitted it in May, but you never picked it up. On Mon, May 24, 2010 at 3:10 PM, Timur Tabi ti...@freescale.com wrote: The device tree (fdt) must always exist in within the bootmap (usually the first 16MB of RAM).  If it doesn't

[U-Boot] [PATCH] fsl: verify writes to the MAC address EEPROM

2010-08-02 Thread Timur Tabi
, the mac save command will display an error message indicating that the write has not succeeded. Signed-off-by: Timur Tabi ti...@freescale.com --- board/freescale/common/sys_eeprom.c | 18 +- 1 files changed, 17 insertions(+), 1 deletions(-) diff --git a/board/freescale/common

Re: [U-Boot] [PATCH] fsl: verify writes to the MAC address EEPROM

2010-08-02 Thread Timur Tabi
Wolfgang Denk wrote: Why is this i2c_read() needed or actually useful? Should the error return code from the i2c_write() above not be sufficient indication that the writing failed? If that was the case, then some other parts of the code need fixing. That's just the way the EEPROM chip works.

Re: [U-Boot] Please pull u-boot-mpc85xx.git

2010-08-02 Thread Timur Tabi
Wolfgang Denk wrote: Without this patch, audio support on the P1022DS is completely broken. We don't have audio support in U-Boot, do we? But we do have P1022DS board support, and my understanding is that part of the function of U-Boot is to prepare the board so that it can be used by Linux,

Re: [U-Boot] Please pull u-boot-mpc85xx.git

2010-08-02 Thread Timur Tabi
Wolfgang Denk wrote: Audio is broken in Linux without this patch. It's something I neglected to incorporate when in my initial P1022DS patch. Well, this is U-Boot, not Linux. I supposed it depends on your definition of a bug. If it's not doing something that it's supposed to be doing, and

Re: [U-Boot] [PATCH] fsl: verify writes to the MAC address EEPROM

2010-08-02 Thread Timur Tabi
Wolfgang Denk wrote: Is it correct to assume the the WP signal is connected to some GPIO pin which can be set / unset in software? No, it's a dip switch on the board. You have to physically set the dip switch in order write protect or unprotect the EEPROM.

Re: [U-Boot] [PATCH] fsl: verify writes to the MAC address EEPROM

2010-08-02 Thread Timur Tabi
Wolfgang Denk wrote: And the setting cannot be read back through some GPIO pin eitheer? Technically, the FPGA (on ngPIXIS boards, of which there are only a few) reads all of the switches and allows software to set and read the values. But I've found it to be unreliable. For example, on one

[U-Boot] [PATCH] p1022ds: add audclk hwconfig setting enable codec reference clock

2010-07-21 Thread Timur Tabi
, 64, and 96KHz. The 11.2896 clock allows 14700, 22050, 29400, 44100, 58800, and 88200Hz. Also configure a pin muxing to select some SSI signals, which will disable I2C1. Signed-off-by: Timur Tabi ti...@freescale.com --- board/freescale/p1022ds/p1022ds.c | 67

[U-Boot] [PATCH] [v2] p1022ds: add audclk hwconfig setting enable codec reference clock

2010-07-21 Thread Timur Tabi
, 64, and 96KHz. The 11.2896 clock allows 14700, 22050, 29400, 44100, 58800, and 88200Hz. Also configure a pin muxing to select some SSI signals, which will disable I2C1. Signed-off-by: Timur Tabi ti...@freescale.com --- board/freescale/p1022ds/p1022ds.c | 67

Re: [U-Boot] [PATCH] p1022ds: add audclk hwconfig setting enable codec reference clock

2010-07-21 Thread Timur Tabi
BTW, the subject line should read: add audclk hwconfig setting to enable codec reference clock Sorry about that. On Wed, Jul 21, 2010 at 4:26 PM, Timur Tabi ti...@freescale.com wrote: The Freescale P1022DS can use either a 12.288MHz or a 11.2896MHz reference clock for the audio codec

[U-Boot] [PATCH] p1022ds: fix indentation of case statement

2010-07-14 Thread Timur Tabi
Signed-off-by: Timur Tabi ti...@freescale.com --- board/freescale/p1022ds/p1022ds.c | 20 ++-- 1 files changed, 10 insertions(+), 10 deletions(-) diff --git a/board/freescale/p1022ds/p1022ds.c b/board/freescale/p1022ds/p1022ds.c index 8cc73ed..be692cb 100644 --- a/board

Re: [U-Boot] [PATCH 8/8] powerpc/85xx: Add memory test feature for mpc85xx.

2010-07-14 Thread Timur Tabi
. It assumes that memory has already been initialized, but on PowerPC parts, we only ever map 2GB of DDR. We need a memory test that covers all of DDR, but that requries PowerPC-specific code to create sliding TLB windows during the memory test. -- Timur Tabi Linux kernel developer at Freescale

Re: [U-Boot] [PATCH v3] fdt: Add function to alloc phandle values

2010-07-10 Thread Timur Tabi
and return the next greater value for a new phandle allocation. Signed-off-by: Kumar Gala ga...@kernel.crashing.org I still think you should merge my fdt_get_max_phandle() function into this patch. The two functions are related and can share code. -- Timur Tabi Linux kernel developer at Freescale

Re: [U-Boot] at91sam9g20 MAC address in EEPROM?

2010-07-10 Thread Timur Tabi
this way, and would rather keep it in EEPROM.  Does anyone have experience with this? Not on that chip, but we do it for Freescale PowerPC SOCs. Take a look at board/freescale/common/sys_eeprom.c -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot

Re: [U-Boot] [PATCH 3/4] powerpc/86xx: Rename PCI1/2 to PCIE1/2 on MPC8641HPCN SBC8641

2010-07-09 Thread Timur Tabi
PCIE1/2 to PCI1/2. Personally, I would drop the 1/2 as well. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 3/4] powerpc/86xx: Rename PCI1/2 to PCIE1/2 on MPC8641HPCN SBC8641

2010-07-09 Thread Timur Tabi
On Fri, Jul 9, 2010 at 9:20 AM, Kumar Gala ga...@kernel.crashing.org wrote: Huh, I'm going from PCI1 - PCIE1 Sorry, I got confused. I really should stop reading patches before I had my coffee. -- Timur Tabi Linux kernel developer at Freescale ___ U

Re: [U-Boot] [PATCH] fdt: Add function to alloc phandle values

2010-07-09 Thread Timur Tabi
. Wolfgang rejected that patch because I didn't include any code that called fdt_get_max_phandle(). I see you're doing the same thing here with fdt_alloc_phandle(). It will be interesting to see if Wolfgang rejects your patch. -- Timur Tabi Linux kernel developer at Freescale

Re: [U-Boot] fdt_resize() bug caused WARNING: could not set linux, initrd-start FDT_ERR_NOSPACE.

2010-07-08 Thread Timur Tabi
building the device tree. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] [v3] powerpc: add support for the Freescale P1022DS reference board

2010-06-30 Thread Timur Tabi
Kumar, please don't forget this patch. On Mon, Jun 14, 2010 at 3:28 PM, Timur Tabi ti...@freescale.com wrote: Add basic suport for the Freescale P1022DS reference board. Specifics: 1) 36-bit only 2) Booting from NOR flash only 3) Environment stored in NOR flash only 4) No SPI support 5

Re: [U-Boot] [PATCH] [v3] powerpc: add support for the Freescale P1022DS reference board

2010-06-30 Thread Timur Tabi
On Wed, Jun 30, 2010 at 9:02 AM, Kumar Gala ga...@kernel.crashing.org wrote: On Jun 30, 2010, at 8:55 AM, Timur Tabi wrote: Kumar, please don't forget this patch. It applies against Wolfgang's 'next' branch. Have you updated your branch to pull in the new Makefile code? -- Timur Tabi Linux

Re: [U-Boot] [PATCH] examples/standalone: Remove relocation compile flags for PowerPC

2010-06-16 Thread Timur Tabi
include it, and just add a comment saying that it only works on gcc 4.2 or higher? -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] examples/standalone: Remove relocation compile flags for PowerPC

2010-06-15 Thread Timur Tabi
Peter Tyser wrote: Previously, standalone applications were compiled with gcc flags that produced relocatable executables on the PowerPC architecture (eg with the -mrelocatable and -fPIC flags). There's no reason for these applications to be fully relocatable at this time since no relocation

Re: [U-Boot] [PATCH] examples/standalone: Remove relocation compile flags for PowerPC

2010-06-15 Thread Timur Tabi
Timur Tabi wrote: Peter Tyser wrote: Previously, standalone applications were compiled with gcc flags that produced relocatable executables on the PowerPC architecture (eg with the -mrelocatable and -fPIC flags). There's no reason for these applications to be fully relocatable at this time

Re: [U-Boot] [PATCH] examples/standalone: Remove relocation compile flags for PowerPC

2010-06-15 Thread Timur Tabi
Peter Tyser wrote: Can you do a 'make clean', then recompile? That *was* with a 'make clean'. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] examples/standalone: Remove relocation compile flags for PowerPC

2010-06-15 Thread Timur Tabi
Wolfgang Denk wrote: I think the timer code is sufficient to show the problem, and that your fix helps. If Timur confirms it's working for his secret code too we should apply this. Doing this at the top of my source file: static int __flash_wp(int argc, char *argv[]); int

Re: [U-Boot] [PATCH] examples/standalone: Remove relocation compile flags for PowerPC

2010-06-15 Thread Timur Tabi
Wolfgang Denk wrote: I think the static int __flash_wp() thing is not even needed assuming that flash_wp() is the first function in your file. That's the thing -- I don't want flash_wp() to be the first function in my file. I would then need a forward reference to all the other functions in my

[U-Boot] [PATCH] [v3] powerpc: add support for the Freescale P1022DS reference board

2010-06-14 Thread Timur Tabi
Add basic suport for the Freescale P1022DS reference board. Specifics: 1) 36-bit only 2) Booting from NOR flash only 3) Environment stored in NOR flash only 4) No SPI support 5) No DIU support Signed-off-by: Timur Tabi ti...@freescale.com --- This patch is for the 'next' branch. v3: Fixed

Re: [U-Boot] [PATCH] display the entry point when compiling a stand-alone program

2010-06-11 Thread Timur Tabi
Detlev Zundel wrote: We should concentrate on creating a correct linker script so that the entry point ends up where we want it to be, namely at STANDALONE_LOAD_ADDR. Can you have a look at this and maybe solve the problem for good then? I have no idea how to do that. I don't know diddly

[U-Boot] [PATCH] display the entry point when compiling a stand-alone program

2010-06-10 Thread Timur Tabi
-by: Timur Tabi ti...@freescale.com --- examples/standalone/Makefile |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/examples/standalone/Makefile b/examples/standalone/Makefile index 6ea3b93..d6a01ea 100644 --- a/examples/standalone/Makefile +++ b/examples/standalone/Makefile

<    1   2   3   4   5   6   7   >