Re: [PATCHv5 10/10] ARM: OMAP2+: pdata-quirks: add PRM data for reset support

[Tero Kristo <t-kristo@xxxxxx>]

On 12/09/2019 20:09, Tony Lindgren wrote:
> * Tero Kristo <t-kristo@xxxxxx> [190912 11:40]:
> > @@ -565,6 +566,12 @@ void omap_pcs_legacy_init(int irq, void (*rearm)(void))
> >   	pcs_pdata.rearm = rearm;
> >   }
> >   
> > +static struct ti_prm_platform_data ti_prm_pdata = {
> > +	.clkdm_deny_idle = clkdm_deny_idle,
> > +	.clkdm_allow_idle = clkdm_allow_idle,
> > +	.clkdm_lookup = clkdm_lookup,
> > +};
> > +
> >   /*
> >    * GPIOs for TWL are initialized by the I2C bus and need custom
> >    * handing until DSS has device tree bindings.
> > @@ -664,6 +671,11 @@ static struct of_dev_auxdata omap_auxdata_lookup[] = {
> >   	/* Common auxdata */
> >   	OF_DEV_AUXDATA("ti,sysc", 0, NULL, &ti_sysc_pdata),
> >   	OF_DEV_AUXDATA("pinctrl-single", 0, NULL, &pcs_pdata),
> > +	OF_DEV_AUXDATA("ti,omap4-prm-inst", 0, NULL, &ti_prm_pdata),
> > +	OF_DEV_AUXDATA("ti,omap5-prm-inst", 0, NULL, &ti_prm_pdata),
> > +	OF_DEV_AUXDATA("ti,dra7-prm-inst", 0, NULL, &ti_prm_pdata),
> > +	OF_DEV_AUXDATA("ti,am3-prm-inst", 0, NULL, &ti_prm_pdata),
> > +	OF_DEV_AUXDATA("ti,am4-prm-inst", 0, NULL, &ti_prm_pdata),
> >   	{ /* sentinel */ },
> >   };
>
> Hmm I think I already commented on this.. Just one entry please:
>
> 	OF_DEV_AUXDATA("ti,omap-prm-inst", 0, NULL, &ti_prm_pdata),
>
> As the auxdata is the same for all of them. Note that all the
> dts files need to have also the generic compatible
> "ti,omap-prm-inst" after the SoC specific one.

Ok that should be fine, sorry for missing it out. I can update this in 
the next rev, just need to update the dt binding also.

-Tero
--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki