Remove various gratuitous blank lines and other minor whitespace and style cleanups. Add some missing commas within group arrays to prevent the compiler from concatenating the strings. Remove unneeded TEGRA_MUX_SAFE entry from enumeration and drop gratuitous _dt suffix from tegra_mux enumeration type name. Drop an unwarranted FIXME comment and rename gmi_ad8_ph0 pin to ph0 since that's how it's referenced in the DS06472001 datasheet. Signed-off-by: Thierry Reding <treding@xxxxxxxxxx> --- drivers/pinctrl/pinctrl-tegra124.c | 27 ++++++++------------------- 1 file changed, 8 insertions(+), 19 deletions(-) diff --git a/drivers/pinctrl/pinctrl-tegra124.c b/drivers/pinctrl/pinctrl-tegra124.c index 592fd6f079b6..632e076115d5 100644 --- a/drivers/pinctrl/pinctrl-tegra124.c +++ b/drivers/pinctrl/pinctrl-tegra124.c @@ -211,7 +211,6 @@ #define TEGRA_PIN_USB_VBUS_EN2_PFF1 _GPIO(249) #define TEGRA_PIN_PFF2 _GPIO(250) - /* All non-GPIO pins follow */ #define NUM_GPIOS (TEGRA_PIN_PFF2 + 1) #define _PIN(offset) (NUM_GPIOS + (offset)) @@ -1183,8 +1182,6 @@ static const unsigned jtag_rtck_pins[] = { TEGRA_PIN_JTAG_RTCK, }; - -/*FIXME */ static const unsigned drive_ao1_pins[] = { TEGRA_PIN_KB_ROW0_PR0, TEGRA_PIN_KB_ROW1_PR1, @@ -1242,7 +1239,7 @@ static const unsigned drive_at2_pins[] = { TEGRA_PIN_PG6, TEGRA_PIN_PG7, TEGRA_PIN_PI0, - TEGRA_PIN_PI0, + TEGRA_PIN_PI1, TEGRA_PIN_PI3, TEGRA_PIN_PI4, TEGRA_PIN_PI7, @@ -1497,8 +1494,7 @@ static const unsigned drive_ao4_pins[] = { TEGRA_PIN_JTAG_RTCK, }; -enum tegra_mux_dt { - TEGRA_MUX_SAFE, +enum tegra_mux { TEGRA_MUX_BLINK, TEGRA_MUX_CEC, TEGRA_MUX_CLDVFS, @@ -1771,7 +1767,6 @@ static const char * const gmi_groups[] = { "sdmmc4_clk_pcc4", "sdmmc4_cmd_pt7", "gmi_clk_lb", - "gpio_x6_aud_px6", "dap1_fs_pn0", "dap1_din_pn1", @@ -1888,7 +1883,7 @@ static const char * const irda_groups[] = { "uart2_rxd_pc3", "uart2_txd_pc2", "kb_row11_ps3", - "kb_row12_ps4" + "kb_row12_ps4", }; static const char * const kbc_groups[] = { @@ -1936,7 +1931,7 @@ static const char * const pwm0_groups[] = { "sdmmc1_dat2_py5", "uart3_rts_n_pc0", "pu3", - "gmi_ad8_ph0", + "ph0", "sdmmc3_dat3_pb4", }; @@ -2001,8 +1996,6 @@ static const char * const rsvd1_groups[] = { "reset_out_n", }; - - static const char * const rsvd2_groups[] = { "pv0", "pv1", @@ -2194,7 +2187,7 @@ static const char * const rsvd4_groups[] = { "dap3_dout_pp2", "pv0", "pv1", - "sdmmc1_clk_pz0" + "sdmmc1_clk_pz0", "clk2_out_pw5", "clk2_req_pcc5", @@ -2305,7 +2298,7 @@ static const char * const rsvd4_groups[] = { "usb_vbus_en1_pn5", "sdmmc3_clk_lb_out_pee4", "sdmmc3_clk_lb_in_pee5", - "gmi_clk_lb" + "gmi_clk_lb", "dp_hpd_pff0", "usb_vbus_en2_pff1", @@ -2313,7 +2306,6 @@ static const char * const rsvd4_groups[] = { }; static const char * const sdmmc1_groups[] = { - "sdmmc1_clk_pz0", "sdmmc1_cmd_pz1", "sdmmc1_dat3_py4", @@ -2700,7 +2692,7 @@ static const char * const tmds_groups[] = { .ngroups = ARRAY_SIZE(fname##_groups), \ } -static const struct tegra_function tegra124_functions[] = { +static const struct tegra_function tegra124_functions[] = { FUNCTION(blink), FUNCTION(cec), FUNCTION(cldvfs), @@ -3105,7 +3097,6 @@ static const struct tegra_pingroup tegra124_groups[] = { DRV_PINGROUP(hv0, 0x9b4, 2, 3, 4, 12, 5, -1, -1, 28, 2, -1, -1, N), DRV_PINGROUP(sdio4, 0x9c4, 2, 3, 4, 12, 5, 20, 5, 28, 2, 30, 2, N), DRV_PINGROUP(ao4, 0x9c8, 2, 3, 4, 12, 7, 20, 7, 28, 2, 30, 2, Y), - }; static const struct tegra_pinctrl_soc_data tegra124_pinctrl = { @@ -3127,6 +3118,7 @@ static struct of_device_id tegra124_pinctrl_of_match[] = { { .compatible = "nvidia,tegra124-pinmux", }, { }, }; +MODULE_DEVICE_TABLE(of, tegra124_pinctrl_of_match); static struct platform_driver tegra124_pinctrl_driver = { .driver = { @@ -3137,11 +3129,8 @@ static struct platform_driver tegra124_pinctrl_driver = { .probe = tegra124_pinctrl_probe, .remove = tegra_pinctrl_remove, }; - module_platform_driver(tegra124_pinctrl_driver); MODULE_AUTHOR("Ashwini Ghuge <aghuge@xxxxxxxxxx>"); MODULE_DESCRIPTION("NVIDIA Tegra124 pinctrl driver"); MODULE_LICENSE("GPL v2"); -MODULE_DEVICE_TABLE(of, tegra124_pinctrl_of_match); - -- 1.8.4.2 -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html