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

Tom Rini trini at konsulko.com
Thu Jun 2 21:25:42 CEST 2016


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 <reinhard.pfau at gdsys.cc>:
> ...
> >> 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/405ep/dlvision-10g.c
> >> @@ -109,7 +109,10 @@ static void print_fpga_info(unsigned dev)
> >>             && !((hardware_version == HWVER_101)
> >>                  && (fpga_state == FPGA_STATE_DONE_FAILED))) {
> >>                 puts("not available\n");
> >> -               print_fpga_state(dev);
> >> +               if (gd->arch.fpga_state[dev] & FPGA_STATE_DONE_FAILED)
> >> +                       puts("       Waiting for FPGA-DONE timed out.\n");
> >> +               if (gd->arch.fpga_state[dev] &
> >> FPGA_STATE_REFLECTION_FAILED)
> >> +                       puts("       FPGA reflection test failed.\n");
> >
> >
> > At this point the FPGA state is already available in the local var
> > "fpga_state".
> > So please replace "gd->arch.fpga_state[dev]" with just "fpga_state".
> >
> >>                 return;
> >>         }
> >>
> > [...]
> >
> 
> Apart from that
> Acked-by: Dirk Eibach <dirk.eibach at gdsys.cc>

Will do, thanks guys!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160602/1a4fd503/attachment.sig>


More information about the U-Boot mailing list