Messages in this thread | | | Date | Fri, 14 May 2004 17:10:28 -0700 | From | Marc Singer <> | Subject | Re: arm-lh7a40x IDE support in 2.6.6 |
| |
On Sat, May 15, 2004 at 01:26:33AM +0200, Bartlomiej Zolnierkiewicz wrote: > On Saturday 15 of May 2004 00:49, Marc Singer wrote: > > On Sat, May 15, 2004 at 12:19:46AM +0200, Bartlomiej Zolnierkiewicz wrote: > > > > > > > > - you are setting IDE_NO_IRQ in ide_init_hwif_ports() which is > > > > > > > used in many places in generic IDE code - anybody wanting to > > > > > > > understand interactions with your code + generic code will have > > > > > > > serious problems (especially if knows _nothing_ about lpd7a40x) > > > > > > > > > > > > I don't know what you mean. I grep for that constant and found it > > > > > > nowhere except for ide-io.c and in my code. It doesn't take much > > > > > > to find the references. > > > > > > > > > > I'm talking about ide_init_hwif_ports() function. > > > > > > > > Most of the ARM arch's use it. Perhaps all of them need a good once > > > > over. > > > > > > Since some time I have a patch killing <asm/arch-*/ide.h>. :) > > > > OK. That raises an interesting question. If a) you as the IDE > > maintainer want to make a policy change, and b) you have a concrete > > action to take, then how do you go about it so that the right thing > > (tm) happens? > > I posted patch to linux-arm-kernel (rmk, I can't find it in l-a-k archives > and I also can't find any mail about it being rejected?) and linux-ide. > [ and to affected arch maintainers of course ]
There is a web page for posting kernel patches. It is most likely to receive attention than posting to the mailing list.
<http://www.arm.linux.org.uk/developer/patches/>
> > One tack would be to post to the ARM list stating that there is > > such-and-such, a new policy, and this requires a change to the > > way-things-work (tm). Then effect a patch that breaks the bad stuff > > so that the users of such bad stuff must cope. > > It is stable kernel series so I paid 'stable kernel' price > and made sure that it shouldn't break anything.
Fair enough. We'll start with lh and see if we can get others to follow. - 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/
| |