1. 17 Nov, 2016 9 commits
  2. 16 Nov, 2016 7 commits
    • Arnd Bergmann's avatar
      staging: vc04_services: clarify firmware dependency · 6fde3789
      Arnd Bergmann authored
      The raspberrypi-firmware driver may be built as a loadable module,
      which causes a link-time failure if the vc04_services driver is
      built-in during compile-testing:
      
      drivers/staging/vc04_services/vchiq.o: In function `vchiq_probe':
      vchiq_connected.c:(.text.vchiq_probe+0x2c): undefined reference to `rpi_firmware_get'
      drivers/staging/vc04_services/vchiq.o: In function `vchiq_platform_init':
      vchiq_connected.c:(.text.vchiq_platform_init+0x1f0): undefined reference to `rpi_firmware_property'
      
      This extends the dependency list to ensure the firmware is either
      reachable, or completely disabled in case of compile-testing.
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      6fde3789
    • Arnd Bergmann's avatar
      staging: vc04_services: remove duplicate mutex_lock_interruptible · b826d73b
      Arnd Bergmann authored
      The driver tries to redefine mutex_lock_interruptible as an open-coded
      mutex_lock_killable, but that definition clashes with the normal
      mutex_lock_interruptible definition when CONFIG_DEBUG_LOCK_ALLOC
      is set:
      
      staging/vc04_services/interface/vchiq_arm/vchiq_killable.h:67:0: error: "mutex_lock_interruptible" redefined [-Werror]
       #define mutex_lock_interruptible mutex_lock_interruptible_killable
      include/linux/mutex.h:161:0: note: this is the location of the previous definition
      
      This simply removes the private implementation and uses the
      normal mutex_lock_killable directly.
      
      We could do the same for the down_interruptible_killable here, but
      it's better to just remove the semaphores entirely from the driver,
      which also takes care of that.
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      b826d73b
    • Arnd Bergmann's avatar
      staging: wilc1000: simplify vif[i]->ndev accesses · 735bb39c
      Arnd Bergmann authored
      With gcc-7, I got a new warning for this driver:
      
      wilc1000/linux_wlan.c: In function 'wilc_netdev_cleanup':
      wilc1000/linux_wlan.c:1224:15: error: 'vif[1]' may be used uninitialized in this function [-Werror=maybe-uninitialized]
      wilc1000/linux_wlan.c:1224:15: error: 'vif[0]' may be used uninitialized in this function [-Werror=maybe-uninitialized]
      
      A closer look at the function reveals that it's more complex than
      it needs to be, given that based on how the device is created
      we always get
      
      	netdev_priv(vif->ndev) == vif
      
      Based on this assumption, I found a few other places in the same file
      that can be simplified. That code appears to be a relic from times
      when the assumption above was not valid.
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      735bb39c
    • Greg Kroah-Hartman's avatar
      Merge tag 'iio-for-4.10c' of... · 2c52b1ef
      Greg Kroah-Hartman authored
      Merge tag 'iio-for-4.10c' of git://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio into staging-next
      
      Jonathan writes:
      
      Third set of IIO new device support, features and cleanup for the 4.10 cycle.
      
      Includes Peter Rosin's interesting drivers for a comparator. First complex
      use we have had with an analog front end made from discrete components.
      
      Brian Masney's work on moving the tsl2583 driver out of staging also
      feature extensively!
      
      New Drivers
      * DAC based on a digital potentiometer
        - New driver for the use of a dpot as a DAC. Includes bindings and Axentia
        entry in vendor prefixes.
      * Envelope detector baed on DAC and a comparator including device tree
        bindings.
      
      Staging Graduation
      * tsl2583.
      
      Core new features
      - Core provision for _available attributes.   This one had been stalled for
        a long time until Peter picked it up and ran with it!
      - In kernel interface helpers to retrieve available info from channels.
      
      Driver new features
      * mcp4531
        - Add range of available raw values (used for the dpot dac driver).
      
      Driver cleanups and fixes for issues introduced
      * ad7766
        - Testing the wrong variable following devm_regulator_bulk_get introduced
        with the driver earlier in this cycle.
      * ad9832
        - Fix a wrong ordering in the probe introduced in the previous set of
          patches.  A use before allocation bug.
      * cros_ec_sensors
        - Testing for an error in a u8 will never work.
      * mpu3050
        - Remove duplicate initializer for the module owner.
        - Add missing i2c dependency.
        - Inform the i2c mux core how it is used - step one in implifying device
        tree bindings.
      * st-sensors
        - Get rid of large number of uninformative defines in favour of putting the
        constants where they are relevant. It is clear what they are from where
        they are used.
      * tsl2583
        - Fix unused function warning when CONFIG_PM disabled and remove the
        ifdefs in favour of __maybe_unused.
        - Refactor taos_chip_on  to only read relevant registers.
        - Make sure calibscale and integration time are being set.
        - Verify chip is in ready to be used before calibration.
        - Remove some repeated checks for chip status (it's protected by a mutex
        so can't change until it's released)
        - Change current state storage from a tristate enum to a boolean seeing as
        only two values are actually used now.
        - Drop a redundant write to the control regiser in taos_probe (it's a noop)
        - Drop the FSF mailing address.
        - Clean up logging to not use hard coded function names (use __func__
        instead).
        - Cleanup up variable and function name prefixes.
        - Alignment of #define fixes.
        - Fix comparison between signed and unsigned integer warnings.
        - Add some newlines in favour of readability.
        - Combine the two sysfs ABI docs that somehow ended up in different places.
        - Fix multiline comment syntax.
        - Move a code block to inside an else statement as it makes more sense there.
        - Change tsl2583_als_calibrate to return 0 rather than a value nothing
        reads.
        - Drop some pointless brackets
        - Don't assume 32bit unsigned int.
        - Change to a per device instance lux table.
        - Add missing tsl2583 to the list of supported devices in the intro comments.
        - Improve commment on clearing of interrupts.
        - Drop some uninformative comments.
        - Drop a memset call that doesn't do anything useful any more.
        - Don't initialize some return variables that are always set.
        - Add Brian Masney as a module author after all these changes.
      2c52b1ef
    • Punit Vara's avatar
      Staging: ks7010: Use preffered kernel types · 81710951
      Punit Vara authored
      Replace uint8_t, uint16_t and uint32_t with preferred kernel types
      u8, u16 and u32 respectively suggested by checkpatch.pl
      Signed-off-by: default avatarPunit Vara <punitvara@gmail.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      81710951
    • Walt Feasel's avatar
      staging: xgifb: vb_table.h Preferred space around · ef50db63
      Walt Feasel authored
      Make suggested modification from checkpatch in reference
      to: CHECK: spaces preferred around that '+'
      Signed-off-by: default avatarWalt Feasel <waltfeasel@gmail.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      ef50db63
    • Walt Feasel's avatar
      staging: xgifb: vb_table.h Blank line after declarations · 9703cad4
      Walt Feasel authored
      Make suggested modification from checkpatch in reference
      to: CHECK: Please use a blank line after
      function/struct/union/enum declarations
      Signed-off-by: default avatarWalt Feasel <waltfeasel@gmail.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      9703cad4
  3. 15 Nov, 2016 7 commits
  4. 14 Nov, 2016 17 commits