[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <97b8fabe-2bc9-4d14-a3b3-bb1356a7e642@gmail.com>
Date: Thu, 2 May 2024 21:53:54 +0530
From: R Sundar <prosunofficial@...il.com>
To: heikki.krogerus@...ux.intel.com, gregkh@...uxfoundation.org,
neil.armstrong@...aro.org, dmitry.baryshkov@...aro.org,
u.kleine-koenig@...gutronix.de, christophe.jaillet@...adoo.fr
Cc: linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org,
skhan@...uxfoundation.org, javier.carrasco.cruz@...il.com
Subject: Re: [PATCH v5 linux-next] usb:typec:mux: remove indentation for
common path
On 26/04/24 22:17, R Sundar wrote:
> Nitpick, Mostly common path will not be indented. so rewritten this
> function to check device_node pointer is null and removed common path
> indentation.
>
> Signed-off-by: R Sundar <prosunofficial@...il.com>
> ---
>
> Fixed nitpicks in code 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.
>
> Patches link:
> ------------
> v1 - https://lore.kernel.org/all/20240420145522.15018-1-prosunofficial@gmail.com/
> v2 - https://lore.kernel.org/linux-usb/20240420164927.15290-1-prosunofficial@gmail.com/
> v3 - https://lore.kernel.org/all/20240421011647.3027-1-prosunofficial@gmail.com/
> v4 - https://lore.kernel.org/all/20240424150718.5006-1-prosunofficial@gmail.com/
>
> drivers/usb/typec/mux/nb7vpq904m.c | 68 +++++++++++++++---------------
> 1 file changed, 34 insertions(+), 34 deletions(-)
>
> diff --git a/drivers/usb/typec/mux/nb7vpq904m.c b/drivers/usb/typec/mux/nb7vpq904m.c
> index b17826713753..f7a00b388876 100644
> --- a/drivers/usb/typec/mux/nb7vpq904m.c
> +++ b/drivers/usb/typec/mux/nb7vpq904m.c
> @@ -320,47 +320,47 @@ static int nb7vpq904m_parse_data_lanes_mapping(struct nb7vpq904m *nb7)
> int ret, i, j;
>
> ep = of_graph_get_endpoint_by_regs(nb7->client->dev.of_node, 1, 0);
> + if (!ep)
> + return 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;
> - }
> -
> - ret = of_property_read_u32_array(ep, "data-lanes", data_lanes, DATA_LANES_COUNT);
> - if (ret)
> - goto out_error;
> + 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;
> + }
>
> - 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_read_u32_array(ep, "data-lanes", data_lanes, DATA_LANES_COUNT);
> + if (ret)
> + goto out_error;
>
> - if (j == DATA_LANES_COUNT)
> + 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:
Hi,
Any feedback for this patch?
Regards,
Sundar
Powered by blists - more mailing lists