* [PATCH 2/7] edac: i5100_edac: use to_delayed_work
2016-01-01 14:59 [PATCH 1/7] char: xillybus: use to_delayed_work Geliang Tang
@ 2016-01-01 14:59 ` Geliang Tang
2016-01-01 17:35 ` Borislav Petkov
2016-01-01 14:59 ` [PATCH 3/7] input: touchscreen: " Geliang Tang
` (5 subsequent siblings)
6 siblings, 1 reply; 12+ messages in thread
From: Geliang Tang @ 2016-01-01 14:59 UTC (permalink / raw)
To: Doug Thompson, Borislav Petkov, Mauro Carvalho Chehab
Cc: Geliang Tang, linux-edac, linux-kernel
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang <geliangtang@163.com>
---
drivers/edac/i5100_edac.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/edac/i5100_edac.c b/drivers/edac/i5100_edac.c
index 4091777..c655162 100644
--- a/drivers/edac/i5100_edac.c
+++ b/drivers/edac/i5100_edac.c
@@ -575,9 +575,7 @@ static void i5100_check_error(struct mem_ctl_info *mci)
static void i5100_refresh_scrubbing(struct work_struct *work)
{
- struct delayed_work *i5100_scrubbing = container_of(work,
- struct delayed_work,
- work);
+ struct delayed_work *i5100_scrubbing = to_delayed_work(work);
struct i5100_priv *priv = container_of(i5100_scrubbing,
struct i5100_priv,
i5100_scrubbing);
--
2.5.0
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 2/7] edac: i5100_edac: use to_delayed_work
2016-01-01 14:59 ` [PATCH 2/7] edac: i5100_edac: " Geliang Tang
@ 2016-01-01 17:35 ` Borislav Petkov
0 siblings, 0 replies; 12+ messages in thread
From: Borislav Petkov @ 2016-01-01 17:35 UTC (permalink / raw)
To: Geliang Tang
Cc: Doug Thompson, Mauro Carvalho Chehab, linux-edac, linux-kernel
On Fri, Jan 01, 2016 at 10:59:07PM +0800, Geliang Tang wrote:
> Use to_delayed_work() instead of open-coding it.
>
> Signed-off-by: Geliang Tang <geliangtang@163.com>
> ---
> drivers/edac/i5100_edac.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/edac/i5100_edac.c b/drivers/edac/i5100_edac.c
> index 4091777..c655162 100644
> --- a/drivers/edac/i5100_edac.c
> +++ b/drivers/edac/i5100_edac.c
> @@ -575,9 +575,7 @@ static void i5100_check_error(struct mem_ctl_info *mci)
>
> static void i5100_refresh_scrubbing(struct work_struct *work)
> {
> - struct delayed_work *i5100_scrubbing = container_of(work,
> - struct delayed_work,
> - work);
> + struct delayed_work *i5100_scrubbing = to_delayed_work(work);
> struct i5100_priv *priv = container_of(i5100_scrubbing,
> struct i5100_priv,
> i5100_scrubbing);
> --
Applied, thanks.
--
Regards/Gruss,
Boris.
ECO tip #101: Trim your mails when you reply.
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 3/7] input: touchscreen: use to_delayed_work
2016-01-01 14:59 [PATCH 1/7] char: xillybus: use to_delayed_work Geliang Tang
2016-01-01 14:59 ` [PATCH 2/7] edac: i5100_edac: " Geliang Tang
@ 2016-01-01 14:59 ` Geliang Tang
2016-01-03 5:07 ` Dmitry Torokhov
2016-01-01 14:59 ` [PATCH 4/7] mmc: sh_mmcif: " Geliang Tang
` (4 subsequent siblings)
6 siblings, 1 reply; 12+ messages in thread
From: Geliang Tang @ 2016-01-01 14:59 UTC (permalink / raw)
To: Dmitry Torokhov; +Cc: Geliang Tang, linux-input, linux-kernel
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang <geliangtang@163.com>
---
drivers/input/touchscreen/pcap_ts.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/input/touchscreen/pcap_ts.c b/drivers/input/touchscreen/pcap_ts.c
index 23a354a..0e3fc41 100644
--- a/drivers/input/touchscreen/pcap_ts.c
+++ b/drivers/input/touchscreen/pcap_ts.c
@@ -87,7 +87,7 @@ static void pcap_ts_read_xy(void *data, u16 res[2])
static void pcap_ts_work(struct work_struct *work)
{
- struct delayed_work *dw = container_of(work, struct delayed_work, work);
+ struct delayed_work *dw = to_delayed_work(work);
struct pcap_ts *pcap_ts = container_of(dw, struct pcap_ts, work);
u8 ch[2];
--
2.5.0
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 3/7] input: touchscreen: use to_delayed_work
2016-01-01 14:59 ` [PATCH 3/7] input: touchscreen: " Geliang Tang
@ 2016-01-03 5:07 ` Dmitry Torokhov
0 siblings, 0 replies; 12+ messages in thread
From: Dmitry Torokhov @ 2016-01-03 5:07 UTC (permalink / raw)
To: Geliang Tang; +Cc: linux-input, linux-kernel
On Fri, Jan 01, 2016 at 10:59:08PM +0800, Geliang Tang wrote:
> Use to_delayed_work() instead of open-coding it.
>
> Signed-off-by: Geliang Tang <geliangtang@163.com>
Applied, thank you.
> ---
> drivers/input/touchscreen/pcap_ts.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/input/touchscreen/pcap_ts.c b/drivers/input/touchscreen/pcap_ts.c
> index 23a354a..0e3fc41 100644
> --- a/drivers/input/touchscreen/pcap_ts.c
> +++ b/drivers/input/touchscreen/pcap_ts.c
> @@ -87,7 +87,7 @@ static void pcap_ts_read_xy(void *data, u16 res[2])
>
> static void pcap_ts_work(struct work_struct *work)
> {
> - struct delayed_work *dw = container_of(work, struct delayed_work, work);
> + struct delayed_work *dw = to_delayed_work(work);
> struct pcap_ts *pcap_ts = container_of(dw, struct pcap_ts, work);
> u8 ch[2];
>
> --
> 2.5.0
>
>
--
Dmitry
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 4/7] mmc: sh_mmcif: use to_delayed_work
2016-01-01 14:59 [PATCH 1/7] char: xillybus: use to_delayed_work Geliang Tang
2016-01-01 14:59 ` [PATCH 2/7] edac: i5100_edac: " Geliang Tang
2016-01-01 14:59 ` [PATCH 3/7] input: touchscreen: " Geliang Tang
@ 2016-01-01 14:59 ` Geliang Tang
2016-01-27 14:15 ` Ulf Hansson
2016-01-01 14:59 ` [PATCH 5/7] mmc: usdhi6rol0: " Geliang Tang
` (3 subsequent siblings)
6 siblings, 1 reply; 12+ messages in thread
From: Geliang Tang @ 2016-01-01 14:59 UTC (permalink / raw)
To: Ulf Hansson, Kuninori Morimoto; +Cc: Geliang Tang, linux-mmc, linux-kernel
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang <geliangtang@163.com>
---
drivers/mmc/host/sh_mmcif.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mmc/host/sh_mmcif.c b/drivers/mmc/host/sh_mmcif.c
index 1ca8a13..e0c076a 100644
--- a/drivers/mmc/host/sh_mmcif.c
+++ b/drivers/mmc/host/sh_mmcif.c
@@ -1395,7 +1395,7 @@ static irqreturn_t sh_mmcif_intr(int irq, void *dev_id)
static void sh_mmcif_timeout_work(struct work_struct *work)
{
- struct delayed_work *d = container_of(work, struct delayed_work, work);
+ struct delayed_work *d = to_delayed_work(work);
struct sh_mmcif_host *host = container_of(d, struct sh_mmcif_host, timeout_work);
struct mmc_request *mrq = host->mrq;
struct device *dev = sh_mmcif_host_to_dev(host);
--
2.5.0
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 4/7] mmc: sh_mmcif: use to_delayed_work
2016-01-01 14:59 ` [PATCH 4/7] mmc: sh_mmcif: " Geliang Tang
@ 2016-01-27 14:15 ` Ulf Hansson
0 siblings, 0 replies; 12+ messages in thread
From: Ulf Hansson @ 2016-01-27 14:15 UTC (permalink / raw)
To: Geliang Tang; +Cc: Kuninori Morimoto, linux-mmc, linux-kernel
On 1 January 2016 at 15:59, Geliang Tang <geliangtang@163.com> wrote:
> Use to_delayed_work() instead of open-coding it.
>
> Signed-off-by: Geliang Tang <geliangtang@163.com>
Thanks, applied for next.
Kind regards
Uffe
> ---
> drivers/mmc/host/sh_mmcif.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mmc/host/sh_mmcif.c b/drivers/mmc/host/sh_mmcif.c
> index 1ca8a13..e0c076a 100644
> --- a/drivers/mmc/host/sh_mmcif.c
> +++ b/drivers/mmc/host/sh_mmcif.c
> @@ -1395,7 +1395,7 @@ static irqreturn_t sh_mmcif_intr(int irq, void *dev_id)
>
> static void sh_mmcif_timeout_work(struct work_struct *work)
> {
> - struct delayed_work *d = container_of(work, struct delayed_work, work);
> + struct delayed_work *d = to_delayed_work(work);
> struct sh_mmcif_host *host = container_of(d, struct sh_mmcif_host, timeout_work);
> struct mmc_request *mrq = host->mrq;
> struct device *dev = sh_mmcif_host_to_dev(host);
> --
> 2.5.0
>
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 5/7] mmc: usdhi6rol0: use to_delayed_work
2016-01-01 14:59 [PATCH 1/7] char: xillybus: use to_delayed_work Geliang Tang
` (2 preceding siblings ...)
2016-01-01 14:59 ` [PATCH 4/7] mmc: sh_mmcif: " Geliang Tang
@ 2016-01-01 14:59 ` Geliang Tang
2016-01-27 14:15 ` Ulf Hansson
2016-01-01 14:59 ` [PATCH 6/7] power: generic-adc-battery: " Geliang Tang
` (2 subsequent siblings)
6 siblings, 1 reply; 12+ messages in thread
From: Geliang Tang @ 2016-01-01 14:59 UTC (permalink / raw)
To: Ulf Hansson, Rabin Vincent; +Cc: Geliang Tang, linux-mmc, linux-kernel
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang <geliangtang@163.com>
---
drivers/mmc/host/usdhi6rol0.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mmc/host/usdhi6rol0.c b/drivers/mmc/host/usdhi6rol0.c
index b47122d..b2752fe 100644
--- a/drivers/mmc/host/usdhi6rol0.c
+++ b/drivers/mmc/host/usdhi6rol0.c
@@ -1630,7 +1630,7 @@ static irqreturn_t usdhi6_cd(int irq, void *dev_id)
*/
static void usdhi6_timeout_work(struct work_struct *work)
{
- struct delayed_work *d = container_of(work, struct delayed_work, work);
+ struct delayed_work *d = to_delayed_work(work);
struct usdhi6_host *host = container_of(d, struct usdhi6_host, timeout_work);
struct mmc_request *mrq = host->mrq;
struct mmc_data *data = mrq ? mrq->data : NULL;
--
2.5.0
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 5/7] mmc: usdhi6rol0: use to_delayed_work
2016-01-01 14:59 ` [PATCH 5/7] mmc: usdhi6rol0: " Geliang Tang
@ 2016-01-27 14:15 ` Ulf Hansson
0 siblings, 0 replies; 12+ messages in thread
From: Ulf Hansson @ 2016-01-27 14:15 UTC (permalink / raw)
To: Geliang Tang; +Cc: Rabin Vincent, linux-mmc, linux-kernel
On 1 January 2016 at 15:59, Geliang Tang <geliangtang@163.com> wrote:
> Use to_delayed_work() instead of open-coding it.
>
> Signed-off-by: Geliang Tang <geliangtang@163.com>
Thanks, applied for next.
Kind regards
Uffe
> ---
> drivers/mmc/host/usdhi6rol0.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mmc/host/usdhi6rol0.c b/drivers/mmc/host/usdhi6rol0.c
> index b47122d..b2752fe 100644
> --- a/drivers/mmc/host/usdhi6rol0.c
> +++ b/drivers/mmc/host/usdhi6rol0.c
> @@ -1630,7 +1630,7 @@ static irqreturn_t usdhi6_cd(int irq, void *dev_id)
> */
> static void usdhi6_timeout_work(struct work_struct *work)
> {
> - struct delayed_work *d = container_of(work, struct delayed_work, work);
> + struct delayed_work *d = to_delayed_work(work);
> struct usdhi6_host *host = container_of(d, struct usdhi6_host, timeout_work);
> struct mmc_request *mrq = host->mrq;
> struct mmc_data *data = mrq ? mrq->data : NULL;
> --
> 2.5.0
>
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 6/7] power: generic-adc-battery: use to_delayed_work
2016-01-01 14:59 [PATCH 1/7] char: xillybus: use to_delayed_work Geliang Tang
` (3 preceding siblings ...)
2016-01-01 14:59 ` [PATCH 5/7] mmc: usdhi6rol0: " Geliang Tang
@ 2016-01-01 14:59 ` Geliang Tang
2016-01-01 14:59 ` [PATCH 7/7] mvsas: " Geliang Tang
2016-01-01 19:47 ` [PATCH 1/7] char: xillybus: " Eli Billauer
6 siblings, 0 replies; 12+ messages in thread
From: Geliang Tang @ 2016-01-01 14:59 UTC (permalink / raw)
To: Sebastian Reichel, Dmitry Eremin-Solenikov, David Woodhouse
Cc: Geliang Tang, linux-pm, linux-kernel
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang <geliangtang@163.com>
---
drivers/power/generic-adc-battery.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/generic-adc-battery.c b/drivers/power/generic-adc-battery.c
index fedc581..edb36bf 100644
--- a/drivers/power/generic-adc-battery.c
+++ b/drivers/power/generic-adc-battery.c
@@ -206,7 +206,7 @@ static void gab_work(struct work_struct *work)
bool is_plugged;
int status;
- delayed_work = container_of(work, struct delayed_work, work);
+ delayed_work = to_delayed_work(work);
adc_bat = container_of(delayed_work, struct gab, bat_work);
pdata = adc_bat->pdata;
status = adc_bat->status;
--
2.5.0
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 7/7] mvsas: use to_delayed_work
2016-01-01 14:59 [PATCH 1/7] char: xillybus: use to_delayed_work Geliang Tang
` (4 preceding siblings ...)
2016-01-01 14:59 ` [PATCH 6/7] power: generic-adc-battery: " Geliang Tang
@ 2016-01-01 14:59 ` Geliang Tang
2016-01-01 19:47 ` [PATCH 1/7] char: xillybus: " Eli Billauer
6 siblings, 0 replies; 12+ messages in thread
From: Geliang Tang @ 2016-01-01 14:59 UTC (permalink / raw)
To: James E.J. Bottomley, Martin K. Petersen, Johannes Thumshirn
Cc: Geliang Tang, linux-scsi, linux-kernel
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang <geliangtang@163.com>
---
drivers/scsi/mvsas/mv_sas.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/scsi/mvsas/mv_sas.c b/drivers/scsi/mvsas/mv_sas.c
index e712fe7..78bf557 100644
--- a/drivers/scsi/mvsas/mv_sas.c
+++ b/drivers/scsi/mvsas/mv_sas.c
@@ -1887,7 +1887,7 @@ static void mvs_phy_disconnected(struct mvs_phy *phy)
static void mvs_work_queue(struct work_struct *work)
{
- struct delayed_work *dw = container_of(work, struct delayed_work, work);
+ struct delayed_work *dw = to_delayed_work(work);
struct mvs_wq *mwq = container_of(dw, struct mvs_wq, work_q);
struct mvs_info *mvi = mwq->mvi;
unsigned long flags;
--
2.5.0
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 1/7] char: xillybus: use to_delayed_work
2016-01-01 14:59 [PATCH 1/7] char: xillybus: use to_delayed_work Geliang Tang
` (5 preceding siblings ...)
2016-01-01 14:59 ` [PATCH 7/7] mvsas: " Geliang Tang
@ 2016-01-01 19:47 ` Eli Billauer
6 siblings, 0 replies; 12+ messages in thread
From: Eli Billauer @ 2016-01-01 19:47 UTC (permalink / raw)
To: Geliang Tang; +Cc: linux-kernel
Hello,
It's fine with me, but this patch needs to go to the char and misc
drivers maintainers (i.e. Arnd Bergmann and Greg Kroah-Hartman). I don't
have a repository of my own.
Thanks,
Eli
Acked-by: Eli Billauer <eli.billauer@gmail.com>
On 01/01/16 16:59, Geliang Tang wrote:
> Use to_delayed_work() instead of open-coding it.
>
> Signed-off-by: Geliang Tang<geliangtang@163.com>
> ---
> drivers/char/xillybus/xillybus_core.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/char/xillybus/xillybus_core.c b/drivers/char/xillybus/xillybus_core.c
> index 77d6c12..3bd36e0 100644
> --- a/drivers/char/xillybus/xillybus_core.c
> +++ b/drivers/char/xillybus/xillybus_core.c
> @@ -1164,8 +1164,7 @@ static int xillybus_flush(struct file *filp, fl_owner_t id)
>
> static void xillybus_autoflush(struct work_struct *work)
> {
> - struct delayed_work *workitem = container_of(
> - work, struct delayed_work, work);
> + struct delayed_work *workitem = to_delayed_work(work);
> struct xilly_channel *channel = container_of(
> workitem, struct xilly_channel, rd_workitem);
> int rc;
>
^ permalink raw reply [flat|nested] 12+ messages in thread