lkml.org 
[lkml]   [2007]   [May]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] ISDN: move card state init to separate function
Karsten Keil wrote:
> So what is the reason to do this now, does this block some changes in
> other places ? I did not change this code for long time because I know
> that this is a nightmare, the code was grown and grown (supporting more and
> more cards/chipsets) and is really nothing which should be used in future.
> So I would say it maybe better, if it doesn't block anything else, to
> replace HiSax completely with a new modular design. I know that here was not
> so much effort in the past years, but now I got some time to push mISDN
> forward and it looks not so bad to get the first parts ready for mainline
> in June.

ISDN is the biggest chunk of code blocking removal of some deprecated
PCI interfaces.

If you want to replace the code completely with a newfangled modular
design, I will gladly step out of the way and let that happen :) I only
know what I see right now, which is code that has sat around for a while
not getting updated to the new PCI API.

Since I do not have any ISDN hardware, my plan was to do equivalent
transformations of the code. Each patch just shuffles code, but
maintains a working state at all time, making it trivial to prove
(mathematically, or with testing, or with git-bisect) that the code
continues to work.

So, just tell me which you would prefer. If you are going to fix all
this stuff, there is plenty of other stuff on my list to work on.
Otherwise I will create equivalent-transform patches that take the
existing code and modify it just enough to be correct for the new PCI API.

Jeff


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2007-05-22 18:49    [W:0.043 / U:0.164 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site