[SeaBIOS] bootorder, ATA-SFF, AHCI and CSM; oh my!

Jonathan A. Kollasch jakllsch at kollasch.net
Thu Jul 23 15:41:46 CEST 2015


On Tue, Jul 14, 2015 at 03:18:25PM -0400, Kevin O'Connor wrote:
> On Wed, Jul 08, 2015 at 10:21:57PM -0500, Jonathan A. Kollasch wrote:
> > On Tue, Jul 07, 2015 at 05:06:03PM -0400, Kevin O'Connor wrote:
> > > Thanks.  Are you willing to respin a simpler version of the patch?
> > 
> > Attached.
> 
> I don't think the user visible drive descriptions or the debug strings
> should change.  How about the patch below instead?
> 
> -Kevin

Looks fine.  But if you're going to add ataid to ata_channel_s you can
probably also avoid any changes to CSM-mode behavior too.  I had shied
away from adding a member to ata_channel_s because it was already nicely
packed (a multiple of 4 bytes).

Also changing the bootorder names is a user-visible change, one we should
change, but it's a change none the less, so maybe changing the descriptions
isn't such a bad thing.  But maybe changing the descriptions should be in a
different changeset if we do decide to do it.

	Jonathan Kollasch



More information about the SeaBIOS mailing list