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@xxxxxxxxxx>
> > > >
> > > > ----------------------------------------------------------------
> > > > 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
