* Re: [PATCH v2 2/2] fpga manager: Add Xilinx slave serial SPI driver
[not found] ` <CAAtXAHeO_+6ZXP51nxFgyLYR9cjqLrqmoYEniqXmZudF=3uhFA@mail.gmail.com>
@ 2017-02-20 20:05 ` Anatolij Gustschin
0 siblings, 0 replies; only message in thread
From: Anatolij Gustschin @ 2017-02-20 20:05 UTC (permalink / raw)
To: Moritz Fischer
Cc: linux-fpga, Devicetree List, Alan Tull, Rob Herring, Mark Rutland
Hi Moritz,
On Mon, 20 Feb 2017 09:27:35 -0800
Moritz Fischer mdf@kernel.org wrote:
>Anatolij,
>
>small nits inline.
...
>> + gpiod_set_value(conf->prog_b, 1);
>> +
>> + udelay(1); /* min is 500 ns */
>
>Maybe make this a named constant.
Okay, I can add
const size_t prog_pulse_us = 1;
...
udelay(prog_pulse_us);
...
>> + gpiod_set_value(conf->prog_b, 0);
>> +
>> + if (gpiod_get_value(conf->done)) {
>> + dev_err(&mgr->dev, "Unexpected DONE pin state...\n");
>> + return -EIO;
>> + }
>> +
>> + /* program latency */
>> + usleep_range(1000, 1500);
>
>Same here.
Okay, will change here as well.
Thanks for review! I'll wait some time for more comments and
will submit v3 then.
Anatolij
^ permalink raw reply [flat|nested] only message in thread