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=-7.2 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=no 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 55068C433EF for ; Tue, 21 Sep 2021 07:20:08 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 2855D610CA for ; Tue, 21 Sep 2021 07:20:08 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230130AbhIUHVf (ORCPT ); Tue, 21 Sep 2021 03:21:35 -0400 Received: from smtp-relay-internal-0.canonical.com ([185.125.188.122]:36752 "EHLO smtp-relay-internal-0.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230071AbhIUHVc (ORCPT ); Tue, 21 Sep 2021 03:21:32 -0400 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 A1A1140262 for ; Tue, 21 Sep 2021 07:20:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=canonical.com; s=20210705; t=1632208802; bh=AEgBA3R77w9KryrrVCKS+IcMBHSWWQYQcCPJfI9REUY=; h=Subject:To:Cc:References:From:Message-ID:Date:MIME-Version: In-Reply-To:Content-Type; b=q5I2OKK/BHIgeadXIt4B504ihWDnqoms3PgcGZQGSft7i44IlbxsDlXvkhVjJFIW3 mzYuofJjMvriDGreAPnbacsXV4600h+II6SoC0/cUriDo8/BwzS1MsBO4ZYx1WoBRY Hhiz/tJZjvxtjhGvANGJlRhJ/ZPg4lKfqDYZ6sYHyc267AUtnWybwqolIxF49/bRVf lYGazccFPhBDNT5/WgSwoDgG70BJdUuJOV58O8soOlGA4x2ifxfz2Vbb4dVsBor53I PeZHGG2I7TKlyOocxsFvmTY7QOZVuO5RA0wJ7vMhnXmk2OGAhpLM5q87yNT9wrvMBU JYcuEZT4dxsQQ== Received: by mail-wr1-f69.google.com with SMTP id c15-20020a5d4ccf000000b0015dff622f39so8027736wrt.21 for ; Tue, 21 Sep 2021 00:20:02 -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=AEgBA3R77w9KryrrVCKS+IcMBHSWWQYQcCPJfI9REUY=; b=06FTt+tXbv20igJ9gMsZPsupPYNQ2n+Th1bEYV2w1rOB2SEpd8LmJScQh2V+8Y689z 3uQAOJpG+IlkPaAmsATfNb/sYgqHCcit7gRJnyadSoqGt2402ucAgluRcTdXAyqc5bzT kl9NMMp47Y6V9Mb/ybuSQoDX1KH1eMcqpZkr2OwdHfRAfkDy5p/hWEKPOTznLkAnrXRC H/kzmOhTcPS80Wu8iMYbuueDkATN9SOCe3bm7ApAoGuAUg/Ui/8PfXkhKQ8c8WWQzbnE YClZtmcATBwwN4ju685BJvig0DDjYVyJ5Xt+mw5+WzAGQ5m1DGG6+MtBwJ50RwZArrCg kUXA== X-Gm-Message-State: AOAM533bQ/kCxZFLYeSPTkg3Bf94H5F86pi7tnacnQfRbH85XicJ/2lN uoG+eL/L638xHSX+yfQSg2BIk5uLbbniduCA1K84UwglPZyaxomFFozCWiHuXmyReh2/+otTeOa Gcz4DAKs9ptUVOe8tomF7GLeAzcM7TK6SB6FTwzdW5g== X-Received: by 2002:a7b:c750:: with SMTP id w16mr2977700wmk.128.1632208802084; Tue, 21 Sep 2021 00:20:02 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzO5jz1FEI8gyX7GyxtVi5eDjZZJeMMrwVlptuHEN5O7i5u5mvto6Hcns9xoz5xze1ScI9CPA== X-Received: by 2002:a7b:c750:: with SMTP id w16mr2977671wmk.128.1632208801861; Tue, 21 Sep 2021 00:20:01 -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 u5sm3543492wrg.57.2021.09.21.00.20.00 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 21 Sep 2021 00:20:01 -0700 (PDT) Subject: Re: [PATCH v1 0/4] arm64: Kconfig: Update ARCH_EXYNOS select configs To: Will McVicker , Catalin Marinas , Will Deacon , Sylwester Nawrocki , Tomasz Figa , Chanwoo Choi , Michael Turquette , Stephen Boyd , Linus Walleij , Alessandro Zummo , Alexandre Belloni Cc: Lee Jones , kernel-team@android.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-samsung-soc@vger.kernel.org, linux-clk@vger.kernel.org, linux-gpio@vger.kernel.org, linux-rtc@vger.kernel.org References: <20210920190350.3860821-1-willmcvicker@google.com> From: Krzysztof Kozlowski Message-ID: <7735b09c-cf1c-5e37-a737-9a330fbacf1e@canonical.com> Date: Tue, 21 Sep 2021 09:19:59 +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-1-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: > This patch series tries to address the issue of ARCH_EXYNOS force selecting > a handful of drivers without allowing the vendor to override any of the > default configs. This takes away from the flexibilty of compiling a generic > kernel with exynos kernel modules. For example, it doesn't allow vendors to > modularize these drivers out of the core kernel in order to share a generic > kernel image across multiple devices that require device-specific kernel > modules. You do not address the issue in these patches. The problem you describe is that drivers are not modules and you are not changing them into modules. > > To address this without impacting the existing behavior, this series > switches the default config logic for the offending configs to use "default > y if ARCH_EXYNOS" versus having ARCH_EXYNOS directly select them. I have > verified that these patches do not impact the default aarch64 .config. Yep, this is what you did but it does not match the described problem. You are not solving it but doing something else. > > Will McVicker (4): > clk: samsung: change COMMON_CLK_SAMSUNG default config logic > soc: samsung: change SOC_SAMSUNG default config logic > pinctrl: samsung: change PINCTRL_EXYNOS default config logic > rtc: change HAVE_S3C_RTC default config logic > I received only two patches from this set. Please resend following get_maintainers.pl script. Best regards, Krzysztof