Message ID | 1276611581-3757-18-git-send-email-kwolf@redhat.com |
---|---|
State | New |
Headers | show |
On 06/15/2010 09:19 AM, Kevin Wolf wrote: > From: Jes Sorensen<Jes.Sorensen@redhat.com> > > Correct definitions for FD_CMD_SAVE and FD_CMD_RESTORE in hw/fdc.c > FYI, the subject is duplicated here so the commit log looks a bit funky. Regards, Anthony Liguori > Per https://bugs.launchpad.net/qemu/+bug/424453 the correct values > for FD_CMD_SAVE is 0x2e and FD_CMD_RESTORE is 0x4e. Verified against > the Intel 82078 manual which can be found at: > http://wiki.qemu.org/Documentation/HardwareManuals page 22. > > Signed-off-by: Jes Sorensen<Jes.Sorensen@redhat.com> > Signed-off-by: Kevin Wolf<kwolf@redhat.com> > --- > hw/fdc.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/hw/fdc.c b/hw/fdc.c > index b978957..45a876d 100644 > --- a/hw/fdc.c > +++ b/hw/fdc.c > @@ -369,9 +369,9 @@ enum { > FD_CMD_PART_ID = 0x18, > FD_CMD_SCAN_LOW_OR_EQUAL = 0x19, > FD_CMD_SCAN_HIGH_OR_EQUAL = 0x1d, > - FD_CMD_SAVE = 0x2c, > + FD_CMD_SAVE = 0x2e, > FD_CMD_OPTION = 0x33, > - FD_CMD_RESTORE = 0x4c, > + FD_CMD_RESTORE = 0x4e, > FD_CMD_DRIVE_SPECIFICATION_COMMAND = 0x8e, > FD_CMD_RELATIVE_SEEK_OUT = 0x8f, > FD_CMD_FORMAT_AND_WRITE = 0xcd, >
Am 15.06.2010 16:28, schrieb Anthony Liguori: > On 06/15/2010 09:19 AM, Kevin Wolf wrote: >> From: Jes Sorensen<Jes.Sorensen@redhat.com> >> >> Correct definitions for FD_CMD_SAVE and FD_CMD_RESTORE in hw/fdc.c >> > > FYI, the subject is duplicated here so the commit log looks a bit funky. I agree. There are some more patches by Jes in master that look the same, so I didn't complain, but it definitely looks a bit strange. Kevin
On 06/15/10 16:42, Kevin Wolf wrote: > Am 15.06.2010 16:28, schrieb Anthony Liguori: >> On 06/15/2010 09:19 AM, Kevin Wolf wrote: >>> From: Jes Sorensen<Jes.Sorensen@redhat.com> >>> >>> Correct definitions for FD_CMD_SAVE and FD_CMD_RESTORE in hw/fdc.c >>> >> >> FYI, the subject is duplicated here so the commit log looks a bit funky. > > I agree. There are some more patches by Jes in master that look the > same, so I didn't complain, but it definitely looks a bit strange. Hmmm maybe I am messing something up. I started with strange duplicate lines because I thought git send-email would cut the first line for the subject line and then just leave the rest in. Probably just me getting used to do it all in git instead of using quilt. Cheers, Jes
Am 15.06.2010 16:47, schrieb Jes Sorensen: > On 06/15/10 16:42, Kevin Wolf wrote: >> Am 15.06.2010 16:28, schrieb Anthony Liguori: >>> On 06/15/2010 09:19 AM, Kevin Wolf wrote: >>>> From: Jes Sorensen<Jes.Sorensen@redhat.com> >>>> >>>> Correct definitions for FD_CMD_SAVE and FD_CMD_RESTORE in hw/fdc.c >>>> >>> >>> FYI, the subject is duplicated here so the commit log looks a bit funky. >> >> I agree. There are some more patches by Jes in master that look the >> same, so I didn't complain, but it definitely looks a bit strange. > > Hmmm maybe I am messing something up. I started with strange duplicate > lines because I thought git send-email would cut the first line for the > subject line and then just leave the rest in. Probably just me getting > used to do it all in git instead of using quilt. It does take the first line for the subject, but when importing your mail, git am puts the subject back into the first line, so the whole text should look the same as in your local repository. Kevin
On 06/15/10 16:52, Kevin Wolf wrote: > Am 15.06.2010 16:47, schrieb Jes Sorensen: >> Hmmm maybe I am messing something up. I started with strange duplicate >> lines because I thought git send-email would cut the first line for the >> subject line and then just leave the rest in. Probably just me getting >> used to do it all in git instead of using quilt. > > It does take the first line for the subject, but when importing your > mail, git am puts the subject back into the first line, so the whole > text should look the same as in your local repository. I see, I wasn't aware of that ... well I'll try to keep that in mind next time :) Thanks Jes
diff --git a/hw/fdc.c b/hw/fdc.c index b978957..45a876d 100644 --- a/hw/fdc.c +++ b/hw/fdc.c @@ -369,9 +369,9 @@ enum { FD_CMD_PART_ID = 0x18, FD_CMD_SCAN_LOW_OR_EQUAL = 0x19, FD_CMD_SCAN_HIGH_OR_EQUAL = 0x1d, - FD_CMD_SAVE = 0x2c, + FD_CMD_SAVE = 0x2e, FD_CMD_OPTION = 0x33, - FD_CMD_RESTORE = 0x4c, + FD_CMD_RESTORE = 0x4e, FD_CMD_DRIVE_SPECIFICATION_COMMAND = 0x8e, FD_CMD_RELATIVE_SEEK_OUT = 0x8f, FD_CMD_FORMAT_AND_WRITE = 0xcd,