[PATCH v4 08/15] drm/i915/display: remove the display platform enum as unnecessary

Jani Nikula jani.nikula at intel.com
Wed Oct 23 13:09:03 UTC 2024


On Tue, 22 Oct 2024, Rodrigo Vivi <rodrigo.vivi at intel.com> wrote:
> On Mon, Oct 21, 2024 at 04:54:09PM +0300, Jani Nikula wrote:
>> @@ -1466,7 +1462,7 @@ find_subplatform_desc(struct pci_dev *pdev, const struct platform_desc *desc)
>>  	const struct subplatform_desc *sp;
>>  	const u16 *id;
>>  
>> -	for (sp = desc->subplatforms; sp && sp->subplatform; sp++)
>> +	for (sp = desc->subplatforms; sp && sp->pciidlist; sp++)
>
> I honestly got confused on this... should it be a separate patch?

Well, I'm removing ->subplatform so obviously I have to check for the
end condition using something else. I can make it a separate patch if
you want.

BR,
Jani.


-- 
Jani Nikula, Intel


More information about the Intel-gfx mailing list