linux-mips.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Wu Zhangjin <wuzhangjin@gmail.com>
To: Alexander Clouter <alex@digriz.org.uk>
Cc: linux-mips@linux-mips.org
Subject: Re: [PATCH] MIPS: add support for gzip/bzip2/lzma compressed kernel images
Date: Mon, 10 Aug 2009 17:11:40 +0800	[thread overview]
Message-ID: <1249895500.20588.10.camel@falcon> (raw)
In-Reply-To: <e4s3l6-dou.ln1@chipmunk.wormnet.eu>

Hi,

On Sun, 2009-08-09 at 22:36 +0100, Alexander Clouter wrote:
> 
> This has made my AR7[1] based Linksys WAG54Gv2 (16MB RAM and 4MB flash, 
> limited to a 768kB kernel) useful, finally...thanks!
> > 
> I got it working (LZMA kernel) however you have hardcoded a lot of bits 
> in there.  It looks to my uneducated eye most of the issues lie in that 
> getting a suitable PORT(x), KERNEL_START, KERNEL_SIZE, FREE_MEM_START 
> and FREE_MEM_END is non-trivial on the MIPS platform currently; 
> probably because of the lack of a generic lzma/gzip/bzip2 framework to 
> be used with.
> 
> For me I used:
>   #define FREE_MEM_START CKSEG0ADDR(0x94a00000)
>   #define FREE_MEM_END CKSEG0ADDR(0x94f00000)
> 
> I had to replace (you probably should move this from dbg.c to dbg.h):
>   #define PORT(offset) (CKSEG1ADDR(AR7_REGS_UART0) + (4 * offset))
> 
> The load address is awkward, but I replaced your 0x8100000 in the 
> Makefile with 0x94400000.
> 
> Now, the big question is how to work this all out at compile time. :)
> 

Thanks very much for your suggestion, just rewritten it and removed
almost all of the hard-coded parts(except the size of the HEAP) and a
new patch was sent out. and this new one _should_ work for the other
MIPS-based machines if not missed something else.

looking forward to your test result on AR7 with the new patch and
welcome to comment to this new E-mail thread: [PATCH -v1] MIPS: add
support for gzip/bzip2/lzma compressed kernel images

> As a side note, I would personally leave the DEBUG non-optional and 
> turned on as it all disappears at runtime anyway, but I'm no kernel 
> developer :)

ooh, in reality, I'm looking forward to the comment of Ralf and somebody
else. to my point of view, this _debug_ interface is only need to help
the developers of the other MIPS variants ensure this patch work for
them.

Thanks & Regards,
Wu Zhangjin

  reply	other threads:[~2009-08-10  9:12 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-08 18:55 [PATCH] MIPS: add support for gzip/bzip2/lzma compressed kernel images Wu Zhangjin
2009-08-09 21:36 ` Alexander Clouter
2009-08-10  9:11   ` Wu Zhangjin [this message]
2009-08-10 13:55   ` Ralf Baechle
     [not found]     ` <4A803455.5010304@team-embedded.nl>
2009-08-10 15:26       ` Ralf Baechle

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=1249895500.20588.10.camel@falcon \
    --to=wuzhangjin@gmail.com \
    --cc=alex@digriz.org.uk \
    --cc=linux-mips@linux-mips.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).