Re: [PATCH v2 3/3] arm64: dts: qcom: glymur-zenbook-a16: Add Embedded Controller

[Abel Vesa]

On 26-09-23 12:04:32, Konrad Dybcio wrote:
> On 9/22/26 10:31 PM, Abel Vesa wrote:
> > On 26-09-22 11:33:38, Konrad Dybcio wrote:
> >> From: Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxxxx>
> >>
> >> The Glymur-based Zenbook A16 features an Embedded Controller running
> >> custom ASUS firmware (which is loosely based on the reference Qualcomm
> >> Hamoa EC one with a lot of modifications). It provides keyboard
> >> backlight control, additional temperature sensor readouts, fan speed
> >> monitoring and a number of miscellaneous functions.
> >>
> >> Describe it.
> >>
> >> Signed-off-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxxxx>
> >> ---
> >>  .../boot/dts/qcom/glymur-asus-zenbook-a16-ux3607oa.dts    | 15 +++++++++++++--
> >>  1 file changed, 13 insertions(+), 2 deletions(-)
> >>
> >> diff --git a/arch/arm64/boot/dts/qcom/glymur-asus-zenbook-a16-ux3607oa.dts b/arch/arm64/boot/dts/qcom/glymur-asus-zenbook-a16-ux3607oa.dts
> >> index 40433dc2d6cf..d3356fcd25f0 100644
> >> --- a/arch/arm64/boot/dts/qcom/glymur-asus-zenbook-a16-ux3607oa.dts
> >> +++ b/arch/arm64/boot/dts/qcom/glymur-asus-zenbook-a16-ux3607oa.dts
> >> @@ -670,8 +670,19 @@ &i2c9 {
> >>  
> >>  	status = "okay";
> >>  
> >> -	/* EC subdevice @ 0x5b */
> > 
> > So what is on this address then ?
> 
> The driver spawns a secondary i2c device there to service some
> of the EC communication

Thanks for clarifying.