On Wed, Apr 15, 2020 at 02:20:35PM +0200, Daniel Vetter wrote:
> On Wed, Apr 15, 2020 at 2:03 PM Alexey Brodkin
> <Alexey.Brodkin@xxxxxxxxxxxx> wrote:
> >
> > Hi Daniel,
> >
> > > -----Original Message-----
> > > From: Sam Ravnborg <sam@xxxxxxxxxxxx>
> > > Sent: Wednesday, April 15, 2020 12:45 PM
> > > To: Daniel Vetter <daniel.vetter@xxxxxxxx>
> > > Cc: Intel Graphics Development <intel-gfx@xxxxxxxxxxxxxxxxxxxxx>; Alexey Brodkin
> > > <abrodkin@xxxxxxxxxxxx>; DRI Development <dri-devel@xxxxxxxxxxxxxxxxxxxxx>; Daniel Vetter
> > > <daniel.vetter@xxxxxxxxx>
> > > Subject: Re: [PATCH 53/59] drm/arc: Move to drm/tiny
> > >
> > > Hi Daniel.
> > > On Wed, Apr 15, 2020 at 09:40:28AM +0200, Daniel Vetter wrote:
> > > > Because it is.
> > > >
> > > > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxxx>
> > > > Cc: Alexey Brodkin <abrodkin@xxxxxxxxxxxx>
> > > > ---
> > > > MAINTAINERS | 2 +-
> > > > drivers/gpu/drm/Kconfig | 2 --
> > > > drivers/gpu/drm/Makefile | 1 -
> > > > drivers/gpu/drm/arc/Kconfig | 10 ----------
> > > > drivers/gpu/drm/arc/Makefile | 3 ---
> > > > drivers/gpu/drm/tiny/Kconfig | 10 ++++++++++
> > > > drivers/gpu/drm/tiny/Makefile | 1 +
> > > > drivers/gpu/drm/{arc/arcpgu_drv.c => tiny/arcpgu.c} | 0
> > > > 8 files changed, 12 insertions(+), 17 deletions(-)
> > > > delete mode 100644 drivers/gpu/drm/arc/Kconfig
> > > > delete mode 100644 drivers/gpu/drm/arc/Makefile
> > > > rename drivers/gpu/drm/{arc/arcpgu_drv.c => tiny/arcpgu.c} (100%)
> > >
> > > We have "DRM: ARC: add HDMI 2.0 TX encoder support" which
> > > adds another platform driver to drm/arc/
> > > This speaks against the move to tiny IMO
> >
> > Indeed that's an interesting question, see v3 series here:
> > https://lists.freedesktop.org/archives/dri-devel/2020-April/262352.html
>
> Looking at this patch series, feels a bit like hand-rolling of bridge
> code, badly. We should get away from that.
>
> Once you have that I think the end result is tiny enough that it can
> stay, bridges intergrate quite well into simple display pipe drivers.
>
> > BTW should I pull that series in my tree and send you a pull-request
> > or that kind of change needs to go through another tree?
> >
> > Also I'd like to test the change we discuss here to make sure stuff
> > still works. Once we do that I'll send an update. Any hint on
> > when that change needs to be acked/nacked?
>
> Simplest is if this can all land through drm-misc, is arc not
> maintained in there? And there's plenty of time for testing, I'm just
> slowly crawling through the tree to get everything polished and
> cleaned up in this area.
Any updates on testing this pile here? First patch landed now, and I've
started to push driver patches. So would be good to get this sorted out
too.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
