linux-fpga.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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

      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).