linux-kernel - Re: [PATCH v2] remove indentation for common path [linux-next]

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date: Sat, 20 Apr 2024 23:47:25 +0300
From: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
To: sundar <prosunofficial@...il.com>
Cc: heikki.krogerus@...ux.intel.com, gregkh@...uxfoundation.org, 
	neil.armstrong@...aro.org, u.kleine-koenig@...gutronix.de, 
	christophe.jaillet@...adoo.fr, linux-usb@...r.kernel.org, 
	linux-kernel@...r.kernel.org, skhan@...uxfoundation.org, 
	javier.carrasco.cruz@...il.com
Subject: Re: [PATCH v2] remove indentation for common path [linux-next]

On Sat, 20 Apr 2024 at 19:49, sundar <prosunofficial@...il.com> wrote:
>
> Added check if pointer is null and removed indentation for common path
>
> Suggested-by: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
> Signed-off-by: sundar <prosunofficial@...il.com>
> ---
>
> Fixed nitpicks in code according to comments received on other patch.
>
> 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
>
> v1 patch link - https://lore.kernel.org/all/20240420145522.15018-1-prosunofficial@gmail.com/
>
>  drivers/usb/typec/mux/nb7vpq904m.c | 49 +++++++++++++++---------------
>  1 file changed, 25 insertions(+), 24 deletions(-)
>
> diff --git a/drivers/usb/typec/mux/nb7vpq904m.c b/drivers/usb/typec/mux/nb7vpq904m.c
> index b17826713753..fe0257840dd5 100644
> --- a/drivers/usb/typec/mux/nb7vpq904m.c
> +++ b/drivers/usb/typec/mux/nb7vpq904m.c
> @@ -321,35 +321,37 @@ 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;
> +       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) {
> +               if (j == DATA_LANES_COUNT)
> +                       break;
> +       }
> +
> +       switch (i) {
>                 case NORMAL_LANE_MAPPING:

switch-cases should also be shifted one level to the left, see
Documentation/process/coding-style.rst

>                         break;
>                 case INVERT_LANE_MAPPING:
> @@ -360,7 +362,6 @@ static int nb7vpq904m_parse_data_lanes_mapping(struct nb7vpq904m *nb7)
>                         dev_err(&nb7->client->dev, "invalid data lanes mapping\n");
>                         ret = -EINVAL;
>                         goto out_error;
> -               }
>         }
>
>  out_done:
> --
> 2.34.1
>


-- 
With best wishes
Dmitry