[PATCH v5 11/12] arm64: dts: qcom: sc8280xp-crd: Enable EDP
Johan Hovold
johan at kernel.org
Tue Dec 13 16:26:36 UTC 2022
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
More information about the dri-devel
mailing list