Re: [PATCH] pinctrl: rockchip: do coding style for mux route struct

[Heiko Stübner <heiko@xxxxxxxxx>]

Hi Jay,

Am Dienstag, 20. April 2021, 10:41:34 CEST schrieb jay.xu@xxxxxxxxxxxxxx:
> Do I need to send v2 with change-id abandon ?

do everything to make a Maintainer's life easier :-) .

So I guess yes and of course try to make sure it applies to the
pinctrl devel branch
[ https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-pinctrl.git/log/?h=devel ]

and add the necessary versioning information, so that people see
which one is newer.


Heiko


> From: HeikoStübner
> Date: 2021-04-20 15:35
> To: linus.walleij; Jianqun Xu
> CC: huangtao; linux-gpio; linux-rockchip; linux-kernel; Jianqun Xu
> Subject: Re: [PATCH] pinctrl: rockchip: do coding style for mux route struct
> Hi,
>  
> Am Dienstag, 6. April 2021, 04:53:56 CEST schrieb Jianqun Xu:
> > The mux route tables take many lines for each SoC, and it will be more
> > instances for newly SoC, that makes the file size increase larger.
> > 
> > This patch only do coding style for mux route struct, by adding a new
> > definition and replace the structs by script which supplied by
> > huangtao@xxxxxxxxxxxxxx
> > 
> > sed -i -e "
> > /static struct rockchip_mux_route_data /bcheck
> > b
> > :append-next-line
> > N
> > :check
> > /^[^;]*$/bappend-next-line
> > s/[[:blank:]]*.bank_num = \([[:digit:]]*,\)\n/\tRK_MUXROUTE_SAME(\1/g
> > s/[[:blank:]]*.pin =[[:blank:]]*0,\n/ RK_PA0,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*1,\n/ RK_PA1,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*2,\n/ RK_PA2,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*3,\n/ RK_PA3,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*4,\n/ RK_PA4,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*5,\n/ RK_PA5,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*6,\n/ RK_PA6,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*7,\n/ RK_PA7,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*8,\n/ RK_PB0,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*9,\n/ RK_PB1,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*10,\n/ RK_PB2,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*11,\n/ RK_PB3,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*12,\n/ RK_PB4,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*13,\n/ RK_PB5,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*14,\n/ RK_PB6,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*15,\n/ RK_PB7,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*16,\n/ RK_PC0,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*17,\n/ RK_PC1,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*18,\n/ RK_PC2,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*19,\n/ RK_PC3,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*20,\n/ RK_PC4,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*21,\n/ RK_PC5,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*22,\n/ RK_PC6,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*23,\n/ RK_PC7,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*24,\n/ RK_PD0,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*25,\n/ RK_PD1,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*26,\n/ RK_PD2,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*27,\n/ RK_PD3,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*28,\n/ RK_PD4,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*29,\n/ RK_PD5,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*30,\n/ RK_PD6,/g
> > s/[[:blank:]]*.pin =[[:blank:]]*31,\n/ RK_PD7,/g
> > s/[[:blank:]]*.func = \([[:digit:]]*,\)\n/ \1/g
> > s/[[:blank:]]*.route_location =[[:blank:]]*\([[:print:]]*,\)\n//g
> > s/[[:blank:]]*.route_offset = \(0x[[:xdigit:]]*,\)\n/ \1/g
> > s/[[:blank:]]*.route_val =[[:blank:]]*\([[:print:]]*\),\n/ \1),/g
> > s/\t{\n//g
> > s/\t}, {\n//g
> > s/\t},//g
> > s/[[:blank:]]*\(\/\*[[:print:]]*\*\/\)\n[[:blank:]]*RK_MUXROUTE_SAME(\([[:print:]]*\)),\n/\tRK_MUXROUTE_SAME(\2), \1\n/g
> > s/[[:blank:]]*\(\/\*[[:print:]]*\*\/\)\n[[:blank:]]*RK_MUXROUTE_SAME(\([[:print:]]*\)),/\tRK_MUXROUTE_SAME(\2), \1\n/g
> > " drivers/pinctrl/pinctrl-rockchip.c
> > 
> > Signed-off-by: Jianqun Xu <jay.xu@xxxxxxxxxxxxxx>
> > Change-Id: Ifc823d9557605b9dfcc9c0455a739f04f3fce5be
>  
> Change-id should not be in here.
>  
> Somehow I remember giving this a "reviewed-by" before in some
> previous version, but my memory may be faulty, so
>  
> Reviewed-by: Heiko Stuebner <heiko@xxxxxxxxx>
>  
> and of course I like this change very much ;-)
>  
>  
> Though it may be really helpful to not send these individual patches but
> instead make it part of the series and put it at the beginning.
> Tracking the order patches should get applied when they're sent individually
> can get very hard.
>  
> Heiko
>  
>  
> > ---
> >  drivers/pinctrl/pinctrl-rockchip.c | 669 +++++------------------------
> >  1 file changed, 99 insertions(+), 570 deletions(-)
> > 
> > diff --git a/drivers/pinctrl/pinctrl-rockchip.c b/drivers/pinctrl/pinctrl-rockchip.c
> > index deabfbc74a01..6ba31c66ef8b 100644
> > --- a/drivers/pinctrl/pinctrl-rockchip.c
> > +++ b/drivers/pinctrl/pinctrl-rockchip.c
> > @@ -292,6 +292,25 @@ struct rockchip_pin_bank {
> >  .pull_type[3] = pull3, \
> >  }
> >  
> > +#define PIN_BANK_MUX_ROUTE_FLAGS(ID, PIN, FUNC, REG, VAL, FLAG) \
> > + { \
> > + .bank_num = ID, \
> > + .pin = PIN, \
> > + .func = FUNC, \
> > + .route_offset = REG, \
> > + .route_val = VAL, \
> > + .route_location = FLAG, \
> > + }
> > +
> > +#define RK_MUXROUTE_SAME(ID, PIN, FUNC, REG, VAL) \
> > + PIN_BANK_MUX_ROUTE_FLAGS(ID, PIN, FUNC, REG, VAL, ROCKCHIP_ROUTE_SAME)
> > +
> > +#define RK_MUXROUTE_GRF(ID, PIN, FUNC, REG, VAL) \
> > + PIN_BANK_MUX_ROUTE_FLAGS(ID, PIN, FUNC, REG, VAL, ROCKCHIP_ROUTE_GRF)
> > +
> > +#define RK_MUXROUTE_PMU(ID, PIN, FUNC, REG, VAL) \
> > + PIN_BANK_MUX_ROUTE_FLAGS(ID, PIN, FUNC, REG, VAL, ROCKCHIP_ROUTE_PMU)
> > +
> >  /**
> >   * struct rockchip_mux_recalced_data: represent a pin iomux data.
> >   * @num: bank number.
> > @@ -803,597 +822,107 @@ static void rockchip_get_recalced_mux(struct rockchip_pin_bank *bank, int pin,
> >  }
> >  
> >  static struct rockchip_mux_route_data px30_mux_route_data[] = {
> > - {
> > - /* cif-d2m0 */
> > - .bank_num = 2,
> > - .pin = 0,
> > - .func = 1,
> > - .route_offset = 0x184,
> > - .route_val = BIT(16 + 7),
> > - }, {
> > - /* cif-d2m1 */
> > - .bank_num = 3,
> > - .pin = 3,
> > - .func = 3,
> > - .route_offset = 0x184,
> > - .route_val = BIT(16 + 7) | BIT(7),
> > - }, {
> > - /* pdm-m0 */
> > - .bank_num = 3,
> > - .pin = 22,
> > - .func = 2,
> > - .route_offset = 0x184,
> > - .route_val = BIT(16 + 8),
> > - }, {
> > - /* pdm-m1 */
> > - .bank_num = 2,
> > - .pin = 22,
> > - .func = 1,
> > - .route_offset = 0x184,
> > - .route_val = BIT(16 + 8) | BIT(8),
> > - }, {
> > - /* uart2-rxm0 */
> > - .bank_num = 1,
> > - .pin = 27,
> > - .func = 2,
> > - .route_offset = 0x184,
> > - .route_val = BIT(16 + 10),
> > - }, {
> > - /* uart2-rxm1 */
> > - .bank_num = 2,
> > - .pin = 14,
> > - .func = 2,
> > - .route_offset = 0x184,
> > - .route_val = BIT(16 + 10) | BIT(10),
> > - }, {
> > - /* uart3-rxm0 */
> > - .bank_num = 0,
> > - .pin = 17,
> > - .func = 2,
> > - .route_offset = 0x184,
> > - .route_val = BIT(16 + 9),
> > - }, {
> > - /* uart3-rxm1 */
> > - .bank_num = 1,
> > - .pin = 15,
> > - .func = 2,
> > - .route_offset = 0x184,
> > - .route_val = BIT(16 + 9) | BIT(9),
> > - },
> > + RK_MUXROUTE_SAME(2, RK_PA0, 1, 0x184, BIT(16 + 7)), /* cif-d2m0 */
> > + RK_MUXROUTE_SAME(3, RK_PA3, 3, 0x184, BIT(16 + 7) | BIT(7)), /* cif-d2m1 */
> > + RK_MUXROUTE_SAME(3, RK_PC6, 2, 0x184, BIT(16 + 8)), /* pdm-m0 */
> > + RK_MUXROUTE_SAME(2, RK_PC6, 1, 0x184, BIT(16 + 8) | BIT(8)), /* pdm-m1 */
> > + RK_MUXROUTE_SAME(1, RK_PD3, 2, 0x184, BIT(16 + 10)), /* uart2-rxm0 */
> > + RK_MUXROUTE_SAME(2, RK_PB6, 2, 0x184, BIT(16 + 10) | BIT(10)), /* uart2-rxm1 */
> > + RK_MUXROUTE_SAME(0, RK_PC1, 2, 0x184, BIT(16 + 9)), /* uart3-rxm0 */
> > + RK_MUXROUTE_SAME(1, RK_PB7, 2, 0x184, BIT(16 + 9) | BIT(9)), /* uart3-rxm1 */
> >  };
> >  
> >  static struct rockchip_mux_route_data rk3128_mux_route_data[] = {
> > - {
> > - /* spi-0 */
> > - .bank_num = 1,
> > - .pin = 10,
> > - .func = 1,
> > - .route_offset = 0x144,
> > - .route_val = BIT(16 + 3) | BIT(16 + 4),
> > - }, {
> > - /* spi-1 */
> > - .bank_num = 1,
> > - .pin = 27,
> > - .func = 3,
> > - .route_offset = 0x144,
> > - .route_val = BIT(16 + 3) | BIT(16 + 4) | BIT(3),
> > - }, {
> > - /* spi-2 */
> > - .bank_num = 0,
> > - .pin = 13,
> > - .func = 2,
> > - .route_offset = 0x144,
> > - .route_val = BIT(16 + 3) | BIT(16 + 4) | BIT(4),
> > - }, {
> > - /* i2s-0 */
> > - .bank_num = 1,
> > - .pin = 5,
> > - .func = 1,
> > - .route_offset = 0x144,
> > - .route_val = BIT(16 + 5),
> > - }, {
> > - /* i2s-1 */
> > - .bank_num = 0,
> > - .pin = 14,
> > - .func = 1,
> > - .route_offset = 0x144,
> > - .route_val = BIT(16 + 5) | BIT(5),
> > - }, {
> > - /* emmc-0 */
> > - .bank_num = 1,
> > - .pin = 22,
> > - .func = 2,
> > - .route_offset = 0x144,
> > - .route_val = BIT(16 + 6),
> > - }, {
> > - /* emmc-1 */
> > - .bank_num = 2,
> > - .pin = 4,
> > - .func = 2,
> > - .route_offset = 0x144,
> > - .route_val = BIT(16 + 6) | BIT(6),
> > - },
> > + RK_MUXROUTE_SAME(1, RK_PB2, 1, 0x144, BIT(16 + 3) | BIT(16 + 4)), /* spi-0 */
> > + RK_MUXROUTE_SAME(1, RK_PD3, 3, 0x144, BIT(16 + 3) | BIT(16 + 4) | BIT(3)), /* spi-1 */
> > + RK_MUXROUTE_SAME(0, RK_PB5, 2, 0x144, BIT(16 + 3) | BIT(16 + 4) | BIT(4)), /* spi-2 */
> > + RK_MUXROUTE_SAME(1, RK_PA5, 1, 0x144, BIT(16 + 5)), /* i2s-0 */
> > + RK_MUXROUTE_SAME(0, RK_PB6, 1, 0x144, BIT(16 + 5) | BIT(5)), /* i2s-1 */
> > + RK_MUXROUTE_SAME(1, RK_PC6, 2, 0x144, BIT(16 + 6)), /* emmc-0 */
> > + RK_MUXROUTE_SAME(2, RK_PA4, 2, 0x144, BIT(16 + 6) | BIT(6)), /* emmc-1 */
> >  };
> >  
> >  static struct rockchip_mux_route_data rk3188_mux_route_data[] = {
> > - {
> > - /* non-iomuxed emmc/flash pins on flash-dqs */
> > - .bank_num = 0,
> > - .pin = 24,
> > - .func = 1,
> > - .route_location = ROCKCHIP_ROUTE_GRF,
> > - .route_offset = 0xa0,
> > - .route_val = BIT(16 + 11),
> > - }, {
> > - /* non-iomuxed emmc/flash pins on emmc-clk */
> > - .bank_num = 0,
> > - .pin = 24,
> > - .func = 2,
> > - .route_location = ROCKCHIP_ROUTE_GRF,
> > - .route_offset = 0xa0,
> > - .route_val = BIT(16 + 11) | BIT(11),
> > - },
> > + RK_MUXROUTE_SAME(0, RK_PD0, 1, 0xa0, BIT(16 + 11)), /* non-iomuxed emmc/flash pins on flash-dqs */
> > + RK_MUXROUTE_SAME(0, RK_PD0, 2, 0xa0, BIT(16 + 11) | BIT(11)), /* non-iomuxed emmc/flash pins on emmc-clk */
> >  };
> >  
> >  static struct rockchip_mux_route_data rk3228_mux_route_data[] = {
> > - {
> > - /* pwm0-0 */
> > - .bank_num = 0,
> > - .pin = 26,
> > - .func = 1,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16),
> > - }, {
> > - /* pwm0-1 */
> > - .bank_num = 3,
> > - .pin = 21,
> > - .func = 1,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16) | BIT(0),
> > - }, {
> > - /* pwm1-0 */
> > - .bank_num = 0,
> > - .pin = 27,
> > - .func = 1,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16 + 1),
> > - }, {
> > - /* pwm1-1 */
> > - .bank_num = 0,
> > - .pin = 30,
> > - .func = 2,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16 + 1) | BIT(1),
> > - }, {
> > - /* pwm2-0 */
> > - .bank_num = 0,
> > - .pin = 28,
> > - .func = 1,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16 + 2),
> > - }, {
> > - /* pwm2-1 */
> > - .bank_num = 1,
> > - .pin = 12,
> > - .func = 2,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16 + 2) | BIT(2),
> > - }, {
> > - /* pwm3-0 */
> > - .bank_num = 3,
> > - .pin = 26,
> > - .func = 1,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16 + 3),
> > - }, {
> > - /* pwm3-1 */
> > - .bank_num = 1,
> > - .pin = 11,
> > - .func = 2,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16 + 3) | BIT(3),
> > - }, {
> > - /* sdio-0_d0 */
> > - .bank_num = 1,
> > - .pin = 1,
> > - .func = 1,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16 + 4),
> > - }, {
> > - /* sdio-1_d0 */
> > - .bank_num = 3,
> > - .pin = 2,
> > - .func = 1,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16 + 4) | BIT(4),
> > - }, {
> > - /* spi-0_rx */
> > - .bank_num = 0,
> > - .pin = 13,
> > - .func = 2,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16 + 5),
> > - }, {
> > - /* spi-1_rx */
> > - .bank_num = 2,
> > - .pin = 0,
> > - .func = 2,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16 + 5) | BIT(5),
> > - }, {
> > - /* emmc-0_cmd */
> > - .bank_num = 1,
> > - .pin = 22,
> > - .func = 2,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16 + 7),
> > - }, {
> > - /* emmc-1_cmd */
> > - .bank_num = 2,
> > - .pin = 4,
> > - .func = 2,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16 + 7) | BIT(7),
> > - }, {
> > - /* uart2-0_rx */
> > - .bank_num = 1,
> > - .pin = 19,
> > - .func = 2,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16 + 8),
> > - }, {
> > - /* uart2-1_rx */
> > - .bank_num = 1,
> > - .pin = 10,
> > - .func = 2,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16 + 8) | BIT(8),
> > - }, {
> > - /* uart1-0_rx */
> > - .bank_num = 1,
> > - .pin = 10,
> > - .func = 1,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16 + 11),
> > - }, {
> > - /* uart1-1_rx */
> > - .bank_num = 3,
> > - .pin = 13,
> > - .func = 1,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16 + 11) | BIT(11),
> > - },
> > + RK_MUXROUTE_SAME(0, RK_PD2, 1, 0x50, BIT(16)), /* pwm0-0 */
> > + RK_MUXROUTE_SAME(3, RK_PC5, 1, 0x50, BIT(16) | BIT(0)), /* pwm0-1 */
> > + RK_MUXROUTE_SAME(0, RK_PD3, 1, 0x50, BIT(16 + 1)), /* pwm1-0 */
> > + RK_MUXROUTE_SAME(0, RK_PD6, 2, 0x50, BIT(16 + 1) | BIT(1)), /* pwm1-1 */
> > + RK_MUXROUTE_SAME(0, RK_PD4, 1, 0x50, BIT(16 + 2)), /* pwm2-0 */
> > + RK_MUXROUTE_SAME(1, RK_PB4, 2, 0x50, BIT(16 + 2) | BIT(2)), /* pwm2-1 */
> > + RK_MUXROUTE_SAME(3, RK_PD2, 1, 0x50, BIT(16 + 3)), /* pwm3-0 */
> > + RK_MUXROUTE_SAME(1, RK_PB3, 2, 0x50, BIT(16 + 3) | BIT(3)), /* pwm3-1 */
> > + RK_MUXROUTE_SAME(1, RK_PA1, 1, 0x50, BIT(16 + 4)), /* sdio-0_d0 */
> > + RK_MUXROUTE_SAME(3, RK_PA2, 1, 0x50, BIT(16 + 4) | BIT(4)), /* sdio-1_d0 */
> > + RK_MUXROUTE_SAME(0, RK_PB5, 2, 0x50, BIT(16 + 5)), /* spi-0_rx */
> > + RK_MUXROUTE_SAME(2, RK_PA0, 2, 0x50, BIT(16 + 5) | BIT(5)), /* spi-1_rx */
> > + RK_MUXROUTE_SAME(1, RK_PC6, 2, 0x50, BIT(16 + 7)), /* emmc-0_cmd */
> > + RK_MUXROUTE_SAME(2, RK_PA4, 2, 0x50, BIT(16 + 7) | BIT(7)), /* emmc-1_cmd */
> > + RK_MUXROUTE_SAME(1, RK_PC3, 2, 0x50, BIT(16 + 8)), /* uart2-0_rx */
> > + RK_MUXROUTE_SAME(1, RK_PB2, 2, 0x50, BIT(16 + 8) | BIT(8)), /* uart2-1_rx */
> > + RK_MUXROUTE_SAME(1, RK_PB2, 1, 0x50, BIT(16 + 11)), /* uart1-0_rx */
> > + RK_MUXROUTE_SAME(3, RK_PB5, 1, 0x50, BIT(16 + 11) | BIT(11)), /* uart1-1_rx */
> >  };
> >  
> >  static struct rockchip_mux_route_data rk3288_mux_route_data[] = {
> > - {
> > - /* edphdmi_cecinoutt1 */
> > - .bank_num = 7,
> > - .pin = 16,
> > - .func = 2,
> > - .route_offset = 0x264,
> > - .route_val = BIT(16 + 12) | BIT(12),
> > - }, {
> > - /* edphdmi_cecinout */
> > - .bank_num = 7,
> > - .pin = 23,
> > - .func = 4,
> > - .route_offset = 0x264,
> > - .route_val = BIT(16 + 12),
> > - },
> > + RK_MUXROUTE_SAME(7, RK_PC0, 2, 0x264, BIT(16 + 12) | BIT(12)), /* edphdmi_cecinoutt1 */
> > + RK_MUXROUTE_SAME(7, RK_PC7, 4, 0x264, BIT(16 + 12)), /* edphdmi_cecinout */
> >  };
> >  
> >  static struct rockchip_mux_route_data rk3308_mux_route_data[] = {
> > - {
> > - /* rtc_clk */
> > - .bank_num = 0,
> > - .pin = 19,
> > - .func = 1,
> > - .route_offset = 0x314,
> > - .route_val = BIT(16 + 0) | BIT(0),
> > - }, {
> > - /* uart2_rxm0 */
> > - .bank_num = 1,
> > - .pin = 22,
> > - .func = 2,
> > - .route_offset = 0x314,
> > - .route_val = BIT(16 + 2) | BIT(16 + 3),
> > - }, {
> > - /* uart2_rxm1 */
> > - .bank_num = 4,
> > - .pin = 26,
> > - .func = 2,
> > - .route_offset = 0x314,
> > - .route_val = BIT(16 + 2) | BIT(16 + 3) | BIT(2),
> > - }, {
> > - /* i2c3_sdam0 */
> > - .bank_num = 0,
> > - .pin = 15,
> > - .func = 2,
> > - .route_offset = 0x608,
> > - .route_val = BIT(16 + 8) | BIT(16 + 9),
> > - }, {
> > - /* i2c3_sdam1 */
> > - .bank_num = 3,
> > - .pin = 12,
> > - .func = 2,
> > - .route_offset = 0x608,
> > - .route_val = BIT(16 + 8) | BIT(16 + 9) | BIT(8),
> > - }, {
> > - /* i2c3_sdam2 */
> > - .bank_num = 2,
> > - .pin = 0,
> > - .func = 3,
> > - .route_offset = 0x608,
> > - .route_val = BIT(16 + 8) | BIT(16 + 9) | BIT(9),
> > - }, {
> > - /* i2s-8ch-1-sclktxm0 */
> > - .bank_num = 1,
> > - .pin = 3,
> > - .func = 2,
> > - .route_offset = 0x308,
> > - .route_val = BIT(16 + 3),
> > - }, {
> > - /* i2s-8ch-1-sclkrxm0 */
> > - .bank_num = 1,
> > - .pin = 4,
> > - .func = 2,
> > - .route_offset = 0x308,
> > - .route_val = BIT(16 + 3),
> > - }, {
> > - /* i2s-8ch-1-sclktxm1 */
> > - .bank_num = 1,
> > - .pin = 13,
> > - .func = 2,
> > - .route_offset = 0x308,
> > - .route_val = BIT(16 + 3) | BIT(3),
> > - }, {
> > - /* i2s-8ch-1-sclkrxm1 */
> > - .bank_num = 1,
> > - .pin = 14,
> > - .func = 2,
> > - .route_offset = 0x308,
> > - .route_val = BIT(16 + 3) | BIT(3),
> > - }, {
> > - /* pdm-clkm0 */
> > - .bank_num = 1,
> > - .pin = 4,
> > - .func = 3,
> > - .route_offset = 0x308,
> > - .route_val =  BIT(16 + 12) | BIT(16 + 13),
> > - }, {
> > - /* pdm-clkm1 */
> > - .bank_num = 1,
> > - .pin = 14,
> > - .func = 4,
> > - .route_offset = 0x308,
> > - .route_val = BIT(16 + 12) | BIT(16 + 13) | BIT(12),
> > - }, {
> > - /* pdm-clkm2 */
> > - .bank_num = 2,
> > - .pin = 6,
> > - .func = 2,
> > - .route_offset = 0x308,
> > - .route_val = BIT(16 + 12) | BIT(16 + 13) | BIT(13),
> > - }, {
> > - /* pdm-clkm-m2 */
> > - .bank_num = 2,
> > - .pin = 4,
> > - .func = 3,
> > - .route_offset = 0x600,
> > - .route_val = BIT(16 + 2) | BIT(2),
> > - }, {
> > - /* spi1_miso */
> > - .bank_num = 3,
> > - .pin = 10,
> > - .func = 3,
> > - .route_offset = 0x314,
> > - .route_val = BIT(16 + 9),
> > - }, {
> > - /* spi1_miso_m1 */
> > - .bank_num = 2,
> > - .pin = 4,
> > - .func = 2,
> > - .route_offset = 0x314,
> > - .route_val = BIT(16 + 9) | BIT(9),
> > - }, {
> > - /* owire_m0 */
> > - .bank_num = 0,
> > - .pin = 11,
> > - .func = 3,
> > - .route_offset = 0x314,
> > - .route_val = BIT(16 + 10) | BIT(16 + 11),
> > - }, {
> > - /* owire_m1 */
> > - .bank_num = 1,
> > - .pin = 22,
> > - .func = 7,
> > - .route_offset = 0x314,
> > - .route_val = BIT(16 + 10) | BIT(16 + 11) | BIT(10),
> > - }, {
> > - /* owire_m2 */
> > - .bank_num = 2,
> > - .pin = 2,
> > - .func = 5,
> > - .route_offset = 0x314,
> > - .route_val = BIT(16 + 10) | BIT(16 + 11) | BIT(11),
> > - }, {
> > - /* can_rxd_m0 */
> > - .bank_num = 0,
> > - .pin = 11,
> > - .func = 2,
> > - .route_offset = 0x314,
> > - .route_val = BIT(16 + 12) | BIT(16 + 13),
> > - }, {
> > - /* can_rxd_m1 */
> > - .bank_num = 1,
> > - .pin = 22,
> > - .func = 5,
> > - .route_offset = 0x314,
> > - .route_val = BIT(16 + 12) | BIT(16 + 13) | BIT(12),
> > - }, {
> > - /* can_rxd_m2 */
> > - .bank_num = 2,
> > - .pin = 2,
> > - .func = 4,
> > - .route_offset = 0x314,
> > - .route_val = BIT(16 + 12) | BIT(16 + 13) | BIT(13),
> > - }, {
> > - /* mac_rxd0_m0 */
> > - .bank_num = 1,
> > - .pin = 20,
> > - .func = 3,
> > - .route_offset = 0x314,
> > - .route_val = BIT(16 + 14),
> > - }, {
> > - /* mac_rxd0_m1 */
> > - .bank_num = 4,
> > - .pin = 2,
> > - .func = 2,
> > - .route_offset = 0x314,
> > - .route_val = BIT(16 + 14) | BIT(14),
> > - }, {
> > - /* uart3_rx */
> > - .bank_num = 3,
> > - .pin = 12,
> > - .func = 4,
> > - .route_offset = 0x314,
> > - .route_val = BIT(16 + 15),
> > - }, {
> > - /* uart3_rx_m1 */
> > - .bank_num = 0,
> > - .pin = 17,
> > - .func = 3,
> > - .route_offset = 0x314,
> > - .route_val = BIT(16 + 15) | BIT(15),
> > - },
> > + RK_MUXROUTE_SAME(0, RK_PC3, 1, 0x314, BIT(16 + 0) | BIT(0)), /* rtc_clk */
> > + RK_MUXROUTE_SAME(1, RK_PC6, 2, 0x314, BIT(16 + 2) | BIT(16 + 3)), /* uart2_rxm0 */
> > + RK_MUXROUTE_SAME(4, RK_PD2, 2, 0x314, BIT(16 + 2) | BIT(16 + 3) | BIT(2)), /* uart2_rxm1 */
> > + RK_MUXROUTE_SAME(0, RK_PB7, 2, 0x608, BIT(16 + 8) | BIT(16 + 9)), /* i2c3_sdam0 */
> > + RK_MUXROUTE_SAME(3, RK_PB4, 2, 0x608, BIT(16 + 8) | BIT(16 + 9) | BIT(8)), /* i2c3_sdam1 */
> > + RK_MUXROUTE_SAME(2, RK_PA0, 3, 0x608, BIT(16 + 8) | BIT(16 + 9) | BIT(9)), /* i2c3_sdam2 */
> > + RK_MUXROUTE_SAME(1, RK_PA3, 2, 0x308, BIT(16 + 3)), /* i2s-8ch-1-sclktxm0 */
> > + RK_MUXROUTE_SAME(1, RK_PA4, 2, 0x308, BIT(16 + 3)), /* i2s-8ch-1-sclkrxm0 */
> > + RK_MUXROUTE_SAME(1, RK_PB5, 2, 0x308, BIT(16 + 3) | BIT(3)), /* i2s-8ch-1-sclktxm1 */
> > + RK_MUXROUTE_SAME(1, RK_PB6, 2, 0x308, BIT(16 + 3) | BIT(3)), /* i2s-8ch-1-sclkrxm1 */
> > + RK_MUXROUTE_SAME(1, RK_PA4, 3, 0x308, BIT(16 + 12) | BIT(16 + 13)), /* pdm-clkm0 */
> > + RK_MUXROUTE_SAME(1, RK_PB6, 4, 0x308, BIT(16 + 12) | BIT(16 + 13) | BIT(12)), /* pdm-clkm1 */
> > + RK_MUXROUTE_SAME(2, RK_PA6, 2, 0x308, BIT(16 + 12) | BIT(16 + 13) | BIT(13)), /* pdm-clkm2 */
> > + RK_MUXROUTE_SAME(2, RK_PA4, 3, 0x600, BIT(16 + 2) | BIT(2)), /* pdm-clkm-m2 */
> > + RK_MUXROUTE_SAME(3, RK_PB2, 3, 0x314, BIT(16 + 9)), /* spi1_miso */
> > + RK_MUXROUTE_SAME(2, RK_PA4, 2, 0x314, BIT(16 + 9) | BIT(9)), /* spi1_miso_m1 */
> > + RK_MUXROUTE_SAME(0, RK_PB3, 3, 0x314, BIT(16 + 10) | BIT(16 + 11)), /* owire_m0 */
> > + RK_MUXROUTE_SAME(1, RK_PC6, 7, 0x314, BIT(16 + 10) | BIT(16 + 11) | BIT(10)), /* owire_m1 */
> > + RK_MUXROUTE_SAME(2, RK_PA2, 5, 0x314, BIT(16 + 10) | BIT(16 + 11) | BIT(11)), /* owire_m2 */
> > + RK_MUXROUTE_SAME(0, RK_PB3, 2, 0x314, BIT(16 + 12) | BIT(16 + 13)), /* can_rxd_m0 */
> > + RK_MUXROUTE_SAME(1, RK_PC6, 5, 0x314, BIT(16 + 12) | BIT(16 + 13) | BIT(12)), /* can_rxd_m1 */
> > + RK_MUXROUTE_SAME(2, RK_PA2, 4, 0x314, BIT(16 + 12) | BIT(16 + 13) | BIT(13)), /* can_rxd_m2 */
> > + RK_MUXROUTE_SAME(1, RK_PC4, 3, 0x314, BIT(16 + 14)), /* mac_rxd0_m0 */
> > + RK_MUXROUTE_SAME(4, RK_PA2, 2, 0x314, BIT(16 + 14) | BIT(14)), /* mac_rxd0_m1 */
> > + RK_MUXROUTE_SAME(3, RK_PB4, 4, 0x314, BIT(16 + 15)), /* uart3_rx */
> > + RK_MUXROUTE_SAME(0, RK_PC1, 3, 0x314, BIT(16 + 15) | BIT(15)), /* uart3_rx_m1 */
> >  };
> >  
> >  static struct rockchip_mux_route_data rk3328_mux_route_data[] = {
> > - {
> > - /* uart2dbg_rxm0 */
> > - .bank_num = 1,
> > - .pin = 1,
> > - .func = 2,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16) | BIT(16 + 1),
> > - }, {
> > - /* uart2dbg_rxm1 */
> > - .bank_num = 2,
> > - .pin = 1,
> > - .func = 1,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16) | BIT(16 + 1) | BIT(0),
> > - }, {
> > - /* gmac-m1_rxd0 */
> > - .bank_num = 1,
> > - .pin = 11,
> > - .func = 2,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16 + 2) | BIT(2),
> > - }, {
> > - /* gmac-m1-optimized_rxd3 */
> > - .bank_num = 1,
> > - .pin = 14,
> > - .func = 2,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16 + 10) | BIT(10),
> > - }, {
> > - /* pdm_sdi0m0 */
> > - .bank_num = 2,
> > - .pin = 19,
> > - .func = 2,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16 + 3),
> > - }, {
> > - /* pdm_sdi0m1 */
> > - .bank_num = 1,
> > - .pin = 23,
> > - .func = 3,
> > - .route_offset = 0x50,
> > - .route_val =  BIT(16 + 3) | BIT(3),
> > - }, {
> > - /* spi_rxdm2 */
> > - .bank_num = 3,
> > - .pin = 2,
> > - .func = 4,
> > - .route_offset = 0x50,
> > - .route_val =  BIT(16 + 4) | BIT(16 + 5) | BIT(5),
> > - }, {
> > - /* i2s2_sdim0 */
> > - .bank_num = 1,
> > - .pin = 24,
> > - .func = 1,
> > - .route_offset = 0x50,
> > - .route_val = BIT(16 + 6),
> > - }, {
> > - /* i2s2_sdim1 */
> > - .bank_num = 3,
> > - .pin = 2,
> > - .func = 6,
> > - .route_offset = 0x50,
> > - .route_val =  BIT(16 + 6) | BIT(6),
> > - }, {
> > - /* card_iom1 */
> > - .bank_num = 2,
> > - .pin = 22,
> > - .func = 3,
> > - .route_offset = 0x50,
> > - .route_val =  BIT(16 + 7) | BIT(7),
> > - }, {
> > - /* tsp_d5m1 */
> > - .bank_num = 2,
> > - .pin = 16,
> > - .func = 3,
> > - .route_offset = 0x50,
> > - .route_val =  BIT(16 + 8) | BIT(8),
> > - }, {
> > - /* cif_data5m1 */
> > - .bank_num = 2,
> > - .pin = 16,
> > - .func = 4,
> > - .route_offset = 0x50,
> > - .route_val =  BIT(16 + 9) | BIT(9),
> > - },
> > + RK_MUXROUTE_SAME(1, RK_PA1, 2, 0x50, BIT(16) | BIT(16 + 1)), /* uart2dbg_rxm0 */
> > + RK_MUXROUTE_SAME(2, RK_PA1, 1, 0x50, BIT(16) | BIT(16 + 1) | BIT(0)), /* uart2dbg_rxm1 */
> > + RK_MUXROUTE_SAME(1, RK_PB3, 2, 0x50, BIT(16 + 2) | BIT(2)), /* gmac-m1_rxd0 */
> > + RK_MUXROUTE_SAME(1, RK_PB6, 2, 0x50, BIT(16 + 10) | BIT(10)), /* gmac-m1-optimized_rxd3 */
> > + RK_MUXROUTE_SAME(2, RK_PC3, 2, 0x50, BIT(16 + 3)), /* pdm_sdi0m0 */
> > + RK_MUXROUTE_SAME(1, RK_PC7, 3, 0x50, BIT(16 + 3) | BIT(3)), /* pdm_sdi0m1 */
> > + RK_MUXROUTE_SAME(3, RK_PA2, 4, 0x50, BIT(16 + 4) | BIT(16 + 5) | BIT(5)), /* spi_rxdm2 */
> > + RK_MUXROUTE_SAME(1, RK_PD0, 1, 0x50, BIT(16 + 6)), /* i2s2_sdim0 */
> > + RK_MUXROUTE_SAME(3, RK_PA2, 6, 0x50, BIT(16 + 6) | BIT(6)), /* i2s2_sdim1 */
> > + RK_MUXROUTE_SAME(2, RK_PC6, 3, 0x50, BIT(16 + 7) | BIT(7)), /* card_iom1 */
> > + RK_MUXROUTE_SAME(2, RK_PC0, 3, 0x50, BIT(16 + 8) | BIT(8)), /* tsp_d5m1 */
> > + RK_MUXROUTE_SAME(2, RK_PC0, 4, 0x50, BIT(16 + 9) | BIT(9)), /* cif_data5m1 */
> >  };
> >  
> >  static struct rockchip_mux_route_data rk3399_mux_route_data[] = {
> > - {
> > - /* uart2dbga_rx */
> > - .bank_num = 4,
> > - .pin = 8,
> > - .func = 2,
> > - .route_offset = 0xe21c,
> > - .route_val = BIT(16 + 10) | BIT(16 + 11),
> > - }, {
> > - /* uart2dbgb_rx */
> > - .bank_num = 4,
> > - .pin = 16,
> > - .func = 2,
> > - .route_offset = 0xe21c,
> > - .route_val = BIT(16 + 10) | BIT(16 + 11) | BIT(10),
> > - }, {
> > - /* uart2dbgc_rx */
> > - .bank_num = 4,
> > - .pin = 19,
> > - .func = 1,
> > - .route_offset = 0xe21c,
> > - .route_val = BIT(16 + 10) | BIT(16 + 11) | BIT(11),
> > - }, {
> > - /* pcie_clkreqn */
> > - .bank_num = 2,
> > - .pin = 26,
> > - .func = 2,
> > - .route_offset = 0xe21c,
> > - .route_val = BIT(16 + 14),
> > - }, {
> > - /* pcie_clkreqnb */
> > - .bank_num = 4,
> > - .pin = 24,
> > - .func = 1,
> > - .route_offset = 0xe21c,
> > - .route_val = BIT(16 + 14) | BIT(14),
> > - },
> > + RK_MUXROUTE_SAME(4, RK_PB0, 2, 0xe21c, BIT(16 + 10) | BIT(16 + 11)), /* uart2dbga_rx */
> > + RK_MUXROUTE_SAME(4, RK_PC0, 2, 0xe21c, BIT(16 + 10) | BIT(16 + 11) | BIT(10)), /* uart2dbgb_rx */
> > + RK_MUXROUTE_SAME(4, RK_PC3, 1, 0xe21c, BIT(16 + 10) | BIT(16 + 11) | BIT(11)), /* uart2dbgc_rx */
> > + RK_MUXROUTE_SAME(2, RK_PD2, 2, 0xe21c, BIT(16 + 14)), /* pcie_clkreqn */
> > + RK_MUXROUTE_SAME(4, RK_PD0, 1, 0xe21c, BIT(16 + 14) | BIT(14)), /* pcie_clkreqnb */
> >  };
> >  
> >  static bool rockchip_get_mux_route(struct rockchip_pin_bank *bank, int pin,
> > 
>  
>  
>  
>  
>  
>  
>  
>