linux-gpio.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 00/28] remove .owner for most platform_drivers: the missing bits
@ 2014-12-21 21:14 Wolfram Sang
  2014-12-21 21:14 ` [PATCH 07/28] gpio: drop owner assignment from platform_drivers Wolfram Sang
  2014-12-21 21:14 ` [PATCH 15/28] pinctrl: intel: " Wolfram Sang
  0 siblings, 2 replies; 6+ messages in thread
From: Wolfram Sang @ 2014-12-21 21:14 UTC (permalink / raw)
  To: linux-kernel
  Cc: Wolfram Sang, alsa-devel, ath5k-devel, Dan Williams, devicetree,
	dmaengine, dri-devel, iommu, linux-arm-kernel, linux-gpio,
	linux-mips, linux-omap, linux-pci, linux-pm, linuxppc-dev,
	linux-rockchip, linux-samsung-soc, linux-scsi, linux-serial,
	linux-usb, linux-watchdog, linux-wireless, netdev,
	openipmi-developer, rtc-linux

Generated with coccinelle. The big cleanup was pulled in this merge window.
This series catches the bits fallen through. The patches shall go in via the
subsystem trees. If possible for 3.19 to increase consistency I'd say, but you
decide, of course.

cocci-file used:

@match1@
declarer name module_platform_driver;
declarer name module_platform_driver_probe;
declarer name for_each_node_by_type;
identifier __driver;
@@
(
	module_platform_driver(__driver);
|
	module_platform_driver_probe(__driver, ...);
)

@fix1 depends on match1@
identifier match1.__driver;
@@
	static struct platform_driver __driver = {
		.driver = {
-			.owner = THIS_MODULE,
		}
	};

@match2@
identifier __driver;
@@
(
	platform_driver_register(&__driver)
|
	platform_driver_probe(&__driver, ...)
|
	platform_create_bundle(&__driver, ...)
)

@fix2 depends on match2@
identifier match2.__driver;
@@
	static struct platform_driver __driver = {
		.driver = {
-			.owner = THIS_MODULE,
		}
	};

Thanks again to Julia Lawall for support. And hey, we fixed a coccinelle bug on
the way :)


Wolfram Sang (28):
  ARM: mach-exynos: drop owner assignment from platform_drivers
  mips: lantiq: xway: drop owner assignment from platform_drivers
  mips: pci: drop owner assignment from platform_drivers
  char: ipmi: drop owner assignment from platform_drivers
  cpufreq: drop owner assignment from platform_drivers
  dma: drop owner assignment from platform_drivers
  gpio: drop owner assignment from platform_drivers
  gpu: drm: rockchip: drop owner assignment from platform_drivers
  iommu: drop owner assignment from platform_drivers
  net: ethernet: stmicro: stmmac: drop owner assignment from
    platform_drivers
  net: wireless: ath: ath5k: drop owner assignment from platform_drivers
  of: drop owner assignment from platform_drivers
  pci: host: drop owner assignment from platform_drivers
  phy: drop owner assignment from platform_drivers
  pinctrl: intel: drop owner assignment from platform_drivers
  rtc: drop owner assignment from platform_drivers
  scsi: drop owner assignment from platform_drivers
  thermal: drop owner assignment from platform_drivers
  thermal: int340x_thermal: drop owner assignment from platform_drivers
  tty: serial: 8250: drop owner assignment from platform_drivers
  usb: gadget: udc: bdc: drop owner assignment from platform_drivers
  watchdog: drop owner assignment from platform_drivers
  ASoC: intel: drop owner assignment from platform_drivers
  ASoC: intel: sst: drop owner assignment from platform_drivers
  ASoC: omap: drop owner assignment from platform_drivers
  ASoC: pxa: drop owner assignment from platform_drivers
  ASoC: samsung: drop owner assignment from platform_drivers
  macintosh: drop owner assignment from platform_drivers

 arch/arm/mach-exynos/pmu.c                            | 1 -
 arch/mips/lantiq/xway/vmmc.c                          | 1 -
 arch/mips/pci/pci-ar2315.c                            | 1 -
 arch/mips/pci/pci-rt2880.c                            | 1 -
 drivers/char/ipmi/ipmi_powernv.c                      | 1 -
 drivers/cpufreq/ls1x-cpufreq.c                        | 1 -
 drivers/dma/at_xdmac.c                                | 1 -
 drivers/gpio/gpio-vf610.c                             | 1 -
 drivers/gpu/drm/rockchip/rockchip_drm_drv.c           | 1 -
 drivers/iommu/rockchip-iommu.c                        | 1 -
 drivers/macintosh/windfarm_pm112.c                    | 1 -
 drivers/macintosh/windfarm_pm72.c                     | 1 -
 drivers/macintosh/windfarm_rm31.c                     | 1 -
 drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c | 1 -
 drivers/net/wireless/ath/ath5k/ahb.c                  | 1 -
 drivers/of/unittest.c                                 | 1 -
 drivers/pci/host/pci-layerscape.c                     | 1 -
 drivers/phy/phy-armada375-usb2.c                      | 1 -
 drivers/phy/phy-berlin-usb.c                          | 1 -
 drivers/phy/phy-miphy28lp.c                           | 1 -
 drivers/pinctrl/intel/pinctrl-cherryview.c            | 1 -
 drivers/rtc/rtc-opal.c                                | 1 -
 drivers/scsi/atari_scsi.c                             | 1 -
 drivers/scsi/mac_scsi.c                               | 1 -
 drivers/scsi/sun3_scsi.c                              | 1 -
 drivers/thermal/int340x_thermal/int3400_thermal.c     | 1 -
 drivers/thermal/int340x_thermal/int3402_thermal.c     | 1 -
 drivers/thermal/rockchip_thermal.c                    | 1 -
 drivers/tty/serial/8250/8250_omap.c                   | 1 -
 drivers/usb/gadget/udc/bdc/bdc_core.c                 | 1 -
 drivers/watchdog/cadence_wdt.c                        | 1 -
 drivers/watchdog/meson_wdt.c                          | 1 -
 sound/soc/intel/bytcr_dpcm_rt5640.c                   | 1 -
 sound/soc/intel/cht_bsw_rt5672.c                      | 1 -
 sound/soc/intel/sst/sst_acpi.c                        | 1 -
 sound/soc/omap/omap-hdmi-audio.c                      | 1 -
 sound/soc/pxa/spitz.c                                 | 1 -
 sound/soc/samsung/arndale_rt5631.c                    | 1 -
 38 files changed, 38 deletions(-)

-- 
2.1.3

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

* [PATCH 07/28] gpio: drop owner assignment from platform_drivers
  2014-12-21 21:14 [PATCH 00/28] remove .owner for most platform_drivers: the missing bits Wolfram Sang
@ 2014-12-21 21:14 ` Wolfram Sang
  2015-01-14  8:15   ` Linus Walleij
  2014-12-21 21:14 ` [PATCH 15/28] pinctrl: intel: " Wolfram Sang
  1 sibling, 1 reply; 6+ messages in thread
From: Wolfram Sang @ 2014-12-21 21:14 UTC (permalink / raw)
  To: linux-kernel; +Cc: Wolfram Sang, Linus Walleij, Alexandre Courbot, linux-gpio

This platform_driver does not need to set an owner, it will be populated by the
driver core.

Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
---
Generated with coccinelle. SmPL file is in the introductory msg. The big
cleanup was pulled in this merge window. This series catches the bits fallen
through. The patches shall go in via the subsystem trees.

 drivers/gpio/gpio-vf610.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/gpio/gpio-vf610.c b/drivers/gpio/gpio-vf610.c
index 4ee4cee832ec..971c73964ef1 100644
--- a/drivers/gpio/gpio-vf610.c
+++ b/drivers/gpio/gpio-vf610.c
@@ -278,7 +278,6 @@ static int vf610_gpio_probe(struct platform_device *pdev)
 static struct platform_driver vf610_gpio_driver = {
 	.driver		= {
 		.name	= "gpio-vf610",
-		.owner	= THIS_MODULE,
 		.of_match_table = vf610_gpio_dt_ids,
 	},
 	.probe		= vf610_gpio_probe,
-- 
2.1.3

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

* [PATCH 15/28] pinctrl: intel: drop owner assignment from platform_drivers
  2014-12-21 21:14 [PATCH 00/28] remove .owner for most platform_drivers: the missing bits Wolfram Sang
  2014-12-21 21:14 ` [PATCH 07/28] gpio: drop owner assignment from platform_drivers Wolfram Sang
@ 2014-12-21 21:14 ` Wolfram Sang
  2014-12-22 13:35   ` Mika Westerberg
  2015-01-09  9:48   ` Linus Walleij
  1 sibling, 2 replies; 6+ messages in thread
From: Wolfram Sang @ 2014-12-21 21:14 UTC (permalink / raw)
  To: linux-kernel
  Cc: Wolfram Sang, Mika Westerberg, Heikki Krogerus, Linus Walleij,
	linux-gpio

This platform_driver does not need to set an owner, it will be populated by the
driver core.

Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
---
Generated with coccinelle. SmPL file is in the introductory msg. The big
cleanup was pulled in this merge window. This series catches the bits fallen
through. The patches shall go in via the subsystem trees.

 drivers/pinctrl/intel/pinctrl-cherryview.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/pinctrl/intel/pinctrl-cherryview.c b/drivers/pinctrl/intel/pinctrl-cherryview.c
index e9f8b39d1a9f..007d25fd5fb7 100644
--- a/drivers/pinctrl/intel/pinctrl-cherryview.c
+++ b/drivers/pinctrl/intel/pinctrl-cherryview.c
@@ -1497,7 +1497,6 @@ static struct platform_driver chv_pinctrl_driver = {
 	.remove = chv_pinctrl_remove,
 	.driver = {
 		.name = "cherryview-pinctrl",
-		.owner = THIS_MODULE,
 		.acpi_match_table = chv_pinctrl_acpi_match,
 	},
 };
-- 
2.1.3


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

* Re: [PATCH 15/28] pinctrl: intel: drop owner assignment from platform_drivers
  2014-12-21 21:14 ` [PATCH 15/28] pinctrl: intel: " Wolfram Sang
@ 2014-12-22 13:35   ` Mika Westerberg
  2015-01-09  9:48   ` Linus Walleij
  1 sibling, 0 replies; 6+ messages in thread
From: Mika Westerberg @ 2014-12-22 13:35 UTC (permalink / raw)
  To: Wolfram Sang; +Cc: linux-kernel, Heikki Krogerus, Linus Walleij, linux-gpio

On Sun, Dec 21, 2014 at 10:14:36PM +0100, Wolfram Sang wrote:
> This platform_driver does not need to set an owner, it will be populated by the
> driver core.
> 
> Signed-off-by: Wolfram Sang <wsa@the-dreams.de>

Acked-by: Mika Westerberg <mika.westerberg@linux.intel.com>

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

* Re: [PATCH 15/28] pinctrl: intel: drop owner assignment from platform_drivers
  2014-12-21 21:14 ` [PATCH 15/28] pinctrl: intel: " Wolfram Sang
  2014-12-22 13:35   ` Mika Westerberg
@ 2015-01-09  9:48   ` Linus Walleij
  1 sibling, 0 replies; 6+ messages in thread
From: Linus Walleij @ 2015-01-09  9:48 UTC (permalink / raw)
  To: Wolfram Sang
  Cc: linux-kernel@vger.kernel.org, Mika Westerberg, Heikki Krogerus,
	linux-gpio@vger.kernel.org

On Sun, Dec 21, 2014 at 10:14 PM, Wolfram Sang <wsa@the-dreams.de> wrote:

> This platform_driver does not need to set an owner, it will be populated by the
> driver core.
>
> Signed-off-by: Wolfram Sang <wsa@the-dreams.de>

Patch applied.

Yours,
Linus Walleij

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

* Re: [PATCH 07/28] gpio: drop owner assignment from platform_drivers
  2014-12-21 21:14 ` [PATCH 07/28] gpio: drop owner assignment from platform_drivers Wolfram Sang
@ 2015-01-14  8:15   ` Linus Walleij
  0 siblings, 0 replies; 6+ messages in thread
From: Linus Walleij @ 2015-01-14  8:15 UTC (permalink / raw)
  To: Wolfram Sang
  Cc: linux-kernel@vger.kernel.org, Alexandre Courbot,
	linux-gpio@vger.kernel.org

On Sun, Dec 21, 2014 at 10:14 PM, Wolfram Sang <wsa@the-dreams.de> wrote:

> This platform_driver does not need to set an owner, it will be populated by the
> driver core.
>
> Signed-off-by: Wolfram Sang <wsa@the-dreams.de>

Patch applied, thanks!

Yours,
Linus Walleij

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

end of thread, other threads:[~2015-01-14  8:15 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-12-21 21:14 [PATCH 00/28] remove .owner for most platform_drivers: the missing bits Wolfram Sang
2014-12-21 21:14 ` [PATCH 07/28] gpio: drop owner assignment from platform_drivers Wolfram Sang
2015-01-14  8:15   ` Linus Walleij
2014-12-21 21:14 ` [PATCH 15/28] pinctrl: intel: " Wolfram Sang
2014-12-22 13:35   ` Mika Westerberg
2015-01-09  9:48   ` Linus Walleij

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