linux-fpga.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [GIT PULL] FPGA Manager changes for 5.15-rc1
@ 2021-07-26  2:55 Moritz Fischer
  2021-07-26  3:00 ` Moritz Fischer
  0 siblings, 1 reply; 8+ messages in thread
From: Moritz Fischer @ 2021-07-26  2:55 UTC (permalink / raw)
  To: gregkh; +Cc: linux-fpga

The following changes since commit e73f0f0ee7541171d89f2e2491130c7771ba58d3:

  Linux 5.14-rc1 (2021-07-11 15:07:40 -0700)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/mdf/linux-fpga tags/fpga-for-5.15

for you to fetch changes up to c485d3bf3cc7790faed2b90c799a38caa2f69268:

  fpga: fpga-mgr: wrap the write_sg() op (2021-07-18 08:05:00 -0700)

----------------------------------------------------------------
FPGA Manager Changes for 5.15-rc1

FPGA Manager

- Navin's change removes a duplicate word in a comment
- Tom's change fixes a spelling mistake
- Mauro's change fixes up documentation
- Tom's second set adds wrappers to allow drivers not having to
  implement empty functions by moving checks into fpga-mgr core code
- My changes address a bunch of warnings

DFL

- Martin's change adds a new PCI ID for Silicom N501x PAC cards

All patches have been reviewed on the mailing list, and have been in the
last linux-next releases (as part of my for-next branch).

I did get a complaint about one of the commit messages w/ a Fixes: tag
which would need a rebase to fix.

Some of the earlier patches were originally meant for 5.14 but missed
the merge window by a couple of days, hence the back-merge of 5.14-rc1.

Signed-offy-by: Moritz Fischer <mdf@kernel.org>

----------------------------------------------------------------
Martin Hundebøll (1):
      fpga: dfl: pci: add device IDs for Silicom N501x PAC cards

Mauro Carvalho Chehab (1):
      docs: driver-api: fpga: avoid using UTF-8 chars

Moritz Fischer (5):
      Merge tag 'v5.14-rc1' into for-next
      fpga: altera-freeze-bridge: Address warning about unused variable
      fpga: xiilnx-spi: Address warning about unused variable
      fpga: xilinx-pr-decoupler: Address warning about unused variable
      fpga: zynqmp-fpga: Address warning about unused variable

Navin Sankar Velliangiri (1):
      fpga: fpga-bridge: removed repeated word

Tom Rix (8):
      fpga: fix spelling mistakes
      fpga: fpga-mgr: wrap the write_init() op
      fpga: fpga-mgr: make write_complete() op optional
      fpga: fpga-mgr: wrap the write() op
      fpga: fpga-mgr: wrap the status() op
      fpga: fpga-mgr: wrap the state() op
      fpga: fpga-mgr: wrap the fpga_remove() op
      fpga: fpga-mgr: wrap the write_sg() op

 Documentation/driver-api/fpga/fpga-bridge.rst      |  10 +-
 Documentation/driver-api/fpga/fpga-mgr.rst         |  12 +--
 Documentation/driver-api/fpga/fpga-programming.rst |   8 +-
 Documentation/driver-api/fpga/fpga-region.rst      |  20 ++--
 Documentation/fpga/dfl.rst                         |   4 +-
 drivers/fpga/altera-cvp.c                          |   2 +-
 drivers/fpga/altera-freeze-bridge.c                |   2 +
 drivers/fpga/dfl-fme-mgr.c                         |   6 --
 drivers/fpga/dfl-fme-pr.c                          |   2 +-
 drivers/fpga/dfl-n3000-nios.c                      |   2 +-
 drivers/fpga/dfl-pci.c                             |   5 +
 drivers/fpga/dfl.h                                 |   2 +-
 drivers/fpga/fpga-bridge.c                         |   8 +-
 drivers/fpga/fpga-mgr.c                            | 111 ++++++++++++++-------
 drivers/fpga/stratix10-soc.c                       |   6 --
 drivers/fpga/ts73xx-fpga.c                         |   6 --
 drivers/fpga/xilinx-pr-decoupler.c                 |   2 +
 drivers/fpga/xilinx-spi.c                          |   2 +
 drivers/fpga/zynq-fpga.c                           |   6 +-
 drivers/fpga/zynqmp-fpga.c                         |  10 +-
 include/linux/fpga/fpga-mgr.h                      |   2 +-
 21 files changed, 127 insertions(+), 101 deletions(-)

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

* Re: [GIT PULL] FPGA Manager changes for 5.15-rc1
  2021-07-26  2:55 [GIT PULL] FPGA Manager changes for 5.15-rc1 Moritz Fischer
@ 2021-07-26  3:00 ` Moritz Fischer
  2021-07-27 14:39   ` Greg KH
  0 siblings, 1 reply; 8+ messages in thread
From: Moritz Fischer @ 2021-07-26  3:00 UTC (permalink / raw)
  To: Moritz Fischer; +Cc: gregkh, linux-fpga

Hi Greg,

On Sun, Jul 25, 2021 at 07:55:26PM -0700, Moritz Fischer wrote:
> The following changes since commit e73f0f0ee7541171d89f2e2491130c7771ba58d3:
> 
>   Linux 5.14-rc1 (2021-07-11 15:07:40 -0700)
> 
> are available in the Git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/mdf/linux-fpga tags/fpga-for-5.15
> 
> for you to fetch changes up to c485d3bf3cc7790faed2b90c799a38caa2f69268:
> 
>   fpga: fpga-mgr: wrap the write_sg() op (2021-07-18 08:05:00 -0700)
> 
> ----------------------------------------------------------------
> FPGA Manager Changes for 5.15-rc1
> 
> FPGA Manager
> 
> - Navin's change removes a duplicate word in a comment
> - Tom's change fixes a spelling mistake
> - Mauro's change fixes up documentation
> - Tom's second set adds wrappers to allow drivers not having to
>   implement empty functions by moving checks into fpga-mgr core code
> - My changes address a bunch of warnings
> 
> DFL
> 
> - Martin's change adds a new PCI ID for Silicom N501x PAC cards
> 
> All patches have been reviewed on the mailing list, and have been in the
> last linux-next releases (as part of my for-next branch).
> 
> I did get a complaint about one of the commit messages w/ a Fixes: tag
> which would need a rebase to fix.
> 
> Some of the earlier patches were originally meant for 5.14 but missed
> the merge window by a couple of days, hence the back-merge of 5.14-rc1.
> 
> Signed-offy-by: Moritz Fischer <mdf@kernel.org>
> 
> ----------------------------------------------------------------
> Martin Hundebøll (1):
>       fpga: dfl: pci: add device IDs for Silicom N501x PAC cards
> 
> Mauro Carvalho Chehab (1):
>       docs: driver-api: fpga: avoid using UTF-8 chars
> 
> Moritz Fischer (5):
>       Merge tag 'v5.14-rc1' into for-next
>       fpga: altera-freeze-bridge: Address warning about unused variable
>       fpga: xiilnx-spi: Address warning about unused variable
>       fpga: xilinx-pr-decoupler: Address warning about unused variable
>       fpga: zynqmp-fpga: Address warning about unused variable
> 
> Navin Sankar Velliangiri (1):
>       fpga: fpga-bridge: removed repeated word
> 
> Tom Rix (8):
>       fpga: fix spelling mistakes
>       fpga: fpga-mgr: wrap the write_init() op
>       fpga: fpga-mgr: make write_complete() op optional
>       fpga: fpga-mgr: wrap the write() op
>       fpga: fpga-mgr: wrap the status() op
>       fpga: fpga-mgr: wrap the state() op
>       fpga: fpga-mgr: wrap the fpga_remove() op
>       fpga: fpga-mgr: wrap the write_sg() op
> 
>  Documentation/driver-api/fpga/fpga-bridge.rst      |  10 +-
>  Documentation/driver-api/fpga/fpga-mgr.rst         |  12 +--
>  Documentation/driver-api/fpga/fpga-programming.rst |   8 +-
>  Documentation/driver-api/fpga/fpga-region.rst      |  20 ++--
>  Documentation/fpga/dfl.rst                         |   4 +-
>  drivers/fpga/altera-cvp.c                          |   2 +-
>  drivers/fpga/altera-freeze-bridge.c                |   2 +
>  drivers/fpga/dfl-fme-mgr.c                         |   6 --
>  drivers/fpga/dfl-fme-pr.c                          |   2 +-
>  drivers/fpga/dfl-n3000-nios.c                      |   2 +-
>  drivers/fpga/dfl-pci.c                             |   5 +
>  drivers/fpga/dfl.h                                 |   2 +-
>  drivers/fpga/fpga-bridge.c                         |   8 +-
>  drivers/fpga/fpga-mgr.c                            | 111 ++++++++++++++-------
>  drivers/fpga/stratix10-soc.c                       |   6 --
>  drivers/fpga/ts73xx-fpga.c                         |   6 --
>  drivers/fpga/xilinx-pr-decoupler.c                 |   2 +
>  drivers/fpga/xilinx-spi.c                          |   2 +
>  drivers/fpga/zynq-fpga.c                           |   6 +-
>  drivers/fpga/zynqmp-fpga.c                         |  10 +-
>  include/linux/fpga/fpga-mgr.h                      |   2 +-
>  21 files changed, 127 insertions(+), 101 deletions(-)

I realize the backwards merge is somewhat messy, as discussed you said
to not rebase.

Alternatively there's a tag (fpga-for-5.15-early) with the changes
rebased onto v5.14-rc1 that also addresses the 'Fixes: ' tag.

If you pull that instead then I could just merge that tag into my
for-next branch and linux-next should be fine again.

When I localy tried it seemed to resolve fine with merge either of the
branches involved (linux-next/master, char-misc-next, for-next).

- Moritz

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

* Re: [GIT PULL] FPGA Manager changes for 5.15-rc1
  2021-07-26  3:00 ` Moritz Fischer
@ 2021-07-27 14:39   ` Greg KH
  2021-07-27 16:13     ` Moritz Fischer
  0 siblings, 1 reply; 8+ messages in thread
From: Greg KH @ 2021-07-27 14:39 UTC (permalink / raw)
  To: Moritz Fischer; +Cc: linux-fpga

On Sun, Jul 25, 2021 at 08:00:49PM -0700, Moritz Fischer wrote:
> Hi Greg,
> 
> On Sun, Jul 25, 2021 at 07:55:26PM -0700, Moritz Fischer wrote:
> > The following changes since commit e73f0f0ee7541171d89f2e2491130c7771ba58d3:
> > 
> >   Linux 5.14-rc1 (2021-07-11 15:07:40 -0700)
> > 
> > are available in the Git repository at:
> > 
> >   git://git.kernel.org/pub/scm/linux/kernel/git/mdf/linux-fpga tags/fpga-for-5.15
> > 
> > for you to fetch changes up to c485d3bf3cc7790faed2b90c799a38caa2f69268:
> > 
> >   fpga: fpga-mgr: wrap the write_sg() op (2021-07-18 08:05:00 -0700)
> > 
> > ----------------------------------------------------------------
> > FPGA Manager Changes for 5.15-rc1
> > 
> > FPGA Manager
> > 
> > - Navin's change removes a duplicate word in a comment
> > - Tom's change fixes a spelling mistake
> > - Mauro's change fixes up documentation
> > - Tom's second set adds wrappers to allow drivers not having to
> >   implement empty functions by moving checks into fpga-mgr core code
> > - My changes address a bunch of warnings
> > 
> > DFL
> > 
> > - Martin's change adds a new PCI ID for Silicom N501x PAC cards
> > 
> > All patches have been reviewed on the mailing list, and have been in the
> > last linux-next releases (as part of my for-next branch).
> > 
> > I did get a complaint about one of the commit messages w/ a Fixes: tag
> > which would need a rebase to fix.
> > 
> > Some of the earlier patches were originally meant for 5.14 but missed
> > the merge window by a couple of days, hence the back-merge of 5.14-rc1.
> > 
> > Signed-offy-by: Moritz Fischer <mdf@kernel.org>
> > 
> > ----------------------------------------------------------------
> > Martin Hundebøll (1):
> >       fpga: dfl: pci: add device IDs for Silicom N501x PAC cards
> > 
> > Mauro Carvalho Chehab (1):
> >       docs: driver-api: fpga: avoid using UTF-8 chars
> > 
> > Moritz Fischer (5):
> >       Merge tag 'v5.14-rc1' into for-next
> >       fpga: altera-freeze-bridge: Address warning about unused variable
> >       fpga: xiilnx-spi: Address warning about unused variable
> >       fpga: xilinx-pr-decoupler: Address warning about unused variable
> >       fpga: zynqmp-fpga: Address warning about unused variable
> > 
> > Navin Sankar Velliangiri (1):
> >       fpga: fpga-bridge: removed repeated word
> > 
> > Tom Rix (8):
> >       fpga: fix spelling mistakes
> >       fpga: fpga-mgr: wrap the write_init() op
> >       fpga: fpga-mgr: make write_complete() op optional
> >       fpga: fpga-mgr: wrap the write() op
> >       fpga: fpga-mgr: wrap the status() op
> >       fpga: fpga-mgr: wrap the state() op
> >       fpga: fpga-mgr: wrap the fpga_remove() op
> >       fpga: fpga-mgr: wrap the write_sg() op
> > 
> >  Documentation/driver-api/fpga/fpga-bridge.rst      |  10 +-
> >  Documentation/driver-api/fpga/fpga-mgr.rst         |  12 +--
> >  Documentation/driver-api/fpga/fpga-programming.rst |   8 +-
> >  Documentation/driver-api/fpga/fpga-region.rst      |  20 ++--
> >  Documentation/fpga/dfl.rst                         |   4 +-
> >  drivers/fpga/altera-cvp.c                          |   2 +-
> >  drivers/fpga/altera-freeze-bridge.c                |   2 +
> >  drivers/fpga/dfl-fme-mgr.c                         |   6 --
> >  drivers/fpga/dfl-fme-pr.c                          |   2 +-
> >  drivers/fpga/dfl-n3000-nios.c                      |   2 +-
> >  drivers/fpga/dfl-pci.c                             |   5 +
> >  drivers/fpga/dfl.h                                 |   2 +-
> >  drivers/fpga/fpga-bridge.c                         |   8 +-
> >  drivers/fpga/fpga-mgr.c                            | 111 ++++++++++++++-------
> >  drivers/fpga/stratix10-soc.c                       |   6 --
> >  drivers/fpga/ts73xx-fpga.c                         |   6 --
> >  drivers/fpga/xilinx-pr-decoupler.c                 |   2 +
> >  drivers/fpga/xilinx-spi.c                          |   2 +
> >  drivers/fpga/zynq-fpga.c                           |   6 +-
> >  drivers/fpga/zynqmp-fpga.c                         |  10 +-
> >  include/linux/fpga/fpga-mgr.h                      |   2 +-
> >  21 files changed, 127 insertions(+), 101 deletions(-)
> 
> I realize the backwards merge is somewhat messy, as discussed you said
> to not rebase.
> 
> Alternatively there's a tag (fpga-for-5.15-early) with the changes
> rebased onto v5.14-rc1 that also addresses the 'Fixes: ' tag.
> 
> If you pull that instead then I could just merge that tag into my
> for-next branch and linux-next should be fine again.
> 
> When I localy tried it seemed to resolve fine with merge either of the
> branches involved (linux-next/master, char-misc-next, for-next).

I would take this, but I get the following warning (and if I ignore it,
you will get it from linux-next):

Commit: 6c17b7ff1d11 ("fpga: zynqmp-fpga: Address warning about unused variable")
	Fixes tag: Fixes: c09f7471127e ("fpga manager: Adding FPGA Manager support for
	Has these problem(s):
	        - Subject has leading but no trailing parentheses
	        - Subject has leading but no trailing quotes

So can you fix this up?  Might as well just rebase and redo the whole
tree, then you can add the additional patch you send after this to it as
well.

thanks,

greg k-h

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

* Re: [GIT PULL] FPGA Manager changes for 5.15-rc1
  2021-07-27 14:39   ` Greg KH
@ 2021-07-27 16:13     ` Moritz Fischer
  2021-07-27 16:45       ` Greg KH
  0 siblings, 1 reply; 8+ messages in thread
From: Moritz Fischer @ 2021-07-27 16:13 UTC (permalink / raw)
  To: Greg KH; +Cc: Moritz Fischer, linux-fpga

On Tue, Jul 27, 2021 at 04:39:46PM +0200, Greg KH wrote:
> On Sun, Jul 25, 2021 at 08:00:49PM -0700, Moritz Fischer wrote:
> > Hi Greg,
> > 
> > On Sun, Jul 25, 2021 at 07:55:26PM -0700, Moritz Fischer wrote:
> > > The following changes since commit e73f0f0ee7541171d89f2e2491130c7771ba58d3:
> > > 
> > >   Linux 5.14-rc1 (2021-07-11 15:07:40 -0700)
> > > 
> > > are available in the Git repository at:
> > > 
> > >   git://git.kernel.org/pub/scm/linux/kernel/git/mdf/linux-fpga tags/fpga-for-5.15
> > > 
> > > for you to fetch changes up to c485d3bf3cc7790faed2b90c799a38caa2f69268:
> > > 
> > >   fpga: fpga-mgr: wrap the write_sg() op (2021-07-18 08:05:00 -0700)
> > > 
> > > ----------------------------------------------------------------
> > > FPGA Manager Changes for 5.15-rc1
> > > 
> > > FPGA Manager
> > > 
> > > - Navin's change removes a duplicate word in a comment
> > > - Tom's change fixes a spelling mistake
> > > - Mauro's change fixes up documentation
> > > - Tom's second set adds wrappers to allow drivers not having to
> > >   implement empty functions by moving checks into fpga-mgr core code
> > > - My changes address a bunch of warnings
> > > 
> > > DFL
> > > 
> > > - Martin's change adds a new PCI ID for Silicom N501x PAC cards
> > > 
> > > All patches have been reviewed on the mailing list, and have been in the
> > > last linux-next releases (as part of my for-next branch).
> > > 
> > > I did get a complaint about one of the commit messages w/ a Fixes: tag
> > > which would need a rebase to fix.
> > > 
> > > Some of the earlier patches were originally meant for 5.14 but missed
> > > the merge window by a couple of days, hence the back-merge of 5.14-rc1.
> > > 
> > > Signed-offy-by: Moritz Fischer <mdf@kernel.org>
> > > 
> > > ----------------------------------------------------------------
> > > Martin Hundebøll (1):
> > >       fpga: dfl: pci: add device IDs for Silicom N501x PAC cards
> > > 
> > > Mauro Carvalho Chehab (1):
> > >       docs: driver-api: fpga: avoid using UTF-8 chars
> > > 
> > > Moritz Fischer (5):
> > >       Merge tag 'v5.14-rc1' into for-next
> > >       fpga: altera-freeze-bridge: Address warning about unused variable
> > >       fpga: xiilnx-spi: Address warning about unused variable
> > >       fpga: xilinx-pr-decoupler: Address warning about unused variable
> > >       fpga: zynqmp-fpga: Address warning about unused variable
> > > 
> > > Navin Sankar Velliangiri (1):
> > >       fpga: fpga-bridge: removed repeated word
> > > 
> > > Tom Rix (8):
> > >       fpga: fix spelling mistakes
> > >       fpga: fpga-mgr: wrap the write_init() op
> > >       fpga: fpga-mgr: make write_complete() op optional
> > >       fpga: fpga-mgr: wrap the write() op
> > >       fpga: fpga-mgr: wrap the status() op
> > >       fpga: fpga-mgr: wrap the state() op
> > >       fpga: fpga-mgr: wrap the fpga_remove() op
> > >       fpga: fpga-mgr: wrap the write_sg() op
> > > 
> > >  Documentation/driver-api/fpga/fpga-bridge.rst      |  10 +-
> > >  Documentation/driver-api/fpga/fpga-mgr.rst         |  12 +--
> > >  Documentation/driver-api/fpga/fpga-programming.rst |   8 +-
> > >  Documentation/driver-api/fpga/fpga-region.rst      |  20 ++--
> > >  Documentation/fpga/dfl.rst                         |   4 +-
> > >  drivers/fpga/altera-cvp.c                          |   2 +-
> > >  drivers/fpga/altera-freeze-bridge.c                |   2 +
> > >  drivers/fpga/dfl-fme-mgr.c                         |   6 --
> > >  drivers/fpga/dfl-fme-pr.c                          |   2 +-
> > >  drivers/fpga/dfl-n3000-nios.c                      |   2 +-
> > >  drivers/fpga/dfl-pci.c                             |   5 +
> > >  drivers/fpga/dfl.h                                 |   2 +-
> > >  drivers/fpga/fpga-bridge.c                         |   8 +-
> > >  drivers/fpga/fpga-mgr.c                            | 111 ++++++++++++++-------
> > >  drivers/fpga/stratix10-soc.c                       |   6 --
> > >  drivers/fpga/ts73xx-fpga.c                         |   6 --
> > >  drivers/fpga/xilinx-pr-decoupler.c                 |   2 +
> > >  drivers/fpga/xilinx-spi.c                          |   2 +
> > >  drivers/fpga/zynq-fpga.c                           |   6 +-
> > >  drivers/fpga/zynqmp-fpga.c                         |  10 +-
> > >  include/linux/fpga/fpga-mgr.h                      |   2 +-
> > >  21 files changed, 127 insertions(+), 101 deletions(-)
> > 
> > I realize the backwards merge is somewhat messy, as discussed you said
> > to not rebase.
> > 
> > Alternatively there's a tag (fpga-for-5.15-early) with the changes
> > rebased onto v5.14-rc1 that also addresses the 'Fixes: ' tag.
> > 
> > If you pull that instead then I could just merge that tag into my
> > for-next branch and linux-next should be fine again.
> > 
> > When I localy tried it seemed to resolve fine with merge either of the
> > branches involved (linux-next/master, char-misc-next, for-next).
> 
> I would take this, but I get the following warning (and if I ignore it,
> you will get it from linux-next):
> 
> Commit: 6c17b7ff1d11 ("fpga: zynqmp-fpga: Address warning about unused variable")
> 	Fixes tag: Fixes: c09f7471127e ("fpga manager: Adding FPGA Manager support for
> 	Has these problem(s):
> 	        - Subject has leading but no trailing parentheses
> 	        - Subject has leading but no trailing quotes
> 
> So can you fix this up?  Might as well just rebase and redo the whole
> tree, then you can add the additional patch you send after this to it as
> well.

The tag mentioned above 'fpga-for-5.15-early', is a rebase that fixes
this on top of v5.14-rc1 (which gets rid of the back-merge, too)

git://git.kernel.org/pub/scm/linux/kernel/git/mdf/linux-fpga tags/fpga-for-5.15-early

I didn't add the patch because it depends on changes in your tree (the
versal driver you applied), which would make for a weird tree if I don't
pull in your changes?

Let me know if you prefer the patch inside anyways or not and I can
send out a new pull request.

Thanks
Moritz

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

* Re: [GIT PULL] FPGA Manager changes for 5.15-rc1
  2021-07-27 16:13     ` Moritz Fischer
@ 2021-07-27 16:45       ` Greg KH
  0 siblings, 0 replies; 8+ messages in thread
From: Greg KH @ 2021-07-27 16:45 UTC (permalink / raw)
  To: Moritz Fischer; +Cc: linux-fpga

On Tue, Jul 27, 2021 at 09:13:39AM -0700, Moritz Fischer wrote:
> On Tue, Jul 27, 2021 at 04:39:46PM +0200, Greg KH wrote:
> > On Sun, Jul 25, 2021 at 08:00:49PM -0700, Moritz Fischer wrote:
> > > Hi Greg,
> > > 
> > > On Sun, Jul 25, 2021 at 07:55:26PM -0700, Moritz Fischer wrote:
> > > > The following changes since commit e73f0f0ee7541171d89f2e2491130c7771ba58d3:
> > > > 
> > > >   Linux 5.14-rc1 (2021-07-11 15:07:40 -0700)
> > > > 
> > > > are available in the Git repository at:
> > > > 
> > > >   git://git.kernel.org/pub/scm/linux/kernel/git/mdf/linux-fpga tags/fpga-for-5.15
> > > > 
> > > > for you to fetch changes up to c485d3bf3cc7790faed2b90c799a38caa2f69268:
> > > > 
> > > >   fpga: fpga-mgr: wrap the write_sg() op (2021-07-18 08:05:00 -0700)
> > > > 
> > > > ----------------------------------------------------------------
> > > > FPGA Manager Changes for 5.15-rc1
> > > > 
> > > > FPGA Manager
> > > > 
> > > > - Navin's change removes a duplicate word in a comment
> > > > - Tom's change fixes a spelling mistake
> > > > - Mauro's change fixes up documentation
> > > > - Tom's second set adds wrappers to allow drivers not having to
> > > >   implement empty functions by moving checks into fpga-mgr core code
> > > > - My changes address a bunch of warnings
> > > > 
> > > > DFL
> > > > 
> > > > - Martin's change adds a new PCI ID for Silicom N501x PAC cards
> > > > 
> > > > All patches have been reviewed on the mailing list, and have been in the
> > > > last linux-next releases (as part of my for-next branch).
> > > > 
> > > > I did get a complaint about one of the commit messages w/ a Fixes: tag
> > > > which would need a rebase to fix.
> > > > 
> > > > Some of the earlier patches were originally meant for 5.14 but missed
> > > > the merge window by a couple of days, hence the back-merge of 5.14-rc1.
> > > > 
> > > > Signed-offy-by: Moritz Fischer <mdf@kernel.org>
> > > > 
> > > > ----------------------------------------------------------------
> > > > Martin Hundebøll (1):
> > > >       fpga: dfl: pci: add device IDs for Silicom N501x PAC cards
> > > > 
> > > > Mauro Carvalho Chehab (1):
> > > >       docs: driver-api: fpga: avoid using UTF-8 chars
> > > > 
> > > > Moritz Fischer (5):
> > > >       Merge tag 'v5.14-rc1' into for-next
> > > >       fpga: altera-freeze-bridge: Address warning about unused variable
> > > >       fpga: xiilnx-spi: Address warning about unused variable
> > > >       fpga: xilinx-pr-decoupler: Address warning about unused variable
> > > >       fpga: zynqmp-fpga: Address warning about unused variable
> > > > 
> > > > Navin Sankar Velliangiri (1):
> > > >       fpga: fpga-bridge: removed repeated word
> > > > 
> > > > Tom Rix (8):
> > > >       fpga: fix spelling mistakes
> > > >       fpga: fpga-mgr: wrap the write_init() op
> > > >       fpga: fpga-mgr: make write_complete() op optional
> > > >       fpga: fpga-mgr: wrap the write() op
> > > >       fpga: fpga-mgr: wrap the status() op
> > > >       fpga: fpga-mgr: wrap the state() op
> > > >       fpga: fpga-mgr: wrap the fpga_remove() op
> > > >       fpga: fpga-mgr: wrap the write_sg() op
> > > > 
> > > >  Documentation/driver-api/fpga/fpga-bridge.rst      |  10 +-
> > > >  Documentation/driver-api/fpga/fpga-mgr.rst         |  12 +--
> > > >  Documentation/driver-api/fpga/fpga-programming.rst |   8 +-
> > > >  Documentation/driver-api/fpga/fpga-region.rst      |  20 ++--
> > > >  Documentation/fpga/dfl.rst                         |   4 +-
> > > >  drivers/fpga/altera-cvp.c                          |   2 +-
> > > >  drivers/fpga/altera-freeze-bridge.c                |   2 +
> > > >  drivers/fpga/dfl-fme-mgr.c                         |   6 --
> > > >  drivers/fpga/dfl-fme-pr.c                          |   2 +-
> > > >  drivers/fpga/dfl-n3000-nios.c                      |   2 +-
> > > >  drivers/fpga/dfl-pci.c                             |   5 +
> > > >  drivers/fpga/dfl.h                                 |   2 +-
> > > >  drivers/fpga/fpga-bridge.c                         |   8 +-
> > > >  drivers/fpga/fpga-mgr.c                            | 111 ++++++++++++++-------
> > > >  drivers/fpga/stratix10-soc.c                       |   6 --
> > > >  drivers/fpga/ts73xx-fpga.c                         |   6 --
> > > >  drivers/fpga/xilinx-pr-decoupler.c                 |   2 +
> > > >  drivers/fpga/xilinx-spi.c                          |   2 +
> > > >  drivers/fpga/zynq-fpga.c                           |   6 +-
> > > >  drivers/fpga/zynqmp-fpga.c                         |  10 +-
> > > >  include/linux/fpga/fpga-mgr.h                      |   2 +-
> > > >  21 files changed, 127 insertions(+), 101 deletions(-)
> > > 
> > > I realize the backwards merge is somewhat messy, as discussed you said
> > > to not rebase.
> > > 
> > > Alternatively there's a tag (fpga-for-5.15-early) with the changes
> > > rebased onto v5.14-rc1 that also addresses the 'Fixes: ' tag.
> > > 
> > > If you pull that instead then I could just merge that tag into my
> > > for-next branch and linux-next should be fine again.
> > > 
> > > When I localy tried it seemed to resolve fine with merge either of the
> > > branches involved (linux-next/master, char-misc-next, for-next).
> > 
> > I would take this, but I get the following warning (and if I ignore it,
> > you will get it from linux-next):
> > 
> > Commit: 6c17b7ff1d11 ("fpga: zynqmp-fpga: Address warning about unused variable")
> > 	Fixes tag: Fixes: c09f7471127e ("fpga manager: Adding FPGA Manager support for
> > 	Has these problem(s):
> > 	        - Subject has leading but no trailing parentheses
> > 	        - Subject has leading but no trailing quotes
> > 
> > So can you fix this up?  Might as well just rebase and redo the whole
> > tree, then you can add the additional patch you send after this to it as
> > well.
> 
> The tag mentioned above 'fpga-for-5.15-early', is a rebase that fixes
> this on top of v5.14-rc1 (which gets rid of the back-merge, too)
> 
> git://git.kernel.org/pub/scm/linux/kernel/git/mdf/linux-fpga tags/fpga-for-5.15-early

Ah, missed that, sorry.  Let me go try that tag...

That worked, so I took that.

> I didn't add the patch because it depends on changes in your tree (the
> versal driver you applied), which would make for a weird tree if I don't
> pull in your changes?
> 
> Let me know if you prefer the patch inside anyways or not and I can
> send out a new pull request.

I'll go take the patch now, no worries.

thanks,

greg k-h

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

* Re: [GIT PULL] FPGA Manager changes for 5.15-rc1
  2021-08-04 15:31 Moritz Fischer
@ 2021-08-05 12:26 ` Greg KH
  0 siblings, 0 replies; 8+ messages in thread
From: Greg KH @ 2021-08-05 12:26 UTC (permalink / raw)
  To: Moritz Fischer; +Cc: linux-fpga, moritzf

On Wed, Aug 04, 2021 at 08:31:54AM -0700, Moritz Fischer wrote:
> The following changes since commit 630211a17055bafd21fb83ae8c0002b2e214ebb2:
> 
>   fpga: fpga-mgr: wrap the write_sg() op (2021-07-24 15:10:31 -0700)
> 
> are available in the Git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/mdf/linux-fpga.git tags/fpga-for-5.15

Pulled and pushed out, thanks.

greg k-h

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

* [GIT PULL] FPGA Manager changes for 5.15-rc1
@ 2021-08-04 15:31 Moritz Fischer
  2021-08-05 12:26 ` Greg KH
  0 siblings, 1 reply; 8+ messages in thread
From: Moritz Fischer @ 2021-08-04 15:31 UTC (permalink / raw)
  To: gregkh; +Cc: linux-fpga, moritzf

The following changes since commit 630211a17055bafd21fb83ae8c0002b2e214ebb2:

  fpga: fpga-mgr: wrap the write_sg() op (2021-07-24 15:10:31 -0700)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/mdf/linux-fpga.git tags/fpga-for-5.15

for you to fetch changes up to 4f45f3404960109843eaa92c8a4a850d6bdd9981:

  spi: spi-altera-dfl: support n5010 feature revision (2021-07-29 13:02:28 -0700)

----------------------------------------------------------------
FPGA Manager changes for 5.15-rc1

FPGA Manager

- Colin's change is a simple spelling cleanup.

DFL

- Martin's fist change exposes DFL feature revision to client drivers
- Martin's second change modifies a SPI driver to populate different
  spi_board_info modaliases based on the DFL feature revision

All patches have been reviewed on the mailing list, and have been in the
last few linux-next releases (as part of my for-next branch) without issues.

Signed-off-by: Moritz Fischer <mdf@kernel.org>

----------------------------------------------------------------
Colin Ian King (1):
      fpga: Fix spelling mistake "eXchnage" -> "exchange" in Kconfig

Martin Hundebøll (2):
      fpga: dfl: expose feature revision from struct dfl_device
      spi: spi-altera-dfl: support n5010 feature revision

 drivers/fpga/Kconfig         |  2 +-
 drivers/fpga/dfl.c           | 27 +++++++++++++++++----------
 drivers/fpga/dfl.h           |  1 +
 drivers/spi/spi-altera-dfl.c | 21 ++++++++++++---------
 include/linux/dfl.h          |  1 +
 5 files changed, 32 insertions(+), 20 deletions(-)

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

* [GIT PULL] FPGA Manager changes for 5.15-rc1
@ 2021-07-26  2:53 Moritz Fischer
  0 siblings, 0 replies; 8+ messages in thread
From: Moritz Fischer @ 2021-07-26  2:53 UTC (permalink / raw)
  To: gregk; +Cc: linux-fpga

The following changes since commit e73f0f0ee7541171d89f2e2491130c7771ba58d3:

  Linux 5.14-rc1 (2021-07-11 15:07:40 -0700)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/mdf/linux-fpga tags/fpga-for-5.15

for you to fetch changes up to c485d3bf3cc7790faed2b90c799a38caa2f69268:

  fpga: fpga-mgr: wrap the write_sg() op (2021-07-18 08:05:00 -0700)

----------------------------------------------------------------
FPGA Manager Changes for 5.15-rc1

FPGA Manager

- Navin's change removes a duplicate word in a comment
- Tom's change fixes a spelling mistake
- Mauro's change fixes up documentation
- Tom's second set adds wrappers to allow drivers not having to
  implement empty functions by moving checks into fpga-mgr core code
- My changes address a bunch of warnings

DFL

- Martin's change adds a new PCI ID for Silicom N501x PAC cards

All patches have been reviewed on the mailing list, and have been in the
last linux-next releases (as part of my for-next branch).

I did get a complaint about one of the commit messages w/ a Fixes: tag
which would need a rebase to fix.

Some of the earlier patches were originally meant for 5.14 but missed
the merge window by a couple of days, hence the back-merge of 5.14-rc1.

Signed-offy-by: Moritz Fischer <mdf@kernel.org>

----------------------------------------------------------------
Martin Hundebøll (1):
      fpga: dfl: pci: add device IDs for Silicom N501x PAC cards

Mauro Carvalho Chehab (1):
      docs: driver-api: fpga: avoid using UTF-8 chars

Moritz Fischer (5):
      Merge tag 'v5.14-rc1' into for-next
      fpga: altera-freeze-bridge: Address warning about unused variable
      fpga: xiilnx-spi: Address warning about unused variable
      fpga: xilinx-pr-decoupler: Address warning about unused variable
      fpga: zynqmp-fpga: Address warning about unused variable

Navin Sankar Velliangiri (1):
      fpga: fpga-bridge: removed repeated word

Tom Rix (8):
      fpga: fix spelling mistakes
      fpga: fpga-mgr: wrap the write_init() op
      fpga: fpga-mgr: make write_complete() op optional
      fpga: fpga-mgr: wrap the write() op
      fpga: fpga-mgr: wrap the status() op
      fpga: fpga-mgr: wrap the state() op
      fpga: fpga-mgr: wrap the fpga_remove() op
      fpga: fpga-mgr: wrap the write_sg() op

 Documentation/driver-api/fpga/fpga-bridge.rst      |  10 +-
 Documentation/driver-api/fpga/fpga-mgr.rst         |  12 +--
 Documentation/driver-api/fpga/fpga-programming.rst |   8 +-
 Documentation/driver-api/fpga/fpga-region.rst      |  20 ++--
 Documentation/fpga/dfl.rst                         |   4 +-
 drivers/fpga/altera-cvp.c                          |   2 +-
 drivers/fpga/altera-freeze-bridge.c                |   2 +
 drivers/fpga/dfl-fme-mgr.c                         |   6 --
 drivers/fpga/dfl-fme-pr.c                          |   2 +-
 drivers/fpga/dfl-n3000-nios.c                      |   2 +-
 drivers/fpga/dfl-pci.c                             |   5 +
 drivers/fpga/dfl.h                                 |   2 +-
 drivers/fpga/fpga-bridge.c                         |   8 +-
 drivers/fpga/fpga-mgr.c                            | 111 ++++++++++++++-------
 drivers/fpga/stratix10-soc.c                       |   6 --
 drivers/fpga/ts73xx-fpga.c                         |   6 --
 drivers/fpga/xilinx-pr-decoupler.c                 |   2 +
 drivers/fpga/xilinx-spi.c                          |   2 +
 drivers/fpga/zynq-fpga.c                           |   6 +-
 drivers/fpga/zynqmp-fpga.c                         |  10 +-
 include/linux/fpga/fpga-mgr.h                      |   2 +-
 21 files changed, 127 insertions(+), 101 deletions(-)

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

end of thread, other threads:[~2021-08-05 12:27 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-26  2:55 [GIT PULL] FPGA Manager changes for 5.15-rc1 Moritz Fischer
2021-07-26  3:00 ` Moritz Fischer
2021-07-27 14:39   ` Greg KH
2021-07-27 16:13     ` Moritz Fischer
2021-07-27 16:45       ` Greg KH
  -- strict thread matches above, loose matches on Subject: below --
2021-08-04 15:31 Moritz Fischer
2021-08-05 12:26 ` Greg KH
2021-07-26  2:53 Moritz Fischer

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