* Adam Ford <aford173@xxxxxxxxx> [190826 13:05]: > On Fri, Aug 23, 2019 at 5:27 PM Tony Lindgren <tony@xxxxxxxxxxx> wrote: > > > > * Adam Ford <aford173@xxxxxxxxx> [190821 12:55]: > > > Based on Tony Lindgren's work for omap34xx, this patch applies the same > > > functionality to the AM3517. > > > > > > The following can be tested via sysfs with the following to ensure the SGX > > > module gets enabled and disabled properly: > > > > > > 0x00010201 > > > > > > Bus error > > > > OK thanks for doing that. > > > > > Note that this patch depends on the PRM rstctrl driver that has > > > been recently posted. If the child device driver(s) need to prevent > > > rstctrl reset on PM runtime suspend, the drivers need to increase > > > the usecount for the shared rstctrl reset that can be mapped also > > > for the child device(s) or accessed via dev->parent. > > > > Care to check the subject and description again.. I don't think > > this one uses rstctrl resets :) > > I am not entirely sure what your patch does, I just copied it from the > 34xx to am3517. Since the code is really yours, would it make sense > for you to push it and I respond with 'tested-by" message? I don't > really want to claim it. ;-) Looks like you copied the commit from am335x patch, not from the omap3 patch. But I'll just drop the info about reset driver. Regards, Tony
