Message ID | 1326613915-3282-5-git-send-email-hpoussin@reactos.org |
---|---|
State | New |
Headers | show |
Am 15.01.2012 08:51, schrieb Hervé Poussineau: > Stepping 1 (S82078B) is not fully i82078 compatible, so better stick to initial revision > > Signed-off-by: Hervé Poussineau <hpoussin@reactos.org> > --- > hw/fdc.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/hw/fdc.c b/hw/fdc.c > index bedaeca..0e167f8 100644 > --- a/hw/fdc.c > +++ b/hw/fdc.c > @@ -1385,7 +1385,7 @@ static void fdctrl_handle_version(FDCtrl *fdctrl, int direction) > > static void fdctrl_handle_partid(FDCtrl *fdctrl, int direction) > { > - fdctrl->fifo[0] = 0x41; /* Stepping 1 */ > + fdctrl->fifo[0] = 0x01; /* Stepping 0 */ > fdctrl_set_fifo(fdctrl, 1, 0); > } > Hm, this is the kind of change that I'm hesitant to make because I don't understand the implications. Can you give some more details what is fixed by this and why you think it's harmless for currently working OSes? My spec says this: 6 3 14 PART ID COMMAND This command can be used to identify the floppy disk controller as an enhanced controller The first stepping of the 82078 (all 44 pin versions) will yield 0x41 in the result phase of this command Any future enhancements on these parts will be denoted by the 5 LSBs (0x01 to 0x1F) It doesn't even define what 0x01 would mean. Kevin
Hi, Kevin Wolf a écrit : > Am 15.01.2012 08:51, schrieb Hervé Poussineau: >> Stepping 1 (S82078B) is not fully i82078 compatible, so better stick to initial revision >>> > Hm, this is the kind of change that I'm hesitant to make because I don't > understand the implications. Can you give some more details what is > fixed by this and why you think it's harmless for currently working OSes? > I was thinking at comments like https://git.kernel.org/?p=linux/kernel/git/stable/linux-stable.git;a=blob;f=drivers/char/ftape/lowlevel/fdc-io.c;h=65c9d2ec60bdd7221a30a776fba377c37b89c078;hb=refs/heads/linux-2.6.18.y#l1119 Anyway, I didn't see any real problem yet with current code. I'll resend this serie without this patch. Regards Hervé
diff --git a/hw/fdc.c b/hw/fdc.c index bedaeca..0e167f8 100644 --- a/hw/fdc.c +++ b/hw/fdc.c @@ -1385,7 +1385,7 @@ static void fdctrl_handle_version(FDCtrl *fdctrl, int direction) static void fdctrl_handle_partid(FDCtrl *fdctrl, int direction) { - fdctrl->fifo[0] = 0x41; /* Stepping 1 */ + fdctrl->fifo[0] = 0x01; /* Stepping 0 */ fdctrl_set_fifo(fdctrl, 1, 0); }
Stepping 1 (S82078B) is not fully i82078 compatible, so better stick to initial revision Signed-off-by: Hervé Poussineau <hpoussin@reactos.org> --- hw/fdc.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)