Re: [PATCH v4 3/7] i2c: qcom-cci: Stop complaining about DT set clock rate

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

 



On 05/09/2024 14:57, Konrad Dybcio wrote:
On 4.09.2024 4:04 AM, Richard Acayan wrote:
From: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx>

It is common practice in the downstream and upstream CCI dt to set CCI
clock rates to 19.2 MHz. It appears to be fairly common for initial code to
set the CCI clock rate to 37.5 MHz.

Applying the widely used CCI clock rates from downstream ought not to cause
warning messages in the upstream kernel where our general policy is to
usually copy downstream hardware clock rates across the range of Qualcomm
drivers.

Drop the warning it is pervasive across CAMSS users but doesn't add any
information or warrant any changes to the DT to align the DT clock rate to
the bootloader clock rate.

Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx>
Reviewed-by: Vladimir Zapolskiy <vladimir.zapolskiy@xxxxxxxxxx>
Link: https://lore.kernel.org/linux-arm-msm/20240824115900.40702-1-bryan.odonoghue@xxxxxxxxxx
Signed-off-by: Richard Acayan <mailingradian@xxxxxxxxx>
---

I.. am not sure this is really a problem? On some platforms the core
clock is only 19.2 Mhz, but e.g. on sdm845 we have:

static const struct freq_tbl ftbl_cam_cc_cci_clk_src[] = {
         F(19200000, P_BI_TCXO, 1, 0, 0),
         F(37500000, P_CAM_CC_PLL0_OUT_EVEN, 16, 0, 0),
         F(50000000, P_CAM_CC_PLL0_OUT_EVEN, 12, 0, 0),
         F(100000000, P_CAM_CC_PLL0_OUT_EVEN, 6, 0, 0),
         { }
};

CCI latches the code from DT and I assume that people submitting dts have actually tested their sensors when they do so.

The complaint about not being 19.2 MHz is surely not valid since, it can be any number of frequencies.

Its a redundant and useless warning.

We can do extra work to align to a set of frequencies sure but, the warning is not a warning about a real thing.

---
bod





[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]

  Powered by Linux