From: Aapo Vienamo <avienamo@nvidia.com>
To: Jon Hunter <jonathanh@nvidia.com>
Cc: Rob Herring <robh+dt@kernel.org>,
Mark Rutland <mark.rutland@arm.com>,
Thierry Reding <thierry.reding@gmail.com>,
Mikko Perttunen <mperttunen@nvidia.com>,
<devicetree@vger.kernel.org>, <linux-tegra@vger.kernel.org>,
<linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 4/6] soc/tegra: pmc: Use X macro to generate IO pad tables
Date: Wed, 11 Jul 2018 12:23:03 +0300 [thread overview]
Message-ID: <20180711122303.45512721@dhcp-10-21-25-168> (raw)
In-Reply-To: <dc369c97-6e23-02d7-8e3f-0f94ae4bf585@nvidia.com>
On Wed, 11 Jul 2018 09:30:57 +0100
Jon Hunter <jonathanh@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 <avienamo@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
next prev parent reply other threads:[~2018-07-11 9:23 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-07-10 12:47 [PATCH 0/6] Tegra PMC pinctrl pad configuration Aapo Vienamo
2018-07-10 12:47 ` [PATCH 1/6] soc/tegra: pmc: Fix pad voltage configuration for Tegra186 Aapo Vienamo
2018-07-11 8:08 ` Jon Hunter
2018-07-10 12:47 ` [PATCH 2/6] soc/tegra: pmc: Factor out DPD register bit calculation Aapo Vienamo
2018-07-11 8:24 ` Jon Hunter
2018-07-10 12:47 ` [PATCH 3/6] soc/tegra: pmc: Implement tegra_io_pad_is_powered() Aapo Vienamo
2018-07-11 8:27 ` Jon Hunter
2018-07-10 12:47 ` [PATCH 4/6] soc/tegra: pmc: Use X macro to generate IO pad tables Aapo Vienamo
2018-07-11 8:30 ` Jon Hunter
2018-07-11 9:23 ` Aapo Vienamo [this message]
2018-07-11 9:43 ` Jon Hunter
2018-07-10 12:54 ` [PATCH 5/6] dt-bindings: Add Tegra PMC pad configuration bindings Aapo Vienamo
2018-07-11 8:32 ` Jon Hunter
2018-07-10 12:54 ` [PATCH 6/6] soc/tegra: pmc: Implement pad configuration via pinctrl Aapo Vienamo
2018-07-11 8:40 ` Jon Hunter
2018-07-11 9:38 ` Aapo Vienamo
2018-07-11 9:46 ` Jon Hunter
2018-07-11 10:12 ` Aapo Vienamo
2018-07-11 11:42 ` Jon Hunter
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180711122303.45512721@dhcp-10-21-25-168 \
--to=avienamo@nvidia.com \
--cc=devicetree@vger.kernel.org \
--cc=jonathanh@nvidia.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=mperttunen@nvidia.com \
--cc=robh+dt@kernel.org \
--cc=thierry.reding@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).