Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-07-22 Thread Doug Anderson
Hi, On Mon, Jul 15, 2024 at 9:40 AM Doug Anderson wrote: > > Hi, > > On Fri, Jun 21, 2024 at 1:46 PM Doug Anderson wrote: > > > > Hi, > > > > On Fri, Jun 21, 2024 at 1:45 PM Douglas Anderson > > wrote: > > > > > > At shutdown if you've got a _properly_ coded DRM modeset driver then > > > you'l

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-07-16 Thread Linus Walleij
On Fri, Jun 21, 2024 at 10:45 PM Douglas Anderson wrote: > At shutdown if you've got a _properly_ coded DRM modeset driver then > you'll get these two warnings at shutdown time: > > Skipping disable of already disabled panel > Skipping unprepare of already unprepared panel > > These warnings

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-07-16 Thread Neil Armstrong
On 21/06/2024 22:44, Douglas Anderson wrote: At shutdown if you've got a _properly_ coded DRM modeset driver then you'll get these two warnings at shutdown time: Skipping disable of already disabled panel Skipping unprepare of already unprepared panel These warnings are ugly and sound con

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-07-15 Thread Doug Anderson
Hi, On Fri, Jun 21, 2024 at 1:46 PM Doug Anderson wrote: > > Hi, > > On Fri, Jun 21, 2024 at 1:45 PM Douglas Anderson > wrote: > > > > At shutdown if you've got a _properly_ coded DRM modeset driver then > > you'll get these two warnings at shutdown time: > > > > Skipping disable of already d

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-21 Thread Doug Anderson
Hi, On Fri, Jun 21, 2024 at 1:45 PM Douglas Anderson wrote: > > At shutdown if you've got a _properly_ coded DRM modeset driver then > you'll get these two warnings at shutdown time: > > Skipping disable of already disabled panel > Skipping unprepare of already unprepared panel > > These warn

[PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-21 Thread Douglas Anderson
At shutdown if you've got a _properly_ coded DRM modeset driver then you'll get these two warnings at shutdown time: Skipping disable of already disabled panel Skipping unprepare of already unprepared panel These warnings are ugly and sound concerning, but they're actually a sign of a properl

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-21 Thread Daniel Vetter
On Tue, Jun 18, 2024 at 04:49:32PM -0700, Doug Anderson wrote: > Hi, > > On Mon, Jun 17, 2024 at 7:22 AM Daniel Vetter wrote: > > > > > I'm really not convinced that hacking with device links in order to > > > get the shutdown notification in the right order is correct, though. > > > The idea is

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-21 Thread Daniel Vetter
On Tue, Jun 18, 2024 at 04:49:22PM -0700, Doug Anderson wrote: > Hi, > > On Mon, Jun 17, 2024 at 7:17 AM Daniel Vetter wrote: > > > > > That all being said, I'm also totally OK with any of the following: > > > > > > 1. Dropping my patch and just accepting that we will have warnings > > > printed

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-18 Thread Doug Anderson
Hi, On Mon, Jun 17, 2024 at 7:22 AM Daniel Vetter wrote: > > > I'm really not convinced that hacking with device links in order to > > get the shutdown notification in the right order is correct, though. > > The idea is that after we're confident that all DRM modeset drivers > > are calling shutd

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-18 Thread Doug Anderson
Hi, On Mon, Jun 17, 2024 at 7:17 AM Daniel Vetter wrote: > > > That all being said, I'm also totally OK with any of the following: > > > > 1. Dropping my patch and just accepting that we will have warnings > > printed out for all DRM drivers that do things correctly and have no > > warnings for b

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-17 Thread Daniel Vetter
On Wed, Jun 12, 2024 at 10:22:49AM -0700, Doug Anderson wrote: > Hi, > > On Wed, Jun 12, 2024 at 9:47 AM Linus Walleij > wrote: > > > > On Wed, Jun 12, 2024 at 5:11 PM Daniel Vetter wrote: > > > On Wed, Jun 12, 2024 at 07:49:31AM -0700, Doug Anderson wrote: > > (...) > > > > The problem is that

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-17 Thread Daniel Vetter
On Wed, Jun 12, 2024 at 09:00:29AM -0700, Doug Anderson wrote: > Hi, > > On Wed, Jun 12, 2024 at 8:11 AM Daniel Vetter wrote: > > > > > The problem is that the ordering is wrong, I think. Even if the OS was > > > calling driver shutdown functions in the perfect order (which I'm not > > > convince

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-17 Thread Daniel Vetter
On Wed, Jun 12, 2024 at 09:52:40AM -0700, Doug Anderson wrote: > Sima, > > On Wed, Jun 12, 2024 at 8:13 AM Daniel Vetter wrote: > > > > > > I ran the coccinelle script we started with, and here are the results: > > > > > > > > ./drivers/gpu/drm/vmwgfx/vmwgfx_drv.c:1640:25-39: ERROR: KMS driver >

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-12 Thread Doug Anderson
Hi, On Wed, Jun 12, 2024 at 9:47 AM Linus Walleij wrote: > > On Wed, Jun 12, 2024 at 5:11 PM Daniel Vetter wrote: > > On Wed, Jun 12, 2024 at 07:49:31AM -0700, Doug Anderson wrote: > (...) > > > The problem is that the ordering is wrong, I think. Even if the OS was > > > calling driver shutdown

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-12 Thread Doug Anderson
Sima, On Wed, Jun 12, 2024 at 8:13 AM Daniel Vetter wrote: > > > > I ran the coccinelle script we started with, and here are the results: > > > > > > ./drivers/gpu/drm/vmwgfx/vmwgfx_drv.c:1640:25-39: ERROR: KMS driver > > > vmw_pci_driver is missing shutdown implementation > > > ./drivers/gpu/dr

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-12 Thread Linus Walleij
On Wed, Jun 12, 2024 at 5:11 PM Daniel Vetter wrote: > On Wed, Jun 12, 2024 at 07:49:31AM -0700, Doug Anderson wrote: (...) > > The problem is that the ordering is wrong, I think. Even if the OS was > > calling driver shutdown functions in the perfect order (which I'm not > > convinced about since

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-12 Thread Doug Anderson
Hi, On Wed, Jun 12, 2024 at 8:03 AM Maxime Ripard wrote: > > > > Why does something like this now work? > > > > > > drm_panel_shutdown_fixup(panel) > > > { > > > /* if you get warnings here, fix your main drm driver to call > > > * drm_atomic_helper_shutdown() > > > */ >

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-12 Thread Doug Anderson
Hi, On Wed, Jun 12, 2024 at 8:11 AM Daniel Vetter wrote: > > > The problem is that the ordering is wrong, I think. Even if the OS was > > calling driver shutdown functions in the perfect order (which I'm not > > convinced about since panels aren't always child "struct device"s of > > the DRM devi

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-12 Thread Doug Anderson
Hi, On Wed, Jun 12, 2024 at 8:13 AM Daniel Vetter wrote: > > > > I ran the coccinelle script we started with, and here are the results: > > > > > > ./drivers/gpu/drm/vmwgfx/vmwgfx_drv.c:1640:25-39: ERROR: KMS driver > > > vmw_pci_driver is missing shutdown implementation > > > ./drivers/gpu/drm/

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-12 Thread Daniel Vetter
On Wed, Jun 12, 2024 at 07:39:01AM -0700, Doug Anderson wrote: > Hi, > > On Wed, Jun 12, 2024 at 1:09 AM Maxime Ripard wrote: > > > > Hi, > > > > On Tue, Jun 11, 2024 at 07:48:51AM GMT, Douglas Anderson wrote: > > > At shutdown if you've got a _properly_ coded DRM modeset driver then > > > you'll

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-12 Thread Daniel Vetter
On Wed, Jun 12, 2024 at 07:49:31AM -0700, Doug Anderson wrote: > Hi, > > On Wed, Jun 12, 2024 at 1:58 AM Daniel Vetter wrote: > > > > On Tue, Jun 11, 2024 at 07:48:51AM -0700, Douglas Anderson wrote: > > > At shutdown if you've got a _properly_ coded DRM modeset driver then > > > you'll get these

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-12 Thread Maxime Ripard
On Wed, Jun 12, 2024 at 07:49:31AM GMT, Doug Anderson wrote: > Hi, > > On Wed, Jun 12, 2024 at 1:58 AM Daniel Vetter wrote: > > > > On Tue, Jun 11, 2024 at 07:48:51AM -0700, Douglas Anderson wrote: > > > At shutdown if you've got a _properly_ coded DRM modeset driver then > > > you'll get these t

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-12 Thread Doug Anderson
Hi, On Wed, Jun 12, 2024 at 1:58 AM Daniel Vetter wrote: > > On Tue, Jun 11, 2024 at 07:48:51AM -0700, Douglas Anderson wrote: > > At shutdown if you've got a _properly_ coded DRM modeset driver then > > you'll get these two warnings at shutdown time: > > > > Skipping disable of already disable

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-12 Thread Doug Anderson
Hi, On Wed, Jun 12, 2024 at 1:09 AM Maxime Ripard wrote: > > Hi, > > On Tue, Jun 11, 2024 at 07:48:51AM GMT, Douglas Anderson wrote: > > At shutdown if you've got a _properly_ coded DRM modeset driver then > > you'll get these two warnings at shutdown time: > > > > Skipping disable of already d

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-12 Thread Daniel Vetter
On Tue, Jun 11, 2024 at 07:48:51AM -0700, Douglas Anderson wrote: > At shutdown if you've got a _properly_ coded DRM modeset driver then > you'll get these two warnings at shutdown time: > > Skipping disable of already disabled panel > Skipping unprepare of already unprepared panel > > These

Re: [PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-12 Thread Maxime Ripard
Hi, On Tue, Jun 11, 2024 at 07:48:51AM GMT, Douglas Anderson wrote: > At shutdown if you've got a _properly_ coded DRM modeset driver then > you'll get these two warnings at shutdown time: > > Skipping disable of already disabled panel > Skipping unprepare of already unprepared panel > > The

[PATCH] drm/panel: Avoid warnings w/ panel-simple/panel-edp at shutdown

2024-06-11 Thread Douglas Anderson
At shutdown if you've got a _properly_ coded DRM modeset driver then you'll get these two warnings at shutdown time: Skipping disable of already disabled panel Skipping unprepare of already unprepared panel These warnings are ugly and sound concerning, but they're actually a sign of a properl