Hi, On Fri, Nov 22, 2013 at 11:41:27AM +0100, Javier Martinez Canillas wrote: > I dropped that patch from my series and posted a v2 that just name the > VPLL2 regulator as vdds_dsi [1]. That way will be safer for Tony and > Benoit to take this series as a fix for the -rc cycle since the > changes are contained within IGEP boards DTS.
I know, but the patch serves as example for me :)
> > My suggestion would be to add something like this pseudocode to
> > omapdss:
> >
> > if(board_is_n900_dt()) {
> > vdds_dsi = devm_regulator_get(&dpi.pdev->dev, "V28");
> > }
> >
> > The problem is, that we do not want to name the regulator
> > "vdds_dsi", since it's not used exclusivly for omapdss.
> >
> > In the future it can get the regulator via phandle of course.
>
> For what is worth I think that your suggestion is a good workaround.
> Please just add a comment specifying that it is a hack and that we
> have to get rid of these once the DSS DT bindings land on mainline and
> we can use a phandle to get the regulator.
I just sent a patchset, that adds DT panel support for N900.
-- Sebastian
signature.asc
Description: Digital signature
