Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1193092 > unrolled thread
| Started by | Matt Fleming <matt@codeblueprint.co.uk> |
|---|---|
| First post | 2015-07-27 15:40 +0200 |
| Last post | 2015-07-28 11:20 +0200 |
| Articles | 3 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
[PATCH 5/5] i2c-i801: fixup regarding watchdog timer Matt Fleming <matt@codeblueprint.co.uk> - 2015-07-27 15:40 +0200
Re: [PATCH 5/5] i2c-i801: fixup regarding watchdog timer Guenter Roeck <linux@roeck-us.net> - 2015-07-27 16:20 +0200
Re: [PATCH 5/5] i2c-i801: fixup regarding watchdog timer Matt Fleming <matt@codeblueprint.co.uk> - 2015-07-28 11:20 +0200
| From | Matt Fleming <matt@codeblueprint.co.uk> |
|---|---|
| Date | 2015-07-27 15:40 +0200 |
| Subject | [PATCH 5/5] i2c-i801: fixup regarding watchdog timer |
| Message-ID | <pQQNk-3iH-13@gated-at.bofh.it> |
From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Change &array[0] to array since it's the same.
It fixes 80 character limit warning as well.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: Jean Delvare <jdelvare@suse.com>
Cc: Wolfram Sang <wsa@the-dreams.de>
Signed-off-by: Matt Fleming <matt.fleming@intel.com>
---
drivers/i2c/busses/i2c-i801.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c
index c79dbe116ccc..e41005927746 100644
--- a/drivers/i2c/busses/i2c-i801.c
+++ b/drivers/i2c/busses/i2c-i801.c
@@ -1173,7 +1173,8 @@ static void i801_add_tco(struct i801_priv *priv)
if (!(tco_ctl & TCOCTL_EN))
return;
- memset(&tco_res[0], 0, sizeof(tco_res));
+ memset(tco_res, 0, sizeof(tco_res));
+
res = &tco_res[ICH_RES_IO_TCO];
res->start = tco_base & ~1;
res->end = res->start + 32 - 1;
@@ -1226,7 +1227,7 @@ static void i801_add_tco(struct i801_priv *priv)
res->flags = IORESOURCE_MEM;
pdev = platform_device_register_resndata(&pci_dev->dev, "iTCO_wdt", -1,
- &tco_res[0], 3, &tco_platform_data,
+ tco_res, 3, &tco_platform_data,
sizeof(tco_platform_data));
if (IS_ERR(pdev)) {
dev_warn(&pci_dev->dev, "failed to create iTCO device\n");
--
2.1.0
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[toc] | [next] | [standalone]
| From | Guenter Roeck <linux@roeck-us.net> |
|---|---|
| Date | 2015-07-27 16:20 +0200 |
| Message-ID | <pQRq3-4iB-43@gated-at.bofh.it> |
| In reply to | #1193092 |
On 07/27/2015 06:38 AM, Matt Fleming wrote:
> From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
>
> Change &array[0] to array since it's the same.
>
> It fixes 80 character limit warning as well.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> Cc: Jean Delvare <jdelvare@suse.com>
> Cc: Wolfram Sang <wsa@the-dreams.de>
> Signed-off-by: Matt Fleming <matt.fleming@intel.com>
Should be merged into patch 2.
Guenter
> ---
> drivers/i2c/busses/i2c-i801.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c
> index c79dbe116ccc..e41005927746 100644
> --- a/drivers/i2c/busses/i2c-i801.c
> +++ b/drivers/i2c/busses/i2c-i801.c
> @@ -1173,7 +1173,8 @@ static void i801_add_tco(struct i801_priv *priv)
> if (!(tco_ctl & TCOCTL_EN))
> return;
>
> - memset(&tco_res[0], 0, sizeof(tco_res));
> + memset(tco_res, 0, sizeof(tco_res));
> +
> res = &tco_res[ICH_RES_IO_TCO];
> res->start = tco_base & ~1;
> res->end = res->start + 32 - 1;
> @@ -1226,7 +1227,7 @@ static void i801_add_tco(struct i801_priv *priv)
> res->flags = IORESOURCE_MEM;
>
> pdev = platform_device_register_resndata(&pci_dev->dev, "iTCO_wdt", -1,
> - &tco_res[0], 3, &tco_platform_data,
> + tco_res, 3, &tco_platform_data,
> sizeof(tco_platform_data));
> if (IS_ERR(pdev)) {
> dev_warn(&pci_dev->dev, "failed to create iTCO device\n");
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Matt Fleming <matt@codeblueprint.co.uk> |
|---|---|
| Date | 2015-07-28 11:20 +0200 |
| Message-ID | <pR9dg-4Rk-1@gated-at.bofh.it> |
| In reply to | #1193170 |
On Mon, 27 Jul, at 07:14:08AM, Guenter Roeck wrote: > On 07/27/2015 06:38 AM, Matt Fleming wrote: > >From: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > > > >Change &array[0] to array since it's the same. > > > >It fixes 80 character limit warning as well. > > > >Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > >Cc: Jean Delvare <jdelvare@suse.com> > >Cc: Wolfram Sang <wsa@the-dreams.de> > >Signed-off-by: Matt Fleming <matt.fleming@intel.com> > > Should be merged into patch 2. OK, will do. -- Matt Fleming, Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web