[SeaBIOS] bootorder, ATA-SFF, AHCI and CSM; oh my!
Kevin O'Connor
kevin at koconnor.net
Sat Jul 25 00:59:43 CEST 2015
On Thu, Jul 23, 2015 at 08:41:46AM -0500, Jonathan A. Kollasch wrote:
> 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.
Thanks. I committed the patch.
I don't think using the PCI bus/device/function id in the user visible
string makes sense as only hardcore developers would know what that
means. (It would be confusing to a general user to see ATA at 00b2 or
any similar non-linear identifier.)
-Kevin
More information about the SeaBIOS
mailing list