Re: [PATCH 0/2] media: imx-mipi-csis: csis clock fixes

[Tomi Valkeinen]

On 22/11/2023 15:21, Fabio Estevam wrote:
> Hi Tomi,
>
> On Wed, Nov 22, 2023 at 10:14 AM Tomi Valkeinen
> <tomi.valkeinen@xxxxxxxxxxxxxxxx> wrote:
> > Two fixes to the csis driver: One to fix remove() another to only enable
> > the clocks when needed.
> >
> > Signed-off-by: Tomi Valkeinen <tomi.valkeinen@xxxxxxxxxxxxxxxx>
> > ---
> > Tomi Valkeinen (2):
> >        media: imx-mipi-csis: Fix clock handling in remove()
> >        media: imx-mipi-csis: Drop extra clock enable at probe()
>
> Shouldn't both patches contain a Fixes tag?

I think the issue is there in the original commit adding the driver:

7807063b862b ("media: staging/imx7: add MIPI CSI-2 receiver subdev for 
i.MX7")

However, the driver has changed along the way, and I'm not sure if the 
original one had an actual bug. Nevertheless, the same pattern (wrt. 
clocks and runtime) is there in the original one, and I think that 
pattern is not correct even if it wouldn't have caused any visible issue.

So I'll add that commit as Fixes-tag, but if someone with more knowledge 
about the driver can verify this, that'd be great.

 Tomi