Re: [PATCH v2 2/2] pinctrl: meson-g12a: add pinctrl driver support

2018-07-16 Thread Yixun Lan
HI Jerome On 07/16/18 17:54, Jerome Brunet wrote: > +/* uart_ao_a_ee */ +static const unsigned int uart_ao_rx_a_c2_pins[]= { GPIOC_2 }; +static const unsigned int uart_ao_tx_a_c3_pins[]= { GPIOC_3 }; >>> >>> Same comment as Martin about naming consistency ... drop c2

Re: [PATCH v2 2/2] pinctrl: meson-g12a: add pinctrl driver support

2018-07-16 Thread Yixun Lan
HI Jerome On 07/16/18 17:54, Jerome Brunet wrote: > +/* uart_ao_a_ee */ +static const unsigned int uart_ao_rx_a_c2_pins[]= { GPIOC_2 }; +static const unsigned int uart_ao_tx_a_c3_pins[]= { GPIOC_3 }; >>> >>> Same comment as Martin about naming consistency ... drop c2

Re: [PATCH v2 2/2] pinctrl: meson-g12a: add pinctrl driver support

2018-07-16 Thread Jerome Brunet
> > > +/* uart_ao_a_ee */ > > > +static const unsigned int uart_ao_rx_a_c2_pins[]= { GPIOC_2 }; > > > +static const unsigned int uart_ao_tx_a_c3_pins[]= { GPIOC_3 }; > > > > Same comment as Martin about naming consistency ... drop c2 and c3 here. > > > > there is already

Re: [PATCH v2 2/2] pinctrl: meson-g12a: add pinctrl driver support

2018-07-16 Thread Jerome Brunet
> > > +/* uart_ao_a_ee */ > > > +static const unsigned int uart_ao_rx_a_c2_pins[]= { GPIOC_2 }; > > > +static const unsigned int uart_ao_tx_a_c3_pins[]= { GPIOC_3 }; > > > > Same comment as Martin about naming consistency ... drop c2 and c3 here. > > > > there is already

Re: [PATCH v2 2/2] pinctrl: meson-g12a: add pinctrl driver support

2018-07-16 Thread Yixun Lan
Hi Jerome thanks for the review, see my comments below On 07/16/18 00:16, Jerome Brunet wrote: > On Sat, 2018-07-14 at 23:27 +, Yixun Lan wrote: >> Add the pinctrl driver for Meson-G12A SoC which share the similar IP as >> the previous Meson-AXG SoC. >> >> Starting from Meson-AXG SoC, the

Re: [PATCH v2 2/2] pinctrl: meson-g12a: add pinctrl driver support

2018-07-16 Thread Yixun Lan
Hi Jerome thanks for the review, see my comments below On 07/16/18 00:16, Jerome Brunet wrote: > On Sat, 2018-07-14 at 23:27 +, Yixun Lan wrote: >> Add the pinctrl driver for Meson-G12A SoC which share the similar IP as >> the previous Meson-AXG SoC. >> >> Starting from Meson-AXG SoC, the

Re: [PATCH v2 2/2] pinctrl: meson-g12a: add pinctrl driver support

2018-07-15 Thread Jerome Brunet
On Sat, 2018-07-14 at 23:27 +, Yixun Lan wrote: > Add the pinctrl driver for Meson-G12A SoC which share the similar IP as > the previous Meson-AXG SoC. > > Starting from Meson-AXG SoC, the pinctrl controller block use 4 > continues register bits to specific the pin mux function, while

Re: [PATCH v2 2/2] pinctrl: meson-g12a: add pinctrl driver support

2018-07-15 Thread Jerome Brunet
On Sat, 2018-07-14 at 23:27 +, Yixun Lan wrote: > Add the pinctrl driver for Meson-G12A SoC which share the similar IP as > the previous Meson-AXG SoC. > > Starting from Meson-AXG SoC, the pinctrl controller block use 4 > continues register bits to specific the pin mux function, while

[PATCH v2 2/2] pinctrl: meson-g12a: add pinctrl driver support

2018-07-14 Thread Yixun Lan
Add the pinctrl driver for Meson-G12A SoC which share the similar IP as the previous Meson-AXG SoC. Starting from Meson-AXG SoC, the pinctrl controller block use 4 continues register bits to specific the pin mux function, while comparing to old generation SoC which using variable length register

[PATCH v2 2/2] pinctrl: meson-g12a: add pinctrl driver support

2018-07-14 Thread Yixun Lan
Add the pinctrl driver for Meson-G12A SoC which share the similar IP as the previous Meson-AXG SoC. Starting from Meson-AXG SoC, the pinctrl controller block use 4 continues register bits to specific the pin mux function, while comparing to old generation SoC which using variable length register