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

[alex@foogod.com]

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/