Bug 230572 - src/sys/mips/nlm/usb_init.c:55: strange bitmask ?
Summary: src/sys/mips/nlm/usb_init.c:55: strange bitmask ?
Status: Closed FIXED
Alias: None
Product: Base System
Classification: Unclassified
Component: kern (show other bugs)
Version: CURRENT
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-bugs (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-08-12 20:29 UTC by David Binderman
Modified: 2019-02-03 08:53 UTC (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description David Binderman 2018-08-12 20:29:12 UTC 
Source code is

        val = USB_CTRL_INTERRUPT_EN  | USB_OHCI_INTERRUPT_EN |
                USB_OHCI_INTERRUPT1_EN | USB_CTRL_INTERRUPT_EN  |
                USB_OHCI_INTERRUPT_EN | USB_OHCI_INTERRUPT2_EN;
Comment 1 commit-hook freebsd_committer freebsd_triage 2019-01-25 19:37:23 UTC 
A commit references this bug:

Author: gonzo
Date: Fri Jan 25 19:36:21 UTC 2019
New revision: 343443
URL: https://svnweb.freebsd.org/changeset/base/343443

Log:
  [mips] remove dublicate values in enable mask in nlm_usb_intr_en

  PR:		230572
  Submitted by:	David Binderman <dcb314@hotmail.com>
  MFC after:	1 week

Changes:
  head/sys/mips/nlm/usb_init.c
Comment 2 Oleksandr Tymoshenko freebsd_committer freebsd_triage 2019-01-25 19:41:57 UTC 
Fix committed. Thanks for submitting the report.
Comment 3 commit-hook freebsd_committer freebsd_triage 2019-02-03 08:53:40 UTC 
A commit references this bug:

Author: gonzo
Date: Sun Feb  3 08:53:04 UTC 2019
New revision: 343709
URL: https://svnweb.freebsd.org/changeset/base/343709

Log:
  MFC r343443, r343446, r343448, r343452

  r343443:
  [mips] remove dublicate values in enable mask in nlm_usb_intr_en

  PR:		230572
  Submitted by:	David Binderman <dcb314@hotmail.com>

  r343446:
  [mips] remove check that is always false (unsinged < 0)

  cpuid and local cpu variable are unsigned so checking if value is less than zero
  always yields false.

  PR:		211088
  Submitted by:	David Binderman <dcb314@hotmail.com>

  r343448:
  [mips] Fix counter mask in jz4780 timer driver

  Fix dublicate value in what is apparent copypaste mistake. The last value
  in mask is supposed to be for counter 7, not counter 3.

  PR:		229790
  Submitted by:	David Binderman <dcb314@hotmail.com>

  r343452:
  [mips] Fix error condition check that always evaluates to false

  Use proper logical operand when checking the value of srcid

  PR:		200988
  Submitted by:	David Binderman <dcb314@hotmail.com>

Changes:
_U  stable/12/
  stable/12/sys/mips/ingenic/jz4780_timer.c
  stable/12/sys/mips/mips/tlb.c
  stable/12/sys/mips/nlm/cms.c
  stable/12/sys/mips/nlm/usb_init.c