From patchwork Wed May 28 10:41:08 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Krzysztof Kozlowski X-Patchwork-Id: 893029 Received: from mail-wm1-f54.google.com (mail-wm1-f54.google.com [209.85.128.54]) (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 68F4E28368A for ; Wed, 28 May 2025 10:42:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.54 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1748428934; cv=none; b=B7ssKVEJburu2v6fXOjUVBsampj5VZbptreKHouBq13iN/JSfRVPwe+Zu38ZmuvSuseyPhy/3skQ+sfWNwgCt3KLQ16iBGS/kqwl4t2RqMYUtrFhKnoa3gpRa/f+ZG0QeUf1u9ZgfMkItfyEqxtJhhrGjNvMH84iDIh3FOKBOEs= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1748428934; c=relaxed/simple; bh=vXeTXdr0B9QOQoZjEzmtipvrdS/pdMRjTxQFNr/h+bw=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=nI2CP716IeJiWAdBfvrjaj2oXSaHkVSYIipz6KoZ5NnyrSFaYpdj5VD/T7i7pDNR/6Fr5W8g06Dhu3p8f826F9IADVrPsPzzd10zGcGxC8Zk1TUOoI/EJQGwyg+zQ/12iQCYfCA6zWSWwRqGsDpWa5RaT2LcxiBb6MpF69w/N+g= 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=F6+4NYfu; arc=none smtp.client-ip=209.85.128.54 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="F6+4NYfu" Received: by mail-wm1-f54.google.com with SMTP id 5b1f17b1804b1-442ea0b3b46so4584515e9.0 for ; Wed, 28 May 2025 03:42:12 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1748428931; x=1749033731; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=qdA/ig8nlywJbkEyEpBGSWcEGO8vU+oGS07oZJY25lE=; b=F6+4NYfuriHoKe6R1qFQs8lZBUxyevcww/9Qiqn3VhH9ldruZf9cSzYsvSs0Pr6qSY CQKV8ZTRYF0G9pw9E+2WEDQMCPgQGWb/NiVkiqx5+DtAcncTthz5Y+e1vl9XHqAqu0GS 4hV6wV0zRAskE0mlt/xzodZQHrWrigaeHvX0qQrAHTn77VNHsYC9+AE9n7LjZMbOpOPL HxgcV6odM93DnsVb5qEOk4V4EP5w7gAU49y59oHEiAD0aG2JxANQQheEkErze/83wq0z dzUfrIgc+c62ZJjg2uqbKSwtJy5zY3fhP8srpYX+hLlU373CSbsbhDQojZCEXM1OuGWz lLJA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1748428931; x=1749033731; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=qdA/ig8nlywJbkEyEpBGSWcEGO8vU+oGS07oZJY25lE=; b=G2cUaRmnqAyqwlITqmHBBwawXpgDvvtwiULR+CfgdXcgrpBhxHfESN11gGkhMHDGx5 +OurBI1Z0BiJTbLUroDapSnmAyvmx3JG47HgXDpEtMJ9MOCDQCFueRFX6AwcxlKMooWi R8Vw0KE45gGDPXdBtH2Y/xEOGEQd42Vh6zcVjW5/M+30n+/OMyIBq679P6PhaN/X3thQ fYccCEZBUcPa/m2tp+fOBtuhVsT1oG/10jC8QS86Jgnlq8xoksJmfHd1sqojr6X+2uXn 2nZr5kv66DnN9zfGDI3HJQ3joXC77ucoQvBovRZGgbhOBHpEhYJ+ApAYarWu1XPMsCTg gf7w== X-Gm-Message-State: AOJu0YyzBl8l41a7mp2tlz3CDxB4PFnibe2mxoQyYmTWQZq6SV355APf TjFOUdMFT+eOBQxf6p6xwmJTFfzLNlgmKR8PkR3bt6v8scokPXLhKxQMkWu8gkqzQMc= X-Gm-Gg: ASbGncssWSmLfPBHMBPQ40LoRimz50bPMckcsp/c6qYrhLhf+88RFznsq+GvJMFkQez E2q0Ph+EIq/iBLcgC3yv5U5cc+gRRVuOojeCEpPexgS5YcRlDhakW0JLzxnL+c9XsV9bdWcz5Z8 Wmju7VpSeumZDL8Xk8i2pa2/hE2l80d+PqmNB5kwEW8dWcfh/ww85kr20oY/HT7Zn8f0Xp3rF0x b0rQuKgtp3AwyBD12HPG7Sqz/6/SCsFQINAO6cmrUFtEmgGZQ/puEcqQiBdr9txzge837SIM72g NYmrBhqhLLyRYLTnHNIoXHOGyju3012fk9we7lSoi6thtgOS46zGYKWnX1GCiYQZD4gFiphUiFP V8JYixA== X-Google-Smtp-Source: AGHT+IHdcQ9yFIlwP3zJEBCO4eEnwvHp4vt7sag5E+m0p9zZBnLpwpM4OjjEhLuG3z+TywmxNPkmLg== X-Received: by 2002:a05:600c:4f0c:b0:439:a30f:2e49 with SMTP id 5b1f17b1804b1-44ff407200bmr6539815e9.5.1748428930677; Wed, 28 May 2025 03:42:10 -0700 (PDT) Received: from [192.168.1.29] ([178.197.223.125]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-450064add8bsm17331595e9.17.2025.05.28.03.42.07 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 28 May 2025 03:42:10 -0700 (PDT) From: Krzysztof Kozlowski Date: Wed, 28 May 2025 12:41:08 +0200 Subject: [PATCH 12/17] pinctrl: max77620: Move fixed assignments to 'pinctrl_desc' definition Precedence: bulk X-Mailing-List: linux-gpio@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Message-Id: <20250528-pinctrl-const-desc-v1-12-76fe97899945@linaro.org> References: <20250528-pinctrl-const-desc-v1-0-76fe97899945@linaro.org> In-Reply-To: <20250528-pinctrl-const-desc-v1-0-76fe97899945@linaro.org> To: Linus Walleij , Basavaraj Natikar , Shyam Sundar S K , Andrew Jeffery , Joel Stanley , Avi Fishman , Tomer Maimon , Tali Perry , Patrick Venture , Nancy Yuen , Benjamin Fair , =?utf-8?q?Jonathan_Neusch=C3=A4fer?= , =?utf-8?b?UmFmYcWCIE1pxYJlY2tp?= , Broadcom internal kernel review list , Ray Jui , Scott Branden , Geert Uytterhoeven , David Rhodes , Richard Fitzgerald , Charles Keepax , Lorenzo Bianconi , Sean Wang , Jesper Nilsson , Lars Persson , Manivannan Sadhasivam , Damien Le Moal , Vladimir Zapolskiy , Michal Simek , Emil Renner Berthing , Jianlong Huang , Hal Feng , Matthias Brugger , AngeloGioacchino Del Regno , Alexandre Belloni , Maxime Coquelin , Alexandre Torgue Cc: linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, linux-aspeed@lists.ozlabs.org, openbmc@lists.ozlabs.org, linux-arm-kernel@lists.infradead.org, linux-renesas-soc@vger.kernel.org, linux-sound@vger.kernel.org, patches@opensource.cirrus.com, linux-mediatek@lists.infradead.org, linux-arm-kernel@axis.com, linux-riscv@lists.infradead.org, linux-rtc@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, Krzysztof Kozlowski X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=1581; i=krzysztof.kozlowski@linaro.org; h=from:subject:message-id; bh=vXeTXdr0B9QOQoZjEzmtipvrdS/pdMRjTxQFNr/h+bw=; b=owEBbQKS/ZANAwAKAcE3ZuaGi4PXAcsmYgBoNuhT4xTFSLShiMHGrzurn8k/RWLwBOEg/kTfX 2gUx5kqw3WJAjMEAAEKAB0WIQTd0mIoPREbIztuuKjBN2bmhouD1wUCaDboUwAKCRDBN2bmhouD 119uEACKrgBEySfxytMKjgXas+UOwi3IzZ36tTWGAtahy8quwEaS8yKLMXGH3UR5R/xfiE/qxgT K0j8KZQkSSoxJgxu/1dPpRloEO6KFrgMS1JNvrxc535yglaGGAF6dztV4S0X8Zw47lxNkSI3oCf kmg6Jop17f+7Kjbsu3oNHrrdaoyBSECGDQxfjPrG1OK1828Xa1Gs20NH8vBU0CQOtNFmjAg4W1q rCrbcRJkrSV7L68wgkZcewjkW+uCHpp8xfiuky/X+XA3LrnVkt3ZWvvtx5EVrBotsW3c605oGMm aIh39HoF2i+U+c1KA9+kRRkUESeOM0m/NNik+1C01XMxfvb6mtTvGtL474OCtItf1zazpXvoj9T ygvQC/oprYke1uID4uognq5UNn/OdoNvQJk6tbxBkAwR2epz1lboNXHXIe1MMdo+aFjl0220apS E2bXflnrkOr9/PrUbwkSiJxaLcUHqAJCywLcj/t0mPNa791g/BPtMYNr8rDFfZ7XmZyrEPbEJKp 5XT8mZQD9JIEGYwbw0cQH/vz9kgOs7p47hN4UApEPGZ0HzG/37d9kmNp+Q9mNDbTUfAr5OPTT0V Bv9d8JMKPn9VoDFIrtdM3+11ONR/OmTdCfKgP1pqzLvz0HwoQl9QvT6II0t/70s7GaWP8Umrwo8 X7EdXcYJodX39FA== X-Developer-Key: i=krzysztof.kozlowski@linaro.org; a=openpgp; fpr=9BD07E0E0C51F8D59677B7541B93437D3B41629B Assign 'struct pinctrl_desc' .pins, .npins and other members in definition to make clear that number of pins is fixed and have less code in the probe. Signed-off-by: Krzysztof Kozlowski --- drivers/pinctrl/pinctrl-max77620.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/pinctrl/pinctrl-max77620.c b/drivers/pinctrl/pinctrl-max77620.c index d236daa7c13ebbcaf7c4a8d1a8bed751aa111c0f..acb945a2574368b2e34659a902c34df8aad193d7 100644 --- a/drivers/pinctrl/pinctrl-max77620.c +++ b/drivers/pinctrl/pinctrl-max77620.c @@ -543,6 +543,10 @@ static struct pinctrl_desc max77620_pinctrl_desc = { .pctlops = &max77620_pinctrl_ops, .pmxops = &max77620_pinmux_ops, .confops = &max77620_pinconf_ops, + .pins = max77620_pins_desc, + .npins = ARRAY_SIZE(max77620_pins_desc), + .num_custom_params = ARRAY_SIZE(max77620_cfg_params), + .custom_params = max77620_cfg_params, }; static int max77620_pinctrl_probe(struct platform_device *pdev) @@ -569,11 +573,6 @@ static int max77620_pinctrl_probe(struct platform_device *pdev) platform_set_drvdata(pdev, mpci); max77620_pinctrl_desc.name = dev_name(&pdev->dev); - max77620_pinctrl_desc.pins = max77620_pins_desc; - max77620_pinctrl_desc.npins = ARRAY_SIZE(max77620_pins_desc); - max77620_pinctrl_desc.num_custom_params = - ARRAY_SIZE(max77620_cfg_params); - max77620_pinctrl_desc.custom_params = max77620_cfg_params; for (i = 0; i < MAX77620_PIN_NUM; ++i) { mpci->fps_config[i].active_fps_src = -1;