Re: [PATCH 2/5] firmware: imx: scu-pd: Add power domains for imx-jpeg

[Daniel Baluta]

On Fri, 2019-12-13 at 10:49 +0100, Hans Verkuil wrote:
> On 11/6/19 4:20 PM, Mirela Rabulea wrote:
> > The power domains are for imx8qxp/imx8qm JPEG encoder & decoder.
> > Each has 4 slots and a wrapper.
> > 
> > Signed-off-by: Mirela Rabulea <mirela.rabulea@xxxxxxx>
> 
> Shawn, Daniel, can I have an Ack from you? It probably makes sense
> that
> this patch gets merged via the media subsystem.

I hope I'm the Daniel you are referring to. It looks good to  me.

Acked-by: Daniel Baluta <daniel.baluta@xxxxxxx>

> 
> Regards,
> 
> 	Hans
> 
> > ---
> >  drivers/firmware/imx/scu-pd.c | 6 ++++++
> >  1 file changed, 6 insertions(+)
> > 
> > diff --git a/drivers/firmware/imx/scu-pd.c
> > b/drivers/firmware/imx/scu-pd.c
> > index b556612..e7bf48a 100644
> > --- a/drivers/firmware/imx/scu-pd.c
> > +++ b/drivers/firmware/imx/scu-pd.c
> > @@ -158,6 +158,12 @@ static const struct imx_sc_pd_range
> > imx8qxp_scu_pd_ranges[] = {
> >  	/* DC SS */
> >  	{ "dc0", IMX_SC_R_DC_0, 1, false, 0 },
> >  	{ "dc0-pll", IMX_SC_R_DC_0_PLL_0, 2, true, 0 },
> > +
> > +	/* IMAGE SS */
> > +	{ "img-jpegdec-mp", IMX_SC_R_MJPEG_DEC_MP, 1, false, 0 },
> > +	{ "img-jpegdec-s0", IMX_SC_R_MJPEG_DEC_S0, 4, true, 0 },
> > +	{ "img-jpegenc-mp", IMX_SC_R_MJPEG_ENC_MP, 1, false, 0 },
> > +	{ "img-jpegenc-s0", IMX_SC_R_MJPEG_ENC_S0, 4, true, 0 },
> >  };
> >  
> >  static const struct imx_sc_pd_soc imx8qxp_scu_pd = {
> > 
> 
>