aboutsummaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2016-03-28Staging: most: Remove atomic_counter_t typedefPrasannaKumar Muralidharan1-4/+2
Remove atomic_counter_t typedef, use int instead. Signed-off-by: PrasannaKumar Muralidharan <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28Staging: iio: ad5933: Remove unnecessary space after cast.Sandhya Bankar1-4/+4
Remove unnecessary space after cast. Signed-off-by: Sandhya Bankar <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28Staging: iio: ade7758: Fix open parentheses alignment issues.Sandhya Bankar1-7/+3
Fix open parentheses alignment issues. Signed-off-by: Sandhya Bankar <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28Staging: iio: ade7758: Use a blank line after function/struct declarations.Sandhya Bankar1-0/+6
Use a blank line after function/struct declarations. Signed-off-by: Sandhya Bankar <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28Staging: iio: ade7758_core: Remove unnecessary blank line.Sandhya Bankar1-1/+0
Remove unnecessary blank line. Signed-off-by: Sandhya Bankar <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28Staging: iio: ade7758_core: Fix open parentheses alignment issues.Sandhya Bankar1-47/+26
Fix open parentheses alignment issues. Signed-off-by: Sandhya Bankar <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: iio: ad5933: remove unused #includesAlison Schofield1-2/+0
Remove #includes no longer used in this module. Signed-off-by: Alison Schofield <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: iio: ad5933: move contents of header file to source fileAlison Schofield2-30/+12
The contents of the header file are used only by this single source file. Move content into .c and remove .h. Signed-off-by: Alison Schofield <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: iio: io-trig-bfin-timer: use dev_get_platdata()Alison Schofield1-1/+2
Use dev_get_platdata() for retrieving the platform data instead of accessing dev->platform_data directly. Move the assignment out of the declaration (avoid lines over 80 char and put it close to usage). Signed-off-by: Alison Schofield <[email protected]> Acked-by: Daniel Baluta <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: iio: ad5933: use dev_get_platdata()Alison Schofield1-1/+1
Use dev_get_platdata() for retrieving the platform data instead of accessing dev->platform_data directly. Signed-off-by: Alison Schofield <[email protected]> Acked-by: Daniel Baluta <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: iio: meter: remove fixme comment on device removeAlison Schofield3-3/+0
This comment was in place in the original drafts of these drivers when the remove function did a whole lot of work: flushed queues, unregistered interrupts, uninitialized rings, unconfigured rings, and a few kfree's. The remove functions have since been reduced to unregistering and stopping the device. This is the inverse of what was done during probe and is correct. Time to remove the comment. Signed-off-by: Alison Schofield <[email protected]> Acked-by: Daniel Baluta <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: iio: tsl2x7x_core: add blank line after struct declarationEva Rachel Retuya1-0/+1
Add the missing blank line after struct declaration. Checkpatch found this issue. CHECK: Please use a blank line after function/struct/union/enum declarations Signed-off-by: Eva Rachel Retuya <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: iio: tsl2x7x_core: add spaces around operatorsEva Rachel Retuya1-2/+2
Address the checkpatch check by adding the necessary whitespace around operators: CHECK: spaces preferred around that '/' (ctx:VxV) Signed-off-by: Eva Rachel Retuya <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: iio: tsl2x7x_core: remove space after a castEva Rachel Retuya1-2/+2
Delete unneeded space after a cast as suggested by checkpatch. CHECK: No space is necessary after a cast. Signed-off-by: Eva Rachel Retuya <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: iio: tsl2x7x_core: use preferred comment styleEva Rachel Retuya1-13/+25
Adjust block comments by adding the necessary trailing */ on a separate line. Checkpatch found this issue. Signed-off-by: Eva Rachel Retuya <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: iio: tsl2x7x_core: adjust alignment to match open parenthesisEva Rachel Retuya1-72/+92
Align the parameters to match open parenthesis. Issue detected using checkpatch. CHECK: Alignment should match open parenthesis Signed-off-by: Eva Rachel Retuya <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: unisys: visorbus: rename create_visor_device gotosDavid Kershner1-9/+9
Away is ambiguous when specifying error vs success. Make return labels more meaningful by marking them as error paths. Signed-off-by: David Kershner <[email protected]> Signed-off-by: Timothy Sell <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: unisys: visorbus: Remove gotos in visorbus_matchDavid Kershner1-14/+10
Gotos in visorbus_match are not needed. Signed-off-by: David Kershner <[email protected]> Signed-off-by: Timothy Sell <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: unisys: visorbus: fix up gotos in visorbus_initDavid Kershner1-13/+11
This patch fixes the gotos in visorbus_init Signed-off-by: David Kershner <[email protected]> Signed-off-by: Timothy Sell <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: unisys: visorbus: remove unused sysfs attribute devmajorminor/*Tim Sell3-216/+0
The sysfs attribute directory at: /sys/bus/visorbus/devices/vbus<x>:dev<y>/devmajorminor/* or /sys/devices/visorbus<x>/vbus<x>:dev<y>/devmajorminor/* previously provided a location where a visorbus function driver could publish information (for usermode use) about possibly-multiple major and minor device numbers for character devices created for a each visorbus device, using visorbus_registerdevnode(). This functionality is not currently used, so it has been removed by this cset. Signed-off-by: Tim Sell <[email protected]> Signed-off-by: David Kershner <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: unisys: visorbus: Fix up visordriver_probeDavid Kershner1-20/+12
Fixup the visordriver_probe function. Rearrange the function to avoid needing gotos and removed unnecessary wmb(). Signed-off-by: David Kershner <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: unisys: visorbus: fix my_create_device goto statementsDavid Kershner1-6/+6
Don't use the abmiguous cleanup, make it more meaningful. Signed-off-by: David Kershner <[email protected]> Signed-off-by: Timothy Sell <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: unisys: visorbus: Cleanup bus_epilog goto statementsDavid Kershner1-6/+12
Cleaned up bus_epilogs vague gotos and in the process discovered some error paths that could unlock a non locked semaphore. Signed-off-by: David Kershner <[email protected]> Signed-off-by: Timothy Sell <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: unisys: visorbus: Cleanup goto in bus_createDavid Kershner1-4/+4
Rename it to what it does instead of the default ambiguous cleanup. Signed-off-by: David Kershner <[email protected]> Signed-off-by: Timothy Sell <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: unisys: visorbus: chipset_init rename gotoDavid Kershner1-2/+2
Change the goto label "cleanup" to something more useful like out_respond. Signed-off-by: David Kershner <[email protected]> Signed-off-by: Timothy Sell <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: unisys: visorbus: cleanup gotos in parser_init_byte_streamDavid Kershner1-25/+14
Clean up the goto in parser_init_byte_stream and make the goto section the error case. Signed-off-by: David Kershner <[email protected]> Signed-off-by: Timothy Sell <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: unisys: visorbus: get rid of gotos in intialize_controlvm_payload_infoDavid Kershner1-21/+9
Get rid of the gotos in initialize_controlvm_payload_info. The check in the error path if payload was valid was never called so get rid of that as well. Signed-off-by: David Kershner <[email protected]> Signed-off-by: Timothy Sell <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: unisys: visorbus: cleanup goto in setup_crash_devices_work_queueDavid Kershner1-9/+5
If visorbus has registered yet just reschedule and exit. The rest of the function doesn't need to reschedule so just move it up to the initial check. Signed-off-by: David Kershner <[email protected]> Signed-off-by: Timothy Sell <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: lustre: cleanup comment style for lnet selftestJames Simmons8-48/+50
Apply a consistent style for comments in the lnet selftest code. Realign some of the comments to make it easier to read. This also fixes a few checkpatch issues as well. Signed-off-by: James Simmons <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: lustre: realign some code in lnet selftest so its readableJames Simmons2-3/+3
Two places to align the code so it is easier to read. Signed-off-by: James Simmons <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: lustre: remove excess blank lines in lnet selftest codeJames Simmons3-6/+0
Remove extra blank lines missed by checkpatch. Signed-off-by: James Simmons <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: lustre: handle error returned from wait_event_timeout seltest timerJames Simmons1-4/+6
The function wait_event_timeout can fail and return an error. Handle this case in stt_timer_main(). Signed-off-by: James Simmons <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: lustre: add missing buffer overflow fix for console.cDmitry Eremin1-1/+1
Patch 9389 change a strncpy call into a strlcpy call. This was missed in the merger into the upstream client. Signed-off-by: Dmitry Eremin <[email protected]> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-4629 Reviewed-on: http://review.whamcloud.com/9389 Reviewed-by: Andreas Dilger <[email protected]> Reviewed-by: Oleg Drokin <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: lustre: Correct missing newline for CERROR call in ↵James Nunez1-1/+1
sfw_handle_server_rpc This is one of the fixes broken out of patch 10000 that was missed in the merger. With this fix the CERROR called in sfw_handle_server_rpc will print out correctly. Signed-off-by: James Nunez <[email protected]> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-4871 Reviewed-on: http://review.whamcloud.com/10000 Reviewed-by: Andreas Dilger <[email protected]> Reviewed-by: John L. Hammond <[email protected]> Reviewed-by: Cliff White <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: lustre: lnet: Replace sg++ with sg = sg_next(sg)Amitoj Kaur Chawla1-2/+2
With scatterlist chaining, simply incrementing the array does not work. sg_next macro was thus introduced to follow the chain links when necessary. So replace sg++ with sg_next. This change was made with the help of the following Coccinelle semantic patch: //<smpl> @@ struct scatterlist *sg; @@ -sg++ +sg = sg_next(sg) //</smpl> Signed-off-by: Amitoj Kaur Chawla <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2016-03-28staging: iio: convert bare unsigned usage to unsigned intAlison Schofield9-61/+61
Use kernel preferred unsigned int declaration style. Patch created using: git ls-files drivers/staging/iio | \ xargs ./scripts/checkpatch.pl -f --fix-inplace --types=unspecified_int Hand edits restored columns in structure definitions. Signed-off-by: Alison Schofield <[email protected]> Signed-off-by: Jonathan Cameron <[email protected]>
2016-03-28iio: st_sensors: simplify buffer address handlingLinus Walleij1-21/+7
The driver goes to some length to dynamically allocate an array to hold the channel addresses. However no ST sensor has more than three channels (x, y, z at most). Instead of kmalloc():ing and kfree():in the address array, just use a fixed array of three elements. Cc: Giuseppe Barba <[email protected]> Signed-off-by: Linus Walleij <[email protected]> Acked-by: Denis Ciocca <[email protected]> Signed-off-by: Jonathan Cameron <[email protected]>
2016-03-26Linux 4.6-rc1Linus Torvalds1-2/+2
2016-03-26Merge branch 'for-linus' of ↵Linus Torvalds22-519/+811
git://git.kernel.org/pub/scm/linux/kernel/git/sage/ceph-client Pull Ceph updates from Sage Weil: "There is quite a bit here, including some overdue refactoring and cleanup on the mon_client and osd_client code from Ilya, scattered writeback support for CephFS and a pile of bug fixes from Zheng, and a few random cleanups and fixes from others" [ I already decided not to pull this because of it having been rebased recently, but ended up changing my mind after all. Next time I'll really hold people to it. Oh well. - Linus ] * 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/sage/ceph-client: (34 commits) libceph: use KMEM_CACHE macro ceph: use kmem_cache_zalloc rbd: use KMEM_CACHE macro ceph: use lookup request to revalidate dentry ceph: kill ceph_get_dentry_parent_inode() ceph: fix security xattr deadlock ceph: don't request vxattrs from MDS ceph: fix mounting same fs multiple times ceph: remove unnecessary NULL check ceph: avoid updating directory inode's i_size accidentally ceph: fix race during filling readdir cache libceph: use sizeof_footer() more ceph: kill ceph_empty_snapc ceph: fix a wrong comparison ceph: replace CURRENT_TIME by current_fs_time() ceph: scattered page writeback libceph: add helper that duplicates last extent operation libceph: enable large, variable-sized OSD requests libceph: osdc->req_mempool should be backed by a slab pool libceph: make r_request msg_size calculation clearer ...
2016-03-26Merge tag 'ofs-pull-tag-1' of ↵Linus Torvalds33-0/+11243
git://git.kernel.org/pub/scm/linux/kernel/git/hubcap/linux Pull orangefs filesystem from Mike Marshall. This finally merges the long-pending orangefs filesystem, which has been much cleaned up with input from Al Viro over the last six months. From the documentation file: "OrangeFS is an LGPL userspace scale-out parallel storage system. It is ideal for large storage problems faced by HPC, BigData, Streaming Video, Genomics, Bioinformatics. Orangefs, originally called PVFS, was first developed in 1993 by Walt Ligon and Eric Blumer as a parallel file system for Parallel Virtual Machine (PVM) as part of a NASA grant to study the I/O patterns of parallel programs. Orangefs features include: - Distributes file data among multiple file servers - Supports simultaneous access by multiple clients - Stores file data and metadata on servers using local file system and access methods - Userspace implementation is easy to install and maintain - Direct MPI support - Stateless" see Documentation/filesystems/orangefs.txt for more in-depth details. * tag 'ofs-pull-tag-1' of git://git.kernel.org/pub/scm/linux/kernel/git/hubcap/linux: (174 commits) orangefs: fix orangefs_superblock locking orangefs: fix do_readv_writev() handling of error halfway through orangefs: have ->kill_sb() evict the VFS side of things first orangefs: sanitize ->llseek() orangefs-bufmap.h: trim unused junk orangefs: saner calling conventions for getting a slot orangefs_copy_{to,from}_bufmap(): don't pass bufmap pointer orangefs: get rid of readdir_handle_s ornagefs: ensure that truncate has an up to date inode size orangefs: move code which sets i_link to orangefs_inode_getattr orangefs: remove needless wrapper around GFP_KERNEL orangefs: remove wrapper around mutex_lock(&inode->i_mutex) orangefs: refactor inode type or link_target change detection orangefs: use new getattr for revalidate and remove old getattr orangefs: use new getattr in inode getattr and permission orangefs: use new orangefs_inode_getattr to get size in write and llseek orangefs: use new orangefs_inode_getattr to create new inodes orangefs: rename orangefs_inode_getattr to orangefs_inode_old_getattr orangefs: remove inode->i_lock wrapper orangefs: put register_chrdev immediately before register_filesystem ...
2016-03-26Merge tag 'ntb-4.6' of git://github.com/jonmason/ntbLinus Torvalds4-70/+79
Pull NTB bug fixes from Jon Mason: "NTB bug fixes for tasklet from spinning forever, link errors, translation window setup, NULL ptr dereference, and ntb-perf errors. Also, a modification to the driver API that makes _addr functions optional" * tag 'ntb-4.6' of git://github.com/jonmason/ntb: NTB: Remove _addr functions from ntb_hw_amd NTB: Make _addr functions optional in the API NTB: Fix incorrect clean up routine in ntb_perf NTB: Fix incorrect return check in ntb_perf ntb: fix possible NULL dereference ntb: add missing setup of translation window ntb: stop link work when we do not have memory ntb: stop tasklet from spinning forever during shutdown. ntb: perf test: fix address space confusion
2016-03-26Merge tag 'scsi-misc' of git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsiLinus Torvalds22-166/+1177
Pull more SCSI updates from James Bottomley: "The only new stuff which missed the first pull request is an update to the UFS driver. The rest is an assortment of bug fixes and minor tweaks which appeared recently (some are fixes for recent code and some are stuff spotted recently by the checkers or the new gcc-6 compiler [most of Arnd's stuff])" * tag 'scsi-misc' of git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi: (32 commits) scsi_common: do not clobber fixed sense information scsi: ufs: select CONFIG_NLS scsi: fc: use get/put_unaligned64 for wwn access fnic: move printk()s outside of the critical code section. qla2xxx: avoid maybe_uninitialized warning megaraid_sas: add missing curly braces in ioctl handler lpfc: fix misleading indentation scsi_transport_sas: add 'scsi_target_id' sysfs attribute scsi_dh_alua: uninitialized variable in alua_check_vpd() scsi: ufs-qcom: add printouts of testbus debug registers scsi: ufs-qcom: enable/disable the device ref clock scsi: ufs-qcom: set PA_Local_TX_LCC_Enable before link startup scsi: ufs: add device quirk delay before putting UFS rails in LPM scsi: ufs: fix leakage during link off state scsi: ufs: tune UniPro parameters to optimize hibern8 exit time scsi: ufs: handle non spec compliant bkops behaviour by device scsi: ufs: add retry for query descriptors scsi: ufs: add error recovery after DL NAC error scsi: ufs: make error handling bit faster scsi: ufs: disable vccq if it's not needed by UFS device ...
2016-03-26f2fs/crypto: fix xts_tweak initializationLinus Torvalds1-1/+1
Commit 0b81d07790726 ("fs crypto: move per-file encryption from f2fs tree to fs/crypto") moved the f2fs crypto files to fs/crypto/ and renamed the symbol prefixes from "f2fs_" to "fscrypt_" (and from "F2FS_" to just "FS" for preprocessor symbols). Because of the symbol renaming, it's a bit hard to see it as a file move: use git show -M30 0b81d07790726 to lower the rename detection to just 30% similarity and make git show the files as renamed (the header file won't be shown as a rename even then - since all it contains is symbol definitions, it looks almost completely different). Even with the renames showing as renames, the diffs are not all that easy to read, since so much is just the renames. But Eric Biggers noticed that it's not just all renames: the initialization of the xts_tweak had been broken too, using the inode number rather than the page offset. That's not right - it makes the xfs_tweak the same for all pages of each inode. It _might_ make sense to make the xfs_tweak contain both the offset _and_ the inode number, but not just the inode number. Reported-by: Eric Biggers <[email protected]> Cc: Jaegeuk Kim <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2016-03-26NTB: Remove _addr functions from ntb_hw_amdAllen Hubbe1-30/+0
Kernel zero day testing warned about address space confusion. A virtual iomem address was used where a physical address is expected. The offending functions implement an optional part of the api, so they are removed. They can be added later, after testing. Fixes: a1b3695820aa490e58915d720a1438069813008b Signed-off-by: Allen Hubbe <[email protected]> Acked-by: Xiangliang Yu <[email protected]> Signed-off-by: Jon Mason <[email protected]>
2016-03-26orangefs: fix orangefs_superblock lockingAl Viro3-58/+47
* switch orangefs_remount() to taking ORANGEFS_SB(sb) instead of sb * remove from the list _before_ orangefs_unmount() - request_mutex in the latter will make sure that nothing observed in the loop in ORANGEFS_DEV_REMOUNT_ALL handling will get freed until the end of loop * on removal, keep the forward pointer and zero the back one. That way we can drop and regain the spinlock in the loop body (again, ORANGEFS_DEV_REMOUNT_ALL one) and still be able to get to the rest of the list. Signed-off-by: Al Viro <[email protected]> Signed-off-by: Mike Marshall <[email protected]>
2016-03-25orangefs: fix do_readv_writev() handling of error halfway throughAl Viro1-1/+1
Error should only be returned if nothing had been read/written. Otherwise we need to report a short read/write instead. Signed-off-by: Al Viro <[email protected]> Signed-off-by: Mike Marshall <[email protected]>
2016-03-25orangefs: have ->kill_sb() evict the VFS side of things firstAl Viro1-3/+3
Signed-off-by: Al Viro <[email protected]> Signed-off-by: Mike Marshall <[email protected]>
2016-03-25orangefs: sanitize ->llseek()Al Viro2-10/+3
a) open files can't have NULL inodes b) it's SEEK_END, not ORANGEFS_SEEK_END; no need to get cute. c) make_bad_inode() on lseek()? Signed-off-by: Al Viro <[email protected]> Signed-off-by: Mike Marshall <[email protected]>
2016-03-25orangefs-bufmap.h: trim unused junkAl Viro1-9/+0
Signed-off-by: Al Viro <[email protected]> Signed-off-by: Mike Marshall <[email protected]>
2016-03-25orangefs: saner calling conventions for getting a slotAl Viro4-28/+16
just have it return the slot number or -E... - the caller checks the sign anyway Signed-off-by: Al Viro <[email protected]> Signed-off-by: Mike Marshall <[email protected]>