lkml.org 
[lkml]   [2026]   [Apr]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 2/2] pinctrl: qcom: Add Shikra pinctrl driver
From


On 4/29/2026 6:41 PM, Komal Bajaj wrote:
> Add pinctrl driver for TLMM block found in Shikra SoC.


[...]

> +#define UFS_RESET(pg_name, ctl, io) \
> + { \
> + .grp = PINCTRL_PINGROUP(#pg_name, \
> + pg_name##_pins, \
> + ARRAY_SIZE(pg_name##_pins)), \
> + .ctl_reg = ctl, \
> + .io_reg = io, \
> + .intr_cfg_reg = 0, \
> + .intr_status_reg = 0, \
> + .mux_bit = -1, \
> + .pull_bit = 3, \
> + .drv_bit = 0, \
> + .oe_bit = -1, \
> + .in_bit = -1, \
> + .out_bit = 0, \
> + .intr_enable_bit = -1, \
> + .intr_status_bit = -1, \
> + .intr_target_bit = -1, \
> + .intr_raw_status_bit = -1, \
> + .intr_polarity_bit = -1, \
> + .intr_detection_bit = -1, \
> + .intr_detection_width = -1, \
> + }

UFS_RESET macro is not used anywhere in the file, please remove it.

I assume the macro keeps getting added since the file pinctrl-<target> is
fully/partially auto generated. Would be good to fix auto generation to
avoid getting this added as default in future.

The unused ones were removed via [1].

[1] https://lore.kernel.org/all/4429f44e-f7e5-449c-824c-83daa339b383@oss.qualcomm.com/

[...]


> +static const struct msm_gpio_wakeirq_map shikra_mpm_map[] = {
> + {1, 9}, {2, 31}, {5, 49}, {6, 53}, {9, 72}, {10, 10},

It would be better to have spacing before/after brackets inline with other pinctrl drivers.
{1, 9}, should be { 1, 9 }.

> + {12, 22}, {14, 26}, {17, 29}, {18, 24}, {20, 32}, {22, 33},
> + {25, 34}, {27, 35}, {28, 36}, {29, 37}, {30, 38}, {31, 39},
> + {32, 40}, {33, 41}, {38, 42}, {40, 43}, {43, 44}, {44, 45},
> + {45, 46}, {46, 47}, {47, 48}, {48, 60}, {50, 50}, {51, 51},
> + {52, 61}, {53, 62}, {57, 52}, {58, 63}, {60, 54}, {63, 64},
> + {73, 55}, {74, 56}, {75, 57}, {77, 3}, {80, 4}, {84, 5},
> + {85, 67}, {86, 69}, {88, 70}, {89, 71}, {90, 73}, {91, 74},
> + {92, 75}, {93, 76}, {94, 77}, {95, 78}, {97, 79}, {99, 80},
> + {100, 11}, {101, 13}, {102, 14}, {103, 15}, {106, 16}, {108, 17},
> + {112, 18}, {116, 19}, {117, 20}, {119, 21}, {120, 23}, {136, 25},
> + {159, 27}, {161, 28},
> +};
> +

Thanks,
Maulik


\
 
 \ /
  Last update: 2026-04-30 06:07    [W:0.189 / U:11.510 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and my Meterkast|Read the blog