cvs commit: src/sys/dev/fdc fdc.c fdc_acpi.c fdc_isa.c fdcvar.h

[Warner Losh]

imp         2004-09-17 18:50:39 UTC

  FreeBSD src repository

  Modified files:
    sys/dev/fdc          fdc.c fdc_acpi.c fdc_isa.c fdcvar.h 
  Log:
  The long dead hand of the past has pushed forward useless bits in this
  driver.  Trim its fingernails by removing some useless bits before
  fixing the 'thread not terminated on detach' problem.
  
  o dmacnt is no longer used now that we allocate at attach time.  Remove
    it from struct fdc_data.
  o ISPNP was only ever set, but never tested.  It used to be used for the
    allocation routines to change how it allocated resources.  Since that's
    no longer necessary, retire the flag.
  o ISPCMICA was only ever tested, but never set.  GC it.  This removes
    a special case in determining the drive type.  The drive type is
    now set in fdc_pcmcia.c, so the hack isn't needed anymore.  Sadly,
    this isn't tested with a Y-E Data pcmcia floppy drive because there
    are a number of other issues that preclude it from working.
  o Fix ifdef for reading from the rtc.  I'm of the opinion that this ifdef
    should be moved into fdc_isa.c, but not today as ideally there'd be
    other fixes to the probing of children.  So now we just read it on
    i386 ! pc98 (there's no #define for MACHINE_ARCH, just MACHINE, hence
    this slightly inelegant kludge) and amd64.  The PC98 exclusion likely
    isn't meaningful since pc98 uses a different driver, but will be when
    merging of the pc98 floppy code into this driver is complete (this is the
    other reason I think this block of code belongs outside fdc.c).
  
  All of these changes are safe to MT5.
  
  Revision  Changes    Path
  1.290     +4 -20     src/sys/dev/fdc/fdc.c
  1.9       +0 -1      src/sys/dev/fdc/fdc_acpi.c
  1.14      +0 -4      src/sys/dev/fdc/fdc_isa.c
  1.6       +0 -3      src/sys/dev/fdc/fdcvar.h