Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1582984

Re: [PATCH 2/4] fpga pr ip: Core driver support for Altera Partial Reconfiguration IP.

Show all headers | View raw

Hi Moritz,

Thanks for the feedback.

On Thu, 16 Feb 2017, Moritz Fischer wrote:

> Hi Matthew,
>
> On Wed, Feb 15, 2017 at 1:10 PM,  <matthew.gerlach@linux.intel.com> wrote:
>
>> +static int alt_pr_fpga_write_complete(struct fpga_manager *mgr,
>> +                                     struct fpga_image_info *info)
>> +{
>> +       u32 i;
>> +
>> +       for (i = 0; i < info->config_complete_timeout_us; i++) {
>> +               switch (alt_pr_fpga_state(mgr)) {
>> +               case FPGA_MGR_STATE_WRITE_ERR:
>> +                       return -EIO;
>> +
>> +               case FPGA_MGR_STATE_OPERATING:
>> +                       dev_info(&mgr->dev,
>> +                                "successful partial reconfiguration\n");
>> +                       return 0;
>> +
>> +               default:
>> +                       break;
>> +               }
>> +               udelay(1);
>
> Does this need to be a udelay? would a usleep_range() do maybe?

The actual timeout required is design specific.  The member,
config_complete_timeout_us, is defined in microseconds, and my experience 
is that a small number of microseconds (e.g. < 10) is usually plenty. 
Other FPGAs and designs might be different.  My quick reading of kernel 
timers says usleep_range() is good for 10 us - 20 ms.  In this driver, if 
usleep_range(1) can put less pressure on the CPU and schedular at the cost 
of little accuracy, I will be happy to switch, but at this time I don't 
know if usleep_range() would be better or not.


>
> Could we maybe pull the timeout part into the framework if all drivers are doing
> is to wait / poll for the state to be a certain value?
>

I think it is safe to say that all variations of FPGAs need some amount of 
time after the bitstream to be delivered before the FPGA is "ready".  If 
that time is exceeded then the fpga programming has failed.  If all FPGA 
variations are doing a poll for some amount of time, then it would be good 
to move the polling up to the framework.  I think such a change would be 
better in its own patch set.

> Thanks,
>
> Moritz
> --
> To unsubscribe from this list: send the line "unsubscribe linux-fpga" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 0/4] Altera Partial Reconfiguration IP matthew.gerlach@linux.intel.com - 2017-02-15 22:20 +0100
  [PATCH 1/4] fpga: add config complete timeout matthew.gerlach@linux.intel.com - 2017-02-15 22:20 +0100
  [PATCH 2/4] fpga pr ip: Core driver support for Altera Partial Reconfiguration IP. matthew.gerlach@linux.intel.com - 2017-02-15 22:20 +0100
    Re: [PATCH 2/4] fpga pr ip: Core driver support for Altera Partial  Reconfiguration IP. Moritz Fischer <moritz.fischer@ettus.com> - 2017-02-16 18:40 +0100
      Re: [PATCH 2/4] fpga pr ip: Core driver support for Altera Partial  Reconfiguration IP. matthew.gerlach@linux.intel.com - 2017-02-16 23:50 +0100
  [PATCH 3/4] fpga dt: bindings for Altera Partial Reconfiguraion IP. matthew.gerlach@linux.intel.com - 2017-02-15 22:20 +0100
    Re: [PATCH 3/4] fpga dt: bindings for Altera Partial Reconfiguraion IP. Dinh Nguyen <dinh.linux@gmail.com> - 2017-02-17 14:40 +0100
      Re: [PATCH 3/4] fpga dt: bindings for Altera Partial Reconfiguraion IP. Moritz Fischer <mdf@kernel.org> - 2017-02-17 16:30 +0100
    Re: [PATCH 3/4] fpga dt: bindings for Altera Partial Reconfiguraion  IP. Rob Herring <robh@kernel.org> - 2017-02-27 15:40 +0100
      Re: [PATCH 3/4] fpga dt: bindings for Altera Partial Reconfiguraion  IP. matthew.gerlach@linux.intel.com - 2017-02-27 17:40 +0100

csiph-web