usb: typec: nb7vpq904m: Remove unneeded indentation
Subsystems (2)
PATCH v6 1/1 usb: typec: nb7vpq904m: Remove unneeded indentation
In function nb7vpq904m_parse_data_lanes_mapping(), the "if (ep)" condition is basically the entire function. Making the code a bit more readable by inverting the condition so that the function returns immediately if there is no "ep".
---
Fixed indentation in common path according to comments received on other patch as below:
[ Nit, this function should be rewritten to not work like this, the "common" path should not be indented, but only the exception (i.e. bail if ep is not allocated properly.) ]
https://lore.kernel.org/all/2024041103-doornail-professor-7c1e@gregkh/
Goal is to get rid of of_node_put,but sending this patch first to do one thing at a time.
Changes since v1 - fixed the typo error for spell from identation to indentation
Changes since v2 - Shifted the indentation to one level left for the switch cases as per coding style.
Changes since v3 - Added descriptive subject for the patch and checked from and sign-off having same name.
Changes since v4 - Fixed name in signed-off-by as in documents.
Changes since v5 - Modified the subject and commit message as suggested.
Patches link:
------------
v1 - https://lore.kernel.org/all/[email protected]/ v2 - https://lore.kernel.org/linux-usb/[email protected]/ v3 - https://lore.kernel.org/all/[email protected]/ v4 - https://lore.kernel.org/all/[email protected]/ v5 - https://lore.kernel.org/all/[email protected]/
drivers/usb/typec/mux/nb7vpq904m.c | 68 +++++++++++++++--------------- 1 file changed, 35 insertions(+), 33 deletions(-)
drivers/usb/typec/mux/nb7vpq904m.c +35 -33
@@ -321,46 +321,48 @@ static int nb7vpq904m_parse_data_lanes_mapping(struct nb7vpq904m *nb7) ep =of_graph_get_endpoint_by_regs (nb7->client->dev.of_node,1 ,0 ); -if (ep) { - ret =of_property_count_u32_elems (ep,"data-lanes" ); -if (ret == -EINVAL ) -/* Property isn't here, consider default mapping */ - goto out_done; -if (ret <0 ) - goto out_error; - -if (ret !=DATA_LANES_COUNT ) { -dev_err (&nb7->client->dev,"expected 4 data lanes\n" ); - ret = -EINVAL ; - goto out_error; - } +if (!ep) +return 0 ; - ret =of_property_read_u32_array (ep,"data-lanes" , data_lanes,DATA_LANES_COUNT ); -if (ret) - goto out_error; -for (i =0 ; i <ARRAY_SIZE (supported_data_lane_mapping); i++) { -for (j =0 ; j <DATA_LANES_COUNT ; j++) { -if (data_lanes[j] != supported_data_lane_mapping[i][j]) -break ; - } + ret =of_property_count_u32_elems (ep,"data-lanes" ); +if (ret == -EINVAL ) +/* Property isn't here, consider default mapping */ + goto out_done; +if (ret <0 ) + goto out_error; + +if (ret !=DATA_LANES_COUNT ) { +dev_err (&nb7->client->dev,"expected 4 data lanes\n" ); + ret = -EINVAL ; + goto out_error; + } -if (j ==DATA_LANES_COUNT ) + ret =of_property_read_u32_array (ep,"data-lanes" , data_lanes,DATA_LANES_COUNT ); +if (ret) + goto out_error; + +for (i =0 ; i <ARRAY_SIZE (supported_data_lane_mapping); i++) { +for (j =0 ; j <DATA_LANES_COUNT ; j++) { +if (data_lanes[j] != supported_data_lane_mapping[i][j])break ; } -switch (i) { -case NORMAL_LANE_MAPPING : -break ; -case INVERT_LANE_MAPPING : - nb7->swap_data_lanes = true; -dev_info (&nb7->client->dev,"using inverted data lanes mapping\n" ); +if (j ==DATA_LANES_COUNT )break ; -default : -dev_err (&nb7->client->dev,"invalid data lanes mapping\n" ); - ret = -EINVAL ; - goto out_error; - } + } + +switch (i) { +case NORMAL_LANE_MAPPING : +break ; +case INVERT_LANE_MAPPING : + nb7->swap_data_lanes = true; +dev_info (&nb7->client->dev,"using inverted data lanes mapping\n" ); +break ; +default : +dev_err (&nb7->client->dev,"invalid data lanes mapping\n" ); + ret = -EINVAL ; + goto out_error; } out_done:
Signature · 1 line
2.34.1