linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [GIT PULL] fpga: Updates for 4.10-rc2
@ 2017-01-04 20:00 Alan Tull
  2017-01-04 21:28 ` Greg Kroah-Hartman
  0 siblings, 1 reply; 5+ messages in thread
From: Alan Tull @ 2017-01-04 20:00 UTC (permalink / raw)
  To: Greg Kroah-Hartman
  Cc: Moritz Fischer, linux-fpga, linux-kernel, atull,
	delicious.quinoa, dinguyen

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [GIT PULL] fpga: Updates for 4.10-rc2
  2017-01-04 20:00 [GIT PULL] fpga: Updates for 4.10-rc2 Alan Tull
@ 2017-01-04 21:28 ` Greg Kroah-Hartman
  2017-01-04 21:53   ` Alan Tull
  0 siblings, 1 reply; 5+ messages in thread
From: Greg Kroah-Hartman @ 2017-01-04 21:28 UTC (permalink / raw)
  To: Alan Tull
  Cc: Moritz Fischer, linux-fpga, linux-kernel, atull,
	delicious.quinoa, dinguyen

On Wed, Jan 04, 2017 at 02:00:23PM -0600, Alan Tull wrote:
> 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

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [GIT PULL] fpga: Updates for 4.10-rc2
  2017-01-04 21:28 ` Greg Kroah-Hartman
@ 2017-01-04 21:53   ` Alan Tull
  2017-01-05  7:51     ` Greg Kroah-Hartman
  0 siblings, 1 reply; 5+ messages in thread
From: Alan Tull @ 2017-01-04 21:53 UTC (permalink / raw)
  To: Greg Kroah-Hartman
  Cc: Moritz Fischer, linux-fpga, linux-kernel, atull, dinguyen

On Wed, Jan 4, 2017 at 3:28 PM, Greg Kroah-Hartman
<gregkh@linuxfoundation.org> wrote:
> On Wed, Jan 04, 2017 at 02:00:23PM -0600, Alan Tull wrote:
>> 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
>
> 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

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [GIT PULL] fpga: Updates for 4.10-rc2
  2017-01-04 21:53   ` Alan Tull
@ 2017-01-05  7:51     ` Greg Kroah-Hartman
  2017-01-05 20:49       ` Alan Tull
  0 siblings, 1 reply; 5+ messages in thread
From: Greg Kroah-Hartman @ 2017-01-05  7:51 UTC (permalink / raw)
  To: Alan Tull; +Cc: Moritz Fischer, linux-fpga, linux-kernel, atull, dinguyen

On Wed, Jan 04, 2017 at 03:53:18PM -0600, Alan Tull wrote:
> On Wed, Jan 4, 2017 at 3:28 PM, Greg Kroah-Hartman
> <gregkh@linuxfoundation.org> wrote:
> > On Wed, Jan 04, 2017 at 02:00:23PM -0600, Alan Tull wrote:
> >> 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
> >
> > 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
> 
> 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.

> 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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [GIT PULL] fpga: Updates for 4.10-rc2
  2017-01-05  7:51     ` Greg Kroah-Hartman
@ 2017-01-05 20:49       ` Alan Tull
  0 siblings, 0 replies; 5+ messages in thread
From: Alan Tull @ 2017-01-05 20:49 UTC (permalink / raw)
  To: Greg Kroah-Hartman
  Cc: Moritz Fischer, linux-fpga, linux-kernel, atull, Dinh Nguyen

On Thu, Jan 5, 2017 at 1:51 AM, Greg Kroah-Hartman
<gregkh@linuxfoundation.org> wrote:
> On Wed, Jan 04, 2017 at 03:53:18PM -0600, Alan Tull wrote:
>> On Wed, Jan 4, 2017 at 3:28 PM, Greg Kroah-Hartman
>> <gregkh@linuxfoundation.org> wrote:
>> > On Wed, Jan 04, 2017 at 02:00:23PM -0600, Alan Tull wrote:
>> >> 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
>> >
>> > 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
>>
>> 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...

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

>
> Send patches, we can go from there please.

OK will do.  Thanks for your help and patience!

Alan

>
> thanks,
>
> greg k-h

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2017-01-05 20:49 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-04 20:00 [GIT PULL] fpga: Updates for 4.10-rc2 Alan Tull
2017-01-04 21:28 ` Greg Kroah-Hartman
2017-01-04 21:53   ` Alan Tull
2017-01-05  7:51     ` Greg Kroah-Hartman
2017-01-05 20:49       ` Alan Tull

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