lkml.org 
[lkml]   [2010]   [Apr]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/7] viafb: package often used basic io functions
Hi Jon,

Jonathan Corbet schrieb:
> On Sat, 17 Apr 2010 19:44:51 +0000
> Florian Tobias Schandinat <FlorianSchandinat@gmx.de> wrote:
>
>> This patch puts redesigned versions of the basic io functions that
>> are used overall the driver in an extra header.
>
> I don't object to the basic idea of these patches, but...as I'm sure
> you can imagine, this stuff will conflict hard with the work I'm trying
> to prepare for submission. I, too, have had to go in to those
> functions and do things like add locking. Is there any chance you
> could hold off for just a *little* bit while I get things together?

Huh, I really didn't expect that to conflict with anything you were
doing as those are basic functions that are only changed a bit and I
didn't find anything that would cause any disturbance in some of your trees.
As for locking I'd rather suggest doing it in the caller of these
functions as they are often bundled together.
Feel free to ignore theses patches if they get in your way. As I said
the next merge window is mostly for you, I just try to get some stuff in
that has a near zero regression potential and that I don't expect to
conflict with yours.

> As I said before, I was traveling this last week. Now I'm home and can
> get back to this...

Thanks,

Florian Tobias Schandinat


\
 
 \ /
  Last update: 2010-04-17 22:57    [from the cache]
©2003-2011 Jasper Spaans