On Fri, Sep 12, 2025 at 02:11:20PM +0800, Peng Fan wrote:
> On Thu, Sep 11, 2025 at 01:13:59AM +0000, Peng Fan wrote:
> >Hi Frank,
> >>
> >> Can you remove 'method' in data struct also?
> >
> >The method is used in other places and other purpose, imx_rproc_detach
> >imx_rproc_put_scu, imx_rproc_remove, it is also referred
> >imx_dsp_rproc.c.
> >
> >Could we keep it for now?
>
> The method could not be removed from the data structure, because it is also
> used in imx_dsp_rproc.c.
>
> I have a few more patches to do further cleanup, but that would make
> the patchset a bit larger. I would like to see Mathieu's view.
>
> Mathieu,
>
> Do you expect me to add more patches in V3 to cleanup other parts or
> we could keep the patchset size as it is, with further cleanup in
> a standalone new patchset?
>
I think there is enough in this set. I will look at it next week.
> Thanks,
> Peng.
>
>
> >
> >Thanks,
> >Peng.
> >
> >>
> >> Frank
> >> > - case IMX_RPROC_NONE:
> >> > + /*
> >> > + * To i.MX{7,8} ULP, Linux is under control of RTOS, no need
> >> > + * dcfg->ops or dcfg->ops->detect_mode, it is state
> >> RPROC_DETACHED.
> >> > + */
> >> > + if (!dcfg->ops || !dcfg->ops->detect_mode) {
> >> > priv->rproc->state = RPROC_DETACHED;
> >> > return 0;
> >> > - default:
> >> > - break;
> >> > }
> >> >
> >> > - return 0;
> >> > + return dcfg->ops->detect_mode(priv->rproc);
> >> > }
> >> >
> >> > static int imx_rproc_clk_enable(struct imx_rproc *priv)
> >> >
> >> > --
> >> > 2.37.1
> >> >
