Received: by 2002:a05:6a10:eb17:0:0:0:0 with SMTP id hx23csp446905pxb; Thu, 9 Sep 2021 04:48:25 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxmw/5FzGwDpYhCKwZfWRXijpfj3MYEVNri0mAVy7jctuXDXjwaYYMKmH8VaP9Lw370nIkw X-Received: by 2002:a17:906:7a07:: with SMTP id d7mr2889921ejo.257.1631188105515; Thu, 09 Sep 2021 04:48:25 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1631188105; cv=none; d=google.com; s=arc-20160816; b=hto05krrWoGlUTjdvGn37yB74u0NJOu4ztI028tno7qnSMBIfW64SZmjtcccKcieXE 4sm1mfB7GtNLFqKP90nUZtzCSX3qJQJt/oVhKg0gyRVf0MKH05WYjxyoyMv+BIEfflgY cmX1hOpvMdxejMUN3OnTiUAG1yxFNTn2EshN7S+m9IpkTBhd+ETv75/rnmT7XGFx8yew rGo4WKevMuuvRw2L7o0v6L83PJ9B+ICOE+482NblqsSXWpVZgyusI1Q6X916j/kVoLRi avtmChee7CT/0rcSyoTU9GtD8CpJ9Sz6v8srm74VO+DGkEF4uf5V3+CjIFpJo6jaQb3B 5KHA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=bCU4vgfxnbmnuIfhnxVBhBnAzRTOoNeJ87FfS267nQ8=; b=MrZlO2lcy001lB0wcH2KA8uv49kXbmGk7EZ7YwuQj+iyi9bDOb2xWk6ino16rbn6wA CGvpK/xVcA5aQm4i3vBsnMRadszPESew20qQWUYCPmly1mvNoBOErWW+zFugKn07yGMU 1SmOyjyozwxC6ZNulNABHTp9IvC401x3e1Bj3zTROd3OxtAhHHv6L9bTTrXm7PVTpikM eVneoIIzs9Lt8ShkInCFprILeAWIQXi8lwdkV+GClLVV9dsBpffpSstmtHiDzQECBSBQ GGcU0DUoKSMYeuk+2OEzJRhtIui+uZUlb1oCLk+goja4B0sl5JgcdfYePPWjEUzra2Zx mNKw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=sthtQsZT; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id js2si1812670ejc.576.2021.09.09.04.48.01; Thu, 09 Sep 2021 04:48:25 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=sthtQsZT; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238488AbhIILry (ORCPT + 99 others); Thu, 9 Sep 2021 07:47:54 -0400 Received: from mail.kernel.org ([198.145.29.99]:46894 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S237911AbhIILpQ (ORCPT ); Thu, 9 Sep 2021 07:45:16 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 56273611CC; Thu, 9 Sep 2021 11:42:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1631187765; bh=ffhCIhguU0o3/doEVsvxrucAN7Ddh5vxISHL95YaV5w=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=sthtQsZTkqkJ6DSGRVW7mlwbmLGK2eG578LiVuXOvUNyXQufKjxKzU+fUNjEOKwsP ZS0uksYcFJdPY1ck505nLqfcLXgBWyaUbVRI4Woo5hMaJPVs/pWQVOuh3Zwal8W7uM VqKhT6mwC/S2LxJ6NMuAI3V6jr+h6Rv+U5BCRFTn5+cliH3W1HbHgfk9YxPUR3d/QU a3crTHtf0ZqWtC0EPIBaD/hQT4Dg8Hz6/s9J/oU+83hEMiGopmR6o7jGcRvGqlQTZY GklyPjeR2HuVdyuimTIS4NaZ/KlAL2+Lnf91RqhGOfDvHmzq9DJH31gSTea1eWD1o5 Gd1nhcUDk+61Q== From: Sasha Levin To: linux-kernel@vger.kernel.org, stable@vger.kernel.org Cc: "Maciej W. Rozycki" , Greg Kroah-Hartman , Sasha Levin , linux-serial@vger.kernel.org Subject: [PATCH AUTOSEL 5.14 077/252] serial: 8250: Define RX trigger levels for OxSemi 950 devices Date: Thu, 9 Sep 2021 07:38:11 -0400 Message-Id: <20210909114106.141462-77-sashal@kernel.org> X-Mailer: git-send-email 2.30.2 In-Reply-To: <20210909114106.141462-1-sashal@kernel.org> References: <20210909114106.141462-1-sashal@kernel.org> MIME-Version: 1.0 X-stable: review X-Patchwork-Hint: Ignore Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: "Maciej W. Rozycki" [ Upstream commit d7aff291d069c4418285f3c8ee27b0ff67ce5998 ] Oxford Semiconductor 950 serial port devices have a 128-byte FIFO and in the enhanced (650) mode, which we select in `autoconfig_has_efr' with the ECB bit set in the EFR register, they support the receive interrupt trigger level selectable with FCR bits 7:6 from the set of 16, 32, 112, 120. This applies to the original OX16C950 discrete UART[1] as well as 950 cores embedded into more complex devices. For these devices we set the default to 112, which sets an excessively high level of 112 or 7/8 of the FIFO capacity, unlike with other port types where we choose at most 1/2 of their respective FIFO capacities. Additionally we don't make the trigger level configurable. Consequently frequent input overruns happen with high bit rates where hardware flow control cannot be used (e.g. terminal applications) even with otherwise highly-performant systems. Lower the default receive interrupt trigger level to 32 then, and make it configurable. Document the trigger levels along with other port types, including the set of 16, 32, 64, 112 for the transmit interrupt as well[2]. References: [1] "OX16C950 rev B High Performance UART with 128 byte FIFOs", Oxford Semiconductor, Inc., DS-0031, Sep 05, Table 10: "Receiver Trigger Levels", p. 22 [2] same, Table 9: "Transmit Interrupt Trigger Levels", p. 22 Signed-off-by: Maciej W. Rozycki Link: https://lore.kernel.org/r/alpine.DEB.2.21.2106260608480.37803@angie.orcam.me.uk Signed-off-by: Greg Kroah-Hartman Signed-off-by: Sasha Levin --- drivers/tty/serial/8250/8250_port.c | 3 ++- include/uapi/linux/serial_reg.h | 1 + 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/tty/serial/8250/8250_port.c b/drivers/tty/serial/8250/8250_port.c index 1da29a219842..66374704747e 100644 --- a/drivers/tty/serial/8250/8250_port.c +++ b/drivers/tty/serial/8250/8250_port.c @@ -122,7 +122,8 @@ static const struct serial8250_config uart_config[] = { .name = "16C950/954", .fifo_size = 128, .tx_loadsz = 128, - .fcr = UART_FCR_ENABLE_FIFO | UART_FCR_R_TRIG_10, + .fcr = UART_FCR_ENABLE_FIFO | UART_FCR_R_TRIG_01, + .rxtrig_bytes = {16, 32, 112, 120}, /* UART_CAP_EFR breaks billionon CF bluetooth card. */ .flags = UART_CAP_FIFO | UART_CAP_SLEEP, }, diff --git a/include/uapi/linux/serial_reg.h b/include/uapi/linux/serial_reg.h index be07b5470f4b..f51bc8f36813 100644 --- a/include/uapi/linux/serial_reg.h +++ b/include/uapi/linux/serial_reg.h @@ -62,6 +62,7 @@ * ST16C654: 8 16 56 60 8 16 32 56 PORT_16654 * TI16C750: 1 16 32 56 xx xx xx xx PORT_16750 * TI16C752: 8 16 56 60 8 16 32 56 + * OX16C950: 16 32 112 120 16 32 64 112 PORT_16C950 * Tegra: 1 4 8 14 16 8 4 1 PORT_TEGRA */ #define UART_FCR_R_TRIG_00 0x00 -- 2.30.2