From patchwork Sat Dec 4 19:57:52 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sam Protsenko X-Patchwork-Id: 12694780 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 08159C433EF for ; Sat, 4 Dec 2021 19:59:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-Id:Date:Subject:Cc :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=iSK0+eQGaaiHnYyb7SYtETWPFSaV+/S5rL8PjnaiGWg=; b=pWxUHA2+EL0PsR n5ia6l8sPnulDBQqz6WEUduChKRyvzkAkQeGQXSGXVVLkypH4Cp8WPTlvN5oSFzug+kGRSgZG7LuN 6EeHfJ/WWMJO9u1qD7WVL7aw9ZzXVVuH6/YFi9b6ysmOC4PDEl13GOIpq08VttlLJY9J+0vAhoM9l 62I9NSKecyksN0yynau0I8AiY4mBO8UeqH73hnjdVNGxo3umJnKFajs8aWE4ldMUA2ZIBCY+Qym7o qQqxWl2lkK6bAy73nDYd4nx4cWBv9Iu1EZ/+Z4zPvwuPddEXo/fjg95jVK0BdftsQbfZw1FJyXzXQ /uJEzfBScEtqs0TkiSkg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mtbAc-000hrg-4s; Sat, 04 Dec 2021 19:58:06 +0000 Received: from mail-wr1-x433.google.com ([2a00:1450:4864:20::433]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mtbAY-000hqT-Du for linux-arm-kernel@lists.infradead.org; Sat, 04 Dec 2021 19:58:03 +0000 Received: by mail-wr1-x433.google.com with SMTP id o13so13263271wrs.12 for ; Sat, 04 Dec 2021 11:58:01 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=5Tae7shI/erDBK4JbFqL57PR9dNyL7JXb+7DeAlnpbE=; b=HYbAMkZQ7OyhNh+oJ+o26au8HLeEYXjmkei/adv8/PCYcWI3IKBVgGPM3+yw6nJ9tq y88Kv2DVc2HfYedBiSmrcsS5KhwHV04zoFQpgf/zhB7jewv63yH4O7cXMViNRPyyU3HM +gWQNlDfAWwyBFvoPVsHKRhirK9leTz7pKic/sHhCuLVsIHkmraUl+6AEdCPvybPxylm 872i28hbddCHq/BTH+RahXxsA6k2muhrWaze1KjveMtx7M4TGLI2nSQvoj0qQVfQbw62 nb8l+UgNLjjUav+pqqccAGAlKAxeDmhDauEf4lE3zzMWe9kBnnsF4bwlxCUs58E6Ub/N 4a3Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=5Tae7shI/erDBK4JbFqL57PR9dNyL7JXb+7DeAlnpbE=; b=fFY8IVWxF2Qks1sHpzvh3AE+qLLhPzw0W+GOBxVnLl2YmXio/VDgCDJk5l0vXGdnPx hY/9HH9hWlILvwhV84nmNwqMMY1flWZhJHJ2krgAyI7aJ8Mh0IAPDbBypbH3gacLx8sm LibMwVGBIdp/hWWFYf+3GSqMHSojU7yfYPOJeAMx9+YG6KjZqt5PA92SoqwaStwXdEmt O/Dj1hmwfI7FzvgiYsHeUkQLVdap7YEdK1pmzqoxy3Dvg9djHMjNoJvzZB5lpJUG5S+3 E8Kpwb7uYK38yp/lzdgo2rht7TUgd7rEC3Idg65HIs4zFXlEHl+YJSJAjOgCsew4Nsxs Mvuw== X-Gm-Message-State: AOAM532OGqO1JOQrHtHta7rZPIX8mSJrpGatTjuOc0W08hbHb0bNL99q jVp8ZphPoAkaeHDWVbDiF8xz/Q== X-Google-Smtp-Source: ABdhPJxwjIqYxZ/dHI2yVxue6xSU9HkKtY/F4RcR68UF2cflsf4OPRwRK95eY5hT0ZZNFe3+ydy2ng== X-Received: by 2002:a5d:404d:: with SMTP id w13mr30057082wrp.293.1638647879454; Sat, 04 Dec 2021 11:57:59 -0800 (PST) Received: from localhost ([31.134.121.151]) by smtp.gmail.com with ESMTPSA id m14sm9185932wrp.28.2021.12.04.11.57.58 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 04 Dec 2021 11:57:58 -0800 (PST) From: Sam Protsenko To: Krzysztof Kozlowski , Rob Herring , Greg Kroah-Hartman Cc: Jaewon Kim , Chanho Park , David Virag , Youngmin Nam , devicetree@vger.kernel.org, linux-serial@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-samsung-soc@vger.kernel.org Subject: [PATCH v3 0/5] soc: samsung: Add USI driver Date: Sat, 4 Dec 2021 21:57:52 +0200 Message-Id: <20211204195757.8600-1-semen.protsenko@linaro.org> X-Mailer: git-send-email 2.30.2 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211204_115802_526103_AFE06A8D X-CRM114-Status: GOOD ( 19.00 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org USIv2 IP-core provides selectable serial protocol (UART, SPI or High-Speed I2C); only one can be chosen at a time. This series implements USIv2 driver, which allows one to select particular USI function in device tree, and also performs USI block initialization. With that driver implemented, it's not needed to do USI initialization in protocol drivers anymore, so that code is removed from the serial driver. Because USI driver is tristate (can be built as a module), serial driver was reworked so it's possible to use its console part as a module too. This way we can load serial driver module from user space and still have serial console functional. Design features: - "reg" property contains USI registers start address (0xc0 offset); it's used in the driver to access USI_CON and USI_OPTION registers. This way all USI initialization (reset, HWACG, etc) can be done in USIv2 driver separately, rather than duplicating that code over UART/SPI/I2C drivers - System Register (system controller node) and its SW_CONF register offset are provided in "samsung,sysreg" property; it's used to select USI function (protocol to be used) - USI function is specified in "samsung,mode" property; integer value is used to simplify parsing - there is "samsung,clkreq-on" bool property, which makes driver disable HWACG control (needed for UART to work properly) - PCLK and IPCLK clocks are both provided to USI node; apparently both need to be enabled to access USI registers - protocol nodes are embedded (as a child nodes) in USI node; it allows correct init order, and reflects HW properly - USI driver is a tristate: can be also useful from Android GKI requirements point of view - driver functions are implemented with further development in mind: - we might want to add some DebugFs interface later - some functions might need to be revealed to serial drivers with EXPORT_SYMBOL(), and provide somehow pointer to needed USI driver instance - another USI revisions could be added (like USIv1) Changes in v3: - Renamed compatible from samsung,exynos-usi-v2 to samsung,exynos850-usi - Used clk_bulk API instead of handling each clock separately - Spell check fixes and coding style fixes - Improved dt-bindings doc Changes in v2: - Renamed all 'usi_v2' wording to just 'usi' everywhere - Removed patches adding dependency on EXYNOS_USI for UART/I2C/SPI drivers - Added patch: "tty: serial: samsung: Fix console registration from module" - Combined dt-bindings doc and dt-bindings header patches - Reworked USI driver to be ready for USIv1 addition - Improved dt-bindings - Added USI_V2_NONE mode value Sam Protsenko (5): dt-bindings: soc: samsung: Add Exynos USI bindings soc: samsung: Add USI driver tty: serial: samsung: Remove USI initialization tty: serial: samsung: Enable console as module tty: serial: samsung: Fix console registration from module .../bindings/soc/samsung/exynos-usi.yaml | 159 ++++++++++ drivers/soc/samsung/Kconfig | 14 + drivers/soc/samsung/Makefile | 2 + drivers/soc/samsung/exynos-usi.c | 285 ++++++++++++++++++ drivers/tty/serial/Kconfig | 2 +- drivers/tty/serial/samsung_tty.c | 78 ++--- include/dt-bindings/soc/samsung,exynos-usi.h | 17 ++ include/linux/serial_s3c.h | 9 - 8 files changed, 518 insertions(+), 48 deletions(-) create mode 100644 Documentation/devicetree/bindings/soc/samsung/exynos-usi.yaml create mode 100644 drivers/soc/samsung/exynos-usi.c create mode 100644 include/dt-bindings/soc/samsung,exynos-usi.h Tested-by: Chanho Park with below patch.