* [PATCH v2 1/2] leds: as3645a: Fix quoted string split warning
@ 2017-12-14 17:37 Dan Murphy
[not found] ` <20171214173727.10070-1-dmurphy-l0cyMroinI0@public.gmane.org>
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Dan Murphy @ 2017-12-14 17:37 UTC (permalink / raw)
To: robh+dt, mark.rutland, rpurdie, jacek.anaszewski, pavel,
sakari.ailus, laurent.pinchart
Cc: devicetree, linux-kernel, linux-leds, Dan Murphy
Fix a warning for a split quoted string across
lines.
WARNING: quoted string split across lines
459: FILE: drivers/leds/leds-as3645a.c:459:
dev_err(dev, "AS3645A not detected "
"(model %d rfu %d)\n", model, rfu);
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Dan Murphy <dmurphy@ti.com>
---
v2 - Split the warning fixes into 2 patches and fixed commit message -
https://patchwork.kernel.org/patch/10108325/
drivers/leds/leds-as3645a.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/leds/leds-as3645a.c b/drivers/leds/leds-as3645a.c
index 9a257f969300..49fdd7c0a6f6 100644
--- a/drivers/leds/leds-as3645a.c
+++ b/drivers/leds/leds-as3645a.c
@@ -455,8 +455,8 @@ static int as3645a_detect(struct as3645a *flash)
/* Verify the chip model and version. */
if (model != 0x01 || rfu != 0x00) {
- dev_err(dev, "AS3645A not detected "
- "(model %d rfu %d)\n", model, rfu);
+ dev_err(dev, "AS3645A not detected (model %d rfu %d)\n",
+ model, rfu);
return -ENODEV;
}
--
2.15.0.124.g7668cbc60
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH v2 2/2] leds: as3645a: Fix line over 80 characters
[not found] ` <20171214173727.10070-1-dmurphy-l0cyMroinI0@public.gmane.org>
@ 2017-12-14 17:37 ` Dan Murphy
0 siblings, 0 replies; 4+ messages in thread
From: Dan Murphy @ 2017-12-14 17:37 UTC (permalink / raw)
To: robh+dt-DgEjT+Ai2ygdnm+yROfE0A, mark.rutland-5wv7dgnIgG8,
rpurdie-Fm38FmjxZ/leoWH0uzbU5w,
jacek.anaszewski-Re5JQEeQqe8AvxtiuMwx3w, pavel-+ZI9xUNit7I,
sakari.ailus-X3B1VOXEql0, laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-leds-u79uwXL29TY76Z2rM5mHXA, Dan Murphy
Fix the warning for line over 80 characters.
WARNING: line over 80 characters
363: FILE: drivers/leds/leds-as3645a.c:363:
flash->flash_current = as3645a_current_to_reg(flash, true, brightness_ua);
Acked-by: Laurent Pinchart <laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org>
Signed-off-by: Dan Murphy <dmurphy-l0cyMroinI0@public.gmane.org>
---
v2 - Split the warning fixes into 2 patches and fixed commit message -
https://patchwork.kernel.org/patch/10108325/
drivers/leds/leds-as3645a.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/leds/leds-as3645a.c b/drivers/leds/leds-as3645a.c
index 49fdd7c0a6f6..f883616d9e60 100644
--- a/drivers/leds/leds-as3645a.c
+++ b/drivers/leds/leds-as3645a.c
@@ -360,7 +360,8 @@ static int as3645a_set_flash_brightness(struct led_classdev_flash *fled,
{
struct as3645a *flash = fled_to_as3645a(fled);
- flash->flash_current = as3645a_current_to_reg(flash, true, brightness_ua);
+ flash->flash_current = as3645a_current_to_reg(flash, true,
+ brightness_ua);
return as3645a_set_current(flash);
}
--
2.15.0.124.g7668cbc60
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2 1/2] leds: as3645a: Fix quoted string split warning
2017-12-14 17:37 [PATCH v2 1/2] leds: as3645a: Fix quoted string split warning Dan Murphy
[not found] ` <20171214173727.10070-1-dmurphy-l0cyMroinI0@public.gmane.org>
@ 2017-12-14 18:04 ` Sakari Ailus
2017-12-14 21:11 ` Jacek Anaszewski
2 siblings, 0 replies; 4+ messages in thread
From: Sakari Ailus @ 2017-12-14 18:04 UTC (permalink / raw)
To: Dan Murphy
Cc: robh+dt, mark.rutland, rpurdie, jacek.anaszewski, pavel,
laurent.pinchart, devicetree, linux-kernel, linux-leds
On Thu, Dec 14, 2017 at 11:37:26AM -0600, Dan Murphy wrote:
> Fix a warning for a split quoted string across
> lines.
>
> WARNING: quoted string split across lines
> 459: FILE: drivers/leds/leds-as3645a.c:459:
> dev_err(dev, "AS3645A not detected "
> "(model %d rfu %d)\n", model, rfu);
>
> Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Signed-off-by: Dan Murphy <dmurphy@ti.com>
Thanks!
For both:
Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
--
Sakari Ailus
e-mail: sakari.ailus@iki.fi
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2 1/2] leds: as3645a: Fix quoted string split warning
2017-12-14 17:37 [PATCH v2 1/2] leds: as3645a: Fix quoted string split warning Dan Murphy
[not found] ` <20171214173727.10070-1-dmurphy-l0cyMroinI0@public.gmane.org>
2017-12-14 18:04 ` [PATCH v2 1/2] leds: as3645a: Fix quoted string split warning Sakari Ailus
@ 2017-12-14 21:11 ` Jacek Anaszewski
2 siblings, 0 replies; 4+ messages in thread
From: Jacek Anaszewski @ 2017-12-14 21:11 UTC (permalink / raw)
To: Dan Murphy, robh+dt, mark.rutland, rpurdie, pavel, sakari.ailus,
laurent.pinchart
Cc: devicetree, linux-kernel, linux-leds
Hi Dan,
Thanks for the patch set.
Bot patches applied to the for-next branch.
--
Best regards,
Jacek Anaszewski
On 12/14/2017 06:37 PM, Dan Murphy wrote:
> Fix a warning for a split quoted string across
> lines.
>
> WARNING: quoted string split across lines
> 459: FILE: drivers/leds/leds-as3645a.c:459:
> dev_err(dev, "AS3645A not detected "
> "(model %d rfu %d)\n", model, rfu);
>
> Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Signed-off-by: Dan Murphy <dmurphy@ti.com>
> ---
>
> v2 - Split the warning fixes into 2 patches and fixed commit message -
> https://patchwork.kernel.org/patch/10108325/
>
> drivers/leds/leds-as3645a.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/leds/leds-as3645a.c b/drivers/leds/leds-as3645a.c
> index 9a257f969300..49fdd7c0a6f6 100644
> --- a/drivers/leds/leds-as3645a.c
> +++ b/drivers/leds/leds-as3645a.c
> @@ -455,8 +455,8 @@ static int as3645a_detect(struct as3645a *flash)
>
> /* Verify the chip model and version. */
> if (model != 0x01 || rfu != 0x00) {
> - dev_err(dev, "AS3645A not detected "
> - "(model %d rfu %d)\n", model, rfu);
> + dev_err(dev, "AS3645A not detected (model %d rfu %d)\n",
> + model, rfu);
> return -ENODEV;
> }
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-12-14 21:12 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-12-14 17:37 [PATCH v2 1/2] leds: as3645a: Fix quoted string split warning Dan Murphy
[not found] ` <20171214173727.10070-1-dmurphy-l0cyMroinI0@public.gmane.org>
2017-12-14 17:37 ` [PATCH v2 2/2] leds: as3645a: Fix line over 80 characters Dan Murphy
2017-12-14 18:04 ` [PATCH v2 1/2] leds: as3645a: Fix quoted string split warning Sakari Ailus
2017-12-14 21:11 ` Jacek Anaszewski
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).