Date:   Mon, 8 Jul 2019 05:54:43 +0000
From:   "james qian wang (Arm Technology China)" <james.qian.wang@....com>
To:     Liviu Dudau <Liviu.Dudau@....com>,
        "airlied@...ux.ie" <airlied@...ux.ie>,
        Brian Starkey <Brian.Starkey@....com>,
        "maarten.lankhorst@...ux.intel.com" 
        <maarten.lankhorst@...ux.intel.com>,
        "sean@...rly.run" <sean@...rly.run>, nd <nd@....com>,
        Ayan Halder <Ayan.Halder@....com>,
        "Oscar Zhang (Arm Technology China)" <Oscar.Zhang@....com>,
        "Tiannan Zhu (Arm Technology China)" <Tiannan.Zhu@....com>,
        "Jonathan Chai (Arm Technology China)" <Jonathan.Chai@....com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "dri-devel@...ts.freedesktop.org" <dri-devel@...ts.freedesktop.org>,
        "Julien Yin (Arm Technology China)" <Julien.Yin@....com>,
        "Channing Chen (Arm Technology China)" <Channing.Chen@....com>,
        "Yiqi Kang (Arm Technology China)" <Yiqi.Kang@....com>,
        "thomas Sun (Arm Technology China)" <thomas.Sun@....com>,
        "Lowry Li (Arm Technology China)" <Lowry.Li@....com>,
        Ben Davis <Ben.Davis@....com>
Subject: Re: [PATCH 1/2] drm/komeda: Disable slave pipeline support

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@....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@...ll.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@...ts.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.

Powered by blists - more mailing lists