michaelh/zephyr
1
0
Fork 0
Code Releases Activity

ITE: drivers/pinctrl: Add alternate function additional setting

Browse source 
When the alternate setting is configured as func3, in addition to
the setting of func3-gcr, some pins require external setting.

Signed-off-by: Tim Lin <tim2.lin@ite.corp-partner.google.com>
This commit is contained in:
Tim Lin 2023-09-18 17:43:59 +08:00 • committed by Carles Cufí
commit dca9cbff08
4 changed files with 53 additions and 22 deletions
Download patch file Download diff file Expand all files Collapse all files

56
drivers/pinctrl/pinctrl_ite_it8xxx2.c
View file

@ -24,6 +24,10 @@ struct pinctrl_it8xxx2_gpio {
uintptr_t func3_gcr[GPIO_GROUP_MEMBERS];
/* function 3 enable mask */
uint8_t func3_en_mask[GPIO_GROUP_MEMBERS];
/* function 3 external control register */
uintptr_t func3_ext[GPIO_GROUP_MEMBERS];
/* function 3 external mask */
uint8_t func3_ext_mask[GPIO_GROUP_MEMBERS];
/* function 4 general control register */
uintptr_t func4_gcr[GPIO_GROUP_MEMBERS];
/* function 4 enable mask */
@ -132,6 +136,7 @@ static int pinctrl_gpio_it8xxx2_configure_pins(const pinctrl_soc_pin_t *pins)
volatile uint8_t *reg_gpcr = (uint8_t *)gpio->reg_gpcr + pin;
volatile uint8_t *reg_func3_gcr = (uint8_t *)(gpio->func3_gcr[pin]);
volatile uint8_t *reg_func4_gcr = (uint8_t *)(gpio->func4_gcr[pin]);
volatile uint8_t *reg_func3_ext = (uint8_t *)(gpio->func3_ext[pin]);
/* Handle PIN configuration. */
if (pinctrl_it8xxx2_set(pins)) {
@ -155,6 +160,11 @@ static int pinctrl_gpio_it8xxx2_configure_pins(const pinctrl_soc_pin_t *pins)
/* Common settings for alternate function. */
*reg_gpcr &= ~(GPCR_PORT_PIN_MODE_INPUT |
GPCR_PORT_PIN_MODE_OUTPUT);
/* Ensure that func3-ext setting is in default state. */
if (reg_func3_ext != NULL) {
*reg_func3_ext &= ~gpio->func3_ext_mask[pin];
}
switch (pins->alt_func) {
case IT8XXX2_ALT_FUNC_1:
/* Func1: Alternate function has been set above. */
@ -169,6 +179,11 @@ static int pinctrl_gpio_it8xxx2_configure_pins(const pinctrl_soc_pin_t *pins)
* Func3 also need to set the general control.
*/
*reg_func3_gcr |= gpio->func3_en_mask[pin];
/* Func3-external: Some pins require external setting. */
if (reg_func3_ext != NULL) {
*reg_func3_ext |= gpio->func3_ext_mask[pin];
}
break;
case IT8XXX2_ALT_FUNC_4:
/*
@ -327,11 +342,6 @@ static int pinctrl_it8xxx2_init(const struct device *dev)
*/
gpio_base->GPIO_GCR &= ~IT8XXX2_GPIO_LPCRSTEN;
/*
* TODO: If UART2 swaps from bit2:1 to bit6:5 in H group, we
* have to set UART1PSEL = 1 in UART1PMR register.
*/
#ifdef CONFIG_SOC_IT8XXX2_REG_SET_V2
/*
* Swap the default I2C2 SMCLK2/SMDAT2 pins from GPC7/GPD0 to GPF6/GPF7,
@ -345,23 +355,25 @@ static int pinctrl_it8xxx2_init(const struct device *dev)
return 0;
}
#define INIT_UNION_CONFIG(inst) \
COND_CODE_1(DT_INST_PROP(inst, gpio_group), \
(.gpio = { \
.reg_gpcr = (uint8_t *)DT_INST_REG_ADDR_BY_IDX(inst, 0), \
.func3_gcr = DT_INST_PROP(inst, func3_gcr), \
.func3_en_mask = DT_INST_PROP(inst, func3_en_mask), \
.func4_gcr = DT_INST_PROP(inst, func4_gcr), \
.func4_en_mask = DT_INST_PROP(inst, func4_en_mask), \
.volt_sel = DT_INST_PROP(inst, volt_sel), \
.volt_sel_mask = DT_INST_PROP(inst, volt_sel_mask), \
}), \
(.ksi_kso = { \
.reg_gctrl = (uint8_t *)DT_INST_REG_ADDR_BY_IDX(inst, 0), \
.reg_ctrl = (uint8_t *)DT_INST_REG_ADDR_BY_IDX(inst, 1), \
.pp_od_mask = (uint8_t)DT_INST_PROP(inst, pp_od_mask), \
.pullup_mask = (uint8_t)DT_INST_PROP(inst, pullup_mask), \
}) \
#define INIT_UNION_CONFIG(inst) \
COND_CODE_1(DT_INST_PROP(inst, gpio_group), \
(.gpio = { \
.reg_gpcr = (uint8_t *)DT_INST_REG_ADDR_BY_IDX(inst, 0), \
.func3_gcr = DT_INST_PROP(inst, func3_gcr), \
.func3_en_mask = DT_INST_PROP(inst, func3_en_mask), \
.func3_ext = DT_INST_PROP_OR(inst, func3_ext, {0}), \
.func3_ext_mask = DT_INST_PROP_OR(inst, func3_ext_mask, {0}), \
.func4_gcr = DT_INST_PROP(inst, func4_gcr), \
.func4_en_mask = DT_INST_PROP(inst, func4_en_mask), \
.volt_sel = DT_INST_PROP(inst, volt_sel), \
.volt_sel_mask = DT_INST_PROP(inst, volt_sel_mask), \
}), \
(.ksi_kso = { \
.reg_gctrl = (uint8_t *)DT_INST_REG_ADDR_BY_IDX(inst, 0), \
.reg_ctrl = (uint8_t *)DT_INST_REG_ADDR_BY_IDX(inst, 1), \
.pp_od_mask = (uint8_t)DT_INST_PROP(inst, pp_od_mask), \
.pullup_mask = (uint8_t)DT_INST_PROP(inst, pullup_mask), \
}) \
)
#define PINCTRL_ITE_INIT(inst) \

9
dts/bindings/pinctrl/ite,it8xxx2-pinctrl-func.yaml
View file

@ -14,6 +14,15 @@ properties:
func3-en-mask:
type: array
func3-ext:
type: array
description: |
When the alternate setting is configured as func3, in addition to
the setting of func3-gcr, some pins require external setting.
func3-ext-mask:
type: array
func4-gcr:
type: array

4
dts/riscv/ite/it81xx2.dtsi
View file

@ -198,6 +198,10 @@
NO_FUNC 0xf016f5 0xf016f5 NO_FUNC>;
func3-en-mask = <0 0x20 0x20 0
0 0x04 0x08 0 >;
func3-ext = <NO_FUNC NO_FUNC NO_FUNC NO_FUNC
NO_FUNC 0xf03a23 0xf03a23 NO_FUNC>;
func3-ext-mask = <0 0 0 0
0 0x01 0x01 0 >;
func4-gcr = <NO_FUNC 0xf016f5 0xf016f5 NO_FUNC
NO_FUNC NO_FUNC NO_FUNC NO_FUNC>;
func4-en-mask = <0 0x04 0x08 0

6
dts/riscv/ite/it8xxx2-pinctrl-map.dtsi
View file

@ -383,6 +383,12 @@
uart2_tx_gph2_default: uart2_tx_gph2_default {
pinmuxs = <&pinctrlh 2 IT8XXX2_ALT_FUNC_4>;
};
uart2_rx_gph5_default: uart2_rx_gph5_default {
pinmuxs = <&pinctrlh 5 IT8XXX2_ALT_FUNC_3>;
};
uart2_tx_gph6_default: uart2_tx_gph6_default {
pinmuxs = <&pinctrlh 6 IT8XXX2_ALT_FUNC_3>;
};
uart2_rx_gpf0_default: uart2_rx_gpf0_default {
pinmuxs = <&pinctrlf 0 IT8XXX2_ALT_FUNC_3>;
};