From: Anatolij Gustschin <agust@denx.de>
To: Alan Tull <atull@kernel.org>
Cc: USB list <linux-usb@vger.kernel.org>,
linux-spi <linux-spi@vger.kernel.org>,
linux-fpga@vger.kernel.org,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Mark Brown <broonie@kernel.org>, Moritz Fischer <mdf@kernel.org>
Subject: Re: [PATCH 0/3] Add support for ARRI FPGA configuration
Date: Mon, 5 Nov 2018 21:29:45 +0100 [thread overview]
Message-ID: <20181105212945.7625cfd3@crub> (raw)
In-Reply-To: <CANk1AXQCTc39hgn7FszzhqHVFRCeMwqddC1FmHe25i2QYffrcw@mail.gmail.com>
Hi Alan,
On Mon, 5 Nov 2018 12:52:03 -0600
Alan Tull atull@kernel.org wrote:
>On Mon, Nov 5, 2018 at 10:19 AM Anatolij Gustschin <agust@denx.de> wrote:
>
>Hi Anatolij,
>
>>
>> This series adds drivers and FPGA manager support required
>> for FT232H based ARRI FPGA configuration adapters.
>
>Is this a repost or a new version of this patchset? If it's a repost,
>you could note that. If there are changes since the previous posting,
>version info helps reviewers know what's new.
This is a new version. Or it is very different compared to my first
attempt to add driver support for this adapter hardware. The initial
series ([1],[2],[3]) didn't include MPSSE SPI driver, it didn't use
vendor specific USB PIDs for FTDI device and had fundamental design
issues. This new version implements the new driver design as summarized
here [4].
Changes in this new version:
- don't use MFD framework
- don't use original FT232H USB PID (it is too generic and collides
with existing ftdi_sio driver)
- don't add separate CBUS GPIO driver based on FT232H MFD device
- redesign/rework the drivers as described under [4]
- add basic FT232H interface driver under drivers/usb/misc/
- use custom reserved USB PIDs in the USB driver for adapter devices
- add notes about borrowed protocol code from libftdi to commit log
and header file
- add MPSSE SPI controller driver supporting USB-SPI bus with
dynamically added SPI slaves from description in platform data
(the initial attempt didn't include USB SPI bus support for PS-
SPI FPGA configuration via USB)
- rework FPP fpga manager driver based on new FT232H interface
driver and extend it according to CPLD changes for additional
support of new hardware revision B.
[1] https://patchwork.kernel.org/patch/9828985/
[2] https://patchwork.kernel.org/patch/9828981/
[3] https://patchwork.kernel.org/patch/9828983/
[4] https://www.spinics.net/lists/linux-usb/msg163441.html
Thanks,
Anatolij
prev parent reply other threads:[~2018-11-06 5:51 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-05 16:19 [PATCH 0/3] Add support for ARRI FPGA configuration Anatolij Gustschin
2018-11-05 16:19 ` [PATCH 1/3] usb: misc: add driver for FT232H based FPGA configuration devices Anatolij Gustschin
2018-11-05 16:19 ` [PATCH 2/3] spi: add FTDI MPSSE SPI controller driver Anatolij Gustschin
2018-11-06 5:07 ` kbuild test robot
2018-11-06 9:58 ` Anatolij Gustschin
2018-11-06 6:29 ` kbuild test robot
2018-11-09 7:53 ` [PATCH v2 " Anatolij Gustschin
2018-11-09 15:30 ` Mark Brown
2018-11-10 10:39 ` Anatolij Gustschin
2018-11-13 18:11 ` Mark Brown
2018-11-13 18:56 ` Anatolij Gustschin
2018-11-05 16:19 ` [PATCH 3/3] fpga: Add fpga manager driver for ARRI Altera FPP Anatolij Gustschin
2018-11-05 18:52 ` [PATCH 0/3] Add support for ARRI FPGA configuration Alan Tull
2018-11-05 20:29 ` Anatolij Gustschin [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20181105212945.7625cfd3@crub \
--to=agust@denx.de \
--cc=atull@kernel.org \
--cc=broonie@kernel.org \
--cc=gregkh@linuxfoundation.org \
--cc=linux-fpga@vger.kernel.org \
--cc=linux-spi@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=mdf@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).