Hi Tony, On Tuesday 10 July 2018 05:12 PM, Tony Lindgren wrote: > Hi, > > * Faiz Abbas <faiz_abbas@xxxxxx> [180709 16:50]: >> Add support for the software reset of a target interconnect >> module using its sysconfig and sysstatus registers. > ... >> --- a/drivers/bus/ti-sysc.c >> +++ b/drivers/bus/ti-sysc.c > ... >> +static int sysc_reset(struct sysc *ddata) >> +{ >> + int offset = ddata->offsets[SYSC_SYSCONFIG]; >> + int val = sysc_read(ddata, offset); > > Testing with omap4 sysc dts data I noticed we already > have modules where there is only rev register and no > SYSC_SYSCONFIG. So not using legacy mode. The > above offset with -ENODEV value will not work :) > > I added some more checks and moved them to the reset > function. Great. Thanks for the fix. Thanks, Faiz -- 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