Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ITE: drivers/i2c: Disable pre-detect on IT82xx2 family #57173

Merged

Conversation

GTLin08
Copy link
Collaborator

@GTLin08 GTLin08 commented Apr 24, 2023

Disable the hardware I2C target detection on the IT82xx2 SoC family.

Note: The register setting of I2C target detection is different in
IT81XX2 and IT82XX2 SOC.

@keith-zephyr
Copy link
Contributor

Please update the commit message to better reflect the change:

ITE: drivers/i2c: Disable pre-detect on IT82xx2 family

Disable the hardware I2C target detection on the IT82xx2 SoC family.

@brockus-zephyr
Copy link
Member

May want to leave a note in the commit message indicating that you are adding IT82xx2 register definition differentiation between V1 and V2.

@@ -278,6 +278,17 @@ static int ite_it8xxx2_init(void)
*/
gctrl_regs->GCTRL_WMCR |= BIT(7);

/*
* Disable this feature that can detect pre-define hardware
* slave A through I2C0. This is for debugging use, so it
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use "target" instead of "slave". #27033

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

Disable the hardware I2C target detection on the IT82xx2 SoC family.

Note: The register setting of I2C target detection is different in
IT81XX2 and IT82XX2 SOC.

Signed-off-by: Tim Lin <[email protected]>
@GTLin08 GTLin08 force-pushed the it8xxx2_i2c_disable_predefine branch from 323aa67 to 408b980 Compare April 25, 2023 02:20
@GTLin08 GTLin08 changed the title ITE: drivers/i2c: Disable pre-define detect ITE: drivers/i2c: Disable pre-detect on IT82xx2 family Apr 25, 2023
@GTLin08
Copy link
Collaborator Author

GTLin08 commented Apr 25, 2023

May want to leave a note in the commit message indicating that you are adding IT82xx2 register definition differentiation between V1 and V2.

Done.

@GTLin08
Copy link
Collaborator Author

GTLin08 commented Apr 25, 2023

Please update the commit message to better reflect the change:

ITE: drivers/i2c: Disable pre-detect on IT82xx2 family

Disable the hardware I2C target detection on the IT82xx2 SoC family.

Done.

@carlescufi carlescufi merged commit 1a2874a into zephyrproject-rtos:main Apr 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: I2C area: RISCV RISCV Architecture (32-bit & 64-bit) platform: ITE ITE
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants