summaryrefslogtreecommitdiffstats
path: root/drivers/staging/rts5208
Commit message (Collapse)AuthorAgeFilesLines
* staging: rts5208: rtsx.c: Fix invalid use of sizeof in rtsx_probe()Wei Yongjun2016-10-251-1/+1
| | | | | | | | | sizeof() when applied to a pointer typed expression gives the size of the pointer, not that of the pointed data. Fixes: 2eb9d8cbb3c3 ("staging: rts5208: rtsx.c: Alloc sizeof struct") Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: rtsx.c: Alignment fixWayne Porter2016-10-161-3/+4
| | | | | | | Line goes too long when aligned with parenthesis, so moved to a new line Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: rtsx_chip.c: Long linesWayne Porter2016-10-161-7/+10
| | | | | | | Fix lines that go over 80 characters to stop warnings from checkpatch Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: rtxs_chip.h: Long linesWayne Porter2016-10-161-6/+10
| | | | | | | Move comments above long definitions so they don't go over 80 characters Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: rtsx_scsi.h: Long linesWayne Porter2016-10-161-4/+6
| | | | | | | | Aligning with parenthesis causes lines to go too long, so the parenthesis are on a new line Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: CamelCase fixWayne Porter2016-10-161-2/+2
| | | | | | | Change label to not use camel case per checkpatch Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: ms.c: Braces fixWayne Porter2016-10-161-2/+2
| | | | | | | Add braces to all arms of if statement Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: ms.c: Multiple assignmentsWayne Porter2016-10-161-1/+2
| | | | | | | Fix check found by checkpatch Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: ms.c: Long LinesWayne Porter2016-10-161-31/+63
| | | | | | | Fix long lines detected by checkpatch Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: ms.c: Parenthesis alignmentWayne Porter2016-10-161-138/+150
| | | | | | | Fix checkpatch messages: Alignment should match open parenthesis Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: ms.h: Parenthesis alignmentWayne Porter2016-10-161-2/+2
| | | | | | | Fix alignment to match open parenthesis Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: rtsx.c: Alloc sizeof structWayne Porter2016-10-161-1/+1
| | | | | | | | Satisfy checkpatch message: Prefer kzalloc(sizeof(*dev->chip)...) over kzalloc(sizeof(struct rtsx_chip)...) Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: rtsx.c: SpacingWayne Porter2016-10-161-1/+0
| | | | | | | Remove blank line after opening { Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: rtsx.c: CamelCaseWayne Porter2016-10-161-4/+4
| | | | | | | Make camel case labels all lowercase Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: rtsx.c: SpacingWayne Porter2016-10-161-1/+4
| | | | | | | Add spaces around operator for readability Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: rtsx.c: Multiple assignmentsWayne Porter2016-10-161-2/+4
| | | | | | | Fix multiple assignments found by checkpatch Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: rtsx.c: AlignmentWayne Porter2016-10-161-2/+2
| | | | | | | Fix to match open parenthesis in order to satisfy checkpatch Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: rtsx.c: Unnecessary parenthesesWayne Porter2016-10-161-5/+5
| | | | | | | Remove parentheses found by checkpatch Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: Comparison to NULLWayne Porter2016-10-161-1/+1
| | | | | | | Fix to resolve checkpatch message Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: SpacingWayne Porter2016-10-161-1/+1
| | | | | | | | Adding space around operator for better readability and to stop checkpatch check messages Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: Unnecessary parenthesesWayne Porter2016-10-161-24/+24
| | | | | | | Remove parentheses found by checkpatch Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: Parenthesis alignmentWayne Porter2016-10-161-20/+22
| | | | | | | Fix alignment to stop checkpath check messages Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: Unecessary paranthesesWayne Porter2016-10-161-1/+1
| | | | | | | Remove parentheses to stop checkpatch check message Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: Parenthesis alignmentWayne Porter2016-10-161-7/+7
| | | | | | | Matching open parenthesis Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: Prefer using BIT macroWayne Porter2016-10-161-15/+15
| | | | | | | | Replace all instances of (1 << x) with BIT(x) to satisfy checkpatch check messages Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: Long lines fixesWayne Porter2016-10-161-28/+57
| | | | | | | Break up long lines in rtsx_chip.h Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: Alignment fixesWayne Porter2016-10-161-3/+3
| | | | | | | Fix alignment issues that checkpatch found Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: Add space around operatorWayne Porter2016-10-161-1/+1
| | | | | | | Spaces are preferred around operators for readability Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: Add braces to if()Wayne Porter2016-10-161-3/+4
| | | | | | | | Braces should be around every part of the if block and not just the else if Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: Remove multiple assignmentWayne Porter2016-10-161-4/+7
| | | | | | | | Assign values to variables on their own lines instead of using multiple assignment Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: CamelCase fixesWayne Porter2016-10-161-21/+21
| | | | | | | Change camel case variables found by checkpatch Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: Remove parenthesesWayne Porter2016-10-161-23/+23
| | | | | | | Unnecessary parentheses found by checkpatch Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: Alignment fixesWayne Porter2016-10-161-100/+101
| | | | | | | Align to match open parenthesis, found by checkpatch Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: Fix typo in function nameWayne Porter2016-10-161-2/+2
| | | | | | | | Searched the entire kernel for any references to this function and it appears safe to fix the typo Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: Alignment fixWayne Porter2016-10-161-2/+2
| | | | | | | Fix issue found by checkpatch Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: Unnecessary parentheses cleanupWayne Porter2016-10-161-2/+2
| | | | | | | Remove parentheses found by checkpatch Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: Alignment fixesWayne Porter2016-10-161-13/+17
| | | | | | | Cleaning up checkpatch issues Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: sd.c: Spacing cleanupWayne Porter2016-10-161-4/+3
| | | | | | | Add/remove spaces to make things more readable Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: sd.c: Fix logical continuationsWayne Porter2016-10-161-15/+15
| | | | | | | Checkpatch detected && at the beginning of new lines Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: sd.c: CamelCase fixesWayne Porter2016-10-161-84/+84
| | | | | | | Convert camel case lables to all lowercase Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: sd.c: Long line fixesWayne Porter2016-10-161-7/+15
| | | | | | | Break up lines over 80 characters Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: sd.c: Alignment fixesWayne Porter2016-10-161-237/+255
| | | | | | | Lining up with open parenthesis found by checkpatch Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: sd.c: Remove unnecessary parenthesesWayne Porter2016-10-161-47/+47
| | | | | | | | Cleanup of &(chip->sd_card) to remove parentheses where they are not needed Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: sd.h: Alignment fixesWayne Porter2016-10-161-2/+3
| | | | | | | Lining up code with open parenthses found by checkpatch Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: spi.c: Alignment fixesWayne Porter2016-10-161-65/+65
| | | | | | | Lining up code with open parantheses found by checkpatch Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: spi.c: Remove unnecessary parenthesisWayne Porter2016-10-161-7/+7
| | | | | | | Style warnings found by checkpatch Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: rts5208: Remove unnecessary parenthesesDilek Uzulmez2016-10-161-6/+6
| | | | | | | | Problem found using checkpatch.pl CHECK: Unnecessary parentheses around dev->dev_mutex Signed-off-by: Dilek Uzulmez <dilekuzulmez@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: xd.c: Remove multiple assignmentsWayne Porter2016-10-161-1/+2
| | | | | | | | Checkpatch found multiple assignments on one line so move them to their own lines. Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: xd.c: Fix CamelCaseWayne Porter2016-10-161-18/+18
| | | | | | | Covert CamelCase as checkpatch suggests Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rts5208: xd.c: Clean up comparison to NULLWayne Porter2016-10-161-5/+5
| | | | | | | Checkpatch recommended changes Signed-off-by: Wayne Porter <wporter82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
OpenPOWER on IntegriCloud