mbox series

[0/3] ramips: fix OKLI lzma-loader

Message ID 20200902062231.2238353-1-gch981213@gmail.com
Headers show
Series ramips: fix OKLI lzma-loader | expand

Message

Chuanhong Guo Sept. 2, 2020, 6:22 a.m. UTC 
OKLI mode of our lzma-loader in ramips still contains
left-over bits from ar71xx and isn't used before.
This patchset fixes it so that it can be used in
future devices.
(e.g. https://github.com/openwrt/openwrt/pull/2645)

Chuanhong Guo (3):
  ramips: lzma-loader: make FLASH_START configurable
  ramips: image: add recipe for OKLI loader
  ramips: image: example usage of OKLI loader

 target/linux/ramips/image/Makefile            | 28 ++++++++++++++++++-
 .../linux/ramips/image/lzma-loader/Makefile   |  2 ++
 .../ramips/image/lzma-loader/src/Makefile     |  5 ++++
 .../ramips/image/lzma-loader/src/loader.c     |  5 +---
 target/linux/ramips/image/mt7620.mk           |  1 +
 target/linux/ramips/image/mt7621.mk           |  2 +-
 6 files changed, 37 insertions(+), 6 deletions(-)

Comments

Adrian Schmutzler Sept. 2, 2020, 5 p.m. UTC | #1 
Hi,

> -----Original Message-----
> From: openwrt-devel [mailto:openwrt-devel-bounces@lists.openwrt.org]
> On Behalf Of Chuanhong Guo
> Sent: Mittwoch, 2. September 2020 08:22
> To: openwrt-devel@lists.openwrt.org
> Cc: Chuanhong Guo <gch981213@gmail.com>
> Subject: [PATCH 0/3] ramips: fix OKLI lzma-loader
> 
> OKLI mode of our lzma-loader in ramips still contains left-over bits from
> ar71xx and isn't used before.
> This patchset fixes it so that it can be used in future devices.
> (e.g. https://github.com/openwrt/openwrt/pull/2645)

Thanks for taking care of that. In case you haven't read already, I have tidied up/rebased the Hootoo PR onto these changes here:
https://git.openwrt.org/?p=openwrt/staging/adrian.git;a=shortlog;h=refs/heads/hootoo

Those are tested by the author now.

Depending on what you prefer, I'd either wait for you to merge your two commits and then push the final one, or just merge all three myself.

Best

Adrian

> 
> Chuanhong Guo (3):
>   ramips: lzma-loader: make FLASH_START configurable
>   ramips: image: add recipe for OKLI loader
>   ramips: image: example usage of OKLI loader
> 
>  target/linux/ramips/image/Makefile            | 28 ++++++++++++++++++-
>  .../linux/ramips/image/lzma-loader/Makefile   |  2 ++
>  .../ramips/image/lzma-loader/src/Makefile     |  5 ++++
>  .../ramips/image/lzma-loader/src/loader.c     |  5 +---
>  target/linux/ramips/image/mt7620.mk           |  1 +
>  target/linux/ramips/image/mt7621.mk           |  2 +-
>  6 files changed, 37 insertions(+), 6 deletions(-)
> 
> --
> 2.26.2
> 
> 
> _______________________________________________
> openwrt-devel mailing list
> openwrt-devel@lists.openwrt.org
> https://lists.openwrt.org/mailman/listinfo/openwrt-devel
Chuanhong Guo Sept. 3, 2020, 1:27 a.m. UTC | #2 
Hi!

On Thu, Sep 3, 2020 at 1:00 AM Adrian Schmutzler
<mail@adrianschmutzler.de> wrote:
> Thanks for taking care of that. In case you haven't read already, I have tidied up/rebased the Hootoo PR onto these changes here:
> https://git.openwrt.org/?p=openwrt/staging/adrian.git;a=shortlog;h=refs/heads/hootoo
>
> Those are tested by the author now.
>
> Depending on what you prefer, I'd either wait for you to merge your two commits and then push the final one, or just merge all three myself.

Feel free to merge all 3 patches :)