On Fri, 30 Jan 2009 08:27:20 -0500 Devin Heitmueller <devin.heitmueller@xxxxxxxxx> wrote: > On Fri, Jan 30, 2009 at 4:20 AM, Trent Piepho <xyzzy@xxxxxxxxxxxxx> wrote: > > On Thu, 29 Jan 2009, Devin Heitmueller wrote: > >> On Thu, Jan 29, 2009 at 8:19 PM, Trent Piepho <xyzzy@xxxxxxxxxxxxx> wrote: > >> > I haven't been able to test this code. It seems my bt848 card doesn't work > >> > with my SATA controller and I sort of need the latter to access the > >> > harddrive. But I think everything should work. It cuts the the bttv > >> > driver to less than half its current size. > >> > > >> > A number of the changes are for specialized cards that likely have few if > >> > any users left. I'm pretty sure some have been broken for quite a while now. > >> > > >> > Please pull from http://linuxtv.org/hg/~tap/bttv > >> > > >> > bttv-cards.c | 1323 ++++++++++++++++++++++------------------------------------ > >> > bttv-driver.c | 90 +-- > >> > bttv-i2c.c | 6 > >> > bttv-if.c | 18 > >> > bttv-risc.c | 4 > >> > bttv-vbi.c | 2 > >> > bttv.h | 84 ++- > >> > bttvp.h | 19 > >> > 8 files changed, 640 insertions(+), 906 deletions(-) > >> > >> Perhaps I am misunderstanding what you said in this email, but are you > >> submitting a PULL request for 1500 lines of code that have had no > >> testing? > > > > Bugs?! In my code? I think not! > > > > If you look at the patches you'll see it's not nearly as large as it might > > seem. > > When I make a five line code change for a given chipset, I won't > submit it unless it's tested against at least three cards. In this > case, it's 1500 lines, and you have tested it on zero cards. You make > one typo or screw up one pointer and we'll have potentially hundreds > of people reporting kernel panics. > > I don't care how safe you think the change is. If it isn't tested it > under at least one card, then it cannot be that important a change as > to incur the risk of regressions. This isn't a one-line change to a > comment. > > I'm just one developer and my opinion doesn't count for much, but > checking in a large body of code blindly is a colossally stupid idea. Devin, I have one bttv hardware installed here available for testing. This one has 4 bttv chips and uses 2 different board entries. I intend to test the changeset before applying. Anyway, it is good if more people could test it, especially with the exotic hardware. Unfortunately, my other bttv hardwares are on another city, so I can't test on them. Cheers, Mauro -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
