Re: [PATCH v5 11/12] arm64: dts: qcom: sc8280xp-crd: Enable EDP

Johan Hovold
2022-12-13T16:26:20Z

On Tue, Dec 13, 2022 at 07:10:14AM -0800, Bjorn Andersson wrote:
> On Fri, Dec 09, 2022 at 11:35:23AM +0100, Johan Hovold wrote:

> > > + edp_reg_en: edp-reg-en-state {
> > > + pins = "gpio25";
> > > + function = "gpio";
> > > + output-enable;
> >
> > 'output-enable' is not valid for tlmm and causes the settings to be
> > rejected:
> >
> > sc8280xp-tlmm f100000.pinctrl: pin_config_group_set op failed for group 25
> > reg-fixed-voltage regulator-edp-3p3: Error applying setting, reverse things back
> >
>
> Thanks for spotting that, it doesn't seem to be needed for the gpio-regulator
> driver anyways...

I noticed that the firmware on both CRD and X13s sets the drive strength
to 16 here. Should we specify that too (and disable the pull up)
instead of relying on the firmware configuration?

Johan