Skip to contentSkip to conversation

fpga: Updates for 4.10-rc2

GIT PULL Alan Tull 2 participants 5 messages Jan 4, 2017 – Jan 5, 2017

lore
Share & tools
lore

Alan Tull

author
#0 lore raw

Hi Greg,

Please pull these changes for FPGA.

Thanks!
Alan

The following changes since commit e3d31bda06e43968cd215ae590eb7cda827f01e9:

Add linux-next specific files for 20161224 (2017-01-04 10:26:49 -0600)

are available in the git repository at:

git://git.kernel.org/pub/scm/linux/kernel/git/atull/linux-fpga.git tags/fpga-for-greg-20170104

for you to fetch changes up to 2dd088da8cce745c008fc7f8b64e1aef33eb37c2:

ARM: ep93xx: Register ts73xx-fpga manager driver for TS-7300 (2017-01-04 10:27:26 -0600)

----------------------------------------------------------------
fpga: Updates for 4.10-rc2

* Add scatterlist based fpga programming
* TS-7300 FPGA manager
* zynq: Check for errors after completing DMA
* fix sparse warnings in fpga-mgr and fpga-bridge

----------------------------------------------------------------
Dinh Nguyen (1):
fpga: fix sparse warnings in fpga-mgr and fpga-bridge

Florian Fainelli (2):
FPGA: Add TS-7300 FPGA manager
ARM: ep93xx: Register ts73xx-fpga manager driver for TS-7300

Jason Gunthorpe (3):
fpga zynq: Check for errors after completing DMA
fpga: Add scatterlist based programming fpga zynq: Use the scatterlist interface

 Documentation/fpga/fpga-mgr.txt |  15 ++-
 arch/arm/mach-ep93xx/ts72xx.c   |  26 +++++
 drivers/fpga/Kconfig            |   7 ++
 drivers/fpga/Makefile           |   1 +
 drivers/fpga/fpga-bridge.c      |   2 +-
 drivers/fpga/fpga-mgr.c         | 234 ++++++++++++++++++++++++++++++++++------
 drivers/fpga/ts73xx-fpga.c      | 163 ++++++++++++++++++++++++++++
 drivers/fpga/zynq-fpga.c        | 208 ++++++++++++++++++++++++++---------
 include/linux/fpga/fpga-mgr.h   |   5 +
 9 files changed, 573 insertions(+), 88 deletions(-)
 create mode 100644 drivers/fpga/ts73xx-fpga.c
#1 lore raw
Quoted 19 lines · Alan Tull

Hi Greg,

Please pull these changes for FPGA.

Thanks!
Alan

The following changes since commit e3d31bda06e43968cd215ae590eb7cda827f01e9:

Add linux-next specific files for 20161224 (2017-01-04 10:26:49 -0600)

are available in the git repository at:

git://git.kernel.org/pub/scm/linux/kernel/git/atull/linux-fpga.git tags/fpga-for-greg-20170104

for you to fetch changes up to 2dd088da8cce745c008fc7f8b64e1aef33eb37c2:

ARM: ep93xx: Register ts73xx-fpga manager driver for TS-7300 (2017-01-04 10:27:26 -0600)

Quoted 7 lines

----------------------------------------------------------------
fpga: Updates for 4.10-rc2

* Add scatterlist based fpga programming
* TS-7300 FPGA manager
* zynq: Check for errors after completing DMA
* fix sparse warnings in fpga-mgr and fpga-bridge

These are all bugfixes or regression fixes? Doesn't seem like adding new functionality and a new driver fits that category to me, why add them now?

Sorry, I can't take this, if you resend them as patches, I can be more specific...

thanks,

greg k-h

author
#2 lore raw
Quoted 32 lines · Greg Kroah-Hartman
Quoted 19 lines · Alan Tull

Hi Greg,

Please pull these changes for FPGA.

Thanks!
Alan

The following changes since commit e3d31bda06e43968cd215ae590eb7cda827f01e9:

Add linux-next specific files for 20161224 (2017-01-04 10:26:49 -0600)

are available in the git repository at:

git://git.kernel.org/pub/scm/linux/kernel/git/atull/linux-fpga.git tags/fpga-for-greg-20170104

for you to fetch changes up to 2dd088da8cce745c008fc7f8b64e1aef33eb37c2:

ARM: ep93xx: Register ts73xx-fpga manager driver for TS-7300 (2017-01-04 10:27:26 -0600)

Quoted 7 lines

----------------------------------------------------------------
fpga: Updates for 4.10-rc2

* Add scatterlist based fpga programming
* TS-7300 FPGA manager
* zynq: Check for errors after completing DMA
* fix sparse warnings in fpga-mgr and fpga-bridge

These are all bugfixes or regression fixes? Doesn't seem like adding new functionality and a new driver fits that category to me, why add them now?

Quoted 6 lines

Sorry, I can't take this, if you resend them as patches, I can be more specific...

thanks,

greg k-h

Hi Greg,

Yes, sorry, I'm still learning here. One patch is a fix (sparse errors), the rest are new functionality.

Would it be appropriate to separate these and send you two pull requests - the sparse error fix for 4.10 and the rest (new functionality) for 4.11?

Alan

#3 lore raw
Quoted 37 lines · Alan Tull
Quoted 35 lines · Greg Kroah-Hartman
Quoted 19 lines · Alan Tull

Hi Greg,

Please pull these changes for FPGA.

Thanks!
Alan

The following changes since commit e3d31bda06e43968cd215ae590eb7cda827f01e9:

Add linux-next specific files for 20161224 (2017-01-04 10:26:49 -0600)

are available in the git repository at:

git://git.kernel.org/pub/scm/linux/kernel/git/atull/linux-fpga.git tags/fpga-for-greg-20170104

for you to fetch changes up to 2dd088da8cce745c008fc7f8b64e1aef33eb37c2:

ARM: ep93xx: Register ts73xx-fpga manager driver for TS-7300 (2017-01-04 10:27:26 -0600)

Quoted 7 lines

----------------------------------------------------------------
fpga: Updates for 4.10-rc2

* Add scatterlist based fpga programming
* TS-7300 FPGA manager
* zynq: Check for errors after completing DMA
* fix sparse warnings in fpga-mgr and fpga-bridge

These are all bugfixes or regression fixes? Doesn't seem like adding new functionality and a new driver fits that category to me, why add them now?

Sorry, I can't take this, if you resend them as patches, I can be more specific...

Quoted 8 lines
Quoted 3 lines

thanks,

greg k-h

Hi Greg,

Yes, sorry, I'm still learning here. One patch is a fix (sparse errors), the rest are new functionality.

Ok, let's stick to patches then, no git pull requests, it makes things easier that way for things to be reviewed properly.

Quoted 3 lines

Would it be appropriate to separate these and send you two pull requests - the sparse error fix for 4.10 and the rest (new functionality) for 4.11?

why would a sparse warning fix be ok for a -rc kernel? Is it a real bug?

Send patches, we can go from there please.

thanks,

greg k-h

author
#4 lore raw
Quoted 42 lines · Greg Kroah-Hartman
Quoted 41 lines · Alan Tull
Quoted 38 lines · Greg Kroah-Hartman
Quoted 19 lines · Alan Tull

Hi Greg,

Please pull these changes for FPGA.

Thanks!
Alan

The following changes since commit e3d31bda06e43968cd215ae590eb7cda827f01e9:

Add linux-next specific files for 20161224 (2017-01-04 10:26:49 -0600)

are available in the git repository at:

git://git.kernel.org/pub/scm/linux/kernel/git/atull/linux-fpga.git tags/fpga-for-greg-20170104

for you to fetch changes up to 2dd088da8cce745c008fc7f8b64e1aef33eb37c2:

ARM: ep93xx: Register ts73xx-fpga manager driver for TS-7300 (2017-01-04 10:27:26 -0600)

Quoted 7 lines

----------------------------------------------------------------
fpga: Updates for 4.10-rc2

* Add scatterlist based fpga programming
* TS-7300 FPGA manager
* zynq: Check for errors after completing DMA
* fix sparse warnings in fpga-mgr and fpga-bridge

These are all bugfixes or regression fixes? Doesn't seem like adding new functionality and a new driver fits that category to me, why add them now?

Sorry, I can't take this, if you resend them as patches, I can be more specific...

thanks,

greg k-h

Quoted 7 lines
Quoted 4 lines

Hi Greg,

Yes, sorry, I'm still learning here. One patch is a fix (sparse errors), the rest are new functionality.

Ok, let's stick to patches then, no git pull requests, it makes things easier that way for things to be reviewed properly.

That's cool.

All these patches were posted and reviewed on lkml and some on linux-fpga list as well. I'm not trying to skip the mailing list.

But I should have submitted them for the next release. Unfortunately, I had received some confusing advice about doing pull requests...

Quoted 7 lines
Quoted 4 lines

Would it be appropriate to separate these and send you two pull requests - the sparse error fix for 4.10 and the rest (new functionality) for 4.11?

why would a sparse warning fix be ok for a -rc kernel? Is it a real bug?

It's better categorized as cleanup rather than a bug, so not needed for a rc.

Quoted 2 lines

Send patches, we can go from there please.

OK will do. Thanks for your help and patience!

Alan

Quoted 4 lines

thanks,

greg k-h