Error: Wacky! Unable to decode failure status

[Thayne Harbaugh]

On Mon, 2003-05-05 at 16:49, Michael Moedt wrote:
> I hope you don't mind my fooling, but I've modified
> the patch yet again,

I often find better ways of doing things if I re-read and edit things
one more time.

> to use if-then blocks instead of
> a switch. (I've heard it's better for removing code at
> compile-time.)

I'm no expert here - I think a good compiler will remove unnecessary
code regardless.  Either way, I'm not so hung up on a switch/case that I
refuse to change.

> Also, it complained to me about CFIDEV_BUSWIDTH_8
> being undeclared, which matches the behavior in
> include/linux/mtd/cfi.h, when the geometry option
> isn't enabled.  I put in something in for this as
> well.  Could you look over my changes?

You'll have to forgive me, but I also changed my original patch.  The
"ones = CMD( ones )" is not needed.

BTW, we should just ignore the CFIDEV_BUSWIDTH_8 - it's not used
anywhere that I know of and it has some known issues that make it icky.

Yeah, I can't believe I used the word "icky."

> Thanks again,
> 
> Mike
> 
> 
> Here's that patch:

Thanks.

-- 
Thayne Harbaugh
Linux Networx
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 232 bytes
Desc: This is a digitally signed message part
Url : http://lists.infradead.org/pipermail/linux-mtd/attachments/20030505/10d4f5d5/attachment.bin