linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [RESEND 0/2] fpga fixes for 4.20
@ 2018-11-12 19:38 Alan Tull
  2018-11-12 19:38 ` [RESEND 1/2] fpga: of-fpga-region: Use platform_set_drvdata Alan Tull
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Alan Tull @ 2018-11-12 19:38 UTC (permalink / raw)
  To: Greg Kroah-Hartman; +Cc: Moritz Fischer, Alan Tull, linux-kernel, linux-fpga

Hi Greg,

Here's a reposting of the two small fixes for fpga, please
take them in.  They've been reviewed on the mailing list
and apply cleanly on today's char-misc-test.

Thanks,
Alan

Moritz Fischer (2):
  fpga: of-fpga-region: Use platform_set_drvdata
  fpga: dfl-fme-region: Use platform_get_drvdata()

 drivers/fpga/dfl-fme-region.c | 2 +-
 drivers/fpga/of-fpga-region.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

-- 
2.7.4


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

* [RESEND 1/2] fpga: of-fpga-region: Use platform_set_drvdata
  2018-11-12 19:38 [RESEND 0/2] fpga fixes for 4.20 Alan Tull
@ 2018-11-12 19:38 ` Alan Tull
  2018-11-12 19:38 ` [RESEND 2/2] fpga: dfl-fme-region: Use platform_get_drvdata() Alan Tull
  2018-11-12 19:46 ` [RESEND 0/2] fpga fixes for 4.20 Greg Kroah-Hartman
  2 siblings, 0 replies; 4+ messages in thread
From: Alan Tull @ 2018-11-12 19:38 UTC (permalink / raw)
  To: Greg Kroah-Hartman; +Cc: Moritz Fischer, Alan Tull, linux-kernel, linux-fpga

From: Moritz Fischer <mdf@kernel.org>

Use platform_set_drvdata rather than dev_set_drvdata
to match the platform_get_drvdata in the _remove()
function of the platform driver.

Signed-off-by: Moritz Fischer <mdf@kernel.org>
Acked-by: Alan Tull <atull@kernel.org>
---
 drivers/fpga/of-fpga-region.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/fpga/of-fpga-region.c b/drivers/fpga/of-fpga-region.c
index 122286f..75f64ab 100644
--- a/drivers/fpga/of-fpga-region.c
+++ b/drivers/fpga/of-fpga-region.c
@@ -421,7 +421,7 @@ static int of_fpga_region_probe(struct platform_device *pdev)
 		goto eprobe_mgr_put;
 
 	of_platform_populate(np, fpga_region_of_match, NULL, &region->dev);
-	dev_set_drvdata(dev, region);
+	platform_set_drvdata(pdev, region);
 
 	dev_info(dev, "FPGA Region probed\n");
 
-- 
2.7.4


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

* [RESEND 2/2] fpga: dfl-fme-region: Use platform_get_drvdata()
  2018-11-12 19:38 [RESEND 0/2] fpga fixes for 4.20 Alan Tull
  2018-11-12 19:38 ` [RESEND 1/2] fpga: of-fpga-region: Use platform_set_drvdata Alan Tull
@ 2018-11-12 19:38 ` Alan Tull
  2018-11-12 19:46 ` [RESEND 0/2] fpga fixes for 4.20 Greg Kroah-Hartman
  2 siblings, 0 replies; 4+ messages in thread
From: Alan Tull @ 2018-11-12 19:38 UTC (permalink / raw)
  To: Greg Kroah-Hartman; +Cc: Moritz Fischer, Alan Tull, linux-kernel, linux-fpga

From: Moritz Fischer <mdf@kernel.org>

Use platform_get_drvdata() in remove() function of
the platform driver rather than dev_get_drvdata()
to match the platform_set_drvdata in the probe().

Signed-off-by: Moritz Fischer <mdf@kernel.org>
Acked-by: Alan Tull <atull@kernel.org>
---
 drivers/fpga/dfl-fme-region.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/fpga/dfl-fme-region.c b/drivers/fpga/dfl-fme-region.c
index ec134ec..1eeb42a 100644
--- a/drivers/fpga/dfl-fme-region.c
+++ b/drivers/fpga/dfl-fme-region.c
@@ -64,7 +64,7 @@ static int fme_region_probe(struct platform_device *pdev)
 
 static int fme_region_remove(struct platform_device *pdev)
 {
-	struct fpga_region *region = dev_get_drvdata(&pdev->dev);
+	struct fpga_region *region = platform_get_drvdata(pdev);
 	struct fpga_manager *mgr = region->mgr;
 
 	fpga_region_unregister(region);
-- 
2.7.4


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

* Re: [RESEND 0/2] fpga fixes for 4.20
  2018-11-12 19:38 [RESEND 0/2] fpga fixes for 4.20 Alan Tull
  2018-11-12 19:38 ` [RESEND 1/2] fpga: of-fpga-region: Use platform_set_drvdata Alan Tull
  2018-11-12 19:38 ` [RESEND 2/2] fpga: dfl-fme-region: Use platform_get_drvdata() Alan Tull
@ 2018-11-12 19:46 ` Greg Kroah-Hartman
  2 siblings, 0 replies; 4+ messages in thread
From: Greg Kroah-Hartman @ 2018-11-12 19:46 UTC (permalink / raw)
  To: Alan Tull; +Cc: Moritz Fischer, linux-kernel, linux-fpga

On Mon, Nov 12, 2018 at 01:38:32PM -0600, Alan Tull wrote:
> Hi Greg,
> 
> Here's a reposting of the two small fixes for fpga, please
> take them in.  They've been reviewed on the mailing list
> and apply cleanly on today's char-misc-test.

This isn't a bugfix, they should just go into my "next" branch to be
merged for the next -rc1 release, right?  I thought that is what I said
last time...

thanks,

greg k-h

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

end of thread, other threads:[~2018-11-12 19:46 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-11-12 19:38 [RESEND 0/2] fpga fixes for 4.20 Alan Tull
2018-11-12 19:38 ` [RESEND 1/2] fpga: of-fpga-region: Use platform_set_drvdata Alan Tull
2018-11-12 19:38 ` [RESEND 2/2] fpga: dfl-fme-region: Use platform_get_drvdata() Alan Tull
2018-11-12 19:46 ` [RESEND 0/2] fpga fixes for 4.20 Greg Kroah-Hartman

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