Hello,

On Saturday 27 August 2011 12:45:41 Andrew wrote:
> Hello all,
> I'm currently working on rtl8196 port, and those weird guys from realtek
> implemented there a different architecture rlx in kernel.
> Basically, much is borrowed from mips, but small differences here and
> there are present. I applied openwrt command hack to rlx, however, that
> didn't work out of the box.
> To get patch-cmdline actually patch rlx's vmlinux I had to widen the
> SEARCH_SPACE (
> http://invyl.ath.cx/cgit/cgit.cgi/openwrt/commit/?id=38a13e566f179a12803a39
> 1ca941cad1f806db08 ). The next thing was, that despite the image gets
> patched, and I see the correct text hexdump'ing the image, the kernel
> command line stays the same. Looks like some other place needs to get
> patched, but I just can't firure where to look. Any ideas?

The bootloader most likely overrides the command-line as well, so make sure 
your prom code concatenates both command-line.

> Thanks in advance.
> 
> I have now fixed the image packing for initial support for different
> boards, and, hopefully I'll get spi flash up&running next week. What should
> the state of the port be, before it's ok to send in the patches to the
> trunk? :)

I had a quick look at the files in your repository:

- please merge support for rtl8196c and rtl8196b, they do not differ that much

- please use the patches that I posted on the mailing-list as much as you can, 
the code is much cleaner and integrates better into the existing mips 
architecture (special checksum, memcpy functions etc ...)

- remove all the dead code ifdefed'out with #if 0

- port your patches to 2.6.39 at least, 3.0 would be even better

- remove all the unnecessary hacks introduced by patches-2.6.30, they are not 
required

- clarify the license of the tools located in image/imgtools, they do not seem 
to be open source (they have no license at all)

Thank you very much.
--
Florian
_______________________________________________
openwrt-devel mailing list
[email protected]
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to