I think for all of your concers one of the following is true:
- they are not valid any more in this version; or
- I cannot understand it.
I'll try to list them all below. Please tell us if the list isn't
complete or if my comments doesn't convince you. You might have to
repeat yourself, but for me it's hard to sort your arguments because
Magnus' suggestion changed over time.
And please, I try to work out the pros and cons in a constructive way
and hope there is nothing in it you will take personal. It's really
that I consider the patch valuable and don't understand your concerns.
In the first thread[1] your unique and open concerns (to the best of my
knowledge and belief) with my comments are:
- "This only works for embedded devices [...]"
OK, this doesn't work with shared IRQs which rules out x86.
I don't claim to know all the 23[2] other architectures but
IMHO if something is good for 3 archs and doesn't hurt the
other 21, you should do it.
- "This would save somebody the trouble to add the above 5 lines
to the 30 lines of board/platform support code he has to write
anyway. That's the only gain, and that is not enough."
IMHO it's worth it. Because if you add the five lines to a
central place you save 5 lines per platform using the driver.
Moreover this might prevent some bugs. (And obviously this
function has the potential to have a buggy implementation as
the comment by Alan Cox shows.)
- "And if you _know_ that on your platform the irq is not
shared, this might really be a one-liner that simply calls
irq_disable. That's OK in board specific code, but not in a
generic driver."
Please note that the patch only introduces a helper that the
platform code *can* use. You still have the freedom not to
use it without any overhead.
- "I won't accept anything that changes the current UIO
behaviour."
Not valid anymore. There is no change in behaviour.
In the second thread[3] I cannot find any open concerns that are not
already listed above.
I think nobody really is surprised that you're not happy with the new
post. But note that Magnus just did what you told him. ("I'm [not] the
big boss who makes the final decision. I can be critized and overridden.
If Greg loves your patch and merges it, fine. Try it.")
In the hope not to have kicked off a flame,
Uwe
[1] http://thread.gmane.org/gmane.linux.kernel/689631
[2] ukleinek@zentaur:~/gsrc/linux-2.6$ ls -l arch/ | grep ^d | wc -l
24
[3] http://thread.gmane.org/gmane.linux.ports.sh.devel/3917/
--
Uwe Kleine-König, Software Engineer
Digi International GmbH Branch Breisach, Küferstrasse 8, 79206 Breisach, Germany
Tax: 315/5781/0242 / VAT: DE153662976 / Reg. Amtsgericht Dortmund HRB 13962
--
