[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