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=-17.2 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,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 4C719C433F5 for ; Tue, 21 Sep 2021 07:36:43 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 3195F60F6B for ; Tue, 21 Sep 2021 07:36:43 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230283AbhIUHiJ (ORCPT ); Tue, 21 Sep 2021 03:38:09 -0400 Received: from smtp-relay-internal-0.canonical.com ([185.125.188.122]:37542 "EHLO smtp-relay-internal-0.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230168AbhIUHiD (ORCPT ); Tue, 21 Sep 2021 03:38:03 -0400 Received: from mail-wr1-f72.google.com (mail-wr1-f72.google.com [209.85.221.72]) (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-internal-0.canonical.com (Postfix) with ESMTPS id 8C36E40262 for ; Tue, 21 Sep 2021 07:36:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=canonical.com; s=20210705; t=1632209794; bh=5xtuLVRlXiTMHH96D8zimgkDHcNXy9X9ihNLTsNCcqE=; h=Subject:To:Cc:References:From:Message-ID:Date:MIME-Version: In-Reply-To:Content-Type; b=KYeovEO7J1SuJYe3ER8quU3K/9bGCFFvYmykARPDPS6wAUODPE/7zPWekYhOdB6sP yWHRVHNsJp4ztdxDOSh2db+egrZn83kHDZ1nqgUcJql6y+sIIoackAJ6IUDY4M6H4N 4yiisoed0JBCFjV/OifNryVt6DqzQ11EnDxoSjNpU5ihZsOJ0S1zDlp2bpldIYguHn nBbBMNU3Igu6Vr40Y5HVkKzZE/GELGqi7mfi7R+N30+XJy+HVNRhMnHbkdLu1uATRQ 1HPaUwT+smKnyeHFwpyKOK8IacFUk0RenPqgUPTlExRD/JaSz8kXiB1NTFSe6I3FaR KVoWmhmBg++cw== Received: by mail-wr1-f72.google.com with SMTP id e1-20020adfa741000000b0015e424fdd01so7614404wrd.11 for ; Tue, 21 Sep 2021 00:36:34 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; 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=5xtuLVRlXiTMHH96D8zimgkDHcNXy9X9ihNLTsNCcqE=; b=fU/tUXL+qRMrfpjaquEQ5o0u1iTgHmrTuzUNfBfTRRD2tEuKklhvVsgWUPnHOt40/1 yU1pLh5NLybAA/Ar0f2xNjtBmJ2Z/S8LM+ikyXAMKKtwUft2ffkThDXbZ2fcPpwLhvkZ eeHWE/2i0itoaV+2loy/DEO3b7qDqA37mTVNhwQJB0qlPyn8KDxqM1IdTEwmWZyqsl4I niBA9BjfibKAGawkjVofcbLPzCUu7lKu0jd9LyL2cp9dwhzLmOZ/VMm+2qJzgP9YK62s r9hcYNNUdQ5AkvCLe+yBnnUBej4TLBe+wqhV4cB8IWAx0ZB3hBL4t4+lkHWuWHYn3vgQ x9jw== X-Gm-Message-State: AOAM531j+nfT/l5ng/TRmAUHoCvJhU91oV6wYOOCtrFUmssBJXyTQb4n 4lGnBVi9Kn7e30aFGytt3qz95dPFG8FYfo+cU7wVSQ1jBiuQqm6D9iB18RtYV+4VyGP8O4SET0s cPrAN3eQiZhf5jfRYAFr8K3Cmp80Pl/SWZLBFG0p8og== X-Received: by 2002:a5d:598f:: with SMTP id n15mr30497162wri.74.1632209794145; Tue, 21 Sep 2021 00:36:34 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzLKQWe41cx6PvZ19KxjufcOkHR0IX8PQ9IjcFMsfMXm6dylIN5TbRIN773VgRDP6J6eJhhWw== X-Received: by 2002:a5d:598f:: with SMTP id n15mr30497148wri.74.1632209793950; Tue, 21 Sep 2021 00:36:33 -0700 (PDT) Received: from [192.168.0.134] (lk.84.20.244.219.dc.cable.static.lj-kabel.net. [84.20.244.219]) by smtp.gmail.com with ESMTPSA id i203sm1353028wma.7.2021.09.21.00.36.32 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 21 Sep 2021 00:36:33 -0700 (PDT) Subject: Re: [PATCH v1 4/4] rtc: change HAVE_S3C_RTC default config logic To: Will McVicker , Catalin Marinas , Will Deacon , Alessandro Zummo , Alexandre Belloni Cc: Lee Jones , kernel-team@android.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-rtc@vger.kernel.org References: <20210920190350.3860821-1-willmcvicker@google.com> <20210920190350.3860821-5-willmcvicker@google.com> From: Krzysztof Kozlowski Message-ID: <1f4efb90-4c6f-0972-de67-eae8e0246f5b@canonical.com> Date: Tue, 21 Sep 2021 09:36:32 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: <20210920190350.3860821-5-willmcvicker@google.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 20/09/2021 21:03, Will McVicker wrote: > Switches the default config logic of HAVE_S3C_RTC to use "default y if > (ARCH_EXYNOS && RTC_CLASS)" versus having ARCH_EXYNOS directly select > it. This provides vendors flexibility to disable the config if desired > or modularize it in the presence of a generic kernel. > > Verified this change doesn't impact the .config. > > Signed-off-by: Will McVicker > --- > arch/arm64/Kconfig.platforms | 1 - > drivers/rtc/Kconfig | 1 + > 2 files changed, 1 insertion(+), 1 deletion(-) > It took me some effort to find this patch because you did not send it to neither me, nor Samsung SoC list. Since you touch arm64 code, this should go either via SoC tree or with SoC ack, therefore you should simply use get_maintainers.pl on entire patchset, not on some pieces. > diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms > index a884e5da8b0f..f9f829aab511 100644 > --- a/arch/arm64/Kconfig.platforms > +++ b/arch/arm64/Kconfig.platforms > @@ -91,7 +91,6 @@ config ARCH_BRCMSTB > > config ARCH_EXYNOS > bool "ARMv8 based Samsung Exynos SoC family" > - select HAVE_S3C_RTC if RTC_CLASS Just remove HAVE_S3C_RTC entirely like in commit: 7dd3cae90d856e97e93bc1c32904e5aed7210f7b > select PINCTRL > select PM_GENERIC_DOMAINS if PM > help > diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig > index e1bc5214494e..40afdb37d2a5 100644 > --- a/drivers/rtc/Kconfig > +++ b/drivers/rtc/Kconfig > @@ -1406,6 +1406,7 @@ config RTC_DRV_OMAP > > config HAVE_S3C_RTC > bool > + default y if (ARCH_EXYNOS && RTC_CLASS) > help > This will include RTC support for Samsung SoCs. If > you want to include RTC support for any machine, kindly > Best regards, Krzysztof 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=-16.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=ham 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 4CD55C433F5 for ; Tue, 21 Sep 2021 07:38:35 +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 1846160F6B for ; Tue, 21 Sep 2021 07:38:35 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 1846160F6B 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=K+pYrCmYHnJhWA1L1Fg0qgzeJdS+J0uaumxWv1PhnjQ=; b=qsPtRAgRPECZIToZgZN91OHZZO nHDCf/gYE8Q0PYI3z2EhkFFV++E7wRw2+kgKY0Cpdizbf6XR6/rzIqJ6IAlp6ZCd7V4Bv89yQKuep jk6ouAWxQBaQDioCYtso2IvcckU8hafeI5dDset3+u6NK8sKOwg9Ue6cAdZJnzCEZ/R/9fnj9wINi Co0XyQYa7akkbuVNpQzOpQT3Gu1ORvEGBluYGaLGG3m/1trI7PW7I90BItu54PUGpvXQz6vJ/iCEY 6sh2ovVGtJ1l2ya6eULU4tF6neqNr+NOK7b9gBjZcsrhx/59Gj/nvQN2TG01OLEaaZUARxMYsl2FY SXp5znOg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mSaKW-003qJO-4S; Tue, 21 Sep 2021 07:36:40 +0000 Received: from smtp-relay-internal-0.canonical.com ([185.125.188.122]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mSaKR-003qJ0-Uz for linux-arm-kernel@lists.infradead.org; Tue, 21 Sep 2021 07:36:37 +0000 Received: from mail-wr1-f69.google.com (mail-wr1-f69.google.com [209.85.221.69]) (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-internal-0.canonical.com (Postfix) with ESMTPS id 852CE40197 for ; Tue, 21 Sep 2021 07:36:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=canonical.com; s=20210705; t=1632209794; bh=5xtuLVRlXiTMHH96D8zimgkDHcNXy9X9ihNLTsNCcqE=; h=Subject:To:Cc:References:From:Message-ID:Date:MIME-Version: In-Reply-To:Content-Type; b=KYeovEO7J1SuJYe3ER8quU3K/9bGCFFvYmykARPDPS6wAUODPE/7zPWekYhOdB6sP yWHRVHNsJp4ztdxDOSh2db+egrZn83kHDZ1nqgUcJql6y+sIIoackAJ6IUDY4M6H4N 4yiisoed0JBCFjV/OifNryVt6DqzQ11EnDxoSjNpU5ihZsOJ0S1zDlp2bpldIYguHn nBbBMNU3Igu6Vr40Y5HVkKzZE/GELGqi7mfi7R+N30+XJy+HVNRhMnHbkdLu1uATRQ 1HPaUwT+smKnyeHFwpyKOK8IacFUk0RenPqgUPTlExRD/JaSz8kXiB1NTFSe6I3FaR KVoWmhmBg++cw== Received: by mail-wr1-f69.google.com with SMTP id s13-20020a5d69cd000000b00159d49442cbso8065118wrw.13 for ; Tue, 21 Sep 2021 00:36:34 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; 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=5xtuLVRlXiTMHH96D8zimgkDHcNXy9X9ihNLTsNCcqE=; b=LEhlleJAlJuuPGc1gMSnrDat/t6oj7rQetCNfJOJjsGoNAK8d9LPNw/O3krqDtNwQM kH0J/U6fP6j7WbXdOreL5bjQXN56shZ7QOU0tQGEA0WN7pYdtPgro2syxH2YZUrHMai/ rR2SoiekLKqczi/fHdVY6FOHQQgrLhvjWLc6kRgH+3DUY2T0AB+J1W+Y2Ufr6KKwEiT9 bMVcKCTocOpNdZdmcdk8xARDDrYXgcmsj15Wdnq933swsRULfXM1X1XTH4hzh3L+9Mh8 O4Klv7YlTUCWwO8YSTZpP9G6jc/Muqquq5prvrI3lUc7U1mVRttX2+v8ppX/3YPjGn0r vYQg== X-Gm-Message-State: AOAM533m/o66F9DPw9aQyvo+Teh4Si6Wg8Myaq56T94d553DLHSNClex DXyC7SGkkXzh+2SRPgp63ooacDwBQqzAa9QrqzzsA6f7kBBWjGbBu+X8VBL0YUkGyWR8VhErJmw RwJ4TkPJnGfL9Hi2fXnUHZvNcQM6LEjmWNPN4W+70Mrrv9Un9k8OD X-Received: by 2002:a5d:598f:: with SMTP id n15mr30497165wri.74.1632209794145; Tue, 21 Sep 2021 00:36:34 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzLKQWe41cx6PvZ19KxjufcOkHR0IX8PQ9IjcFMsfMXm6dylIN5TbRIN773VgRDP6J6eJhhWw== X-Received: by 2002:a5d:598f:: with SMTP id n15mr30497148wri.74.1632209793950; Tue, 21 Sep 2021 00:36:33 -0700 (PDT) Received: from [192.168.0.134] (lk.84.20.244.219.dc.cable.static.lj-kabel.net. [84.20.244.219]) by smtp.gmail.com with ESMTPSA id i203sm1353028wma.7.2021.09.21.00.36.32 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 21 Sep 2021 00:36:33 -0700 (PDT) Subject: Re: [PATCH v1 4/4] rtc: change HAVE_S3C_RTC default config logic To: Will McVicker , Catalin Marinas , Will Deacon , Alessandro Zummo , Alexandre Belloni Cc: Lee Jones , kernel-team@android.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-rtc@vger.kernel.org References: <20210920190350.3860821-1-willmcvicker@google.com> <20210920190350.3860821-5-willmcvicker@google.com> From: Krzysztof Kozlowski Message-ID: <1f4efb90-4c6f-0972-de67-eae8e0246f5b@canonical.com> Date: Tue, 21 Sep 2021 09:36:32 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: <20210920190350.3860821-5-willmcvicker@google.com> Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210921_003636_260182_D75A54F2 X-CRM114-Status: GOOD ( 25.84 ) 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 20/09/2021 21:03, Will McVicker wrote: > Switches the default config logic of HAVE_S3C_RTC to use "default y if > (ARCH_EXYNOS && RTC_CLASS)" versus having ARCH_EXYNOS directly select > it. This provides vendors flexibility to disable the config if desired > or modularize it in the presence of a generic kernel. > > Verified this change doesn't impact the .config. > > Signed-off-by: Will McVicker > --- > arch/arm64/Kconfig.platforms | 1 - > drivers/rtc/Kconfig | 1 + > 2 files changed, 1 insertion(+), 1 deletion(-) > It took me some effort to find this patch because you did not send it to neither me, nor Samsung SoC list. Since you touch arm64 code, this should go either via SoC tree or with SoC ack, therefore you should simply use get_maintainers.pl on entire patchset, not on some pieces. > diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms > index a884e5da8b0f..f9f829aab511 100644 > --- a/arch/arm64/Kconfig.platforms > +++ b/arch/arm64/Kconfig.platforms > @@ -91,7 +91,6 @@ config ARCH_BRCMSTB > > config ARCH_EXYNOS > bool "ARMv8 based Samsung Exynos SoC family" > - select HAVE_S3C_RTC if RTC_CLASS Just remove HAVE_S3C_RTC entirely like in commit: 7dd3cae90d856e97e93bc1c32904e5aed7210f7b > select PINCTRL > select PM_GENERIC_DOMAINS if PM > help > diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig > index e1bc5214494e..40afdb37d2a5 100644 > --- a/drivers/rtc/Kconfig > +++ b/drivers/rtc/Kconfig > @@ -1406,6 +1406,7 @@ config RTC_DRV_OMAP > > config HAVE_S3C_RTC > bool > + default y if (ARCH_EXYNOS && RTC_CLASS) > help > This will include RTC support for Samsung SoCs. If > you want to include RTC support for any machine, kindly > Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel