From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752121AbeEGJFu (ORCPT ); Mon, 7 May 2018 05:05:50 -0400 Received: from lb1-smtp-cloud9.xs4all.net ([194.109.24.22]:48995 "EHLO lb1-smtp-cloud9.xs4all.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751940AbeEGJFq (ORCPT ); Mon, 7 May 2018 05:05:46 -0400 Subject: Re: [PATCH] media: zoran: move to dma-mapping interface To: Mauro Carvalho Chehab , Arnd Bergmann Cc: Christoph Hellwig , Mauro Carvalho Chehab , Hans Verkuil , Arvind Yadav , mjpeg-users@lists.sourceforge.net, Linux Media Mailing List , Linux Kernel Mailing List , Trent Piepho References: <20180424204158.2764095-1-arnd@arndb.de> <20180425061537.GA23383@infradead.org> <20180425072138.GA16375@infradead.org> <20180425152636.GC27076@infradead.org> <20180425142229.25d756ed@vento.lan> From: Hans Verkuil Message-ID: <36e4effa-135d-daeb-1abb-f55500a9dfee@xs4all.nl> Date: Mon, 7 May 2018 11:05:40 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 MIME-Version: 1.0 In-Reply-To: <20180425142229.25d756ed@vento.lan> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-CMAE-Envelope: MS4wfEcXUSWCwfSDUDDxlD29obxENsyNf4KqFkxicZ3r/2j8PlCvC9lbg+DZlWaPSJpPNa+QQv88PJYB9YsVjUpEz0z6rcqmAevwTyiG99zuvRiDjtKlMexH CaIU41+uSvMknqry83RTY4IkaWJWQiBUo5a4Jbul4GJg5kJicrZvolSiubcq79IDp5mhRSQgTo6cDKfi/DCe0naB6gjW72p5OvqmP+y4HETKJ1E0Bc25CKH7 XObMNtP68UepGifEiaaJZrFuTVj3xqFZvdeI5ziu2XH3JPfaO/BWvHiSsR1VnvtkDMif6AA8hvfIYqO0oSHsUyX7Z3AwbIADzLep+A9XRJstRlObCM2QAdd9 BP48yr/WoqB/i50+AaL6bV5Lt7N27z1zTXzjqquupFj8uSoSuZpPhRMcfTBewp/Jqd9rbYyZW351/cVlAyRW8QXlt3s6mCdxvXQj4YFKdQoRerEm2VFsv2+h aSRrA/KChQHzajXfftfvOW6rsAs7WVVR+pg4AGmFzLytk0qff6nlP0Ob9XwxQF+3gchwzoSatIfRTYoWd0FU/+iol1IH3NAhIf/buVSrXpVJCX4a1Df4GRK+ 7i6oSd+MYMhjylBvkiP1OacgDwLbvPgyX1yK2G1JlMWvrA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 25/04/18 19:22, Mauro Carvalho Chehab wrote: > Em Wed, 25 Apr 2018 17:58:25 +0200 > Arnd Bergmann escreveu: > >> On Wed, Apr 25, 2018 at 5:26 PM, Christoph Hellwig wrote: >>> On Wed, Apr 25, 2018 at 01:15:18PM +0200, Arnd Bergmann wrote: >>>> That thought had occurred to me as well. I removed the oldest ISDN >>>> drivers already some years ago, and the OSS sound drivers >>>> got removed as well, and comedi got converted to the dma-mapping >>>> interfaces, so there isn't much left at all now. This is what we >>>> have as of v4.17-rc1: >>> >>> Yes, I've been looking at various grotty old bits to purge. Usually >>> I've been looking for some non-tree wide patches and CCed the last >>> active people to see if they care. In a few cases people do, but >>> most often no one does. >> >> Let's start with this one (zoran) then, as Mauro is keen on having >> all media drivers compile-testable on x86-64 and arm. >> >> Trent Piepho and Hans Verkuil both worked on this driver in the >> 2008/2009 timeframe and those were the last commits from anyone >> who appears to have tested their patches on actual hardware. > > Zoran is a driver for old hardware. I don't doubt that are people > out there still using it, but who knows? > > I have a few those boards packed somewhere. I haven't work with PCI > hardware for a while. If needed, I can try to seek for them and > do some tests. I need first to unpack a machine with PCI slots... > the NUCs I generally use for development don't have any :-) > > Anyway, except for virt_to_bus() and related stuff, I think that this > driver is in good shape, as Hans did a lot of work in the past to > make it to use the current media framework. > >> >> Trent, Hans: do you have reason to believe that there might still >> be users out there? I have no way of knowing this. However, I think they are easily replaced by much cheaper USB alternatives today. I did some work on the zoran driver several years ago, but it doesn't use the vb2 framework (and not even the older vb1 framework!) so I'm sure there are all sorts of bugs in that driver. Personally I would be fine with moving this driver to staging and removing it by, say, the end of the year. Nobody is going to work on it and I think it is time to retire it. Regards, Hans