From nobody Wed Oct 8 07:26:38 2025 Received: from mail-wm1-f52.google.com (mail-wm1-f52.google.com [209.85.128.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 E8FC1221281; Mon, 30 Jun 2025 20:23:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.52 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751315010; cv=none; b=XHoGzsMx2gdqm4bHODAVIehXyk49skLVFlmWb8+54Vdc8H7qrMpvodobh3daIJACYHcLV+brCfb2DKuJcGjo9Ur66QNAmeST9w07Irj1y2aPGnaIEvdLzAnkmNlLmH4rHeKzVXKUhRdZPd2GRxNn8yPeitvljetshEuUzYIHD6E= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751315010; c=relaxed/simple; bh=/8DJjbLRpzn8h6yjM5nQT9QMImWnjpAlNmHtz/RluvQ=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=AMIyTZSBmQ2RWufUBp9nz6wXIQdFTm5VXWwvbt2+rd4jnfN2ElRZGP7/POH1JX/E9X30t/AEmi7pxm1tOTMBqZyW6wVINGc9ttlNMvIiETqXKlBmi06ceN61sZXE0xGbYOCVLB3hdXOK4EVPKWIvU7XBkgHTJEXJL23QR4DY4RE= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=e7np1oE3; arc=none smtp.client-ip=209.85.128.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="e7np1oE3" Received: by mail-wm1-f52.google.com with SMTP id 5b1f17b1804b1-453749af004so26534075e9.1; Mon, 30 Jun 2025 13:23:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1751315007; x=1751919807; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=TSm5b2N+s6cLS/r0R+fGwy1pGElCb0YnFGyDH01M4j8=; b=e7np1oE3d16ZAElMX5F+yoOycqaj2juvqOCrPRKgvhVQklufKyhepP9RdapZdnKx4x yxEjc1XZhyvtuS5OzD1l/i1pxE59enVQ0rl00409aMWmAziOl3MGUgIqS7Wstyb6QHMK BWzHLcinQEcDCQdaDFEz99W85uecKFQNGjmMxARa1cDR5/G/wV9V7N56uEkMcyNXpwjD z4NbShW/N7tavA6+2BIWK75APehOxBoSsDKGt6weMV19s/CamXmux4ADmrV5wbQp9Nyr qASks5g296t7WZV63oVd3+/a2jiGcOlZXy+9Vw6waLbatl/OBHsGJYhspIZfY92C3Ww5 jKeQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1751315007; x=1751919807; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=TSm5b2N+s6cLS/r0R+fGwy1pGElCb0YnFGyDH01M4j8=; b=ct0lPshta3gi8Q6afgpvkGR2cyOoDSnoaEMDvGWf9K3LI4SUqHnxjk11NXPv0cvC8u 9b71u3Ai8lk67DWvjLb+/LUH8DD51gW6fxrqlPuke+rwLAq7TJ1gcpqxRGZX7CZUCO03 FfTTGPL/TKtj5yR9TV6xl53+n6URBJ/Dmrw+AI3WEEDxoKGQvnL0KWDoN/y4fGueWw+7 P/1iHC6iTlzSQTxnFM0c3AI8MYJM5Nl2xgFKNUDPLjyayvS05EHRH/aYDRVnNj/wpghE ho/l4Yt6Tf5P9Ceumtndr9aZ6guNhafz1K/ZEHDIidK6S4dUNFmKwYfUaItLDl6sM+Ul Fjog== X-Forwarded-Encrypted: i=1; AJvYcCVVWUR+tkPDvyWZiHvLmibiImIgAFVzxUjmthvjk3dnYBrTPxYjGc4MjTzHATQPDNQlDddiSWV0iAAl2exZ@vger.kernel.org, AJvYcCVW8HRBF0a4OEV/FqOrPSecPK6Bg0quZFbpejPwC4PxUca2wV6ZDgVKaz1Iq9ZwSPuilwVmPzjb64Ei7uzEoIQQvXg=@vger.kernel.org, AJvYcCW55Mg7gyaa0BtK7RO6O/ZtVjfgg4UHeYSa6sw3DbzlIEX8hUEA228tg5EYWHLFtjLzM5Wclmw1Hys2@vger.kernel.org X-Gm-Message-State: AOJu0Yx6Ae8vmH42vvNyBgS30yIWkYMvJWxNjtZLzHqTIb9graOfZc77 B7RjrWkwYsXBw7yLBKsctW1dh041xcqCl4L8HMvagT9jcs3wGsmIbUyj X-Gm-Gg: ASbGnculOqZTzlN4WlduUKqTtVm13E1CaYDkhC5cVoqUz7d3Cvz5C9ZRf0nG8DX4fjZ sDylTeB49YbOfGWw2b1G+iZq8L5XFrdX/4Wbp6vaIwuzsxmx2qX7syW2VBnSJ5IpZ01iac1lRk2 Yk79pWm6ivreka1D8pTgR+aMccdHFFrgH/7vmmUY7UYFXrOKtIhtowaIiIq3XDaAH1nb1aqh1g1 mkRf3x4oIVb8R6LnoQ/INYYG6DTrHP/YLjl1SEEq1TXiVo8wUlV/Ma/ncfytRJRLOjxaVY2QkDp Sn0Zrah4LZmkPjDOKejGznEl1L1G7FngQNdrvwdg6xGGLgs8taRETzN5Xc3UcTEgR1m32WpMNPF ncSgvrH51Bb8= X-Google-Smtp-Source: AGHT+IG3vceWMeuYOYPskzAnafa+nAvj4f0vu5vUS+G4zt8jMdSBtdirJNXDwZmLFDEUukeVlPK2Hg== X-Received: by 2002:a05:600c:1c08:b0:442:e9eb:cba2 with SMTP id 5b1f17b1804b1-4539ab9c02bmr93279575e9.0.1751315006873; Mon, 30 Jun 2025 13:23:26 -0700 (PDT) Received: from iku.Home ([2a06:5906:61b:2d00:a065:f77f:7ac4:1c25]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4538a3a5b7fsm148523195e9.10.2025.06.30.13.23.25 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 30 Jun 2025 13:23:26 -0700 (PDT) From: Prabhakar X-Google-Original-From: Prabhakar To: Greg Kroah-Hartman , Jiri Slaby , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Geert Uytterhoeven , Magnus Damm , Wolfram Sang , linux-serial@vger.kernel.org, devicetree@vger.kernel.org Cc: linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org, Prabhakar , Biju Das , Fabrizio Castro , Lad Prabhakar Subject: [PATCH v14 1/5] dt-bindings: serial: renesas,rsci: Add optional secondary clock input Date: Mon, 30 Jun 2025 21:23:19 +0100 Message-ID: <20250630202323.279809-2-prabhakar.mahadev-lad.rj@bp.renesas.com> X-Mailer: git-send-email 2.49.0 In-Reply-To: <20250630202323.279809-1-prabhakar.mahadev-lad.rj@bp.renesas.com> References: <20250630202323.279809-1-prabhakar.mahadev-lad.rj@bp.renesas.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Thierry Bultel Update the RSCI binding to support an optional secondary clock input on the RZ/T2H SoC. At boot, the RSCI operates using the default synchronous clock (PCLKM core clock), which is enabled by the bootloader. However, to support a wider range of baud rates, the hardware also requires an asynchronous external clock input. Clock selection is controlled internally by the CCR3 register in the RSCI block. Due to an incomplete understanding of the hardware, the original binding defined only a single clock ("fck"), which is insufficient to describe the full capabilities of the RSCI on RZ/T2H. This update corrects the binding by allowing up to three clocks and defining the `clock-names` as "operation", "bus", and optionally "sck" for the asynchronous clock input. This is an ABI change, as it modifies the expected number and names of clocks. However, since there are no in-kernel consumers of this binding yet, the change is considered safe and non-disruptive. Also remove the unneeded `serial0` alias from the DTS example and use the R9A09G077_CLK_PCLKM macro for core clock. Signed-off-by: Thierry Bultel Signed-off-by: Lad Prabhakar Reviewed-by: Geert Uytterhoeven Reviewed-by: Rob Herring (Arm) --- v13->v14: - Dropped using `R9A09G077_CLK_PCLKM` macro in binding. v12->v13: - Rebased on latest linux-next. - Updated commit message to clarify the ABI change. - Used `R9A09G077_CLK_PCLKM` macro for core clock --- .../bindings/serial/renesas,rsci.yaml | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/Documentation/devicetree/bindings/serial/renesas,rsci.yaml b/D= ocumentation/devicetree/bindings/serial/renesas,rsci.yaml index 4aacc44bb509..f20de85d5304 100644 --- a/Documentation/devicetree/bindings/serial/renesas,rsci.yaml +++ b/Documentation/devicetree/bindings/serial/renesas,rsci.yaml @@ -41,10 +41,15 @@ properties: - const: tei =20 clocks: - maxItems: 1 + minItems: 2 + maxItems: 3 =20 clock-names: - const: fck # UART functional clock + minItems: 2 + items: + - const: operation + - const: bus + - const: sck # optional external clock input =20 power-domains: maxItems: 1 @@ -66,10 +71,6 @@ examples: #include #include =20 - aliases { - serial0 =3D &sci0; - }; - sci0: serial@80005000 { compatible =3D "renesas,r9a09g077-rsci"; reg =3D <0x80005000 0x400>; @@ -78,7 +79,7 @@ examples: , ; interrupt-names =3D "eri", "rxi", "txi", "tei"; - clocks =3D <&cpg CPG_MOD 108>; - clock-names =3D "fck"; + clocks =3D <&cpg CPG_MOD 8>, <&cpg CPG_CORE 13>; + clock-names =3D "operation", "bus"; power-domains =3D <&cpg>; }; --=20 2.49.0 From nobody Wed Oct 8 07:26:38 2025 Received: from mail-wm1-f53.google.com (mail-wm1-f53.google.com [209.85.128.53]) (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 4C91622D9E3; Mon, 30 Jun 2025 20:23:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.53 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751315012; cv=none; b=F0++18IBLkWwzLmrG8fslrEy9ITA3s+u1wb7IKvgMATTT5a4qvOhxNGr99UJ1In/tNt88oYyLnF2tCsAbcPqk/0Urn+vC+7yvnScLunfxQOsAmvcD4UW3tFM8I+F9ipVoLOfbJZDFOXhQ5HF7534x+RAEQcAKJNrt0rREDgtaB0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751315012; c=relaxed/simple; bh=q1k2ya7ntd+84OBNE4fCXhUPBLrExVHnmfq512xeKB4=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=aI5UagVHhZFJLtVThV3cNuYKT9dX6LlSIrnnaKZ8hGcvTpgOL2uopRJlWqGa8lvClGLLMhMN0dlA25Oskk1K2AN1+8fVMqV1iCiDlMOeoN8cEMStySK+2IAa3THBSqlSczy8CErj2888MGFBjvaC36qcuA3WSXBBYq0BgSUJGUQ= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=WgleltUZ; arc=none smtp.client-ip=209.85.128.53 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="WgleltUZ" Received: by mail-wm1-f53.google.com with SMTP id 5b1f17b1804b1-45363645a8eso34186615e9.1; Mon, 30 Jun 2025 13:23:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1751315009; x=1751919809; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=tRRkV+fcxIvFVti07imS12JhpI7oaLjvmCHQCmufjlc=; b=WgleltUZWFxpDtQu+IAyinuUluTXKN5AH1cRGO4U6vOP8AR0aBtbiWqFOCZekuFu8K fK/7Yq7xTi88fONS6CJZLwWXB1l2XaN2XQPqAgWxSj7Fi/iR5WK1BOlGUicBq9flMzDz ne+5m2OLBKuPMk40qqeynFf6vqysibqvM86mMOEK5byImY6FVzVqXT21qgvyOZ9oy0We B0dGcmrnW3nVuh21nIbUEec7eTJVj39QfgQBh+Uou+W2Xl1T00QexzhCVcKWRwrVEVMu 8PkGvZ2lAqkWI2aaWlc2m96IOucQzLikc6CSyYp3gc3GRvAqtbba9Ka7Eluf1IV/ed1E P/uw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1751315009; x=1751919809; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=tRRkV+fcxIvFVti07imS12JhpI7oaLjvmCHQCmufjlc=; b=ZkOXi0704Khp0Tg4eJFD1gaTMv5djbe+rALkTyp2Gi1psRygN/pUTD6YiLQVSyxtdd qeX/2ZxW+NI5NLvzMPrVz6ayGY3P8SCoICr9t8Knb+D3b1vQ1WK2Dp5zeTrAW4fCpZWX 70W4uHBdTQ19u5vNHi4F4hAkD6GvJfEY5qEnAOHawv2yKcNSQ3tOQThnIGrcJBSPzWdL 4OUywQAksJ9CRX9lVk3DGYGmqBUH5O+PR8G2aD++xfnugeMNb3FdyemWqijsYzlfG+97 /UjeH3NQKx0o5NWw2zZH4Sb4Xhkt5ol7P9Z99GEhm7VB7f4BRKcGwOW5w6B2Azsu3r8I r7nQ== X-Forwarded-Encrypted: i=1; AJvYcCU+FIDXWrEZw8ZrF0ZkZcsy9fudEkrpdBxKDodgnLV/wAYWb3eeY/gXsw5eULrXOqYN3uENdtlPXD7YYtNM@vger.kernel.org, AJvYcCUsMS1iexrMaZsqbMNqQ7JKpynfBa7deifJTa+RZPweRtp/kTYd8fFbUnn/heCfZnqU9DY5SS9sE033TjoCAOLkM84=@vger.kernel.org, AJvYcCViO4tjVXikRk4Quv1KTVk7LCAYfxvNu6c1EyVVX9aKCM61KjASwefI/KSNGp9dpW+YfABTkYbdJQlj@vger.kernel.org X-Gm-Message-State: AOJu0Yy3nVkWcb9hDgKRpg0b9iUmMO1AtO9tCQtiV2YflNP3bBDns47J yD4phKXtUj8KWBOiBgggh2iegKdxUnAv1isxQ5W5uF+MxlP3gj+x8298 X-Gm-Gg: ASbGnctNRBLJZzmdeOE4rHuIUguFT8otLEH/HDqeMvRdIpgMzXMeBMwC9sbUIKWb8Ue Zl+YccHSwhWVb/pzWN9/lSw2VXu5Xl9VEQM2zzvw3RcSwzSlnLBB4NOdMd4ydPtXapw7AjIUfhc QsZSAXt4tvPh/feVMtin+bc/C+XhfOmY0cB7FT5ASkwO0tcnWtqXFgth7mvRzEMLagZwsMo4VZN f//SaKz0r8oXOzx22KHoJPzhyYmUUZdQeDmCXSnt0hGX6yQ0G1H9j8EYKiXvi1e2/vZxrWdqGyn Kx8WKKtCbfdepzvdNKtg3wkTnrwvUooZQTbaeSdDCbe1F1mR+Yqgl0GUJJtBPzH2LaNYdB4KAu4 lFfkZnSBy/bk= X-Google-Smtp-Source: AGHT+IFrXJra6NuB7SL0le7cHxAKMCh3buaoVInV13ZQw3kBeIKxu9AD0KJpOVAfTvhgnImo7oUCJg== X-Received: by 2002:a05:600c:828f:b0:450:d00d:588b with SMTP id 5b1f17b1804b1-45392bc88damr135090365e9.9.1751315008308; Mon, 30 Jun 2025 13:23:28 -0700 (PDT) Received: from iku.Home ([2a06:5906:61b:2d00:a065:f77f:7ac4:1c25]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4538a3a5b7fsm148523195e9.10.2025.06.30.13.23.27 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 30 Jun 2025 13:23:27 -0700 (PDT) From: Prabhakar X-Google-Original-From: Prabhakar To: Greg Kroah-Hartman , Jiri Slaby , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Geert Uytterhoeven , Magnus Damm , Wolfram Sang , linux-serial@vger.kernel.org, devicetree@vger.kernel.org Cc: linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org, Prabhakar , Biju Das , Fabrizio Castro , Lad Prabhakar Subject: [PATCH v14 2/5] dt-bindings: serial: rsci: Update maintainer entry Date: Mon, 30 Jun 2025 21:23:20 +0100 Message-ID: <20250630202323.279809-3-prabhakar.mahadev-lad.rj@bp.renesas.com> X-Mailer: git-send-email 2.49.0 In-Reply-To: <20250630202323.279809-1-prabhakar.mahadev-lad.rj@bp.renesas.com> References: <20250630202323.279809-1-prabhakar.mahadev-lad.rj@bp.renesas.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Lad Prabhakar Add myself as the maintainer for the Renesas RSCI device tree binding, as Thierry Bultel no longer works for Renesas. Signed-off-by: Lad Prabhakar Reviewed-by: Geert Uytterhoeven Acked-by: Rob Herring (Arm) --- v13->v14: - No changes. v12->v13: - Rebased on latest linux-next. - Added Reviewed-by and Acked-by tags. --- Documentation/devicetree/bindings/serial/renesas,rsci.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/serial/renesas,rsci.yaml b/D= ocumentation/devicetree/bindings/serial/renesas,rsci.yaml index f20de85d5304..f50d8e02f476 100644 --- a/Documentation/devicetree/bindings/serial/renesas,rsci.yaml +++ b/Documentation/devicetree/bindings/serial/renesas,rsci.yaml @@ -8,7 +8,7 @@ title: Renesas RSCI Serial Communication Interface =20 maintainers: - Geert Uytterhoeven - - Thierry Bultel + - Lad Prabhakar =20 allOf: - $ref: serial.yaml# --=20 2.49.0 From nobody Wed Oct 8 07:26:38 2025 Received: from mail-wr1-f42.google.com (mail-wr1-f42.google.com [209.85.221.42]) (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 179D023183B; Mon, 30 Jun 2025 20:23:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.42 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751315013; cv=none; b=QjvLnSHp89ZZjecZJebjPGXHpOkPXbeaSOgFwFKSrG4Bxjxkt5Gjpr8+LhToPKEfbiLD7ZG4Zr7cQoa1EJjhjtBpXoxVQHTR9RzzvNQiBwy6bNgIAf+nh82xfQxyPuTURhG1oFN4hDi+wsRfNnafMLs1N5CUFaUeeGk/HY32b2Q= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751315013; c=relaxed/simple; bh=g6hqr+J2pF1c03qptdqCZ+8L3bzLFJeiz0AkDf6Ry0Q=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=VwfBuqaoMVL4/SYkL0HTqEVm5P8xu1H1/em+Gx+YVXw5Ir4EwDzM37SmkMIm0l+tFTG7ZV3dT2kVSUoCA+ENIkzYQTIdc3AvO8cl/55ELgzHnlMKS7DoEfWXliDlug8LMu9vvhCSNMEFtGmvliQ5yzeh6VPPAot7mhyF8OkFhCE= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=KUTIXmwf; arc=none smtp.client-ip=209.85.221.42 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="KUTIXmwf" Received: by mail-wr1-f42.google.com with SMTP id ffacd0b85a97d-3a4ef2c2ef3so2287894f8f.2; Mon, 30 Jun 2025 13:23:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1751315009; x=1751919809; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=0odGTbRuyJr049EUwp2mzcTgQO9lLHgZa8OkN0rK/U4=; b=KUTIXmwf/LOm24GmjMaJpqOSf6VlPITJNyTmb1AMtpLb1CGbrF7Qci43g4CrnDUbV0 GqDPYrSo1Cr/vi4f52QYn+U9oyzgteMuTEQy6460ohHB2PP5o0Jo4ys2BmFMZyqXs30W YHNC4PQ1qryKbG8QyKa8rMarywVehv3RgoYJnrUboInW7rKf/LwCrIZ7Um22S14TonlF juENqbtg39bEMpUOOVONP34DuuolC/Rm4BqgBIK+y1JXjsPYlkstQAahHRUamIY2e5s2 fhMUmQXUMREUzMxTp9saHke5Rk0EjDq1fAlsSPBUB88+O4yxkwhyAaXWBuROI+TkfrPB qIaA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1751315009; x=1751919809; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=0odGTbRuyJr049EUwp2mzcTgQO9lLHgZa8OkN0rK/U4=; b=hcGIwqoX6GSmfPW24YDWF7nSPWdxTv53H/PJw3L2NZhZpwZkrNhxffBuaxfdIrqlYk Nk9TI34kqO5ac88to/qDSPi3cZsUtfC3yFgjwD4GIDYX2y5g3g4083DWSMAzU0ACz37a M7/cgNA2R/6JW0VrQsPBhXdGQIrrV7VDe5eLC0PNY2yKwU4S96AuYpDhDe7zQ+ON4mCi uFJJEiFf9/kOYg9//t9iThqTI0DVVBjPnSeDdVQ+gP8DAh5jTA/QrrbwIoLAKiWVcLvn 4pxrLWgXwpXk+42AdGj1UwTQ5e+jayWVV7sbMr7U6+eySByJHGeCOPUwL/MzVYFtnEXM yepA== X-Forwarded-Encrypted: i=1; AJvYcCUOh3ho5xq+JFZFlnh7nxeAOxs1PqWZAVLcvlOMuOqfH+r9h2d+18U8hNzAQw6ablEFRY9h472narBdj5koO3PwJ6Y=@vger.kernel.org, AJvYcCWSnZDpsM3pZOfLpJwJFIjrCGZegwPMHniWDiNwHwggJR/n+8w/PUolGXmKP0sSsXNPUKn2IsHkg6va@vger.kernel.org, AJvYcCWUFdmXmu8cTm5o0pNYFPyBuz8Q1Px5xUEnUqcg6Mwo7UVodldotH76kkXP5NUwagXHp6szt+rRQKtSVE/y@vger.kernel.org X-Gm-Message-State: AOJu0YyPuJ0h9daHv2fS5PYZW7q3oqk13VCBpg1ZZsXeMG9Wi8zBDGS1 QRgplhxem4vn4MqH7Wr23C22O0TjiTORN4D3s+P+N9z/n/seeYzdjkpQ X-Gm-Gg: ASbGnctVmiagMI1H1lhO6PzdxJ/qDBRLbIOAVwklt4HOnr2Irf3sDrefeg+kxaYwtPM eiX4QEe5J+PTCu1I6er1h5BIU1MEtWwz+ovDaXgFkQOhvt0AM73dLtPT1qM1znUe+WfPiUdU5Ms HTyLg+l4igur/vH9BbZl3k69zlMQr6oKVJXvtz6RAHAkrdqdmvSD0lxfQEO+UufNHDFdyyP3Lt1 S6D24/qsLCIzNNFbIWGNtYcuNdz6Xb9xXJnnSTPiaR/4ojyDmda51kolTRMAyQLEOcqYdM9qBQ+ lbXFscebSADOz8q79N2XAafbBn1+6wUi+KL4p/HQdLd0d4sDfYRy79nGNlqIiFbVQ6V11QMOT2y sap3p74lDAR8= X-Google-Smtp-Source: AGHT+IG4558x/ocshOI7mIikuZNBrzyiaVVFRj1v/lyejyahiKo53X3c1hz5JOh9H5+u5GzFLCFpEA== X-Received: by 2002:a05:6000:4182:b0:3a4:f6ba:51c8 with SMTP id ffacd0b85a97d-3a8f482bc72mr13259113f8f.14.1751315009254; Mon, 30 Jun 2025 13:23:29 -0700 (PDT) Received: from iku.Home ([2a06:5906:61b:2d00:a065:f77f:7ac4:1c25]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4538a3a5b7fsm148523195e9.10.2025.06.30.13.23.28 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 30 Jun 2025 13:23:28 -0700 (PDT) From: Prabhakar X-Google-Original-From: Prabhakar To: Greg Kroah-Hartman , Jiri Slaby , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Geert Uytterhoeven , Magnus Damm , Wolfram Sang , linux-serial@vger.kernel.org, devicetree@vger.kernel.org Cc: linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org, Prabhakar , Biju Das , Fabrizio Castro , Lad Prabhakar Subject: [PATCH v14 3/5] serial: sh-sci: Replace direct stop_rx/stop_tx calls with port ops in sci_shutdown() Date: Mon, 30 Jun 2025 21:23:21 +0100 Message-ID: <20250630202323.279809-4-prabhakar.mahadev-lad.rj@bp.renesas.com> X-Mailer: git-send-email 2.49.0 In-Reply-To: <20250630202323.279809-1-prabhakar.mahadev-lad.rj@bp.renesas.com> References: <20250630202323.279809-1-prabhakar.mahadev-lad.rj@bp.renesas.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Lad Prabhakar Replace direct calls to sci_stop_rx() and sci_stop_tx() with port ops callbacks in sci_shutdown(). This enables the RSCI driver, which reuses the SCI core but implements its own stop_rx and stop_tx logic, to reuse sci_shutdown() without duplicating code. Signed-off-by: Lad Prabhakar Reviewed-by: Geert Uytterhoeven --- v13->v14: - No changes. v12->v13: - Added Reviewed-by tag. --- drivers/tty/serial/sh-sci.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/tty/serial/sh-sci.c b/drivers/tty/serial/sh-sci.c index 1c356544a832..5c4283ce542d 100644 --- a/drivers/tty/serial/sh-sci.c +++ b/drivers/tty/serial/sh-sci.c @@ -2289,8 +2289,8 @@ void sci_shutdown(struct uart_port *port) mctrl_gpio_disable_ms_sync(to_sci_port(port)->gpios); =20 uart_port_lock_irqsave(port, &flags); - sci_stop_rx(port); - sci_stop_tx(port); + s->port.ops->stop_rx(port); + s->port.ops->stop_tx(port); s->ops->shutdown_complete(port); uart_port_unlock_irqrestore(port, flags); =20 --=20 2.49.0 From nobody Wed Oct 8 07:26:38 2025 Received: from mail-wm1-f41.google.com (mail-wm1-f41.google.com [209.85.128.41]) (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 B3952239E77; Mon, 30 Jun 2025 20:23:32 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.41 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751315015; cv=none; b=TLBvKb9g4ugmq4M8jNg5QcFKzF0De6Ba/6CUtZGewzcO96QUyBJGrJbCP5qLSOMlj6hqMzk7EMuGZleRiePJh50aFixsK+WX0ATlhPsPiZRel+W+oacjy8A+zNf8mTnM3y+M+FMlDKFFoihmVtnZQiHDMx8J2P86FoQH2DSUhJ4= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751315015; c=relaxed/simple; bh=9pdVkTs4Si7PNdiyl39aMdKw6FlWaUV6VhKbAzpaiX0=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=fdm7lPwBw5S+nC1jWusmPN9VouJka4I0k75utypygtRIsbHxOzKD41BHr70MaZlQdH8L7eQY+C+STpz3sbAw/5ucT8tFKcJghRzilzLP8r7ryF7nr9xNw1mY+qpNJzxrKo4HI/wS+/azJTrgo1M+uqUtDCz14bCu2F4SsxMQvdk= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=hJokfx0x; arc=none smtp.client-ip=209.85.128.41 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="hJokfx0x" Received: by mail-wm1-f41.google.com with SMTP id 5b1f17b1804b1-450cb2ddd46so28220535e9.2; Mon, 30 Jun 2025 13:23:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1751315011; x=1751919811; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=H+9MLLKvB77GFqpwuxhiI+9ewTvoWB8hmqcwjd3kz4A=; b=hJokfx0xGWZEIr5e1WmwmVI2a4H96EHAFfwoUc7ICJ6oCOyzRB1E4BV6doeW/qgdnw yQDkDJpD1fD8FBWIq4+/CFOH1dSkrKdrbLWokmi6daWUKpoyRZ1mi8ZorlW354/tB9ro JJf38J9WBoe8h6LUHCEkDsj7T1Wj6d23CIpno/GUV86Vou8XfT5f5kXqlexlC97oSH+S rXMfuPj7yzElCqOzD5YQf55DiaGJJtZCtpmjNRrsTPONtnK0koU62RsUTTpfX9HHqYZe /RIqlIzBlvL8Osmj03yUCjHWgYn0J5t/qVO4QQbVANVaejp4IRgtH6tqdgQwwmZMxeQY O/lw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1751315011; x=1751919811; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=H+9MLLKvB77GFqpwuxhiI+9ewTvoWB8hmqcwjd3kz4A=; b=vuU3V83fgECBVFkatDURZx0Rvb7H+rFzdgGxPPQzzMHHSUp3rm20BCwrPXuyKZkbjQ QvB5XywWHAAbdP/uIFWIOY2KHrs6J+zb4yZiGSSwnSzJAqugm/D9yEnj5kY5P8v7NLty +HoYv/t9Fo1RLrZdUs+gUfrAcqV+4GKBS8QyXNtBBq5PzECFNSmHReTr7I/3xiGB3SEl 5uAziPsDSXdQCELrY1/H7vB+7tOFfGeKt7VhaOFfMRReYmwYd8v5BvPoxTyDgNlu6myj ON11GqgItfql9aLcZRxgCpT26KzUYlOkFKph85udsCGIpImk2BZLScc0DlcU/8XGRvs9 FzFg== X-Forwarded-Encrypted: i=1; AJvYcCUhWPvKQX44oFs2mBTaYzMtJmmtHlXqxh1EWEtc+SUypFia1H7LAlwZ2jkjcfdyoFU8B1fVT94+Fx3mN0XIHvVROlQ=@vger.kernel.org, AJvYcCWVdfTGc718/fd2lrY3nfYLx8FjgKsWIqM85JfpKawdypgjRQ6Xte1tTrPQHr6QKe8j7J3AqeavcFMF@vger.kernel.org, AJvYcCXUg8OZDe/2lFlQMSPdz+JwFNDMeWHKzljEPzt0FCCW2JDGjJpYjHd+LccIpbH1QnYjqLtEuWs80tmgrDuK@vger.kernel.org X-Gm-Message-State: AOJu0YzYQRo4lQjvA2aUFAX32TIje1pCv6pLGvnB2olBHDQHLPwSmNOG GgmE8LTLIMe5iARK9A7oYuKySfnlbK+g+9A+9CfGrHTyI3RXkB89lMNY9ywYVGJ9Dc0= X-Gm-Gg: ASbGncsblTJQwBNUEwDReLw5oaMhWSw0Ko4rl9VV8k1TZ9qUEb6QUEv04pt/UO/V/mo tHcvFlAOxslOeb1b58Iw762gtzuKmfSuZDDyI+F9Zw3jJYyXP57+BiW5qJH00jd5F8ZRuaGFJtT zmat8tH52QMV0GL38jjLc4jxINrBWIsdI3uwsQ4tFN/A+V0605DuYbfL+q8ycwJ7fsvLnbWmjy6 Y+rgw43Km3xWAYLOji5p7b9lylpOLQtQsjt2PFmLxc36CQ8Q1etc8O5AhGqPGqUu3A2P1aEerw4 gX3qxauf5vBMdSwfbcQ1rZxExQN0eBI8q7fk6DUuJUc6iwEEkqCP3bIeFRSDCsIRiYFrErJm0R4 97JkDq2QfbDttTgcONE1pXg== X-Google-Smtp-Source: AGHT+IHNJPTMyyGYHOeNeyeOBVm11AQsRftSBhify6BskdSgiUtWDERhSBDkasoubxGwysswWagFkw== X-Received: by 2002:a05:600c:1d28:b0:453:cd0:903c with SMTP id 5b1f17b1804b1-4538fd62fa8mr162317205e9.2.1751315010696; Mon, 30 Jun 2025 13:23:30 -0700 (PDT) Received: from iku.Home ([2a06:5906:61b:2d00:a065:f77f:7ac4:1c25]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4538a3a5b7fsm148523195e9.10.2025.06.30.13.23.29 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 30 Jun 2025 13:23:29 -0700 (PDT) From: Prabhakar X-Google-Original-From: Prabhakar To: Greg Kroah-Hartman , Jiri Slaby , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Geert Uytterhoeven , Magnus Damm , Wolfram Sang , linux-serial@vger.kernel.org, devicetree@vger.kernel.org Cc: linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org, Prabhakar , Biju Das , Fabrizio Castro , Lad Prabhakar Subject: [PATCH v14 4/5] serial: sh-sci: Use private port ID Date: Mon, 30 Jun 2025 21:23:22 +0100 Message-ID: <20250630202323.279809-5-prabhakar.mahadev-lad.rj@bp.renesas.com> X-Mailer: git-send-email 2.49.0 In-Reply-To: <20250630202323.279809-1-prabhakar.mahadev-lad.rj@bp.renesas.com> References: <20250630202323.279809-1-prabhakar.mahadev-lad.rj@bp.renesas.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Thierry Bultel New port types cannot be added in serial_core.h, which is shared with userspace. In order to support new port types, the coming new ones will have BIT(7) set in the id value, and in this case, uartport->type is set to PORT_GENERIC. This commit therefore changes all the places where the port type is read, by not relying on uartport->type but on the private value stored in struct sci_port. Signed-off-by: Thierry Bultel Signed-off-by: Lad Prabhakar Reviewed-by: Wolfram Sang Reviewed-by: Geert Uytterhoeven --- v13->v14: - No changes. v12->v13: - Fixed checkpatch warnings. --- drivers/tty/serial/sh-sci-common.h | 3 + drivers/tty/serial/sh-sci.c | 161 ++++++++++++++++------------- 2 files changed, 93 insertions(+), 71 deletions(-) diff --git a/drivers/tty/serial/sh-sci-common.h b/drivers/tty/serial/sh-sci= -common.h index bd9d9cfac1c8..fcddf66780c9 100644 --- a/drivers/tty/serial/sh-sci-common.h +++ b/drivers/tty/serial/sh-sci-common.h @@ -142,6 +142,9 @@ struct sci_port { int rx_fifo_timeout; u16 hscif_tot; =20 + u8 type; + u8 regtype; + const struct sci_port_ops *ops; =20 bool has_rtscts; diff --git a/drivers/tty/serial/sh-sci.c b/drivers/tty/serial/sh-sci.c index 5c4283ce542d..26536ff2eda1 100644 --- a/drivers/tty/serial/sh-sci.c +++ b/drivers/tty/serial/sh-sci.c @@ -75,6 +75,8 @@ =20 #define SCI_NPORTS CONFIG_SERIAL_SH_SCI_NR_UARTS =20 +#define SCI_PUBLIC_PORT_ID(port) (((port) & BIT(7)) ? PORT_GENERIC : (port= )) + static struct sci_port sci_ports[SCI_NPORTS]; static unsigned long sci_ports_in_use; static struct uart_driver sci_uart_driver; @@ -580,7 +582,7 @@ static void sci_start_tx(struct uart_port *port) unsigned short ctrl; =20 #ifdef CONFIG_SERIAL_SH_SCI_DMA - if (port->type =3D=3D PORT_SCIFA || port->type =3D=3D PORT_SCIFB) { + if (s->type =3D=3D PORT_SCIFA || s->type =3D=3D PORT_SCIFB) { u16 new, scr =3D sci_serial_in(port, SCSCR); if (s->chan_tx) new =3D scr | SCSCR_TDRQE; @@ -592,7 +594,7 @@ static void sci_start_tx(struct uart_port *port) =20 if (s->chan_tx && !kfifo_is_empty(&port->state->port.xmit_fifo) && dma_submit_error(s->cookie_tx)) { - if (s->cfg->regtype =3D=3D SCIx_RZ_SCIFA_REGTYPE) + if (s->regtype =3D=3D SCIx_RZ_SCIFA_REGTYPE) /* Switch irq from SCIF to DMA */ disable_irq_nosync(s->irqs[SCIx_TXI_IRQ]); =20 @@ -601,8 +603,8 @@ static void sci_start_tx(struct uart_port *port) } #endif =20 - if (!s->chan_tx || s->cfg->regtype =3D=3D SCIx_RZ_SCIFA_REGTYPE || - port->type =3D=3D PORT_SCIFA || port->type =3D=3D PORT_SCIFB) { + if (!s->chan_tx || s->regtype =3D=3D SCIx_RZ_SCIFA_REGTYPE || + s->type =3D=3D PORT_SCIFA || s->type =3D=3D PORT_SCIFB) { /* Set TIE (Transmit Interrupt Enable) bit in SCSCR */ ctrl =3D sci_serial_in(port, SCSCR); =20 @@ -611,7 +613,7 @@ static void sci_start_tx(struct uart_port *port) * (transmit interrupt enable) or in the same instruction to start * the transmit process. */ - if (port->type =3D=3D PORT_SCI) + if (s->type =3D=3D PORT_SCI) ctrl |=3D SCSCR_TE; =20 sci_serial_out(port, SCSCR, ctrl | SCSCR_TIE); @@ -620,12 +622,13 @@ static void sci_start_tx(struct uart_port *port) =20 static void sci_stop_tx(struct uart_port *port) { + struct sci_port *s =3D to_sci_port(port); unsigned short ctrl; =20 /* Clear TIE (Transmit Interrupt Enable) bit in SCSCR */ ctrl =3D sci_serial_in(port, SCSCR); =20 - if (port->type =3D=3D PORT_SCIFA || port->type =3D=3D PORT_SCIFB) + if (s->type =3D=3D PORT_SCIFA || s->type =3D=3D PORT_SCIFB) ctrl &=3D ~SCSCR_TDRQE; =20 ctrl &=3D ~SCSCR_TIE; @@ -633,21 +636,22 @@ static void sci_stop_tx(struct uart_port *port) sci_serial_out(port, SCSCR, ctrl); =20 #ifdef CONFIG_SERIAL_SH_SCI_DMA - if (to_sci_port(port)->chan_tx && - !dma_submit_error(to_sci_port(port)->cookie_tx)) { - dmaengine_terminate_async(to_sci_port(port)->chan_tx); - to_sci_port(port)->cookie_tx =3D -EINVAL; + if (s->chan_tx && + !dma_submit_error(s->cookie_tx)) { + dmaengine_terminate_async(s->chan_tx); + s->cookie_tx =3D -EINVAL; } #endif } =20 static void sci_start_rx(struct uart_port *port) { + struct sci_port *s =3D to_sci_port(port); unsigned short ctrl; =20 ctrl =3D sci_serial_in(port, SCSCR) | port_rx_irq_mask(port); =20 - if (port->type =3D=3D PORT_SCIFA || port->type =3D=3D PORT_SCIFB) + if (s->type =3D=3D PORT_SCIFA || s->type =3D=3D PORT_SCIFB) ctrl &=3D ~SCSCR_RDRQE; =20 sci_serial_out(port, SCSCR, ctrl); @@ -655,11 +659,12 @@ static void sci_start_rx(struct uart_port *port) =20 static void sci_stop_rx(struct uart_port *port) { + struct sci_port *s =3D to_sci_port(port); unsigned short ctrl; =20 ctrl =3D sci_serial_in(port, SCSCR); =20 - if (port->type =3D=3D PORT_SCIFA || port->type =3D=3D PORT_SCIFB) + if (s->type =3D=3D PORT_SCIFA || s->type =3D=3D PORT_SCIFB) ctrl &=3D ~SCSCR_RDRQE; =20 ctrl &=3D ~port_rx_irq_mask(port); @@ -669,10 +674,12 @@ static void sci_stop_rx(struct uart_port *port) =20 static void sci_clear_SCxSR(struct uart_port *port, unsigned int mask) { - if (port->type =3D=3D PORT_SCI) { + struct sci_port *s =3D to_sci_port(port); + + if (s->type =3D=3D PORT_SCI) { /* Just store the mask */ sci_serial_out(port, SCxSR, mask); - } else if (to_sci_port(port)->params->overrun_mask =3D=3D SCIFA_ORER) { + } else if (s->params->overrun_mask =3D=3D SCIFA_ORER) { /* SCIFA/SCIFB and SCIF on SH7705/SH7720/SH7721 */ /* Only clear the status bits we want to clear */ sci_serial_out(port, SCxSR, sci_serial_in(port, SCxSR) & mask); @@ -742,13 +749,13 @@ static void sci_init_pins(struct uart_port *port, uns= igned int cflag) return; } =20 - if (port->type =3D=3D PORT_SCIFA || port->type =3D=3D PORT_SCIFB) { + if (s->type =3D=3D PORT_SCIFA || s->type =3D=3D PORT_SCIFB) { u16 data =3D sci_serial_in(port, SCPDR); u16 ctrl =3D sci_serial_in(port, SCPCR); =20 /* Enable RXD and TXD pin functions */ ctrl &=3D ~(SCPCR_RXDC | SCPCR_TXDC); - if (to_sci_port(port)->has_rtscts) { + if (s->has_rtscts) { /* RTS# is output, active low, unless autorts */ if (!(port->mctrl & TIOCM_RTS)) { ctrl |=3D SCPCR_RTSC; @@ -765,7 +772,7 @@ static void sci_init_pins(struct uart_port *port, unsig= ned int cflag) } sci_serial_out(port, SCPDR, data); sci_serial_out(port, SCPCR, ctrl); - } else if (sci_getreg(port, SCSPTR)->size && s->cfg->regtype !=3D SCIx_RZ= V2H_SCIF_REGTYPE) { + } else if (sci_getreg(port, SCSPTR)->size && s->regtype !=3D SCIx_RZV2H_S= CIF_REGTYPE) { u16 status =3D sci_serial_in(port, SCSPTR); =20 /* RTS# is always output; and active low, unless autorts */ @@ -852,8 +859,8 @@ static void sci_transmit_chars(struct uart_port *port) c =3D port->x_char; port->x_char =3D 0; } else if (stopped || !kfifo_get(&tport->xmit_fifo, &c)) { - if (port->type =3D=3D PORT_SCI && - kfifo_is_empty(&tport->xmit_fifo)) { + if (s->type =3D=3D PORT_SCI && + kfifo_is_empty(&tport->xmit_fifo)) { ctrl =3D sci_serial_in(port, SCSCR); ctrl &=3D ~SCSCR_TE; sci_serial_out(port, SCSCR, ctrl); @@ -873,7 +880,7 @@ static void sci_transmit_chars(struct uart_port *port) if (kfifo_len(&tport->xmit_fifo) < WAKEUP_CHARS) uart_write_wakeup(port); if (kfifo_is_empty(&tport->xmit_fifo)) { - if (port->type =3D=3D PORT_SCI) { + if (s->type =3D=3D PORT_SCI) { ctrl =3D sci_serial_in(port, SCSCR); ctrl &=3D ~SCSCR_TIE; ctrl |=3D SCSCR_TEIE; @@ -904,7 +911,7 @@ static void sci_receive_chars(struct uart_port *port) if (count =3D=3D 0) break; =20 - if (port->type =3D=3D PORT_SCI) { + if (s->type =3D=3D PORT_SCI) { char c =3D sci_serial_in(port, SCxRDR); if (uart_handle_sysrq_char(port, c)) count =3D 0; @@ -914,8 +921,8 @@ static void sci_receive_chars(struct uart_port *port) for (i =3D 0; i < count; i++) { char c; =20 - if (port->type =3D=3D PORT_SCIF || - port->type =3D=3D PORT_HSCIF) { + if (s->type =3D=3D PORT_SCIF || + s->type =3D=3D PORT_HSCIF) { status =3D sci_serial_in(port, SCxSR); c =3D sci_serial_in(port, SCxRDR); } else { @@ -1052,6 +1059,7 @@ static int sci_handle_breaks(struct uart_port *port) =20 static int scif_set_rtrg(struct uart_port *port, int rx_trig) { + struct sci_port *s =3D to_sci_port(port); unsigned int bits; =20 if (rx_trig >=3D port->fifosize) @@ -1065,7 +1073,7 @@ static int scif_set_rtrg(struct uart_port *port, int = rx_trig) return rx_trig; } =20 - switch (port->type) { + switch (s->type) { case PORT_SCIF: if (rx_trig < 4) { bits =3D 0; @@ -1150,7 +1158,7 @@ static ssize_t rx_fifo_trigger_store(struct device *d= ev, return ret; =20 sci->rx_trigger =3D sci->ops->set_rtrg(port, r); - if (port->type =3D=3D PORT_SCIFA || port->type =3D=3D PORT_SCIFB) + if (sci->type =3D=3D PORT_SCIFA || sci->type =3D=3D PORT_SCIFB) sci->ops->set_rtrg(port, 1); =20 return count; @@ -1166,7 +1174,7 @@ static ssize_t rx_fifo_timeout_show(struct device *de= v, struct sci_port *sci =3D to_sci_port(port); int v; =20 - if (port->type =3D=3D PORT_HSCIF) + if (sci->type =3D=3D PORT_HSCIF) v =3D sci->hscif_tot >> HSSCR_TOT_SHIFT; else v =3D sci->rx_fifo_timeout; @@ -1188,7 +1196,7 @@ static ssize_t rx_fifo_timeout_store(struct device *d= ev, if (ret) return ret; =20 - if (port->type =3D=3D PORT_HSCIF) { + if (sci->type =3D=3D PORT_HSCIF) { if (r < 0 || r > 3) return -EINVAL; sci->hscif_tot =3D r << HSSCR_TOT_SHIFT; @@ -1229,11 +1237,11 @@ static void sci_dma_tx_complete(void *arg) schedule_work(&s->work_tx); } else { s->cookie_tx =3D -EINVAL; - if (port->type =3D=3D PORT_SCIFA || port->type =3D=3D PORT_SCIFB || - s->cfg->regtype =3D=3D SCIx_RZ_SCIFA_REGTYPE) { + if (s->type =3D=3D PORT_SCIFA || s->type =3D=3D PORT_SCIFB || + s->regtype =3D=3D SCIx_RZ_SCIFA_REGTYPE) { u16 ctrl =3D sci_serial_in(port, SCSCR); sci_serial_out(port, SCSCR, ctrl & ~SCSCR_TIE); - if (s->cfg->regtype =3D=3D SCIx_RZ_SCIFA_REGTYPE) { + if (s->regtype =3D=3D SCIx_RZ_SCIFA_REGTYPE) { /* Switch irq from DMA to SCIF */ dmaengine_pause(s->chan_tx_saved); enable_irq(s->irqs[SCIx_TXI_IRQ]); @@ -1315,10 +1323,10 @@ static void sci_dma_rx_reenable_irq(struct sci_port= *s) =20 /* Direct new serial port interrupts back to CPU */ scr =3D sci_serial_in(port, SCSCR); - if (port->type =3D=3D PORT_SCIFA || port->type =3D=3D PORT_SCIFB || - s->cfg->regtype =3D=3D SCIx_RZ_SCIFA_REGTYPE) { + if (s->type =3D=3D PORT_SCIFA || s->type =3D=3D PORT_SCIFB || + s->regtype =3D=3D SCIx_RZ_SCIFA_REGTYPE) { enable_irq(s->irqs[SCIx_RXI_IRQ]); - if (s->cfg->regtype =3D=3D SCIx_RZ_SCIFA_REGTYPE) + if (s->regtype =3D=3D SCIx_RZ_SCIFA_REGTYPE) s->ops->set_rtrg(port, s->rx_trigger); else scr &=3D ~SCSCR_RDRQE; @@ -1558,8 +1566,8 @@ static enum hrtimer_restart sci_dma_rx_timer_fn(struc= t hrtimer *t) tty_flip_buffer_push(&port->state->port); } =20 - if (port->type =3D=3D PORT_SCIFA || port->type =3D=3D PORT_SCIFB || - s->cfg->regtype =3D=3D SCIx_RZ_SCIFA_REGTYPE) + if (s->type =3D=3D PORT_SCIFA || s->type =3D=3D PORT_SCIFB || + s->regtype =3D=3D SCIx_RZ_SCIFA_REGTYPE) sci_dma_rx_submit(s, true); =20 sci_dma_rx_reenable_irq(s); @@ -1682,8 +1690,8 @@ static void sci_request_dma(struct uart_port *port) =20 s->chan_rx_saved =3D s->chan_rx =3D chan; =20 - if (port->type =3D=3D PORT_SCIFA || port->type =3D=3D PORT_SCIFB || - s->cfg->regtype =3D=3D SCIx_RZ_SCIFA_REGTYPE) + if (s->type =3D=3D PORT_SCIFA || s->type =3D=3D PORT_SCIFB || + s->regtype =3D=3D SCIx_RZ_SCIFA_REGTYPE) sci_dma_rx_submit(s, false); } } @@ -1753,10 +1761,10 @@ static irqreturn_t sci_rx_interrupt(int irq, void *= ptr) u16 ssr =3D sci_serial_in(port, SCxSR); =20 /* Disable future Rx interrupts */ - if (port->type =3D=3D PORT_SCIFA || port->type =3D=3D PORT_SCIFB || - s->cfg->regtype =3D=3D SCIx_RZ_SCIFA_REGTYPE) { + if (s->type =3D=3D PORT_SCIFA || s->type =3D=3D PORT_SCIFB || + s->regtype =3D=3D SCIx_RZ_SCIFA_REGTYPE) { disable_irq_nosync(s->irqs[SCIx_RXI_IRQ]); - if (s->cfg->regtype =3D=3D SCIx_RZ_SCIFA_REGTYPE) { + if (s->regtype =3D=3D SCIx_RZ_SCIFA_REGTYPE) { s->ops->set_rtrg(port, 1); scr |=3D SCSCR_RIE; } else { @@ -1820,7 +1828,7 @@ static irqreturn_t sci_tx_end_interrupt(int irq, void= *ptr) unsigned long flags; u32 ctrl; =20 - if (port->type !=3D PORT_SCI) + if (s->type !=3D PORT_SCI) return sci_tx_interrupt(irq, ptr); =20 uart_port_lock_irqsave(port, &flags); @@ -1867,7 +1875,7 @@ static irqreturn_t sci_er_interrupt(int irq, void *pt= r) } =20 /* Handle errors */ - if (port->type =3D=3D PORT_SCI) { + if (s->type =3D=3D PORT_SCI) { if (sci_handle_errors(port)) { /* discard character in rx buffer */ sci_serial_in(port, SCxSR); @@ -2091,7 +2099,9 @@ static unsigned int sci_tx_empty(struct uart_port *po= rt) =20 static void sci_set_rts(struct uart_port *port, bool state) { - if (port->type =3D=3D PORT_SCIFA || port->type =3D=3D PORT_SCIFB) { + struct sci_port *s =3D to_sci_port(port); + + if (s->type =3D=3D PORT_SCIFA || s->type =3D=3D PORT_SCIFB) { u16 data =3D sci_serial_in(port, SCPDR); =20 /* Active low */ @@ -2118,7 +2128,9 @@ static void sci_set_rts(struct uart_port *port, bool = state) =20 static bool sci_get_cts(struct uart_port *port) { - if (port->type =3D=3D PORT_SCIFA || port->type =3D=3D PORT_SCIFB) { + struct sci_port *s =3D to_sci_port(port); + + if (s->type =3D=3D PORT_SCIFA || s->type =3D=3D PORT_SCIFB) { /* Active low */ return !(sci_serial_in(port, SCPDR) & SCPDR_CTSD); } else if (sci_getreg(port, SCSPTR)->size) { @@ -2164,21 +2176,21 @@ static void sci_set_mctrl(struct uart_port *port, u= nsigned int mctrl) =20 if (!(mctrl & TIOCM_RTS)) { /* Disable Auto RTS */ - if (s->cfg->regtype !=3D SCIx_RZV2H_SCIF_REGTYPE) + if (s->regtype !=3D SCIx_RZV2H_SCIF_REGTYPE) sci_serial_out(port, SCFCR, sci_serial_in(port, SCFCR) & ~SCFCR_MCE); =20 /* Clear RTS */ sci_set_rts(port, 0); } else if (s->autorts) { - if (port->type =3D=3D PORT_SCIFA || port->type =3D=3D PORT_SCIFB) { + if (s->type =3D=3D PORT_SCIFA || s->type =3D=3D PORT_SCIFB) { /* Enable RTS# pin function */ sci_serial_out(port, SCPCR, sci_serial_in(port, SCPCR) & ~SCPCR_RTSC); } =20 /* Enable Auto RTS */ - if (s->cfg->regtype !=3D SCIx_RZV2H_SCIF_REGTYPE) + if (s->regtype !=3D SCIx_RZV2H_SCIF_REGTYPE) sci_serial_out(port, SCFCR, sci_serial_in(port, SCFCR) | SCFCR_MCE); } else { @@ -2315,7 +2327,7 @@ static int sci_sck_calc(struct sci_port *s, unsigned = int bps, int err, min_err =3D INT_MAX; unsigned int sr; =20 - if (s->port.type !=3D PORT_HSCIF) + if (s->type !=3D PORT_HSCIF) freq *=3D 2; =20 for_each_sr(sr, s) { @@ -2342,7 +2354,7 @@ static int sci_brg_calc(struct sci_port *s, unsigned = int bps, int err, min_err =3D INT_MAX; unsigned int sr, dl; =20 - if (s->port.type !=3D PORT_HSCIF) + if (s->type !=3D PORT_HSCIF) freq *=3D 2; =20 for_each_sr(sr, s) { @@ -2375,7 +2387,7 @@ static int sci_scbrr_calc(struct sci_port *s, unsigne= d int bps, unsigned int sr, br, prediv, scrate, c; int err, min_err =3D INT_MAX; =20 - if (s->port.type !=3D PORT_HSCIF) + if (s->type !=3D PORT_HSCIF) freq *=3D 2; =20 /* @@ -2460,8 +2472,8 @@ static void sci_reset(struct uart_port *port) s->ops->set_rtrg(port, 1); timer_setup(&s->rx_fifo_timer, rx_fifo_timer_fn, 0); } else { - if (port->type =3D=3D PORT_SCIFA || - port->type =3D=3D PORT_SCIFB) + if (s->type =3D=3D PORT_SCIFA || + s->type =3D=3D PORT_SCIFB) s->ops->set_rtrg(port, 1); else s->ops->set_rtrg(port, s->rx_trigger); @@ -2521,8 +2533,8 @@ static void sci_set_termios(struct uart_port *port, s= truct ktermios *termios, */ =20 /* Optional Undivided External Clock */ - if (s->clk_rates[SCI_SCK] && port->type !=3D PORT_SCIFA && - port->type !=3D PORT_SCIFB) { + if (s->clk_rates[SCI_SCK] && s->type !=3D PORT_SCIFA && + s->type !=3D PORT_SCIFB) { err =3D sci_sck_calc(s, baud, &srr1); if (abs(err) < abs(min_err)) { best_clk =3D SCI_SCK; @@ -2607,7 +2619,7 @@ static void sci_set_termios(struct uart_port *port, s= truct ktermios *termios, sci_serial_out(port, SEMR, 0); =20 if (best_clk >=3D 0) { - if (port->type =3D=3D PORT_SCIFA || port->type =3D=3D PORT_SCIFB) + if (s->type =3D=3D PORT_SCIFA || s->type =3D=3D PORT_SCIFB) switch (srr + 1) { case 5: smr_val |=3D SCSMR_SRC_5; break; case 7: smr_val |=3D SCSMR_SRC_7; break; @@ -2692,12 +2704,12 @@ static void sci_set_termios(struct uart_port *port,= struct ktermios *termios, * (transmit interrupt enable) or in the same instruction to * start the transmitting process. So skip setting TE here for SCI. */ - if (port->type !=3D PORT_SCI) + if (s->type !=3D PORT_SCI) scr_val |=3D SCSCR_TE; scr_val |=3D SCSCR_RE | (s->cfg->scscr & ~(SCSCR_CKE1 | SCSCR_CKE0)); sci_serial_out(port, SCSCR, scr_val | s->hscif_tot); if ((srr + 1 =3D=3D 5) && - (port->type =3D=3D PORT_SCIFA || port->type =3D=3D PORT_SCIFB)) { + (s->type =3D=3D PORT_SCIFA || s->type =3D=3D PORT_SCIFB)) { /* * In asynchronous mode, when the sampling rate is 1/5, first * received data may become invalid on some SCIFA and SCIFB. @@ -2741,7 +2753,9 @@ void sci_pm(struct uart_port *port, unsigned int stat= e, =20 static const char *sci_type(struct uart_port *port) { - switch (port->type) { + struct sci_port *s =3D to_sci_port(port); + + switch (s->type) { case PORT_IRDA: return "irda"; case PORT_SCI: @@ -2825,8 +2839,7 @@ void sci_config_port(struct uart_port *port, int flag= s) { if (flags & UART_CONFIG_TYPE) { struct sci_port *sport =3D to_sci_port(port); - - port->type =3D sport->cfg->type; + port->type =3D SCI_PUBLIC_PORT_ID(sport->type); sci_request_port(port); } } @@ -2964,7 +2977,7 @@ static int sci_init_clocks(struct sci_port *sci_port,= struct device *dev) struct clk *clk; unsigned int i; =20 - if (sci_port->cfg->type =3D=3D PORT_HSCIF) + if (sci_port->type =3D=3D PORT_HSCIF) clk_names[SCI_SCK] =3D "hsck"; =20 for (i =3D 0; i < SCI_NUM_CLKS; i++) { @@ -3050,6 +3063,9 @@ static int sci_init_single(struct platform_device *de= v, =20 sci_port->cfg =3D p; =20 + sci_port->type =3D p->type; + sci_port->regtype =3D p->regtype; + port->iotype =3D UPIO_MEM; port->line =3D index; port->has_sysrq =3D IS_ENABLED(CONFIG_SERIAL_SH_SCI_CONSOLE); @@ -3128,11 +3144,11 @@ static int sci_init_single(struct platform_device *= dev, return ret; } =20 - port->type =3D p->type; + port->type =3D SCI_PUBLIC_PORT_ID(p->type); port->flags =3D UPF_FIXED_PORT | UPF_BOOT_AUTOCONF | p->flags; port->fifosize =3D sci_port->params->fifosize; =20 - if (port->type =3D=3D PORT_SCI && !dev->dev.of_node) { + if (p->type =3D=3D PORT_SCI && !dev->dev.of_node) { if (sci_port->reg_size >=3D 0x20) port->regshift =3D 2; else @@ -3322,13 +3338,13 @@ static struct uart_driver sci_uart_driver =3D { =20 static void sci_remove(struct platform_device *dev) { - struct sci_port *port =3D platform_get_drvdata(dev); - unsigned int type =3D port->port.type; /* uart_remove_... clears it */ + struct sci_port *s =3D platform_get_drvdata(dev); + unsigned int type =3D s->type; /* uart_remove_... clears it */ =20 - sci_ports_in_use &=3D ~BIT(port->port.line); - uart_remove_one_port(&sci_uart_driver, &port->port); + sci_ports_in_use &=3D ~BIT(s->port.line); + uart_remove_one_port(&sci_uart_driver, &s->port); =20 - if (port->port.fifosize > 1) + if (s->port.fifosize > 1) device_remove_file(&dev->dev, &dev_attr_rx_fifo_trigger); if (type =3D=3D PORT_SCIFA || type =3D=3D PORT_SCIFB || type =3D=3D PORT_= HSCIF) device_remove_file(&dev->dev, &dev_attr_rx_fifo_timeout); @@ -3682,8 +3698,8 @@ static int sci_probe(struct platform_device *dev) if (ret) return ret; } - if (sp->port.type =3D=3D PORT_SCIFA || sp->port.type =3D=3D PORT_SCIFB || - sp->port.type =3D=3D PORT_HSCIF) { + if (sp->type =3D=3D PORT_SCIFA || sp->type =3D=3D PORT_SCIFB || + sp->type =3D=3D PORT_HSCIF) { ret =3D device_create_file(&dev->dev, &dev_attr_rx_fifo_timeout); if (ret) { if (sp->port.fifosize > 1) { @@ -3799,8 +3815,11 @@ int __init scix_early_console_setup(struct earlycon_= device *device, if (!device->port.membase) return -ENODEV; =20 - device->port.type =3D data->type; + device->port.type =3D SCI_PUBLIC_PORT_ID(data->type); + sci_ports[0].port =3D device->port; + sci_ports[0].type =3D data->type; + sci_ports[0].regtype =3D data->regtype; =20 port_cfg.type =3D data->type; port_cfg.regtype =3D data->regtype; --=20 2.49.0 From nobody Wed Oct 8 07:26:38 2025 Received: from mail-wr1-f45.google.com (mail-wr1-f45.google.com [209.85.221.45]) (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 EAA9B292B4D; Mon, 30 Jun 2025 20:23:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.45 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751315016; cv=none; b=koDT5h023Rljnkx+quPuy1WZ2ObwmkFjG9WleQ1M+wo4vKWTzAP7r41yUQgOIrbjdgguVTnb7zZz22S4KZoquqHaOU1Zo5vFclwdfdzUJaF8rtW5QOkHDi0nyRAIu0n2/rh4Hr4OAD+/rTNeYbPBZyVp/0Sv4xPA8QIhF53q8BU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751315016; c=relaxed/simple; bh=OkjUanlnS7e68ROV3fxoeqPS6k8vxvE+BBo/IEa0KQI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=kfPpS4Yd2toBftehjUpimTVrNPdt8BlaPTADflhIA1prYFw3V3IxBTKXZd5TY0Dy/wL6jWrHEx7wMEuigx23Hzcm0AXLn5qDLcbd/s58IjNc2iNUZN1bx++gYGWFLumHOKPb39Tq409/kgOea1YlDxkdX2UYZW/T1iSoJ7Ylxoc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=RJ9scd6K; arc=none smtp.client-ip=209.85.221.45 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="RJ9scd6K" Received: by mail-wr1-f45.google.com with SMTP id ffacd0b85a97d-3a582e09144so1968678f8f.1; Mon, 30 Jun 2025 13:23:33 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1751315012; x=1751919812; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=vOAmUo64oafhZlpyGfS3XJ5Wo8a09Il5eFelgtOy+VE=; b=RJ9scd6KHOoa/2BCI4OwKpFYkWrBM+ioRZILS743Rcvfa08zFt+YYdJzgmKqj9fWcx YlalQr1DrDunmLLSXGhxNZ2zO3YYM9hY1CF540wSDk5a41YZMfNxGgPkmADisRVH+rl8 ROSZPmELTqYZgR3S3ZbKrGmjmKSCRTyKXu3IuNxwMA/lO6Rw4iR0D7Pk8qyrOQVKRU0K iqc+Nxx6Mhofi/vZw8OpKVoUPFYhpG/kOB/Qp0sCVEl3yJsfymxFw8pB5Sfjapvcn2hs 7tuRAyPi+L3wg2eqlsm1FdXyeXgMb6fr2ZrHls9twZ+5T6GWJeYFxrUEsC46i5Re1nF6 Mcgg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1751315012; x=1751919812; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=vOAmUo64oafhZlpyGfS3XJ5Wo8a09Il5eFelgtOy+VE=; b=ojgcWxb3X0o02Ms5OaT+dzuHr+hmUPYq7oF+RzjO4PJEG8dq67l3whLYZlzS03vC2x 1TIEONnP64FIL/6t/LDsZaayBlh01geEloOtA5VQox/KMfXuM0T9y9ANLnMELqti34i9 RvIiptLR1DPeE5PN4liDTfqpKpZUIvFS3u7Bp6Co6J7BQGLPAupTGRtkhljdpv08DvwV PUa4emg/SamlkwUWcgHRFBlRveCugiQyR94Z7lnSZcvb+loup7SyTWUqcmewu06rBvNz JpEssJVwu1yp+U9K8VRW3CkSCoxc2Q3NT8CA6rp9xsrQB7v2h1L0H2Ae19c6hd8+d0uW AiEw== X-Forwarded-Encrypted: i=1; AJvYcCVVmT2oDG5hwW17hSvdH5P0a14yZcasjYPZThskB957qTcf5urlJJNxAS0IeqD/JaCfAa4Q3SaZ/chYp5Jj@vger.kernel.org, AJvYcCW6lUbn+mS4All7PEV8Sg2oA4/m846bMyIOv7sostbyFN1zNf3kA3WRw6u4lAmnsz7mqgyh6NoBO9l3z4sUrYtvu4Y=@vger.kernel.org, AJvYcCXULnXuxN4ZgI0/75vMd1D7/1bYnkjR2+rINZtdo2AwoI6mW5nkAQO+WtTkA6hYXRoTOSJ3LO0/JsWl@vger.kernel.org X-Gm-Message-State: AOJu0YwD2D5W0h2AVio1CQGGZU01OE0RSb5iOi9zvAenbEMHq1tvT8vm 4o35Alkm3YKl8q/TZVjF1rZ2bLIjMqsKF2ROyFMv9eb2k9tE9m9n+a03 X-Gm-Gg: ASbGnctB+PECU8qfBHxuPsV8tVVcI7K4sMycs9RVB/e1zUPOdbchn9izrtzcgq2/GXs MdoHRaZARqwaMBRghGje+hdSW+tvtmJvrJOcLNfmqK0QScP943Bi6OqC60GiYoMlOpJL8mcPbOo v4Ct54/hg0in2ek7EsE9zOyM2DxIluYgpACoek9z381dSJRq6yKRYv+dd7zLvPIKJUrMRcBPiZD zNvVVXqrI0S6fbPshsQA6peMQY0xHSITefQTeyJtY3ZExDep3XgMlCUxexsBaTDrBFrw8E+hOdE 4JsLwxxA3ge4/+YI+ktYps/2TOy0UysWhliaywa2vLtBV1MQrXw0QhGwAkCuiRwMf6LPir50P1X BHrd+Qj76EKdyp5V4NRVdWg== X-Google-Smtp-Source: AGHT+IEr4i1YSWrQOsR+d1vGp9CXZdB++h+DmaG8U3R2ROpXFIfuCgJMXpyJKyF9SobTOh3k2c/Ikg== X-Received: by 2002:adf:9c8d:0:b0:3a4:febd:39f2 with SMTP id ffacd0b85a97d-3a8ff614c78mr8399937f8f.37.1751315012044; Mon, 30 Jun 2025 13:23:32 -0700 (PDT) Received: from iku.Home ([2a06:5906:61b:2d00:a065:f77f:7ac4:1c25]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4538a3a5b7fsm148523195e9.10.2025.06.30.13.23.30 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 30 Jun 2025 13:23:31 -0700 (PDT) From: Prabhakar X-Google-Original-From: Prabhakar To: Greg Kroah-Hartman , Jiri Slaby , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Geert Uytterhoeven , Magnus Damm , Wolfram Sang , linux-serial@vger.kernel.org, devicetree@vger.kernel.org Cc: linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org, Prabhakar , Biju Das , Fabrizio Castro , Lad Prabhakar Subject: [PATCH v14 5/5] serial: sh-sci: Add support for RZ/T2H SCI Date: Mon, 30 Jun 2025 21:23:23 +0100 Message-ID: <20250630202323.279809-6-prabhakar.mahadev-lad.rj@bp.renesas.com> X-Mailer: git-send-email 2.49.0 In-Reply-To: <20250630202323.279809-1-prabhakar.mahadev-lad.rj@bp.renesas.com> References: <20250630202323.279809-1-prabhakar.mahadev-lad.rj@bp.renesas.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Thierry Bultel Define a new RSCI port type, and the RSCI 32 bits registers set. The RZ/T2H SCI has a a fifo, and a quite different set of registers from the original SH SCI ones. DMA is not supported yet. Signed-off-by: Thierry Bultel Signed-off-by: Lad Prabhakar Reviewed-by: Wolfram Sang Reviewed-by: Geert Uytterhoeven --- v13->v14: - Switched to using `EXPORT_SYMBOL_NS_GPL` for all exported symbols in the sh-sci driver to allow RSCI driver to use SH-SCI symbols. - Added MODULE_IMPORT_NS for SH_SCI to allow RSCI driver to use SH-SCI symbols. v12->v13: - No changes --- drivers/tty/serial/Kconfig | 7 + drivers/tty/serial/Makefile | 1 + drivers/tty/serial/rsci.c | 480 +++++++++++++++++++++++++++++ drivers/tty/serial/rsci.h | 10 + drivers/tty/serial/sh-sci-common.h | 5 + drivers/tty/serial/sh-sci.c | 53 +++- 6 files changed, 546 insertions(+), 10 deletions(-) create mode 100644 drivers/tty/serial/rsci.c create mode 100644 drivers/tty/serial/rsci.h diff --git a/drivers/tty/serial/Kconfig b/drivers/tty/serial/Kconfig index 79a8186d3361..44427415a80d 100644 --- a/drivers/tty/serial/Kconfig +++ b/drivers/tty/serial/Kconfig @@ -675,6 +675,13 @@ config SERIAL_SH_SCI_DMA depends on SERIAL_SH_SCI && DMA_ENGINE default ARCH_RENESAS =20 +config SERIAL_RSCI + tristate "Support for Renesas RZ/T2H SCI variant" + depends on SERIAL_SH_SCI + help + Support for the RZ/T2H SCI variant with fifo. + Say Y if you want to be able to use the RZ/T2H SCI serial port. + config SERIAL_HS_LPC32XX tristate "LPC32XX high speed serial port support" depends on ARCH_LPC32XX || COMPILE_TEST diff --git a/drivers/tty/serial/Makefile b/drivers/tty/serial/Makefile index d58d9f719889..a2ccbc508ec5 100644 --- a/drivers/tty/serial/Makefile +++ b/drivers/tty/serial/Makefile @@ -71,6 +71,7 @@ obj-$(CONFIG_SERIAL_QCOM_GENI) +=3D qcom_geni_serial.o obj-$(CONFIG_SERIAL_QE) +=3D ucc_uart.o obj-$(CONFIG_SERIAL_RDA) +=3D rda-uart.o obj-$(CONFIG_SERIAL_RP2) +=3D rp2.o +obj-$(CONFIG_SERIAL_RSCI) +=3D rsci.o obj-$(CONFIG_SERIAL_SA1100) +=3D sa1100.o obj-$(CONFIG_SERIAL_SAMSUNG) +=3D samsung_tty.o obj-$(CONFIG_SERIAL_SB1250_DUART) +=3D sb1250-duart.o diff --git a/drivers/tty/serial/rsci.c b/drivers/tty/serial/rsci.c new file mode 100644 index 000000000000..b3c48dc1e07d --- /dev/null +++ b/drivers/tty/serial/rsci.c @@ -0,0 +1,480 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Copyright (C) 2025 Renesas Electronics Corp. + */ + +#include +#include +#include +#include +#include +#include +#include +#include +#include "rsci.h" + +MODULE_IMPORT_NS("SH_SCI"); + +/* RSCI registers */ +#define RDR 0x00 +#define TDR 0x04 +#define CCR0 0x08 +#define CCR1 0x0C +#define CCR2 0x10 +#define CCR3 0x14 +#define CCR4 0x18 +#define FCR 0x24 +#define DCR 0x30 +#define CSR 0x48 +#define FRSR 0x50 +#define FTSR 0x54 +#define CFCLR 0x68 +#define FFCLR 0x70 + +/* RDR (Receive Data Register) */ +#define RDR_FFER BIT(12) /* FIFO Framing Error */ +#define RDR_FPER BIT(11) /* FIFO Parity Error */ +#define RDR_RDAT_MSK GENMASK(8, 0) + +/* TDR (Transmit Data Register) */ +#define TDR_MPBT BIT(9) /* Multiprocessor Transfer */ +#define TDR_TDAT_9BIT_LSHIFT 0 +#define TDR_TDAT_9BIT_VAL 0x1FF +#define TDR_TDAT_9BIT_MSK (TDR_TDAT_9BIT_VAL << TDR_TDAT_9BIT_LSHIFT) + +/* CCR0 (Common Control Register 0) */ +#define CCR0_SSE BIT(24) /* SSn# Pin Function Enable */ +#define CCR0_TEIE BIT(21) /* Transmit End Interrupt Enable */ +#define CCR0_TIE BIT(20) /* Transmit Interrupt Enable */ +#define CCR0_RIE BIT(16) /* Receive Interrupt Enable */ +#define CCR0_IDSEL BIT(10) /* ID Frame Select */ +#define CCR0_DCME BIT(9) /* Data Compare Match Enable */ +#define CCR0_MPIE BIT(8) /* Multiprocessor Interrupt Enable */ +#define CCR0_TE BIT(4) /* Transmit Enable */ +#define CCR0_RE BIT(0) /* Receive Enable */ + +/* CCR1 (Common Control Register 1) */ +#define CCR1_NFEN BIT(28) /* Digital Noise Filter Function */ +#define CCR1_SHARPS BIT(20) /* Half -duplex Communication Select */ +#define CCR1_SPLP BIT(16) /* Loopback Control */ +#define CCR1_RINV BIT(13) /* RxD invert */ +#define CCR1_TINV BIT(12) /* TxD invert */ +#define CCR1_PM BIT(9) /* Parity Mode */ +#define CCR1_PE BIT(8) /* Parity Enable */ +#define CCR1_SPB2IO BIT(5) /* Serial Port Break I/O */ +#define CCR1_SPB2DT BIT(4) /* Serial Port Break Data Select */ +#define CCR1_CTSPEN BIT(1) /* CTS External Pin Enable */ +#define CCR1_CTSE BIT(0) /* CTS Enable */ + +/* FCR (FIFO Control Register) */ +#define FCR_RFRST BIT(23) /* Receive FIFO Data Register Reset */ +#define FCR_TFRST BIT(15) /* Transmit FIFO Data Register Reset */ +#define FCR_DRES BIT(0) /* Incoming Data Ready Error Select */ +#define FCR_RTRG4_0 GENMASK(20, 16) +#define FCR_TTRG GENMASK(12, 8) + +/* CSR (Common Status Register) */ +#define CSR_RDRF BIT(31) /* Receive Data Full */ +#define CSR_TEND BIT(30) /* Transmit End Flag */ +#define CSR_TDRE BIT(29) /* Transmit Data Empty */ +#define CSR_FER BIT(28) /* Framing Error */ +#define CSR_PER BIT(27) /* Parity Error */ +#define CSR_MFF BIT(26) /* Mode Fault Error */ +#define CSR_ORER BIT(24) /* Overrun Error */ +#define CSR_DFER BIT(18) /* Data Compare Match Framing Error */ +#define CSR_DPER BIT(17) /* Data Compare Match Parity Error */ +#define CSR_DCMF BIT(16) /* Data Compare Match */ +#define CSR_RXDMON BIT(15) /* Serial Input Data Monitor */ +#define CSR_ERS BIT(4) /* Error Signal Status */ + +#define SCxSR_ERRORS(port) (to_sci_port(port)->params->error_mask) +#define SCxSR_ERROR_CLEAR(port) (to_sci_port(port)->params->error_clear) + +#define RSCI_DEFAULT_ERROR_MASK (CSR_PER | CSR_FER) + +#define RSCI_RDxF_CLEAR (CFCLR_RDRFC) +#define RSCI_ERROR_CLEAR (CFCLR_PERC | CFCLR_FERC) +#define RSCI_TDxE_CLEAR (CFCLR_TDREC) +#define RSCI_BREAK_CLEAR (CFCLR_PERC | CFCLR_FERC | CFCLR_ORERC) + +/* FRSR (FIFO Receive Status Register) */ +#define FRSR_R5_0 GENMASK(13, 8) /* Receive FIFO Data Count */ +#define FRSR_DR BIT(0) /* Receive Data Ready */ + +/* CFCLR (Common Flag CLear Register) */ +#define CFCLR_RDRFC BIT(31) /* RDRF Clear */ +#define CFCLR_TDREC BIT(29) /* TDRE Clear */ +#define CFCLR_FERC BIT(28) /* FER Clear */ +#define CFCLR_PERC BIT(27) /* PER Clear */ +#define CFCLR_MFFC BIT(26) /* MFF Clear */ +#define CFCLR_ORERC BIT(24) /* ORER Clear */ +#define CFCLR_DFERC BIT(18) /* DFER Clear */ +#define CFCLR_DPERC BIT(17) /* DPER Clear */ +#define CFCLR_DCMFC BIT(16) /* DCMF Clear */ +#define CFCLR_ERSC BIT(4) /* ERS Clear */ +#define CFCLR_CLRFLAG (CFCLR_RDRFC | CFCLR_FERC | CFCLR_PERC | \ + CFCLR_MFFC | CFCLR_ORERC | CFCLR_DFERC | \ + CFCLR_DPERC | CFCLR_DCMFC | CFCLR_ERSC) + +/* FFCLR (FIFO Flag CLear Register) */ +#define FFCLR_DRC BIT(0) /* DR Clear */ + +#define DCR_DEPOL BIT(0) + +static u32 rsci_serial_in(struct uart_port *p, int offset) +{ + return readl(p->membase + offset); +} + +static void rsci_serial_out(struct uart_port *p, int offset, int value) +{ + writel(value, p->membase + offset); +} + +static void rsci_clear_DRxC(struct uart_port *port) +{ + rsci_serial_out(port, CFCLR, CFCLR_RDRFC); + rsci_serial_out(port, FFCLR, FFCLR_DRC); +} + +static void rsci_clear_SCxSR(struct uart_port *port, unsigned int mask) +{ + rsci_serial_out(port, CFCLR, mask); +} + +static void rsci_start_rx(struct uart_port *port) +{ + unsigned int ctrl; + + ctrl =3D rsci_serial_in(port, CCR0); + ctrl |=3D CCR0_RIE; + rsci_serial_out(port, CCR0, ctrl); +} + +static void rsci_set_termios(struct uart_port *port, struct ktermios *term= ios, + const struct ktermios *old) +{ + struct sci_port *s =3D to_sci_port(port); + unsigned long flags; + + sci_port_enable(s); + uart_port_lock_irqsave(port, &flags); + + /* For now, only RX enabling is supported */ + if (termios->c_cflag & CREAD) + rsci_start_rx(port); + + uart_port_unlock_irqrestore(port, flags); + sci_port_disable(s); +} + +static int rsci_txfill(struct uart_port *port) +{ + return rsci_serial_in(port, FTSR); +} + +static int rsci_rxfill(struct uart_port *port) +{ + u32 val =3D rsci_serial_in(port, FRSR); + + return FIELD_GET(FRSR_R5_0, val); +} + +static unsigned int rsci_tx_empty(struct uart_port *port) +{ + unsigned int status =3D rsci_serial_in(port, CSR); + unsigned int in_tx_fifo =3D rsci_txfill(port); + + return (status & CSR_TEND) && !in_tx_fifo ? TIOCSER_TEMT : 0; +} + +static void rsci_set_mctrl(struct uart_port *port, unsigned int mctrl) +{ + /* Not supported yet */ +} + +static unsigned int rsci_get_mctrl(struct uart_port *port) +{ + /* Not supported yet */ + return 0; +} + +static void rsci_clear_CFC(struct uart_port *port, unsigned int mask) +{ + rsci_serial_out(port, CFCLR, mask); +} + +static void rsci_start_tx(struct uart_port *port) +{ + struct sci_port *sp =3D to_sci_port(port); + u32 ctrl; + + if (sp->chan_tx) + return; + + /* + * TE (Transmit Enable) must be set after setting TIE + * (Transmit Interrupt Enable) or in the same instruction + * to start the transmit process. + */ + ctrl =3D rsci_serial_in(port, CCR0); + ctrl |=3D CCR0_TIE | CCR0_TE; + rsci_serial_out(port, CCR0, ctrl); +} + +static void rsci_stop_tx(struct uart_port *port) +{ + u32 ctrl; + + ctrl =3D rsci_serial_in(port, CCR0); + ctrl &=3D ~CCR0_TIE; + rsci_serial_out(port, CCR0, ctrl); +} + +static void rsci_stop_rx(struct uart_port *port) +{ + u32 ctrl; + + ctrl =3D rsci_serial_in(port, CCR0); + ctrl &=3D ~CCR0_RIE; + rsci_serial_out(port, CCR0, ctrl); +} + +static int rsci_txroom(struct uart_port *port) +{ + return port->fifosize - rsci_txfill(port); +} + +static void rsci_transmit_chars(struct uart_port *port) +{ + unsigned int stopped =3D uart_tx_stopped(port); + struct tty_port *tport =3D &port->state->port; + u32 status, ctrl; + int count; + + status =3D rsci_serial_in(port, CSR); + if (!(status & CSR_TDRE)) { + ctrl =3D rsci_serial_in(port, CCR0); + if (kfifo_is_empty(&tport->xmit_fifo)) + ctrl &=3D ~CCR0_TIE; + else + ctrl |=3D CCR0_TIE; + rsci_serial_out(port, CCR0, ctrl); + return; + } + + count =3D rsci_txroom(port); + + do { + unsigned char c; + + if (port->x_char) { + c =3D port->x_char; + port->x_char =3D 0; + } else if (stopped || !kfifo_get(&tport->xmit_fifo, &c)) { + break; + } + + rsci_clear_CFC(port, CFCLR_TDREC); + rsci_serial_out(port, TDR, c); + + port->icount.tx++; + } while (--count > 0); + + if (kfifo_len(&tport->xmit_fifo) < WAKEUP_CHARS) + uart_write_wakeup(port); + + if (kfifo_is_empty(&tport->xmit_fifo)) { + ctrl =3D rsci_serial_in(port, CCR0); + ctrl &=3D ~CCR0_TIE; + ctrl |=3D CCR0_TEIE; + rsci_serial_out(port, CCR0, ctrl); + } +} + +static void rsci_receive_chars(struct uart_port *port) +{ + struct tty_port *tport =3D &port->state->port; + u32 rdat, status, frsr_status =3D 0; + int i, count, copied =3D 0; + unsigned char flag; + + status =3D rsci_serial_in(port, CSR); + frsr_status =3D rsci_serial_in(port, FRSR); + + if (!(status & CSR_RDRF) && !(frsr_status & FRSR_DR)) + return; + + while (1) { + /* Don't copy more bytes than there is room for in the buffer */ + count =3D tty_buffer_request_room(tport, rsci_rxfill(port)); + + /* If for any reason we can't copy more data, we're done! */ + if (count =3D=3D 0) + break; + + for (i =3D 0; i < count; i++) { + char c; + + rdat =3D rsci_serial_in(port, RDR); + /* 9-bits data is not supported yet */ + c =3D rdat & RDR_RDAT_MSK; + + if (uart_handle_sysrq_char(port, c)) { + count--; + i--; + continue; + } + + /* Store data and status. + * Non FIFO mode is not supported + */ + if (rdat & RDR_FFER) { + flag =3D TTY_FRAME; + port->icount.frame++; + } else if (rdat & RDR_FPER) { + flag =3D TTY_PARITY; + port->icount.parity++; + } else { + flag =3D TTY_NORMAL; + } + + tty_insert_flip_char(tport, c, flag); + } + + rsci_serial_in(port, CSR); /* dummy read */ + rsci_clear_DRxC(port); + + copied +=3D count; + port->icount.rx +=3D count; + } + + if (copied) { + /* Tell the rest of the system the news. New characters! */ + tty_flip_buffer_push(tport); + } else { + /* TTY buffers full; read from RX reg to prevent lockup */ + rsci_serial_in(port, RDR); + rsci_serial_in(port, CSR); /* dummy read */ + rsci_clear_DRxC(port); + } +} + +static void rsci_poll_put_char(struct uart_port *port, unsigned char c) +{ + u32 status; + int ret; + + ret =3D readl_relaxed_poll_timeout_atomic(port->membase + CSR, status, + (status & CSR_TDRE), 100, + USEC_PER_SEC); + if (ret !=3D 0) { + dev_err(port->dev, + "Error while sending data in UART TX : %d\n", ret); + goto done; + } + rsci_serial_out(port, TDR, c); +done: + rsci_clear_SCxSR(port, CFCLR_TDREC); +} + +static void rsci_prepare_console_write(struct uart_port *port, u32 ctrl) +{ + struct sci_port *s =3D to_sci_port(port); + u32 ctrl_temp =3D + s->params->param_bits->rxtx_enable | CCR0_TIE | + s->hscif_tot; + rsci_serial_out(port, CCR0, ctrl_temp); +} + +static const char *rsci_type(struct uart_port *port) +{ + return "rsci"; +} + +static size_t rsci_suspend_regs_size(void) +{ + return 0; +} + +static void rsci_shutdown_complete(struct uart_port *port) +{ + /* + * Stop RX and TX, disable related interrupts, keep clock source + */ + rsci_serial_out(port, CCR0, 0); +} + +static const struct sci_common_regs rsci_common_regs =3D { + .status =3D CSR, + .control =3D CCR0, +}; + +static const struct sci_port_params_bits rsci_port_param_bits =3D { + .rxtx_enable =3D CCR0_RE | CCR0_TE, + .te_clear =3D CCR0_TE | CCR0_TEIE, + .poll_sent_bits =3D CSR_TDRE | CSR_TEND, +}; + +static const struct sci_port_params rsci_port_params =3D { + .fifosize =3D 16, + .overrun_reg =3D CSR, + .overrun_mask =3D CSR_ORER, + .sampling_rate_mask =3D SCI_SR(32), + .error_mask =3D RSCI_DEFAULT_ERROR_MASK, + .error_clear =3D RSCI_ERROR_CLEAR, + .param_bits =3D &rsci_port_param_bits, + .common_regs =3D &rsci_common_regs, +}; + +static const struct uart_ops rsci_uart_ops =3D { + .tx_empty =3D rsci_tx_empty, + .set_mctrl =3D rsci_set_mctrl, + .get_mctrl =3D rsci_get_mctrl, + .start_tx =3D rsci_start_tx, + .stop_tx =3D rsci_stop_tx, + .stop_rx =3D rsci_stop_rx, + .startup =3D sci_startup, + .shutdown =3D sci_shutdown, + .set_termios =3D rsci_set_termios, + .pm =3D sci_pm, + .type =3D rsci_type, + .release_port =3D sci_release_port, + .request_port =3D sci_request_port, + .config_port =3D sci_config_port, + .verify_port =3D sci_verify_port, +}; + +static const struct sci_port_ops rsci_port_ops =3D { + .read_reg =3D rsci_serial_in, + .write_reg =3D rsci_serial_out, + .clear_SCxSR =3D rsci_clear_SCxSR, + .transmit_chars =3D rsci_transmit_chars, + .receive_chars =3D rsci_receive_chars, + .poll_put_char =3D rsci_poll_put_char, + .prepare_console_write =3D rsci_prepare_console_write, + .suspend_regs_size =3D rsci_suspend_regs_size, + .shutdown_complete =3D rsci_shutdown_complete, +}; + +struct sci_of_data of_sci_rsci_data =3D { + .type =3D SCI_PORT_RSCI, + .ops =3D &rsci_port_ops, + .uart_ops =3D &rsci_uart_ops, + .params =3D &rsci_port_params, +}; + +#ifdef CONFIG_SERIAL_SH_SCI_EARLYCON + +static int __init rsci_early_console_setup(struct earlycon_device *device, + const char *opt) +{ + return scix_early_console_setup(device, &of_sci_rsci_data); +} + +OF_EARLYCON_DECLARE(rsci, "renesas,r9a09g077-rsci", rsci_early_console_set= up); + +#endif /* CONFIG_SERIAL_SH_SCI_EARLYCON */ + +MODULE_LICENSE("GPL"); +MODULE_DESCRIPTION("RSCI serial driver"); diff --git a/drivers/tty/serial/rsci.h b/drivers/tty/serial/rsci.h new file mode 100644 index 000000000000..2af3f28b465a --- /dev/null +++ b/drivers/tty/serial/rsci.h @@ -0,0 +1,10 @@ +/* SPDX-License-Identifier: GPL-2.0 */ + +#ifndef __RSCI_H__ +#define __RSCI_H__ + +#include "sh-sci-common.h" + +extern struct sci_of_data of_sci_rsci_data; + +#endif /* __RSCI_H__ */ diff --git a/drivers/tty/serial/sh-sci-common.h b/drivers/tty/serial/sh-sci= -common.h index fcddf66780c9..e3c028df14f1 100644 --- a/drivers/tty/serial/sh-sci-common.h +++ b/drivers/tty/serial/sh-sci-common.h @@ -5,6 +5,11 @@ =20 #include =20 +/* Private port IDs */ +enum SCI_PORT_TYPE { + SCI_PORT_RSCI =3D BIT(7) | 0, +}; + enum SCI_CLKS { SCI_FCK, /* Functional Clock */ SCI_SCK, /* Optional External Clock */ diff --git a/drivers/tty/serial/sh-sci.c b/drivers/tty/serial/sh-sci.c index 26536ff2eda1..805be97e9146 100644 --- a/drivers/tty/serial/sh-sci.c +++ b/drivers/tty/serial/sh-sci.c @@ -54,6 +54,7 @@ #include #endif =20 +#include "rsci.h" #include "serial_mctrl_gpio.h" #include "sh-sci.h" #include "sh-sci-common.h" @@ -550,6 +551,7 @@ void sci_port_enable(struct sci_port *sci_port) } sci_port->port.uartclk =3D sci_port->clk_rates[SCI_FCK]; } +EXPORT_SYMBOL_NS_GPL(sci_port_enable, "SH_SCI"); =20 void sci_port_disable(struct sci_port *sci_port) { @@ -563,6 +565,7 @@ void sci_port_disable(struct sci_port *sci_port) =20 pm_runtime_put_sync(sci_port->port.dev); } +EXPORT_SYMBOL_NS_GPL(sci_port_disable, "SH_SCI"); =20 static inline unsigned long port_rx_irq_mask(struct uart_port *port) { @@ -1828,7 +1831,7 @@ static irqreturn_t sci_tx_end_interrupt(int irq, void= *ptr) unsigned long flags; u32 ctrl; =20 - if (s->type !=3D PORT_SCI) + if (s->type !=3D PORT_SCI && s->type !=3D SCI_PORT_RSCI) return sci_tx_interrupt(irq, ptr); =20 uart_port_lock_irqsave(port, &flags); @@ -2289,6 +2292,7 @@ int sci_startup(struct uart_port *port) =20 return 0; } +EXPORT_SYMBOL_NS_GPL(sci_startup, "SH_SCI"); =20 void sci_shutdown(struct uart_port *port) { @@ -2319,6 +2323,7 @@ void sci_shutdown(struct uart_port *port) sci_free_irq(s); sci_free_dma(port); } +EXPORT_SYMBOL_NS_GPL(sci_shutdown, "SH_SCI"); =20 static int sci_sck_calc(struct sci_port *s, unsigned int bps, unsigned int *srr) @@ -2750,6 +2755,7 @@ void sci_pm(struct uart_port *port, unsigned int stat= e, break; } } +EXPORT_SYMBOL_NS_GPL(sci_pm, "SH_SCI"); =20 static const char *sci_type(struct uart_port *port) { @@ -2812,6 +2818,7 @@ void sci_release_port(struct uart_port *port) =20 release_mem_region(port->mapbase, sport->reg_size); } +EXPORT_SYMBOL_NS_GPL(sci_release_port, "SH_SCI"); =20 int sci_request_port(struct uart_port *port) { @@ -2834,6 +2841,7 @@ int sci_request_port(struct uart_port *port) =20 return 0; } +EXPORT_SYMBOL_NS_GPL(sci_request_port, "SH_SCI"); =20 void sci_config_port(struct uart_port *port, int flags) { @@ -2843,6 +2851,7 @@ void sci_config_port(struct uart_port *port, int flag= s) sci_request_port(port); } } +EXPORT_SYMBOL_NS_GPL(sci_config_port, "SH_SCI"); =20 int sci_verify_port(struct uart_port *port, struct serial_struct *ser) { @@ -2852,6 +2861,7 @@ int sci_verify_port(struct uart_port *port, struct se= rial_struct *ser) =20 return 0; } +EXPORT_SYMBOL_NS_GPL(sci_verify_port, "SH_SCI"); =20 static void sci_prepare_console_write(struct uart_port *port, u32 ctrl) { @@ -2977,14 +2987,27 @@ static int sci_init_clocks(struct sci_port *sci_por= t, struct device *dev) struct clk *clk; unsigned int i; =20 - if (sci_port->type =3D=3D PORT_HSCIF) + if (sci_port->type =3D=3D PORT_HSCIF) { clk_names[SCI_SCK] =3D "hsck"; + } else if (sci_port->type =3D=3D SCI_PORT_RSCI) { + clk_names[SCI_FCK] =3D "operation"; + clk_names[SCI_BRG_INT] =3D "bus"; + } =20 for (i =3D 0; i < SCI_NUM_CLKS; i++) { - clk =3D devm_clk_get_optional(dev, clk_names[i]); + const char *name =3D clk_names[i]; + + clk =3D devm_clk_get_optional(dev, name); if (IS_ERR(clk)) return PTR_ERR(clk); =20 + if (!clk && sci_port->type =3D=3D SCI_PORT_RSCI && + (i =3D=3D SCI_FCK || i =3D=3D SCI_BRG_INT)) { + return dev_err_probe(dev, -ENODEV, + "failed to get %s\n", + name); + } + if (!clk && i =3D=3D SCI_FCK) { /* * Not all SH platforms declare a clock lookup entry @@ -2995,13 +3018,13 @@ static int sci_init_clocks(struct sci_port *sci_por= t, struct device *dev) if (IS_ERR(clk)) return dev_err_probe(dev, PTR_ERR(clk), "failed to get %s\n", - clk_names[i]); + name); } =20 if (!clk) - dev_dbg(dev, "failed to get %s\n", clk_names[i]); + dev_dbg(dev, "failed to get %s\n", name); else - dev_dbg(dev, "clk %s is %pC rate %lu\n", clk_names[i], + dev_dbg(dev, "clk %s is %pC rate %lu\n", name, clk, clk_get_rate(clk)); sci_port->clks[i] =3D clk; } @@ -3085,10 +3108,10 @@ static int sci_init_single(struct platform_device *= dev, } =20 /* - * The fourth interrupt on SCI port is transmit end interrupt, so + * The fourth interrupt on SCI and RSCI port is transmit end interrupt, so * shuffle the interrupts. */ - if (p->type =3D=3D PORT_SCI) + if (p->type =3D=3D PORT_SCI || p->type =3D=3D SCI_PORT_RSCI) swap(sci_port->irqs[SCIx_BRI_IRQ], sci_port->irqs[SCIx_TEI_IRQ]); =20 /* The SCI generates several interrupts. They can be muxed together or @@ -3122,6 +3145,9 @@ static int sci_init_single(struct platform_device *de= v, else sci_port->rx_trigger =3D 8; break; + case SCI_PORT_RSCI: + sci_port->rx_trigger =3D 15; + break; default: sci_port->rx_trigger =3D 1; break; @@ -3346,7 +3372,8 @@ static void sci_remove(struct platform_device *dev) =20 if (s->port.fifosize > 1) device_remove_file(&dev->dev, &dev_attr_rx_fifo_trigger); - if (type =3D=3D PORT_SCIFA || type =3D=3D PORT_SCIFB || type =3D=3D PORT_= HSCIF) + if (type =3D=3D PORT_SCIFA || type =3D=3D PORT_SCIFB || type =3D=3D PORT_= HSCIF || + type =3D=3D SCI_PORT_RSCI) device_remove_file(&dev->dev, &dev_attr_rx_fifo_timeout); } =20 @@ -3440,6 +3467,12 @@ static const struct of_device_id of_sci_match[] __ma= ybe_unused =3D { .compatible =3D "renesas,scif-r9a09g057", .data =3D &of_sci_scif_rzv2h, }, +#ifdef CONFIG_SERIAL_RSCI + { + .compatible =3D "renesas,r9a09g077-rsci", + .data =3D &of_sci_rsci_data, + }, +#endif /* CONFIG_SERIAL_RSCI */ /* Family-specific types */ { .compatible =3D "renesas,rcar-gen1-scif", @@ -3699,7 +3732,7 @@ static int sci_probe(struct platform_device *dev) return ret; } if (sp->type =3D=3D PORT_SCIFA || sp->type =3D=3D PORT_SCIFB || - sp->type =3D=3D PORT_HSCIF) { + sp->type =3D=3D PORT_HSCIF || sp->type =3D=3D SCI_PORT_RSCI) { ret =3D device_create_file(&dev->dev, &dev_attr_rx_fifo_timeout); if (ret) { if (sp->port.fifosize > 1) { --=20 2.49.0