1. 26 May, 2015 17 commits
  2. 22 May, 2015 1 commit
    • Arnd Bergmann's avatar
      usb: renesas_usbhs: avoid uninitialized variable use · 672bfdaa
      Arnd Bergmann authored
      After the renesas_usbhs driver is enabled in ARM multi_v7_defconfig,
      we now get a new warning:
      
      renesas_usbhs/mod.c: In function 'usbhs_interrupt':
      renesas_usbhs/mod.c:246:7: warning: 'intenb1' may be used uninitialized in this function [-Wmaybe-uninitialized]
      
      gcc correctly points to a problem here, for the case that the
      device is in host mode, we use the intenb1 variable without
      having assigned it first. The state->intsts1 has a similar
      problem, but gcc cannot know that.
      
      This avoids the problem by initializing both sides of the
      comparison to zero when we don't read them from the respective
      registers.
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Fixes: 88a25e02 ("usb: renesas_usbhs: Add access control for INTSTS1 and INTENB1 register")
      Acked-by: default avatarYoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
      Signed-off-by: default avatarFelipe Balbi <balbi@ti.com>
      672bfdaa
  3. 21 May, 2015 3 commits
  4. 14 May, 2015 1 commit
  5. 13 May, 2015 13 commits
  6. 07 May, 2015 5 commits
    • Bin Liu's avatar
      usb: musb: add softconnect for host mode · ffc1d299
      Bin Liu authored
      Add a debugfs interface - softconnect - for host mode to
      connect/disconnect the devices without physically remove the
      them.
      
      This adds the capability to re-enumerate the devices which are
      permanently mounted on the board with the MUSB controller
      together.
      Signed-off-by: default avatarBin Liu <b-liu@ti.com>
      Signed-off-by: default avatarFelipe Balbi <balbi@ti.com>
      ffc1d299
    • Andrzej Pietrasiewicz's avatar
      usb: gadget: rndis: remove the limit of available rndis connections · d6d22922
      Andrzej Pietrasiewicz authored
      RNDIS function has a limitation on the number of allowed instances.
      So far it has been RNDIS_MAX_CONFIGS, which happens to be one.
      In order to eliminate this kind of arbitrary limitation we should not
      preallocate a predefined (RNDIS_MAX_CONFIGS) array of struct rndis_params
      instances but instead allow allocating them on demand.
      
      This patch allocates struct rndis_params on demand in rndis_register().
      Coversly, the structure is free()'d in rndis_deregister().
      If CONFIG_USB_GADGET_DEBUG_FILES is set, the proc files are created which
      is the same behaviour as before, but the moment of creation is delayed
      until struct rndis_params is actually allocated.
      
      rnids_init() and rndis_exit() have nothing to do, so they are eliminated.
      Signed-off-by: default avatarAndrzej Pietrasiewicz <andrzej.p@samsung.com>
      Signed-off-by: default avatarFelipe Balbi <balbi@ti.com>
      d6d22922
    • Andrzej Pietrasiewicz's avatar
      usb: gadget: rndis: style correction · 6122b151
      Andrzej Pietrasiewicz authored
      Don't use a space between function name and parameter list opening bracket.
      All other functions in this file comply wich checkpatch rules.
      Signed-off-by: default avatarAndrzej Pietrasiewicz <andrzej.p@samsung.com>
      Signed-off-by: default avatarFelipe Balbi <balbi@ti.com>
      6122b151
    • Andrzej Pietrasiewicz's avatar
      usb: gadget: rndis: use rndis_params instead of configNr · 83210e59
      Andrzej Pietrasiewicz authored
      RNDIS function has a limitation on the number of allowed instances.
      So far it has been RNDIS_MAX_CONFIGS, which happens to be one.
      In order to eliminate this kind of arbitrary limitation we should not
      preallocate a predefined (RNDIS_MAX_CONFIGS) array of struct rndis_params
      instances but instead allow allocating them on demand.
      
      This patch prepares the elimination of the said limit by converting all the
      functions which accept rndis config number to accept a pointer to the
      actual struct rndis_params. Consequently, rndis_register() returns
      a pointer to a corresponding struct rndis_params instance. The pointer
      is then always used by f_rndis.c instead of config number when it talks
      to rndis.c API.
      
      A nice side-effect of the changes is that many lines of code in rndis.c
      become shorter and fit in 80 columns.
      
      If a function prototype changes in rndis.h a style cleanup is made
      at the same time, otherwise checkpatch complains that the patch
      has style problems.
      Signed-off-by: default avatarAndrzej Pietrasiewicz <andrzej.p@samsung.com>
      Signed-off-by: default avatarFelipe Balbi <balbi@ti.com>
      83210e59
    • Joe Perches's avatar
      usb: gadget: s3c2410_udc: Remove static char buffer, use vsprintf extension %pV · d74c23d3
      Joe Perches authored
      Using unnecessary static char buffers isn't good.
      Use the %pV extension instead.
      
      Miscellanea:
      
      o the dprintk return value is unused, make it void
      o add __printf format and argument verification
      Signed-off-by: default avatarJoe Perches <joe@perches.com>
      Signed-off-by: default avatarFelipe Balbi <balbi@ti.com>
      d74c23d3