1. 17 May, 2016 22 commits
  2. 16 May, 2016 1 commit
  3. 15 May, 2016 1 commit
  4. 13 May, 2016 2 commits
  5. 12 May, 2016 1 commit
  6. 10 May, 2016 2 commits
  7. 07 May, 2016 2 commits
  8. 06 May, 2016 9 commits
    • Peng Fan's avatar
      usb: gadget: dfu: discard dead code · 12ff19db
      Peng Fan authored
      Reported by Coverity:
      Logically dead code (DEADCODE)
      dead_error_line: Execution cannot reach this statement:
      (f_dfu->strings + --i).s = ....
      
      If calloc failed, i is still 0 and no need to call free,
      so discard the dead code.
      Signed-off-by: default avatarPeng Fan <van.freenix@gmail.com>
      Cc: "Łukasz Majewski" <l.majewski@samsung.com>
      Cc: Marek Vasut <marex@denx.de>
      12ff19db
    • Peng Fan's avatar
      dfu: avoid memory leak · 5d8fae79
      Peng Fan authored
      When dfu_fill_entity fail, need to free dfu to avoid memory leak.
      
      Reported by Coverity:
      "
      Resource leak (RESOURCE_LEAK)
      leaked_storage: Variable dfu going out of scope leaks the storage
      it points to.
      "
      Signed-off-by: default avatarPeng Fan <van.freenix@gmail.com>
      Cc: "Łukasz Majewski" <l.majewski@samsung.com>
      Cc: Marek Vasut <marex@denx.de>
      5d8fae79
    • Stefan Roese's avatar
      usb: dwc2: Add delay to fix the USB detection problem on SoCFPGA · 2bf352f0
      Stefan Roese authored
      With patch c998da0d (usb: Change power-on / scanning timeout handling),
      the USB scanning is started earlier and with a smaller timeout. This
      resulted on SoCFPGA (using the DWC2 driver) in some USB sticks not
      getting detected any more. This patch now adds a 1 second delay (in
      the host mode only) to the DWC2 driver before the scanning is started.
      With this delay, now all problematic USB keys are detected successfully
      again. And there is no need any more to change the delay / timeout
      in the common USB code (usb_hub.c).
      Signed-off-by: default avatarStefan Roese <sr@denx.de>
      Cc: Chin Liang See <clsee@altera.com>
      Cc: Dinh Nguyen <dinguyen@opensource.altera.com>
      Cc: Hans de Goede <hdegoede@redhat.com>
      Cc: Stephen Warren <swarren@nvidia.com>
      Cc: Marek Vasut <marex@denx.de>
      2bf352f0
    • Marek Vasut's avatar
      usb: hub: Don't continue on get_port_status failure · d81db48d
      Marek Vasut authored
      The code shouldn't continue probing the port if get_port_status() failed.
      Signed-off-by: default avatarMarek Vasut <marex@denx.de>
      Cc: Chin Liang See <clsee@altera.com>
      Cc: Dinh Nguyen <dinguyen@opensource.altera.com>
      Cc: Hans de Goede <hdegoede@redhat.com>
      Cc: Stefan Roese <sr@denx.de>
      Cc: Stephen Warren <swarren@nvidia.com>
      d81db48d
    • Marek Vasut's avatar
      usb: Assure Get Descriptor request is in separate microframe · ef71290b
      Marek Vasut authored
      The Kingston DT Ultimate USB 3.0 stick is sensitive to this first
      Get Descriptor request and if the request is not in a separate
      microframe, the stick refuses to operate. Add slight delay, which
      is enough for one microframe to pass on any USB spec revision.
      Signed-off-by: default avatarMarek Vasut <marex@denx.de>
      Cc: Chin Liang See <clsee@altera.com>
      Cc: Dinh Nguyen <dinguyen@opensource.altera.com>
      Cc: Hans de Goede <hdegoede@redhat.com>
      Cc: Stefan Roese <sr@denx.de>
      Cc: Stephen Warren <swarren@nvidia.com>
      ef71290b
    • Marek Vasut's avatar
      usb: Wait after sending Set Configuration request · f647bf0b
      Marek Vasut authored
      Some devices, like the SanDisk Cruzer Pop need some time to process
      the Set Configuration request, so wait a little until they are ready.
      Signed-off-by: default avatarMarek Vasut <marex@denx.de>
      Cc: Chin Liang See <clsee@altera.com>
      Cc: Dinh Nguyen <dinguyen@opensource.altera.com>
      Cc: Hans de Goede <hdegoede@redhat.com>
      Cc: Stefan Roese <sr@denx.de>
      Cc: Stephen Warren <swarren@nvidia.com>
      f647bf0b
    • Anatolij Gustschin's avatar
      socfpga: fix broken build if CONFIG_ETH_DESIGNWARE disabled · 5289c5fa
      Anatolij Gustschin authored
      Building without ethernet driver doesn't work. Fix it.
      Signed-off-by: default avatarAnatolij Gustschin <agust@denx.de>
      Cc: Marek Vasut <marex@denx.de>
      5289c5fa
    • Marek Vasut's avatar
      mtd: cqspi: Simplify indirect read code · 5a824c49
      Marek Vasut authored
      The indirect read code is a pile of nastiness. This patch replaces
      the whole unmaintainable indirect read implementation with the one
      from upcoming Linux CQSPI driver, which went through multiple rounds
      of thorough review and testing. All the patch does is it plucks out
      duplicate ad-hoc code distributed across the driver and replaces it
      with more compact code doing exactly the same thing. There is no
      speed change of the read operation.
      Signed-off-by: default avatarMarek Vasut <marex@denx.de>
      Cc: Anatolij Gustschin <agust@denx.de>
      Cc: Chin Liang See <clsee@altera.com>
      Cc: Dinh Nguyen <dinguyen@opensource.altera.com>
      Cc: Jagan Teki <jteki@openedev.com>
      Cc: Pavel Machek <pavel@denx.de>
      Cc: Stefan Roese <sr@denx.de>
      Cc: Vignesh R <vigneshr@ti.com>
      5a824c49
    • Marek Vasut's avatar
      mtd: cqspi: Simplify indirect write code · 26da6353
      Marek Vasut authored
      The indirect write code is buggy pile of nastiness which fails horribly
      when the system runs fast enough to saturate the controller. The failure
      results in some pages (256B) not being written to the flash. This can be
      observed on systems which run with Dcache enabled and L2 cache enabled,
      like the Altera SoCFPGA.
      
      This patch replaces the whole unmaintainable indirect write implementation
      with the one from upcoming Linux CQSPI driver, which went through multiple
      rounds of thorough review and testing. While this makes the patch look
      terrifying and violates all best-practices of software development, all
      the patch does is it plucks out duplicate ad-hoc code distributed across
      the driver and replaces it with more compact code doing exactly the same
      thing.
      Signed-off-by: default avatarMarek Vasut <marex@denx.de>
      Cc: Anatolij Gustschin <agust@denx.de>
      Cc: Chin Liang See <clsee@altera.com>
      Cc: Dinh Nguyen <dinguyen@opensource.altera.com>
      Cc: Jagan Teki <jteki@openedev.com>
      Cc: Pavel Machek <pavel@denx.de>
      Cc: Stefan Roese <sr@denx.de>
      Cc: Vignesh R <vigneshr@ti.com>
      26da6353