On Wed, 28 May 2025 at 19:50, Rob Clark <[email protected]> wrote: > > On Tue, May 27, 2025 at 11:18 AM Dmitry Baryshkov > <[email protected]> wrote: > > > > On Tue, May 27, 2025 at 12:55:06PM +0200, Konrad Dybcio wrote: > > > On 5/26/25 5:28 PM, Rob Clark wrote: > > > > On Mon, May 26, 2025 at 1:36 AM Dmitry Baryshkov > > > > <[email protected]> wrote: > > > >> > > > >> On Sun, May 25, 2025 at 09:43:36PM +0200, Aleksandrs Vinarskis wrote: > > > >>> On Sun, 25 May 2025 at 15:33, Dmitry Baryshkov > > > >>> <[email protected]> wrote: > > > >>>> > > > > > > [...] > > > > > > >> Yes, please. Don't set okay status for the GPU until it gets enabled. > > > > > > > > Drive-by: Shouldn't the dtb describe the hw and not the state of the > > > > linux kernel's support for the hw? Ie. if bad things happen if we > > > > describe hw which is missing driver support, shouldn't we fix that in > > > > the driver. > > > > > > > > (In the case of the GPU there is the slight wrinkle that we don't have > > > > a gpu-id yet so there is no compatible in the dtb yet.) > > > > > > My two cents are that it's okay to enable it, at least in this case.. > > > > That would most likely make display unavailable as DPU driver uses GPU > > as one of the components. > > Hmm, perhaps we should allow the gpu to probe, but just fail to create > priv->gpu, similarly to how we handle missing firmware?
Ack from my side. I'd also like to remind you about my split-GPU-from display series. > > BR, > -R -- With best wishes Dmitry
