From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f52.google.com (mail-ej1-f52.google.com [209.85.218.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 130F1280CC1 for ; Mon, 18 Aug 2025 07:44:42 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.52 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1755503086; cv=none; b=hv0my1F37vTk3cYMRJ7pjXHCclaCweSEuMQFmUOO6c1Y5pgdK39375V+Q0TRvKZeVaZzAhTyw4gixSyc3uvEqy/KT6fUnu81uKw87kQKmdVxZf8VWoyA0JyVI60ximf+hwQ93kYGytmTmXbO3CyLnoNtVKUh1mzVgCsWoefniNU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1755503086; c=relaxed/simple; bh=jO41a6HejjBPXSF3nuomJXZzUSmXK9fA1x8Yvu3KM8Y=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=BDG2bA3QD69zDFNRVejdIBuWq+ym/3C58YMcg4w7BSjy3vshb6ZqgXLK55Z5SEJ3eUxidtKiMJcroBThcmy4iRZ8io0fbH/DBG4lzWYlN6kP0Uu1E58VD28DhtRMXRJFXVloC3k+n3dGocEn2bMnqao1aGPzeDYGeRSGJsVh6gU= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=xNTwlCMH; arc=none smtp.client-ip=209.85.218.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="xNTwlCMH" Received: by mail-ej1-f52.google.com with SMTP id a640c23a62f3a-afcb6856dfbso663779766b.1 for ; Mon, 18 Aug 2025 00:44:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1755503081; x=1756107881; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=0q3D1hCM7sezdvpVcrp0vH4dqDUryB04vg1xMtc6bwU=; b=xNTwlCMHBXYmuzYqVCHoSOTrT9mBaaG6SL1rT6wJBN8S4OK5kREyVtELyDNwF0hBnf VbuT4UaOvQJyzR0jtpB0V+uXXGV1V0rd+0+fFbB2UDsGsX6/h9XgzsiqDCWoeIAgOb3n 6odwMT2FmLXYD4gUKDbItWpfdmYDvz9CHOXXCJteROarBTvxcA7RYgHFiFR87RPdjPlB wT8x7ku8j604BacDjmghAM+ZRU2pt47lAmhZn1/+mZyI4SC1wLxGHmedO/RTZYKdRz9S nFBwPKvrw5n6qO+m5H7+mTKf7+JqUeObO3dXZWFkp9KhbIi2sViom5aoyTpc7Q9WG5Qj kJwA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1755503081; x=1756107881; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=0q3D1hCM7sezdvpVcrp0vH4dqDUryB04vg1xMtc6bwU=; b=r5lopNkKWsJWvgo8nLedWyjxQw91cAI9JBs/TBw8JfpAin3s6J0UaGm+44pE02Ck+t MXJ9sgu64Whhe15gODlTxpLUxcCXSFlg77X/iVzDDv3SMmgFKwwQD5gPLbHtKmOJMeS/ 5/ZziVh8YRpTztafS5rPmFjWL5O7O/+2XTNWkYuTRAI5N/lB1FadqmquhN5Muxhd4CVf qkt8NVMEOOvWxk4IAWI2RmREnTAoBT5x9zbDLpVCzpxaaDRxAloDBecvtfT/QMEyzB8k XUSggYf7Jha4amZPVt137qi5+T4KYd7vI+yWd2/L0jOU7/ceMnzhq62imGqMusq9y9l4 tr/Q== X-Forwarded-Encrypted: i=1; AJvYcCUR+Tbdq54QvCDHFjlTR8oOM3mc4pTTdAYt9NcjXYyhuDm3PPEFuaX5MeNeOLorhLXVHtQSverHAGRnkou9@vger.kernel.org X-Gm-Message-State: AOJu0YzCr2SwTuC5W9XmVlNK4VKv+mpvdQm1AC8ZBAqdRlP1VJjVyMBR VhSPD1gbsAu0wozSM+AIHStiKcefDuHzdnQfnfDij7OFJPWpCQ+3RCWZTXo9VFmwrHE= X-Gm-Gg: ASbGncsicg45tySjVdkH7o6qgQZIHKGXk6NzeLAjFkPvOk0OLvaENUeGnWlbSEaxJLd d90BWcU3zQeih9h4afBgeCDw9KjcjeKpIjcAyYXcSEMcKrgsDiw9h9mhDHnsXj3VNzbDwsTYEpm mnrIFudcDGVWSjtNyKZPxS7chEBMkVdeMa6NGg5N9roMkcHYNebrajXJWxCiVdcDQi+9CZlA9qe d4Q70isMSdeLTNNKg2eZqaXlGH2chgXwK7ce+nD8BPs/OFq+SXfVt/4IPyNjahjX90DXMPOL2zG 9pwNRGywBEfM5xf0wNg1mXCOaCzs4nfJhomf9L7DG+9oafbEHR06meKYOqwqqd3mi/1TtOIhoCL Py2qlQTVkId3WiwbWTKiMCHSyTwPgnM0HhA== X-Google-Smtp-Source: AGHT+IFYRa5/G/vkuyAEvzPFOvv12naV5coa7rSBsKS6OmFNdjfJsZKTFi2E19B+6H4KXxIuA2nubw== X-Received: by 2002:a17:907:7f90:b0:afb:513a:f62f with SMTP id a640c23a62f3a-afcbd8974c4mr1402578566b.20.1755503081196; Mon, 18 Aug 2025 00:44:41 -0700 (PDT) Received: from linaro.org ([2a02:2454:ff21:ef30:6db9:9ad4:f47a:99b]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-afcdd010e1bsm733850266b.93.2025.08.18.00.44.40 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 18 Aug 2025 00:44:40 -0700 (PDT) Date: Mon, 18 Aug 2025 09:44:36 +0200 From: Stephan Gerhold To: Dmitry Baryshkov , Bjorn Andersson , Konrad Dybcio Cc: Aleksandrs Vinarskis , Abel Vesa , Xilin Wu , Jens Glathe , Srinivas Kandagatla , Sibi Sankar , Rajendra Nayak , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Johan Hovold , Christopher Obbard Subject: Re: [PATCH 1/9] arm64: dts: qcom: x1-asus-zenbook-a14: Add missing pinctrl for eDP HPD Message-ID: References: <20250814-x1e80100-add-edp-hpd-v1-0-a52804db53f6@linaro.org> <20250814-x1e80100-add-edp-hpd-v1-1-a52804db53f6@linaro.org> Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Sat, Aug 16, 2025 at 01:06:50AM +0300, Dmitry Baryshkov wrote: > On Thu, Aug 14, 2025 at 03:30:28PM +0200, Stephan Gerhold wrote: > > At the moment, we indirectly rely on the boot firmware to set up the > > pinctrl for the eDP HPD line coming from the internal display. If the boot > > firmware does not configure the display (e.g. because a different display > > is selected for output in the UEFI settings), then the display fails to > > come up and there are several errors in the kernel log: > > > > [drm:dpu_encoder_phys_vid_wait_for_commit_done:544] [dpu error]vblank timeout: 80020041 > > [drm:dpu_kms_wait_for_commit_done:524] [dpu error]wait for commit done returned -110 > > [drm:dpu_encoder_frame_done_timeout:2715] [dpu error]enc40 frame done timeout > > ... > > > > Fix this by adding the missing pinctrl for gpio119 (func1/edp0_hot and > > bias-disable according to the ACPI DSDT). > > > > Fixes: 6516961352a1 ("arm64: dts: qcom: Add support for X1-based Asus Zenbook A14") > > Signed-off-by: Stephan Gerhold > > --- > > arch/arm64/boot/dts/qcom/x1-asus-zenbook-a14.dtsi | 9 +++++++++ > > 1 file changed, 9 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/qcom/x1-asus-zenbook-a14.dtsi b/arch/arm64/boot/dts/qcom/x1-asus-zenbook-a14.dtsi > > index 16d045cf64c08c02c420787e000f4f45cfc2c6ff..613c675aac296f931293a1ba3d8506c6663bad21 100644 > > --- a/arch/arm64/boot/dts/qcom/x1-asus-zenbook-a14.dtsi > > +++ b/arch/arm64/boot/dts/qcom/x1-asus-zenbook-a14.dtsi > > @@ -1001,6 +1001,9 @@ &mdss_dp1_out { > > &mdss_dp3 { > > /delete-property/ #sound-dai-cells; > > > > + pinctrl-0 = <&edp_hpd_default>; > > + pinctrl-names = "default"; > > + > > status = "okay"; > > > > aux-bus { > > @@ -1236,6 +1239,12 @@ cam_indicator_en: cam-indicator-en-state { > > bias-disable; > > }; > > > > + edp_hpd_default: edp-hpd-default-state { > > + pins = "gpio119"; > > + function = "edp0_hot"; > > + bias-disable; > > + }; > > I think this is common enough. Let's maybe push this into the SoC dtsi > instead of copying it to all devices? > I had it there before, but Johan commented on the patch set from Chris that he would prefer to keep the potentially board-specific pinctrl out of the SoC dtsi [1]. So I can either address his feedback or yours. :-) There isn't really a convention for X1E either - we have a wild mix where some pinctrl is defined in the SoC dtsi (UART, I2C, SDHCI, ...) and others is copied for each board (e.g. PCIe). The reason I chose this approach is that I didn't feel it is guaranteed that the HPD pin has external pull down. It seems to be the case on most devices, but in theory a device could maybe rely on the internal pull down. Might be better to have it explicitly defined, the 5 additional lines are not that much at the end. I'm also fine with moving it. Perhaps Bjorn or Konrad could comment if they would prefer to have the pinctrl template defined in the SoC dtsi? Thanks, Stephan [1]: https://lore.kernel.org/linux-arm-msm/Z_kB3jOH04-zFnym@hovoldconsulting.com/