Re: [PATCH 22/38] arm: dts: qcom: ipq8064: Break long property array data

[Konrad Dybcio]

On 10/2/26 12:36 AM, Waqar Hameed wrote:
> There is an awkward line break at the last elements expression. Put all
> elements on a newline instead. Found/fixed with dtfmt (Devicetree Source
> formatter).
> 
> Signed-off-by: Waqar Hameed <waqar.hameed@xxxxxxxx>
> ---
>  arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi b/arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi
> index 24632af28d52e..d2b576a37c47d 100644
> --- a/arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi
> +++ b/arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi
> @@ -278,8 +278,9 @@ memory {
>  
>  	cpu-pmu {
>  		compatible = "qcom,krait-pmu";
> -		interrupts = <GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(2) |
> -					  IRQ_TYPE_LEVEL_HIGH)>;
> +		interrupts = <
> +			GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(2) | IRQ_TYPE_LEVEL_HIGH)
> +		>;

Let's wrap it at 100 chars. For lines that overflow, I think we may
need some brainstorming from the wider audience.

For reference, we have a couple of very long lines in the Qualcomm
soundwire nodes (qcom,soundwire-vXXX), and we just kinda let them
overflow for the time being

Konrad