From: Ivaylo Dimitrov <ivo.g.dimitrov.75@gmail.com>
To: Tony Lindgren <tony@atomide.com>, Suman Anna <s-anna@ti.com>
Cc: "devel@driverdev.osuosl.org" <devel@driverdev.osuosl.org>,
"Greg KH" <gregkh@linuxfoundation.org>,
"Peter Meerwald" <pmeerw@pmeerw.net>,
"Pali Rohár" <pali.rohar@gmail.com>,
"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>
Subject: Re: Anybody working on tidspbridge?
Date: Thu, 10 Jul 2014 08:54:18 +0300 [thread overview]
Message-ID: <53BE2A8A.5080001@gmail.com> (raw)
In-Reply-To: <20140709070726.GF28884@atomide.com>
On 9.07.2014 10:07, Tony Lindgren wrote:
> * Suman Anna <s-anna@ti.com> [140708 11:40]:
>> Hi Peter,
>>
>> On 07/08/2014 09:36 AM, Greg KH wrote:
>>> On Tue, Jul 08, 2014 at 03:03:58PM +0200, Peter Meerwald wrote:
>>>> Hello,
>>>>
>>>>>> Given the total lack of response here, I suggest just deleting the
>>>>>> driver. No one has ever done the "real work" that is going to be
>>>>>> required to get this code out of staging. It has had build errors
>>>>>> causing it to not even be usable for some kernel versions with no one
>>>>>> noticing, so I doubt anyone cares about it anymore here.
>>>>>
>>>>> Cc'ing some more people who might be interested. If no one offers to
>>>>> work on the driver in the next couple of days, I'll send a patch to
>>>>> remove it.
>>>>
>>>> I'm using the driver (with kernel 3.7) and it works reasonably well for
>>>> me; removing it seems a bit harsh.
>>>
>>> Using it is different from being able to maintain the code and move it
>>> out of the staging tree. Also, 3.7 is really old and obsolete, not much
>>> we can do with that kernel version :)
>>>
>>> Are you able to work on the code and do the effort needed to get it out
>>> of the staging tree? If so, great, if not, we are going to have to
>>> delete it, sorry.
>>
>> I agree with Greg here. In fact, the current TODO does not do enough
>> justice to the amount of work required to make it even work on the
>> latest kernel. Most of the TIers who worked on this driver have moved on
>> as Kristina would have figured with her bounced emails. So I do suggest
>> that this driver be deleted from the kernel tree. If there are enough
>> number of folks using it (not sure how many are out there), it can be
>> worked on out-of-tree and brought back in a cleaner fashion rather than
>> keeping a broken stale driver in the kernel.
>
> I agree, not much has improved with this driver since it got added into
> staging except just compile fixes.
>
> Regards,
>
> Tony
Well, recently I've sent a couple of patches which implement stuff from
TODO [1]. However, with the migration to DT, my focus now is to have a
kernel/userspace that boots at all and this leaves no free cycles for
DSP. Maybe tidspbridge can be left in staging until DT migration is
finished, that way me (and maybe other people) will have the time needed
to try to implement what remains in TODO. Also, keep in mind there will
(hopefully) be another omap3 end-user device released by the end of the
year (Neo900), which most probably will gain more developers interested
in fixing the DSP driver.
Regards,
Ivo
[1]
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/drivers/staging/tidspbridge?id=559c71fe5dc3bf2ecc55afb336145db7f0abf810
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/drivers/staging/tidspbridge?id=a3b22220a48bd14c9ba4ca8d051b0c92d5bc3866
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/drivers/staging/tidspbridge?id=d30555853052bbec8497260ba888b7d696bed9b8
next prev parent reply other threads:[~2014-07-10 5:54 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <53A4569B.70604@gmail.com>
[not found] ` <53A45BBD.4040201@gmail.com>
[not found] ` <20140629164250.GA31625@kroah.com>
2014-07-08 12:48 ` Anybody working on tidspbridge? Kristina Martšenko
2014-07-08 13:03 ` Peter Meerwald
2014-07-08 14:36 ` Greg KH
2014-07-08 18:38 ` Suman Anna
2014-07-09 7:07 ` Tony Lindgren
2014-07-10 5:54 ` Ivaylo Dimitrov [this message]
2014-07-10 16:11 ` Greg KH
2014-07-10 16:18 ` Jason Kridner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=53BE2A8A.5080001@gmail.com \
--to=ivo.g.dimitrov.75@gmail.com \
--cc=devel@driverdev.osuosl.org \
--cc=gregkh@linuxfoundation.org \
--cc=linux-omap@vger.kernel.org \
--cc=pali.rohar@gmail.com \
--cc=pmeerw@pmeerw.net \
--cc=s-anna@ti.com \
--cc=tony@atomide.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox