All of lore.kernel.org
 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: 25+ 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   ` [1/3] " Anatolij Gustschin
2018-11-05 16:19 ` [PATCH 2/3] spi: add FTDI MPSSE SPI controller driver Anatolij Gustschin
2018-11-05 16:19   ` [2/3] " Anatolij Gustschin
2018-11-06  5:07   ` [PATCH 2/3] " kbuild test robot
2018-11-06  5:07     ` [2/3] " kbuild test robot
2018-11-06  9:58     ` [PATCH 2/3] " Anatolij Gustschin
2018-11-06  9:58       ` [2/3] " Anatolij Gustschin
2018-11-06  6:29   ` [PATCH 2/3] " kbuild test robot
2018-11-06  6:29     ` [2/3] " kbuild test robot
2018-11-09  7:53   ` [PATCH v2 2/3] " Anatolij Gustschin
2018-11-09  7:53     ` [v2,2/3] " Anatolij Gustschin
2018-11-09 15:30     ` [PATCH v2 2/3] " Mark Brown
2018-11-09 15:30       ` [v2,2/3] " Mark Brown
2018-11-10 10:39       ` [PATCH v2 2/3] " Anatolij Gustschin
2018-11-10 10:39         ` [v2,2/3] " Anatolij Gustschin
2018-11-13 18:11         ` [PATCH v2 2/3] " Mark Brown
2018-11-13 18:11           ` [v2,2/3] " Mark Brown
2018-11-13 18:56           ` [PATCH v2 2/3] " Anatolij Gustschin
2018-11-13 18:56             ` [v2,2/3] " Anatolij Gustschin
2018-11-05 16:19 ` [PATCH 3/3] fpga: Add fpga manager driver for ARRI Altera FPP Anatolij Gustschin
2018-11-05 16:19   ` [3/3] " 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.