From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mout01.posteo.de (mout01.posteo.de [185.67.36.65]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id EE9238248B for ; Sat, 7 Mar 2026 17:20:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.67.36.65 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772904012; cv=none; b=Dc7j38SfIcqdRq8SkawSi8BiVJlbvzyfjBUYmL0neCPWAPbksBxDMetsYSICEHrY3BHTDyH3cfmvuTDDrr9WnBBbzAwPmpJ58qWVPB+pomu0M/mVHepvlOverB27uwKLGa8syU2xYEWzZKAhXRjItrjAw36AhXNIZbmZcMCNXHU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772904012; c=relaxed/simple; bh=41c5YB62NkVj4QXCQGXjaY+g6aVFcAXV/YuneE0W+eA=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=D4CxpEWjKAFOR0VUy5N2h2dWlBRMfOabNBzoqewidsWuBbBuLVunl4BupDPbOOQaYL+r3Qb6Q1+72bljyMqy0VIAJBg78a5aGipMCsHsV6n36dYb1A+/EX48QyE8+H/O5SE/9cGcFwqPi7Mg3CCSDL7Yn9ppfp3kC8fSdcKAoEk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=posteo.de; spf=pass smtp.mailfrom=posteo.de; dkim=pass (2048-bit key) header.d=posteo.de header.i=@posteo.de header.b=DYyM21Js; arc=none smtp.client-ip=185.67.36.65 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=posteo.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=posteo.de Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=posteo.de header.i=@posteo.de header.b="DYyM21Js" Received: from submission (posteo.de [185.67.36.169]) by mout01.posteo.de (Postfix) with ESMTPS id 33BA1240027 for ; Sat, 7 Mar 2026 18:20:09 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=posteo.de; s=2017; t=1772904009; bh=TeLxd/uTrJmEOGEZ+zvsDQYaz9O1UZJAm/pt9kYs3jc=; h=Date:From:To:Cc:Subject:Message-ID:MIME-Version:Content-Type: Content-Disposition:From; b=DYyM21Jsw6HGFIq+iKmRwlI972EckAKed3Bet4l2g4AiDmrGs4m5bvpaDyaldwGfv 7AdVJWDQVVTRotuzxNbgk2iN3fI4ahR3rIS4YnGT0XGn/Nv3pFpekfupgg+sbiNx5J 3x62Z8SOJwryvCfgO3a5m9eh8JcndoEfUo10D7snLweSe4T/4HOqdD4oPtU+3iivZu OqVyCbkXHnGLuTMTInToJmoI7wmEZTwinJ7fp5go2wowxywBCYXimR81DB84dnPxjQ bDsaXKyoefyIcek9qMVsxcUYKRk+S5YggumoqGJaWVzJZvk/mRv54B6Q1MFi1p4ZvD 0RBdShfudgZuA== Received: from customer (localhost [127.0.0.1]) by submission (posteo.de) with ESMTPSA id 4fSqlq38qwz6tws; Sat, 7 Mar 2026 18:20:07 +0100 (CET) Date: Sat, 07 Mar 2026 17:20:08 +0000 From: Paul Adam To: Stephan Gerhold Cc: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Bjorn Andersson , Konrad Dybcio , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, ~postmarketos/upstreaming@lists.sr.ht, phone-devel@vger.kernel.org Subject: Re: [PATCH v3 3/3] arm64: dts: qcom: msm8916-wiko-chuppito: add initial devicetree Message-ID: References: <20260301-wiko-chuppito-v3-0-9b36a2a7aed5@posteo.de> <20260301-wiko-chuppito-v3-3-9b36a2a7aed5@posteo.de> Precedence: bulk X-Mailing-List: devicetree@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 26/03/05 10:47, Stephan Gerhold wrote: > Did you mean "sleep" instead of "suspend" here? By default, there is > "default", "init", "idle" and "sleep" [1], although drivers can request > custom pinctrl states as well. > > Note that even with "sleep", the pinctrl state will be rarely used on > Linux. Drivers need to explicitly call pinctrl_pm_select_sleep_state() > to trigger these, but only few drivers do that [2]. From a quick look, > I suspect that only the &sdhc_2 sleep pinctrl here is currently used on > Linux. > > That doesn't mean that you can't add these pinctrl states (the DT is not > supposed to be specific to one particular operating system), but it does > mean that most of these are effectively untested right now. I would > personally omit them. > > [1]: https://elixir.bootlin.com/linux/v7.0-rc1/source/include/linux/pinctrl/pinctrl-state.h#L36 > [2]: https://elixir.bootlin.com/linux/v7.0-rc1/A/ident/pinctrl_pm_select_sleep_state Thanks, I will remove them. [...] > Does this need to be always-on? Still works without, will take it out. Paul