mbox series

[GIT,PULL] FPGA Manager changes for 5.15-rc1

Message ID YP4kHpn7CDzCYrbU@epycbox.lan (mailing list archive)
State New
Headers show
Series [GIT,PULL] FPGA Manager changes for 5.15-rc1 | expand

Pull-request

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

Message

Moritz Fischer July 26, 2021, 2:55 a.m. UTC
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(-)

Comments

Moritz Fischer July 26, 2021, 3 a.m. UTC | #1
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
Greg KH July 27, 2021, 2:39 p.m. UTC | #2
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
Moritz Fischer July 27, 2021, 4:13 p.m. UTC | #3
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
Greg KH July 27, 2021, 4:45 p.m. UTC | #4
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