On 13.08.2021 23:23, Rob Herring wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > On Fri, Aug 06, 2021 at 02:29:57PM +0300, Claudiu Beznea wrote: >> Hi, >> >> This series adds support for selecting the delay b/w reset and power >> lines on pwrseq-sd8787. With this the driver could also be used by >> other devices (e.g. WILC1000/WILC3000 which needs a delay of 5ms). Along >> with this added the DT bindings for WILC3000 device present on >> SAMA5D27 WLSOM1 board. > > A 300ms delay doesn't work? It should work as the datasheet specifies a minimum of 5ms is necessary. > Is powering on the device so timing > critical? > > Add a new compatible for your device and imply the timing from that. > That way we're not extending the binding a property at a time in a > poorly designed way for each new timing quirk. OK, will do it in v2. Thank you for your review, Claudiu Beznea > > Rob >
