Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Damien Le Moal <Damien.LeMoal@wdc.com>
To:     Palmer Dabbelt <palmer@dabbelt.com>,
        "heinrich.schuchardt@canonical.com" 
        <heinrich.schuchardt@canonical.com>
CC:     Paul Walmsley <paul.walmsley@sifive.com>,
        "aou@eecs.berkeley.edu" <aou@eecs.berkeley.edu>,
        Anup Patel <Anup.Patel@wdc.com>,
        "axboe@kernel.dk" <axboe@kernel.dk>,
        "sagar.kadam@sifive.com" <sagar.kadam@sifive.com>,
        Atish Patra <Atish.Patra@wdc.com>,
        "linux-riscv@lists.infradead.org" <linux-riscv@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 1/1] riscv: defconfig: enable DRM_NOUVEAU
Thread-Topic: [PATCH 1/1] riscv: defconfig: enable DRM_NOUVEAU
Thread-Index: AQHXv4kNtwgxdmgy0k6ZFdVRkGjXuw==
Date:   Wed, 27 Oct 2021 02:31:40 +0000
Message-ID: <DM6PR04MB7081C64D27D133CEDE728393E7859@DM6PR04MB7081.namprd04.prod.outlook.com>
References: <mhng-5dc2aa26-9f1c-4764-9877-6d99569210b4@palmerdabbelt-glaptop>
Accept-Language: en-US
Content-Language: en-US
wdcipoutbound: EOP-TRUE
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?FJo1A3Ozz7XlKMG74kcqtyg0TSmJJihINvufJMra8iTgKLaAZRhU9HFFL3sA?=
 =?us-ascii?Q?h7vwvTJAkPydFbqAfedfe++bol57u0Wwk5o22ZW7QmHLkg1zWfDrfXj8fDbh?=
 =?us-ascii?Q?hcbWoUdj0n8F5vMVUTvtWGjB1It+Q3hgWyrET8489JWAhg2nCigbq6dotOxv?=
 =?us-ascii?Q?nNxjrsHgWn1R4utIIA+71cY0GY4VTCCVSuZRBKrsSEbZfCIMaKZm0RHB+7ko?=
 =?us-ascii?Q?sLY8pztcXPkjI57T2ibp330o20iZRI8IXUUKmK89Cr1CyCixEbV20jaIXwqE?=
 =?us-ascii?Q?Cowd03gWMQm7fNoIPj/2d4mILyUfEDW4EgmPEDk/cXDEl9zsB73ZqvSIjaDY?=
 =?us-ascii?Q?9Iv3JKvrgiP4yRTwOnLiFUC05D1iN9iJIViS1nKb5fdpqXErM8OYeUSRfZJe?=
 =?us-ascii?Q?0Y8Re+jyORKgTNMOgNZMOs/mCfR1ktQKQ4pwkC23dv5roSJi6qitaN7r2prn?=
 =?us-ascii?Q?lRN1YaablePlSqtP937gDKdAFVNIKKaeihs/aa3LJhL0AX0kDuHWvosPYqYp?=
 =?us-ascii?Q?Tm9SYmtSnY1IwK5UxfaLItrtb5Z0Cwvn516OBasjxNSJVjGDoKpd2d/vwVk8?=
 =?us-ascii?Q?kCFpcNmswKACIslgaTiTDYVfSDnNmc7mV0MqXUJjxa3taK5bCB84Ie1ywz2p?=
 =?us-ascii?Q?7K63/Msm0oqbMPQfi2uT5gSZHDq4tuNdnBLeOJ/aOO908rDg6yrCxMPj6TKm?=
 =?us-ascii?Q?a4grfi9SC4oCGQ6gNtq/6qJTvmwnGSTSJH6fsQaSbwQFJe71AJBEPtecgJW+?=
 =?us-ascii?Q?Qo5pn13dtiUZgz6I6yZBP/ACkWwFdzRyZzus9mXgtUcpouHfVFgJE8p1Mh3/?=
 =?us-ascii?Q?VCNqdlWu4t7u7rl3olXFHbssWMv/slGkjViTRCrywdPDCHRB4ZtmZv6z+Xn8?=
 =?us-ascii?Q?WthkLTW4mz0MHiQlOQUAdFu9PIZi1zCcgsW/x7/aN4VkmLPUWfUB/cs12Rtl?=
 =?us-ascii?Q?wh+aVIagWP6BReCoKRTd96+RWnHgQe6PxM5dLs3S8YPFXVPS8ikO1Kjcqyuo?=
 =?us-ascii?Q?TSX2rg8DjFaDJ8pGximEsPeKPvWtziYFh/Tkyzuxss8ua1x5bjshpOSqq5Et?=
 =?us-ascii?Q?eOx4I58e+dm8hZY87QulASv9dmWTuvoeYfAFsHhuCwaWiIUMPzBzH+NH+lWO?=
 =?us-ascii?Q?E5EnCMznuFKZIpCU/0FL0dX6tzIwglt4ZNnbI+JG9PIeGYX12nrD1cl9n638?=
 =?us-ascii?Q?O0U2Pi8YC1wiiIuqRR0lF0hT10yxH2gSPX6klh9+N0BZXo5AH2te2scx0pUq?=
 =?us-ascii?Q?P2DT8OjqE9lMoWUGyEzJHu8iGNPakMx2SCrS0h6xxsi2YGzNnVNleW8akP7g?=
 =?us-ascii?Q?GbTSFBUroL4NoXquUzd7lU+jn2uba5GsANa0WuMCncUW71qhumtACCLqOxwk?=
 =?us-ascii?Q?UmRNjthzAtPicNm+Mn8OliF3V5LXSMuMvDi7LpzX2+VbvgF3Ry/re9F0+dvY?=
 =?us-ascii?Q?T00uxM8easwp2l1Fd7hFtCZ+Lbuqi10UG0Pe7QGkbdgxgiyN3o31oNW0sTWB?=
 =?us-ascii?Q?GSH+xhptRejWfCg4fQ5GXFDE3fqZKRJU5OI2WP1ARQRkBoepX/yTOGz5USV4?=
 =?us-ascii?Q?xAszZSaaTZ+RX+UIJ7C+66pXLy7uQkvaKkOmww9kCq/O5xay0NyrmNTWfRj2?=
 =?us-ascii?Q?wUDOUxyVV3LfuYcL3bTHxpU5SXIiSCHaykd51UVVDf+k?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: wdc.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DM6PR04MB7081.namprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f796243e-2fbf-4e84-8c9e-08d998f1e884
X-MS-Exchange-CrossTenant-originalarrivaltime: 27 Oct 2021 02:31:40.3703
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: b61c8803-16f3-4c35-9b17-6f65f441df86
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: nRTBsE3pUFcYdsMLyI8vIP+xBELZ+HXWJPtX/NdWEAEFmiDks/HCFSHGOJnfDbFSkCfUiIjzPs8jnVvhEgc2tg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR04MB0813
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 2021/10/27 8:58, Palmer Dabbelt wrote:=0A=
> On Tue, 12 Oct 2021 09:46:58 PDT (-0700), heinrich.schuchardt@canonical.c=
om wrote:=0A=
>> Both RADEON and NOUVEAU graphics cards are supported on RISC-V. Enabling=
=0A=
>> the one and not the other does not make sense.=0A=
>>=0A=
>> As typically at most one of RADEON, NOUVEAU, or VIRTIO GPU support will =
be=0A=
>> needed DRM drivers should be compiled as modules.=0A=
> =0A=
> Do you have an nVidia card that works on real hardware?  Last I checked =
=0A=
> was a while ago, but they weren't working at the time (IIRC it was =0A=
> something to do with PCIe addressing, but it was a hardware limitation =
=0A=
> so I don't remember exactly how it all fits together).=0A=
> =0A=
> If they work then I'm happy to flip them on.=0A=
=0A=
Why enable this at all ? If they do not work, then disabling theses cards m=
ake=0A=
sense. But if they do work, isn't leaving the config to be the default defi=
ned=0A=
by the driver the preferred approach ? Otherwise, we will eventually end-up=
 with=0A=
a defconfig that has everything enabled...=0A=
=0A=
> =0A=
>> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>=
=0A=
>> ---=0A=
>>  arch/riscv/configs/defconfig | 7 ++++---=0A=
>>  1 file changed, 4 insertions(+), 3 deletions(-)=0A=
>>=0A=
>> diff --git a/arch/riscv/configs/defconfig b/arch/riscv/configs/defconfig=
=0A=
>> index 4ebc80315f01..c252fd5706d2 100644=0A=
>> --- a/arch/riscv/configs/defconfig=0A=
>> +++ b/arch/riscv/configs/defconfig=0A=
>> @@ -72,9 +72,10 @@ CONFIG_GPIOLIB=3Dy=0A=
>>  CONFIG_GPIO_SIFIVE=3Dy=0A=
>>  # CONFIG_PTP_1588_CLOCK is not set=0A=
>>  CONFIG_POWER_RESET=3Dy=0A=
>> -CONFIG_DRM=3Dy=0A=
>> -CONFIG_DRM_RADEON=3Dy=0A=
>> -CONFIG_DRM_VIRTIO_GPU=3Dy=0A=
>> +CONFIG_DRM=3Dm=0A=
>> +CONFIG_DRM_RADEON=3Dm=0A=
>> +CONFIG_DRM_NOUVEAU=3Dm=0A=
>> +CONFIG_DRM_VIRTIO_GPU=3Dm=0A=
>>  CONFIG_FRAMEBUFFER_CONSOLE=3Dy=0A=
>>  CONFIG_USB=3Dy=0A=
>>  CONFIG_USB_XHCI_HCD=3Dy=0A=
> =0A=
> _______________________________________________=0A=
> linux-riscv mailing list=0A=
> linux-riscv@lists.infradead.org=0A=
> http://lists.infradead.org/mailman/listinfo/linux-riscv=0A=
> =0A=
=0A=
=0A=
-- =0A=
Damien Le Moal=0A=
Western Digital Research=0A=