- 27 Oct, 2014 40 commits
-
-
Roberta Dobrescu authored
This patch removes uneeded return variables, just using 'return _SUCCESS' or 'return HCI_STATUS_SUCCESS' instead. This fixes the following warning detected using coccinelle: Unneeded variable. It was done using the following semantic patch: @@ identifier ret; type T; expression e; @@ -T ret = e; ... when != ret when strict -return ret; +return e; Signed-off-by:Roberta Dobrescu <roberta.dobrescu@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Roberta Dobrescu authored
This patch fixes the following warning detected using coccinelle: Unneeded semicolon. Signed-off-by:
-
Roberta Dobrescu authored
This patch removes unnecessary comparisons to bool. It fixes the following warning detected using coccinelle tool: WARNING: Comparison to bool. Signed-off-by:
-
Tina Ruchandani authored
This patch fixes the following checkpatch warning: ERROR: Macros with complex values should be enclosed in parentheses 1535: FILE: drivers/staging/rtl8723au/core/rtw_mlme.c:1535: Signed-off-by:
Tina Ruchandani <ruchandani.tina@gmail.com> Signed-off-by:
-
Jiayi Ye authored
Successive assignments to the same location is meaningless and can be deleted. The Coccinelle semantic patch was used to find cases. @@ expression e1,e2,e3; @@ ( (<+...e1++...+>)=e2; | (<+...e1--...+>)=e2; | (<+...++e1...+>)=e2; | (<+...--e1...+>)=e2; | e1=e2; e1 = <+...e1...+>; | *e1=e2; *e1=e3; ) Signed-off-by:
Jiayi Ye <yejiayily@gmail.com> Reviewed-by:
Josh Triplett <josh@joshtriplett.org> Signed-off-by:
-
Vaishali Thakkar authored
This patch replaces driver specific functions(RT_usb_endpoint_is_bulk_in, RT_usb_endpoint_is_bulk_out, RT_usb_endpoint_is_int_in) with USB API functions(usb_endpoint_is_bulk_in, usb_endpoint_is_bulk_out, usb_endpoint_is_int_in). Also, this patch removes these three RT functions as they are no longer needed after replacement. This patch also have one line over 80 characters as limiting it to 80 characters does not make code look good. Signed-off-by:
Vaishali Thakkar <vthakkar1994@gmail.com> Signed-off-by:
-
Benjamin Romer authored
Rename the struct to spar_vbus_channel_protocol. Fix CamelCase member names: ChannelHeader => channel_header HdrInfo => hdr_info ChpInfo => chp_info BusInfo => bus_info DevInfo => dev_info Update all references to use the changed names. Signed-off-by:
Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by:
-
Benjamin Romer authored
Remove the typedef and rename to struct spar_vbus_headerinfo. Fix CamelCase member names: structBytes => struct_bytes deviceInfoStructBytes => device_info_struct_bytes devInfoCount => dev_info_count chpInfoByteOffset => chp_info_offset busInfoByteOffset => bus_info_offset devInfoByteOffset => dev_info_offset Update all references to changed names. Signed-off-by:
-
Benjamin Romer authored
Fix the name to SPAR_VBUS_CHANNEL_OK_SERVER and fix the CamelCase parameter: actualBytes => actual_bytes Signed-off-by:
-
Benjamin Romer authored
Remove the unused logCtx parameter and fix the CamelCase name: pChannel => ch Update calls to the macro to remove the unused second parameter. Signed-off-by:
-
Benjamin Romer authored
Fix the CamelCase name: UltraVbusChannelProtocolGuid => spar_vbus_channel_protocol_uuid Update related macro names to match. Update references to changed names. Fix indentation in macros where names changed. Signed-off-by:
-
Benjamin Romer authored
The following macros and functions are unused, and should be deleted: NUMSIGNALS IO_CHANNEL_SIZE QSLOTSFROMBYTES QSIZEFROMBYTES SignalQInit INIT_CLIENTSTRING ULTRA_IO_CHANNEL_SERVER_READY ULTRA_IO_CHANNEL_SERVER_NOTREADY ULTRA_VHBA_init_channel ULTRA_VHBA_set_max ULTRA_VNIC_init_channel Signed-off-by:
-
Benjamin Romer authored
Remove the typedef from ULTRA_IO_CHANNEL_PROTOCOL and use the name struct spar_io_channel_protocol instead. Fix CamelCase member names: ChannelHeader => channel_header cmdQ => cmd_q rspQ => rsp_q zoneGuid => zone_uuid ClientString => client_string Update all references to changed names. Signed-off-by:
-
Benjamin Romer authored
Fix the CamelCase member of this structure: vHba => v_hba Signed-off-by:
-
Benjamin Romer authored
Fix CamelCase names: UniqueNum => unique_num RcvsDroppedDelta => rcvs_dropped_delta Signed-off-by:
-
Benjamin Romer authored
Fix the CamelCase name in this struct: UniqueNum => unique_num Signed-off-by:
-
Benjamin Romer authored
Fix CamelCase names: ErrorCode => error_code Valid => valid SegmentNumber => segment_number SenseKey => sense_key Reserved => reserved IncorrectLength => incorrect_length EndOfMedia => end_of_media FileMark => file_mark Information => information AdditionalSenseLength => additional_sense_length CommandSpecificInformation => commmand_specific_information AdditionalSenseCode => additional_sense_code AdditionalSenseCodeQualifier => additional_sense_code_qualifier FieldReplaceableUnitCode => fru_code SenseKeySpecific => sense_key_specific Signed-off-by:
-
Benjamin Romer authored
Get rid of the typedef and use enum vdisk_mgmt_types instead. Reformat the enumeration names and update any references to use the enum directly. Signed-off-by:
-
Benjamin Romer authored
Get rid of the typedef and use enum task_mgmt_types in its place. Clean up the indentation of the enumeration values and update all use of the name to the new name. Signed-off-by:
-
Benjamin Romer authored
Fix the CamelCase parameter in this macro: Address => address Signed-off-by:
-
Benjamin Romer authored
Take out the typedef and just use enum net_types instead. Signed-off-by:
-
Benjamin Romer authored
Nobody is using this one either... Signed-off-by:
-
Benjamin Romer authored
Remove the unused logCtx parameter and fix the CamelCase name: pChannel => ch Signed-off-by:
-
Benjamin Romer authored
This macro isn't being used so take it out. Signed-off-by:
-
Benjamin Romer authored
This macro isn't being used either, so delete it. Signed-off-by:
-
Benjamin Romer authored
This macro is not used so remove it. Signed-off-by:
-
Benjamin Romer authored
Remove the unused logCtx parameter and fix CamelCase parameter name: pChannel => ch Signed-off-by:
-
Benjamin Romer authored
Fix all the spaces between typecasts and their targets in iochannel.h. Signed-off-by:
-
Benjamin Romer authored
Get rid of extra blank lines in iochannel.h Signed-off-by:
-
Benjamin Romer authored
Get rid of the typedef and use struct spar_diag_channel_protocol instead. Fix CamelCase member names, and update references and comments to match. struct channel_header CommonChannelHeader; QueueHeader => queue_header DiagChannelHeader => diag_channel_header Events => events Signed-off-by:
-
Benjamin Romer authored
Get rid of the typedef and use struct diag_channel_protocol_header instead. Fix CamelCase member names and update references and comments. DiagLock => diag_lock IsChannelInitialized => channel_initialized Reserved => reserved SubsystemSeverityFilter => subsystem_severity_filter Signed-off-by:
-
Benjamin Romer authored
Remove the typedef from DIAG_CAUSE and just use enum diag_cause instead. Signed-off-by:
-
Benjamin Romer authored
Take out the extra blank lines in diagchannel.h. Signed-off-by:
-
Benjamin Romer authored
Get rid of the typedef for DIAG_SEVERITY and just use the enum diag_severity instead. Signed-off-by:
-
Benjamin Romer authored
Get rid of the typedef on DIAG_CHANNEL_EVENT, and use struct diag_channel_event instead. Update comments and fix CamelCase member names: EventId => event_id Severity => severity ModuleName => module_name LineNumber => line_number Timestamp => timestamp PartitionNumber => partition_number VirtualProcessorNumber => vcpu_number LogicalProcessorNumber => lcpu_number ComponentType => component_type Subsystem => subsystem Reserved0 => reserved0 BlockNumber => block_no BlockNumberHigh => block_no_high EventNumber => event_no EventNumberHigh => event_no_high Reserved => reserved AdditionalInfo => additional_info Signed-off-by:
-
Benjamin Romer authored
Get rid of the typedef for ULTRA_COMPONENT_TYPES and use enum spar_component_types instead. Signed-off-by:
-
Benjamin Romer authored
Get rid of the typedef and use struct diag_efi_time instead. Fix CamelCase member names and update references to changed names. Year => year Month => month Day => day Hour => hour Minute => minute Second => second Pad1 => pad1 Nanosecond => nanosecond TimeZone => timezone Daylight => daylight Pad2 => pad2 Signed-off-by:
-
Benjamin Romer authored
Rename this macro to SPAR_DIAG_CHANNEL_OK_SERVER and fix the CamelCase parameter. ActualBytes => bytes Fix indentation and add a blank line after the macro for better readability. Signed-off-by:
-
Benjamin Romer authored
Rename this macro to SPAR_DIAG_CHANNEL_OK_CLIENT, fix the CamelCase parameter and get rid of the unused LogCtx parameter. pChannel => ch Fix the indentation and add a blank line after the macro definition so it looks nicer. Signed-off-by:
-
Benjamin Romer authored
Fix the CamelCase name: UltraDiagChannelProtocolGuid => spar_diag_channel_protocol_uuid Also, rename ULTRA_DIAG_CHANNEL_PROTOCOL_GUID to SPAR_DIAG_CHANNEL_PROTOCOL_UUID to match the new variable name. Update all references to the changed names. Signed-off-by:
-
