[U-Boot] [PATCH] gdsys: Drop print_fpga_state function

2016-07-15 Thread Tom Rini
On most platforms the print_fpga_state function is never called. Only on dlvision-10g do we, so in that case inline it. Drop it from everywhere else to avoid extra strings. Signed-off-by: Tom Rini Acked-by: Reinhard Pfau Acked-by: Dirk Eibach --- Changes in v2: - Per Reinhard, use fpga_state

Re: [U-Boot] [PATCH] gdsys: Drop print_fpga_state function

2016-06-02 Thread Tom Rini
On Thu, Jun 02, 2016 at 08:54:12PM +0200, Dirk Eibach wrote: > Hi Tom, > > sorry for the delay, and thanks for keeping iocon alive :) > > 2016-06-02 16:05 GMT+02:00 Reinhard Pfau : > ... > >> diff --git a/board/gdsys/405ep/dlvision-10g.c > >> b/board/gdsys/405ep/dlvision-10g.c > >> index 54c7eb3.

Re: [U-Boot] [PATCH] gdsys: Drop print_fpga_state function

2016-06-02 Thread Dirk Eibach
Hi Tom, sorry for the delay, and thanks for keeping iocon alive :) 2016-06-02 16:05 GMT+02:00 Reinhard Pfau : ... >> diff --git a/board/gdsys/405ep/dlvision-10g.c >> b/board/gdsys/405ep/dlvision-10g.c >> index 54c7eb3..def4f9c 100644 >> --- a/board/gdsys/405ep/dlvision-10g.c >> +++ b/board/gdsys/

Re: [U-Boot] [PATCH] gdsys: Drop print_fpga_state function

2016-06-02 Thread Reinhard Pfau
Hi Tom, I think, Dirk had missed this message. Sorry for a late reply. Inlining the func is basically OK. But could you please make a little change at the inlined code (see below)? Beside this: Acked-by: Reinhard Pfau Greetings, Reinhard. On 2016-01-14 00:12, Tom Rini wrote: [...] diff

[U-Boot] [PATCH] gdsys: Drop print_fpga_state function

2016-01-13 Thread Tom Rini
On most platforms the print_fpga_state function is never called. Only on dlvision-10g do we, so in that case inline it. Drop it from everywhere else to avoid extra strings. Cc: Dirk Eibach Signed-off-by: Tom Rini --- board/gdsys/405ep/405ep.c|8 board/gdsys/405ep/dlvision