Re: [patch] I-Opener fix (again)

From: alex@foogod.com
Date: Wed Dec 13 2000 - 14:40:46 EST


On Tue, Dec 12, 2000 at 10:47:35PM -0800, Andre Hedrick wrote:
>
> Basically if the setting of
>
> * "hdx=flash" : allows for more than one ata_flash disk to be
> * registered. In most cases, only one device
> * will be present.
>
> fails then I will look into this but, the breaking of laptops that have
> CFA devices that do not come on channels in a pair canb not happen.
> If you have a vender unique setting that will follow always the way
> I-Opener's are setup then that is better.

Ok, there are two things here:

1) "hdx=flash" _does_ fail, because the flash-related code clobbers hda
   _after_ hda's detection phase, when it's looking at hdb (which is flash).

2) I can see no situation where hdb detection should ever override the
   _already_performed_ detection of hda in this way.

Basically, as is, the kernel finds hda (traditional IDE device), configures it
normally, then finds hdb (flash), and clobbers all the correct info it already
detected for hda. This seems just plain wrong.

-alex
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Fri Dec 15 2000 - 21:00:27 EST