Re: Re: [BUG] binman: Add a library to access binman entries

2020-02-16 Thread Frank Wunderlich
Hi Simon,Ryder The binman-option is introduced by this commit: https://gitlab.denx.de/u-boot/u-boot/commit/cbd2fba1eca11300649052e289685e7404e0b81c add basic support for MT7629 boards @ryder is binman used (mt7629)? R2 and r64 are affected from this issue (boot broken by binman_fdt). Have it

Re: Re: [BUG] binman: Add a library to access binman entries

2020-02-16 Thread Simon Glass
Hi Frank, On Thu, 13 Feb 2020 at 10:51, Frank Wunderlich wrote: > > Hi, > > i guess you mean this (board is bananapi R64, not M64): > > board/mediatek/mt7622/mt7622_rfb.c > arch/arm/dts/mt7622.dtsi > arch/arm/dts/mt7622-rfb.dts > > currently i have added > > CONFIG_BINMAN_FDT=n > > to configs/mt7

Aw: Re: [BUG] binman: Add a library to access binman entries

2020-02-13 Thread Frank Wunderlich
13. Februar 2020 um 18:40 Uhr > Von: "Simon Glass" > Betreff: Re: [BUG] binman: Add a library to access binman entries > > Hi Frank, > > On Wed, 12 Feb 2020 at 11:02, Frank Wunderlich > wrote: > > > > Hi > > > > Why not just disable binman_fdt (

Re: [BUG] binman: Add a library to access binman entries

2020-02-13 Thread Simon Glass
Hi Frank, On Wed, 12 Feb 2020 at 11:02, Frank Wunderlich wrote: > > Hi > > Why not just disable binman_fdt (or not default y)? Your way sounds more > complex. As i do not understand it and see no benefit for this board, i would > leave it disabled. This leave time for a thoughtful solution > Ma

Re: [BUG] binman: Add a library to access binman entries

2020-02-12 Thread Frank Wunderlich
Hi Why not just disable binman_fdt (or not default y)? Your way sounds more complex. As i do not understand it and see no benefit for this board, i would leave it disabled. This leave time for a thoughtful solution Maybe mtk knows a better way... Sunxi sounds wrong because board is not with all

Re: [BUG] binman: Add a library to access binman entries

2020-02-12 Thread Simon Glass
Hi Frank, On Wed, 12 Feb 2020 at 04:50, Frank Wunderlich wrote: > > Am 12. Februar 2020 00:03:18 MEZ schrieb Simon Glass : > >Hi Frank, > > > >Sorry for the delay. Stephen hit this also. > > Hi Simon, > good that my mail was not lost > > >> > >> this line [1] (in my case) breaks the init-chain: >

Re: [BUG] binman: Add a library to access binman entries

2020-02-12 Thread Frank Wunderlich
Am 12. Februar 2020 00:03:18 MEZ schrieb Simon Glass : >Hi Frank, > >Sorry for the delay. Stephen hit this also. Hi Simon, good that my mail was not lost 😁 >> >> this line [1] (in my case) breaks the init-chain: >> >> return log_msg_ret("binman node", -EINVAL); >> >> the binman_init [2] is added

Re: [BUG] binman: Add a library to access binman entries

2020-02-11 Thread Simon Glass
Hi Frank, On Fri, 24 Jan 2020 at 11:15, Frank Wunderlich wrote: > > Hi, > > a bit more info about this... Sorry for the delay. Stephen hit this also. > > this line [1] (in my case) breaks the init-chain: > > return log_msg_ret("binman node", -EINVAL); > > the binman_init [2] is added to init_se

Re: [BUG] binman: Add a library to access binman entries

2020-01-24 Thread Frank Wunderlich
Hi, a bit more info about this... this line [1] (in my case) breaks the init-chain: return log_msg_ret("binman node", -EINVAL); the binman_init [2] is added to init_sequence_r[] which is executed by initcall_run_list ./common/board_r.c:897: if (initcall_run_list(init_sequence_r)) exiting the

[BUG] binman: Add a library to access binman entries

2020-01-22 Thread Frank Wunderlich
Hi, this Patch seems to break init-chain at least on bpi-r64 if BINMAN_FDT not disabled (enabled by default). adding some code for debugging 38 int binman_init(void) 39 { 40 binman = malloc(sizeof(struct binman_info)); 41 printf("%s:%d",__FUNCTION__,__LINE__); 42 if (!bin