On Wed, 11 Jul 2018 09:30:57 +0100
Jon Hunter <jonath...@nvidia.com> wrote:
> On 10/07/18 13:47, Aapo Vienamo wrote:
> > Refactor the IO pad tables into macro tables so that they can be reused
> > to generate pinctrl pin descriptors. Also add a name field which is
> > needed by pinctrl.
> >
> > Signed-off-by: Aapo Vienamo <avien...@nvidia.com>
> > ---
> > drivers/soc/tegra/pmc.c | 231
> > ++++++++++++++++++++++++++----------------------
> > 1 file changed, 126 insertions(+), 105 deletions(-)
> >
> > diff --git a/drivers/soc/tegra/pmc.c b/drivers/soc/tegra/pmc.c
> > index 3f5b69fd..b833334 100644
> > --- a/drivers/soc/tegra/pmc.c
> > +++ b/drivers/soc/tegra/pmc.c
> > @@ -137,6 +137,7 @@ struct tegra_io_pad_soc {
> > enum tegra_io_pad id;
> > unsigned int dpd;
> > unsigned int voltage;
> > + const char *name;
> > };
> >
> > struct tegra_pmc_regs {
> > @@ -1697,37 +1698,49 @@ static const u8 tegra124_cpu_powergates[] = {
> > TEGRA_POWERGATE_CPU3,
> > };
> >
> > +#define TEGRA_IO_PAD(_id, _dpd, _voltage, _name) \
> > + (struct tegra_io_pad_soc) { \
> > + .id = _id, \
> > + .dpd = _dpd, \
> > + .voltage = _voltage, \
> > + .name = _name, \
> > + },
> > +
> > +#define TEGRA124_IO_PAD_TABLE(_pad)
> > \
> > + /* .id .dpd .voltage .name */ \
> > + _pad(TEGRA_IO_PAD_AUDIO, 17, UINT_MAX, "audio") \
> > + _pad(TEGRA_IO_PAD_BB, 15, UINT_MAX, "bb") \
> > + _pad(TEGRA_IO_PAD_CAM, 36, UINT_MAX, "cam") \
> > + _pad(TEGRA_IO_PAD_COMP, 22, UINT_MAX, "comp") \
> > + _pad(TEGRA_IO_PAD_CSIA, 0, UINT_MAX, "csia") \
> > + _pad(TEGRA_IO_PAD_CSIB, 1, UINT_MAX, "csb") \
> > + _pad(TEGRA_IO_PAD_CSIE, 44, UINT_MAX, "cse") \
> > + _pad(TEGRA_IO_PAD_DSI, 2, UINT_MAX, "dsi") \
> > + _pad(TEGRA_IO_PAD_DSIB, 39, UINT_MAX, "dsib") \
> > + _pad(TEGRA_IO_PAD_DSIC, 40, UINT_MAX, "dsic") \
> > + _pad(TEGRA_IO_PAD_DSID, 41, UINT_MAX, "dsid") \
> > + _pad(TEGRA_IO_PAD_HDMI, 28, UINT_MAX, "hdmi") \
> > + _pad(TEGRA_IO_PAD_HSIC, 19, UINT_MAX, "hsic") \
> > + _pad(TEGRA_IO_PAD_HV, 38, UINT_MAX, "hv") \
> > + _pad(TEGRA_IO_PAD_LVDS, 57, UINT_MAX, "lvds") \
> > + _pad(TEGRA_IO_PAD_MIPI_BIAS, 3, UINT_MAX, "mipi-bias") \
> > + _pad(TEGRA_IO_PAD_NAND, 13, UINT_MAX, "nand") \
> > + _pad(TEGRA_IO_PAD_PEX_BIAS, 4, UINT_MAX, "pex-bias") \
> > + _pad(TEGRA_IO_PAD_PEX_CLK1, 5, UINT_MAX, "pex-clk1") \
> > + _pad(TEGRA_IO_PAD_PEX_CLK2, 6, UINT_MAX, "pex-clk2") \
> > + _pad(TEGRA_IO_PAD_PEX_CNTRL, 32, UINT_MAX, "pex-cntrl") \
> > + _pad(TEGRA_IO_PAD_SDMMC1, 33, UINT_MAX, "sdmmc1") \
> > + _pad(TEGRA_IO_PAD_SDMMC3, 34, UINT_MAX, "sdmmc3") \
> > + _pad(TEGRA_IO_PAD_SDMMC4, 35, UINT_MAX, "sdmmc4") \
> > + _pad(TEGRA_IO_PAD_SYS_DDC, 58, UINT_MAX, "sys_ddc") \
> > + _pad(TEGRA_IO_PAD_UART, 14, UINT_MAX, "uart") \
> > + _pad(TEGRA_IO_PAD_USB0, 9, UINT_MAX, "usb0") \
> > + _pad(TEGRA_IO_PAD_USB1, 10, UINT_MAX, "usb1") \
> > + _pad(TEGRA_IO_PAD_USB2, 11, UINT_MAX, "usb2") \
> > + _pad(TEGRA_IO_PAD_USB_BIAS, 12, UINT_MAX, "usb_bias")
> > +
> > static const struct tegra_io_pad_soc tegra124_io_pads[] = {
> > - { .id = TEGRA_IO_PAD_AUDIO, .dpd = 17, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_BB, .dpd = 15, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_CAM, .dpd = 36, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_COMP, .dpd = 22, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_CSIA, .dpd = 0, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_CSIB, .dpd = 1, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_CSIE, .dpd = 44, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_DSI, .dpd = 2, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_DSIB, .dpd = 39, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_DSIC, .dpd = 40, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_DSID, .dpd = 41, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_HDMI, .dpd = 28, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_HSIC, .dpd = 19, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_HV, .dpd = 38, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_LVDS, .dpd = 57, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_MIPI_BIAS, .dpd = 3, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_NAND, .dpd = 13, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_PEX_BIAS, .dpd = 4, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_PEX_CLK1, .dpd = 5, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_PEX_CLK2, .dpd = 6, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_PEX_CNTRL, .dpd = 32, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_SDMMC1, .dpd = 33, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_SDMMC3, .dpd = 34, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_SDMMC4, .dpd = 35, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_SYS_DDC, .dpd = 58, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_UART, .dpd = 14, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_USB0, .dpd = 9, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_USB1, .dpd = 10, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_USB2, .dpd = 11, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_USB_BIAS, .dpd = 12, .voltage = UINT_MAX },
> > + TEGRA124_IO_PAD_TABLE(TEGRA_IO_PAD)
> > };
> >
> > static const struct tegra_pmc_soc tegra124_pmc_soc = {
> > @@ -1779,45 +1792,49 @@ static const u8 tegra210_cpu_powergates[] = {
> > TEGRA_POWERGATE_CPU3,
> > };
> >
> > +#define TEGRA210_IO_PAD_TABLE(_pad)
> > \
> > + /* .id .dpd .voltage .name
> > */ \
> > + _pad(TEGRA_IO_PAD_AUDIO, 17, 5,
> > "audio") \
> > + _pad(TEGRA_IO_PAD_AUDIO_HV, 61, 18,
> > "audio-hv") \
> > + _pad(TEGRA_IO_PAD_CAM, 36, 10, "cam")
> > \
> > + _pad(TEGRA_IO_PAD_CSIA, 0, UINT_MAX, "csia")
> > \
> > + _pad(TEGRA_IO_PAD_CSIB, 1, UINT_MAX, "csib")
> > \
> > + _pad(TEGRA_IO_PAD_CSIC, 42, UINT_MAX, "csic")
> > \
> > + _pad(TEGRA_IO_PAD_CSID, 43, UINT_MAX, "csid")
> > \
> > + _pad(TEGRA_IO_PAD_CSIE, 44, UINT_MAX, "csie")
> > \
> > + _pad(TEGRA_IO_PAD_CSIF, 45, UINT_MAX, "csif")
> > \
> > + _pad(TEGRA_IO_PAD_DBG, 25, 19, "dbg")
> > \
> > + _pad(TEGRA_IO_PAD_DEBUG_NONAO, 26, UINT_MAX,
> > "debug-nonao") \
> > + _pad(TEGRA_IO_PAD_DMIC, 50, 20, "dmic")
> > \
> > + _pad(TEGRA_IO_PAD_DP, 51, UINT_MAX, "dp")
> > \
> > + _pad(TEGRA_IO_PAD_DSI, 2, UINT_MAX, "dsi")
> > \
> > + _pad(TEGRA_IO_PAD_DSIB, 39, UINT_MAX, "dsib")
> > \
> > + _pad(TEGRA_IO_PAD_DSIC, 40, UINT_MAX, "dsic")
> > \
> > + _pad(TEGRA_IO_PAD_DSID, 41, UINT_MAX, "dsid")
> > \
> > + _pad(TEGRA_IO_PAD_EMMC, 35, UINT_MAX, "emmc")
> > \
> > + _pad(TEGRA_IO_PAD_EMMC2, 37, UINT_MAX,
> > "emmc2") \
> > + _pad(TEGRA_IO_PAD_GPIO, 27, 21, "gpio")
> > \
> > + _pad(TEGRA_IO_PAD_HDMI, 28, UINT_MAX, "hdmi")
> > \
> > + _pad(TEGRA_IO_PAD_HSIC, 19, UINT_MAX, "hsic")
> > \
> > + _pad(TEGRA_IO_PAD_LVDS, 57, UINT_MAX, "lvds")
> > \
> > + _pad(TEGRA_IO_PAD_MIPI_BIAS, 3, UINT_MAX,
> > "mipi-bias") \
> > + _pad(TEGRA_IO_PAD_PEX_BIAS, 4, UINT_MAX,
> > "pex-bias") \
> > + _pad(TEGRA_IO_PAD_PEX_CLK1, 5, UINT_MAX,
> > "pex-clk1") \
> > + _pad(TEGRA_IO_PAD_PEX_CLK2, 6, UINT_MAX,
> > "pex-clk2") \
> > + _pad(TEGRA_IO_PAD_PEX_CNTRL, UINT_MAX, 11,
> > "pex-cntrl") \
> > + _pad(TEGRA_IO_PAD_SDMMC1, 33, 12,
> > "sdmmc1") \
> > + _pad(TEGRA_IO_PAD_SDMMC3, 34, 13,
> > "sdmmc3") \
> > + _pad(TEGRA_IO_PAD_SPI, 46, 22, "spi")
> > \
> > + _pad(TEGRA_IO_PAD_SPI_HV, 47, 23,
> > "spi-hv") \
> > + _pad(TEGRA_IO_PAD_UART, 14, 2, "uart")
> > \
> > + _pad(TEGRA_IO_PAD_USB0, 9, UINT_MAX, "usb0")
> > \
> > + _pad(TEGRA_IO_PAD_USB1, 10, UINT_MAX, "usb1")
> > \
> > + _pad(TEGRA_IO_PAD_USB2, 11, UINT_MAX, "usb2")
> > \
> > + _pad(TEGRA_IO_PAD_USB3, 18, UINT_MAX, "usb3")
> > \
> > + _pad(TEGRA_IO_PAD_USB_BIAS, 12, UINT_MAX,
> > "usb-bias")
> > +
> > static const struct tegra_io_pad_soc tegra210_io_pads[] = {
> > - { .id = TEGRA_IO_PAD_AUDIO, .dpd = 17, .voltage = 5 },
> > - { .id = TEGRA_IO_PAD_AUDIO_HV, .dpd = 61, .voltage = 18 },
> > - { .id = TEGRA_IO_PAD_CAM, .dpd = 36, .voltage = 10 },
> > - { .id = TEGRA_IO_PAD_CSIA, .dpd = 0, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_CSIB, .dpd = 1, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_CSIC, .dpd = 42, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_CSID, .dpd = 43, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_CSIE, .dpd = 44, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_CSIF, .dpd = 45, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_DBG, .dpd = 25, .voltage = 19 },
> > - { .id = TEGRA_IO_PAD_DEBUG_NONAO, .dpd = 26, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_DMIC, .dpd = 50, .voltage = 20 },
> > - { .id = TEGRA_IO_PAD_DP, .dpd = 51, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_DSI, .dpd = 2, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_DSIB, .dpd = 39, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_DSIC, .dpd = 40, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_DSID, .dpd = 41, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_EMMC, .dpd = 35, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_EMMC2, .dpd = 37, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_GPIO, .dpd = 27, .voltage = 21 },
> > - { .id = TEGRA_IO_PAD_HDMI, .dpd = 28, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_HSIC, .dpd = 19, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_LVDS, .dpd = 57, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_MIPI_BIAS, .dpd = 3, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_PEX_BIAS, .dpd = 4, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_PEX_CLK1, .dpd = 5, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_PEX_CLK2, .dpd = 6, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_PEX_CNTRL, .dpd = UINT_MAX, .voltage = 11 },
> > - { .id = TEGRA_IO_PAD_SDMMC1, .dpd = 33, .voltage = 12 },
> > - { .id = TEGRA_IO_PAD_SDMMC3, .dpd = 34, .voltage = 13 },
> > - { .id = TEGRA_IO_PAD_SPI, .dpd = 46, .voltage = 22 },
> > - { .id = TEGRA_IO_PAD_SPI_HV, .dpd = 47, .voltage = 23 },
> > - { .id = TEGRA_IO_PAD_UART, .dpd = 14, .voltage = 2 },
> > - { .id = TEGRA_IO_PAD_USB0, .dpd = 9, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_USB1, .dpd = 10, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_USB2, .dpd = 11, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_USB3, .dpd = 18, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_USB_BIAS, .dpd = 12, .voltage = UINT_MAX },
> > + TEGRA210_IO_PAD_TABLE(TEGRA_IO_PAD)
> > };
> >
> > static const struct tegra_pmc_soc tegra210_pmc_soc = {
> > @@ -1836,44 +1853,48 @@ static const struct tegra_pmc_soc tegra210_pmc_soc
> > = {
> > .setup_irq_polarity = tegra20_pmc_setup_irq_polarity,
> > };
> >
> > +#define TEGRA186_IO_PAD_TABLE(_pad)
> > \
> > + /* .id .dpd .voltage .name */
> > \
> > + _pad(TEGRA_IO_PAD_CSIA, 0, UINT_MAX, "csia")
> > \
> > + _pad(TEGRA_IO_PAD_CSIB, 1, UINT_MAX, "csib")
> > \
> > + _pad(TEGRA_IO_PAD_DSI, 2, UINT_MAX, "dsi")
> > \
> > + _pad(TEGRA_IO_PAD_MIPI_BIAS, 3, UINT_MAX, "mipi-bias")
> > \
> > + _pad(TEGRA_IO_PAD_PEX_CLK_BIAS, 4, UINT_MAX, "pex-clk-bias")
> > \
> > + _pad(TEGRA_IO_PAD_PEX_CLK3, 5, UINT_MAX, "pex-clk3")
> > \
> > + _pad(TEGRA_IO_PAD_PEX_CLK2, 6, UINT_MAX, "pex-clk2")
> > \
> > + _pad(TEGRA_IO_PAD_PEX_CLK1, 7, UINT_MAX, "pex-clk1")
> > \
> > + _pad(TEGRA_IO_PAD_USB0, 9, UINT_MAX, "usb0")
> > \
> > + _pad(TEGRA_IO_PAD_USB1, 10, UINT_MAX, "usb1")
> > \
> > + _pad(TEGRA_IO_PAD_USB2, 11, UINT_MAX, "usb2")
> > \
> > + _pad(TEGRA_IO_PAD_USB_BIAS, 12, UINT_MAX, "usb-bias")
> > \
> > + _pad(TEGRA_IO_PAD_UART, 14, UINT_MAX, "uart")
> > \
> > + _pad(TEGRA_IO_PAD_AUDIO, 17, UINT_MAX, "audio")
> > \
> > + _pad(TEGRA_IO_PAD_HSIC, 19, UINT_MAX, "hsic")
> > \
> > + _pad(TEGRA_IO_PAD_DBG, 25, UINT_MAX, "dbg")
> > \
> > + _pad(TEGRA_IO_PAD_HDMI_DP0, 28, UINT_MAX, "hdmi-dp0")
> > \
> > + _pad(TEGRA_IO_PAD_HDMI_DP1, 29, UINT_MAX, "hdmi-dp1")
> > \
> > + _pad(TEGRA_IO_PAD_PEX_CNTRL, 32, UINT_MAX, "pex-cntrl")
> > \
> > + _pad(TEGRA_IO_PAD_SDMMC2_HV, 34, 5, "sdmmc2-hv")
> > \
> > + _pad(TEGRA_IO_PAD_SDMMC4, 36, UINT_MAX, "sdmmc4")
> > \
> > + _pad(TEGRA_IO_PAD_CAM, 38, UINT_MAX, "cam")
> > \
> > + _pad(TEGRA_IO_PAD_DSIB, 40, UINT_MAX, "dsib")
> > \
> > + _pad(TEGRA_IO_PAD_DSIC, 41, UINT_MAX, "dsic")
> > \
> > + _pad(TEGRA_IO_PAD_DSID, 42, UINT_MAX, "dsid")
> > \
> > + _pad(TEGRA_IO_PAD_CSIC, 43, UINT_MAX, "csic")
> > \
> > + _pad(TEGRA_IO_PAD_CSID, 44, UINT_MAX, "csid")
> > \
> > + _pad(TEGRA_IO_PAD_CSIE, 45, UINT_MAX, "csie")
> > \
> > + _pad(TEGRA_IO_PAD_CSIF, 46, UINT_MAX, "csif")
> > \
> > + _pad(TEGRA_IO_PAD_SPI, 47, UINT_MAX, "spi")
> > \
> > + _pad(TEGRA_IO_PAD_UFS, 49, UINT_MAX, "ufs")
> > \
> > + _pad(TEGRA_IO_PAD_DMIC_HV, 52, 2, "dmic-hv")
> > \
> > + _pad(TEGRA_IO_PAD_EDP, 53, UINT_MAX, "edp")
> > \
> > + _pad(TEGRA_IO_PAD_SDMMC1_HV, 55, 4, "sdmmc1-hv")
> > \
> > + _pad(TEGRA_IO_PAD_SDMMC3_HV, 56, 6, "sdmmc3-hv")
> > \
> > + _pad(TEGRA_IO_PAD_CONN, 60, UINT_MAX, "conn")
> > \
> > + _pad(TEGRA_IO_PAD_AUDIO_HV, 61, 1, "audio-hv")
> > +
> > static const struct tegra_io_pad_soc tegra186_io_pads[] = {
> > - { .id = TEGRA_IO_PAD_CSIA, .dpd = 0, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_CSIB, .dpd = 1, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_DSI, .dpd = 2, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_MIPI_BIAS, .dpd = 3, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_PEX_CLK_BIAS, .dpd = 4, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_PEX_CLK3, .dpd = 5, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_PEX_CLK2, .dpd = 6, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_PEX_CLK1, .dpd = 7, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_USB0, .dpd = 9, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_USB1, .dpd = 10, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_USB2, .dpd = 11, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_USB_BIAS, .dpd = 12, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_UART, .dpd = 14, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_AUDIO, .dpd = 17, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_HSIC, .dpd = 19, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_DBG, .dpd = 25, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_HDMI_DP0, .dpd = 28, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_HDMI_DP1, .dpd = 29, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_PEX_CNTRL, .dpd = 32, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_SDMMC2_HV, .dpd = 34, .voltage = 5 },
> > - { .id = TEGRA_IO_PAD_SDMMC4, .dpd = 36, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_CAM, .dpd = 38, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_DSIB, .dpd = 40, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_DSIC, .dpd = 41, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_DSID, .dpd = 42, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_CSIC, .dpd = 43, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_CSID, .dpd = 44, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_CSIE, .dpd = 45, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_CSIF, .dpd = 46, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_SPI, .dpd = 47, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_UFS, .dpd = 49, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_DMIC_HV, .dpd = 52, .voltage = 2 },
> > - { .id = TEGRA_IO_PAD_EDP, .dpd = 53, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_SDMMC1_HV, .dpd = 55, .voltage = 4 },
> > - { .id = TEGRA_IO_PAD_SDMMC3_HV, .dpd = 56, .voltage = 6 },
> > - { .id = TEGRA_IO_PAD_CONN, .dpd = 60, .voltage = UINT_MAX },
> > - { .id = TEGRA_IO_PAD_AUDIO_HV, .dpd = 61, .voltage = 1 },
> > + TEGRA186_IO_PAD_TABLE(TEGRA_IO_PAD)
> > };
> >
> > static const struct tegra_pmc_regs tegra186_pmc_regs = {
>
> Can you run checkpatch on this and fix up the warnings?
>
> Cheers
> Jon
>
I did run checkpatch on it. The remaining errors and warnings are either
about the table rows being too long or about the macro values used to
generate them not being enclosed in parentheses. I felt that fixing them
could be somewhat detrimental to the overall readability. I probably
could fix the errors about macro parens, but I'd rather not give up the
aligned columns of the tables. I guess I can try to get them squeezed a
bit tighter together by not aligning to tab stops.
-Aapo
