Re: [PATCH 1/7] viafb: package often used basic io functions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Jon,

Jonathan Corbet schrieb:
On Sat, 17 Apr 2010 19:44:51 +0000
Florian Tobias Schandinat <FlorianSchandinat@xxxxxx> 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
--
To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Video for Linux]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Tourism]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux