* Faiz Abbas <faiz_ab...@ti.com> [180606 06:14]:
> +static int sysc_reset(struct sysc *ddata)
> +{
> +     int offset = ddata->offsets[SYSC_SYSCONFIG];
> +     int val = sysc_read(ddata, offset);
> +
> +     val |= (0x1 << ddata->cap->regbits->srst_shift);
> +     sysc_write(ddata, offset, val);
> +
> +     /* Poll on reset status */
> +     if (ddata->cfg.quirks & SYSC_QUIRK_RESET_STATUS) {
> +             offset = ddata->offsets[SYSC_SYSSTATUS];
> +
> +             return readl_poll_timeout(ddata->module_va + offset, val,
> +                             (val & ddata->cfg.syss_mask) == 0x0,
> +                             100, MAX_MODULE_SOFTRESET_WAIT);
> +     }
> +
> +     return 0;
> +}

I wonder if we should also add SYSS_QUIRK_RESET_STATUS in
addition to SYSC_QUIRK_RESET status to make it easy to
read the right register?

Then we can add support for SYSC_QUIRK_RESET_STATUS later on
when tested and return error for now.

Regards,

Tony

Reply via email to