- 28 1月, 2017 3 次提交
-
-
由 Maxime Rossi Bellom 提交于
Replace a mix of tabs and spaces indentation by tabs only. Errors were reported by checkpatch.pl as WARNING: Statements should start on a tabstop WARNING: suspect code indent for conditional statement Signed-off-by: NMaxime Rossi Bellom <mrossibellom@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 William Blough 提交于
Convert leading spaces to tabs Signed-off-by: NWilliam Blough <devel@blough.us> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 sayli karnik 提交于
This patch resolves the checkpatch issue: multiple assignments should be avoided Signed-off-by: Nsayli karnik <karniksayli1995@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
- 27 1月, 2017 13 次提交
-
-
由 Michael Zoran 提交于
Initial cleanup of bcm2835-audio driver for the bcm2535(Raspberry PI) Driver provides HDMI audio through ALSA and is built on top of the vc04_services driver. Original version of the driver is available at: http://www.github.com/raspberry/linux Driver compiles without any build errors or warnings. Tested on a RPI 3 running in ARM64 mode with the vlc player and alsautils. Signed-off-by: NMichael Zoran <mzoran@crowfest.net> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Colin Vidal 提交于
Fields frag_size and playload_size of struct ieee80211_txb are encoded as short little-endian. This patch adds conversions to / from cpu byte order when copy / write these values in variables of architecture independent byte order. It also avoid a sparse type warning. Signed-off-by: NColin Vidal <colin@cvidal.org> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 simran singhal 提交于
This patch fixes the checkpatch issue by adding space after ','. Signed-off-by: Nsimran singhal <singhalsimran0@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 simran singhal 提交于
This patch fixes the checkpatch issue by adding space around '='. Signed-off-by: Nsimran singhal <singhalsimran0@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Fix multiple checkpatch.pl warnings: function definition argument '...' should also have an identifier name Signed-off-by: NSimon Sandström <simon@nikanor.nu> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Ernestas Kulik 提交于
Using octal permissions instead of symbolic ones is preferred. Signed-off-by: NErnestas Kulik <ernestas.kulik@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Ernestas Kulik 提交于
Using octal permissions instead of symbolic ones is preferred. Signed-off-by: NErnestas Kulik <ernestas.kulik@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Arnd Bergmann 提交于
The newly added function triggers a harmless warning: drivers/staging/lustre/lustre/ptlrpc/pack_generic.c: In function 'lustre_shrink_msg': drivers/staging/lustre/lustre/ptlrpc/pack_generic.c:472:1: error: control reaches end of non-void function [-Werror=return-type] This probably happens because LASSERTF() contains an 'unlikely()' that sometimes prevents gcc from analysing the control flow correctly. Adding a return statement here seems harmless and lets us keep that unlikely(). Fixes: 96049bd1 ("staging: lustre: ptlrpc: embed highest XID in each request") Signed-off-by: NArnd Bergmann <arnd@arndb.de> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Sebastian Haas 提交于
Replace symbolic permissions with their octect representation to fix checkpatch warnings. Signed-off-by: NSebastian Haas <sehaas@deebas.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Umang Raghuvanshi 提交于
Fix the following formatting issues: CHECK: Alignment should match open parenthesis Signed-off-by: NUmang Raghuvanshi <u@umangis.me> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Johan Hovold 提交于
The loopback driver allows the user to set a minimum delay of up to one second to be inserted between test iterations (i.e. request submissions). The delay is currently specified in microseconds and is implemented using udelay. Busy looping for long periods is not just anti-social; udelay must not be used for delays longer than a few milliseconds due to the risk of integer overflow. Replace the broken udelay with a usleep_range with a 100 us range for short delays (< 20 ms) and otherwise revert to using msleep. Fixes: b36f04fa ("greybus: loopback: Convert thread delay to microseconds") Signed-off-by: NJohan Hovold <johan@kernel.org> Cc: stable <stable@vger.kernel.org> # 4.9+ Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Franck Demathieu 提交于
It fixes the following issue reported by checkpatch.pl: Prefer kernel type 'u32' over 'uint32_t' Signed-off-by: NFranck Demathieu <fdemathieu@gmail.com> Acked-by: NRui Miguel Silva <rmfrfs@gmail.com> Acked-by: NViresh Kumar <viresh.kumar@linaro.org> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Johan Hovold 提交于
This driver does not have a low-latency mode and should not report anything else. Also drop the skip-test flag which isn't used either. Signed-off-by: NJohan Hovold <johan@kernel.org> Reviewed-by: NViresh Kumar <viresh.kumar@linaro.org> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
- 25 1月, 2017 3 次提交
-
-
由 Abdul Rauf 提交于
Fix the following errors: Use 4 digit octal (0777) not decimal permissions Signed-off-by: NAbdul Rauf <abdulraufmujahid@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Joseph Wright 提交于
Found by checkpatch: - s/cacluated/calculated/ - s/convertor/converter/ - s/psudo/pseudo/ - s/halfs/halves/ Signed-off-by: NJoseph Wright <rjosephwright@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Amitesh Singh 提交于
There are multiple entries for ili9225 display defined in flexfb_chip_table array. remove duplicate entries and keep single entry. Signed-off-by: NAmitesh Singh <singh.amitesh@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
- 23 1月, 2017 14 次提交
-
-
由 Sudip Mukherjee 提交于
Modify lirc_parallel driver to use the new parallel port device model. Signed-off-by: NSudip Mukherjee <sudip.mukherjee@codethink.co.uk> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 James Simmons 提交于
Modify all the msg union fields from typedefs to to proper structures. Signed-off-by: NJames Simmons <uja.ornl@yahoo.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6142 Reviewed-on: https://review.whamcloud.com/24566Reviewed-by: NDoug Oucharek <doug.s.oucharek@intel.com> Reviewed-by: NOlaf Weber <olaf@sgi.com> Reviewed-by: NAndreas Dilger <andreas.dilger@intel.com> Reviewed-by: NDmitry Eremin <dmitry.eremin@intel.com> Signed-off-by: NJames Simmons <jsimmons@infradead.org> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 James Simmons 提交于
Change lnet_hdr_t from typedef to proper structure. Signed-off-by: NJames Simmons <uja.ornl@yahoo.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6142 Reviewed-on: https://review.whamcloud.com/24566Reviewed-by: NDoug Oucharek <doug.s.oucharek@intel.com> Reviewed-by: NOlaf Weber <olaf@sgi.com> Reviewed-by: NAndreas Dilger <andreas.dilger@intel.com> Reviewed-by: NDmitry Eremin <dmitry.eremin@intel.com> Signed-off-by: NJames Simmons <jsimmons@infradead.org> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 James Simmons 提交于
Change lnet_magicversion_t from typedef to proper structure. Signed-off-by: NJames Simmons <uja.ornl@yahoo.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6142 Reviewed-on: https://review.whamcloud.com/24566Reviewed-by: NDoug Oucharek <doug.s.oucharek@intel.com> Reviewed-by: NOlaf Weber <olaf@sgi.com> Reviewed-by: NAndreas Dilger <andreas.dilger@intel.com> Reviewed-by: NDmitry Eremin <dmitry.eremin@intel.com> Signed-off-by: NJames Simmons <jsimmons@infradead.org> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 James Simmons 提交于
Change lnet_acceptor_connreq_t from typedef to proper structure. Signed-off-by: NJames Simmons <uja.ornl@yahoo.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6142 Reviewed-on: https://review.whamcloud.com/24566Reviewed-by: NDoug Oucharek <doug.s.oucharek@intel.com> Reviewed-by: NOlaf Weber <olaf@sgi.com> Reviewed-by: NAndreas Dilger <andreas.dilger@intel.com> Reviewed-by: NDmitry Eremin <dmitry.eremin@intel.com> Signed-off-by: NJames Simmons <jsimmons@infradead.org> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 James Simmons 提交于
Change lnet_process_id_packed_t from typedef to proper structure. Signed-off-by: NJames Simmons <uja.ornl@yahoo.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6142 Reviewed-on: https://review.whamcloud.com/24566Reviewed-by: NDoug Oucharek <doug.s.oucharek@intel.com> Reviewed-by: NOlaf Weber <olaf@sgi.com> Reviewed-by: NAndreas Dilger <andreas.dilger@intel.com> Reviewed-by: NDmitry Eremin <dmitry.eremin@intel.com> Signed-off-by: NJames Simmons <jsimmons@infradead.org> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 James Simmons 提交于
Change lnet_ni_status_t from typedef to proper structure. Signed-off-by: NJames Simmons <uja.ornl@yahoo.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6142 Reviewed-on: https://review.whamcloud.com/24566Reviewed-by: NDoug Oucharek <doug.s.oucharek@intel.com> Reviewed-by: NOlaf Weber <olaf@sgi.com> Reviewed-by: NAndreas Dilger <andreas.dilger@intel.com> Reviewed-by: NDmitry Eremin <dmitry.eremin@intel.com> Signed-off-by: NJames Simmons <jsimmons@infradead.org> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 James Simmons 提交于
Change lnet_ping_info_t from typedef to proper structure. Signed-off-by: NJames Simmons <uja.ornl@yahoo.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6142 Reviewed-on: https://review.whamcloud.com/24566Reviewed-by: NDoug Oucharek <doug.s.oucharek@intel.com> Reviewed-by: NOlaf Weber <olaf@sgi.com> Reviewed-by: NAndreas Dilger <andreas.dilger@intel.com> Reviewed-by: NDmitry Eremin <dmitry.eremin@intel.com> Signed-off-by: NJames Simmons <jsimmons@infradead.org> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 James Simmons 提交于
Change lnet_handle_wire_t from typedef to proper structure. Signed-off-by: NJames Simmons <uja.ornl@yahoo.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6142 Reviewed-on: https://review.whamcloud.com/24566Reviewed-by: NDoug Oucharek <doug.s.oucharek@intel.com> Reviewed-by: NOlaf Weber <olaf@sgi.com> Reviewed-by: NAndreas Dilger <andreas.dilger@intel.com> Reviewed-by: NDmitry Eremin <dmitry.eremin@intel.com> Signed-off-by: NJames Simmons <jsimmons@infradead.org> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Marcos Paulo de Souza 提交于
Change uint32_t to u32, solved the issue reported by checkpatch.pl: CHECK: Prefer kernel type 'u32' over 'uint32_t' + uint32_t *format, uint32_t *rate, u8 *channels, CHECK: Prefer kernel type 'u32' over 'uint32_t' + uint32_t format, uint32_t rate, u8 channels, Signed-off-by: NMarcos Paulo de Souza <marcos.souza.org@gmail.com> Acked-by: NMark Greer <mgreer@animalcreek.com> Acked-by: NVaibhav Agarwal <vaibhav.sr@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Derek Robson 提交于
Fixed comparison, moved the constant to the right side of the test Found using checkpatch Signed-off-by: NDerek Robson <robsonde@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 simran singhal 提交于
This patch fixes the checkpatch issue by adding space around '='. Signed-off-by: Nsimran singhal <singhalsimran0@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Linus Walleij 提交于
commit 76222772 "iio: accel: st_accel: handle deprecated bindings" attempted to be smart and let users use the old I2C driver and the new SPI driver, or the old SPI driver and the new I2C driver in combination for the LIS3LV02, and put the restrictions on the I2C and SPI subdrivers not not be combined with the old subdrivers. This doesn't work since the IIO ST accel top-level component selects the I2C and SPI subdrivers, resulting in the following Kconfig noise: warning: (IIO_ST_ACCEL_3AXIS) selects IIO_ST_ACCEL_I2C_3AXIS which has unmet direct dependencies (IIO && !SENSORS_LIS3_I2C && IIO_ST_ACCEL_3AXIS && IIO_ST_SENSORS_I2C) warning: (IIO_ST_ACCEL_3AXIS) selects IIO_ST_ACCEL_SPI_3AXIS which has unmet direct dependencies (IIO && !SENSORS_LIS3_SPI && IIO_ST_ACCEL_3AXIS && IIO_ST_SENSORS_SPI) (...) This fixes the issue by putting the dependencies directly in the top-level component instead, so that it never gets to select its unselectable subcomponent. Fixes: 76222772 ("iio: accel: st_accel: handle deprecated bindings") Reported-by: NStephen Rothwell <sfr@canb.auug.org.au> Signed-off-by: NLinus Walleij <linus.walleij@linaro.org> Acked-by: NJonathan Cameron <jic23@kernel.org> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Alison Schofield 提交于
These stand-alone trigger drivers were using iio_trigger_put() where they should have been using iio_trigger_free(). The iio_trigger_put() adds a module_put which is bad since they never did a module_get. In the sysfs driver, module_get/put's are used as triggers are added & removed. This extra module_put() occurs on an error path in the probe routine (probably rare). In the bfin-timer & interrupt trigger drivers, the module resources are not explicitly managed, so it's doing a put on something that was never get'd. It occurs on the probe error path and on the remove path (not so rare). Tested with the sysfs trigger driver. The bfin & interrupt drivers were build tested & inspected only. Signed-off-by: NAlison Schofield <amsfield22@gmail.com> Signed-off-by: NJonathan Cameron <jic23@kernel.org>
-
- 22 1月, 2017 7 次提交
-
-
由 Dan Carpenter 提交于
1) Reverse the test for kmalloc() failure so we can pull everything back one tab. 2) Use gotos for unwinding. 3) Some of the extra line breaks for the 80 character limit are no longer needed so we can remove them. Signed-off-by: NDan Carpenter <dan.carpenter@oracle.com> Signed-off-by: NJonathan Cameron <jic23@kernel.org>
-
由 Bhumika Goyal 提交于
Declare iio_info structures as const as they are only stored in the info field of a iio_dev structure. This field is of type const, so iio_info structures having similar properties can be made const too. File size before: text data bss dec hex filename 6944 792 0 7736 1e38 drivers/iio/adc/ti-ads1015.o File size after: text data bss dec hex filename 7264 472 0 7736 1e38 drivers/iio/adc/ti-ads1015.o Signed-off-by: NBhumika Goyal <bhumirks@gmail.com> Signed-off-by: NJonathan Cameron <jic23@kernel.org>
-
由 Bhumika Goyal 提交于
Declare device_type structure as const as it is only stored in the type field of a device structure. This field is of type const, so add const to declaration of device_type structure. File size before: text data bss dec hex filename 5389 208 48 5645 160d iio/industrialio-trigger.o File size after: text data bss dec hex filename 5453 176 48 5677 162d iio/industrialio-trigger.o Signed-off-by: NBhumika Goyal <bhumirks@gmail.com> Signed-off-by: NJonathan Cameron <jic23@kernel.org>
-
由 Alison Schofield 提交于
In iio_trigger_write_current() we find the trigger we want while holding mutex on the list of triggers, but we don't actually do a get on it while holding mutex. We wait until further validations are completed and we're sure it's the one we want. Race condition is that it could be freed by the time we do the get. Solution is to grab the trigger (iio_trigger_get) as soon as we find it while holding mutex on the list of triggers. If later we decide it's not the right one, put it back. (iio_trigger_put). Signed-off-by: NAlison Schofield <amsfield22@gmail.com> Suggested-by: NLars-Peter Clausen <lars@metafoo.de> Signed-off-by: NJonathan Cameron <jic23@kernel.org>
-
由 Lorenzo Bianconi 提交于
Signed-off-by: NLorenzo Bianconi <lorenzo.bianconi@st.com> Signed-off-by: NJonathan Cameron <jic23@kernel.org>
-
由 Brian Masney 提交于
This patch adds runtime power management support to the isl29028 driver. It defaults to powering off the device after two seconds of inactivity. isl29028_chip_init_and_power_on() currently only zeros the CONFIGURE register on the chip, which will cause the chip to turn off. This patch also renames that function to isl29028_clear_configure_reg() since it is now used in several places. Signed-off-by: NBrian Masney <masneyb@onstation.org> Signed-off-by: NJonathan Cameron <jic23@kernel.org>
-
由 Brian Masney 提交于
isl29028_chip_init_and_power_on() calls isl29028_set_als_scale() and this is not needed until the user actually needs to take a reading from the ALS/IR sensor. This patch moves the isl29028_set_als_scale() call from isl29028_chip_init_and_power_on() to isl29028_set_als_ir_mode(). This sets the stage for faster resume times from runtime power management if the user is only querying the proximity sensor. Signed-off-by: NBrian Masney <masneyb@onstation.org> Signed-off-by: NJonathan Cameron <jic23@kernel.org>
-