Linux ARM-MSM sub-architecture
 help / color / mirror / Atom feed
From: Alex Elder <elder@ieee.org>
To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>,
	Andy Gross <agross@kernel.org>,
	Bjorn Andersson <andersson@kernel.org>,
	Konrad Dybcio <konrad.dybcio@linaro.org>,
	Stephen Boyd <sboyd@kernel.org>,
	Michael Turquette <mturquette@baylibre.com>,
	Taniya Das <quic_tdas@quicinc.com>
Cc: linux-arm-msm@vger.kernel.org, linux-clk@vger.kernel.org
Subject: Re: [PATCH v2 10/18] clk: qcom: smd-rpm: fix alignment of line breaking backslashes
Date: Wed, 7 Dec 2022 07:48:55 -0600	[thread overview]
Message-ID: <115194d9-5bfb-486d-71d4-05fb9212627c@ieee.org> (raw)
In-Reply-To: <20221207001503.93790-11-dmitry.baryshkov@linaro.org>

On 12/6/22 6:14 PM, Dmitry Baryshkov wrote:
> The commit 52a436e0b7fe ("clk: qcom: smd-rpm: Switch to parent_data")
> introduced ragged right alignment for the line breaking backslash. Fix
> it to make the code look consistently.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>

Very pretty.

Reviewed-by: Alex Elder <elder@linaro.org>

> ---
>   drivers/clk/qcom/clk-smd-rpm.c | 32 ++++++++++++++++----------------
>   1 file changed, 16 insertions(+), 16 deletions(-)
> 
> diff --git a/drivers/clk/qcom/clk-smd-rpm.c b/drivers/clk/qcom/clk-smd-rpm.c
> index 2075cfd34f99..7e43ecdda763 100644
> --- a/drivers/clk/qcom/clk-smd-rpm.c
> +++ b/drivers/clk/qcom/clk-smd-rpm.c
> @@ -36,10 +36,10 @@
>   		.hw.init = &(struct clk_init_data){			      \
>   			.ops = &clk_smd_rpm_ops,			      \
>   			.name = #_name,					      \
> -			.parent_data =  &(const struct clk_parent_data){ \
> -					.fw_name = "xo",		\
> -					.name = "xo_board",		\
> -			},						\
> +			.parent_data =  &(const struct clk_parent_data){      \
> +					.fw_name = "xo",		      \
> +					.name = "xo_board",		      \
> +			},						      \
>   			.num_parents = 1,				      \
>   		},							      \
>   	};								      \
> @@ -53,10 +53,10 @@
>   		.hw.init = &(struct clk_init_data){			      \
>   			.ops = &clk_smd_rpm_ops,			      \
>   			.name = #_active,				      \
> -			.parent_data =  &(const struct clk_parent_data){ \
> -					.fw_name = "xo",		\
> -					.name = "xo_board",		\
> -			},						\
> +			.parent_data =  &(const struct clk_parent_data){      \
> +					.fw_name = "xo",		      \
> +					.name = "xo_board",		      \
> +			},						      \
>   			.num_parents = 1,				      \
>   		},							      \
>   	}
> @@ -74,10 +74,10 @@
>   		.hw.init = &(struct clk_init_data){			      \
>   			.ops = &clk_smd_rpm_branch_ops,			      \
>   			.name = #_name,					      \
> -			.parent_data =  &(const struct clk_parent_data){ \
> -					.fw_name = "xo",		\
> -					.name = "xo_board",		\
> -			},						\
> +			.parent_data =  &(const struct clk_parent_data){      \
> +					.fw_name = "xo",		      \
> +					.name = "xo_board",		      \
> +			},						      \
>   			.num_parents = 1,				      \
>   		},							      \
>   	};								      \
> @@ -92,10 +92,10 @@
>   		.hw.init = &(struct clk_init_data){			      \
>   			.ops = &clk_smd_rpm_branch_ops,			      \
>   			.name = #_active,				      \
> -			.parent_data =  &(const struct clk_parent_data){ \
> -					.fw_name = "xo",		\
> -					.name = "xo_board",		\
> -			},						\
> +			.parent_data =  &(const struct clk_parent_data){      \
> +					.fw_name = "xo",		      \
> +					.name = "xo_board",		      \
> +			},						      \
>   			.num_parents = 1,				      \
>   		},							      \
>   	}


  reply	other threads:[~2022-12-07 13:49 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-07  0:14 [PATCH v2 00/18] clk: qcom: smd-rpm: drop platform names Dmitry Baryshkov
2022-12-07  0:14 ` [PATCH v2 01/18] clk: qcom: smd-rpm: remove duplication between MMXI and MMAXI defines Dmitry Baryshkov
2022-12-07 13:48   ` Alex Elder
2022-12-07  0:14 ` [PATCH v2 02/18] clk: qcom: smd-rpm: enable pin-controlled ln_bb_clk clocks on qcs404 Dmitry Baryshkov
2022-12-09 16:14   ` Krzysztof Kozlowski
2022-12-09 16:28     ` Dmitry Baryshkov
2022-12-09 17:05       ` Krzysztof Kozlowski
2022-12-07  0:14 ` [PATCH v2 03/18] clk: qcom: smd-rpm: remove duplication between qcs404 and qcm2290 clocks Dmitry Baryshkov
2022-12-07 13:48   ` Alex Elder
2022-12-07  0:14 ` [PATCH v2 04/18] clk: qcom: smd-rpm: add missing ln_bb_clkN clocks Dmitry Baryshkov
2022-12-07 13:48   ` Alex Elder
2022-12-07  0:14 ` [PATCH v2 05/18] clk: qcom: smd-rpm: use msm8998_ln_bb_clk2 for qcm2290 SoC Dmitry Baryshkov
2022-12-07  0:14 ` [PATCH v2 06/18] clk: qcom: smd-rpm: rename msm8992_ln_bb_* clocks to qcs404_ln_bb_* Dmitry Baryshkov
2022-12-07 13:48   ` Alex Elder
2022-12-07  0:14 ` [PATCH v2 07/18] clk: qcom: smd-rpm: remove duplication between sm6375 and sm6125 clocks Dmitry Baryshkov
2022-12-07 13:48   ` Alex Elder
2022-12-07  0:14 ` [PATCH v2 08/18] clk: qcom: smd-rpm: add XO_BUFFER clock for each XO_BUFFER_PINCTRL clock Dmitry Baryshkov
2022-12-07  0:14 ` [PATCH v2 09/18] clk: qcom: smd-rpm: drop the rpm_status_id field Dmitry Baryshkov
2022-12-07  0:14 ` [PATCH v2 10/18] clk: qcom: smd-rpm: fix alignment of line breaking backslashes Dmitry Baryshkov
2022-12-07 13:48   ` Alex Elder [this message]
2022-12-07  0:14 ` [PATCH v2 11/18] clk: qcom: smd-rpm: move clock definitions together Dmitry Baryshkov
2022-12-07  0:14 ` [PATCH v2 12/18] clk: qcom: smd-rpm: rename some msm8974 active-only clocks Dmitry Baryshkov
2022-12-07  0:14 ` [PATCH v2 13/18] clk: qcom: smd-rpm: simplify XO_BUFFER clocks definitions Dmitry Baryshkov
2022-12-07  0:14 ` [PATCH v2 14/18] clk: qcom: smd-rpm: simplify SMD_RPM/_BRANCH/_QDSS clock definitions Dmitry Baryshkov
2022-12-07  0:15 ` [PATCH v2 15/18] clk: qcom: smd-rpm: rename SMD_RPM_BRANCH clock symbols Dmitry Baryshkov
2022-12-07  0:15 ` [PATCH v2 16/18] clk: qcom: smd-rpm: rename the qcm2290 rf_clk3 clocks Dmitry Baryshkov
2022-12-07  0:15 ` [PATCH v2 17/18] clk: qcom: smd-rpm: rename SMD_RPM_BUS clocks Dmitry Baryshkov
2022-12-07 13:48   ` Alex Elder
2022-12-07  0:15 ` [PATCH v2 18/18] clk: qcom: smd-rpm: remove usage of platform name Dmitry Baryshkov
2022-12-09 16:15 ` [PATCH v2 00/18] clk: qcom: smd-rpm: drop platform names Krzysztof Kozlowski

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=115194d9-5bfb-486d-71d4-05fb9212627c@ieee.org \
    --to=elder@ieee.org \
    --cc=agross@kernel.org \
    --cc=andersson@kernel.org \
    --cc=dmitry.baryshkov@linaro.org \
    --cc=konrad.dybcio@linaro.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-clk@vger.kernel.org \
    --cc=mturquette@baylibre.com \
    --cc=quic_tdas@quicinc.com \
    --cc=sboyd@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox