From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-io1-f50.google.com (mail-io1-f50.google.com [209.85.166.50]) (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 0A5622046BF for ; Wed, 12 Feb 2025 13:50:19 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.166.50 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1739368221; cv=none; b=sdU2WaJomaeR+2U2TtlAdvaM6N8/1SrDAvof03beHXqn//EJzKowGg29qcdoHRAl6k3t0RMc/5pOa85pe+lpm9CRCWrkb9o/l4TgDm0xrG8ayMmBjDgLeLKsh4f/90ycWj/dr7fJRY4O+KpaSwyaWjkIoohbnTFbxqCgvzfUtsQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1739368221; c=relaxed/simple; bh=b578F3G4ksg9uYuuIbDmEdH1PzTG2rkqiAEAgGBywv8=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=pcgJ1sMbXlujD9i2jyj4k+rqsh6/23J0BO/ORw1G9JHn1BQJER/3+OSmvVAxgYiZ/UHMaFW/EibiGePvq3ux/YpWnNJ356YA1I9jShn4NwjbCp7cNwAExurWLiaU7nKkQBSm6IhVegsRhWDBE7B8bjRnoFf46foQ8npnEd+Py8s= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=riscstar.com; spf=pass smtp.mailfrom=riscstar.com; dkim=pass (2048-bit key) header.d=riscstar-com.20230601.gappssmtp.com header.i=@riscstar-com.20230601.gappssmtp.com header.b=JEoTr1iU; arc=none smtp.client-ip=209.85.166.50 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=riscstar.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=riscstar.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=riscstar-com.20230601.gappssmtp.com header.i=@riscstar-com.20230601.gappssmtp.com header.b="JEoTr1iU" Received: by mail-io1-f50.google.com with SMTP id ca18e2360f4ac-854a68f5aeeso145528239f.0 for ; Wed, 12 Feb 2025 05:50:19 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=riscstar-com.20230601.gappssmtp.com; s=20230601; t=1739368219; x=1739973019; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=L++QIvyizjU09jC6Ra79XUphzeMezUQZEHB1o4PFgv4=; b=JEoTr1iU02+HdmXf12s1+WhKYBOYAqZ7ZXunK8wuc4u3+bk7/kLfSP+9opAPDzLeze 7l7Dc5KWM94MFpb3jldq5EdGY+QTqIA+xH4NtNzk38h8cicctdViXeBNmDjRzV3rYhCl iGZvpqXDOr6MAZfBG9DB7oIurXo2YHntaT29fpN+CCtUty+HYC4FhdutcyFQkcrfyojB ZjCo+I8fQMhWeBou4zVNjsXvwDDH69LHSfuF6BqIqrYqd17NdiMwr6ZRWTfGgo/bTQYg u3xwCgloMh4Kut7fzGZVmAlhyyTJGrifambbjVsDdxg9nmNB+Str6HtFD2quZb5xj0kn /QCQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1739368219; x=1739973019; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=L++QIvyizjU09jC6Ra79XUphzeMezUQZEHB1o4PFgv4=; b=Pv0zn+6VsQxz4oDTWsUoyXQ5uILqwUnK1NJ+dcEedFteRJdGD0V9KGuWUgeMxzDqNk nxaxeH5jm68FN9NDgxbTck+OAXw6py9vqsII0Av0sNQMLysYRc6KrcfKtZg1re3M4i42 EPQxEnKwGegvH6mZjtvIqPGs1+Le5cPaNZR5ADscZlniH+CQXRwU4l/TrmeasBAsEoKl yVIAxlPIKBSOx66tG1IamKBDb3lFqrSoDHWIaO8xF73NBpUe9vgMYuDyPAgzAO+9hJq3 HrxH23TJtsn+Hqe5kI41fy+0IJRTuq/in4sQPyNf5AtVWlvaBbDqE+HHM4QJMNuHGSt7 41PQ== X-Forwarded-Encrypted: i=1; AJvYcCW6Vrxdx6uFfvN4wHz0Ejp1FpwyQz69yT2EOUdDjbeiaIqNIhY2eqTJ0H5U6xABM+t3nKJRbNkxKj/a@vger.kernel.org X-Gm-Message-State: AOJu0YwyPJeSagt+aHb1b4806KIHv9CQdo102N5AlNGxJC7v1hJlv4/W s0TsgnEyphT461QnB1+2UALdyKlYo3+iE3+dwP3k6vXkVNa4qnfwEEeT+8GN4kg= X-Gm-Gg: ASbGncvxRIY3H5WsYrp4zgZ5FUEFecZkMp6o3EPDQfXJjkcrW2jMDuZsd0EFqXwZ4vd slRoyGh+skOKUgGPHtb/IA46jlmieG6AFrx0OpgBwavfmEu28gUhCMLKs/lQszZCHGKWJcC0526 jCN3w75CUCcjUPHau6nkiIvSYxYRzXYScY/8uQ75au/d4mGwx8+o8KBqmZBIlynktugJ/bgrrIQ dEAL48ogPPvp0cQXV5hQvIDuZ5DLnhkprUgQ90lPdf8ByOBtSI8pIM2UUM9Gq91hGVTaF+nDq3u 0S+2YcvpiOGdNfE3osMyKJ3f59b7vURj4CyTcMyTU56tHetNpWlTbWs= X-Google-Smtp-Source: AGHT+IGZhne98LVx62ktvxV2QQrzKBPSzDo+6zahs2aSXaoHvmnrhXadwbf8Kww9ZmRSX7qfmJgLgQ== X-Received: by 2002:a92:c56d:0:b0:3d0:4700:db18 with SMTP id e9e14a558f8ab-3d17c1090damr31382265ab.20.1739368219019; Wed, 12 Feb 2025 05:50:19 -0800 (PST) Received: from [172.22.22.28] (c-73-228-159-35.hsd1.mn.comcast.net. [73.228.159.35]) by smtp.gmail.com with ESMTPSA id e9e14a558f8ab-3d05e96f2cdsm32337095ab.50.2025.02.12.05.50.18 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 12 Feb 2025 05:50:18 -0800 (PST) Message-ID: Date: Wed, 12 Feb 2025 07:50:17 -0600 Precedence: bulk X-Mailing-List: linux-gpio@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2] pinctrl: spacemit: enable config option To: Yixun Lan , Linus Walleij , Paul Walmsley , Palmer Dabbelt Cc: Alex Elder , linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org, linux-gpio@vger.kernel.org, spacemit@lists.linux.dev, Conor Dooley References: <20250212-k1-pinctrl-option-v2-1-bde7da0bc0d9@gentoo.org> Content-Language: en-US From: Alex Elder In-Reply-To: <20250212-k1-pinctrl-option-v2-1-bde7da0bc0d9@gentoo.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 2/12/25 2:27 AM, Yixun Lan wrote: > Pinctrl is an essential driver for SpacemiT's SoC, > so let's enable it by default for this SoC. > > The CONFIG_PINCTRL_SPACEMIT_K1 isn't enabled when using > 'make defconfig' to select kernel configuration options. > This result in a broken uart driver where fail at probe() > stage due to no pins found. > > Fixes: a83c29e1d145 ("pinctrl: spacemit: add support for SpacemiT K1 SoC") > Reported-by: Alex Elder > Acked-by: Conor Dooley > Tested-by: Alex Elder > Signed-off-by: Yixun Lan I just tested this version of the patch. By default PINCTRL_SPACEMIT_K1 is "y". But since it's tristate, perhaps it should be default=m so it's not built in for everyone. Yixun I assume the K1 pinctrl driver actually *works* as a kernel module. Anyway, I suggest this change to be a module; Conor should weigh in. Either way is good for me, and I have tested both. Reviewed-by: Alex Elder > --- > This should fix problem that CONFIG_PINCTRL_SPACEMIT_K1 is not enabled > when using make defconfig, thus fail to initilize uart driver which requst > pins during probe stage. > --- > Changes in v2: > - set default as y > - Link to v1: https://lore.kernel.org/r/20250207-k1-pinctrl-option-v1-1-e8a7e4d8404f@gentoo.org > --- > arch/riscv/Kconfig.socs | 1 + > drivers/pinctrl/spacemit/Kconfig | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/arch/riscv/Kconfig.socs b/arch/riscv/Kconfig.socs > index 1916cf7ba450ec9958265de2ca41dc504d4d2f7c..17606940bb5239d0fdfc6b5aefb50eeb982d14aa 100644 > --- a/arch/riscv/Kconfig.socs > +++ b/arch/riscv/Kconfig.socs > @@ -26,6 +26,7 @@ config ARCH_SOPHGO > > config ARCH_SPACEMIT > bool "SpacemiT SoCs" > + select PINCTRL > help > This enables support for SpacemiT SoC platform hardware. > > diff --git a/drivers/pinctrl/spacemit/Kconfig b/drivers/pinctrl/spacemit/Kconfig > index 168f8a5ffbb952cbeae3e3401c11149558e0a84b..c18d879274e72df251e0bc82a308603ce23738bd 100644 > --- a/drivers/pinctrl/spacemit/Kconfig > +++ b/drivers/pinctrl/spacemit/Kconfig > @@ -7,6 +7,7 @@ config PINCTRL_SPACEMIT_K1 > tristate "SpacemiT K1 SoC Pinctrl driver" > depends on ARCH_SPACEMIT || COMPILE_TEST > depends on OF > + default y > select GENERIC_PINCTRL_GROUPS > select GENERIC_PINMUX_FUNCTIONS > select GENERIC_PINCONF > > --- > base-commit: 2014c95afecee3e76ca4a56956a936e23283f05b > change-id: 20250207-k1-pinctrl-option-de5bdfd6b42e > > Best regards,