From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-19.7 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 12677C4338F for ; Mon, 16 Aug 2021 09:48:17 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id D0D0061B73 for ; Mon, 16 Aug 2021 09:48:15 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org D0D0061B73 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=canonical.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org 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:In-Reply-To:MIME-Version:Date: Message-ID:From:References:Cc:To:Subject:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Owner; bh=9aQ1yXIKviBXxkUowOSA0UPOn8HzcFsbS+HQo8UMQJY=; b=RIjkSrnTB2HTSgv92zinorwhOu u8CKLgZnnCBp9NjBIm6iZU4aOMf6neVo9JUTxgyAvX1tE3TbiriYxKVW5ssr9WQc/S2oXdZcXjHCl LMu8mUMUedaX3yAEvthLbzonD3csJKZyBUsHrCP5s7ymlM/gqCJ5XZbUUf6uUBtlW/naDcszaJQvi +ZAw8GEWHN1sZ33DZVIVj9A5LsDygoLxwwNcBF7JDGAuM/bOjbLQCR6Lco1+mOdmbufuVR8VhBu0d DnDg/VFvhafWyBZfuW3qxwTfzU+3o5B4Mcrl42G9LodyUbSpYlWlkm1vKwsYWllsVOd8wFYjEJ7WP vT4l/tiw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mFZC1-00Gcvn-8P; Mon, 16 Aug 2021 09:46:05 +0000 Received: from smtp-relay-canonical-0.canonical.com ([185.125.188.120]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mFZBi-00Gctl-IE for linux-arm-kernel@lists.infradead.org; Mon, 16 Aug 2021 09:45:50 +0000 Received: from mail-lj1-f199.google.com (mail-lj1-f199.google.com [209.85.208.199]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by smtp-relay-canonical-0.canonical.com (Postfix) with ESMTPS id CE31540C9D for ; Mon, 16 Aug 2021 09:45:37 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=canonical.com; s=20210705; t=1629107137; bh=0YPeosyPSKYRu+3IMfidh4Xy4sl5Ld4HC2vVfEQcmh8=; h=Subject:To:Cc:References:From:Message-ID:Date:MIME-Version: In-Reply-To:Content-Type; b=YV8aFj31OZFWGEcBknUAR9e6RIgx5VsBbkk4Zzo0VJyvciLVNR78qk4tU1FZ8Q/3v s8kG0EVfd015mVGgBk2OiyXqAOfAIcTfgN1KQIyilhw/HW7kPEjNtV31Yt5CJOATKK 02M+4vajd7s+EhbyoueI1As3WATHoY7fdMwo9+0ULNL5mFztunviFTusGauFjMSXlk dgGWECxQXvhiYas/xDmu/Fc/i7FNkUR5lErbdMeki0fD5tbv+BiWCjf4mkZsgrdmtr iH1Xrh/T6k+wURTYunvWk/tczHhCkucI2OYLeexiZr20S4wuebtI6lux0zIFfV09nW evvPwVsh8+YsA== Received: by mail-lj1-f199.google.com with SMTP id l12-20020a2ea80c0000b02900f5b2b52da7so5764869ljq.8 for ; Mon, 16 Aug 2021 02:45:37 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=0YPeosyPSKYRu+3IMfidh4Xy4sl5Ld4HC2vVfEQcmh8=; b=NXUIc6MWOU8pXCmdUkpoNMUTRvDsOZL6MqhfWiIaSidlEnhO/F9fTRB8rk0DNKqXTn fKImojKwu+nqZs9jCx1pHCGBF76mQZq7WnM001ArDlVBtreCyUwm26kXX0ECqyZ+kcSB ZSZEqYV1wwmZabx2KsBGIahxcdlO1uU5I2lYkv04gyU0Og1eP9L/xS06r2pZQJd9W68I kfxBfflWf44kV6ATcRCEDCNAihgFL2gFXGgblMkrQsB0jY9EiffFRfGWcLHjmIX7y6EG R8rED7ygkWiFe/xo3sgMsl7pY1ojHc9iC32sKB/2h7L7PixXzkOhupziX5FW3MR6hVAa iCLw== X-Gm-Message-State: AOAM532fdPz3Hz1GYbMgGqyiZlPs/qCxzGJtGDBPuTPNSSvJ4pRcjHZa 6RnkFQ8Y3Q0z/EZGtpWUoRbYur0B+eIPs+jMyLBmRwH0Y+4Pb5zqQU3yH93azj+obwQ+Tg5mRqF 6TkaDjlWJ70kLO87qxkzRGZszryFYpm6SlY8c5mK3Hl3oIklUSH/b X-Received: by 2002:a17:906:b195:: with SMTP id w21mr3439815ejy.12.1629107126855; Mon, 16 Aug 2021 02:45:26 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwuvp8D3ZgBwPmEOii5wrRifN9jfWsAO8nKcRBli/sHMv7eXnxEkHJaNyywLt+pQvHAdcMCQg== X-Received: by 2002:a17:906:b195:: with SMTP id w21mr3439794ejy.12.1629107126702; Mon, 16 Aug 2021 02:45:26 -0700 (PDT) Received: from [192.168.8.102] ([86.32.42.198]) by smtp.gmail.com with ESMTPSA id wc16sm3479626ejb.15.2021.08.16.02.45.24 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 16 Aug 2021 02:45:26 -0700 (PDT) Subject: Re: [PATCH v3 7/7] arm64: dts: exynos: Add Exynos850 SoC support To: Sam Protsenko Cc: Sylwester Nawrocki , =?UTF-8?Q?Pawe=c5=82_Chmiel?= , Chanwoo Choi , Linus Walleij , Tomasz Figa , Marc Zyngier , Rob Herring , Stephen Boyd , Michael Turquette , Jiri Slaby , Greg Kroah-Hartman , Charles Keepax , Ryu Euiyoul , Tom Gall , Sumit Semwal , John Stultz , Amit Pundir , devicetree , linux-arm Mailing List , "open list:GPIO SUBSYSTEM" , Linux Kernel Mailing List , Linux Samsung SOC , "open list:SERIAL DRIVERS" References: <20210811114827.27322-1-semen.protsenko@linaro.org> <20210811114827.27322-8-semen.protsenko@linaro.org> From: Krzysztof Kozlowski Message-ID: <40dd9a0a-eccc-1b0c-70ec-06edc8b91177@canonical.com> Date: Mon, 16 Aug 2021 11:45:24 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0 MIME-Version: 1.0 In-Reply-To: Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210816_024546_941960_3167E4AB X-CRM114-Status: GOOD ( 23.11 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 12/08/2021 19:42, Sam Protsenko wrote: > On Thu, 12 Aug 2021 at 11:17, Krzysztof Kozlowski > wrote: >> >> On 11/08/2021 13:48, Sam Protsenko wrote: >>> Samsung Exynos850 is ARMv8-based mobile-oriented SoC. >>> >>> This patch adds minimal SoC support by including next Device Tree nodes: >>> >>> 1. Octa cores (Cortex-A55), supporting PSCI v1.0 >>> 2. ARM architecture timer (armv8-timer) >>> 3. Interrupt controller (GIC-400) >>> 4. Pinctrl nodes for GPIO >>> 5. Serial node >>> >>> Signed-off-by: Sam Protsenko >>> --- >>> Changes in v3: >>> - Used generic fixed clock for serial >>> >>> Changes in v2: >>> * Commit message: >>> - Documented added dts features instead of CPU features >>> >>> * exynos850-usi.dtsi: >>> - Removed, moved everything to exynos850.dtsi >>> >>> * exynos850.dtsi: >>> - Root node: >>> - Added comment about engineering name (Exynos3830) >>> - Renamed pinctrl nodes, adding domain names >>> - Used hard coded IRQ numbers instead of named constants everywhere >>> - Added soc node, moved next nodes there: gic, clock, pinctrls and >>> serial >>> - Used address-cells=1 for soc node and removed unneeded 0x0 from >>> reg properties >>> - Moved exynos850-pinctrl.dtsi include line to the end of >>> exynos850.dtsi >>> - Coding style fixes >>> - cpus: >>> - Used address-cells=1 for cpus node >>> - Renamed cpu@0001 to cpu@1, and so on >>> - Left only "arm,cortex-a55" for cpus compatible >>> - Renamed reg = <0x0001> to <0x1> for cpus >>> - armv8 timer: >>> - Add comment about missing HV timer IRQ to armv8 timer node >>> - Removed not existing properties from armv8 timer node >>> - Fixed cpu number in CPU_MASK() >>> - Removed obsolete clock-frequency property >>> - GIC: >>> - Fixed GIC type to be GIC-400 >>> - Fixed size of GIC's 2nd region to be 0x2000 >>> - serial node: >>> - Hard coded clock number for serial_0 for now; will replace with >>> named const once proper clock driver is implemented >>> - Removed gate_uart_clk0 clock from serial_0, as that clock is not >>> supported in serial driver anyway (yet) >>> - clock node: >>> - Fixed clock controller node name (@0x12.. -> @12..) >>> >>> * exynos850-pinctrl.dtsi: >>> - Referenced pinctrl nodes instead of defining those again in root node >>> - Fixed interrupt-cells (3 -> 2) >>> - Fixed USI related comments for pin config nodes >>> - Removed decon_f_te_* and fm_lna_en nodes (won't be used) >>> - Reordered pin config nodes by pin numbers >>> - Improved all comments >>> - Used existing named constants for pin-function and pin-pud >>> - Fixed node names (used hyphens instead of underscore) >>> - Fixed warnings found in W=1 build >>> >>> .../boot/dts/exynos/exynos850-pinctrl.dtsi | 748 ++++++++++++++++++ >>> arch/arm64/boot/dts/exynos/exynos850.dtsi | 261 ++++++ >>> 2 files changed, 1009 insertions(+) >>> create mode 100644 arch/arm64/boot/dts/exynos/exynos850-pinctrl.dtsi >>> create mode 100644 arch/arm64/boot/dts/exynos/exynos850.dtsi >>> >>> diff --git a/arch/arm64/boot/dts/exynos/exynos850-pinctrl.dtsi b/arch/arm64/boot/dts/exynos/exynos850-pinctrl.dtsi >>> new file mode 100644 >>> index 000000000000..ba5d5f33e2f6 >>> --- /dev/null >>> +++ b/arch/arm64/boot/dts/exynos/exynos850-pinctrl.dtsi >>> @@ -0,0 +1,748 @@ >>> +// SPDX-License-Identifier: GPL-2.0 >>> +/* >>> + * Samsung's Exynos850 SoC pin-mux and pin-config device tree source >>> + * >>> + * Copyright (C) 2017 Samsung Electronics Co., Ltd. >>> + * Copyright (C) 2021 Linaro Ltd. >>> + * >>> + * Samsung's Exynos850 SoC pin-mux and pin-config options are listed as device >>> + * tree nodes in this file. >>> + */ >>> + >>> +#include >>> +#include >>> + >>> +&pinctrl_alive { >>> + gpa0: gpa0 { >>> + gpio-controller; >>> + #gpio-cells = <2>; >>> + >>> + interrupt-controller; >>> + #interrupt-cells = <2>; >>> + interrupt-parent = <&gic>; >>> + interrupts = , >>> + , >>> + , >>> + , >>> + , >>> + , >>> + , >>> + ; >>> + }; >>> + >>> + gpa1: gpa1 { >>> + gpio-controller; >>> + #gpio-cells = <2>; >>> + >>> + interrupt-controller; >>> + #interrupt-cells = <2>; >>> + interrupt-parent = <&gic>; >>> + interrupts = , >>> + , >>> + , >>> + , >>> + , >>> + , >>> + , >>> + ; >>> + }; >>> + >>> + gpa2: gpa2 { >>> + gpio-controller; >>> + #gpio-cells = <2>; >>> + >>> + interrupt-controller; >>> + #interrupt-cells = <2>; >>> + interrupt-parent = <&gic>; >>> + interrupts = , >>> + , >>> + , >>> + , >>> + , >>> + , >>> + , >>> + ; >>> + }; >>> + >>> + gpa3: gpa3 { >>> + gpio-controller; >>> + #gpio-cells = <2>; >>> + >>> + interrupt-controller; >>> + #interrupt-cells = <2>; >>> + interrupt-parent = <&gic>; >>> + interrupts = , >>> + , >>> + , >>> + , >>> + , >>> + , >>> + , >>> + ; >>> + }; >>> + >>> + gpa4: gpa4 { >>> + gpio-controller; >>> + #gpio-cells = <2>; >>> + >>> + interrupt-controller; >>> + #interrupt-cells = <2>; >>> + interrupt-parent = <&gic>; >>> + interrupts = , >>> + , >>> + , >>> + ; >>> + }; >>> + >>> + gpq0: gpq0 { >>> + gpio-controller; >>> + #gpio-cells = <2>; >>> + >>> + interrupt-controller; >>> + #interrupt-cells = <2>; >>> + }; >>> + >>> + /* I2C5 (also called CAM_PMIC_I2C in TRM) */ >>> + i2c5_bus: i2c5-bus { >>> + samsung,pins = "gpa3-5", "gpa3-6"; >>> + samsung,pin-function = ; >>> + samsung,pin-pud = ; >>> + samsung,pin-drv = <0>; >>> + }; >>> + >>> + /* I2C6 (also called MOTOR_I2C in TRM) */ >>> + i2c6_bus: i2c6-bus { >>> + samsung,pins = "gpa3-7", "gpa4-0"; >>> + samsung,pin-function = ; >>> + samsung,pin-pud = ; >>> + samsung,pin-drv = <0>; >>> + }; >>> + >>> + /* USI: UART */ >>> + uart0_bus: uart0-bus { >>> + samsung,pins = "gpq0-0", "gpq0-1"; >>> + samsung,pin-function = ; >>> + samsung,pin-pud = ; >>> + }; >>> +}; >>> + >>> +&pinctrl_cmgp { >>> + gpm0: gpm0 { >>> + gpio-controller; >>> + #gpio-cells = <2>; >>> + >>> + interrupt-controller; >>> + #interrupt-cells = <2>; >>> + interrupt-parent = <&gic>; >>> + interrupts = ; >>> + }; >>> + >>> + gpm1: gpm1 { >>> + gpio-controller; >>> + #gpio-cells = <2>; >>> + >>> + interrupt-controller; >>> + #interrupt-cells = <2>; >>> + interrupt-parent = <&gic>; >>> + interrupts = ; >>> + }; >>> + >>> + gpm2: gpm2 { >>> + gpio-controller; >>> + #gpio-cells = <2>; >>> + >>> + interrupt-controller; >>> + #interrupt-cells = <2>; >>> + interrupt-parent = <&gic>; >>> + interrupts = ; >>> + }; >>> + >>> + gpm3: gpm3 { >>> + gpio-controller; >>> + #gpio-cells = <2>; >>> + >>> + interrupt-controller; >>> + #interrupt-cells = <2>; >>> + interrupt-parent = <&gic>; >>> + interrupts = ; >>> + }; >>> + >>> + gpm4: gpm4 { >>> + gpio-controller; >>> + #gpio-cells = <2>; >>> + >>> + interrupt-controller; >>> + #interrupt-cells = <2>; >>> + interrupt-parent = <&gic>; >>> + interrupts = ; >>> + }; >>> + >>> + gpm5: gpm5 { >>> + gpio-controller; >>> + #gpio-cells = <2>; >>> + >>> + interrupt-controller; >>> + #interrupt-cells = <2>; >>> + interrupt-parent = <&gic>; >>> + interrupts = ; >>> + }; >>> + >>> + /* USI_CMGP0: HSI2C function */ >>> + hsi2c3_bus: hsi2c3-bus { >>> + samsung,pins = "gpm0-0", "gpm1-0"; >>> + samsung,pin-function = ; >>> + samsung,pin-pud = ; >>> + samsung,pin-drv = <0>; >> >> There are also macros for DRV. >> > > Unfortunately, existing DRV macros won't work for Exynos850. DRV > constants have different meaning for different GPIO domains in > Exynos850, so I thought introducing several groups of DRV constants > might be confusing. But please let me know if you still want me do > that. > Oh, damn, raw values are ok then. Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel