From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
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
Subject: Re: [PATCH v2] arm64: defconfig: Enable HDA INTEL config for ARM64
Date: Fri, 20 Jan 2023 07:20:01 +0100 [thread overview]
Message-ID: <cfb9b0f2-a8fa-9bee-a04d-d195d69e4918@linaro.org> (raw)
In-Reply-To: <35eb1396-b91b-8a7f-6585-30d2f2adcf85@nvidia.com>
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
next prev parent reply other threads:[~2023-01-20 6:20 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-17 18:16 [PATCH v2] arm64: defconfig: Enable HDA INTEL config for ARM64 Mohan Kumar
2023-01-18 7:34 ` Krzysztof Kozlowski
2023-01-18 11:46 ` Mohan Kumar D
2023-01-18 12:36 ` Krzysztof Kozlowski
2023-01-20 5:48 ` Mohan Kumar D
2023-01-20 6:20 ` Krzysztof Kozlowski [this message]
2023-01-20 16:56 ` Catalin Marinas
2023-01-20 17:00 ` Krzysztof Kozlowski
2023-01-23 15:58 ` Thierry Reding
2023-01-23 16:05 ` Krzysztof Kozlowski
2023-01-23 17:26 ` Thierry Reding
2023-01-23 17:34 ` Krzysztof Kozlowski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cfb9b0f2-a8fa-9bee-a04d-d195d69e4918@linaro.org \
--to=krzysztof.kozlowski@linaro.org \
--cc=catalin.marinas@arm.com \
--cc=dmitry.baryshkov@linaro.org \
--cc=jonathanh@nvidia.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mkumard@nvidia.com \
--cc=shawnguo@kernel.org \
--cc=treding@nvidia.com \
--cc=will@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).