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

Reply via email to