On 12/3/18 11:44 AM, Sasha Levin wrote: > On Mon, Dec 03, 2018 at 11:30:29AM -0600, Dinh Nguyen wrote: >> From: Ramiro Oliveira <Ramiro.Oliveira@xxxxxxxxxxxx> >> >> commit bb475230b8e59a547ab66ac3b02572df21a580e9 upstream. >> >> The *_get_optional_* functions weren't really optional so this patch >> makes them really optional. >> >> These *_get_optional_* functions will now return NULL instead of an error >> if no matching reset phandle is found in the DT, and all the >> reset_control_* functions now accept NULL rstc pointers. >> >> Signed-off-by: Ramiro Oliveira <Ramiro.Oliveira@xxxxxxxxxxxx> >> Signed-off-by: Philipp Zabel <p.zabel@xxxxxxxxxxxxxx> >> --- >> Please apply to v4.9.y stable tree. > > The original commit had a few upstream fixes: > > bb6c7768385b reset: remove remaining WARN_ON() in <linux/reset.h> > 1554bbd4ad40 reset: make device_reset_optional() really optional > 62e24c5775ec reset: add exported __reset_control_get, return NULL if > optional > 0ca10b60ceeb reset: fix optional reset_control_get stubs to return NULL > 4891486fb2c8 reset: core: fix reset_control_put > > We'll need to backport them to 4.9 as well if you backport this commit. > Hmm..I was able to fix an ethernet driver load issue by just backporting this patch. Should I investigate backport of the other patches as well? Dinh