From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id CC3DAE77376 for ; Sat, 30 Sep 2023 15:37:37 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234289AbjI3Phi (ORCPT ); Sat, 30 Sep 2023 11:37:38 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35116 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234257AbjI3Phh (ORCPT ); Sat, 30 Sep 2023 11:37:37 -0400 Received: from mail-yw1-x112e.google.com (mail-yw1-x112e.google.com [IPv6:2607:f8b0:4864:20::112e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id EF708E6 for ; Sat, 30 Sep 2023 08:37:31 -0700 (PDT) Received: by mail-yw1-x112e.google.com with SMTP id 00721157ae682-5a1d0fee86aso113098177b3.2 for ; Sat, 30 Sep 2023 08:37:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1696088251; x=1696693051; darn=vger.kernel.org; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=Iij7GZPs4ta7mpI/DDP/sOC6B5pAs6vBvsF5JU59Dng=; b=sYzo6XUj/VGXsS/c+8iIR/YiKUJBwnimrOMTrD+9AUnOIlUUa+wIQpkqJpMyHu/bf3 d4+1cDx5AvDqd5noI/NmQZhb10XCf2l8YeZJ+vjZgD/EBXHWLMG0DGxPXcTDNouaD37Y KptpNMaeTeRLAfOvYv4NWBe6a6fVZvxUlO7i8GMXc8wVhc5FpGVv285qRKt5d9Yz6M4a 1jh5qc3Bj6I612sXZ9g4DcjZS0+7mfrkOudLL53a6W8DsPdyyCnMvvXD5BY4B3bJJJ5O /vNzBAyqjW70SEluDVl5R9zhVXvcSV6igSvORQ4pfcYfRmUEV0/CJ5ttuXWQddP3dS0N r+sw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1696088251; x=1696693051; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=Iij7GZPs4ta7mpI/DDP/sOC6B5pAs6vBvsF5JU59Dng=; b=lU2PuAB197ONwhWCoxZVaiIW8nUugAHzcaFFv740DmcUUX6z5aSdDcwfEyu7QSxx8X A/xgfeZ+eeBjptUaSTYz0/lBeC6dQ1OzHfZOlT5ojKVAMwhd7rUbEjHz9Nq15LGSLzzS XLeIGMnEoenvhQOFKyiMRYKEk8AC3WXo5B9meEuWyzNn1nwK9q8g7Ce2LSaYDP3ePkdH dhzTR7vFIsGbmWk6z6olgdqV4zynouBHENekWAwTZ82h/NVLxzeC9nVnE/G519vwdDle 0qPR/zjo2wEPcNy+g9nT1Jv475DiTmUTck/zGWS2CHUqGh/LpItG1FsczP2nxOZ/fphy /BFQ== X-Gm-Message-State: AOJu0YytgnQo5y7HASHGRzvd9yqRsL6SMNlyFi1n902eMUQ5Sh7dhdl0 ShHH7wJmA1JH7zlXb/XZsFT5oFaQ8aZ6lMyp06/OjA== X-Google-Smtp-Source: AGHT+IGFEGxNW9iad1m1hopLGKbkXI8hjBmpaUfewF+ozAfJ+lrOwUFw+LD1rcN0USj40MfR4GqG12cZk9y2+fvzoII= X-Received: by 2002:a0d:ddc1:0:b0:5a1:d4bc:7faa with SMTP id g184-20020a0dddc1000000b005a1d4bc7faamr7394744ywe.18.1696088251028; Sat, 30 Sep 2023 08:37:31 -0700 (PDT) MIME-Version: 1.0 References: <20230929-pxa1908-lkml-v5-0-5aa5a1109c5f@skole.hr> <20230929-pxa1908-lkml-v5-7-5aa5a1109c5f@skole.hr> <5715527.DvuYhMxLoT@radijator> In-Reply-To: <5715527.DvuYhMxLoT@radijator> From: Linus Walleij Date: Sat, 30 Sep 2023 17:37:19 +0200 Message-ID: Subject: Re: [PATCH RESEND v5 7/8] arm64: dts: Add DTS for Marvell PXA1908 and samsung,coreprimevelte To: =?UTF-8?Q?Duje_Mihanovi=C4=87?= , Chris Packham Cc: Robert Jarzmik , Bartosz Golaszewski , Andy Shevchenko , Michael Turquette , Stephen Boyd , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Lubomir Rintel , Catalin Marinas , Will Deacon , Kees Cook , Tony Luck , "Guilherme G. Piccoli" , linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-hardening@vger.kernel.org, ~postmarketos/upstreaming@lists.sr.ht, phone-devel@vger.kernel.org, afaerber@suse.de, balejk@matfyz.cz Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-gpio@vger.kernel.org On Sat, Sep 30, 2023 at 10:25=E2=80=AFAM Duje Mihanovi=C4=87 wrote: > On Saturday, September 30, 2023 12:05:41 AM CEST Linus Walleij wrote: > > But it exists, so I can't say you can't use it. Not my choice. > > I understand it is convenient. > > > > It is possible to switch later, but only if you have a unique > > pin controller compatible so please add that. > > Maybe a dumb question. I might want to do this at some point to clean up = the > device tree a bit, are there any such pinctrl drivers I can use as a > reference? Since it's Marvell after all (albeit a descendant of the 20 yo PXA platform!) I would expect new Marvell SoCs to be more alike the AC5 bindings that Chris Packham merged only last year: Documentation/devicetree/bindings/pinctrl/marvell,ac5-pinctrl.yaml Driver: drivers/pinctrl/mvebu/pinctrl-armada-xp.c drivers/pinctrl/mvebu/pinctrl-mvebu.c But if this pin controller is more related to PXA (Intel) hardware than to either Kirkwood or Armada, you might want to do something entirely different. It depends a bit on hardware. Hardware such as pinctrl-single.c with one mux configuration register per pin usually follow the Qualcomm way of doing things, which is to simply have one group per pin, then that can be associated with desired functions: Documentation/devicetree/bindings/pinctrl/qcom,tlmm-common.yaml this has the upside of using all the standard bindings for bias etc. Driver: drivers/pinctrl/qcom/pinctrl-msm.c then qualcomm have subdrivers for each SoC calling into this so you have to check "real" bindings and drivers such as: Documentation/devicetree/bindings/pinctrl/qcom,sm8550-tlmm.yaml drivers/pinctrl/qcom/pinctrl-sm8550.c Yours, Linus Walleij