headers
Marek Vasut | 1 May 2010 04:48
Picon

RFC: U-Boot OneNAND IPL TEXT_BASE

Hey,

I've been tinkering with OneNAND IPL in uboot. I found out it wan't to load 
itself to the address specified in board/$(BOARDDIR)/config.mk . That's fine in 
most cases, but in my case that wasn't possible.

In my case, SDRAM init didn't fit into the IPL, so I had to copy U-Boot into 
SRAM, then execute it and let it relocate itself into SDRAM. One more time:
IPL: OneNAND->SRAM
U-Boot: SRAM->SDRAM

It all works fine, but when compiling the IPL, I had to alter TEXT_BASE not to 
point into SDRAM but SRAM too (because of stack). I introduced a variable called 
IPL which allows using config.mk from (for example) onenand-
ipl/board/$(BOARDDIR)/config.mk rather than the board/$(BOARDDIR)/config.mk one, 
which is in my opinion a correct behaviour.

Any opinions?

Thanks in advance.

diff --git a/config.mk b/config.mk
index 73b5195..8639580 100644
--- a/config.mk
+++ b/config.mk
 <at>  <at>  -130,9 +130,13  <at>  <at>  BOARDDIR = $(VENDOR)/$(BOARD)
 else
 BOARDDIR = $(BOARD)
 endif
+ifdef  IPL
(Continue reading)

Permalink | Reply | 4 comments |
headers
Marek Vasut | 1 May 2010 05:24
Picon

Re: [OT] - Seeking information Re: Memory Mapped CF Card (IDE) and Linux

Dne Pá 30. dubna 2010 16:11:53 Graeme Russ napsal(a):
> Hi All,
> 
> I'm looking for information on how to access a Compact Flash card which is
> memory-mapped (from 0x20001000)
> 
> From what I have read, there a 3 address lines (8 addresses) which access
> IDE registers within the CF chip
> 
> My question is, how do I access such a custom address range within Linux?
> 
> I'm looking at the 'OF-platform PATA driver' which appears to be what I
> want (a very basic IDE driver) but I am a bit lost on how to set up a
> specific base address.
> 
> Where should I look for examples of CF cards mapped to specific address
> spaces?
> 
> TIA

Hey, look at pata_platform.c driver. Without any additional setup, your drive 
will work. You just specify IRQ line (if any) and CTL and CMD addresses.

Cheers
> 
> Graeme
> _______________________________________________
> U-Boot mailing list
> U-Boot <at> lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
(Continue reading)

Permalink | Reply | 6 comments |
headers
Vipin Kumar | 1 May 2010 12:09
Picon

Re: [PATCH v2 00/17] Network support for spear platform and spear1300 support

On Fri, Apr 30, 2010 at 6:18 PM, Tom Rix <tom <at> bumblecow.com> wrote:
> Vipin KUMAR wrote:
>> Hello Tom/Wolfgang,
>> Didn't see any feedbacks on this patchset.
>> Please let me know if the patch is good or should I make changes and resubmit
>>
>
> Vipin,
> I will provide you with complete review of your patch set this weekend.
>
> Please use MAKEALL arm to check for regressions.
> There is a compile error for spear310 and spear320.
> Here is the spear310 error, spear320 is similar
>
>        spear310.c: In function 'board_eth_init':
>        spear310.c:64: warning: implicit declaration of function 'dw_mii_initialize'
>        board/spear/spear310/libspear310.a(spear310.o): In function `board_eth_init':
>        .../u-boot/src/u-boot-arm/board/spear/spear310/spear310.c:64: undefined
> reference to `dw_mii_initialize'
>
Yes, I noticed this problem immediately after sending the patch
Would be resolved in v3.

> This error must be resolved.
>
> FYI my email address has changed to tom <at> bumblecow.com.
>
Yes, I got a bounce on earlier mail-id

Thanks
(Continue reading)

Permalink | Reply | 0 comments |
headers
Markus Gothe | 1 May 2010 13:11
Picon
Picon
Picon
Favicon

Re: Exception in U-Boot

And I thought this mailling-list was dead... :-&

//Markus
On 29 Apr 2010, at 16:04, Sanjay Kumar wrote:

> Hi All,
>      We are porting U-boot. 2010.03  version for MIPS32 processor  
> (Au1350). Currently we are facing one problem:
>
> When I compile u-boot and program to NOR Flash using bdiGDB through  
> BDI3000 Debugger, I am observing u-boot hangs at exception i.e. at  
> address 0xbfc00570 after reset .
>
> error log is below:
>
>
> Au1350>ti
>    Core number       : 0
>    Core state        : Debug Mode
>    Debug entry cause : single step
>    Current PC        : 0xbfc00400
>    Current SR        : 0x00400004
>    Current LR  (r31) : 0xbfc0f6e4
>    Current SP  (r29) : 0x803fff00
> Au1350>go
> Au1350>halt
>    Core number       : 0
>    Core state        : Debug Mode
>    Debug entry cause : JTAG break request
>    Current PC        : 0xbfc00570
(Continue reading)

Permalink | Reply | 0 comments |
headers
Stefano Babic | 1 May 2010 13:49
Picon
Picon
Favicon

Re: [PATCH 1/2] mtdparts: fix write through NULL pointer

Wolfgang Denk wrote:

> I tend to prefer my version, as it seems to be more defensive, and it
> also adds an assert() for the 3rd parameter.
> 
> If you agree, I'll apply my version?

Yes, of course.

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-0 Fax: +49-8142-66989-80  Email: office <at> denx.de
=====================================================================
Permalink | Reply | 0 comments |
headers
Anatolij Gustschin | 1 May 2010 20:46
Picon
Picon
Favicon

Re: [PATCH 2/3] video: add support for display controller in MB86R0x SoCs

Dear Matthias,

On Thu, 22 Apr 2010 12:30:56 +0200
Matthias Weisser <weisserm <at> arcor.de> wrote:
...
> diff --git a/drivers/video/cfb_console.c b/drivers/video/cfb_console.c
> index d1f47c9..4769cdb 100644
> --- a/drivers/video/cfb_console.c
> +++ b/drivers/video/cfb_console.c
>  <at>  <at>  -153,6 +153,14  <at>  <at>  CONFIG_VIDEO_HW_CURSOR:	     - Uses the hardware cursor capability of the
>  #endif
>  
>  /*****************************************************************************/
> +/* Defines for the MB86R0xGDC driver					     */
> +/*****************************************************************************/
> +#ifdef CONFIG_VIDEO_MB86R0xGDC
> +
> +#define VIDEO_FB_16BPP_WORD_SWAP
> +#endif

Please do not add VIDEO_FB_16BPP_WORD_SWAP #define here.
Add it into your include/configs/jadecpu.h file in the video
related section like other boards do. Thanks!

Best regards,
Anatolij
Permalink | Reply | 0 comments |
headers
Anatolij Gustschin | 1 May 2010 20:51
Picon
Picon
Favicon

Re: [PATCH 2/3] video: add support for display controller in MB86R0x SoCs

Dear Matthias,

On Thu, 22 Apr 2010 15:03:55 +0200
Matthias Weißer <weisserm <at> arcor.de> wrote:
...
> >> +/*
> >> + * Set a RGB color in the LUT
> >> + */
> >> +void video_set_lut(unsigned int index, unsigned char r,
> >> +					unsigned char g, unsigned char b)
> >> +{
> >> +
> >> +}
> >
> > Code seems to be missing?
> 
> The driver doesn't support palletized color format at the moment but 
> removing this function leads to a linker error.
> 
> Maybe we should add a config option to disable palletized color format 
> or add a weak function somewhere. Maybe Anatolij can comment on this 
> issue also.

I will send a patch adding weak default video_set_lut() to cfb_console
driver. You can remove empty function then.

Best regards,
Anatolij
_______________________________________________
U-Boot mailing list
(Continue reading)

Permalink | Reply | 0 comments |
headers
Anatolij Gustschin | 1 May 2010 21:05
Picon
Picon
Favicon

[PATCH] video: cfb_console: add weak default video_set_lut()

Do not enforce drivers to provide empty video_set_lut()
if they do not implement indexed color (8 bpp) frame
buffer support. Add default function to the cfb_console
driver and remove empty video_set_lut() functions.

Signed-off-by: Anatolij Gustschin <agust <at> denx.de>
---
 arch/powerpc/cpu/mpc512x/diu.c                |   14 --------------
 board/freescale/mpc8610hpcd/mpc8610hpcd_diu.c |    9 ---------
 drivers/video/cfb_console.c                   |   12 ++++++++++++
 drivers/video/sm501.c                         |   13 -------------
 4 files changed, 12 insertions(+), 36 deletions(-)

diff --git a/arch/powerpc/cpu/mpc512x/diu.c b/arch/powerpc/cpu/mpc512x/diu.c
index f8d19a0..1cce6a2 100644
--- a/arch/powerpc/cpu/mpc512x/diu.c
+++ b/arch/powerpc/cpu/mpc512x/diu.c
 <at>  <at>  -179,18 +179,4  <at>  <at>  void *video_hw_init(void)
 	return (void *)pGD;
 }

-/**
-  * Set the LUT
-  *
-  *  <at> index: color number
-  *  <at> r: red
-  *  <at> b: blue
-  *  <at> g: green
-  */
-void video_set_lut
(Continue reading)

Permalink | Reply | 2 comments |
headers
Tom Rix | 1 May 2010 23:50

ARM pull request

Wolfgang,
Please pull from arm/master.
Tom

The following changes since commit 3699c28e6d16b563629c285311a0ce62a2c4c5d0:
   Wolfgang Denk (1):
         Merge branch 'master' of git://git.denx.de/u-boot-video

are available in the git repository at:

   git://git.denx.de/u-boot-arm master

Alexander Holler (1):
       at91: add defines for RTT and GPBR

Asen Dimov (5):
       pm9261 converted to at91 soc access
       pm9261: remove CONFIG_CMD_AUTOSCRIPT
       pm9263: remove CONFIG_CMD_AUTOSCRIPT
       at91: define matrix registers bit fields
       pm9263 converted to at91 soc access

Fabio Estevam (1):
       MX51EVK: Remove CPLD related code

Frans Meulenbroeks (2):
       configs/sheevaplug: added a few additional commands
       configs/openrd_base.h: reordered macros

Heiko Schocher (2):
(Continue reading)

Permalink | Reply | 1 comment |
headers
Kyungmin Park | 2 May 2010 05:54
Favicon

Re: RFC: U-Boot OneNAND IPL TEXT_BASE

Hi,

Which CPU do you use? In most ARM cpu, CPU load the IPL into its
internal SRAM, and runs at here.
But in your case it's not. So you maybe redefine IPL address.

Instead of modifying the config.mk, how about to define IPL_TEXT_BASE.

e.g.,

In OneNAND IPL code,

#ifndef BOARD_IPL_TEXT_BASE
#define ONENAND_IPL_TEXT_BASE BOARD_IPL_TEXT_BASE
#endif

load IPL code ONENAND_IPL_TEXT_BASE instead of current code.

Thank you,
Kyungmin Park

On Sat, May 1, 2010 at 11:48 AM, Marek Vasut <marek.vasut <at> gmail.com> wrote:
> Hey,
>
> I've been tinkering with OneNAND IPL in uboot. I found out it wan't to load
> itself to the address specified in board/$(BOARDDIR)/config.mk . That's fine in
> most cases, but in my case that wasn't possible.
>
> In my case, SDRAM init didn't fit into the IPL, so I had to copy U-Boot into
> SRAM, then execute it and let it relocate itself into SDRAM. One more time:
(Continue reading)

Permalink | Reply | 3 comments |

Gmane