Re: [PATCH 02/13] fbdev: add remove_conflicting_pci_framebuffers()

From: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
To: Daniel Vetter <daniel@ffwll.ch>
Cc: "Teddy Wang" <teddy.wang@siliconmotion.com>,
	"David Airlie" <airlied@linux.ie>,
	"Greg KH" <gregkh@linuxfoundation.org>,
	amd-gfx@lists.freedesktop.org,
	"Michał Mirosław" <mirq-linux@rere.qmqm.pl>,
	dri-devel@lists.freedesktop.org,
	"Alex Deucher" <alexander.deucher@amd.com>,
	virtualization@lists.linux-foundation.org,
	"Christian König" <christian.koenig@amd.com>,
	"Gerd Hoffmann" <kraxel@redhat.com>
Subject: Re: [PATCH 02/13] fbdev: add remove_conflicting_pci_framebuffers()
Date: Thu, 30 Nov 2017 23:49:53 +0000	[thread overview]
Message-ID: <20171130234953.GA5542@sudip-laptop> (raw)
In-Reply-To: <20171129095634.ymeld4l7zaehbuvn@phenom.ffwll.local>

Hi Daniel,

On Wed, Nov 29, 2017 at 10:56:34AM +0100, Daniel Vetter wrote:
> On Tue, Nov 28, 2017 at 12:30:30PM +0000, Sudip Mukherjee wrote:
> > On Tue, Nov 28, 2017 at 12:32:38PM +0100, Greg KH wrote:
> > > On Tue, Nov 28, 2017 at 11:22:17AM +0100, Daniel Vetter wrote:
> > > > On Mon, Nov 27, 2017 at 08:52:19PM +0000, Sudip Mukherjee wrote:
> > > > > On Mon, Nov 27, 2017 at 11:27:59AM +0100, Daniel Vetter wrote:
> > > > > > On Fri, Nov 24, 2017 at 06:53:31PM +0100, Michał Mirosław wrote:
> > > > > > > Almost all drivers using remove_conflicting_framebuffers() wrap it with
> > > > > > > the same code. Extract common part from PCI drivers into separate
> > > > > > > remove_conflicting_pci_framebuffers().
> > > > > > > 

<snip>

> > > > 
> > > > Greg?
> > > 
> > > Yes, if no one is working to get it out of staging, that means no one
> > > cares about it, and it needs to be removed from the tree.
> > 
> > Agreed. I was not working on getting it out of staging as there is no
> > place for it to go.
> > But, Teddy Wang told me that they have a working drm driver for it, but
> > it is not atomic like Daniel was asking for. If it is ok, then I can send
> > in a patch to remove the existing sm750 from staging and add the new sm750
> > drm driver to staging. And after it is ready, we can go ahead with moving
> > it out of staging to drm.
> 
> Please keep the todo item that it needs to be converted to atomic. And
> tbh, it's probably faster if you just submit it to dri-devel, assuming you
> have time to work on it. For small drivers we tend to be fairly quick in
> getting them into good enough shape.

I have received the driver from Teddy and pushed it to
https://github.com/sudipm-mukherjee/parport/tree/drm_smi for your first
look into it. It is not even building with next-20171130 and has lots of
build warnings. I will have to do a lot of work on it before I can even
submit it to dri-devel.

Time will be the problem, as this is not part of my day job.

> 
> Staging is also a major pain for drm subsystem refactorings, I really,
> really, really prefer we don't add more than the vbox pain we have
> already.

I am hoping that after seeing it, you will agree to have it in staging. :)

--
Regards
Sudip
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel