diff mbox series

[1/2] drm/komeda: Disable slave pipeline support

Message ID 20190705114357.17403-1-james.qian.wang@arm.com (mailing list archive)
State New, archived
Headers show
Series [1/2] drm/komeda: Disable slave pipeline support | expand

Commit Message

James Qian Wang July 5, 2019, 11:44 a.m. UTC
Since the property slave_planes have been removed, to avoid the resource
assignment problem in user disable slave pipeline support temporarily.

Signed-off-by: james qian wang (Arm Technology China) <james.qian.wang@arm.com>
---
 drivers/gpu/drm/arm/display/komeda/komeda_crtc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Daniel Vetter July 5, 2019, 12:23 p.m. UTC | #1
On Fri, Jul 05, 2019 at 11:44:16AM +0000, james qian wang (Arm Technology China) wrote:
> Since the property slave_planes have been removed, to avoid the resource
> assignment problem in user disable slave pipeline support temporarily.
> 
> Signed-off-by: james qian wang (Arm Technology China) <james.qian.wang@arm.com>

I guess the way you have to enumerate the planes listing the slave planes
wont just automatically work in any fashion and force a lot more fallbacks
to primary plane only. At least until virtualization of plane hw is done.
So makes sense to outright disable all the slave plane stuff for now. And
I think it's ok to keep all the code still, we'll use it again.

Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>

> ---
>  drivers/gpu/drm/arm/display/komeda/komeda_crtc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_crtc.c b/drivers/gpu/drm/arm/display/komeda/komeda_crtc.c
> index f4400788ab94..8ee879ee3ddc 100644
> --- a/drivers/gpu/drm/arm/display/komeda/komeda_crtc.c
> +++ b/drivers/gpu/drm/arm/display/komeda/komeda_crtc.c
> @@ -481,7 +481,7 @@ int komeda_kms_setup_crtcs(struct komeda_kms_dev *kms,
>  		master = mdev->pipelines[i];
>  
>  		crtc->master = master;
> -		crtc->slave  = komeda_pipeline_get_slave(master);

This might cause an unused function warning, might need to annotate it
with __unused.
-Daniel
> +		crtc->slave  = NULL;
>  
>  		if (crtc->slave)
>  			sprintf(str, "pipe-%d", crtc->slave->id);
> -- 
> 2.20.1
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
James Qian Wang July 8, 2019, 5:54 a.m. UTC | #2
On Fri, Jul 05, 2019 at 02:23:48PM +0200, Daniel Vetter wrote:
> On Fri, Jul 05, 2019 at 11:44:16AM +0000, james qian wang (Arm Technology China) wrote:
> > Since the property slave_planes have been removed, to avoid the resource
> > assignment problem in user disable slave pipeline support temporarily.
> >
> > Signed-off-by: james qian wang (Arm Technology China) <james.qian.wang@arm.com>
>
> I guess the way you have to enumerate the planes listing the slave planes
> wont just automatically work in any fashion and force a lot more fallbacks
> to primary plane only. At least until virtualization of plane hw is done.
> So makes sense to outright disable all the slave plane stuff for now. And
> I think it's ok to keep all the code still, we'll use it again.
>
> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
>
> > ---
> >  drivers/gpu/drm/arm/display/komeda/komeda_crtc.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_crtc.c b/drivers/gpu/drm/arm/display/komeda/komeda_crtc.c
> > index f4400788ab94..8ee879ee3ddc 100644
> > --- a/drivers/gpu/drm/arm/display/komeda/komeda_crtc.c
> > +++ b/drivers/gpu/drm/arm/display/komeda/komeda_crtc.c
> > @@ -481,7 +481,7 @@ int komeda_kms_setup_crtcs(struct komeda_kms_dev *kms,
> >  master = mdev->pipelines[i];
> >
> >  crtc->master = master;
> > -crtc->slave  = komeda_pipeline_get_slave(master);
>
> This might cause an unused function warning, might need to annotate it
> with __unused.
> -Daniel

If so, I'd like to drop this change.

Since even with this change, that still can not pass all our tests.
we have to update the user tests or the komeda implementation.
Once we finished the updating, we still need to revert this change.
So maybe just drop it is more better.

Thanks
James

> > +crtc->slave  = NULL;
> >
> >  if (crtc->slave)
> >  sprintf(str, "pipe-%d", crtc->slave->id);
> > --
> > 2.20.1
> >
> > _______________________________________________
> > dri-devel mailing list
> > dri-devel@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/dri-devel
>
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
diff mbox series

Patch

diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_crtc.c b/drivers/gpu/drm/arm/display/komeda/komeda_crtc.c
index f4400788ab94..8ee879ee3ddc 100644
--- a/drivers/gpu/drm/arm/display/komeda/komeda_crtc.c
+++ b/drivers/gpu/drm/arm/display/komeda/komeda_crtc.c
@@ -481,7 +481,7 @@  int komeda_kms_setup_crtcs(struct komeda_kms_dev *kms,
 		master = mdev->pipelines[i];
 
 		crtc->master = master;
-		crtc->slave  = komeda_pipeline_get_slave(master);
+		crtc->slave  = NULL;
 
 		if (crtc->slave)
 			sprintf(str, "pipe-%d", crtc->slave->id);