* 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
only message in thread, other threads:[~2017-02-20 20:05 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1487600724-17607-1-git-send-email-agust@denx.de>
[not found] ` <1487600724-17607-3-git-send-email-agust@denx.de>
[not found] ` <CAAtXAHeO_+6ZXP51nxFgyLYR9cjqLrqmoYEniqXmZudF=3uhFA@mail.gmail.com>
2017-02-20 20:05 ` [PATCH v2 2/2] fpga manager: Add Xilinx slave serial SPI driver Anatolij Gustschin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).