All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
To: linux-iio@vger.kernel.org
Cc: Lars-Peter Clausen <lars@metafoo.de>,
	Jonathan Cameron <jic23@kernel.org>,
	Jonathan Cameron <Jonathan.Cameron@huawei.com>
Subject: [PATCH 26/45] iio:triggers: drop assign iio_info.driver_module and iio_trigger_ops.owner
Date: Sun, 23 Jul 2017 17:26:08 +0100	[thread overview]
Message-ID: <20170723162627.22534-27-Jonathan.Cameron@huawei.com> (raw)
In-Reply-To: <20170723162627.22534-1-Jonathan.Cameron@huawei.com>

From: Jonathan Cameron <jic23@kernel.org>

The equivalent of both of these are now done via macro magic when
the relevant register calls are made.  The actual structure
elements will shortly go away.

Note that stm32-timer-trigger has expanded rather beyond triggers
(to include encoder input counting for example) and hence has an
iio_info structure.

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
---
 drivers/iio/trigger/iio-trig-hrtimer.c    | 1 -
 drivers/iio/trigger/iio-trig-interrupt.c  | 1 -
 drivers/iio/trigger/iio-trig-loop.c       | 1 -
 drivers/iio/trigger/iio-trig-sysfs.c      | 1 -
 drivers/iio/trigger/stm32-timer-trigger.c | 2 --
 5 files changed, 6 deletions(-)

diff --git a/drivers/iio/trigger/iio-trig-hrtimer.c b/drivers/iio/trigger/iio-trig-hrtimer.c
index a1cad6cc2e0f..3ee92160053f 100644
--- a/drivers/iio/trigger/iio-trig-hrtimer.c
+++ b/drivers/iio/trigger/iio-trig-hrtimer.c
@@ -114,7 +114,6 @@ static int iio_trig_hrtimer_set_state(struct iio_trigger *trig, bool state)
 }
 
 static const struct iio_trigger_ops iio_hrtimer_trigger_ops = {
-	.owner = THIS_MODULE,
 	.set_trigger_state = iio_trig_hrtimer_set_state,
 };
 
diff --git a/drivers/iio/trigger/iio-trig-interrupt.c b/drivers/iio/trigger/iio-trig-interrupt.c
index e18f12b74610..171c4ed03543 100644
--- a/drivers/iio/trigger/iio-trig-interrupt.c
+++ b/drivers/iio/trigger/iio-trig-interrupt.c
@@ -29,7 +29,6 @@ static irqreturn_t iio_interrupt_trigger_poll(int irq, void *private)
 }
 
 static const struct iio_trigger_ops iio_interrupt_trigger_ops = {
-	.owner = THIS_MODULE,
 };
 
 static int iio_interrupt_trigger_probe(struct platform_device *pdev)
diff --git a/drivers/iio/trigger/iio-trig-loop.c b/drivers/iio/trigger/iio-trig-loop.c
index dc6be28f96fe..b4b02dbd6e8a 100644
--- a/drivers/iio/trigger/iio-trig-loop.c
+++ b/drivers/iio/trigger/iio-trig-loop.c
@@ -74,7 +74,6 @@ static int iio_loop_trigger_set_state(struct iio_trigger *trig, bool state)
 
 static const struct iio_trigger_ops iio_loop_trigger_ops = {
 	.set_trigger_state = iio_loop_trigger_set_state,
-	.owner = THIS_MODULE,
 };
 
 static struct iio_sw_trigger *iio_trig_loop_probe(const char *name)
diff --git a/drivers/iio/trigger/iio-trig-sysfs.c b/drivers/iio/trigger/iio-trig-sysfs.c
index 202e8b89caf2..3f0dc9a1a514 100644
--- a/drivers/iio/trigger/iio-trig-sysfs.c
+++ b/drivers/iio/trigger/iio-trig-sysfs.c
@@ -127,7 +127,6 @@ static const struct attribute_group *iio_sysfs_trigger_attr_groups[] = {
 };
 
 static const struct iio_trigger_ops iio_sysfs_trigger_ops = {
-	.owner = THIS_MODULE,
 };
 
 static int iio_sysfs_trigger_probe(int id)
diff --git a/drivers/iio/trigger/stm32-timer-trigger.c b/drivers/iio/trigger/stm32-timer-trigger.c
index d22bc56dd9fc..41462e495586 100644
--- a/drivers/iio/trigger/stm32-timer-trigger.c
+++ b/drivers/iio/trigger/stm32-timer-trigger.c
@@ -318,7 +318,6 @@ static const struct attribute_group *stm32_trigger_attr_groups[] = {
 };
 
 static const struct iio_trigger_ops timer_trigger_ops = {
-	.owner = THIS_MODULE,
 };
 
 static int stm32_setup_iio_triggers(struct stm32_timer_trigger *priv)
@@ -442,7 +441,6 @@ static int stm32_counter_validate_trigger(struct iio_dev *indio_dev,
 }
 
 static const struct iio_info stm32_trigger_info = {
-	.driver_module = THIS_MODULE,
 	.validate_trigger = stm32_counter_validate_trigger,
 	.read_raw = stm32_counter_read_raw,
 	.write_raw = stm32_counter_write_raw
-- 
2.13.3


  parent reply	other threads:[~2017-07-23 16:26 UTC|newest]

Thread overview: 55+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-23 16:25 [PATCH V2 00/45] IIO: Drop manual assignment of THIS_MODULE Jonathan Cameron
2017-07-23 16:25 ` [PATCH 01/45] iio: Use macro magic to avoid manual assign of driver_module Jonathan Cameron
2017-07-23 16:25 ` [PATCH 02/45] iio: triggers: Use macros to avoid boilerplate assignment of owner Jonathan Cameron
2017-07-23 16:25 ` [PATCH 03/45] iio:trigger: Remove necessity to have a trig->ops structure Jonathan Cameron
2017-07-23 16:25 ` [PATCH 04/45] coccinelle: Add an iio_no_owner semantic patch to drop driver_owner Jonathan Cameron
2017-07-23 16:25 ` [PATCH 05/45] iio:adc: drop assign iio_info.driver_module and iio_trigger_ops.owner Jonathan Cameron
2017-07-23 16:25 ` [PATCH 06/45] iio:accel: " Jonathan Cameron
2017-07-23 16:25 ` [PATCH 07/45] iio:amplifiers:ad8366 " Jonathan Cameron
2017-07-23 16:25 ` [PATCH 08/45] iio:chemical: " Jonathan Cameron
2017-07-23 16:25 ` [PATCH 09/45] iio:common: " Jonathan Cameron
2017-07-23 16:25 ` [PATCH 10/45] iio:counter:104-quad-8 drop assign iio_info.driver_module Jonathan Cameron
2017-07-23 16:25 ` [PATCH 11/45] iio:dac: drop assignment of iio_info.driver_module Jonathan Cameron
2017-07-23 16:25 ` [PATCH 12/45] iio:dummy: " Jonathan Cameron
2017-07-23 16:25 ` [PATCH 13/45] iio:frequency: drop assign iio_info.driver_module Jonathan Cameron
2017-07-23 16:25 ` [PATCH 14/45] iio:gyro: drop assign iio_info.driver_module and iio_trigger_ops.owner Jonathan Cameron
2017-07-23 16:25 ` [PATCH 15/45] iio:health: " Jonathan Cameron
2017-07-23 16:25 ` [PATCH 16/45] iio:humidity: " Jonathan Cameron
2017-07-23 16:25 ` [PATCH 17/45] iio:imu: " Jonathan Cameron
2017-07-23 16:26 ` [PATCH 18/45] iio:light: " Jonathan Cameron
2017-07-23 16:26 ` [PATCH 19/45] iio:magnetometer: " Jonathan Cameron
2017-07-23 16:26 ` [PATCH 20/45] iio:orientation: drop assign iio_info.driver_module Jonathan Cameron
2017-07-23 16:26 ` [PATCH 21/45] iio:dpot: " Jonathan Cameron
2017-07-23 16:26 ` [PATCH 22/45] iio:potentiostat:lmp91000 drop assign iio_info.driver_module and iio_trigger_ops.owner Jonathan Cameron
2017-07-23 16:26 ` [PATCH 23/45] iio:pressure: " Jonathan Cameron
2017-07-23 16:26 ` [PATCH 24/45] iio:proximity: " Jonathan Cameron
2017-07-23 16:26 ` [PATCH 25/45] iio:temperature: drop assignment of iio_info.driver_module Jonathan Cameron
2017-07-23 16:26 ` Jonathan Cameron [this message]
2017-07-23 16:26 ` [PATCH 27/45] staging:iio:accel: " Jonathan Cameron
2017-07-23 16:26 ` [PATCH 28/45] staging:iio:adc: drop assign iio_info.driver_module Jonathan Cameron
2017-07-23 16:26 ` [PATCH 29/45] staging:iio:cdc: " Jonathan Cameron
2017-07-23 16:26 ` [PATCH 30/45] staging:iio:frequency: " Jonathan Cameron
2017-07-23 16:26 ` [PATCH 31/45] staging:iio:gyro:adis16060 " Jonathan Cameron
2017-07-23 16:26 ` [PATCH 32/45] staging:iio:ad5933: " Jonathan Cameron
2017-07-23 16:26 ` [PATCH 33/45] staging:iio:meter: drop assign iio_info.driver_module and iio_trigger_ops.owner Jonathan Cameron
2017-07-23 16:26 ` [PATCH 34/45] iio:resolver: drop assignment of iio_info.driver_module Jonathan Cameron
2017-07-23 16:26 ` [PATCH 35/45] iio:adc: drop assignment of iio_trigger_ops.owner Jonathan Cameron
2017-07-23 16:26 ` [PATCH 36/45] staging:iio:light:tsl2x7x drop assignment of driver_module Jonathan Cameron
     [not found] ` <20170723162627.22534-1-Jonathan.Cameron-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
2017-07-23 16:26   ` [PATCH 37/45] input: tsc2007 - drop the driver_module assignment in iio interface Jonathan Cameron
2017-07-23 16:26     ` Jonathan Cameron
2017-07-23 16:26   ` [PATCH 38/45] platform/x86: toshiba_acpi: drop assignment of iio_info.driver_module Jonathan Cameron
2017-07-23 16:26     ` Jonathan Cameron
2017-07-23 16:26 ` [PATCH 39/45] iio: drop iio_info.driver_module and iio_trigger_ops.owner Jonathan Cameron
2017-07-23 16:26 ` [PATCH 40/45] iio:adc:max9611: Drop explicit setting of the i2c module owner Jonathan Cameron
2017-07-23 16:26 ` [PATCH 41/45] iio:adc:ti-ads8688: Drop manual setting of the driver owner field Jonathan Cameron
2017-07-23 16:26 ` [PATCH 42/45] iio:potentiometer:max5481 drop explicit setting of the owner module Jonathan Cameron
2017-07-23 16:26 ` [PATCH 43/45] iio:potentiometer:max5487: Drop explicit setting of module owner Jonathan Cameron
2017-07-23 16:26 ` [PATCH 44/45] staging:iio:docs: drop reference to setting trig_ops->owner Jonathan Cameron
2017-07-23 16:26 ` [PATCH 45/45] staging:iio:docs: drop setting of THIS_MODULE from staging docs Jonathan Cameron
2017-08-20 15:13 ` [PATCH V2 00/45] IIO: Drop manual assignment of THIS_MODULE Jonathan Cameron
2017-08-21  7:42   ` Lars-Peter Clausen
2017-08-21 12:38     ` Jonathan Cameron
2017-08-22 21:19       ` Jonathan Cameron
2017-09-03 11:23         ` Jonathan Cameron
2017-09-03 13:35           ` Peter Rosin
2017-09-03 17:10             ` Jonathan Cameron

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170723162627.22534-27-Jonathan.Cameron@huawei.com \
    --to=jonathan.cameron@huawei.com \
    --cc=jic23@kernel.org \
    --cc=lars@metafoo.de \
    --cc=linux-iio@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.