The local #define of ACPI_VIDEO_NOTIFY_PROBE was only added temporarily to avoid a dependency between the acpi and nouveau trees while merging.
This is now properly defined in acpi/video.h and the local #define can be dropped.
Signed-off-by: Hans de Goede hdegoede@redhat.com --- drivers/gpu/drm/nouveau/nouveau_display.c | 9 --------- 1 file changed, 9 deletions(-)
diff --git a/drivers/gpu/drm/nouveau/nouveau_display.c b/drivers/gpu/drm/nouveau/nouveau_display.c index 549763f5e17d..70de2079b2cc 100644 --- a/drivers/gpu/drm/nouveau/nouveau_display.c +++ b/drivers/gpu/drm/nouveau/nouveau_display.c @@ -369,15 +369,6 @@ nouveau_display_hpd_work(struct work_struct *work)
#ifdef CONFIG_ACPI
-/* - * Hans de Goede: This define belongs in acpi/video.h, I've submitted a patch - * to the acpi subsys to move it there from drivers/acpi/acpi_video.c . - * This should be dropped once that is merged. - */ -#ifndef ACPI_VIDEO_NOTIFY_PROBE -#define ACPI_VIDEO_NOTIFY_PROBE 0x81 -#endif - static int nouveau_display_acpi_ntfy(struct notifier_block *nb, unsigned long val, void *data)
Hi,
On 26-05-17 09:35, Hans De Goede wrote:
The local #define of ACPI_VIDEO_NOTIFY_PROBE was only added temporarily to avoid a dependency between the acpi and nouveau trees while merging.
This is now properly defined in acpi/video.h and the local #define can be dropped.
Signed-off-by: Hans de Goede hdegoede@redhat.com
ping, what is the status of this patch ?
Regards,
Hans
drivers/gpu/drm/nouveau/nouveau_display.c | 9 --------- 1 file changed, 9 deletions(-)
diff --git a/drivers/gpu/drm/nouveau/nouveau_display.c b/drivers/gpu/drm/nouveau/nouveau_display.c index 549763f5e17d..70de2079b2cc 100644 --- a/drivers/gpu/drm/nouveau/nouveau_display.c +++ b/drivers/gpu/drm/nouveau/nouveau_display.c @@ -369,15 +369,6 @@ nouveau_display_hpd_work(struct work_struct *work)
#ifdef CONFIG_ACPI
-/*
- Hans de Goede: This define belongs in acpi/video.h, I've submitted a patch
- to the acpi subsys to move it there from drivers/acpi/acpi_video.c .
- This should be dropped once that is merged.
- */
-#ifndef ACPI_VIDEO_NOTIFY_PROBE -#define ACPI_VIDEO_NOTIFY_PROBE 0x81 -#endif
- static int nouveau_display_acpi_ntfy(struct notifier_block *nb, unsigned long val, void *data)
Reviewed-by: Pierre Moreau pierre.morrow@free.fr
On 2017-06-21 — 15:19, Hans de Goede wrote:
Hi,
On 26-05-17 09:35, Hans De Goede wrote:
The local #define of ACPI_VIDEO_NOTIFY_PROBE was only added temporarily to avoid a dependency between the acpi and nouveau trees while merging.
This is now properly defined in acpi/video.h and the local #define can be dropped.
Signed-off-by: Hans de Goede hdegoede@redhat.com
ping, what is the status of this patch ?
Regards,
Hans
drivers/gpu/drm/nouveau/nouveau_display.c | 9 --------- 1 file changed, 9 deletions(-)
diff --git a/drivers/gpu/drm/nouveau/nouveau_display.c b/drivers/gpu/drm/nouveau/nouveau_display.c index 549763f5e17d..70de2079b2cc 100644 --- a/drivers/gpu/drm/nouveau/nouveau_display.c +++ b/drivers/gpu/drm/nouveau/nouveau_display.c @@ -369,15 +369,6 @@ nouveau_display_hpd_work(struct work_struct *work) #ifdef CONFIG_ACPI -/*
- Hans de Goede: This define belongs in acpi/video.h, I've submitted a patch
- to the acpi subsys to move it there from drivers/acpi/acpi_video.c .
- This should be dropped once that is merged.
- */
-#ifndef ACPI_VIDEO_NOTIFY_PROBE -#define ACPI_VIDEO_NOTIFY_PROBE 0x81 -#endif
- static int nouveau_display_acpi_ntfy(struct notifier_block *nb, unsigned long val, void *data)
dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Ping
On 2017-07-22 — 13:54, Pierre Moreau wrote:
Reviewed-by: Pierre Moreau pierre.morrow@free.fr
On 2017-06-21 — 15:19, Hans de Goede wrote:
Hi,
On 26-05-17 09:35, Hans De Goede wrote:
The local #define of ACPI_VIDEO_NOTIFY_PROBE was only added temporarily to avoid a dependency between the acpi and nouveau trees while merging.
This is now properly defined in acpi/video.h and the local #define can be dropped.
Signed-off-by: Hans de Goede hdegoede@redhat.com
ping, what is the status of this patch ?
Regards,
Hans
drivers/gpu/drm/nouveau/nouveau_display.c | 9 --------- 1 file changed, 9 deletions(-)
diff --git a/drivers/gpu/drm/nouveau/nouveau_display.c b/drivers/gpu/drm/nouveau/nouveau_display.c index 549763f5e17d..70de2079b2cc 100644 --- a/drivers/gpu/drm/nouveau/nouveau_display.c +++ b/drivers/gpu/drm/nouveau/nouveau_display.c @@ -369,15 +369,6 @@ nouveau_display_hpd_work(struct work_struct *work) #ifdef CONFIG_ACPI -/*
- Hans de Goede: This define belongs in acpi/video.h, I've submitted a patch
- to the acpi subsys to move it there from drivers/acpi/acpi_video.c .
- This should be dropped once that is merged.
- */
-#ifndef ACPI_VIDEO_NOTIFY_PROBE -#define ACPI_VIDEO_NOTIFY_PROBE 0x81 -#endif
- static int nouveau_display_acpi_ntfy(struct notifier_block *nb, unsigned long val, void *data)
dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau
dri-devel@lists.freedesktop.org
-
Hans de Goede -
Pierre Moreau