On Mon, May 14, 2018 at 10:40 PM, Dan Murphy <dmurphy@xxxxxx> wrote: > On 05/11/2018 06:56 AM, Dan Murphy wrote: >>>> + ret = of_property_read_string(led->strobe_node, "label", &name); >>>> + ret = of_property_read_u32(led->strobe_node, >>>> + ret = of_property_read_u32(led->strobe_node, >>> Common LED bindings state that flash-max-microamp and >>> flash-max-timeout-us properties are mandatory. >> >> OK. > > OK I looked at the max776973 driver and well if the flash-max-microamp and > flash-max-timeout-us nodes are missing it sets a default value for each if the > node is not present. > > So should we remove this code from the Max77693 driver too and fail probe as being asked > in this driver? I would also add that using device_property_*() API is much better then using OF specific one. It will help IoT / DIY entusiasts use them on non-DT platforms. -- With Best Regards, Andy Shevchenko -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
