Re: [PATCH 2/2] clk: qcom: dispcc-sm6350: Add missing parent_map for a clock

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Luca Weiss <luca.weiss@xxxxxxxxxxxxx>, Bjorn Andersson <andersson@xxxxxxxxxx>, Michael Turquette <mturquette@xxxxxxxxxxxx>, Stephen Boyd <sboyd@xxxxxxxxxx>, AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxxx>, Konrad Dybcio <konradybcio@xxxxxxxxxx>
Subject: Re: [PATCH 2/2] clk: qcom: dispcc-sm6350: Add missing parent_map for a clock
From: Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxxxx>
Date: Fri, 20 Dec 2024 10:42:20 +0100
Cc: ~postmarketos/upstreaming@xxxxxxxxxxx, phone-devel@xxxxxxxxxxxxxxx, linux-arm-msm@xxxxxxxxxxxxxxx, linux-clk@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, stable@xxxxxxxxxxxxxxx
In-reply-to: <20241220-sm6350-parent_map-v1-2-64f3d04cb2eb@fairphone.com>
References: <20241220-sm6350-parent_map-v1-0-64f3d04cb2eb@fairphone.com> <20241220-sm6350-parent_map-v1-2-64f3d04cb2eb@fairphone.com>
User-agent: Mozilla Thunderbird

On 20.12.2024 10:03 AM, Luca Weiss wrote:
> If a clk_rcg2 has a parent, it should also have parent_map defined,
> otherwise we'll get a NULL pointer dereference when calling clk_set_rate
> like the following:

Same as patch 1

Konrad

References:
- [PATCH 0/2] Add missing parent_map to clocks in SM6350 clock drivers
  - From: Luca Weiss
- [PATCH 2/2] clk: qcom: dispcc-sm6350: Add missing parent_map for a clock
  - From: Luca Weiss

Prev by Date: Re: [PATCH 1/2] clk: qcom: gcc-sm6350: Add missing parent_map for two clocks
Next by Date: [PATCH v4] dmaengine: qcom: bam_dma: Avoid writing unavailable register
Previous by thread: [PATCH 2/2] clk: qcom: dispcc-sm6350: Add missing parent_map for a clock
Next by thread: Re: [PATCH 2/2] clk: qcom: dispcc-sm6350: Add missing parent_map for a clock
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [Linux for Sparc] [IETF Annouce] [Security] [Bugtraq] [Linux MIPS] [ECOS] [Asterisk Internet PBX] [Linux API]