[PATCH v4 08/15] drm/i915/display: remove the display platform enum as unnecessary
Rodrigo Vivi
rodrigo.vivi at intel.com
Wed Oct 23 14:40:50 UTC 2024
On Wed, Oct 23, 2024 at 04:09:03PM +0300, Jani Nikula wrote:
> 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.
oh, okay... I was in doubt if pciidlist was the right check to ensure
same condition. But taking a deeper look on it here again it indeed sounds
the case.
Reviewed-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
>
> BR,
> Jani.
>
>
> --
> Jani Nikula, Intel
More information about the Intel-xe
mailing list