GitLab

Add workaround for incorrect sizeof(usb_status_t), disable debug_set_stamp_debug(TRUE) to fix interrupt issue

Detail:
  build/c/ehcimodule, build/c/ohcimodule, build/c/usbmodule - Make sure that timestamped debug messages aren't used, as they appear to cause interrupts to be briefly enabled, which causes all kinds of problems in sensitive bits of the code.
  dev/usb/c/usbdi, dev/usb/c/usbdi_util, dev/usb/h/usb - Because DISABLE_PACKED is being used to compile the USB drivers, sizeof(usb_status_t) was being calculated as 4 bytes instead of 2. Somehow this works fine with the EHCI driver (and presumably the OHCI driver), but it was causing problems with the MUSBDriver, so a hardcoded size of 2 is now used.
Admin:
  Tested on rev C2 beagleboard.


Version 0.48. Tagged as 'NetBSD-0_48'

re-merged some items missed by Dan, and corrected usbmodule for 1 item missed earlier.. interrupt out endpoints noe demonstrated working

Detail:
Admin:  jb  Castle added IP


Version 0.45. Tagged as 'NetBSD-0_45'

Detail:
  There were many places where this was broken: the devicefs
interface was always starting off a read regardless of
whether it was an IN or OUT transfer, usbdi.c was setting
OUT interrupt transfers as repeating, which doesn't make sense,
and ohci.c was always making interrupt transfers OUT (this is
fixed in the latest BSD sources). Also the code which
unhalts endpoints at start of day has been removed, as this is
no longer in the BSD sources having been removed after it was
seend to have caused problems.

Admin:
  Tested with a CATC analyzer on an AverMedia FM radio. If now
operates properly with the code from Dave Higton.

Version 0.43. Tagged as 'NetBSD-0_43'

Added USB$Ignore_VVVV_PPPP variable.. if it exists then the internal hub/kbd/mouse checks are not done.

	it is left ONLY to external programs
Detail:
Admin:


Version 0.42. Tagged as 'NetBSD-0_42'

Detail:
	Devices with HID kbd and mouse in separate interfaces were
	not being correctly 'forgotten' when the device was claimed
	by a 3rd party App (!HID). This was because a re-entrancy issue
	led to the second claimed i/f being removed before the first
	was fully removed.
	This led to a muddle in the dev->subdevs array. Solution was to
	unlink it from the subdevs array before 'detaching' it,
	instead of after.
Admin:
	Castle added IP. Tested at Castle. Will be initially released via
	the beta test site.


Version 0.41. Tagged as 'NetBSD-0_41'

Detail:
	Includes several bug fixes and null pointer traps.
	Rhenium version reviewed and reset should be improved.
Admin:
	Tested in rhenium desktop build and iyonix build at castle
   	Castle added IP


Version 0.40. Tagged as 'NetBSD-0_40'

        Added (and fixed code underlying) to give periodic
        explore of USB busses (cf NetBSD).
        Added USBDisover command to let user provoke this too.
        Reworked USB device number allocation to roll
        around at 999 to avoid field overflow in
        a number of places.
Detail:
Admin:  castle added IP. to be beta tested on beta test site


Version 0.37. Tagged as 'NetBSD-0_37'

Detail:
         - VarVal launching.. string length was incorrectly calculated.
         - USB VarVals now correctly cleared if device failed during
           enumeration, after first SetVarVal call.
         - Triggercbs() modified to check interrupt context and be
           refuse (quietly) to trigger callbacks if interrupts threaded
           (result of possible issue .. but perhaps wise)
         - Added third QueryDelay variable, and an Info file to describe
           what does what.
Admin:
         checked at Castle.. due for extended check via beta test site
         before formal release.

         Castle + D Ellis added IP


Version 0.35. Tagged as 'NetBSD-0_35'

        USB2 further debugged.
Detail:
        Check in prior to second beta release. Improvements also to
        softloading of replacement modules.
Admin:
        Checked with JB .. further crosschecking is still needed for rhenium.
        Castle/D Ellis added ip from NetBSD sources


Version 0.33. Tagged as 'NetBSD-0_33'

Update of EHCIDriver to current NetBSD, together with needed changes in both OHCIDriver and USBDriver.

Detail:
        Code may change before formal release .. will be beta released
        ... Entered to CVS to get new version numbers, and for
        the record.
Admin:
        Castle/netbsd added IP .. will be tested more fully before
        formal release. Still to be verified on Rhenium.


Version 0.31. Tagged as 'NetBSD-0_31'

  'blocked' several holes that could cause freezes with awkward
  usb devices.  Added latest changes from Dan Ellis too , especially for USB2
Detail:
 many
Admin:
 Tested in ROM at Castle, and beta tested with users


Version 0.28. Tagged as 'NetBSD-0_28'

Version 0.27. Tagged as 'NetBSD-0_27'

Version 0.26. Tagged as 'NetBSD-0_26'

Detail:
  * All modules will now rmkill/rmload more reliably
  * Keyboard driver disables kernel debounce on new enough kernels, needed
    for many PS2-USB adaptors, bar code scanners etc
  * Mouse driver will scroll wheel scroll any windows that do respond to
    scroll events, otherwise it issues PointerReason_WheelChange (9) to an
    external WimpScroll module
  * Several bugs and an interrupt hole resolved
Admin:
  Tested at Framlingham.

Version 0.25. Tagged as 'NetBSD-0_25'

Version 0.23. Tagged as 'NetBSD-0_23'

Version 0.22. Tagged as 'NetBSD-0_22'

> Reduce root port reset delay to 50ms from 250ms to work with Chicony
> keyboards (again).  Comment out duplicate free(udev) at end of
> usbmodule:config_detach.

Version 0.20. Tagged as 'NetBSD-0_20'

USB transfer failed reason strings in the messages files were incorrectly being tokenised. Also added a couple of bits of debug.

Version 0.18. Tagged as 'NetBSD-0_18'

Added new CallDevice reason code (&80000005) to clear a stall condition on a pipe with a given FileSwitch handle

New usbdevs and usbdevs_data from NetBSD
Rewrote power source detection to skirt round dodgy devices which say
they're self powered,set MaxPower=0,and don't respond to a getstatus()
Copes better with devices being unplugged while a transfer using them is active

Version 0.17. Tagged as 'NetBSD-0_17'

Detail:
  * Errors now use the allocated error number base, &819000, instead of a
    base in the private range, &50000000. "USB transfer failed" errors now
    have use a sub-range of error numbers from &819020; the original BSD
    error number can be retrieved from the bottom 5 bits of the RISC OS
    error number.
  * Service_USB 1 now returns the list in the order in which the devices,
    rather than in the reverse order as previous versions of the module did.
  * A little tweaking of h/usb_port to reduce compiler warnings.
  * Some compiler-satisfying changes to allow use of packed structures.
    (However, the compiler doesn't seem to work correctly yet, so this is
    currently switched out using -DDISABLE_PACKED on the command line.)
  * Added full stops to end of star command help strings for consistency
    with rest of OS.
  * New exports: USBDevFS (moved here from ^.h) and a few files in dev/usb/h
    and sys/h.
Admin:
  Error number overhaul and...

Null vendor names no longer say oflaofla.

Version 0.14. Tagged as 'NetBSD-0_14'

Added stuff to delete the DeviceFS$USB*$Options variables when the module is
RMkilled.

Version 0.12. Tagged as 'NetBSD-0_12'

Fix for erratic behaviour after having unplugged hubs: TAILQ was being given an entry which wasn't on its list, and the behaviour was to break the list and poke a zero into the reset vector!

Also, some extra IFDEF's so that debug builds will work again.
A few typos/spelling errors changed.

Version 0.11. Tagged as 'NetBSD-0_11'

Before I forget,unset the DeviceFS$USBnnn$Options variable due to not reusing numbers and DeviceFS not deleting them - is there such thing as variable leaks?

Version 0.10. Tagged as 'NetBSD-0_10'

Version 0.09. Tagged as 'NetBSD-0_09'

Found a "busses" lingering in the messages file,also removed a \n which must have been copied from a printf somewhere.

Old service call detail removed from c.port as it's defined in another
file differently!
USBDiscover is now only available for debugging purposes.
Internationalised the mouse name (default is still "USB mouse").

Version 0.08. Tagged as 'NetBSD-0_08'

Another fix to the previous buffer overrun fix,such that this time the command aliases actually get executed now.

Timing code changed to work round HAL call not returning the right value
so temporarily wired to 2000000.
Change to Makefile so ROM builds don't have annoying throwback windows
popping up,this is now in the !Mk file.Also "-ff" function names away.

Version 0.05. Tagged as 'NetBSD-0_05'

Version 0.04. Tagged as 'NetBSD-0_04'

Minor bug fix,the system variable was getting run before the device was fully established in DeviceFS - which could cause printing problems.

Version 0.03. Tagged as 'NetBSD-0_03'

Correction of spelling of busses to buses (noun plural).
OHCIdriver only responds to PCI service call to lookup vendor name when
the device is of class OHCI controller.

Version 0.01. Tagged as 'NetBSD-0_01'