mtd-20000119 release
David Woodhouse
dwmw2 at infradead.org
Wed Jan 19 06:07:18 EST 2000
pavel_roskin at geocities.com said:
> That all for today. Don't hesitate to put a new diff on the website.
> This will save time and labor of people interested in getting your
> stuff working on PowerPC.
I've included your patches, with the exception of the change from switch() to
if-else chains, along with the other stuff I've changed or been sent.
I think that I've also applied any other patches which I've been send - if
anyone else has sent me stuff which I've omitted - I'm sorry, let me know
and I'll apply them.
I'm waiting for a response on l-k and from Paul before I make a final
judgement on the switch() stuff, but I'm inclined to believe that we were
doing the right thing, and that it's the PPC swab macros which are buggy
because they don't handle constant expressions properly.
If it's not fixable any other way, I'd rather fix it by changing to:
switch(le16_to_cpu(status)) {
case BLOCK_FREE:
than by changing to if-else chains. I had it like that in the first place, in
fact - but swapped it round as an optimisation - because I believed that the
swap would then be done at compile-time rather than at run time.
I still believe that's what _should_ happen.
--
dwmw2
To unsubscribe, send "unsubscribe mtd" to majordomo at infradead.org
More information about the linux-mtd
mailing list