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?
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
>> >
