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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 72DE5E92733 for ; Thu, 5 Oct 2023 19:29:36 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230168AbjJET3f (ORCPT ); Thu, 5 Oct 2023 15:29:35 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57742 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229939AbjJET3e (ORCPT ); Thu, 5 Oct 2023 15:29:34 -0400 Received: from mail-ej1-x62e.google.com (mail-ej1-x62e.google.com [IPv6:2a00:1450:4864:20::62e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id F359CDE for ; Thu, 5 Oct 2023 12:29:31 -0700 (PDT) Received: by mail-ej1-x62e.google.com with SMTP id a640c23a62f3a-9ada2e6e75fso255389866b.2 for ; Thu, 05 Oct 2023 12:29:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1696534170; x=1697138970; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:autocrypt:from:references:cc :to:content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=8FcIrqBncC9Mrs6EWL94iGiE6FYJb1dG9INjYIsj2wg=; b=Sp+kRiJ60ClUIP33/zrpqR68uGfgHZeMm4+XnQVqUKnwHYLTHDTgipSVBvkvO7lFpR 7h1tSOVKgN8l2i/GTjnG0M9DaNczXTUEAG71AvnBnBn42TQYQFhqer5jGKafpJT7WvR6 tA5eRjDWdtxxNTE/q8IpFKLILd1+IrqsFz0+HQ5cEedOjtug1P7ljlT+kr5h9L3rcov2 VKexLghFPEYp5xgZyK/VHS3hOqF56CEwfJOgLieboNi4pMVXZ0WB5qAlU8Uv79zbLbB7 2QvjoKpbF9H6OzYsCphE03nxvt9/6JUQ6Gl1HpjSo29aiG46TVAeLjoE4qZlXqKcJ2ie BVeQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1696534170; x=1697138970; h=content-transfer-encoding:in-reply-to:autocrypt:from:references:cc :to:content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=8FcIrqBncC9Mrs6EWL94iGiE6FYJb1dG9INjYIsj2wg=; b=T2jHAbG1uYqxMjnfxpeJFJXPkWTUfC2x57IYJSCZJPUwiH6Z5CXrUf9us/Uwbt9CK+ NedGYZeUiYbgwT2WgumJ97v+P03P2yw6xNSc5s+BFHSml0VJUok/fQkv0QpbK2mzDWcQ Rmq9eA4HVoUsiB0VPtP/C9fR2zS+m8rAyO9VEOE53PzgXQ08K3/G4QuaU+7tuhfY020q Ew8j+W1NwON93stRhRcJKTVMJY/4SaFVUOKcrptI2s2MrWdWWq/LcsrMRKndVROHbqd0 lZio1LQqnV5qAsVLn+fT2tu5s8DUFyZd93TWW5KpPh/erG2pxW9n83FgDnqCPbHTmzRz P8PA== X-Gm-Message-State: AOJu0YzwjsXYS82K0BcdXUVIypqQn8TKiFe8d+stUpf5dh0o33VeDa+k WFOG8v+3rBzfYD24PblCZYXE6Q== X-Google-Smtp-Source: AGHT+IH5OVqZVwFMXRaCd1BDzWuuagJUNL+cVtINgJ+JuCQ4BBFliTN5odpOdP1Nfa2kVem3dld3Pg== X-Received: by 2002:a17:906:220c:b0:9b8:a556:87a5 with SMTP id s12-20020a170906220c00b009b8a55687a5mr5241419ejs.22.1696534170424; Thu, 05 Oct 2023 12:29:30 -0700 (PDT) Received: from [192.168.1.197] (5-157-101-10.dyn.eolo.it. [5.157.101.10]) by smtp.gmail.com with ESMTPSA id u2-20020a17090617c200b009ad8084e08asm1640184eje.0.2023.10.05.12.29.27 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 05 Oct 2023 12:29:29 -0700 (PDT) Message-ID: Date: Thu, 5 Oct 2023 21:29:26 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 18/21] arm64: dts: google: Add initial Google gs101 SoC support Content-Language: en-US To: Greg KH Cc: William McVicker , Peter Griffin , robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, mturquette@baylibre.com, conor+dt@kernel.org, sboyd@kernel.org, tomasz.figa@gmail.com, s.nawrocki@samsung.com, linus.walleij@linaro.org, wim@linux-watchdog.org, linux@roeck-us.net, catalin.marinas@arm.com, will@kernel.org, arnd@arndb.de, olof@lixom.net, cw00.choi@samsung.com, tudor.ambarus@linaro.org, andre.draszik@linaro.org, semen.protsenko@linaro.org, soc@kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-clk@vger.kernel.org, linux-gpio@vger.kernel.org, linux-watchdog@vger.kernel.org, kernel-team@google.com References: <20231005155618.700312-1-peter.griffin@linaro.org> <20231005155618.700312-19-peter.griffin@linaro.org> <2023100520-cleaver-sinless-fbae@gregkh> <99419159-cab0-4c79-a4a0-12229bfad3c0@linaro.org> <2023100513-mashing-scrubber-ea59@gregkh> From: Krzysztof Kozlowski Autocrypt: addr=krzysztof.kozlowski@linaro.org; keydata= xsFNBFVDQq4BEAC6KeLOfFsAvFMBsrCrJ2bCalhPv5+KQF2PS2+iwZI8BpRZoV+Bd5kWvN79 cFgcqTTuNHjAvxtUG8pQgGTHAObYs6xeYJtjUH0ZX6ndJ33FJYf5V3yXqqjcZ30FgHzJCFUu JMp7PSyMPzpUXfU12yfcRYVEMQrmplNZssmYhiTeVicuOOypWugZKVLGNm0IweVCaZ/DJDIH gNbpvVwjcKYrx85m9cBVEBUGaQP6AT7qlVCkrf50v8bofSIyVa2xmubbAwwFA1oxoOusjPIE J3iadrwpFvsZjF5uHAKS+7wHLoW9hVzOnLbX6ajk5Hf8Pb1m+VH/E8bPBNNYKkfTtypTDUCj NYcd27tjnXfG+SDs/EXNUAIRefCyvaRG7oRYF3Ec+2RgQDRnmmjCjoQNbFrJvJkFHlPeHaeS BosGY+XWKydnmsfY7SSnjAzLUGAFhLd/XDVpb1Een2XucPpKvt9ORF+48gy12FA5GduRLhQU vK4tU7ojoem/G23PcowM1CwPurC8sAVsQb9KmwTGh7rVz3ks3w/zfGBy3+WmLg++C2Wct6nM Pd8/6CBVjEWqD06/RjI2AnjIq5fSEH/BIfXXfC68nMp9BZoy3So4ZsbOlBmtAPvMYX6U8VwD TNeBxJu5Ex0Izf1NV9CzC3nNaFUYOY8KfN01X5SExAoVTr09ewARAQABzTRLcnp5c3p0b2Yg S296bG93c2tpIDxrcnp5c3p0b2Yua296bG93c2tpQGxpbmFyby5vcmc+wsGUBBMBCgA+FiEE m9B+DgxR+NWWd7dUG5NDfTtBYpsFAmI+BxMCGwMFCRRfreEFCwkIBwIGFQoJCAsCBBYCAwEC HgECF4AACgkQG5NDfTtBYptgbhAAjAGunRoOTduBeC7V6GGOQMYIT5n3OuDSzG1oZyM4kyvO XeodvvYv49/ng473E8ZFhXfrre+c1olbr1A8pnz9vKVQs9JGVa6wwr/6ddH7/yvcaCQnHRPK mnXyP2BViBlyDWQ71UC3N12YCoHE2cVmfrn4JeyK/gHCvcW3hUW4i5rMd5M5WZAeiJj3rvYh v8WMKDJOtZFXxwaYGbvFJNDdvdTHc2x2fGaWwmXMJn2xs1ZyFAeHQvrp49mS6PBQZzcx0XL5 cU9ZjhzOZDn6Apv45/C/lUJvPc3lo/pr5cmlOvPq1AsP6/xRXsEFX/SdvdxJ8w9KtGaxdJuf rpzLQ8Ht+H0lY2On1duYhmro8WglOypHy+TusYrDEry2qDNlc/bApQKtd9uqyDZ+rx8bGxyY qBP6bvsQx5YACI4p8R0J43tSqWwJTP/R5oPRQW2O1Ye1DEcdeyzZfifrQz58aoZrVQq+innR aDwu8qDB5UgmMQ7cjDSeAQABdghq7pqrA4P8lkA7qTG+aw8Z21OoAyZdUNm8NWJoQy8m4nUP gmeeQPRc0vjp5JkYPgTqwf08cluqO6vQuYL2YmwVBIbO7cE7LNGkPDA3RYMu+zPY9UUi/ln5 dcKuEStFZ5eqVyqVoZ9eu3RTCGIXAHe1NcfcMT9HT0DPp3+ieTxFx6RjY3kYTGLOwU0EVUNc NAEQAM2StBhJERQvgPcbCzjokShn0cRA4q2SvCOvOXD+0KapXMRFE+/PZeDyfv4dEKuCqeh0 hihSHlaxTzg3TcqUu54w2xYskG8Fq5tg3gm4kh1Gvh1LijIXX99ABA8eHxOGmLPRIBkXHqJY oHtCvPc6sYKNM9xbp6I4yF56xVLmHGJ61KaWKf5KKWYgA9kfHufbja7qR0c6H79LIsiYqf92 H1HNq1WlQpu/fh4/XAAaV1axHFt/dY/2kU05tLMj8GjeQDz1fHas7augL4argt4e+jum3Nwt yupodQBxncKAUbzwKcDrPqUFmfRbJ7ARw8491xQHZDsP82JRj4cOJX32sBg8nO2N5OsFJOcd 5IE9v6qfllkZDAh1Rb1h6DFYq9dcdPAHl4zOj9EHq99/CpyccOh7SrtWDNFFknCmLpowhct9 5ZnlavBrDbOV0W47gO33WkXMFI4il4y1+Bv89979rVYn8aBohEgET41SpyQz7fMkcaZU+ok/ +HYjC/qfDxT7tjKXqBQEscVODaFicsUkjheOD4BfWEcVUqa+XdUEciwG/SgNyxBZepj41oVq FPSVE+Ni2tNrW/e16b8mgXNngHSnbsr6pAIXZH3qFW+4TKPMGZ2rZ6zITrMip+12jgw4mGjy 5y06JZvA02rZT2k9aa7i9dUUFggaanI09jNGbRA/ABEBAAHCwXwEGAEKACYCGwwWIQSb0H4O DFH41ZZ3t1Qbk0N9O0FimwUCYDzvagUJFF+UtgAKCRAbk0N9O0Fim9JzD/0auoGtUu4mgnna oEEpQEOjgT7l9TVuO3Qa/SeH+E0m55y5Fjpp6ZToc481za3xAcxK/BtIX5Wn1mQ6+szfrJQ6 59y2io437BeuWIRjQniSxHz1kgtFECiV30yHRgOoQlzUea7FgsnuWdstgfWi6LxstswEzxLZ Sj1EqpXYZE4uLjh6dW292sO+j4LEqPYr53hyV4I2LPmptPE9Rb9yCTAbSUlzgjiyyjuXhcwM qf3lzsm02y7Ooq+ERVKiJzlvLd9tSe4jRx6Z6LMXhB21fa5DGs/tHAcUF35hSJrvMJzPT/+u /oVmYDFZkbLlqs2XpWaVCo2jv8+iHxZZ9FL7F6AHFzqEFdqGnJQqmEApiRqH6b4jRBOgJ+cY qc+rJggwMQcJL9F+oDm3wX47nr6jIsEB5ZftdybIzpMZ5V9v45lUwmdnMrSzZVgC4jRGXzsU EViBQt2CopXtHtYfPAO5nAkIvKSNp3jmGxZw4aTc5xoAZBLo0OV+Ezo71pg3AYvq0a3/oGRG KQ06ztUMRrj8eVtpImjsWCd0bDWRaaR4vqhCHvAG9iWXZu4qh3ipie2Y0oSJygcZT7H3UZxq fyYKiqEmRuqsvv6dcbblD8ZLkz1EVZL6djImH5zc5x8qpVxlA0A0i23v5QvN00m6G9NFF0Le D2GYIS41Kv4Isx2dEFh+/Q== In-Reply-To: <2023100513-mashing-scrubber-ea59@gregkh> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-gpio@vger.kernel.org On 05/10/2023 21:23, Greg KH wrote: > On Thu, Oct 05, 2023 at 09:18:48PM +0200, Krzysztof Kozlowski wrote: >>>> I'd like to bring up this thread and discuss the option of not introducing >>>> another ARCH_* config: >>>> >>>> https://lore.kernel.org/all/20200306103652.GA3634389@kroah.com/ >>> >>> I agree, PLEASE don't add platform config options as that makes it >>> impossible to make a unified kernel image that works for more than one >>> platform at the same time. >> >> There is no single problem in making unified image as we were doing >> since beginning of ARM64. The ARCH_* is not a obstacle for this. > > Then why are the ARCH_* options needed at all? What does this help out > with? It helps all the people and distros who do not want to build/package drivers or modules for unrelated hardware or architectures. Let's take Samsung Exynos UART driver. It will never, 100% never, work on x86, x86_64. There is no single need to package it for kernels build for these products. It will not work on nVidia Tegra ARM64, Qualcomm ARM64 SoC, so if you do not want to run on Exynos, then you do no select ARCH_EXYNOS and have significantly smaller image. Now, there is no problem to have one kernel for nVidia Tegra + Qualcomm + Samsung Exynos with everything you need. The ARCH_EXYNOS or SOC_EXYNOS or SOC_GOOGLE serves only the purpose to allow distros and people customize build for specific hardware. It does not limit anyone on anything. > >>>> I especially don't like the "depends on ARCH_EXYNOS" because that forces one to >>>> include all the other Exynos drivers that ARCH_EXYNOS selects that Google >>>> Tensor SoCs don't need. Can we consider using SOC_GOOGLE instead and for all >>>> drivers that actually depend on the SoC hardware, we can just add "depends on >>>> SOC_GOOGLE"? >>> >>> Why do any of this at all? It should not be needed. >>> >>>> The idea is that drivers should be tied to hardware -- not a specific vendor. >>> >>> And drivers should be auto-loaded. >>> >>> All of these drivers are not vendor-specific at all, they are based on >>> the same IP blocks as others, so that is how they should be unified. >> >> They are vendor specific. All of them are specifically for Exynos >> hardwre, because this is Exynos. We call it Google GS/Tensor SoC just >> for fancy convenience, but this just Exynos. > > Ok, then why is this ARCH_ option needed if these IP blocks really are > from something else and are part of other drivers? For the same reason above, because if I want to build kernel for Qualcomm, I want to drop easily anything not related. If I want to build kernel without I2C, I disable I2C bus which effectively disables all drivers which work on I2C. If I want to build kernel without Exynos, I disable ARCH_EXYNOS which effectively disables entire Exynos hardware. Think of SoC as a bus or interface. 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 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 D0B23E92730 for ; Thu, 5 Oct 2023 19:30:05 +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:In-Reply-To:From:References:Cc:To: Subject:MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=tlDYtnOnDeNPALwQiPa3xFs3VJ5Jw/7eb9Y0WUWkSiY=; b=kYTNwiBOz0DOxg HtFSktNtvwUYy3sVzoM1ifXQ+zOXGVZTVyL9aoYJve+bAT2EoZ3tVXLwW6ioCpQmUGPM/iZ53pi1g TS10Ke5brnuvJ1Hq7xgp7/YcTeBQDJ2HhnYkUPsVnTbqCgHIbYb0FLfMyMJVAHd2VAum9J5opd5fQ jfAtYmJOHKZ9siiE6jniVkf4W9V/tIvgfJCCE1aHpcXt25Hk7dQayIPfCuLKQkorilc/yvSVVtsY2 mO/1+Nu4+QOmTCY6Gm8Z53+Vru7A0IyJ278X06s/vkxPZl9MZTWlt1XzCWRmezrlzNzrXFpmS7axI uO1VC7LLMJv0lOtATKQw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qoU2T-004NGf-1c; Thu, 05 Oct 2023 19:29:37 +0000 Received: from mail-ej1-x634.google.com ([2a00:1450:4864:20::634]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qoU2P-004NG7-1G for linux-arm-kernel@lists.infradead.org; Thu, 05 Oct 2023 19:29:35 +0000 Received: by mail-ej1-x634.google.com with SMTP id a640c23a62f3a-9ada2e6e75fso255390266b.2 for ; Thu, 05 Oct 2023 12:29:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1696534170; x=1697138970; darn=lists.infradead.org; h=content-transfer-encoding:in-reply-to:autocrypt:from:references:cc :to:content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=8FcIrqBncC9Mrs6EWL94iGiE6FYJb1dG9INjYIsj2wg=; b=brM3rxoCh0HWCtCiWR9dbjyW+Z3G7bdfo/pwnjLhnXxYJh00azDSQPYXDkTNgHH8/L xFdgPKiX5GVOZRfCxcb35JGBXv18/tes6jkoJ/z+sqf2Jc3EHexwpPalXApZkSoa5V/d VyNTMbqZf/yGKnd7/MQQdJpawsv7o6eVZk4hpkOrBi4YolknF0SXDJfkpxSgOaIiBoMY KtyM3pFwRhbjUVO/YgAdGKUiP5gQaclX8U2TilcQ4sWMhRxSG3EPGw5porLNbbtFICFt Jy71M2qW90OIKqU+LKl3fC/p/ZPGBtQ49jfWZXg54OYmcDXBZ+kn3+dEyzA5pksr4RGb c/4A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1696534170; x=1697138970; h=content-transfer-encoding:in-reply-to:autocrypt:from:references:cc :to:content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=8FcIrqBncC9Mrs6EWL94iGiE6FYJb1dG9INjYIsj2wg=; b=Yz3xi8hHCFqZ+lTkb1JMVCaxnsupv979JAYmqyFdZ5NBFEaynkr+ndbDcmeuP0RR8k jWnXdT3+175bMT3VxLEj4grt3VjE2sNMIYyc9A6zMt5m+HoDTOE5QR5Bvj3f4Ct3CPnU Vsf/nwqTR7EQhUr4sDKMdbuHqMkuxrA4cS1ebk2NgufuU4B/9IeSHb2HSA3Cu7QDOz13 kTBWeDHhLdyvS7scVm16nnAtJWh9ZsDFE8ToQkLWJvBY38ihAE+L8c5tEYICck8fBv16 VEB5V2lWS4bO+hKiw4LTDwuhYzrTfjqj76+NZ51z3FQta3BzDQ7ijvg0sbnC2c576pxj 3SCQ== X-Gm-Message-State: AOJu0YwLv9V/v7KS6akTOS/4iios44D+cCuvRK/z/6qQmTeWZVLV2E/j Ge8qC0aqx5Qrwzpl2cUks5tuPQ== X-Google-Smtp-Source: AGHT+IH5OVqZVwFMXRaCd1BDzWuuagJUNL+cVtINgJ+JuCQ4BBFliTN5odpOdP1Nfa2kVem3dld3Pg== X-Received: by 2002:a17:906:220c:b0:9b8:a556:87a5 with SMTP id s12-20020a170906220c00b009b8a55687a5mr5241419ejs.22.1696534170424; Thu, 05 Oct 2023 12:29:30 -0700 (PDT) Received: from [192.168.1.197] (5-157-101-10.dyn.eolo.it. [5.157.101.10]) by smtp.gmail.com with ESMTPSA id u2-20020a17090617c200b009ad8084e08asm1640184eje.0.2023.10.05.12.29.27 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 05 Oct 2023 12:29:29 -0700 (PDT) Message-ID: Date: Thu, 5 Oct 2023 21:29:26 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 18/21] arm64: dts: google: Add initial Google gs101 SoC support Content-Language: en-US To: Greg KH Cc: William McVicker , Peter Griffin , robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, mturquette@baylibre.com, conor+dt@kernel.org, sboyd@kernel.org, tomasz.figa@gmail.com, s.nawrocki@samsung.com, linus.walleij@linaro.org, wim@linux-watchdog.org, linux@roeck-us.net, catalin.marinas@arm.com, will@kernel.org, arnd@arndb.de, olof@lixom.net, cw00.choi@samsung.com, tudor.ambarus@linaro.org, andre.draszik@linaro.org, semen.protsenko@linaro.org, soc@kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-clk@vger.kernel.org, linux-gpio@vger.kernel.org, linux-watchdog@vger.kernel.org, kernel-team@google.com References: <20231005155618.700312-1-peter.griffin@linaro.org> <20231005155618.700312-19-peter.griffin@linaro.org> <2023100520-cleaver-sinless-fbae@gregkh> <99419159-cab0-4c79-a4a0-12229bfad3c0@linaro.org> <2023100513-mashing-scrubber-ea59@gregkh> From: Krzysztof Kozlowski Autocrypt: addr=krzysztof.kozlowski@linaro.org; keydata= xsFNBFVDQq4BEAC6KeLOfFsAvFMBsrCrJ2bCalhPv5+KQF2PS2+iwZI8BpRZoV+Bd5kWvN79 cFgcqTTuNHjAvxtUG8pQgGTHAObYs6xeYJtjUH0ZX6ndJ33FJYf5V3yXqqjcZ30FgHzJCFUu JMp7PSyMPzpUXfU12yfcRYVEMQrmplNZssmYhiTeVicuOOypWugZKVLGNm0IweVCaZ/DJDIH gNbpvVwjcKYrx85m9cBVEBUGaQP6AT7qlVCkrf50v8bofSIyVa2xmubbAwwFA1oxoOusjPIE J3iadrwpFvsZjF5uHAKS+7wHLoW9hVzOnLbX6ajk5Hf8Pb1m+VH/E8bPBNNYKkfTtypTDUCj NYcd27tjnXfG+SDs/EXNUAIRefCyvaRG7oRYF3Ec+2RgQDRnmmjCjoQNbFrJvJkFHlPeHaeS BosGY+XWKydnmsfY7SSnjAzLUGAFhLd/XDVpb1Een2XucPpKvt9ORF+48gy12FA5GduRLhQU vK4tU7ojoem/G23PcowM1CwPurC8sAVsQb9KmwTGh7rVz3ks3w/zfGBy3+WmLg++C2Wct6nM Pd8/6CBVjEWqD06/RjI2AnjIq5fSEH/BIfXXfC68nMp9BZoy3So4ZsbOlBmtAPvMYX6U8VwD TNeBxJu5Ex0Izf1NV9CzC3nNaFUYOY8KfN01X5SExAoVTr09ewARAQABzTRLcnp5c3p0b2Yg S296bG93c2tpIDxrcnp5c3p0b2Yua296bG93c2tpQGxpbmFyby5vcmc+wsGUBBMBCgA+FiEE m9B+DgxR+NWWd7dUG5NDfTtBYpsFAmI+BxMCGwMFCRRfreEFCwkIBwIGFQoJCAsCBBYCAwEC HgECF4AACgkQG5NDfTtBYptgbhAAjAGunRoOTduBeC7V6GGOQMYIT5n3OuDSzG1oZyM4kyvO XeodvvYv49/ng473E8ZFhXfrre+c1olbr1A8pnz9vKVQs9JGVa6wwr/6ddH7/yvcaCQnHRPK mnXyP2BViBlyDWQ71UC3N12YCoHE2cVmfrn4JeyK/gHCvcW3hUW4i5rMd5M5WZAeiJj3rvYh v8WMKDJOtZFXxwaYGbvFJNDdvdTHc2x2fGaWwmXMJn2xs1ZyFAeHQvrp49mS6PBQZzcx0XL5 cU9ZjhzOZDn6Apv45/C/lUJvPc3lo/pr5cmlOvPq1AsP6/xRXsEFX/SdvdxJ8w9KtGaxdJuf rpzLQ8Ht+H0lY2On1duYhmro8WglOypHy+TusYrDEry2qDNlc/bApQKtd9uqyDZ+rx8bGxyY qBP6bvsQx5YACI4p8R0J43tSqWwJTP/R5oPRQW2O1Ye1DEcdeyzZfifrQz58aoZrVQq+innR aDwu8qDB5UgmMQ7cjDSeAQABdghq7pqrA4P8lkA7qTG+aw8Z21OoAyZdUNm8NWJoQy8m4nUP gmeeQPRc0vjp5JkYPgTqwf08cluqO6vQuYL2YmwVBIbO7cE7LNGkPDA3RYMu+zPY9UUi/ln5 dcKuEStFZ5eqVyqVoZ9eu3RTCGIXAHe1NcfcMT9HT0DPp3+ieTxFx6RjY3kYTGLOwU0EVUNc NAEQAM2StBhJERQvgPcbCzjokShn0cRA4q2SvCOvOXD+0KapXMRFE+/PZeDyfv4dEKuCqeh0 hihSHlaxTzg3TcqUu54w2xYskG8Fq5tg3gm4kh1Gvh1LijIXX99ABA8eHxOGmLPRIBkXHqJY oHtCvPc6sYKNM9xbp6I4yF56xVLmHGJ61KaWKf5KKWYgA9kfHufbja7qR0c6H79LIsiYqf92 H1HNq1WlQpu/fh4/XAAaV1axHFt/dY/2kU05tLMj8GjeQDz1fHas7augL4argt4e+jum3Nwt yupodQBxncKAUbzwKcDrPqUFmfRbJ7ARw8491xQHZDsP82JRj4cOJX32sBg8nO2N5OsFJOcd 5IE9v6qfllkZDAh1Rb1h6DFYq9dcdPAHl4zOj9EHq99/CpyccOh7SrtWDNFFknCmLpowhct9 5ZnlavBrDbOV0W47gO33WkXMFI4il4y1+Bv89979rVYn8aBohEgET41SpyQz7fMkcaZU+ok/ +HYjC/qfDxT7tjKXqBQEscVODaFicsUkjheOD4BfWEcVUqa+XdUEciwG/SgNyxBZepj41oVq FPSVE+Ni2tNrW/e16b8mgXNngHSnbsr6pAIXZH3qFW+4TKPMGZ2rZ6zITrMip+12jgw4mGjy 5y06JZvA02rZT2k9aa7i9dUUFggaanI09jNGbRA/ABEBAAHCwXwEGAEKACYCGwwWIQSb0H4O DFH41ZZ3t1Qbk0N9O0FimwUCYDzvagUJFF+UtgAKCRAbk0N9O0Fim9JzD/0auoGtUu4mgnna oEEpQEOjgT7l9TVuO3Qa/SeH+E0m55y5Fjpp6ZToc481za3xAcxK/BtIX5Wn1mQ6+szfrJQ6 59y2io437BeuWIRjQniSxHz1kgtFECiV30yHRgOoQlzUea7FgsnuWdstgfWi6LxstswEzxLZ Sj1EqpXYZE4uLjh6dW292sO+j4LEqPYr53hyV4I2LPmptPE9Rb9yCTAbSUlzgjiyyjuXhcwM qf3lzsm02y7Ooq+ERVKiJzlvLd9tSe4jRx6Z6LMXhB21fa5DGs/tHAcUF35hSJrvMJzPT/+u /oVmYDFZkbLlqs2XpWaVCo2jv8+iHxZZ9FL7F6AHFzqEFdqGnJQqmEApiRqH6b4jRBOgJ+cY qc+rJggwMQcJL9F+oDm3wX47nr6jIsEB5ZftdybIzpMZ5V9v45lUwmdnMrSzZVgC4jRGXzsU EViBQt2CopXtHtYfPAO5nAkIvKSNp3jmGxZw4aTc5xoAZBLo0OV+Ezo71pg3AYvq0a3/oGRG KQ06ztUMRrj8eVtpImjsWCd0bDWRaaR4vqhCHvAG9iWXZu4qh3ipie2Y0oSJygcZT7H3UZxq fyYKiqEmRuqsvv6dcbblD8ZLkz1EVZL6djImH5zc5x8qpVxlA0A0i23v5QvN00m6G9NFF0Le D2GYIS41Kv4Isx2dEFh+/Q== In-Reply-To: <2023100513-mashing-scrubber-ea59@gregkh> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231005_122933_440146_132FC029 X-CRM114-Status: GOOD ( 22.25 ) 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 05/10/2023 21:23, Greg KH wrote: > On Thu, Oct 05, 2023 at 09:18:48PM +0200, Krzysztof Kozlowski wrote: >>>> I'd like to bring up this thread and discuss the option of not introducing >>>> another ARCH_* config: >>>> >>>> https://lore.kernel.org/all/20200306103652.GA3634389@kroah.com/ >>> >>> I agree, PLEASE don't add platform config options as that makes it >>> impossible to make a unified kernel image that works for more than one >>> platform at the same time. >> >> There is no single problem in making unified image as we were doing >> since beginning of ARM64. The ARCH_* is not a obstacle for this. > > Then why are the ARCH_* options needed at all? What does this help out > with? It helps all the people and distros who do not want to build/package drivers or modules for unrelated hardware or architectures. Let's take Samsung Exynos UART driver. It will never, 100% never, work on x86, x86_64. There is no single need to package it for kernels build for these products. It will not work on nVidia Tegra ARM64, Qualcomm ARM64 SoC, so if you do not want to run on Exynos, then you do no select ARCH_EXYNOS and have significantly smaller image. Now, there is no problem to have one kernel for nVidia Tegra + Qualcomm + Samsung Exynos with everything you need. The ARCH_EXYNOS or SOC_EXYNOS or SOC_GOOGLE serves only the purpose to allow distros and people customize build for specific hardware. It does not limit anyone on anything. > >>>> I especially don't like the "depends on ARCH_EXYNOS" because that forces one to >>>> include all the other Exynos drivers that ARCH_EXYNOS selects that Google >>>> Tensor SoCs don't need. Can we consider using SOC_GOOGLE instead and for all >>>> drivers that actually depend on the SoC hardware, we can just add "depends on >>>> SOC_GOOGLE"? >>> >>> Why do any of this at all? It should not be needed. >>> >>>> The idea is that drivers should be tied to hardware -- not a specific vendor. >>> >>> And drivers should be auto-loaded. >>> >>> All of these drivers are not vendor-specific at all, they are based on >>> the same IP blocks as others, so that is how they should be unified. >> >> They are vendor specific. All of them are specifically for Exynos >> hardwre, because this is Exynos. We call it Google GS/Tensor SoC just >> for fancy convenience, but this just Exynos. > > Ok, then why is this ARCH_ option needed if these IP blocks really are > from something else and are part of other drivers? For the same reason above, because if I want to build kernel for Qualcomm, I want to drop easily anything not related. If I want to build kernel without I2C, I disable I2C bus which effectively disables all drivers which work on I2C. If I want to build kernel without Exynos, I disable ARCH_EXYNOS which effectively disables entire Exynos hardware. Think of SoC as a bus or interface. Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel