Re: [PATCH 3/4] arm64: dts: qcom: sc8280xp: gaokun3: add front camera sensor node

Pengyu Luo
2026-04-29T11:24:49Z

On Wed, Apr 29, 2026 at 6:08 PM Konrad Dybcio
<konrad.dybcio@oss.qualcomm.com> wrote:
>
> On 4/29/26 8:36 AM, Pengyu Luo wrote:
> > On Mon, Apr 27, 2026 at 9:00 PM Konrad Dybcio
> > <konrad.dybcio@oss.qualcomm.com> wrote:
> >>
> >> On 4/25/26 12:52 PM, Pengyu Luo wrote:
> >>> Hi846 is found on my Gaokun3, descripting it.
> >>>
> >>> Note that it seems that only Goakun3(3.0GHz) version is equipped with
> >>> Hi846. Goakun3(2.69GHz) version is equipped with S5K4H7.
> >>
> >> Are there any other noticeable hw differences between the two variants,
> >> that could warrant the DT to be split?
> >>
> >
> > Except for the cameras and cpu, I think no.(But I couldn't tell all
> > 3.0 variants are equipped with hi846) Before camera support, this DT
> > worked for both of them.
>
> Do the ACPI tables perhaps have some sort of a hwid mechanism? Perhaps
> two nodes defined for both cameras and the _STA method referencing some
> object?
>

No. It may poll sensormodule binarys, register lists are in
sensormodule, the slave addr may be here too.

In qccamfrontsensor_extension8280.cab

CAMF_RES_MTP_ALT.bin
CAMF_RES_MTP.bin
CAMF_RES_QRD.bin
com.qti.sensormodule.hlt_hi846.bin
com.qti.sensormodule.xinli_s5k4h7.bin
com.qti.tuned.default.bin # for hi846
com.qti.tuned.hlt_hi846.bin
com.qti.tuned.xinli_s5k4h7.bin
qccamfrontsensor_extension8280.cat
qccamfrontsensor_extension8280.inf
SCF1_FRONT_MTP.bin
SCFG_FRONT_MTP.bin
SCFG_FRONT_QRD.bin

in SCFG_FRONT_QRD.bin

Package ()
{
"com.qti.sensormodule.hlt_hi846.bin",
"com.qti.tuned.hlt_hi846.bin",
0x0000000000150041,
0x0000000046080F16, # id, id_addr, this can be found in the binary too
0x0000000000000000,
},

If this fails, then try to load com.qti.sensormodule.xinli_s5k4h7.bin?

Best wishes,
Pengyu

> Konrad