Re: [PATCH 1/2] pata_legacy: export functionality to ide

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Alan Cox
Date: Sunday, August 3, 2008 - 8:22 am

> conventions, it is still the best one considering the other possibilities:

I think you might want to start somewhere else if that worries you. Its
also a mostly bogus reasoning as almost nobody builds with both, in fact
you have to be pretty careful if you do that or it all falls over in a
heap.

For previous cases (eg ide timing) it has actually made more sense to
split the code. The moment you get future different behaviour between
libata and old IDE on any of these devices the sharing will just break
again (eg if one or the other drops chipset support for one of those
devices or adds one for another device)


That would be stunningly dumb. I happen to think we have developers whose
minds extent to adding libata-common.c and pulling in a single file if we
do that. Yes people have the past done stupid stuff like pulling all of
CONFIG_IDE in for a single USB device but that was because nobody noticed
and fixed it promptly not because it was a good idea.

Alan
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] ide-floppy fix, Borislav Petkov, (Mon Jul 14, 10:33 pm)
Re: [PATCH] ide-floppy fix, Borislav Petkov, (Mon Jul 14, 10:40 pm)
Re: [PATCH] ide-floppy fix, Borislav Petkov, (Tue Jul 15, 1:39 pm)
Re: [PATCH] ide-floppy fix, Bartlomiej Zolnierki ..., (Tue Jul 15, 1:58 pm)
Re: [PATCH] ide-floppy fix, Borislav Petkov, (Tue Jul 15, 10:20 pm)
Re: [PATCH] ide-floppy fix, Bartlomiej Zolnierki ..., (Wed Jul 16, 8:59 am)
Re: [PATCH] ide-floppy fix, Bartlomiej Zolnierki ..., (Wed Jul 16, 10:56 am)
Re: [PATCH] ide-floppy fix, Borislav Petkov, (Sun Jul 20, 5:06 am)
Re: [PATCH] ide-floppy fix, Bartlomiej Zolnierki ..., (Mon Jul 21, 12:03 pm)
Re: [PATCH] ide-floppy fix, Borislav Petkov, (Mon Jul 21, 10:27 pm)
Re: [PATCH] ide-floppy fix, Bartlomiej Zolnierki ..., (Tue Jul 22, 12:49 pm)
Re: [PATCH] ide-floppy fix, Borislav Petkov, (Tue Jul 22, 11:32 pm)
Re: [PATCH] ide-floppy fix, Bartlomiej Zolnierki ..., (Wed Jul 23, 11:51 am)
Re: [PATCH] ide-generic: skip automatic probing of legacy ..., Bartlomiej Zolnierki ..., (Sat Aug 2, 10:02 am)
Re: [PATCH] ide-generic: skip automatic probing of legacy ..., Bartlomiej Zolnierki ..., (Sat Aug 2, 11:46 am)
[PATCH 1/2] pata_legacy: export functionality to ide, Borislav Petkov, (Sun Aug 3, 12:37 am)
[PATCH 2/2] ide-generic: handle probing of legacy io-ports, Borislav Petkov, (Sun Aug 3, 12:38 am)
Re: [PATCH 1/2] pata_legacy: export functionality to ide, Bartlomiej Zolnierki ..., (Sun Aug 3, 6:37 am)
Re: [PATCH 2/2] ide-generic: handle probing of legacy io-ports, Bartlomiej Zolnierki ..., (Sun Aug 3, 7:11 am)
Re: [PATCH 1/2] pata_legacy: export functionality to ide, Borislav Petkov, (Sun Aug 3, 7:38 am)
Re: [PATCH 2/2] ide-generic: handle probing of legacy io-ports, Bartlomiej Zolnierki ..., (Sun Aug 3, 7:59 am)
Re: [PATCH 1/2] pata_legacy: export functionality to ide, Alan Cox, (Sun Aug 3, 8:22 am)
Re: [PATCH 1/2] pata_legacy: export functionality to ide, Borislav Petkov, (Sun Aug 3, 9:51 am)
Re: [PATCH 1/2] pata_legacy: export functionality to ide, Sergei Shtylyov, (Tue Aug 5, 7:26 am)
Re: [PATCH 1/2] pata_legacy: export functionality to ide, Bartlomiej Zolnierki ..., (Tue Aug 5, 7:41 am)
Re: [PATCH 1/2] pata_legacy: export functionality to ide, Borislav Petkov, (Tue Aug 5, 11:10 pm)
Re: [PATCH 1/2] pata_legacy: export functionality to ide, Sergei Shtylyov, (Wed Aug 6, 4:34 am)
Re: [PATCH 1/2] pata_legacy: export functionality to ide, Sergei Shtylyov, (Wed Aug 6, 8:57 am)
Re: [PATCH 1/2] pata_legacy: export functionality to ide, Borislav Petkov, (Wed Aug 6, 12:47 pm)
Re: [PATCH 1/2] pata_legacy: export functionality to ide, Sergei Shtylyov, (Wed Aug 6, 1:04 pm)
Re: [PATCH] ide-generic: handle probing of legacy io-ports ..., Bartlomiej Zolnierki ..., (Thu Aug 7, 10:06 am)