Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <cfb9b0f2-a8fa-9bee-a04d-d195d69e4918@linaro.org>
Date:   Fri, 20 Jan 2023 07:20:01 +0100
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.7.0
Subject: Re: [PATCH v2] arm64: defconfig: Enable HDA INTEL config for ARM64
To:     Mohan Kumar D <mkumard@nvidia.com>, catalin.marinas@arm.com,
        will@kernel.org, dmitry.baryshkov@linaro.org, shawnguo@kernel.org
Cc:     linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        treding@nvidia.com, jonathanh@nvidia.com
References: <20230117181658.17010-1-mkumard@nvidia.com>
 <e871dae2-58ac-7ad9-c198-c4e90d26c69b@linaro.org>
 <1afa38ad-716f-49f8-efd1-ed37bd8dbf6e@nvidia.com>
 <ec440deb-2970-69b8-e54d-6519c95875a8@linaro.org>
 <35eb1396-b91b-8a7f-6585-30d2f2adcf85@nvidia.com>
Content-Language: en-US
From:   Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
In-Reply-To: <35eb1396-b91b-8a7f-6585-30d2f2adcf85@nvidia.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 20/01/2023 06:48, Mohan Kumar D wrote:
> 
> On 18-01-2023 18:06, Krzysztof Kozlowski wrote:
>> External email: Use caution opening links or attachments
>>
>>
>> On 18/01/2023 12:46, Mohan Kumar D wrote:
>>> On 18-01-2023 13:04, Krzysztof Kozlowski wrote:
>>>> External email: Use caution opening links or attachments
>>>>
>>>>
>>>> On 17/01/2023 19:16, Mohan Kumar wrote:
>>>>> Enable CONFIG_SND_HDA_INTEL for NVIDIA PCI based graphics sound card for
>>>>> ARM64 based platforms as Intel PCI driver was used for registering the
>>>>> sound card.
>>>> It's not a part of SoC, not a common device used during debugging or
>>>> development, so I don't think it is reasonable to enable it. We do not
>>>> enable driver just because someone uses them. Otherwise please clarify
>>>> which board has this device embedded (not pluggable by user, but embedded).
>>> This change is required for enabling HDA sound registration for Nvidia
>>> discrete GPU cards based on PCI and pluggable to Nvidia Jetson Platforms.
>> You can plug anything to PCI slot and we do not enable every such PCI
>> adapter.
> Without this config enabled, the Intel hda audio driver won't be built 
> and dGPU won't be able to register sound card. Do you have any 
> suggestion here?

Without hundreds of other drivers they also won't be built and won't be
usable. Anyway, this is just defconfig, so it does not matter. You can
always enable it in your setup, why is this a problem?

Again, we do not enable drivers for every PCI card.

Best regards,
Krzysztof