Messages in this thread |  | | From | Alan Tull <> | Date | Sun, 11 Feb 2018 22:16:20 -0600 | Subject | Re: [PATCH v3 0/4] fpga: don't use drvdata in common code |
| |
On Fri, Feb 9, 2018 at 11:18 AM, Moritz Fischer <mdf@kernel.org> wrote: > On Wed, Feb 07, 2018 at 03:05:26PM -0600, Alan Tull wrote: >> Setting drvdata is fine for DT based devices which have one manager, >> bridge, or region device per platform device. However, PCIe based >> devices may have multiple FPGA mgr/bridge/regions under one PCIe >> device. Without these changes, PCIe-based solutions have to create an >> extra device for each child mgr/bridge/region to hold drvdata. >> >> Also changing the fpga_*_register/unregister functions to take the >> corresponding struct as the only parameter wherever they weren't already. >> >> * int fpga_bridge_register(struct fpga_bridge *br) >> * int fpga_mgr_register(struct fpga_manager *mgr) >> * int fpga_region_register(struct fpga_region *region) >> >> * void fpga_bridge_unregister(struct fpga_bridge *br) >> * void fpga_mgr_unregister(struct fpga_manager *mgr) >> * int fpga_region_unregister(struct fpga_region *region) >> >> Changes from v2: >> * Drop patch 5 which fixed a DT overlay issue. The underlying >> issue was fixed in the main DT overlay code. >> * Minor code cleanup to make it more obvious that the changes >> were not breaking anything >> * Added Moritz' ack on patch 1 >> >> Alan Tull (4): >> fpga: region: don't use drvdata in common fpga code >> fpga: manager: don't use drvdata in common fpga code >> fpga: bridge: don't use drvdata in common fpga code >> fpga: region: change fpga_region_register to have one param > > Acked-by: Moritz Fischer <mdf@kernel.org> for the ones I haven't ack'd > so far.
Thanks Moritz!
Alan
> > Thanks, > > Moritz >> >> Documentation/fpga/fpga-mgr.txt | 24 +++++++++++++++------ >> Documentation/fpga/fpga-region.txt | 3 +-- >> drivers/fpga/altera-cvp.c | 18 ++++++++++++---- >> drivers/fpga/altera-fpga2sdram.c | 20 +++++++++++++---- >> drivers/fpga/altera-freeze-bridge.c | 18 +++++++++++++--- >> drivers/fpga/altera-hps2fpga.c | 16 +++++++++++--- >> drivers/fpga/altera-pr-ip-core.c | 17 +++++++++++++-- >> drivers/fpga/altera-ps-spi.c | 18 +++++++++++++--- >> drivers/fpga/fpga-bridge.c | 43 ++++++++++++++----------------------- >> drivers/fpga/fpga-mgr.c | 39 ++++++++++++--------------------- >> drivers/fpga/fpga-region.c | 9 ++++++-- >> drivers/fpga/ice40-spi.c | 20 +++++++++++++---- >> drivers/fpga/of-fpga-region.c | 4 +++- >> drivers/fpga/socfpga-a10.c | 16 +++++++++++--- >> drivers/fpga/socfpga.c | 18 +++++++++++++--- >> drivers/fpga/ts73xx-fpga.c | 18 +++++++++++++--- >> drivers/fpga/xilinx-pr-decoupler.c | 15 ++++++++++--- >> drivers/fpga/xilinx-spi.c | 18 +++++++++++++--- >> drivers/fpga/zynq-fpga.c | 16 +++++++++++--- >> include/linux/fpga/fpga-bridge.h | 7 +++--- >> include/linux/fpga/fpga-mgr.h | 8 +++---- >> include/linux/fpga/fpga-region.h | 4 +++- >> 22 files changed, 256 insertions(+), 113 deletions(-) >> >> -- >> 2.7.4 >>
|  |