[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <edde330e-516b-ecaa-4139-0bfa766a9c08@ti.com>
Date: Fri, 26 Jul 2019 09:38:57 -0500
From: Dan Murphy <dmurphy@...com>
To: Pavel Machek <pavel@....cz>
CC: <linux-omap@...r.kernel.org>, <tony@...mide.com>, <sre@...nel.org>,
<nekit1000@...il.com>, <mpartap@....net>, <merlijn@...zup.org>,
<jacek.anaszewski@...il.com>, <linux-kernel@...r.kernel.org>,
<linux-leds@...r.kernel.org>
Subject: Re: Backlight in motorola Droid 4
Pavel
On 7/24/19 10:22 AM, Dan Murphy wrote:
> Pavel
>
> On 7/24/19 7:45 AM, Pavel Machek wrote:
> <snip>
>>
>> +
>> static int lm3532_parse_node(struct lm3532_data *priv)
>> {
>> struct fwnode_handle *child = NULL;
>> @@ -536,11 +579,13 @@ static int lm3532_parse_node(struct lm3532_data
>> *priv)
>> ret = fwnode_property_read_u32(child, "ti,led-mode",
>> &led->mode);
>> if (ret) {
>> + /* FIXME: should just default to non-als mod */
>
> Looking at the rest of the code.
>
> The DT doc indicated that this was a required child property so this
> is why it
>
> errors out.
>
> Dan
>
> <snip>
>
Did you want me to fix up the LM3532 and send the patch?
Or do you have a patch ready to go?
Dan
Powered by blists - more mailing lists