Re: [PATCH] reset: socfpga: use arch_initcall for early initialization

[atull]

On Thu, 9 Oct 2014, Steffen Trumtrar wrote:

Hi Steffen,

> Hi!
> 
> On Thu, Oct 09, 2014 at 09:57:49AM -0500, atull wrote:
> > On Thu, 9 Oct 2014, Steffen Trumtrar wrote:
> > 
> > > On Thu, Oct 09, 2014 at 08:16:18AM -0500, Dinh Nguyen wrote:
> > > > Hi Philipp,
> > > > 
> > > > On 10/9/14, 4:03 AM, Philipp Zabel wrote:
> > > > > Am Mittwoch, den 08.10.2014, 21:44 -0500 schrieb
> > > > > dinguyen@xxxxxxxxxxxxxxxxxxxxx:
> > > > >> From: Dinh Nguyen <dinguyen@xxxxxxxxxxxxxxxxxxxxx>
> > > > >>
> > > > >> There are certain drivers that are required to get loaded very early using
> > > > >> arch_initcall. An example of such a driver is the SOCFPGA's FPGA bridge driver.
> > > > >> This driver has to get loaded early because it needs to enable FPGA components
> > > > >> that are connected to the bridge.
> > > > >>
> > > > >> This FPGA bridge driver will using the reset controller API to toggle it's
> > > > >> reset bits, thus, it needs the reset driver to be loaded as early as possible
> > > > >> in order for it to get used properly.
> > > > > 
> > > > > Without knowing the details, this sounds like the wrong approach. Can't
> > > > > the bridge driver return -EPROBE_DEFER until the reset controller is
> > > > > available?
> > > > > 
> > 
> > I don't think we can do deferred probing for arch_initcall.
> > 
> > > > 
> > > > The bridge driver is also using arch_initcall, as it also needs to get
> > > > loaded early for FPGA IPs to work, and so later driver loading will work
> > > > for the FPGA IPs.
> > > > 
> > > 
> > > For the bridge driver the same is true. I guess that there *might* be
> > > IP cores where you need to be very early, but that shouldn't be the normal
> > > case. If the driver can't get loaded properly, the right thing would be to
> > > fix the driver.
> > 
> > Some fpga ip drivers will need to be early (and will assume that the fpga
> > was programmed by the bootloader).  We want to support that case.
> >
> 
> Do you have an example where this is really needed?

My last version of the fpga manager framework 
(https://lkml.org/lkml/2014/8/1/518)
added fpga_mgr_firmware_write().  This can be called from a device driver's 
probe function to request a fpga image be loaded.  I want to support FPGA 
based functionality being seen pretty similar to really hard hardware.  So 
the FPGA could have a PCI bus or something else that would want to be 
early.

> 
> > > 
> > > I have developed a bridge driver, too (which only needs the devicetree binding
> > > docu for a v1) and I have a driver+IP core that is directly connected to the
> > > bridge. I don't need any messing around with the initcalls to work properly.
> > > -EPROBE_DEFER works just fine.
> > 
> > Is this based on the "proposed fpga bridge framework" that I posted?
> > What we are trying to do is update that bridge driver to use the reset driver for 
> > submitting v2.
> 
> Are you talking about the FPGA manager patches? Then: no.
> If not, I seem to have missed the patches.
> 
> The driver just probes and then releases the resets for the according bridge.
> It shall however support the GPV configuration also (which it doesn't at the moment)
> 
> But this reminds me, why I didn't send the driver earlier. IIRC you want to reset
> the bridges after writing the bitstream to the FPGA manager.
> So, I would need to add that first so you can access that in your fpga manager
> driver.

We need to hold the bridges in reset as long as the FPGA is not programmed
or is being programmed.  We want to release the bridges from reset after
programming.  If we are going to re-program the FPGA, we will need to
unload any device drivers, set the bridges into reset, then reprogram
the FPGA, release the bridges, load the appropriate device drivers.
The latest fpga manager code also handles reprogramming the FPGA when
device tree fragments are loaded.

Alan

> 
> Regards,
> Steffen
> 
> -- 
> Pengutronix e.K.                           |                             |
> Industrial Linux Solutions                 | http://www.pengutronix.de/  |
> Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
> Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/