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
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
> >>
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
>> ---
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,
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
5 matches
Mail list logo