Message ID | 20221226123602.56856-1-kever.yang@rock-chips.com |
---|---|
State | Changes Requested |
Delegated to: | Tom Rini |
Headers | show |
Series | Pull request: u-boot-rockchip/for-next | expand |
On Mon, Dec 26, 2022 at 08:36:02PM +0800, Kever Yang wrote: > Hi Tom, > > Please pull the updates for rockchip platform for next: > - Add support for rv1126 soc and rv1126 neu2 io board; > - Add support for rk3399 pine64 pinephone pro board; > - dts sync from linux for rk3399 and px30; > > Travis: > https://source.denx.de/u-boot/custodians/u-boot-rockchip/-/jobs/546342 > > Thanks, > - Kever > > The following changes since commit 3e4cbe184a0f6537abb457d6afa61224396c4e46: > > Merge branch '2022-12-23-complete-phase1-CONFIG-migration' into next (2022-12-23 22:19:39 -0500) > > are available in the Git repository at: > > https://source.denx.de/u-boot/custodians/u-boot-rockchip.git for-next > > for you to fetch changes up to 9636b196c7046f1efa16fcfe7e1df9dcbb0f77fc: > > board: rockchip: Add Edgeble Neu2 IO Board (2022-12-26 13:53:26 +0800) > The build for pinephone-pro-rk3399 fails. Note that it's not that CONFIG_SYS_TEXT_BASE became CFG_SYS_TEXT_BASE but rather it was renamed too CONFIG_TEXT_BASE, as I see that's where the build fails: $ make CROSS_COMPILE=~/.buildman-toolchains/gcc-12.2.0-nolibc/aarch64-linux/bin/aarch64-linux- -sj pinephone-pro-rk3399_config all .config:50:warning: symbol value '' invalid for LNX_KRNL_IMG_TEXT_OFFSET_BASE .config:180:warning: symbol value '' invalid for TEXT_BASE
On Mon, Dec 26, 2022 at 3:10 PM Tom Rini <trini@konsulko.com> wrote: > > On Mon, Dec 26, 2022 at 08:36:02PM +0800, Kever Yang wrote: > > > Hi Tom, > > > > Please pull the updates for rockchip platform for next: > > - Add support for rv1126 soc and rv1126 neu2 io board; > > - Add support for rk3399 pine64 pinephone pro board; > > - dts sync from linux for rk3399 and px30; > > > > Travis: > > https://source.denx.de/u-boot/custodians/u-boot-rockchip/-/jobs/546342 > > > > Thanks, > > - Kever > > > > The following changes since commit 3e4cbe184a0f6537abb457d6afa61224396c4e46: > > > > Merge branch '2022-12-23-complete-phase1-CONFIG-migration' into next (2022-12-23 22:19:39 -0500) > > > > are available in the Git repository at: > > > > https://source.denx.de/u-boot/custodians/u-boot-rockchip.git for-next > > > > for you to fetch changes up to 9636b196c7046f1efa16fcfe7e1df9dcbb0f77fc: > > > > board: rockchip: Add Edgeble Neu2 IO Board (2022-12-26 13:53:26 +0800) > > > > The build for pinephone-pro-rk3399 fails. Note that it's not that > CONFIG_SYS_TEXT_BASE became CFG_SYS_TEXT_BASE but rather it was renamed > too CONFIG_TEXT_BASE, as I see that's where the build fails: > $ make CROSS_COMPILE=~/.buildman-toolchains/gcc-12.2.0-nolibc/aarch64-linux/bin/aarch64-linux- -sj pinephone-pro-rk3399_config all > .config:50:warning: symbol value '' invalid for LNX_KRNL_IMG_TEXT_OFFSET_BASE > .config:180:warning: symbol value '' invalid for TEXT_BASE Happy to either do a new rev or to do a small patch on top of that which Tom can apply directly, which ever is easier. P
On 2022/12/27 09:11, Peter Robinson wrote: > On Mon, Dec 26, 2022 at 3:10 PM Tom Rini <trini@konsulko.com> wrote: >> On Mon, Dec 26, 2022 at 08:36:02PM +0800, Kever Yang wrote: >> >>> Hi Tom, >>> >>> Please pull the updates for rockchip platform for next: >>> - Add support for rv1126 soc and rv1126 neu2 io board; >>> - Add support for rk3399 pine64 pinephone pro board; >>> - dts sync from linux for rk3399 and px30; >>> >>> Travis: >>> https://source.denx.de/u-boot/custodians/u-boot-rockchip/-/jobs/546342 >>> >>> Thanks, >>> - Kever >>> >>> The following changes since commit 3e4cbe184a0f6537abb457d6afa61224396c4e46: >>> >>> Merge branch '2022-12-23-complete-phase1-CONFIG-migration' into next (2022-12-23 22:19:39 -0500) >>> >>> are available in the Git repository at: >>> >>> https://source.denx.de/u-boot/custodians/u-boot-rockchip.git for-next >>> >>> for you to fetch changes up to 9636b196c7046f1efa16fcfe7e1df9dcbb0f77fc: >>> >>> board: rockchip: Add Edgeble Neu2 IO Board (2022-12-26 13:53:26 +0800) >>> >> The build for pinephone-pro-rk3399 fails. Note that it's not that >> CONFIG_SYS_TEXT_BASE became CFG_SYS_TEXT_BASE but rather it was renamed >> too CONFIG_TEXT_BASE, as I see that's where the build fails: >> $ make CROSS_COMPILE=~/.buildman-toolchains/gcc-12.2.0-nolibc/aarch64-linux/bin/aarch64-linux- -sj pinephone-pro-rk3399_config all Sorry, Tom, I only got error with "bl31.bin", I though it was not from new patches of rockchip changes. >> .config:50:warning: symbol value '' invalid for LNX_KRNL_IMG_TEXT_OFFSET_BASE >> .config:180:warning: symbol value '' invalid for TEXT_BASE > Happy to either do a new rev or to do a small patch on top of that > which Tom can apply directly, which ever is easier. Peter, A new rev will be better for me, please help to pass buildman. I will pick the new patch and make a new PR. Thanks, - Kever > > P
Hi Kever and Tom, > >> The build for pinephone-pro-rk3399 fails. Note that it's not that > >> CONFIG_SYS_TEXT_BASE became CFG_SYS_TEXT_BASE but rather it was renamed > >> too CONFIG_TEXT_BASE, as I see that's where the build fails: > >> $ make CROSS_COMPILE=~/.buildman-toolchains/gcc-12.2.0-nolibc/aarch64-linux/bin/aarch64-linux- -sj pinephone-pro-rk3399_config all > Sorry, Tom, I only got error with "bl31.bin", I though it was not from > new patches of rockchip changes. > >> .config:50:warning: symbol value '' invalid for LNX_KRNL_IMG_TEXT_OFFSET_BASE > >> .config:180:warning: symbol value '' invalid for TEXT_BASE > > Happy to either do a new rev or to do a small patch on top of that > > which Tom can apply directly, which ever is easier. > > Peter, A new rev will be better for me, please help to pass buildman. > > I will pick the new patch and make a new PR. I sent out a v2, there were some other changes that needed to be synced too like the DM_VIDEO -> VIDEO change, I also picked up some other fixes, in testing I had some issues with the base rk3399.dtsi sync which i couldn't easily get to the bottom of so I've dropped that patch for the time being and will revisit it again later. Sorry for the delay. Peter