* [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h
@ 2024-01-15 9:24 ` Alex Bee
0 siblings, 0 replies; 20+ messages in thread
From: Alex Bee @ 2024-01-15 9:24 UTC (permalink / raw)
To: Sandy Huang, Heiko Stübner, Andy Yan, Maarten Lankhorst,
Maxime Ripard, Thomas Zimmermann
Cc: David Airlie, Daniel Vetter, Alex Bee, dri-devel,
linux-arm-kernel, linux-rockchip, linux-kernel, kernel test robot
Commit d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set")
started using drm_atomic_get_new_connector_state and
drm_atomic_get_new_crtc_state which are defined in drm_atomic.h
Building does currently only work if CONFIG_OF and CONFIG_DRM_PANEL_BRIDGE
are enabled since this will include drm_atomic.h via drm_bridge.h (see
drm_of.h).
Explicitly include drm_atomic.h in inno_hdmi.c to fix this.
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202401100949.ZVRr0pIa-lkp@intel.com/
Closes: https://lore.kernel.org/oe-kbuild-all/202401081720.UtgAZgba-lkp@intel.com/
Fixes: d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set")
Signed-off-by: Alex Bee <knaerzche@gmail.com>
---
drivers/gpu/drm/rockchip/inno_hdmi.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/rockchip/inno_hdmi.c b/drivers/gpu/drm/rockchip/inno_hdmi.c
index 925320fef0a8..3bb8f1382612 100644
--- a/drivers/gpu/drm/rockchip/inno_hdmi.c
+++ b/drivers/gpu/drm/rockchip/inno_hdmi.c
@@ -15,6 +15,7 @@
#include <linux/mutex.h>
#include <linux/platform_device.h>
+#include <drm/drm_atomic.h>
#include <drm/drm_atomic_helper.h>
#include <drm/drm_edid.h>
#include <drm/drm_of.h>
--
2.43.0
_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h @ 2024-01-15 9:24 ` Alex Bee 0 siblings, 0 replies; 20+ messages in thread From: Alex Bee @ 2024-01-15 9:24 UTC (permalink / raw) To: Sandy Huang, Heiko Stübner, Andy Yan, Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann Cc: David Airlie, Daniel Vetter, Alex Bee, dri-devel, linux-arm-kernel, linux-rockchip, linux-kernel, kernel test robot Commit d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") started using drm_atomic_get_new_connector_state and drm_atomic_get_new_crtc_state which are defined in drm_atomic.h Building does currently only work if CONFIG_OF and CONFIG_DRM_PANEL_BRIDGE are enabled since this will include drm_atomic.h via drm_bridge.h (see drm_of.h). Explicitly include drm_atomic.h in inno_hdmi.c to fix this. Reported-by: kernel test robot <lkp@intel.com> Closes: https://lore.kernel.org/oe-kbuild-all/202401100949.ZVRr0pIa-lkp@intel.com/ Closes: https://lore.kernel.org/oe-kbuild-all/202401081720.UtgAZgba-lkp@intel.com/ Fixes: d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") Signed-off-by: Alex Bee <knaerzche@gmail.com> --- drivers/gpu/drm/rockchip/inno_hdmi.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/rockchip/inno_hdmi.c b/drivers/gpu/drm/rockchip/inno_hdmi.c index 925320fef0a8..3bb8f1382612 100644 --- a/drivers/gpu/drm/rockchip/inno_hdmi.c +++ b/drivers/gpu/drm/rockchip/inno_hdmi.c @@ -15,6 +15,7 @@ #include <linux/mutex.h> #include <linux/platform_device.h> +#include <drm/drm_atomic.h> #include <drm/drm_atomic_helper.h> #include <drm/drm_edid.h> #include <drm/drm_of.h> -- 2.43.0 ^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h @ 2024-01-15 9:24 ` Alex Bee 0 siblings, 0 replies; 20+ messages in thread From: Alex Bee @ 2024-01-15 9:24 UTC (permalink / raw) To: Sandy Huang, Heiko Stübner, Andy Yan, Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann Cc: kernel test robot, Alex Bee, dri-devel, linux-kernel, linux-rockchip, linux-arm-kernel Commit d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") started using drm_atomic_get_new_connector_state and drm_atomic_get_new_crtc_state which are defined in drm_atomic.h Building does currently only work if CONFIG_OF and CONFIG_DRM_PANEL_BRIDGE are enabled since this will include drm_atomic.h via drm_bridge.h (see drm_of.h). Explicitly include drm_atomic.h in inno_hdmi.c to fix this. Reported-by: kernel test robot <lkp@intel.com> Closes: https://lore.kernel.org/oe-kbuild-all/202401100949.ZVRr0pIa-lkp@intel.com/ Closes: https://lore.kernel.org/oe-kbuild-all/202401081720.UtgAZgba-lkp@intel.com/ Fixes: d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") Signed-off-by: Alex Bee <knaerzche@gmail.com> --- drivers/gpu/drm/rockchip/inno_hdmi.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/rockchip/inno_hdmi.c b/drivers/gpu/drm/rockchip/inno_hdmi.c index 925320fef0a8..3bb8f1382612 100644 --- a/drivers/gpu/drm/rockchip/inno_hdmi.c +++ b/drivers/gpu/drm/rockchip/inno_hdmi.c @@ -15,6 +15,7 @@ #include <linux/mutex.h> #include <linux/platform_device.h> +#include <drm/drm_atomic.h> #include <drm/drm_atomic_helper.h> #include <drm/drm_edid.h> #include <drm/drm_of.h> -- 2.43.0 ^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h @ 2024-01-15 9:24 ` Alex Bee 0 siblings, 0 replies; 20+ messages in thread From: Alex Bee @ 2024-01-15 9:24 UTC (permalink / raw) To: Sandy Huang, Heiko Stübner, Andy Yan, Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann Cc: David Airlie, Daniel Vetter, Alex Bee, dri-devel, linux-arm-kernel, linux-rockchip, linux-kernel, kernel test robot Commit d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") started using drm_atomic_get_new_connector_state and drm_atomic_get_new_crtc_state which are defined in drm_atomic.h Building does currently only work if CONFIG_OF and CONFIG_DRM_PANEL_BRIDGE are enabled since this will include drm_atomic.h via drm_bridge.h (see drm_of.h). Explicitly include drm_atomic.h in inno_hdmi.c to fix this. Reported-by: kernel test robot <lkp@intel.com> Closes: https://lore.kernel.org/oe-kbuild-all/202401100949.ZVRr0pIa-lkp@intel.com/ Closes: https://lore.kernel.org/oe-kbuild-all/202401081720.UtgAZgba-lkp@intel.com/ Fixes: d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") Signed-off-by: Alex Bee <knaerzche@gmail.com> --- drivers/gpu/drm/rockchip/inno_hdmi.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/rockchip/inno_hdmi.c b/drivers/gpu/drm/rockchip/inno_hdmi.c index 925320fef0a8..3bb8f1382612 100644 --- a/drivers/gpu/drm/rockchip/inno_hdmi.c +++ b/drivers/gpu/drm/rockchip/inno_hdmi.c @@ -15,6 +15,7 @@ #include <linux/mutex.h> #include <linux/platform_device.h> +#include <drm/drm_atomic.h> #include <drm/drm_atomic_helper.h> #include <drm/drm_edid.h> #include <drm/drm_of.h> -- 2.43.0 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 20+ messages in thread
* Re: (subset) [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h 2024-01-15 9:24 ` Alex Bee (?) (?) @ 2024-01-17 9:46 ` Maxime Ripard -1 siblings, 0 replies; 20+ messages in thread From: Maxime Ripard @ 2024-01-17 9:46 UTC (permalink / raw) To: Sandy Huang, Heiko Stübner, Andy Yan, Maarten Lankhorst, Thomas Zimmermann, Alex Bee Cc: David Airlie, Daniel Vetter, dri-devel, linux-arm-kernel, linux-rockchip, linux-kernel, kernel test robot On Mon, 15 Jan 2024 10:24:35 +0100, Alex Bee wrote: > Commit d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") > started using drm_atomic_get_new_connector_state and > drm_atomic_get_new_crtc_state which are defined in drm_atomic.h > Building does currently only work if CONFIG_OF and CONFIG_DRM_PANEL_BRIDGE > are enabled since this will include drm_atomic.h via drm_bridge.h (see > drm_of.h). > > [...] Applied to drm/drm-misc (drm-misc-next). Thanks! Maxime _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: (subset) [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h @ 2024-01-17 9:46 ` Maxime Ripard 0 siblings, 0 replies; 20+ messages in thread From: Maxime Ripard @ 2024-01-17 9:46 UTC (permalink / raw) To: Sandy Huang, Heiko Stübner, Andy Yan, Maarten Lankhorst, Thomas Zimmermann, Alex Bee Cc: David Airlie, Daniel Vetter, dri-devel, linux-arm-kernel, linux-rockchip, linux-kernel, kernel test robot On Mon, 15 Jan 2024 10:24:35 +0100, Alex Bee wrote: > Commit d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") > started using drm_atomic_get_new_connector_state and > drm_atomic_get_new_crtc_state which are defined in drm_atomic.h > Building does currently only work if CONFIG_OF and CONFIG_DRM_PANEL_BRIDGE > are enabled since this will include drm_atomic.h via drm_bridge.h (see > drm_of.h). > > [...] Applied to drm/drm-misc (drm-misc-next). Thanks! Maxime ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: (subset) [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h @ 2024-01-17 9:46 ` Maxime Ripard 0 siblings, 0 replies; 20+ messages in thread From: Maxime Ripard @ 2024-01-17 9:46 UTC (permalink / raw) To: Sandy Huang, Heiko Stübner, Andy Yan, Maarten Lankhorst, Thomas Zimmermann, Alex Bee Cc: kernel test robot, linux-kernel, dri-devel, linux-rockchip, Daniel Vetter, David Airlie, linux-arm-kernel On Mon, 15 Jan 2024 10:24:35 +0100, Alex Bee wrote: > Commit d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") > started using drm_atomic_get_new_connector_state and > drm_atomic_get_new_crtc_state which are defined in drm_atomic.h > Building does currently only work if CONFIG_OF and CONFIG_DRM_PANEL_BRIDGE > are enabled since this will include drm_atomic.h via drm_bridge.h (see > drm_of.h). > > [...] Applied to drm/drm-misc (drm-misc-next). Thanks! Maxime ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: (subset) [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h @ 2024-01-17 9:46 ` Maxime Ripard 0 siblings, 0 replies; 20+ messages in thread From: Maxime Ripard @ 2024-01-17 9:46 UTC (permalink / raw) To: Sandy Huang, Heiko Stübner, Andy Yan, Maarten Lankhorst, Thomas Zimmermann, Alex Bee Cc: David Airlie, Daniel Vetter, dri-devel, linux-arm-kernel, linux-rockchip, linux-kernel, kernel test robot On Mon, 15 Jan 2024 10:24:35 +0100, Alex Bee wrote: > Commit d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") > started using drm_atomic_get_new_connector_state and > drm_atomic_get_new_crtc_state which are defined in drm_atomic.h > Building does currently only work if CONFIG_OF and CONFIG_DRM_PANEL_BRIDGE > are enabled since this will include drm_atomic.h via drm_bridge.h (see > drm_of.h). > > [...] Applied to drm/drm-misc (drm-misc-next). Thanks! Maxime _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: (subset) [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h 2024-01-17 9:46 ` Maxime Ripard (?) (?) @ 2024-01-17 9:52 ` Heiko Stübner -1 siblings, 0 replies; 20+ messages in thread From: Heiko Stübner @ 2024-01-17 9:52 UTC (permalink / raw) To: Sandy Huang, Andy Yan, Maarten Lankhorst, Thomas Zimmermann, Alex Bee, Maxime Ripard Cc: David Airlie, Daniel Vetter, dri-devel, linux-arm-kernel, linux-rockchip, linux-kernel, kernel test robot Hi Maxime, Am Mittwoch, 17. Januar 2024, 10:46:57 CET schrieb Maxime Ripard: > On Mon, 15 Jan 2024 10:24:35 +0100, Alex Bee wrote: > > Commit d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") > > started using drm_atomic_get_new_connector_state and > > drm_atomic_get_new_crtc_state which are defined in drm_atomic.h > > Building does currently only work if CONFIG_OF and CONFIG_DRM_PANEL_BRIDGE > > are enabled since this will include drm_atomic.h via drm_bridge.h (see > > drm_of.h). > > > > [...] > > Applied to drm/drm-misc (drm-misc-next). wouldn't have drm-misc-next-fixes been more appropriate? Because I _think_ the change causing the issue made it in during the current merge-window? Heiko _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: (subset) [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h @ 2024-01-17 9:52 ` Heiko Stübner 0 siblings, 0 replies; 20+ messages in thread From: Heiko Stübner @ 2024-01-17 9:52 UTC (permalink / raw) To: Sandy Huang, Andy Yan, Maarten Lankhorst, Thomas Zimmermann, Alex Bee, Maxime Ripard Cc: David Airlie, Daniel Vetter, dri-devel, linux-arm-kernel, linux-rockchip, linux-kernel, kernel test robot Hi Maxime, Am Mittwoch, 17. Januar 2024, 10:46:57 CET schrieb Maxime Ripard: > On Mon, 15 Jan 2024 10:24:35 +0100, Alex Bee wrote: > > Commit d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") > > started using drm_atomic_get_new_connector_state and > > drm_atomic_get_new_crtc_state which are defined in drm_atomic.h > > Building does currently only work if CONFIG_OF and CONFIG_DRM_PANEL_BRIDGE > > are enabled since this will include drm_atomic.h via drm_bridge.h (see > > drm_of.h). > > > > [...] > > Applied to drm/drm-misc (drm-misc-next). wouldn't have drm-misc-next-fixes been more appropriate? Because I _think_ the change causing the issue made it in during the current merge-window? Heiko ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: (subset) [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h @ 2024-01-17 9:52 ` Heiko Stübner 0 siblings, 0 replies; 20+ messages in thread From: Heiko Stübner @ 2024-01-17 9:52 UTC (permalink / raw) To: Sandy Huang, Andy Yan, Maarten Lankhorst, Thomas Zimmermann, Alex Bee, Maxime Ripard Cc: kernel test robot, linux-kernel, dri-devel, linux-rockchip, Daniel Vetter, David Airlie, linux-arm-kernel Hi Maxime, Am Mittwoch, 17. Januar 2024, 10:46:57 CET schrieb Maxime Ripard: > On Mon, 15 Jan 2024 10:24:35 +0100, Alex Bee wrote: > > Commit d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") > > started using drm_atomic_get_new_connector_state and > > drm_atomic_get_new_crtc_state which are defined in drm_atomic.h > > Building does currently only work if CONFIG_OF and CONFIG_DRM_PANEL_BRIDGE > > are enabled since this will include drm_atomic.h via drm_bridge.h (see > > drm_of.h). > > > > [...] > > Applied to drm/drm-misc (drm-misc-next). wouldn't have drm-misc-next-fixes been more appropriate? Because I _think_ the change causing the issue made it in during the current merge-window? Heiko ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: (subset) [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h @ 2024-01-17 9:52 ` Heiko Stübner 0 siblings, 0 replies; 20+ messages in thread From: Heiko Stübner @ 2024-01-17 9:52 UTC (permalink / raw) To: Sandy Huang, Andy Yan, Maarten Lankhorst, Thomas Zimmermann, Alex Bee, Maxime Ripard Cc: David Airlie, Daniel Vetter, dri-devel, linux-arm-kernel, linux-rockchip, linux-kernel, kernel test robot Hi Maxime, Am Mittwoch, 17. Januar 2024, 10:46:57 CET schrieb Maxime Ripard: > On Mon, 15 Jan 2024 10:24:35 +0100, Alex Bee wrote: > > Commit d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") > > started using drm_atomic_get_new_connector_state and > > drm_atomic_get_new_crtc_state which are defined in drm_atomic.h > > Building does currently only work if CONFIG_OF and CONFIG_DRM_PANEL_BRIDGE > > are enabled since this will include drm_atomic.h via drm_bridge.h (see > > drm_of.h). > > > > [...] > > Applied to drm/drm-misc (drm-misc-next). wouldn't have drm-misc-next-fixes been more appropriate? Because I _think_ the change causing the issue made it in during the current merge-window? Heiko _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: Re: (subset) [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h 2024-01-17 9:52 ` Heiko Stübner (?) (?) @ 2024-01-17 13:47 ` Maxime Ripard -1 siblings, 0 replies; 20+ messages in thread From: Maxime Ripard @ 2024-01-17 13:47 UTC (permalink / raw) To: Heiko Stübner Cc: Sandy Huang, Andy Yan, Maarten Lankhorst, Thomas Zimmermann, Alex Bee, David Airlie, Daniel Vetter, dri-devel, linux-arm-kernel, linux-rockchip, linux-kernel, kernel test robot [-- Attachment #1.1: Type: text/plain, Size: 893 bytes --] On Wed, Jan 17, 2024 at 10:52:04AM +0100, Heiko Stübner wrote: > Hi Maxime, > > Am Mittwoch, 17. Januar 2024, 10:46:57 CET schrieb Maxime Ripard: > > On Mon, 15 Jan 2024 10:24:35 +0100, Alex Bee wrote: > > > Commit d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") > > > started using drm_atomic_get_new_connector_state and > > > drm_atomic_get_new_crtc_state which are defined in drm_atomic.h > > > Building does currently only work if CONFIG_OF and CONFIG_DRM_PANEL_BRIDGE > > > are enabled since this will include drm_atomic.h via drm_bridge.h (see > > > drm_of.h). > > > > > > [...] > > > > Applied to drm/drm-misc (drm-misc-next). > > wouldn't have drm-misc-next-fixes been more appropriate? > Because I _think_ the change causing the issue made it in during the > current merge-window? AFAIK, the offending commit is in drm-misc-next only Maxime [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] [-- Attachment #2: Type: text/plain, Size: 170 bytes --] _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: Re: (subset) [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h @ 2024-01-17 13:47 ` Maxime Ripard 0 siblings, 0 replies; 20+ messages in thread From: Maxime Ripard @ 2024-01-17 13:47 UTC (permalink / raw) To: Heiko Stübner Cc: Sandy Huang, Andy Yan, Maarten Lankhorst, Thomas Zimmermann, Alex Bee, David Airlie, Daniel Vetter, dri-devel, linux-arm-kernel, linux-rockchip, linux-kernel, kernel test robot [-- Attachment #1: Type: text/plain, Size: 893 bytes --] On Wed, Jan 17, 2024 at 10:52:04AM +0100, Heiko Stübner wrote: > Hi Maxime, > > Am Mittwoch, 17. Januar 2024, 10:46:57 CET schrieb Maxime Ripard: > > On Mon, 15 Jan 2024 10:24:35 +0100, Alex Bee wrote: > > > Commit d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") > > > started using drm_atomic_get_new_connector_state and > > > drm_atomic_get_new_crtc_state which are defined in drm_atomic.h > > > Building does currently only work if CONFIG_OF and CONFIG_DRM_PANEL_BRIDGE > > > are enabled since this will include drm_atomic.h via drm_bridge.h (see > > > drm_of.h). > > > > > > [...] > > > > Applied to drm/drm-misc (drm-misc-next). > > wouldn't have drm-misc-next-fixes been more appropriate? > Because I _think_ the change causing the issue made it in during the > current merge-window? AFAIK, the offending commit is in drm-misc-next only Maxime [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: Re: (subset) [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h @ 2024-01-17 13:47 ` Maxime Ripard 0 siblings, 0 replies; 20+ messages in thread From: Maxime Ripard @ 2024-01-17 13:47 UTC (permalink / raw) To: Heiko Stübner Cc: Daniel Vetter, linux-kernel, Sandy Huang, dri-devel, Alex Bee, linux-rockchip, Thomas Zimmermann, Andy Yan, David Airlie, linux-arm-kernel, kernel test robot [-- Attachment #1: Type: text/plain, Size: 893 bytes --] On Wed, Jan 17, 2024 at 10:52:04AM +0100, Heiko Stübner wrote: > Hi Maxime, > > Am Mittwoch, 17. Januar 2024, 10:46:57 CET schrieb Maxime Ripard: > > On Mon, 15 Jan 2024 10:24:35 +0100, Alex Bee wrote: > > > Commit d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") > > > started using drm_atomic_get_new_connector_state and > > > drm_atomic_get_new_crtc_state which are defined in drm_atomic.h > > > Building does currently only work if CONFIG_OF and CONFIG_DRM_PANEL_BRIDGE > > > are enabled since this will include drm_atomic.h via drm_bridge.h (see > > > drm_of.h). > > > > > > [...] > > > > Applied to drm/drm-misc (drm-misc-next). > > wouldn't have drm-misc-next-fixes been more appropriate? > Because I _think_ the change causing the issue made it in during the > current merge-window? AFAIK, the offending commit is in drm-misc-next only Maxime [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: Re: (subset) [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h @ 2024-01-17 13:47 ` Maxime Ripard 0 siblings, 0 replies; 20+ messages in thread From: Maxime Ripard @ 2024-01-17 13:47 UTC (permalink / raw) To: Heiko Stübner Cc: Sandy Huang, Andy Yan, Maarten Lankhorst, Thomas Zimmermann, Alex Bee, David Airlie, Daniel Vetter, dri-devel, linux-arm-kernel, linux-rockchip, linux-kernel, kernel test robot [-- Attachment #1.1: Type: text/plain, Size: 893 bytes --] On Wed, Jan 17, 2024 at 10:52:04AM +0100, Heiko Stübner wrote: > Hi Maxime, > > Am Mittwoch, 17. Januar 2024, 10:46:57 CET schrieb Maxime Ripard: > > On Mon, 15 Jan 2024 10:24:35 +0100, Alex Bee wrote: > > > Commit d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") > > > started using drm_atomic_get_new_connector_state and > > > drm_atomic_get_new_crtc_state which are defined in drm_atomic.h > > > Building does currently only work if CONFIG_OF and CONFIG_DRM_PANEL_BRIDGE > > > are enabled since this will include drm_atomic.h via drm_bridge.h (see > > > drm_of.h). > > > > > > [...] > > > > Applied to drm/drm-misc (drm-misc-next). > > wouldn't have drm-misc-next-fixes been more appropriate? > Because I _think_ the change causing the issue made it in during the > current merge-window? AFAIK, the offending commit is in drm-misc-next only Maxime [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] [-- Attachment #2: Type: text/plain, Size: 176 bytes --] _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: (subset) [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h 2024-01-17 13:47 ` Maxime Ripard (?) (?) @ 2024-01-17 14:22 ` Heiko Stübner -1 siblings, 0 replies; 20+ messages in thread From: Heiko Stübner @ 2024-01-17 14:22 UTC (permalink / raw) To: Maxime Ripard Cc: Sandy Huang, Andy Yan, Maarten Lankhorst, Thomas Zimmermann, Alex Bee, David Airlie, Daniel Vetter, dri-devel, linux-arm-kernel, linux-rockchip, linux-kernel, kernel test robot Am Mittwoch, 17. Januar 2024, 14:47:48 CET schrieb Maxime Ripard: > On Wed, Jan 17, 2024 at 10:52:04AM +0100, Heiko Stübner wrote: > > Hi Maxime, > > > > Am Mittwoch, 17. Januar 2024, 10:46:57 CET schrieb Maxime Ripard: > > > On Mon, 15 Jan 2024 10:24:35 +0100, Alex Bee wrote: > > > > Commit d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") > > > > started using drm_atomic_get_new_connector_state and > > > > drm_atomic_get_new_crtc_state which are defined in drm_atomic.h > > > > Building does currently only work if CONFIG_OF and CONFIG_DRM_PANEL_BRIDGE > > > > are enabled since this will include drm_atomic.h via drm_bridge.h (see > > > > drm_of.h). > > > > > > > > [...] > > > > > > Applied to drm/drm-misc (drm-misc-next). > > > > wouldn't have drm-misc-next-fixes been more appropriate? > > Because I _think_ the change causing the issue made it in during the > > current merge-window? > > AFAIK, the offending commit is in drm-misc-next only ah, you're of course right. Mistook that for the one in the rk3066_hdmi but that was fixed back in november already. So all is well. Heiko _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: (subset) [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h @ 2024-01-17 14:22 ` Heiko Stübner 0 siblings, 0 replies; 20+ messages in thread From: Heiko Stübner @ 2024-01-17 14:22 UTC (permalink / raw) To: Maxime Ripard Cc: Sandy Huang, Andy Yan, Maarten Lankhorst, Thomas Zimmermann, Alex Bee, David Airlie, Daniel Vetter, dri-devel, linux-arm-kernel, linux-rockchip, linux-kernel, kernel test robot Am Mittwoch, 17. Januar 2024, 14:47:48 CET schrieb Maxime Ripard: > On Wed, Jan 17, 2024 at 10:52:04AM +0100, Heiko Stübner wrote: > > Hi Maxime, > > > > Am Mittwoch, 17. Januar 2024, 10:46:57 CET schrieb Maxime Ripard: > > > On Mon, 15 Jan 2024 10:24:35 +0100, Alex Bee wrote: > > > > Commit d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") > > > > started using drm_atomic_get_new_connector_state and > > > > drm_atomic_get_new_crtc_state which are defined in drm_atomic.h > > > > Building does currently only work if CONFIG_OF and CONFIG_DRM_PANEL_BRIDGE > > > > are enabled since this will include drm_atomic.h via drm_bridge.h (see > > > > drm_of.h). > > > > > > > > [...] > > > > > > Applied to drm/drm-misc (drm-misc-next). > > > > wouldn't have drm-misc-next-fixes been more appropriate? > > Because I _think_ the change causing the issue made it in during the > > current merge-window? > > AFAIK, the offending commit is in drm-misc-next only ah, you're of course right. Mistook that for the one in the rk3066_hdmi but that was fixed back in november already. So all is well. Heiko ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: (subset) [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h @ 2024-01-17 14:22 ` Heiko Stübner 0 siblings, 0 replies; 20+ messages in thread From: Heiko Stübner @ 2024-01-17 14:22 UTC (permalink / raw) To: Maxime Ripard Cc: Daniel Vetter, linux-kernel, Sandy Huang, dri-devel, Alex Bee, linux-rockchip, Thomas Zimmermann, Andy Yan, David Airlie, linux-arm-kernel, kernel test robot Am Mittwoch, 17. Januar 2024, 14:47:48 CET schrieb Maxime Ripard: > On Wed, Jan 17, 2024 at 10:52:04AM +0100, Heiko Stübner wrote: > > Hi Maxime, > > > > Am Mittwoch, 17. Januar 2024, 10:46:57 CET schrieb Maxime Ripard: > > > On Mon, 15 Jan 2024 10:24:35 +0100, Alex Bee wrote: > > > > Commit d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") > > > > started using drm_atomic_get_new_connector_state and > > > > drm_atomic_get_new_crtc_state which are defined in drm_atomic.h > > > > Building does currently only work if CONFIG_OF and CONFIG_DRM_PANEL_BRIDGE > > > > are enabled since this will include drm_atomic.h via drm_bridge.h (see > > > > drm_of.h). > > > > > > > > [...] > > > > > > Applied to drm/drm-misc (drm-misc-next). > > > > wouldn't have drm-misc-next-fixes been more appropriate? > > Because I _think_ the change causing the issue made it in during the > > current merge-window? > > AFAIK, the offending commit is in drm-misc-next only ah, you're of course right. Mistook that for the one in the rk3066_hdmi but that was fixed back in november already. So all is well. Heiko ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: (subset) [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h @ 2024-01-17 14:22 ` Heiko Stübner 0 siblings, 0 replies; 20+ messages in thread From: Heiko Stübner @ 2024-01-17 14:22 UTC (permalink / raw) To: Maxime Ripard Cc: Sandy Huang, Andy Yan, Maarten Lankhorst, Thomas Zimmermann, Alex Bee, David Airlie, Daniel Vetter, dri-devel, linux-arm-kernel, linux-rockchip, linux-kernel, kernel test robot Am Mittwoch, 17. Januar 2024, 14:47:48 CET schrieb Maxime Ripard: > On Wed, Jan 17, 2024 at 10:52:04AM +0100, Heiko Stübner wrote: > > Hi Maxime, > > > > Am Mittwoch, 17. Januar 2024, 10:46:57 CET schrieb Maxime Ripard: > > > On Mon, 15 Jan 2024 10:24:35 +0100, Alex Bee wrote: > > > > Commit d3e040f450ec ("drm/rockchip: inno_hdmi: Get rid of mode_set") > > > > started using drm_atomic_get_new_connector_state and > > > > drm_atomic_get_new_crtc_state which are defined in drm_atomic.h > > > > Building does currently only work if CONFIG_OF and CONFIG_DRM_PANEL_BRIDGE > > > > are enabled since this will include drm_atomic.h via drm_bridge.h (see > > > > drm_of.h). > > > > > > > > [...] > > > > > > Applied to drm/drm-misc (drm-misc-next). > > > > wouldn't have drm-misc-next-fixes been more appropriate? > > Because I _think_ the change causing the issue made it in during the > > current merge-window? > > AFAIK, the offending commit is in drm-misc-next only ah, you're of course right. Mistook that for the one in the rk3066_hdmi but that was fixed back in november already. So all is well. Heiko _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 20+ messages in thread
end of thread, other threads:[~2024-01-17 14:23 UTC | newest] Thread overview: 20+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-01-15 9:24 [PATCH] drm/rockchip: inno_hdmi: Explicitly include drm_atomic.h Alex Bee 2024-01-15 9:24 ` Alex Bee 2024-01-15 9:24 ` Alex Bee 2024-01-15 9:24 ` Alex Bee 2024-01-17 9:46 ` (subset) " Maxime Ripard 2024-01-17 9:46 ` Maxime Ripard 2024-01-17 9:46 ` Maxime Ripard 2024-01-17 9:46 ` Maxime Ripard 2024-01-17 9:52 ` Heiko Stübner 2024-01-17 9:52 ` Heiko Stübner 2024-01-17 9:52 ` Heiko Stübner 2024-01-17 9:52 ` Heiko Stübner 2024-01-17 13:47 ` Maxime Ripard 2024-01-17 13:47 ` Maxime Ripard 2024-01-17 13:47 ` Maxime Ripard 2024-01-17 13:47 ` Maxime Ripard 2024-01-17 14:22 ` Heiko Stübner 2024-01-17 14:22 ` Heiko Stübner 2024-01-17 14:22 ` Heiko Stübner 2024-01-17 14:22 ` Heiko Stübner
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.