Re: [PATCH v2 2/4] drm/via: add VIA_WAIT_ON()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, 20 Jul 2019 at 09:46, Sam Ravnborg <sam@xxxxxxxxxxxx> wrote:
>
> VIA_WAIT_ON() is a direct copy of DRM_WAIT_ON() from
> drm_os_linux.h.
> The copy is made so we can avoid the dependency on the legacy header.
> A more involved approach had been to introduce wait_event_* but for this
> legacy driver the simpler and more safe approach with a copy of the
> macro was selected.
> Added the relevant header files for the functions used in VIA_WAIT_ON.
>
> Users of the macro will come in a follow-up patch.
>
Since nothing "new" is added here I would change the summary to
"drm/via: copy DRM_WAIT_ON as VIA_WAIT_ON and use it"
IMHO there's little point in splitting introduction and usage.

-Emil
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux