* [PATCH 00/35 linux-next] constify of_device_id array @ 2015-03-16 19:17 Fabian Frederick 2015-03-16 19:17 ` [PATCH 05/35 linux-next] tty: " Fabian Frederick ` (5 more replies) 0 siblings, 6 replies; 49+ messages in thread From: Fabian Frederick @ 2015-03-16 19:17 UTC (permalink / raw) To: linux-arm-kernel This small patchset adds const to of_device_id arrays in drivers branch. Fabian Frederick (35): ata: constify of_device_id array regulator: constify of_device_id array thermal: constify of_device_id array tty/hvc_opal: constify of_device_id array tty: constify of_device_id array power: constify of_device_id array char: constify of_device_id array dma: constify of_device_id array iio: constify of_device_id array misc: constify of_device_id array usb: gadget: constify of_device_id array mtd: constify of_device_id array w1: constify of_device_id array ide: pmac: constify of_device_id array spi: constify of_device_id array video: constify of_device_id array coresight-replicator: constify of_device_id array macintosh: constify of_device_id array virtio_mmio: constify of_device_id array swim3: constify of_device_id array mfd: constify of_device_id array soc: ti: constify of_device_id array [media]: constify of_device_id array Input: constify of_device_id array PCI: constify of_device_id array hwmon: constify of_device_id array reset: sti: constify of_device_id array uio: constify of_device_id array gpu: constify of_device_id array devfreq: constify of_device_id array EDAC: constify of_device_id array clk: constify of_device_id array mmc: constify of_device_id array Staging: octeon: constify of_device_id array pinctrl: constify of_device_id array drivers/ata/pata_macio.c | 2 +- drivers/ata/pata_mpc52xx.c | 2 +- drivers/ata/pata_octeon_cf.c | 2 +- drivers/ata/pata_of_platform.c | 2 +- drivers/ata/sata_fsl.c | 2 +- drivers/ata/sata_mv.c | 2 +- drivers/ata/sata_rcar.c | 2 +- drivers/block/swim3.c | 2 +- drivers/char/hw_random/pasemi-rng.c | 2 +- drivers/char/hw_random/powernv-rng.c | 2 +- drivers/char/hw_random/ppc4xx-rng.c | 2 +- drivers/char/ipmi/ipmi_si_intf.c | 4 ++-- drivers/char/xillybus/xillybus_of.c | 2 +- drivers/clk/clk-palmas.c | 2 +- drivers/clk/st/clkgen-fsyn.c | 2 +- drivers/clk/st/clkgen-mux.c | 8 ++++---- drivers/clk/st/clkgen-pll.c | 4 ++-- drivers/clk/ti/clk-dra7-atl.c | 2 +- drivers/clk/ti/clockdomain.c | 2 +- drivers/clk/versatile/clk-vexpress-osc.c | 2 +- drivers/coresight/coresight-replicator.c | 2 +- drivers/devfreq/event/exynos-ppmu.c | 2 +- drivers/devfreq/tegra-devfreq.c | 2 +- drivers/dma/bestcomm/bestcomm.c | 4 ++-- drivers/dma/k3dma.c | 2 +- drivers/dma/mmp_pdma.c | 2 +- drivers/dma/mmp_tdma.c | 2 +- drivers/dma/mpc512x_dma.c | 2 +- drivers/dma/mv_xor.c | 2 +- drivers/dma/sirf-dma.c | 2 +- drivers/dma/sun6i-dma.c | 2 +- drivers/edac/highbank_mc_edac.c | 2 +- drivers/edac/mpc85xx_edac.c | 4 ++-- drivers/edac/ppc4xx_edac.c | 2 +- drivers/edac/synopsys_edac.c | 2 +- drivers/gpio/gpio-mpc8xxx.c | 2 +- drivers/gpio/gpio-octeon.c | 2 +- drivers/gpio/gpio-tz1090-pdc.c | 2 +- drivers/gpio/gpio-tz1090.c | 2 +- drivers/gpio/gpio-zynq.c | 2 +- drivers/gpu/drm/armada/armada_crtc.c | 2 +- drivers/gpu/drm/exynos/exynos_drm_dsi.c | 2 +- drivers/gpu/drm/exynos/exynos_hdmi.c | 2 +- drivers/gpu/drm/exynos/exynos_mixer.c | 2 +- drivers/gpu/drm/panel/panel-ld9040.c | 2 +- drivers/gpu/drm/panel/panel-s6e8aa0.c | 2 +- drivers/gpu/drm/sti/sti_dvo.c | 2 +- drivers/gpu/drm/sti/sti_hqvdp.c | 2 +- drivers/gpu/drm/tilcdc/tilcdc_drv.c | 4 ++-- drivers/gpu/drm/tilcdc/tilcdc_panel.c | 2 +- drivers/gpu/drm/tilcdc/tilcdc_slave.c | 4 ++-- drivers/gpu/drm/tilcdc/tilcdc_tfp410.c | 4 ++-- drivers/gpu/host1x/dev.c | 2 +- drivers/gpu/host1x/mipi.c | 2 +- drivers/hwmon/pwm-fan.c | 2 +- drivers/hwmon/vexpress.c | 2 +- drivers/ide/pmac.c | 2 +- drivers/iio/common/ssp_sensors/ssp_dev.c | 2 +- drivers/input/misc/palmas-pwrbutton.c | 2 +- drivers/input/misc/regulator-haptic.c | 2 +- drivers/input/misc/tps65218-pwrbutton.c | 2 +- drivers/input/touchscreen/ar1021_i2c.c | 2 +- drivers/macintosh/mediabay.c | 2 +- drivers/macintosh/rack-meter.c | 2 +- drivers/media/i2c/adv7604.c | 2 +- drivers/media/platform/fsl-viu.c | 2 +- drivers/media/platform/soc_camera/rcar_vin.c | 2 +- drivers/media/rc/gpio-ir-recv.c | 2 +- drivers/media/rc/ir-hix5hd2.c | 2 +- drivers/media/rc/st_rc.c | 2 +- drivers/mfd/hi6421-pmic-core.c | 2 +- drivers/mfd/rk808.c | 2 +- drivers/mfd/twl4030-power.c | 2 +- drivers/misc/carma/carma-fpga-program.c | 2 +- drivers/misc/carma/carma-fpga.c | 2 +- drivers/misc/lis3lv02d/lis3lv02d_i2c.c | 2 +- drivers/misc/lis3lv02d/lis3lv02d_spi.c | 2 +- drivers/misc/sram.c | 2 +- drivers/mmc/host/mmc_spi.c | 2 +- drivers/mmc/host/wmt-sdmmc.c | 2 +- drivers/mtd/devices/docg3.c | 2 +- drivers/mtd/maps/physmap_of.c | 4 ++-- drivers/mtd/nand/mpc5121_nfc.c | 2 +- drivers/mtd/spi-nor/fsl-quadspi.c | 2 +- drivers/pci/host/pci-rcar-gen2.c | 2 +- drivers/pci/host/pcie-xilinx.c | 2 +- drivers/pinctrl/bcm/pinctrl-bcm2835.c | 2 +- drivers/pinctrl/mediatek/pinctrl-mt8135.c | 2 +- drivers/pinctrl/mediatek/pinctrl-mt8173.c | 2 +- drivers/pinctrl/mvebu/pinctrl-armada-370.c | 2 +- drivers/pinctrl/mvebu/pinctrl-armada-375.c | 2 +- drivers/pinctrl/mvebu/pinctrl-armada-38x.c | 2 +- drivers/pinctrl/mvebu/pinctrl-armada-39x.c | 2 +- drivers/pinctrl/mvebu/pinctrl-armada-xp.c | 2 +- drivers/pinctrl/mvebu/pinctrl-kirkwood.c | 2 +- drivers/pinctrl/mvebu/pinctrl-orion.c | 2 +- drivers/pinctrl/pinctrl-as3722.c | 2 +- drivers/pinctrl/pinctrl-at91.c | 4 ++-- drivers/pinctrl/pinctrl-palmas.c | 2 +- drivers/pinctrl/pinctrl-single.c | 4 ++-- drivers/pinctrl/pinctrl-st.c | 2 +- drivers/pinctrl/pinctrl-tz1090-pdc.c | 2 +- drivers/pinctrl/pinctrl-tz1090.c | 2 +- drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c | 2 +- drivers/pinctrl/sunxi/pinctrl-sun5i-a10s.c | 2 +- drivers/pinctrl/sunxi/pinctrl-sun5i-a13.c | 2 +- drivers/pinctrl/sunxi/pinctrl-sun6i-a31-r.c | 2 +- drivers/pinctrl/sunxi/pinctrl-sun6i-a31.c | 2 +- drivers/pinctrl/sunxi/pinctrl-sun6i-a31s.c | 2 +- drivers/pinctrl/sunxi/pinctrl-sun7i-a20.c | 2 +- drivers/pinctrl/sunxi/pinctrl-sun8i-a23-r.c | 2 +- drivers/pinctrl/sunxi/pinctrl-sun8i-a23.c | 2 +- drivers/pinctrl/sunxi/pinctrl-sun9i-a80.c | 2 +- drivers/pinctrl/vt8500/pinctrl-vt8500.c | 2 +- drivers/pinctrl/vt8500/pinctrl-wm8505.c | 2 +- drivers/pinctrl/vt8500/pinctrl-wm8650.c | 2 +- drivers/pinctrl/vt8500/pinctrl-wm8750.c | 2 +- drivers/pinctrl/vt8500/pinctrl-wm8850.c | 2 +- drivers/power/charger-manager.c | 2 +- drivers/power/reset/at91-poweroff.c | 2 +- drivers/power/reset/at91-reset.c | 4 ++-- drivers/power/reset/hisi-reboot.c | 2 +- drivers/power/reset/keystone-reset.c | 2 +- drivers/power/reset/st-poweroff.c | 2 +- drivers/power/reset/syscon-reboot.c | 2 +- drivers/power/reset/vexpress-poweroff.c | 2 +- drivers/power/reset/xgene-reboot.c | 2 +- drivers/power/tps65090-charger.c | 2 +- drivers/regulator/palmas-regulator.c | 2 +- drivers/reset/sti/reset-stih407.c | 2 +- drivers/reset/sti/reset-stih415.c | 2 +- drivers/reset/sti/reset-stih416.c | 2 +- drivers/soc/ti/knav_dma.c | 2 +- drivers/soc/ti/knav_qmss_queue.c | 2 +- drivers/spi/spi-mpc512x-psc.c | 2 +- drivers/spi/spi-octeon.c | 2 +- drivers/spi/spi-st-ssc4.c | 2 +- drivers/staging/octeon-usb/octeon-hcd.c | 2 +- drivers/staging/octeon/ethernet.c | 2 +- drivers/thermal/st/st_thermal_memmap.c | 2 +- drivers/thermal/st/st_thermal_syscfg.c | 2 +- drivers/tty/hvc/hvc_opal.c | 2 +- drivers/tty/serial/apbuart.c | 2 +- drivers/tty/serial/cpm_uart/cpm_uart_core.c | 2 +- drivers/tty/serial/fsl_lpuart.c | 2 +- drivers/tty/serial/mpc52xx_uart.c | 2 +- drivers/tty/serial/mxs-auart.c | 2 +- drivers/tty/serial/of_serial.c | 4 ++-- drivers/tty/serial/pmac_zilog.c | 2 +- drivers/tty/serial/pxa.c | 2 +- drivers/tty/serial/serial-tegra.c | 2 +- drivers/tty/serial/sirfsoc_uart.c | 2 +- drivers/tty/serial/st-asc.c | 2 +- drivers/tty/serial/uartlite.c | 2 +- drivers/tty/serial/ucc_uart.c | 2 +- drivers/tty/serial/xilinx_uartps.c | 2 +- drivers/uio/uio_pdrv_genirq.c | 2 +- drivers/usb/gadget/udc/pxa27x_udc.c | 2 +- drivers/video/backlight/gpio_backlight.c | 2 +- drivers/video/backlight/pwm_bl.c | 2 +- drivers/video/fbdev/fsl-diu-fb.c | 2 +- drivers/video/fbdev/grvga.c | 2 +- drivers/video/fbdev/mb862xx/mb862xxfbdrv.c | 2 +- drivers/video/fbdev/ocfb.c | 2 +- drivers/video/fbdev/platinumfb.c | 2 +- drivers/video/fbdev/xilinxfb.c | 2 +- drivers/virtio/virtio_mmio.c | 2 +- drivers/w1/masters/mxc_w1.c | 2 +- drivers/w1/masters/omap_hdq.c | 2 +- drivers/w1/masters/w1-gpio.c | 2 +- 170 files changed, 185 insertions(+), 185 deletions(-) -- 2.1.0 ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 05/35 linux-next] tty: constify of_device_id array 2015-03-16 19:17 [PATCH 00/35 linux-next] constify of_device_id array Fabian Frederick @ 2015-03-16 19:17 ` Fabian Frederick 2015-03-16 19:20 ` Timur Tabi ` (2 more replies) 2015-03-16 19:17 ` [PATCH 06/35 linux-next] power: " Fabian Frederick ` (4 subsequent siblings) 5 siblings, 3 replies; 49+ messages in thread From: Fabian Frederick @ 2015-03-16 19:17 UTC (permalink / raw) To: linux-arm-kernel of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick <fabf@skynet.be> --- drivers/tty/serial/apbuart.c | 2 +- drivers/tty/serial/cpm_uart/cpm_uart_core.c | 2 +- drivers/tty/serial/fsl_lpuart.c | 2 +- drivers/tty/serial/mpc52xx_uart.c | 2 +- drivers/tty/serial/mxs-auart.c | 2 +- drivers/tty/serial/of_serial.c | 4 ++-- drivers/tty/serial/pmac_zilog.c | 2 +- drivers/tty/serial/pxa.c | 2 +- drivers/tty/serial/serial-tegra.c | 2 +- drivers/tty/serial/sirfsoc_uart.c | 2 +- drivers/tty/serial/st-asc.c | 2 +- drivers/tty/serial/uartlite.c | 2 +- drivers/tty/serial/ucc_uart.c | 2 +- drivers/tty/serial/xilinx_uartps.c | 2 +- 14 files changed, 15 insertions(+), 15 deletions(-) diff --git a/drivers/tty/serial/apbuart.c b/drivers/tty/serial/apbuart.c index 4f0f95e..f3af317 100644 --- a/drivers/tty/serial/apbuart.c +++ b/drivers/tty/serial/apbuart.c @@ -572,7 +572,7 @@ static int apbuart_probe(struct platform_device *op) return 0; } -static struct of_device_id apbuart_match[] = { +static const struct of_device_id apbuart_match[] = { { .name = "GAISLER_APBUART", }, diff --git a/drivers/tty/serial/cpm_uart/cpm_uart_core.c b/drivers/tty/serial/cpm_uart/cpm_uart_core.c index fddb1fd..08431ad 100644 --- a/drivers/tty/serial/cpm_uart/cpm_uart_core.c +++ b/drivers/tty/serial/cpm_uart/cpm_uart_core.c @@ -1435,7 +1435,7 @@ static int cpm_uart_remove(struct platform_device *ofdev) return uart_remove_one_port(&cpm_reg, &pinfo->port); } -static struct of_device_id cpm_uart_match[] = { +static const struct of_device_id cpm_uart_match[] = { { .compatible = "fsl,cpm1-smc-uart", }, diff --git a/drivers/tty/serial/fsl_lpuart.c b/drivers/tty/serial/fsl_lpuart.c index b1893f3..946273a 100644 --- a/drivers/tty/serial/fsl_lpuart.c +++ b/drivers/tty/serial/fsl_lpuart.c @@ -257,7 +257,7 @@ struct lpuart_port { struct timer_list lpuart_timer; }; -static struct of_device_id lpuart_dt_ids[] = { +static const struct of_device_id lpuart_dt_ids[] = { { .compatible = "fsl,vf610-lpuart", }, diff --git a/drivers/tty/serial/mpc52xx_uart.c b/drivers/tty/serial/mpc52xx_uart.c index 3308ef2..1589f17 100644 --- a/drivers/tty/serial/mpc52xx_uart.c +++ b/drivers/tty/serial/mpc52xx_uart.c @@ -1717,7 +1717,7 @@ static struct uart_driver mpc52xx_uart_driver = { /* OF Platform Driver */ /* ======================================================================== */ -static struct of_device_id mpc52xx_uart_of_match[] = { +static const struct of_device_id mpc52xx_uart_of_match[] = { #ifdef CONFIG_PPC_MPC52xx { .compatible = "fsl,mpc5200b-psc-uart", .data = &mpc5200b_psc_ops, }, { .compatible = "fsl,mpc5200-psc-uart", .data = &mpc52xx_psc_ops, }, diff --git a/drivers/tty/serial/mxs-auart.c b/drivers/tty/serial/mxs-auart.c index 20a863b..f7e5825 100644 --- a/drivers/tty/serial/mxs-auart.c +++ b/drivers/tty/serial/mxs-auart.c @@ -176,7 +176,7 @@ static struct platform_device_id mxs_auart_devtype[] = { }; MODULE_DEVICE_TABLE(platform, mxs_auart_devtype); -static struct of_device_id mxs_auart_dt_ids[] = { +static const struct of_device_id mxs_auart_dt_ids[] = { { .compatible = "fsl,imx28-auart", .data = &mxs_auart_devtype[IMX28_AUART] diff --git a/drivers/tty/serial/of_serial.c b/drivers/tty/serial/of_serial.c index 33fb94f..d8c4eea 100644 --- a/drivers/tty/serial/of_serial.c +++ b/drivers/tty/serial/of_serial.c @@ -155,7 +155,7 @@ out: /* * Try to register a serial port */ -static struct of_device_id of_platform_serial_table[]; +static const struct of_device_id of_platform_serial_table[]; static int of_platform_serial_probe(struct platform_device *ofdev) { const struct of_device_id *match; @@ -320,7 +320,7 @@ static SIMPLE_DEV_PM_OPS(of_serial_pm_ops, of_serial_suspend, of_serial_resume); /* * A few common types, add more as needed. */ -static struct of_device_id of_platform_serial_table[] = { +static const struct of_device_id of_platform_serial_table[] = { { .compatible = "ns8250", .data = (void *)PORT_8250, }, { .compatible = "ns16450", .data = (void *)PORT_16450, }, { .compatible = "ns16550a", .data = (void *)PORT_16550A, }, diff --git a/drivers/tty/serial/pmac_zilog.c b/drivers/tty/serial/pmac_zilog.c index 8f51579..e156e39 100644 --- a/drivers/tty/serial/pmac_zilog.c +++ b/drivers/tty/serial/pmac_zilog.c @@ -1846,7 +1846,7 @@ static int __init pmz_register(void) #ifdef CONFIG_PPC_PMAC -static struct of_device_id pmz_match[] = +static const struct of_device_id pmz_match[] = { { .name = "ch-a", diff --git a/drivers/tty/serial/pxa.c b/drivers/tty/serial/pxa.c index d5d0626..9becba6 100644 --- a/drivers/tty/serial/pxa.c +++ b/drivers/tty/serial/pxa.c @@ -824,7 +824,7 @@ static const struct dev_pm_ops serial_pxa_pm_ops = { }; #endif -static struct of_device_id serial_pxa_dt_ids[] = { +static const struct of_device_id serial_pxa_dt_ids[] = { { .compatible = "mrvl,pxa-uart", }, { .compatible = "mrvl,mmp-uart", }, {} diff --git a/drivers/tty/serial/serial-tegra.c b/drivers/tty/serial/serial-tegra.c index 48e6e41..1d5ea39 100644 --- a/drivers/tty/serial/serial-tegra.c +++ b/drivers/tty/serial/serial-tegra.c @@ -1251,7 +1251,7 @@ static struct tegra_uart_chip_data tegra30_uart_chip_data = { .support_clk_src_div = true, }; -static struct of_device_id tegra_uart_of_match[] = { +static const struct of_device_id tegra_uart_of_match[] = { { .compatible = "nvidia,tegra30-hsuart", .data = &tegra30_uart_chip_data, diff --git a/drivers/tty/serial/sirfsoc_uart.c b/drivers/tty/serial/sirfsoc_uart.c index 27ed0e9..9de3eab 100644 --- a/drivers/tty/serial/sirfsoc_uart.c +++ b/drivers/tty/serial/sirfsoc_uart.c @@ -1269,7 +1269,7 @@ static struct uart_driver sirfsoc_uart_drv = { #endif }; -static struct of_device_id sirfsoc_uart_ids[] = { +static const struct of_device_id sirfsoc_uart_ids[] = { { .compatible = "sirf,prima2-uart", .data = &sirfsoc_uart,}, { .compatible = "sirf,atlas7-uart", .data = &sirfsoc_uart}, { .compatible = "sirf,prima2-usp-uart", .data = &sirfsoc_usp}, diff --git a/drivers/tty/serial/st-asc.c b/drivers/tty/serial/st-asc.c index 712b03a..d625664 100644 --- a/drivers/tty/serial/st-asc.c +++ b/drivers/tty/serial/st-asc.c @@ -720,7 +720,7 @@ static struct asc_port *asc_of_get_asc_port(struct platform_device *pdev) } #ifdef CONFIG_OF -static struct of_device_id asc_match[] = { +static const struct of_device_id asc_match[] = { { .compatible = "st,asc", }, {}, }; diff --git a/drivers/tty/serial/uartlite.c b/drivers/tty/serial/uartlite.c index 189f52e..708eead 100644 --- a/drivers/tty/serial/uartlite.c +++ b/drivers/tty/serial/uartlite.c @@ -622,7 +622,7 @@ static int ulite_release(struct device *dev) #if defined(CONFIG_OF) /* Match table for of_platform binding */ -static struct of_device_id ulite_of_match[] = { +static const struct of_device_id ulite_of_match[] = { { .compatible = "xlnx,opb-uartlite-1.00.b", }, { .compatible = "xlnx,xps-uartlite-1.00.a", }, {} diff --git a/drivers/tty/serial/ucc_uart.c b/drivers/tty/serial/ucc_uart.c index 14d10fc..7d2532b 100644 --- a/drivers/tty/serial/ucc_uart.c +++ b/drivers/tty/serial/ucc_uart.c @@ -1473,7 +1473,7 @@ static int ucc_uart_remove(struct platform_device *ofdev) return 0; } -static struct of_device_id ucc_uart_match[] = { +static const struct of_device_id ucc_uart_match[] = { { .type = "serial", .compatible = "ucc_uart", diff --git a/drivers/tty/serial/xilinx_uartps.c b/drivers/tty/serial/xilinx_uartps.c index 8f9c543..09258de 100644 --- a/drivers/tty/serial/xilinx_uartps.c +++ b/drivers/tty/serial/xilinx_uartps.c @@ -1459,7 +1459,7 @@ static int cdns_uart_remove(struct platform_device *pdev) } /* Match table for of_platform binding */ -static struct of_device_id cdns_uart_of_match[] = { +static const struct of_device_id cdns_uart_of_match[] = { { .compatible = "xlnx,xuartps", }, { .compatible = "cdns,uart-r1p8", }, {} -- 2.1.0 ^ permalink raw reply related [flat|nested] 49+ messages in thread
* [PATCH 05/35 linux-next] tty: constify of_device_id array 2015-03-16 19:17 ` [PATCH 05/35 linux-next] tty: " Fabian Frederick @ 2015-03-16 19:20 ` Timur Tabi 2015-03-16 19:51 ` Peter Korsgaard 2015-03-17 8:10 ` Patrice Chotard 2 siblings, 0 replies; 49+ messages in thread From: Timur Tabi @ 2015-03-16 19:20 UTC (permalink / raw) To: linux-arm-kernel On 03/16/2015 02:17 PM, Fabian Frederick wrote: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > > Signed-off-by: Fabian Frederick<fabf@skynet.be> ... > drivers/tty/serial/ucc_uart.c | 2 +- For this driver: Acked-by: Timur Tabi <timur@tabi.org> ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 05/35 linux-next] tty: constify of_device_id array 2015-03-16 19:17 ` [PATCH 05/35 linux-next] tty: " Fabian Frederick 2015-03-16 19:20 ` Timur Tabi @ 2015-03-16 19:51 ` Peter Korsgaard 2015-03-17 8:10 ` Patrice Chotard 2 siblings, 0 replies; 49+ messages in thread From: Peter Korsgaard @ 2015-03-16 19:51 UTC (permalink / raw) To: linux-arm-kernel >>>>> "Fabian" == Fabian Frederick <fabf@skynet.be> writes: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > Signed-off-by: Fabian Frederick <fabf@skynet.be> > --- For this: > drivers/tty/serial/uartlite.c | 2 +- Acked-by: Peter Korsgaard <peter@korsgaard.com> -- Bye, Peter Korsgaard ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 05/35 linux-next] tty: constify of_device_id array 2015-03-16 19:17 ` [PATCH 05/35 linux-next] tty: " Fabian Frederick 2015-03-16 19:20 ` Timur Tabi 2015-03-16 19:51 ` Peter Korsgaard @ 2015-03-17 8:10 ` Patrice Chotard 2 siblings, 0 replies; 49+ messages in thread From: Patrice Chotard @ 2015-03-17 8:10 UTC (permalink / raw) To: linux-arm-kernel Hi Fabian On 03/16/2015 08:17 PM, Fabian Frederick wrote: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > > Signed-off-by: Fabian Frederick <fabf@skynet.be> > --- .... > drivers/tty/serial/st-asc.c | 2 +- > For this driver Acked-by: Patrice Chotard <patrice.chotard@st.com> Thanks ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 06/35 linux-next] power: constify of_device_id array 2015-03-16 19:17 [PATCH 00/35 linux-next] constify of_device_id array Fabian Frederick 2015-03-16 19:17 ` [PATCH 05/35 linux-next] tty: " Fabian Frederick @ 2015-03-16 19:17 ` Fabian Frederick 2015-03-17 11:38 ` Sudeep Holla 2015-03-20 12:32 ` Sebastian Reichel 2015-03-16 19:17 ` [PATCH 08/35 linux-next] dma: " Fabian Frederick ` (3 subsequent siblings) 5 siblings, 2 replies; 49+ messages in thread From: Fabian Frederick @ 2015-03-16 19:17 UTC (permalink / raw) To: linux-arm-kernel of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick <fabf@skynet.be> --- drivers/power/charger-manager.c | 2 +- drivers/power/reset/at91-poweroff.c | 2 +- drivers/power/reset/at91-reset.c | 4 ++-- drivers/power/reset/hisi-reboot.c | 2 +- drivers/power/reset/keystone-reset.c | 2 +- drivers/power/reset/st-poweroff.c | 2 +- drivers/power/reset/syscon-reboot.c | 2 +- drivers/power/reset/vexpress-poweroff.c | 2 +- drivers/power/reset/xgene-reboot.c | 2 +- drivers/power/tps65090-charger.c | 2 +- 10 files changed, 11 insertions(+), 11 deletions(-) diff --git a/drivers/power/charger-manager.c b/drivers/power/charger-manager.c index 14b0d85..bfd3842 100644 --- a/drivers/power/charger-manager.c +++ b/drivers/power/charger-manager.c @@ -1459,7 +1459,7 @@ static int cm_init_thermal_data(struct charger_manager *cm, return ret; } -static struct of_device_id charger_manager_match[] = { +static const struct of_device_id charger_manager_match[] = { { .compatible = "charger-manager", }, diff --git a/drivers/power/reset/at91-poweroff.c b/drivers/power/reset/at91-poweroff.c index 4b72ea5..9847cfb 100644 --- a/drivers/power/reset/at91-poweroff.c +++ b/drivers/power/reset/at91-poweroff.c @@ -140,7 +140,7 @@ static int at91_poweroff_probe(struct platform_device *pdev) return 0; } -static struct of_device_id at91_poweroff_of_match[] = { +static const struct of_device_id at91_poweroff_of_match[] = { { .compatible = "atmel,at91sam9260-shdwc", }, { .compatible = "atmel,at91sam9rl-shdwc", }, { .compatible = "atmel,at91sam9x5-shdwc", }, diff --git a/drivers/power/reset/at91-reset.c b/drivers/power/reset/at91-reset.c index 13584e2..68b1896 100644 --- a/drivers/power/reset/at91-reset.c +++ b/drivers/power/reset/at91-reset.c @@ -152,14 +152,14 @@ static void __init at91_reset_status(struct platform_device *pdev) pr_info("AT91: Starting after %s\n", reason); } -static struct of_device_id at91_ramc_of_match[] = { +static const struct of_device_id at91_ramc_of_match[] = { { .compatible = "atmel,at91sam9260-sdramc", }, { .compatible = "atmel,at91sam9g45-ddramc", }, { .compatible = "atmel,sama5d3-ddramc", }, { /* sentinel */ } }; -static struct of_device_id at91_reset_of_match[] = { +static const struct of_device_id at91_reset_of_match[] = { { .compatible = "atmel,at91sam9260-rstc", .data = at91sam9260_restart }, { .compatible = "atmel,at91sam9g45-rstc", .data = at91sam9g45_restart }, { /* sentinel */ } diff --git a/drivers/power/reset/hisi-reboot.c b/drivers/power/reset/hisi-reboot.c index 5385460..9ab7f56 100644 --- a/drivers/power/reset/hisi-reboot.c +++ b/drivers/power/reset/hisi-reboot.c @@ -64,7 +64,7 @@ static int hisi_reboot_probe(struct platform_device *pdev) return err; } -static struct of_device_id hisi_reboot_of_match[] = { +static const struct of_device_id hisi_reboot_of_match[] = { { .compatible = "hisilicon,sysctrl" }, {} }; diff --git a/drivers/power/reset/keystone-reset.c b/drivers/power/reset/keystone-reset.c index faedf16..c70f1bf 100644 --- a/drivers/power/reset/keystone-reset.c +++ b/drivers/power/reset/keystone-reset.c @@ -70,7 +70,7 @@ static struct notifier_block rsctrl_restart_nb = { .priority = 128, }; -static struct of_device_id rsctrl_of_match[] = { +static const struct of_device_id rsctrl_of_match[] = { {.compatible = "ti,keystone-reset", }, {}, }; diff --git a/drivers/power/reset/st-poweroff.c b/drivers/power/reset/st-poweroff.c index 27383de..a488877 100644 --- a/drivers/power/reset/st-poweroff.c +++ b/drivers/power/reset/st-poweroff.c @@ -97,7 +97,7 @@ static struct notifier_block st_restart_nb = { .priority = 192, }; -static struct of_device_id st_reset_of_match[] = { +static const struct of_device_id st_reset_of_match[] = { { .compatible = "st,stih415-restart", .data = (void *)&stih415_reset, diff --git a/drivers/power/reset/syscon-reboot.c b/drivers/power/reset/syscon-reboot.c index c4049f4..d3c7d24 100644 --- a/drivers/power/reset/syscon-reboot.c +++ b/drivers/power/reset/syscon-reboot.c @@ -76,7 +76,7 @@ static int syscon_reboot_probe(struct platform_device *pdev) return err; } -static struct of_device_id syscon_reboot_of_match[] = { +static const struct of_device_id syscon_reboot_of_match[] = { { .compatible = "syscon-reboot" }, {} }; diff --git a/drivers/power/reset/vexpress-poweroff.c b/drivers/power/reset/vexpress-poweroff.c index be12d9b..6a9bf70 100644 --- a/drivers/power/reset/vexpress-poweroff.c +++ b/drivers/power/reset/vexpress-poweroff.c @@ -80,7 +80,7 @@ DEVICE_ATTR(active, S_IRUGO | S_IWUSR, vexpress_reset_active_show, enum vexpress_reset_func { FUNC_RESET, FUNC_SHUTDOWN, FUNC_REBOOT }; -static struct of_device_id vexpress_reset_of_match[] = { +static const struct of_device_id vexpress_reset_of_match[] = { { .compatible = "arm,vexpress-reset", .data = (void *)FUNC_RESET, diff --git a/drivers/power/reset/xgene-reboot.c b/drivers/power/reset/xgene-reboot.c index b0e5002..f07e93c 100644 --- a/drivers/power/reset/xgene-reboot.c +++ b/drivers/power/reset/xgene-reboot.c @@ -87,7 +87,7 @@ static int xgene_reboot_probe(struct platform_device *pdev) return err; } -static struct of_device_id xgene_reboot_of_match[] = { +static const struct of_device_id xgene_reboot_of_match[] = { { .compatible = "apm,xgene-reboot" }, {} }; diff --git a/drivers/power/tps65090-charger.c b/drivers/power/tps65090-charger.c index 0f4e597..1c2edcd 100644 --- a/drivers/power/tps65090-charger.c +++ b/drivers/power/tps65090-charger.c @@ -344,7 +344,7 @@ static int tps65090_charger_remove(struct platform_device *pdev) return 0; } -static struct of_device_id of_tps65090_charger_match[] = { +static const struct of_device_id of_tps65090_charger_match[] = { { .compatible = "ti,tps65090-charger", }, { /* end */ } }; -- 2.1.0 ^ permalink raw reply related [flat|nested] 49+ messages in thread
* [PATCH 06/35 linux-next] power: constify of_device_id array 2015-03-16 19:17 ` [PATCH 06/35 linux-next] power: " Fabian Frederick @ 2015-03-17 11:38 ` Sudeep Holla 2015-03-20 12:32 ` Sebastian Reichel 1 sibling, 0 replies; 49+ messages in thread From: Sudeep Holla @ 2015-03-17 11:38 UTC (permalink / raw) To: linux-arm-kernel On 16/03/15 19:17, Fabian Frederick wrote: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > Not sure if you really need acks for simple change like this, anyways for vexpress parts in patch 06, 26 and 32 Acked-by: Sudeep Holla <sudeep.holla@arm.com> Regards, Sudeep ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 06/35 linux-next] power: constify of_device_id array 2015-03-16 19:17 ` [PATCH 06/35 linux-next] power: " Fabian Frederick 2015-03-17 11:38 ` Sudeep Holla @ 2015-03-20 12:32 ` Sebastian Reichel 1 sibling, 0 replies; 49+ messages in thread From: Sebastian Reichel @ 2015-03-20 12:32 UTC (permalink / raw) To: linux-arm-kernel Hi Fabian, On Mon, Mar 16, 2015 at 08:17:12PM +0100, Fabian Frederick wrote: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > > Signed-off-by: Fabian Frederick <fabf@skynet.be> Thanks, applied: http://git.infradead.org/battery-2.6.git/commit/8fb0885504cc029bd48877fa89ba91c33c0110b5 -- Sebastian -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150320/f070b921/attachment-0001.sig> ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 08/35 linux-next] dma: constify of_device_id array 2015-03-16 19:17 [PATCH 00/35 linux-next] constify of_device_id array Fabian Frederick 2015-03-16 19:17 ` [PATCH 05/35 linux-next] tty: " Fabian Frederick 2015-03-16 19:17 ` [PATCH 06/35 linux-next] power: " Fabian Frederick @ 2015-03-16 19:17 ` Fabian Frederick 2015-03-18 16:43 ` Vinod Koul [not found] ` <1426533651-25586-1-git-send-email-fabf@skynet.be> ` (2 subsequent siblings) 5 siblings, 1 reply; 49+ messages in thread From: Fabian Frederick @ 2015-03-16 19:17 UTC (permalink / raw) To: linux-arm-kernel of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick <fabf@skynet.be> --- drivers/dma/bestcomm/bestcomm.c | 4 ++-- drivers/dma/k3dma.c | 2 +- drivers/dma/mmp_pdma.c | 2 +- drivers/dma/mmp_tdma.c | 2 +- drivers/dma/mpc512x_dma.c | 2 +- drivers/dma/mv_xor.c | 2 +- drivers/dma/sirf-dma.c | 2 +- drivers/dma/sun6i-dma.c | 2 +- 8 files changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/dma/bestcomm/bestcomm.c b/drivers/dma/bestcomm/bestcomm.c index fa378d8..180fedb 100644 --- a/drivers/dma/bestcomm/bestcomm.c +++ b/drivers/dma/bestcomm/bestcomm.c @@ -30,7 +30,7 @@ #define DRIVER_NAME "bestcomm-core" /* MPC5200 device tree match tables */ -static struct of_device_id mpc52xx_sram_ids[] = { +static const struct of_device_id mpc52xx_sram_ids[] = { { .compatible = "fsl,mpc5200-sram", }, { .compatible = "mpc5200-sram", }, {} @@ -481,7 +481,7 @@ static int mpc52xx_bcom_remove(struct platform_device *op) return 0; } -static struct of_device_id mpc52xx_bcom_of_match[] = { +static const struct of_device_id mpc52xx_bcom_of_match[] = { { .compatible = "fsl,mpc5200-bestcomm", }, { .compatible = "mpc5200-bestcomm", }, {}, diff --git a/drivers/dma/k3dma.c b/drivers/dma/k3dma.c index 6f7f435..3ac9256 100644 --- a/drivers/dma/k3dma.c +++ b/drivers/dma/k3dma.c @@ -654,7 +654,7 @@ static void k3_dma_free_desc(struct virt_dma_desc *vd) kfree(ds); } -static struct of_device_id k3_pdma_dt_ids[] = { +static const struct of_device_id k3_pdma_dt_ids[] = { { .compatible = "hisilicon,k3-dma-1.0", }, {} }; diff --git a/drivers/dma/mmp_pdma.c b/drivers/dma/mmp_pdma.c index eb41004..462a022 100644 --- a/drivers/dma/mmp_pdma.c +++ b/drivers/dma/mmp_pdma.c @@ -973,7 +973,7 @@ static int mmp_pdma_chan_init(struct mmp_pdma_device *pdev, int idx, int irq) return 0; } -static struct of_device_id mmp_pdma_dt_ids[] = { +static const struct of_device_id mmp_pdma_dt_ids[] = { { .compatible = "marvell,pdma-1.0", }, {} }; diff --git a/drivers/dma/mmp_tdma.c b/drivers/dma/mmp_tdma.c index b6f4e1f..449e785 100644 --- a/drivers/dma/mmp_tdma.c +++ b/drivers/dma/mmp_tdma.c @@ -613,7 +613,7 @@ struct dma_chan *mmp_tdma_xlate(struct of_phandle_args *dma_spec, return dma_request_channel(mask, mmp_tdma_filter_fn, ¶m); } -static struct of_device_id mmp_tdma_dt_ids[] = { +static const struct of_device_id mmp_tdma_dt_ids[] = { { .compatible = "marvell,adma-1.0", .data = (void *)MMP_AUD_TDMA}, { .compatible = "marvell,pxa910-squ", .data = (void *)PXA910_SQU}, {} diff --git a/drivers/dma/mpc512x_dma.c b/drivers/dma/mpc512x_dma.c index 57d2457..9553a8d 100644 --- a/drivers/dma/mpc512x_dma.c +++ b/drivers/dma/mpc512x_dma.c @@ -1072,7 +1072,7 @@ static int mpc_dma_remove(struct platform_device *op) return 0; } -static struct of_device_id mpc_dma_match[] = { +static const struct of_device_id mpc_dma_match[] = { { .compatible = "fsl,mpc5121-dma", }, { .compatible = "fsl,mpc8308-dma", }, {}, diff --git a/drivers/dma/mv_xor.c b/drivers/dma/mv_xor.c index b03e813..9b2fa54 100644 --- a/drivers/dma/mv_xor.c +++ b/drivers/dma/mv_xor.c @@ -1249,7 +1249,7 @@ static int mv_xor_remove(struct platform_device *pdev) } #ifdef CONFIG_OF -static struct of_device_id mv_xor_dt_ids[] = { +static const struct of_device_id mv_xor_dt_ids[] = { { .compatible = "marvell,orion-xor", }, {}, }; diff --git a/drivers/dma/sirf-dma.c b/drivers/dma/sirf-dma.c index d0086e9..a1afda4 100644 --- a/drivers/dma/sirf-dma.c +++ b/drivers/dma/sirf-dma.c @@ -896,7 +896,7 @@ static const struct dev_pm_ops sirfsoc_dma_pm_ops = { SET_SYSTEM_SLEEP_PM_OPS(sirfsoc_dma_pm_suspend, sirfsoc_dma_pm_resume) }; -static struct of_device_id sirfsoc_dma_match[] = { +static const struct of_device_id sirfsoc_dma_match[] = { { .compatible = "sirf,prima2-dmac", }, { .compatible = "sirf,marco-dmac", }, {}, diff --git a/drivers/dma/sun6i-dma.c b/drivers/dma/sun6i-dma.c index 7ebcf9b..5358453 100644 --- a/drivers/dma/sun6i-dma.c +++ b/drivers/dma/sun6i-dma.c @@ -896,7 +896,7 @@ static struct sun6i_dma_config sun8i_a23_dma_cfg = { .nr_max_vchans = 37, }; -static struct of_device_id sun6i_dma_match[] = { +static const struct of_device_id sun6i_dma_match[] = { { .compatible = "allwinner,sun6i-a31-dma", .data = &sun6i_a31_dma_cfg }, { .compatible = "allwinner,sun8i-a23-dma", .data = &sun8i_a23_dma_cfg }, { /* sentinel */ } -- 2.1.0 ^ permalink raw reply related [flat|nested] 49+ messages in thread
* [PATCH 08/35 linux-next] dma: constify of_device_id array 2015-03-16 19:17 ` [PATCH 08/35 linux-next] dma: " Fabian Frederick @ 2015-03-18 16:43 ` Vinod Koul 0 siblings, 0 replies; 49+ messages in thread From: Vinod Koul @ 2015-03-18 16:43 UTC (permalink / raw) To: linux-arm-kernel On Mon, Mar 16, 2015 at 08:17:14PM +0100, Fabian Frederick wrote: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > Applied, thanks -- ~Vinod ^ permalink raw reply [flat|nested] 49+ messages in thread
[parent not found: <1426533651-25586-1-git-send-email-fabf@skynet.be>]
* [PATCH 11/35 linux-next] usb: gadget: constify of_device_id array [not found] ` <1426533651-25586-1-git-send-email-fabf@skynet.be> @ 2015-03-16 19:20 ` Fabian Frederick 2015-03-16 19:20 ` [PATCH 16/35 linux-next] video: " Fabian Frederick 2015-03-16 19:20 ` [PATCH 17/35 linux-next] coresight-replicator: " Fabian Frederick 2 siblings, 0 replies; 49+ messages in thread From: Fabian Frederick @ 2015-03-16 19:20 UTC (permalink / raw) To: linux-arm-kernel of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick <fabf@skynet.be> --- drivers/usb/gadget/udc/pxa27x_udc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/gadget/udc/pxa27x_udc.c b/drivers/usb/gadget/udc/pxa27x_udc.c index 486f754..b51226a 100644 --- a/drivers/usb/gadget/udc/pxa27x_udc.c +++ b/drivers/usb/gadget/udc/pxa27x_udc.c @@ -2387,7 +2387,7 @@ static struct pxa_udc memory = { }; #if defined(CONFIG_OF) -static struct of_device_id udc_pxa_dt_ids[] = { +static const struct of_device_id udc_pxa_dt_ids[] = { { .compatible = "marvell,pxa270-udc" }, {} }; -- 2.1.0 ^ permalink raw reply related [flat|nested] 49+ messages in thread
* [PATCH 16/35 linux-next] video: constify of_device_id array [not found] ` <1426533651-25586-1-git-send-email-fabf@skynet.be> 2015-03-16 19:20 ` [PATCH 11/35 linux-next] usb: gadget: " Fabian Frederick @ 2015-03-16 19:20 ` Fabian Frederick 2015-03-16 19:29 ` Timur Tabi 2015-03-16 19:20 ` [PATCH 17/35 linux-next] coresight-replicator: " Fabian Frederick 2 siblings, 1 reply; 49+ messages in thread From: Fabian Frederick @ 2015-03-16 19:20 UTC (permalink / raw) To: linux-arm-kernel of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick <fabf@skynet.be> --- drivers/video/backlight/gpio_backlight.c | 2 +- drivers/video/backlight/pwm_bl.c | 2 +- drivers/video/fbdev/fsl-diu-fb.c | 2 +- drivers/video/fbdev/grvga.c | 2 +- drivers/video/fbdev/mb862xx/mb862xxfbdrv.c | 2 +- drivers/video/fbdev/ocfb.c | 2 +- drivers/video/fbdev/platinumfb.c | 2 +- drivers/video/fbdev/xilinxfb.c | 2 +- 8 files changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/video/backlight/gpio_backlight.c b/drivers/video/backlight/gpio_backlight.c index 439feb2..4751463 100644 --- a/drivers/video/backlight/gpio_backlight.c +++ b/drivers/video/backlight/gpio_backlight.c @@ -142,7 +142,7 @@ static int gpio_backlight_probe(struct platform_device *pdev) } #ifdef CONFIG_OF -static struct of_device_id gpio_backlight_of_match[] = { +static const struct of_device_id gpio_backlight_of_match[] = { { .compatible = "gpio-backlight" }, { /* sentinel */ } }; diff --git a/drivers/video/backlight/pwm_bl.c b/drivers/video/backlight/pwm_bl.c index 3a145a6..2f1de60 100644 --- a/drivers/video/backlight/pwm_bl.c +++ b/drivers/video/backlight/pwm_bl.c @@ -178,7 +178,7 @@ static int pwm_backlight_parse_dt(struct device *dev, return 0; } -static struct of_device_id pwm_backlight_of_match[] = { +static const struct of_device_id pwm_backlight_of_match[] = { { .compatible = "pwm-backlight" }, { } }; diff --git a/drivers/video/fbdev/fsl-diu-fb.c b/drivers/video/fbdev/fsl-diu-fb.c index 7fa2e6f..5783e66 100644 --- a/drivers/video/fbdev/fsl-diu-fb.c +++ b/drivers/video/fbdev/fsl-diu-fb.c @@ -1865,7 +1865,7 @@ static int __init fsl_diu_setup(char *options) } #endif -static struct of_device_id fsl_diu_match[] = { +static const struct of_device_id fsl_diu_match[] = { #ifdef CONFIG_PPC_MPC512x { .compatible = "fsl,mpc5121-diu", diff --git a/drivers/video/fbdev/grvga.c b/drivers/video/fbdev/grvga.c index b471f92..bced3f3 100644 --- a/drivers/video/fbdev/grvga.c +++ b/drivers/video/fbdev/grvga.c @@ -535,7 +535,7 @@ static int grvga_remove(struct platform_device *device) return 0; } -static struct of_device_id svgactrl_of_match[] = { +static const struct of_device_id svgactrl_of_match[] = { { .name = "GAISLER_SVGACTRL", }, diff --git a/drivers/video/fbdev/mb862xx/mb862xxfbdrv.c b/drivers/video/fbdev/mb862xx/mb862xxfbdrv.c index 9b8bebd..f56dbea 100644 --- a/drivers/video/fbdev/mb862xx/mb862xxfbdrv.c +++ b/drivers/video/fbdev/mb862xx/mb862xxfbdrv.c @@ -821,7 +821,7 @@ static int of_platform_mb862xx_remove(struct platform_device *ofdev) /* * common types */ -static struct of_device_id of_platform_mb862xx_tbl[] = { +static const struct of_device_id of_platform_mb862xx_tbl[] = { { .compatible = "fujitsu,MB86276", }, { .compatible = "fujitsu,lime", }, { .compatible = "fujitsu,MB86277", }, diff --git a/drivers/video/fbdev/ocfb.c b/drivers/video/fbdev/ocfb.c index de98196..718ad60 100644 --- a/drivers/video/fbdev/ocfb.c +++ b/drivers/video/fbdev/ocfb.c @@ -395,7 +395,7 @@ static int ocfb_remove(struct platform_device *pdev) return 0; } -static struct of_device_id ocfb_match[] = { +static const struct of_device_id ocfb_match[] = { { .compatible = "opencores,ocfb", }, {}, }; diff --git a/drivers/video/fbdev/platinumfb.c b/drivers/video/fbdev/platinumfb.c index 518d1fd..37a77bc 100644 --- a/drivers/video/fbdev/platinumfb.c +++ b/drivers/video/fbdev/platinumfb.c @@ -667,7 +667,7 @@ static int platinumfb_remove(struct platform_device* odev) return 0; } -static struct of_device_id platinumfb_match[] = +static const struct of_device_id platinumfb_match[] = { { .name = "platinum", diff --git a/drivers/video/fbdev/xilinxfb.c b/drivers/video/fbdev/xilinxfb.c index 17dc119..fe2673b 100644 --- a/drivers/video/fbdev/xilinxfb.c +++ b/drivers/video/fbdev/xilinxfb.c @@ -482,7 +482,7 @@ static int xilinxfb_of_remove(struct platform_device *op) } /* Match table for of_platform binding */ -static struct of_device_id xilinxfb_of_match[] = { +static const struct of_device_id xilinxfb_of_match[] = { { .compatible = "xlnx,xps-tft-1.00.a", }, { .compatible = "xlnx,xps-tft-2.00.a", }, { .compatible = "xlnx,xps-tft-2.01.a", }, -- 2.1.0 ^ permalink raw reply related [flat|nested] 49+ messages in thread
* [PATCH 16/35 linux-next] video: constify of_device_id array 2015-03-16 19:20 ` [PATCH 16/35 linux-next] video: " Fabian Frederick @ 2015-03-16 19:29 ` Timur Tabi 0 siblings, 0 replies; 49+ messages in thread From: Timur Tabi @ 2015-03-16 19:29 UTC (permalink / raw) To: linux-arm-kernel On 03/16/2015 02:20 PM, Fabian Frederick wrote: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > > Signed-off-by: Fabian Frederick<fabf@skynet.be> ... > drivers/video/fbdev/fsl-diu-fb.c | 2 +- For this driver: Acked-by: Timur Tabi <timur@tabi.org> ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 17/35 linux-next] coresight-replicator: constify of_device_id array [not found] ` <1426533651-25586-1-git-send-email-fabf@skynet.be> 2015-03-16 19:20 ` [PATCH 11/35 linux-next] usb: gadget: " Fabian Frederick 2015-03-16 19:20 ` [PATCH 16/35 linux-next] video: " Fabian Frederick @ 2015-03-16 19:20 ` Fabian Frederick 2 siblings, 0 replies; 49+ messages in thread From: Fabian Frederick @ 2015-03-16 19:20 UTC (permalink / raw) To: linux-arm-kernel of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick <fabf@skynet.be> --- drivers/coresight/coresight-replicator.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/coresight/coresight-replicator.c b/drivers/coresight/coresight-replicator.c index cdf0553..75b9abd 100644 --- a/drivers/coresight/coresight-replicator.c +++ b/drivers/coresight/coresight-replicator.c @@ -107,7 +107,7 @@ static int replicator_remove(struct platform_device *pdev) return 0; } -static struct of_device_id replicator_match[] = { +static const struct of_device_id replicator_match[] = { {.compatible = "arm,coresight-replicator"}, {} }; -- 2.1.0 ^ permalink raw reply related [flat|nested] 49+ messages in thread
* [PATCH 22/35 linux-next] soc: ti: constify of_device_id array 2015-03-16 19:17 [PATCH 00/35 linux-next] constify of_device_id array Fabian Frederick ` (3 preceding siblings ...) [not found] ` <1426533651-25586-1-git-send-email-fabf@skynet.be> @ 2015-03-16 19:54 ` Fabian Frederick 2015-03-16 19:54 ` [PATCH 23/35 linux-next] [media] " Fabian Frederick ` (6 more replies) 2015-03-16 19:59 ` [PATCH 32/35 linux-next] clk: " Fabian Frederick 5 siblings, 7 replies; 49+ messages in thread From: Fabian Frederick @ 2015-03-16 19:54 UTC (permalink / raw) To: linux-arm-kernel of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick <fabf@skynet.be> --- drivers/soc/ti/knav_dma.c | 2 +- drivers/soc/ti/knav_qmss_queue.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/soc/ti/knav_dma.c b/drivers/soc/ti/knav_dma.c index bc1b80e..c467d7a 100644 --- a/drivers/soc/ti/knav_dma.c +++ b/drivers/soc/ti/knav_dma.c @@ -791,7 +791,7 @@ static int knav_dma_remove(struct platform_device *pdev) return 0; } -static struct of_device_id of_match[] = { +static const struct of_device_id of_match[] = { { .compatible = "ti,keystone-navigator-dma", }, {}, }; diff --git a/drivers/soc/ti/knav_qmss_queue.c b/drivers/soc/ti/knav_qmss_queue.c index 6d8646d..dd474e4 100644 --- a/drivers/soc/ti/knav_qmss_queue.c +++ b/drivers/soc/ti/knav_qmss_queue.c @@ -1803,7 +1803,7 @@ static int knav_queue_remove(struct platform_device *pdev) } /* Match table for of_platform binding */ -static struct of_device_id keystone_qmss_of_match[] = { +static const struct of_device_id keystone_qmss_of_match[] = { { .compatible = "ti,keystone-navigator-qmss", }, {}, }; -- 2.1.0 ^ permalink raw reply related [flat|nested] 49+ messages in thread
* [PATCH 23/35 linux-next] [media] constify of_device_id array 2015-03-16 19:54 ` [PATCH 22/35 linux-next] soc: ti: " Fabian Frederick @ 2015-03-16 19:54 ` Fabian Frederick 2015-03-17 8:12 ` Patrice Chotard 2015-03-16 19:54 ` [PATCH 25/35 linux-next] PCI: " Fabian Frederick ` (5 subsequent siblings) 6 siblings, 1 reply; 49+ messages in thread From: Fabian Frederick @ 2015-03-16 19:54 UTC (permalink / raw) To: linux-arm-kernel of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick <fabf@skynet.be> --- drivers/media/i2c/adv7604.c | 2 +- drivers/media/platform/fsl-viu.c | 2 +- drivers/media/platform/soc_camera/rcar_vin.c | 2 +- drivers/media/rc/gpio-ir-recv.c | 2 +- drivers/media/rc/ir-hix5hd2.c | 2 +- drivers/media/rc/st_rc.c | 2 +- 6 files changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/media/i2c/adv7604.c b/drivers/media/i2c/adv7604.c index d228b7c..5a7c938 100644 --- a/drivers/media/i2c/adv7604.c +++ b/drivers/media/i2c/adv7604.c @@ -2598,7 +2598,7 @@ static struct i2c_device_id adv7604_i2c_id[] = { }; MODULE_DEVICE_TABLE(i2c, adv7604_i2c_id); -static struct of_device_id adv7604_of_id[] __maybe_unused = { +static const struct of_device_id adv7604_of_id[] __maybe_unused = { { .compatible = "adi,adv7611", .data = &adv7604_chip_info[ADV7611] }, { } }; diff --git a/drivers/media/platform/fsl-viu.c b/drivers/media/platform/fsl-viu.c index bbf4281..5b76e3d 100644 --- a/drivers/media/platform/fsl-viu.c +++ b/drivers/media/platform/fsl-viu.c @@ -1664,7 +1664,7 @@ static int viu_resume(struct platform_device *op) /* * Initialization and module stuff */ -static struct of_device_id mpc512x_viu_of_match[] = { +static const struct of_device_id mpc512x_viu_of_match[] = { { .compatible = "fsl,mpc5121-viu", }, diff --git a/drivers/media/platform/soc_camera/rcar_vin.c b/drivers/media/platform/soc_camera/rcar_vin.c index 279ab9f..bbbaa0a 100644 --- a/drivers/media/platform/soc_camera/rcar_vin.c +++ b/drivers/media/platform/soc_camera/rcar_vin.c @@ -1818,7 +1818,7 @@ static struct soc_camera_host_ops rcar_vin_host_ops = { }; #ifdef CONFIG_OF -static struct of_device_id rcar_vin_of_table[] = { +static const struct of_device_id rcar_vin_of_table[] = { { .compatible = "renesas,vin-r8a7794", .data = (void *)RCAR_GEN2 }, { .compatible = "renesas,vin-r8a7793", .data = (void *)RCAR_GEN2 }, { .compatible = "renesas,vin-r8a7791", .data = (void *)RCAR_GEN2 }, diff --git a/drivers/media/rc/gpio-ir-recv.c b/drivers/media/rc/gpio-ir-recv.c index 229853d..707df54 100644 --- a/drivers/media/rc/gpio-ir-recv.c +++ b/drivers/media/rc/gpio-ir-recv.c @@ -59,7 +59,7 @@ static int gpio_ir_recv_get_devtree_pdata(struct device *dev, return 0; } -static struct of_device_id gpio_ir_recv_of_match[] = { +static const struct of_device_id gpio_ir_recv_of_match[] = { { .compatible = "gpio-ir-receiver", }, { }, }; diff --git a/drivers/media/rc/ir-hix5hd2.c b/drivers/media/rc/ir-hix5hd2.c index b0df629..0a11d55 100644 --- a/drivers/media/rc/ir-hix5hd2.c +++ b/drivers/media/rc/ir-hix5hd2.c @@ -327,7 +327,7 @@ static int hix5hd2_ir_resume(struct device *dev) static SIMPLE_DEV_PM_OPS(hix5hd2_ir_pm_ops, hix5hd2_ir_suspend, hix5hd2_ir_resume); -static struct of_device_id hix5hd2_ir_table[] = { +static const struct of_device_id hix5hd2_ir_table[] = { { .compatible = "hisilicon,hix5hd2-ir", }, {}, }; diff --git a/drivers/media/rc/st_rc.c b/drivers/media/rc/st_rc.c index 0e758ae..50ea09d 100644 --- a/drivers/media/rc/st_rc.c +++ b/drivers/media/rc/st_rc.c @@ -381,7 +381,7 @@ static int st_rc_resume(struct device *dev) static SIMPLE_DEV_PM_OPS(st_rc_pm_ops, st_rc_suspend, st_rc_resume); #ifdef CONFIG_OF -static struct of_device_id st_rc_match[] = { +static const struct of_device_id st_rc_match[] = { { .compatible = "st,comms-irb", }, {}, }; -- 2.1.0 ^ permalink raw reply related [flat|nested] 49+ messages in thread
* [PATCH 23/35 linux-next] [media] constify of_device_id array 2015-03-16 19:54 ` [PATCH 23/35 linux-next] [media] " Fabian Frederick @ 2015-03-17 8:12 ` Patrice Chotard 0 siblings, 0 replies; 49+ messages in thread From: Patrice Chotard @ 2015-03-17 8:12 UTC (permalink / raw) To: linux-arm-kernel Hi Fabian On 03/16/2015 08:54 PM, Fabian Frederick wrote: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > > Signed-off-by: Fabian Frederick <fabf@skynet.be> > --- ... > drivers/media/rc/st_rc.c | 2 +- For this driver Acked-by: Patrice Chotard <patrice.chotard@st.com> Thanks ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 25/35 linux-next] PCI: constify of_device_id array 2015-03-16 19:54 ` [PATCH 22/35 linux-next] soc: ti: " Fabian Frederick 2015-03-16 19:54 ` [PATCH 23/35 linux-next] [media] " Fabian Frederick @ 2015-03-16 19:54 ` Fabian Frederick 2015-03-16 20:23 ` Sergei Shtylyov 2015-03-16 19:54 ` [PATCH 26/35 linux-next] hwmon: " Fabian Frederick ` (4 subsequent siblings) 6 siblings, 1 reply; 49+ messages in thread From: Fabian Frederick @ 2015-03-16 19:54 UTC (permalink / raw) To: linux-arm-kernel of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick <fabf@skynet.be> --- drivers/gpio/gpio-mpc8xxx.c | 2 +- drivers/gpio/gpio-octeon.c | 2 +- drivers/gpio/gpio-tz1090-pdc.c | 2 +- drivers/gpio/gpio-tz1090.c | 2 +- drivers/gpio/gpio-zynq.c | 2 +- drivers/pci/host/pci-rcar-gen2.c | 2 +- drivers/pci/host/pcie-xilinx.c | 2 +- 7 files changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/gpio/gpio-mpc8xxx.c b/drivers/gpio/gpio-mpc8xxx.c index a65b751..59c4c11 100644 --- a/drivers/gpio/gpio-mpc8xxx.c +++ b/drivers/gpio/gpio-mpc8xxx.c @@ -334,7 +334,7 @@ static struct irq_domain_ops mpc8xxx_gpio_irq_ops = { .xlate = irq_domain_xlate_twocell, }; -static struct of_device_id mpc8xxx_gpio_ids[] = { +static const struct of_device_id mpc8xxx_gpio_ids[] = { { .compatible = "fsl,mpc8349-gpio", }, { .compatible = "fsl,mpc8572-gpio", }, { .compatible = "fsl,mpc8610-gpio", }, diff --git a/drivers/gpio/gpio-octeon.c b/drivers/gpio/gpio-octeon.c index 62ae251..41b5a4f 100644 --- a/drivers/gpio/gpio-octeon.c +++ b/drivers/gpio/gpio-octeon.c @@ -133,7 +133,7 @@ static int octeon_gpio_remove(struct platform_device *pdev) return 0; } -static struct of_device_id octeon_gpio_match[] = { +static const struct of_device_id octeon_gpio_match[] = { { .compatible = "cavium,octeon-3860-gpio", }, diff --git a/drivers/gpio/gpio-tz1090-pdc.c b/drivers/gpio/gpio-tz1090-pdc.c index ede7e40..d04d814 100644 --- a/drivers/gpio/gpio-tz1090-pdc.c +++ b/drivers/gpio/gpio-tz1090-pdc.c @@ -222,7 +222,7 @@ static int tz1090_pdc_gpio_probe(struct platform_device *pdev) return 0; } -static struct of_device_id tz1090_pdc_gpio_of_match[] = { +static const struct of_device_id tz1090_pdc_gpio_of_match[] = { { .compatible = "img,tz1090-pdc-gpio" }, { }, }; diff --git a/drivers/gpio/gpio-tz1090.c b/drivers/gpio/gpio-tz1090.c index 445660a..44f9dc7 100644 --- a/drivers/gpio/gpio-tz1090.c +++ b/drivers/gpio/gpio-tz1090.c @@ -585,7 +585,7 @@ static int tz1090_gpio_probe(struct platform_device *pdev) return 0; } -static struct of_device_id tz1090_gpio_of_match[] = { +static const struct of_device_id tz1090_gpio_of_match[] = { { .compatible = "img,tz1090-gpio" }, { }, }; diff --git a/drivers/gpio/gpio-zynq.c b/drivers/gpio/gpio-zynq.c index 184c4b1..8ff311a 100644 --- a/drivers/gpio/gpio-zynq.c +++ b/drivers/gpio/gpio-zynq.c @@ -695,7 +695,7 @@ static int zynq_gpio_remove(struct platform_device *pdev) return 0; } -static struct of_device_id zynq_gpio_of_match[] = { +static const struct of_device_id zynq_gpio_of_match[] = { { .compatible = "xlnx,zynq-gpio-1.0", }, { /* end of table */ } }; diff --git a/drivers/pci/host/pci-rcar-gen2.c b/drivers/pci/host/pci-rcar-gen2.c index dd6b84e..db52c6a 100644 --- a/drivers/pci/host/pci-rcar-gen2.c +++ b/drivers/pci/host/pci-rcar-gen2.c @@ -356,7 +356,7 @@ static int rcar_pci_probe(struct platform_device *pdev) return 0; } -static struct of_device_id rcar_pci_of_match[] = { +static const struct of_device_id rcar_pci_of_match[] = { { .compatible = "renesas,pci-r8a7790", }, { .compatible = "renesas,pci-r8a7791", }, { }, diff --git a/drivers/pci/host/pcie-xilinx.c b/drivers/pci/host/pcie-xilinx.c index f1a06a0..caae2e4 100644 --- a/drivers/pci/host/pcie-xilinx.c +++ b/drivers/pci/host/pcie-xilinx.c @@ -869,7 +869,7 @@ static int xilinx_pcie_remove(struct platform_device *pdev) return 0; } -static struct of_device_id xilinx_pcie_of_match[] = { +static const struct of_device_id xilinx_pcie_of_match[] = { { .compatible = "xlnx,axi-pcie-host-1.00.a", }, {} }; -- 2.1.0 ^ permalink raw reply related [flat|nested] 49+ messages in thread
* [PATCH 25/35 linux-next] PCI: constify of_device_id array 2015-03-16 19:54 ` [PATCH 25/35 linux-next] PCI: " Fabian Frederick @ 2015-03-16 20:23 ` Sergei Shtylyov 2015-03-16 20:30 ` Fabian Frederick 0 siblings, 1 reply; 49+ messages in thread From: Sergei Shtylyov @ 2015-03-16 20:23 UTC (permalink / raw) To: linux-arm-kernel Hello. On 03/16/2015 10:54 PM, Fabian Frederick wrote: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > Signed-off-by: Fabian Frederick <fabf@skynet.be> > --- > drivers/gpio/gpio-mpc8xxx.c | 2 +- > drivers/gpio/gpio-octeon.c | 2 +- > drivers/gpio/gpio-tz1090-pdc.c | 2 +- > drivers/gpio/gpio-tz1090.c | 2 +- > drivers/gpio/gpio-zynq.c | 2 +- What the GPIO driver changes are doing here, in the PCI patch? > drivers/pci/host/pci-rcar-gen2.c | 2 +- > drivers/pci/host/pcie-xilinx.c | 2 +- > 7 files changed, 7 insertions(+), 7 deletions(-) [...] WBR, Sergei ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 25/35 linux-next] PCI: constify of_device_id array 2015-03-16 20:23 ` Sergei Shtylyov @ 2015-03-16 20:30 ` Fabian Frederick 0 siblings, 0 replies; 49+ messages in thread From: Fabian Frederick @ 2015-03-16 20:30 UTC (permalink / raw) To: linux-arm-kernel > On 16 March 2015 at 21:23 Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> > wrote: > > > Hello. > > On 03/16/2015 10:54 PM, Fabian Frederick wrote: > > > of_device_id is always used as const. > > (See driver.of_match_table and open firmware functions) > > > Signed-off-by: Fabian Frederick <fabf@skynet.be> > > --- > >? ?drivers/gpio/gpio-mpc8xxx.c? ? ? | 2 +- > >? ?drivers/gpio/gpio-octeon.c? ? ? ?| 2 +- > >? ?drivers/gpio/gpio-tz1090-pdc.c? ?| 2 +- > >? ?drivers/gpio/gpio-tz1090.c? ? ? ?| 2 +- > >? ?drivers/gpio/gpio-zynq.c? ? ? ? ?| 2 +- > >? ? ?What the GPIO driver changes are doing here, in the PCI patch? Nice catch :) I'll split that patch and only send those to avoid too much noise. Thanks, Fabian > > >? ?drivers/pci/host/pci-rcar-gen2.c | 2 +- > >? ?drivers/pci/host/pcie-xilinx.c? ?| 2 +- > >? ?7 files changed, 7 insertions(+), 7 deletions(-) > > [...] > > WBR, Sergei > ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 26/35 linux-next] hwmon: constify of_device_id array 2015-03-16 19:54 ` [PATCH 22/35 linux-next] soc: ti: " Fabian Frederick 2015-03-16 19:54 ` [PATCH 23/35 linux-next] [media] " Fabian Frederick 2015-03-16 19:54 ` [PATCH 25/35 linux-next] PCI: " Fabian Frederick @ 2015-03-16 19:54 ` Fabian Frederick 2015-03-17 3:55 ` Guenter Roeck 2015-03-16 19:54 ` [PATCH 27/35 linux-next] reset: sti: " Fabian Frederick ` (3 subsequent siblings) 6 siblings, 1 reply; 49+ messages in thread From: Fabian Frederick @ 2015-03-16 19:54 UTC (permalink / raw) To: linux-arm-kernel of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick <fabf@skynet.be> --- drivers/hwmon/pwm-fan.c | 2 +- drivers/hwmon/vexpress.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/hwmon/pwm-fan.c b/drivers/hwmon/pwm-fan.c index 4170728..31d793b 100644 --- a/drivers/hwmon/pwm-fan.c +++ b/drivers/hwmon/pwm-fan.c @@ -322,7 +322,7 @@ static int pwm_fan_resume(struct device *dev) static SIMPLE_DEV_PM_OPS(pwm_fan_pm, pwm_fan_suspend, pwm_fan_resume); -static struct of_device_id of_pwm_fan_match[] = { +static const struct of_device_id of_pwm_fan_match[] = { { .compatible = "pwm-fan", }, {}, }; diff --git a/drivers/hwmon/vexpress.c b/drivers/hwmon/vexpress.c index cf1848b..8ba419d 100644 --- a/drivers/hwmon/vexpress.c +++ b/drivers/hwmon/vexpress.c @@ -193,7 +193,7 @@ static struct vexpress_hwmon_type vexpress_hwmon_energy = { }, }; -static struct of_device_id vexpress_hwmon_of_match[] = { +static const struct of_device_id vexpress_hwmon_of_match[] = { #if !defined(CONFIG_REGULATOR_VEXPRESS) { .compatible = "arm,vexpress-volt", -- 2.1.0 ^ permalink raw reply related [flat|nested] 49+ messages in thread
* [PATCH 26/35 linux-next] hwmon: constify of_device_id array 2015-03-16 19:54 ` [PATCH 26/35 linux-next] hwmon: " Fabian Frederick @ 2015-03-17 3:55 ` Guenter Roeck 0 siblings, 0 replies; 49+ messages in thread From: Guenter Roeck @ 2015-03-17 3:55 UTC (permalink / raw) To: linux-arm-kernel On 03/16/2015 12:54 PM, Fabian Frederick wrote: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > > Signed-off-by: Fabian Frederick <fabf@skynet.be> > --- Applied to hwmon-next. Thanks, Guenter ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 27/35 linux-next] reset: sti: constify of_device_id array 2015-03-16 19:54 ` [PATCH 22/35 linux-next] soc: ti: " Fabian Frederick ` (2 preceding siblings ...) 2015-03-16 19:54 ` [PATCH 26/35 linux-next] hwmon: " Fabian Frederick @ 2015-03-16 19:54 ` Fabian Frederick 2015-03-17 8:13 ` Patrice Chotard ` (2 more replies) 2015-03-16 19:54 ` [PATCH 29/35 linux-next] gpu: " Fabian Frederick ` (2 subsequent siblings) 6 siblings, 3 replies; 49+ messages in thread From: Fabian Frederick @ 2015-03-16 19:54 UTC (permalink / raw) To: linux-arm-kernel of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick <fabf@skynet.be> --- drivers/reset/sti/reset-stih407.c | 2 +- drivers/reset/sti/reset-stih415.c | 2 +- drivers/reset/sti/reset-stih416.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/reset/sti/reset-stih407.c b/drivers/reset/sti/reset-stih407.c index d83db5d7..e250604 100644 --- a/drivers/reset/sti/reset-stih407.c +++ b/drivers/reset/sti/reset-stih407.c @@ -126,7 +126,7 @@ static const struct syscfg_reset_controller_data stih407_picophyreset_controller .channels = stih407_picophyresets, }; -static struct of_device_id stih407_reset_match[] = { +static const struct of_device_id stih407_reset_match[] = { { .compatible = "st,stih407-powerdown", .data = &stih407_powerdown_controller, diff --git a/drivers/reset/sti/reset-stih415.c b/drivers/reset/sti/reset-stih415.c index 8dad603..fd84238 100644 --- a/drivers/reset/sti/reset-stih415.c +++ b/drivers/reset/sti/reset-stih415.c @@ -89,7 +89,7 @@ static struct syscfg_reset_controller_data stih415_softreset_controller = { .channels = stih415_softresets, }; -static struct of_device_id stih415_reset_match[] = { +static const struct of_device_id stih415_reset_match[] = { { .compatible = "st,stih415-powerdown", .data = &stih415_powerdown_controller, }, { .compatible = "st,stih415-softreset", diff --git a/drivers/reset/sti/reset-stih416.c b/drivers/reset/sti/reset-stih416.c index 79aed70..c7ccaed 100644 --- a/drivers/reset/sti/reset-stih416.c +++ b/drivers/reset/sti/reset-stih416.c @@ -120,7 +120,7 @@ static struct syscfg_reset_controller_data stih416_softreset_controller = { .channels = stih416_softresets, }; -static struct of_device_id stih416_reset_match[] = { +static const struct of_device_id stih416_reset_match[] = { { .compatible = "st,stih416-powerdown", .data = &stih416_powerdown_controller, }, { .compatible = "st,stih416-softreset", -- 2.1.0 ^ permalink raw reply related [flat|nested] 49+ messages in thread
* [PATCH 27/35 linux-next] reset: sti: constify of_device_id array 2015-03-16 19:54 ` [PATCH 27/35 linux-next] reset: sti: " Fabian Frederick @ 2015-03-17 8:13 ` Patrice Chotard 2015-03-17 8:33 ` Maxime Coquelin 2015-03-17 9:52 ` Philipp Zabel 2 siblings, 0 replies; 49+ messages in thread From: Patrice Chotard @ 2015-03-17 8:13 UTC (permalink / raw) To: linux-arm-kernel Hi Fabian On 03/16/2015 08:54 PM, Fabian Frederick wrote: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > > Signed-off-by: Fabian Frederick <fabf@skynet.be> > --- > drivers/reset/sti/reset-stih407.c | 2 +- > drivers/reset/sti/reset-stih415.c | 2 +- > drivers/reset/sti/reset-stih416.c | 2 +- > 3 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/reset/sti/reset-stih407.c b/drivers/reset/sti/reset-stih407.c > index d83db5d7..e250604 100644 > --- a/drivers/reset/sti/reset-stih407.c > +++ b/drivers/reset/sti/reset-stih407.c > @@ -126,7 +126,7 @@ static const struct syscfg_reset_controller_data stih407_picophyreset_controller > .channels = stih407_picophyresets, > }; > > -static struct of_device_id stih407_reset_match[] = { > +static const struct of_device_id stih407_reset_match[] = { > { > .compatible = "st,stih407-powerdown", > .data = &stih407_powerdown_controller, > diff --git a/drivers/reset/sti/reset-stih415.c b/drivers/reset/sti/reset-stih415.c > index 8dad603..fd84238 100644 > --- a/drivers/reset/sti/reset-stih415.c > +++ b/drivers/reset/sti/reset-stih415.c > @@ -89,7 +89,7 @@ static struct syscfg_reset_controller_data stih415_softreset_controller = { > .channels = stih415_softresets, > }; > > -static struct of_device_id stih415_reset_match[] = { > +static const struct of_device_id stih415_reset_match[] = { > { .compatible = "st,stih415-powerdown", > .data = &stih415_powerdown_controller, }, > { .compatible = "st,stih415-softreset", > diff --git a/drivers/reset/sti/reset-stih416.c b/drivers/reset/sti/reset-stih416.c > index 79aed70..c7ccaed 100644 > --- a/drivers/reset/sti/reset-stih416.c > +++ b/drivers/reset/sti/reset-stih416.c > @@ -120,7 +120,7 @@ static struct syscfg_reset_controller_data stih416_softreset_controller = { > .channels = stih416_softresets, > }; > > -static struct of_device_id stih416_reset_match[] = { > +static const struct of_device_id stih416_reset_match[] = { > { .compatible = "st,stih416-powerdown", > .data = &stih416_powerdown_controller, }, > { .compatible = "st,stih416-softreset", Acked-by: Patrice Chotard <patrice.chotard@st.com> Thanks ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 27/35 linux-next] reset: sti: constify of_device_id array 2015-03-16 19:54 ` [PATCH 27/35 linux-next] reset: sti: " Fabian Frederick 2015-03-17 8:13 ` Patrice Chotard @ 2015-03-17 8:33 ` Maxime Coquelin 2015-03-17 9:52 ` Philipp Zabel 2 siblings, 0 replies; 49+ messages in thread From: Maxime Coquelin @ 2015-03-17 8:33 UTC (permalink / raw) To: linux-arm-kernel On 03/16/2015 08:54 PM, Fabian Frederick wrote: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > > Signed-off-by: Fabian Frederick <fabf@skynet.be> > --- > drivers/reset/sti/reset-stih407.c | 2 +- > drivers/reset/sti/reset-stih415.c | 2 +- > drivers/reset/sti/reset-stih416.c | 2 +- > 3 files changed, 3 insertions(+), 3 deletions(-) > > Acked-by: Maxime Coquelin <maxime.coquelin@st.com> Thanks! Maxime ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 27/35 linux-next] reset: sti: constify of_device_id array 2015-03-16 19:54 ` [PATCH 27/35 linux-next] reset: sti: " Fabian Frederick 2015-03-17 8:13 ` Patrice Chotard 2015-03-17 8:33 ` Maxime Coquelin @ 2015-03-17 9:52 ` Philipp Zabel 2 siblings, 0 replies; 49+ messages in thread From: Philipp Zabel @ 2015-03-17 9:52 UTC (permalink / raw) To: linux-arm-kernel Hi Fabian, Am Montag, den 16.03.2015, 20:54 +0100 schrieb Fabian Frederick: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > > Signed-off-by: Fabian Frederick <fabf@skynet.be> > --- > drivers/reset/sti/reset-stih407.c | 2 +- > drivers/reset/sti/reset-stih415.c | 2 +- > drivers/reset/sti/reset-stih416.c | 2 +- > 3 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/reset/sti/reset-stih407.c b/drivers/reset/sti/reset-stih407.c > index d83db5d7..e250604 100644 > --- a/drivers/reset/sti/reset-stih407.c > +++ b/drivers/reset/sti/reset-stih407.c > @@ -126,7 +126,7 @@ static const struct syscfg_reset_controller_data stih407_picophyreset_controller > .channels = stih407_picophyresets, > }; > > -static struct of_device_id stih407_reset_match[] = { > +static const struct of_device_id stih407_reset_match[] = { > { > .compatible = "st,stih407-powerdown", > .data = &stih407_powerdown_controller, > diff --git a/drivers/reset/sti/reset-stih415.c b/drivers/reset/sti/reset-stih415.c > index 8dad603..fd84238 100644 > --- a/drivers/reset/sti/reset-stih415.c > +++ b/drivers/reset/sti/reset-stih415.c > @@ -89,7 +89,7 @@ static struct syscfg_reset_controller_data stih415_softreset_controller = { > .channels = stih415_softresets, > }; > > -static struct of_device_id stih415_reset_match[] = { > +static const struct of_device_id stih415_reset_match[] = { > { .compatible = "st,stih415-powerdown", > .data = &stih415_powerdown_controller, }, > { .compatible = "st,stih415-softreset", > diff --git a/drivers/reset/sti/reset-stih416.c b/drivers/reset/sti/reset-stih416.c > index 79aed70..c7ccaed 100644 > --- a/drivers/reset/sti/reset-stih416.c > +++ b/drivers/reset/sti/reset-stih416.c > @@ -120,7 +120,7 @@ static struct syscfg_reset_controller_data stih416_softreset_controller = { > .channels = stih416_softresets, > }; > > -static struct of_device_id stih416_reset_match[] = { > +static const struct of_device_id stih416_reset_match[] = { > { .compatible = "st,stih416-powerdown", > .data = &stih416_powerdown_controller, }, > { .compatible = "st,stih416-softreset", Applied, thanks! regards Philipp ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 29/35 linux-next] gpu: constify of_device_id array 2015-03-16 19:54 ` [PATCH 22/35 linux-next] soc: ti: " Fabian Frederick ` (3 preceding siblings ...) 2015-03-16 19:54 ` [PATCH 27/35 linux-next] reset: sti: " Fabian Frederick @ 2015-03-16 19:54 ` Fabian Frederick 2015-03-16 19:54 ` [PATCH 30/35 linux-next] devfreq: " Fabian Frederick 2015-03-16 19:54 ` [PATCH 31/35 linux-next] EDAC: " Fabian Frederick 6 siblings, 0 replies; 49+ messages in thread From: Fabian Frederick @ 2015-03-16 19:54 UTC (permalink / raw) To: linux-arm-kernel of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick <fabf@skynet.be> --- drivers/gpu/drm/armada/armada_crtc.c | 2 +- drivers/gpu/drm/exynos/exynos_drm_dsi.c | 2 +- drivers/gpu/drm/exynos/exynos_hdmi.c | 2 +- drivers/gpu/drm/exynos/exynos_mixer.c | 2 +- drivers/gpu/drm/panel/panel-ld9040.c | 2 +- drivers/gpu/drm/panel/panel-s6e8aa0.c | 2 +- drivers/gpu/drm/sti/sti_dvo.c | 2 +- drivers/gpu/drm/sti/sti_hqvdp.c | 2 +- drivers/gpu/drm/tilcdc/tilcdc_drv.c | 4 ++-- drivers/gpu/drm/tilcdc/tilcdc_panel.c | 2 +- drivers/gpu/drm/tilcdc/tilcdc_slave.c | 4 ++-- drivers/gpu/drm/tilcdc/tilcdc_tfp410.c | 4 ++-- drivers/gpu/host1x/dev.c | 2 +- drivers/gpu/host1x/mipi.c | 2 +- 14 files changed, 17 insertions(+), 17 deletions(-) diff --git a/drivers/gpu/drm/armada/armada_crtc.c b/drivers/gpu/drm/armada/armada_crtc.c index 42d2ffa..517542f 100644 --- a/drivers/gpu/drm/armada/armada_crtc.c +++ b/drivers/gpu/drm/armada/armada_crtc.c @@ -1201,7 +1201,7 @@ static int armada_lcd_remove(struct platform_device *pdev) return 0; } -static struct of_device_id armada_lcd_of_match[] = { +static const struct of_device_id armada_lcd_of_match[] = { { .compatible = "marvell,dove-lcd", .data = &armada510_ops, diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c b/drivers/gpu/drm/exynos/exynos_drm_dsi.c index 05fe93d..78afb45 100644 --- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c +++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c @@ -330,7 +330,7 @@ static struct exynos_dsi_driver_data exynos5_dsi_driver_data = { .plltmr_reg = 0x58, }; -static struct of_device_id exynos_dsi_of_match[] = { +static const struct of_device_id exynos_dsi_of_match[] = { { .compatible = "samsung,exynos3250-mipi-dsi", .data = &exynos3_dsi_driver_data }, { .compatible = "samsung,exynos4210-mipi-dsi", diff --git a/drivers/gpu/drm/exynos/exynos_hdmi.c b/drivers/gpu/drm/exynos/exynos_hdmi.c index 229b361..0294402 100644 --- a/drivers/gpu/drm/exynos/exynos_hdmi.c +++ b/drivers/gpu/drm/exynos/exynos_hdmi.c @@ -2274,7 +2274,7 @@ err_data: return NULL; } -static struct of_device_id hdmi_match_types[] = { +static const struct of_device_id hdmi_match_types[] = { { .compatible = "samsung,exynos5-hdmi", .data = &exynos5_hdmi_driver_data, diff --git a/drivers/gpu/drm/exynos/exynos_mixer.c b/drivers/gpu/drm/exynos/exynos_mixer.c index 3518bc4..6297105 100644 --- a/drivers/gpu/drm/exynos/exynos_mixer.c +++ b/drivers/gpu/drm/exynos/exynos_mixer.c @@ -1224,7 +1224,7 @@ static struct platform_device_id mixer_driver_types[] = { } }; -static struct of_device_id mixer_match_types[] = { +static const struct of_device_id mixer_match_types[] = { { .compatible = "samsung,exynos4210-mixer", .data = &exynos4210_mxr_drv_data, diff --git a/drivers/gpu/drm/panel/panel-ld9040.c b/drivers/gpu/drm/panel/panel-ld9040.c index 08cf2c5..0ab5b69 100644 --- a/drivers/gpu/drm/panel/panel-ld9040.c +++ b/drivers/gpu/drm/panel/panel-ld9040.c @@ -367,7 +367,7 @@ static int ld9040_remove(struct spi_device *spi) return 0; } -static struct of_device_id ld9040_of_match[] = { +static const struct of_device_id ld9040_of_match[] = { { .compatible = "samsung,ld9040" }, { } }; diff --git a/drivers/gpu/drm/panel/panel-s6e8aa0.c b/drivers/gpu/drm/panel/panel-s6e8aa0.c index 144b273..3005110 100644 --- a/drivers/gpu/drm/panel/panel-s6e8aa0.c +++ b/drivers/gpu/drm/panel/panel-s6e8aa0.c @@ -1041,7 +1041,7 @@ static int s6e8aa0_remove(struct mipi_dsi_device *dsi) return 0; } -static struct of_device_id s6e8aa0_of_match[] = { +static const struct of_device_id s6e8aa0_of_match[] = { { .compatible = "samsung,s6e8aa0" }, { } }; diff --git a/drivers/gpu/drm/sti/sti_dvo.c b/drivers/gpu/drm/sti/sti_dvo.c index aeb5070..f619ae2 100644 --- a/drivers/gpu/drm/sti/sti_dvo.c +++ b/drivers/gpu/drm/sti/sti_dvo.c @@ -537,7 +537,7 @@ static int sti_dvo_remove(struct platform_device *pdev) return 0; } -static struct of_device_id dvo_of_match[] = { +static const struct of_device_id dvo_of_match[] = { { .compatible = "st,stih407-dvo", }, { /* end node */ } }; diff --git a/drivers/gpu/drm/sti/sti_hqvdp.c b/drivers/gpu/drm/sti/sti_hqvdp.c index b0eb62d..d4324f1 100644 --- a/drivers/gpu/drm/sti/sti_hqvdp.c +++ b/drivers/gpu/drm/sti/sti_hqvdp.c @@ -1050,7 +1050,7 @@ static int sti_hqvdp_remove(struct platform_device *pdev) return 0; } -static struct of_device_id hqvdp_of_match[] = { +static const struct of_device_id hqvdp_of_match[] = { { .compatible = "st,stih407-hqvdp", }, { /* end node */ } }; diff --git a/drivers/gpu/drm/tilcdc/tilcdc_drv.c b/drivers/gpu/drm/tilcdc/tilcdc_drv.c index 095fca9..5f7e602 100644 --- a/drivers/gpu/drm/tilcdc/tilcdc_drv.c +++ b/drivers/gpu/drm/tilcdc/tilcdc_drv.c @@ -47,7 +47,7 @@ void tilcdc_slave_probedefer(bool defered) slave_probing = defered; } -static struct of_device_id tilcdc_of_match[]; +static const struct of_device_id tilcdc_of_match[]; static struct drm_framebuffer *tilcdc_fb_create(struct drm_device *dev, struct drm_file *file_priv, struct drm_mode_fb_cmd2 *mode_cmd) @@ -634,7 +634,7 @@ static int tilcdc_pdev_remove(struct platform_device *pdev) return 0; } -static struct of_device_id tilcdc_of_match[] = { +static const struct of_device_id tilcdc_of_match[] = { { .compatible = "ti,am33xx-tilcdc", }, { }, }; diff --git a/drivers/gpu/drm/tilcdc/tilcdc_panel.c b/drivers/gpu/drm/tilcdc/tilcdc_panel.c index 7a03158..8a374c7 100644 --- a/drivers/gpu/drm/tilcdc/tilcdc_panel.c +++ b/drivers/gpu/drm/tilcdc/tilcdc_panel.c @@ -450,7 +450,7 @@ static int panel_remove(struct platform_device *pdev) return 0; } -static struct of_device_id panel_of_match[] = { +static const struct of_device_id panel_of_match[] = { { .compatible = "ti,tilcdc,panel", }, { }, }; diff --git a/drivers/gpu/drm/tilcdc/tilcdc_slave.c b/drivers/gpu/drm/tilcdc/tilcdc_slave.c index 3775fd4..1cab937 100644 --- a/drivers/gpu/drm/tilcdc/tilcdc_slave.c +++ b/drivers/gpu/drm/tilcdc/tilcdc_slave.c @@ -304,7 +304,7 @@ static const struct tilcdc_module_ops slave_module_ops = { * Device: */ -static struct of_device_id slave_of_match[]; +static const struct of_device_id slave_of_match[]; static int slave_probe(struct platform_device *pdev) { @@ -385,7 +385,7 @@ static int slave_remove(struct platform_device *pdev) return 0; } -static struct of_device_id slave_of_match[] = { +static const struct of_device_id slave_of_match[] = { { .compatible = "ti,tilcdc,slave", }, { }, }; diff --git a/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c b/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c index 354c47c..1bd8e81 100644 --- a/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c +++ b/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c @@ -304,7 +304,7 @@ static const struct tilcdc_module_ops tfp410_module_ops = { * Device: */ -static struct of_device_id tfp410_of_match[]; +static const struct of_device_id tfp410_of_match[]; static int tfp410_probe(struct platform_device *pdev) { @@ -394,7 +394,7 @@ static int tfp410_remove(struct platform_device *pdev) return 0; } -static struct of_device_id tfp410_of_match[] = { +static const struct of_device_id tfp410_of_match[] = { { .compatible = "ti,tilcdc,tfp410", }, { }, }; diff --git a/drivers/gpu/host1x/dev.c b/drivers/gpu/host1x/dev.c index 53d3d1d..f33fd77 100644 --- a/drivers/gpu/host1x/dev.c +++ b/drivers/gpu/host1x/dev.c @@ -87,7 +87,7 @@ static const struct host1x_info host1x04_info = { .sync_offset = 0x2100, }; -static struct of_device_id host1x_of_match[] = { +static const struct of_device_id host1x_of_match[] = { { .compatible = "nvidia,tegra124-host1x", .data = &host1x04_info, }, { .compatible = "nvidia,tegra114-host1x", .data = &host1x02_info, }, { .compatible = "nvidia,tegra30-host1x", .data = &host1x01_info, }, diff --git a/drivers/gpu/host1x/mipi.c b/drivers/gpu/host1x/mipi.c index fbc6ee6..ba54db4 100644 --- a/drivers/gpu/host1x/mipi.c +++ b/drivers/gpu/host1x/mipi.c @@ -277,7 +277,7 @@ static const struct tegra_mipi_soc tegra124_mipi_soc = { .num_pads = ARRAY_SIZE(tegra124_mipi_pads), }; -static struct of_device_id tegra_mipi_of_match[] = { +static const struct of_device_id tegra_mipi_of_match[] = { { .compatible = "nvidia,tegra114-mipi", .data = &tegra114_mipi_soc }, { .compatible = "nvidia,tegra124-mipi", .data = &tegra124_mipi_soc }, { }, -- 2.1.0 ^ permalink raw reply related [flat|nested] 49+ messages in thread
* [PATCH 30/35 linux-next] devfreq: constify of_device_id array 2015-03-16 19:54 ` [PATCH 22/35 linux-next] soc: ti: " Fabian Frederick ` (4 preceding siblings ...) 2015-03-16 19:54 ` [PATCH 29/35 linux-next] gpu: " Fabian Frederick @ 2015-03-16 19:54 ` Fabian Frederick 2015-03-16 19:54 ` [PATCH 31/35 linux-next] EDAC: " Fabian Frederick 6 siblings, 0 replies; 49+ messages in thread From: Fabian Frederick @ 2015-03-16 19:54 UTC (permalink / raw) To: linux-arm-kernel of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick <fabf@skynet.be> --- drivers/devfreq/event/exynos-ppmu.c | 2 +- drivers/devfreq/tegra-devfreq.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/devfreq/event/exynos-ppmu.c b/drivers/devfreq/event/exynos-ppmu.c index ad83473..5afb851 100644 --- a/drivers/devfreq/event/exynos-ppmu.c +++ b/drivers/devfreq/event/exynos-ppmu.c @@ -354,7 +354,7 @@ static int exynos_ppmu_remove(struct platform_device *pdev) return 0; } -static struct of_device_id exynos_ppmu_id_match[] = { +static const struct of_device_id exynos_ppmu_id_match[] = { { .compatible = "samsung,exynos-ppmu", }, { /* sentinel */ }, }; diff --git a/drivers/devfreq/tegra-devfreq.c b/drivers/devfreq/tegra-devfreq.c index 3479096..244d8db 100644 --- a/drivers/devfreq/tegra-devfreq.c +++ b/drivers/devfreq/tegra-devfreq.c @@ -695,7 +695,7 @@ static SIMPLE_DEV_PM_OPS(tegra_devfreq_pm_ops, tegra_devfreq_suspend, tegra_devfreq_resume); -static struct of_device_id tegra_devfreq_of_match[] = { +static const struct of_device_id tegra_devfreq_of_match[] = { { .compatible = "nvidia,tegra124-actmon" }, { }, }; -- 2.1.0 ^ permalink raw reply related [flat|nested] 49+ messages in thread
* [PATCH 31/35 linux-next] EDAC: constify of_device_id array 2015-03-16 19:54 ` [PATCH 22/35 linux-next] soc: ti: " Fabian Frederick ` (5 preceding siblings ...) 2015-03-16 19:54 ` [PATCH 30/35 linux-next] devfreq: " Fabian Frederick @ 2015-03-16 19:54 ` Fabian Frederick 2015-03-20 16:57 ` Borislav Petkov 6 siblings, 1 reply; 49+ messages in thread From: Fabian Frederick @ 2015-03-16 19:54 UTC (permalink / raw) To: linux-arm-kernel of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick <fabf@skynet.be> --- drivers/edac/highbank_mc_edac.c | 2 +- drivers/edac/mpc85xx_edac.c | 4 ++-- drivers/edac/ppc4xx_edac.c | 2 +- drivers/edac/synopsys_edac.c | 2 +- 4 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/edac/highbank_mc_edac.c b/drivers/edac/highbank_mc_edac.c index 4754847..11260cc 100644 --- a/drivers/edac/highbank_mc_edac.c +++ b/drivers/edac/highbank_mc_edac.c @@ -146,7 +146,7 @@ static struct hb_mc_settings mw_settings = { .int_offset = MW_DDR_ECC_INT_BASE, }; -static struct of_device_id hb_ddr_ctrl_of_match[] = { +static const struct of_device_id hb_ddr_ctrl_of_match[] = { { .compatible = "calxeda,hb-ddr-ctrl", .data = &hb_settings }, { .compatible = "calxeda,ecx-2000-ddr-ctrl", .data = &mw_settings }, {}, diff --git a/drivers/edac/mpc85xx_edac.c b/drivers/edac/mpc85xx_edac.c index 7be979c..68bf234 100644 --- a/drivers/edac/mpc85xx_edac.c +++ b/drivers/edac/mpc85xx_edac.c @@ -670,7 +670,7 @@ static int mpc85xx_l2_err_remove(struct platform_device *op) return 0; } -static struct of_device_id mpc85xx_l2_err_of_match[] = { +static const struct of_device_id mpc85xx_l2_err_of_match[] = { /* deprecate the fsl,85.. forms in the future, 2.6.30? */ { .compatible = "fsl,8540-l2-cache-controller", }, { .compatible = "fsl,8541-l2-cache-controller", }, @@ -1160,7 +1160,7 @@ static int mpc85xx_mc_err_remove(struct platform_device *op) return 0; } -static struct of_device_id mpc85xx_mc_err_of_match[] = { +static const struct of_device_id mpc85xx_mc_err_of_match[] = { /* deprecate the fsl,85.. forms in the future, 2.6.30? */ { .compatible = "fsl,8540-memory-controller", }, { .compatible = "fsl,8541-memory-controller", }, diff --git a/drivers/edac/ppc4xx_edac.c b/drivers/edac/ppc4xx_edac.c index 1b64fd0..3515b38 100644 --- a/drivers/edac/ppc4xx_edac.c +++ b/drivers/edac/ppc4xx_edac.c @@ -193,7 +193,7 @@ static int ppc4xx_edac_remove(struct platform_device *device); * Device tree node type and compatible tuples this driver can match * on. */ -static struct of_device_id ppc4xx_edac_match[] = { +static const struct of_device_id ppc4xx_edac_match[] = { { .compatible = "ibm,sdram-4xx-ddr2" }, diff --git a/drivers/edac/synopsys_edac.c b/drivers/edac/synopsys_edac.c index 1c96915..fc153ae 100644 --- a/drivers/edac/synopsys_edac.c +++ b/drivers/edac/synopsys_edac.c @@ -512,7 +512,7 @@ static int synps_edac_mc_remove(struct platform_device *pdev) return 0; } -static struct of_device_id synps_edac_match[] = { +static const struct of_device_id synps_edac_match[] = { { .compatible = "xlnx,zynq-ddrc-a05", }, { /* end of table */ } }; -- 2.1.0 ^ permalink raw reply related [flat|nested] 49+ messages in thread
* [PATCH 31/35 linux-next] EDAC: constify of_device_id array 2015-03-16 19:54 ` [PATCH 31/35 linux-next] EDAC: " Fabian Frederick @ 2015-03-20 16:57 ` Borislav Petkov 0 siblings, 0 replies; 49+ messages in thread From: Borislav Petkov @ 2015-03-20 16:57 UTC (permalink / raw) To: linux-arm-kernel On Mon, Mar 16, 2015 at 08:54:41PM +0100, Fabian Frederick wrote: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > > Signed-off-by: Fabian Frederick <fabf@skynet.be> > --- > drivers/edac/highbank_mc_edac.c | 2 +- > drivers/edac/mpc85xx_edac.c | 4 ++-- > drivers/edac/ppc4xx_edac.c | 2 +- > drivers/edac/synopsys_edac.c | 2 +- > 4 files changed, 5 insertions(+), 5 deletions(-) Applied, thanks. -- Regards/Gruss, Boris. ECO tip #101: Trim your mails when you reply. -- ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 32/35 linux-next] clk: constify of_device_id array 2015-03-16 19:17 [PATCH 00/35 linux-next] constify of_device_id array Fabian Frederick ` (4 preceding siblings ...) 2015-03-16 19:54 ` [PATCH 22/35 linux-next] soc: ti: " Fabian Frederick @ 2015-03-16 19:59 ` Fabian Frederick 2015-03-16 19:59 ` [PATCH 33/35 linux-next] mmc: " Fabian Frederick ` (2 more replies) 5 siblings, 3 replies; 49+ messages in thread From: Fabian Frederick @ 2015-03-16 19:59 UTC (permalink / raw) To: linux-arm-kernel of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick <fabf@skynet.be> --- drivers/clk/clk-palmas.c | 2 +- drivers/clk/st/clkgen-fsyn.c | 2 +- drivers/clk/st/clkgen-mux.c | 8 ++++---- drivers/clk/st/clkgen-pll.c | 4 ++-- drivers/clk/ti/clk-dra7-atl.c | 2 +- drivers/clk/ti/clockdomain.c | 2 +- drivers/clk/versatile/clk-vexpress-osc.c | 2 +- 7 files changed, 11 insertions(+), 11 deletions(-) diff --git a/drivers/clk/clk-palmas.c b/drivers/clk/clk-palmas.c index 8d45992..45a535a 100644 --- a/drivers/clk/clk-palmas.c +++ b/drivers/clk/clk-palmas.c @@ -161,7 +161,7 @@ static struct palmas_clks_of_match_data palmas_of_clk32kgaudio = { }, }; -static struct of_device_id palmas_clks_of_match[] = { +static const struct of_device_id palmas_clks_of_match[] = { { .compatible = "ti,palmas-clk32kg", .data = &palmas_of_clk32kg, diff --git a/drivers/clk/st/clkgen-fsyn.c b/drivers/clk/st/clkgen-fsyn.c index af94ed8..a917c4c 100644 --- a/drivers/clk/st/clkgen-fsyn.c +++ b/drivers/clk/st/clkgen-fsyn.c @@ -1057,7 +1057,7 @@ static struct clk * __init st_clk_register_quadfs_fsynth( return clk; } -static struct of_device_id quadfs_of_match[] = { +static const struct of_device_id quadfs_of_match[] = { { .compatible = "st,stih416-quadfs216", .data = &st_fs216c65_416 diff --git a/drivers/clk/st/clkgen-mux.c b/drivers/clk/st/clkgen-mux.c index 9a15ec3..fdcff10 100644 --- a/drivers/clk/st/clkgen-mux.c +++ b/drivers/clk/st/clkgen-mux.c @@ -341,7 +341,7 @@ static struct clkgena_divmux_data st_divmux_c32odf3 = { .fb_start_bit_idx = 24, }; -static struct of_device_id clkgena_divmux_of_match[] = { +static const struct of_device_id clkgena_divmux_of_match[] = { { .compatible = "st,clkgena-divmux-c65-hs", .data = &st_divmux_c65hs, @@ -479,7 +479,7 @@ static struct clkgena_prediv_data prediv_c32_data = { .table = prediv_table16, }; -static struct of_device_id clkgena_prediv_of_match[] = { +static const struct of_device_id clkgena_prediv_of_match[] = { { .compatible = "st,clkgena-prediv-c65", .data = &prediv_c65_data }, { .compatible = "st,clkgena-prediv-c32", .data = &prediv_c32_data }, {} @@ -586,7 +586,7 @@ static struct clkgen_mux_data stih407_a9_mux_data = { .width = 2, }; -static struct of_device_id mux_of_match[] = { +static const struct of_device_id mux_of_match[] = { { .compatible = "st,stih416-clkgenc-vcc-hd", .data = &clkgen_mux_c_vcc_hd_416, @@ -693,7 +693,7 @@ static struct clkgen_vcc_data st_clkgenf_vcc_416 = { .lock = &clkgenf_lock, }; -static struct of_device_id vcc_of_match[] = { +static const struct of_device_id vcc_of_match[] = { { .compatible = "st,stih416-clkgenc", .data = &st_clkgenc_vcc_416 }, { .compatible = "st,stih416-clkgenf", .data = &st_clkgenf_vcc_416 }, {} diff --git a/drivers/clk/st/clkgen-pll.c b/drivers/clk/st/clkgen-pll.c index 29769d7..d204ba8 100644 --- a/drivers/clk/st/clkgen-pll.c +++ b/drivers/clk/st/clkgen-pll.c @@ -593,7 +593,7 @@ static struct clk * __init clkgen_odf_register(const char *parent_name, return clk; } -static struct of_device_id c32_pll_of_match[] = { +static const struct of_device_id c32_pll_of_match[] = { { .compatible = "st,plls-c32-a1x-0", .data = &st_pll3200c32_a1x_0, @@ -708,7 +708,7 @@ err: } CLK_OF_DECLARE(clkgen_c32_pll, "st,clkgen-plls-c32", clkgen_c32_pll_setup); -static struct of_device_id c32_gpu_pll_of_match[] = { +static const struct of_device_id c32_gpu_pll_of_match[] = { { .compatible = "st,stih415-gpu-pll-c32", .data = &st_pll1200c32_gpu_415, diff --git a/drivers/clk/ti/clk-dra7-atl.c b/drivers/clk/ti/clk-dra7-atl.c index 59bb4b3..d86bc46 100644 --- a/drivers/clk/ti/clk-dra7-atl.c +++ b/drivers/clk/ti/clk-dra7-atl.c @@ -294,7 +294,7 @@ static int of_dra7_atl_clk_remove(struct platform_device *pdev) return 0; } -static struct of_device_id of_dra7_atl_clk_match_tbl[] = { +static const struct of_device_id of_dra7_atl_clk_match_tbl[] = { { .compatible = "ti,dra7-atl", }, {}, }; diff --git a/drivers/clk/ti/clockdomain.c b/drivers/clk/ti/clockdomain.c index b4c5fac..35fe108 100644 --- a/drivers/clk/ti/clockdomain.c +++ b/drivers/clk/ti/clockdomain.c @@ -52,7 +52,7 @@ static void __init of_ti_clockdomain_setup(struct device_node *node) } } -static struct of_device_id ti_clkdm_match_table[] __initdata = { +static const struct of_device_id ti_clkdm_match_table[] __initconst = { { .compatible = "ti,clockdomain" }, { } }; diff --git a/drivers/clk/versatile/clk-vexpress-osc.c b/drivers/clk/versatile/clk-vexpress-osc.c index 765f1e0..89c0609 100644 --- a/drivers/clk/versatile/clk-vexpress-osc.c +++ b/drivers/clk/versatile/clk-vexpress-osc.c @@ -110,7 +110,7 @@ static int vexpress_osc_probe(struct platform_device *pdev) return 0; } -static struct of_device_id vexpress_osc_of_match[] = { +static const struct of_device_id vexpress_osc_of_match[] = { { .compatible = "arm,vexpress-osc", }, {} }; -- 2.1.0 ^ permalink raw reply related [flat|nested] 49+ messages in thread
* [PATCH 33/35 linux-next] mmc: constify of_device_id array 2015-03-16 19:59 ` [PATCH 32/35 linux-next] clk: " Fabian Frederick @ 2015-03-16 19:59 ` Fabian Frederick 2015-03-17 9:49 ` Ulf Hansson 2015-03-16 19:59 ` [PATCH 35/35 linux-next] pinctrl: " Fabian Frederick 2015-03-18 14:15 ` [PATCH 32/35 linux-next] clk: " Michael Turquette 2 siblings, 1 reply; 49+ messages in thread From: Fabian Frederick @ 2015-03-16 19:59 UTC (permalink / raw) To: linux-arm-kernel of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick <fabf@skynet.be> --- drivers/mmc/host/mmc_spi.c | 2 +- drivers/mmc/host/wmt-sdmmc.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/host/mmc_spi.c b/drivers/mmc/host/mmc_spi.c index e4a0754..ae19d83 100644 --- a/drivers/mmc/host/mmc_spi.c +++ b/drivers/mmc/host/mmc_spi.c @@ -1507,7 +1507,7 @@ static int mmc_spi_remove(struct spi_device *spi) return 0; } -static struct of_device_id mmc_spi_of_match_table[] = { +static const struct of_device_id mmc_spi_of_match_table[] = { { .compatible = "mmc-spi-slot", }, {}, }; diff --git a/drivers/mmc/host/wmt-sdmmc.c b/drivers/mmc/host/wmt-sdmmc.c index dd2e1aa..5af0055 100644 --- a/drivers/mmc/host/wmt-sdmmc.c +++ b/drivers/mmc/host/wmt-sdmmc.c @@ -744,7 +744,7 @@ static struct wmt_mci_caps wm8505_caps = { .max_blk_size = 2048, }; -static struct of_device_id wmt_mci_dt_ids[] = { +static const struct of_device_id wmt_mci_dt_ids[] = { { .compatible = "wm,wm8505-sdhc", .data = &wm8505_caps }, { /* Sentinel */ }, }; -- 2.1.0 ^ permalink raw reply related [flat|nested] 49+ messages in thread
* [PATCH 33/35 linux-next] mmc: constify of_device_id array 2015-03-16 19:59 ` [PATCH 33/35 linux-next] mmc: " Fabian Frederick @ 2015-03-17 9:49 ` Ulf Hansson 2015-03-20 8:56 ` Ulf Hansson 0 siblings, 1 reply; 49+ messages in thread From: Ulf Hansson @ 2015-03-17 9:49 UTC (permalink / raw) To: linux-arm-kernel On 16 March 2015 at 20:59, Fabian Frederick <fabf@skynet.be> wrote: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > > Signed-off-by: Fabian Frederick <fabf@skynet.be> Acked-by: Ulf Hansson <ulf.hansson@linaro.org> > --- > drivers/mmc/host/mmc_spi.c | 2 +- > drivers/mmc/host/wmt-sdmmc.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/mmc/host/mmc_spi.c b/drivers/mmc/host/mmc_spi.c > index e4a0754..ae19d83 100644 > --- a/drivers/mmc/host/mmc_spi.c > +++ b/drivers/mmc/host/mmc_spi.c > @@ -1507,7 +1507,7 @@ static int mmc_spi_remove(struct spi_device *spi) > return 0; > } > > -static struct of_device_id mmc_spi_of_match_table[] = { > +static const struct of_device_id mmc_spi_of_match_table[] = { > { .compatible = "mmc-spi-slot", }, > {}, > }; > diff --git a/drivers/mmc/host/wmt-sdmmc.c b/drivers/mmc/host/wmt-sdmmc.c > index dd2e1aa..5af0055 100644 > --- a/drivers/mmc/host/wmt-sdmmc.c > +++ b/drivers/mmc/host/wmt-sdmmc.c > @@ -744,7 +744,7 @@ static struct wmt_mci_caps wm8505_caps = { > .max_blk_size = 2048, > }; > > -static struct of_device_id wmt_mci_dt_ids[] = { > +static const struct of_device_id wmt_mci_dt_ids[] = { > { .compatible = "wm,wm8505-sdhc", .data = &wm8505_caps }, > { /* Sentinel */ }, > }; > -- > 2.1.0 > ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 33/35 linux-next] mmc: constify of_device_id array 2015-03-17 9:49 ` Ulf Hansson @ 2015-03-20 8:56 ` Ulf Hansson 2015-03-21 20:09 ` Fabian Frederick 0 siblings, 1 reply; 49+ messages in thread From: Ulf Hansson @ 2015-03-20 8:56 UTC (permalink / raw) To: linux-arm-kernel On 17 March 2015 at 10:49, Ulf Hansson <ulf.hansson@linaro.org> wrote: > On 16 March 2015 at 20:59, Fabian Frederick <fabf@skynet.be> wrote: >> of_device_id is always used as const. >> (See driver.of_match_table and open firmware functions) >> >> Signed-off-by: Fabian Frederick <fabf@skynet.be> > > Acked-by: Ulf Hansson <ulf.hansson@linaro.org> I realized that you might wanted me to pick this up and queue it through my mmc tree? Just tell me if that's what you want. Kind regards Uffe > >> --- >> drivers/mmc/host/mmc_spi.c | 2 +- >> drivers/mmc/host/wmt-sdmmc.c | 2 +- >> 2 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/mmc/host/mmc_spi.c b/drivers/mmc/host/mmc_spi.c >> index e4a0754..ae19d83 100644 >> --- a/drivers/mmc/host/mmc_spi.c >> +++ b/drivers/mmc/host/mmc_spi.c >> @@ -1507,7 +1507,7 @@ static int mmc_spi_remove(struct spi_device *spi) >> return 0; >> } >> >> -static struct of_device_id mmc_spi_of_match_table[] = { >> +static const struct of_device_id mmc_spi_of_match_table[] = { >> { .compatible = "mmc-spi-slot", }, >> {}, >> }; >> diff --git a/drivers/mmc/host/wmt-sdmmc.c b/drivers/mmc/host/wmt-sdmmc.c >> index dd2e1aa..5af0055 100644 >> --- a/drivers/mmc/host/wmt-sdmmc.c >> +++ b/drivers/mmc/host/wmt-sdmmc.c >> @@ -744,7 +744,7 @@ static struct wmt_mci_caps wm8505_caps = { >> .max_blk_size = 2048, >> }; >> >> -static struct of_device_id wmt_mci_dt_ids[] = { >> +static const struct of_device_id wmt_mci_dt_ids[] = { >> { .compatible = "wm,wm8505-sdhc", .data = &wm8505_caps }, >> { /* Sentinel */ }, >> }; >> -- >> 2.1.0 >> ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 33/35 linux-next] mmc: constify of_device_id array 2015-03-20 8:56 ` Ulf Hansson @ 2015-03-21 20:09 ` Fabian Frederick 2015-03-23 10:06 ` Ulf Hansson 0 siblings, 1 reply; 49+ messages in thread From: Fabian Frederick @ 2015-03-21 20:09 UTC (permalink / raw) To: linux-arm-kernel > On 20 March 2015 at 09:56 Ulf Hansson <ulf.hansson@linaro.org> wrote: > > > On 17 March 2015 at 10:49, Ulf Hansson <ulf.hansson@linaro.org> wrote: > > On 16 March 2015 at 20:59, Fabian Frederick <fabf@skynet.be> wrote: > >> of_device_id is always used as const. > >> (See driver.of_match_table and open firmware functions) > >> > >> Signed-off-by: Fabian Frederick <fabf@skynet.be> > > > > Acked-by: Ulf Hansson <ulf.hansson@linaro.org> > > I realized that you might wanted me to pick this up and queue it > through my mmc tree? Just tell me if that's what you want. > > Kind regards > Uffe Hi Ulf, ? ? ? ? Yes of course :) (it has not been added to linux-next yet). Thanks, Fabian > > > > >> --- > >>? drivers/mmc/host/mmc_spi.c? ?| 2 +- > >>? drivers/mmc/host/wmt-sdmmc.c | 2 +- > >>? 2 files changed, 2 insertions(+), 2 deletions(-) > >> > >> diff --git a/drivers/mmc/host/mmc_spi.c b/drivers/mmc/host/mmc_spi.c > >> index e4a0754..ae19d83 100644 > >> --- a/drivers/mmc/host/mmc_spi.c > >> +++ b/drivers/mmc/host/mmc_spi.c > >> @@ -1507,7 +1507,7 @@ static int mmc_spi_remove(struct spi_device *spi) > >>? ? ? ? ?return 0; > >>? } > >> > >> -static struct of_device_id mmc_spi_of_match_table[] = { > >> +static const struct of_device_id mmc_spi_of_match_table[] = { > >>? ? ? ? ?{ .compatible = "mmc-spi-slot", }, > >>? ? ? ? ?{}, > >>? }; > >> diff --git a/drivers/mmc/host/wmt-sdmmc.c b/drivers/mmc/host/wmt-sdmmc.c > >> index dd2e1aa..5af0055 100644 > >> --- a/drivers/mmc/host/wmt-sdmmc.c > >> +++ b/drivers/mmc/host/wmt-sdmmc.c > >> @@ -744,7 +744,7 @@ static struct wmt_mci_caps wm8505_caps = { > >>? ? ? ? ?.max_blk_size = 2048, > >>? }; > >> > >> -static struct of_device_id wmt_mci_dt_ids[] = { > >> +static const struct of_device_id wmt_mci_dt_ids[] = { > >>? ? ? ? ?{ .compatible = "wm,wm8505-sdhc", .data = &wm8505_caps }, > >>? ? ? ? ?{ /* Sentinel */ }, > >>? }; > >> -- > >> 2.1.0 > >> ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 33/35 linux-next] mmc: constify of_device_id array 2015-03-21 20:09 ` Fabian Frederick @ 2015-03-23 10:06 ` Ulf Hansson 0 siblings, 0 replies; 49+ messages in thread From: Ulf Hansson @ 2015-03-23 10:06 UTC (permalink / raw) To: linux-arm-kernel On 21 March 2015 at 21:09, Fabian Frederick <fabf@skynet.be> wrote: > > >> On 20 March 2015 at 09:56 Ulf Hansson <ulf.hansson@linaro.org> wrote: >> >> >> On 17 March 2015 at 10:49, Ulf Hansson <ulf.hansson@linaro.org> wrote: >> > On 16 March 2015 at 20:59, Fabian Frederick <fabf@skynet.be> wrote: >> >> of_device_id is always used as const. >> >> (See driver.of_match_table and open firmware functions) >> >> >> >> Signed-off-by: Fabian Frederick <fabf@skynet.be> >> > >> > Acked-by: Ulf Hansson <ulf.hansson@linaro.org> >> >> I realized that you might wanted me to pick this up and queue it >> through my mmc tree? Just tell me if that's what you want. >> >> Kind regards >> Uffe > > Hi Ulf, > > Yes of course :) (it has not been added to linux-next yet). Thanks! Applied. Kind regards Uffe > > Thanks, > Fabian >> >> > >> >> --- >> >> drivers/mmc/host/mmc_spi.c | 2 +- >> >> drivers/mmc/host/wmt-sdmmc.c | 2 +- >> >> 2 files changed, 2 insertions(+), 2 deletions(-) >> >> >> >> diff --git a/drivers/mmc/host/mmc_spi.c b/drivers/mmc/host/mmc_spi.c >> >> index e4a0754..ae19d83 100644 >> >> --- a/drivers/mmc/host/mmc_spi.c >> >> +++ b/drivers/mmc/host/mmc_spi.c >> >> @@ -1507,7 +1507,7 @@ static int mmc_spi_remove(struct spi_device *spi) >> >> return 0; >> >> } >> >> >> >> -static struct of_device_id mmc_spi_of_match_table[] = { >> >> +static const struct of_device_id mmc_spi_of_match_table[] = { >> >> { .compatible = "mmc-spi-slot", }, >> >> {}, >> >> }; >> >> diff --git a/drivers/mmc/host/wmt-sdmmc.c b/drivers/mmc/host/wmt-sdmmc.c >> >> index dd2e1aa..5af0055 100644 >> >> --- a/drivers/mmc/host/wmt-sdmmc.c >> >> +++ b/drivers/mmc/host/wmt-sdmmc.c >> >> @@ -744,7 +744,7 @@ static struct wmt_mci_caps wm8505_caps = { >> >> .max_blk_size = 2048, >> >> }; >> >> >> >> -static struct of_device_id wmt_mci_dt_ids[] = { >> >> +static const struct of_device_id wmt_mci_dt_ids[] = { >> >> { .compatible = "wm,wm8505-sdhc", .data = &wm8505_caps }, >> >> { /* Sentinel */ }, >> >> }; >> >> -- >> >> 2.1.0 >> >> ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 35/35 linux-next] pinctrl: constify of_device_id array 2015-03-16 19:59 ` [PATCH 32/35 linux-next] clk: " Fabian Frederick 2015-03-16 19:59 ` [PATCH 33/35 linux-next] mmc: " Fabian Frederick @ 2015-03-16 19:59 ` Fabian Frederick 2015-03-17 4:03 ` Jean-Christophe PLAGNIOL-VILLARD ` (5 more replies) 2015-03-18 14:15 ` [PATCH 32/35 linux-next] clk: " Michael Turquette 2 siblings, 6 replies; 49+ messages in thread From: Fabian Frederick @ 2015-03-16 19:59 UTC (permalink / raw) To: linux-arm-kernel of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick <fabf@skynet.be> --- drivers/pinctrl/bcm/pinctrl-bcm2835.c | 2 +- drivers/pinctrl/mediatek/pinctrl-mt8135.c | 2 +- drivers/pinctrl/mediatek/pinctrl-mt8173.c | 2 +- drivers/pinctrl/mvebu/pinctrl-armada-370.c | 2 +- drivers/pinctrl/mvebu/pinctrl-armada-375.c | 2 +- drivers/pinctrl/mvebu/pinctrl-armada-38x.c | 2 +- drivers/pinctrl/mvebu/pinctrl-armada-39x.c | 2 +- drivers/pinctrl/mvebu/pinctrl-armada-xp.c | 2 +- drivers/pinctrl/mvebu/pinctrl-kirkwood.c | 2 +- drivers/pinctrl/mvebu/pinctrl-orion.c | 2 +- drivers/pinctrl/pinctrl-as3722.c | 2 +- drivers/pinctrl/pinctrl-at91.c | 4 ++-- drivers/pinctrl/pinctrl-palmas.c | 2 +- drivers/pinctrl/pinctrl-single.c | 4 ++-- drivers/pinctrl/pinctrl-st.c | 2 +- drivers/pinctrl/pinctrl-tz1090-pdc.c | 2 +- drivers/pinctrl/pinctrl-tz1090.c | 2 +- drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c | 2 +- drivers/pinctrl/sunxi/pinctrl-sun5i-a10s.c | 2 +- drivers/pinctrl/sunxi/pinctrl-sun5i-a13.c | 2 +- drivers/pinctrl/sunxi/pinctrl-sun6i-a31-r.c | 2 +- drivers/pinctrl/sunxi/pinctrl-sun6i-a31.c | 2 +- drivers/pinctrl/sunxi/pinctrl-sun6i-a31s.c | 2 +- drivers/pinctrl/sunxi/pinctrl-sun7i-a20.c | 2 +- drivers/pinctrl/sunxi/pinctrl-sun8i-a23-r.c | 2 +- drivers/pinctrl/sunxi/pinctrl-sun8i-a23.c | 2 +- drivers/pinctrl/sunxi/pinctrl-sun9i-a80.c | 2 +- drivers/pinctrl/vt8500/pinctrl-vt8500.c | 2 +- drivers/pinctrl/vt8500/pinctrl-wm8505.c | 2 +- drivers/pinctrl/vt8500/pinctrl-wm8650.c | 2 +- drivers/pinctrl/vt8500/pinctrl-wm8750.c | 2 +- drivers/pinctrl/vt8500/pinctrl-wm8850.c | 2 +- 32 files changed, 34 insertions(+), 34 deletions(-) diff --git a/drivers/pinctrl/bcm/pinctrl-bcm2835.c b/drivers/pinctrl/bcm/pinctrl-bcm2835.c index 9aa8a3f..4d08b85 100644 --- a/drivers/pinctrl/bcm/pinctrl-bcm2835.c +++ b/drivers/pinctrl/bcm/pinctrl-bcm2835.c @@ -1051,7 +1051,7 @@ static int bcm2835_pinctrl_remove(struct platform_device *pdev) return 0; } -static struct of_device_id bcm2835_pinctrl_match[] = { +static const struct of_device_id bcm2835_pinctrl_match[] = { { .compatible = "brcm,bcm2835-gpio" }, {} }; diff --git a/drivers/pinctrl/mediatek/pinctrl-mt8135.c b/drivers/pinctrl/mediatek/pinctrl-mt8135.c index 1296d6d..82c4af4 100644 --- a/drivers/pinctrl/mediatek/pinctrl-mt8135.c +++ b/drivers/pinctrl/mediatek/pinctrl-mt8135.c @@ -347,7 +347,7 @@ static int mt8135_pinctrl_probe(struct platform_device *pdev) return mtk_pctrl_init(pdev, &mt8135_pinctrl_data); } -static struct of_device_id mt8135_pctrl_match[] = { +static const struct of_device_id mt8135_pctrl_match[] = { { .compatible = "mediatek,mt8135-pinctrl", }, { diff --git a/drivers/pinctrl/mediatek/pinctrl-mt8173.c b/drivers/pinctrl/mediatek/pinctrl-mt8173.c index f07cafb..594f7b5 100644 --- a/drivers/pinctrl/mediatek/pinctrl-mt8173.c +++ b/drivers/pinctrl/mediatek/pinctrl-mt8173.c @@ -427,7 +427,7 @@ static int mt8173_pinctrl_probe(struct platform_device *pdev) return mtk_pctrl_init(pdev, &mt8173_pinctrl_data); } -static struct of_device_id mt8173_pctrl_match[] = { +static const struct of_device_id mt8173_pctrl_match[] = { { .compatible = "mediatek,mt8173-pinctrl", }, { diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-370.c b/drivers/pinctrl/mvebu/pinctrl-armada-370.c index c4f51d0..42f930f 100644 --- a/drivers/pinctrl/mvebu/pinctrl-armada-370.c +++ b/drivers/pinctrl/mvebu/pinctrl-armada-370.c @@ -379,7 +379,7 @@ static struct mvebu_mpp_mode mv88f6710_mpp_modes[] = { static struct mvebu_pinctrl_soc_info armada_370_pinctrl_info; -static struct of_device_id armada_370_pinctrl_of_match[] = { +static const struct of_device_id armada_370_pinctrl_of_match[] = { { .compatible = "marvell,mv88f6710-pinctrl" }, { }, }; diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-375.c b/drivers/pinctrl/mvebu/pinctrl-armada-375.c index cd7c8f5..ca1e757 100644 --- a/drivers/pinctrl/mvebu/pinctrl-armada-375.c +++ b/drivers/pinctrl/mvebu/pinctrl-armada-375.c @@ -399,7 +399,7 @@ static struct mvebu_mpp_mode mv88f6720_mpp_modes[] = { static struct mvebu_pinctrl_soc_info armada_375_pinctrl_info; -static struct of_device_id armada_375_pinctrl_of_match[] = { +static const struct of_device_id armada_375_pinctrl_of_match[] = { { .compatible = "marvell,mv88f6720-pinctrl" }, { }, }; diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-38x.c b/drivers/pinctrl/mvebu/pinctrl-armada-38x.c index 7302f66..83bbcc7 100644 --- a/drivers/pinctrl/mvebu/pinctrl-armada-38x.c +++ b/drivers/pinctrl/mvebu/pinctrl-armada-38x.c @@ -389,7 +389,7 @@ static struct mvebu_mpp_mode armada_38x_mpp_modes[] = { static struct mvebu_pinctrl_soc_info armada_38x_pinctrl_info; -static struct of_device_id armada_38x_pinctrl_of_match[] = { +static const struct of_device_id armada_38x_pinctrl_of_match[] = { { .compatible = "marvell,mv88f6810-pinctrl", .data = (void *) V_88F6810, diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-39x.c b/drivers/pinctrl/mvebu/pinctrl-armada-39x.c index 5963411..4249162 100644 --- a/drivers/pinctrl/mvebu/pinctrl-armada-39x.c +++ b/drivers/pinctrl/mvebu/pinctrl-armada-39x.c @@ -362,7 +362,7 @@ static struct mvebu_mpp_mode armada_39x_mpp_modes[] = { static struct mvebu_pinctrl_soc_info armada_39x_pinctrl_info; -static struct of_device_id armada_39x_pinctrl_of_match[] = { +static const struct of_device_id armada_39x_pinctrl_of_match[] = { { .compatible = "marvell,mv88f6920-pinctrl", .data = (void *) V_88F6920, diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-xp.c b/drivers/pinctrl/mvebu/pinctrl-armada-xp.c index fc33761..7daedc1 100644 --- a/drivers/pinctrl/mvebu/pinctrl-armada-xp.c +++ b/drivers/pinctrl/mvebu/pinctrl-armada-xp.c @@ -361,7 +361,7 @@ static struct mvebu_mpp_mode armada_xp_mpp_modes[] = { static struct mvebu_pinctrl_soc_info armada_xp_pinctrl_info; -static struct of_device_id armada_xp_pinctrl_of_match[] = { +static const struct of_device_id armada_xp_pinctrl_of_match[] = { { .compatible = "marvell,mv78230-pinctrl", .data = (void *) V_MV78230, diff --git a/drivers/pinctrl/mvebu/pinctrl-kirkwood.c b/drivers/pinctrl/mvebu/pinctrl-kirkwood.c index dbc673c..0f07dc5 100644 --- a/drivers/pinctrl/mvebu/pinctrl-kirkwood.c +++ b/drivers/pinctrl/mvebu/pinctrl-kirkwood.c @@ -456,7 +456,7 @@ static struct mvebu_pinctrl_soc_info mv98dx4122_info = { .ngpioranges = ARRAY_SIZE(mv88f628x_gpio_ranges), }; -static struct of_device_id kirkwood_pinctrl_of_match[] = { +static const struct of_device_id kirkwood_pinctrl_of_match[] = { { .compatible = "marvell,88f6180-pinctrl", .data = &mv88f6180_info }, { .compatible = "marvell,88f6190-pinctrl", .data = &mv88f6190_info }, { .compatible = "marvell,88f6192-pinctrl", .data = &mv88f6192_info }, diff --git a/drivers/pinctrl/mvebu/pinctrl-orion.c b/drivers/pinctrl/mvebu/pinctrl-orion.c index 3a632ef..3b7122d 100644 --- a/drivers/pinctrl/mvebu/pinctrl-orion.c +++ b/drivers/pinctrl/mvebu/pinctrl-orion.c @@ -211,7 +211,7 @@ static struct mvebu_pinctrl_soc_info mv88f5281_info = { * There are multiple variants of the Orion SoCs, but in terms of pin * muxing, they are identical. */ -static struct of_device_id orion_pinctrl_of_match[] = { +static const struct of_device_id orion_pinctrl_of_match[] = { { .compatible = "marvell,88f5181l-pinctrl", .data = &mv88f5181l_info }, { .compatible = "marvell,88f5182-pinctrl", .data = &mv88f5182_info }, { .compatible = "marvell,88f5281-pinctrl", .data = &mv88f5281_info }, diff --git a/drivers/pinctrl/pinctrl-as3722.c b/drivers/pinctrl/pinctrl-as3722.c index 169b1bf..db0571f 100644 --- a/drivers/pinctrl/pinctrl-as3722.c +++ b/drivers/pinctrl/pinctrl-as3722.c @@ -625,7 +625,7 @@ static int as3722_pinctrl_remove(struct platform_device *pdev) return 0; } -static struct of_device_id as3722_pinctrl_of_match[] = { +static const struct of_device_id as3722_pinctrl_of_match[] = { { .compatible = "ams,as3722-pinctrl", }, { }, }; diff --git a/drivers/pinctrl/pinctrl-at91.c b/drivers/pinctrl/pinctrl-at91.c index 6f1eba3..6b32817 100644 --- a/drivers/pinctrl/pinctrl-at91.c +++ b/drivers/pinctrl/pinctrl-at91.c @@ -1114,7 +1114,7 @@ static int at91_pinctrl_parse_functions(struct device_node *np, return 0; } -static struct of_device_id at91_pinctrl_of_match[] = { +static const struct of_device_id at91_pinctrl_of_match[] = { { .compatible = "atmel,sama5d3-pinctrl", .data = &sama5d3_ops }, { .compatible = "atmel,at91sam9x5-pinctrl", .data = &at91sam9x5_ops }, { .compatible = "atmel,at91rm9200-pinctrl", .data = &at91rm9200_ops }, @@ -1690,7 +1690,7 @@ static struct gpio_chip at91_gpio_template = { .ngpio = MAX_NB_GPIO_PER_BANK, }; -static struct of_device_id at91_gpio_of_match[] = { +static const struct of_device_id at91_gpio_of_match[] = { { .compatible = "atmel,at91sam9x5-gpio", .data = &at91sam9x5_ops, }, { .compatible = "atmel,at91rm9200-gpio", .data = &at91rm9200_ops }, { /* sentinel */ } diff --git a/drivers/pinctrl/pinctrl-palmas.c b/drivers/pinctrl/pinctrl-palmas.c index 26461e3..2631df0 100644 --- a/drivers/pinctrl/pinctrl-palmas.c +++ b/drivers/pinctrl/pinctrl-palmas.c @@ -987,7 +987,7 @@ static struct palmas_pinctrl_data tps80036_pinctrl_data = { .num_pin_groups = ARRAY_SIZE(tps80036_pingroups), }; -static struct of_device_id palmas_pinctrl_of_match[] = { +static const struct of_device_id palmas_pinctrl_of_match[] = { { .compatible = "ti,palmas-pinctrl", .data = &tps65913_pinctrl_data}, { .compatible = "ti,tps65913-pinctrl", .data = &tps65913_pinctrl_data}, { .compatible = "ti,tps80036-pinctrl", .data = &tps80036_pinctrl_data}, diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c index 69e8442..13b45f2 100644 --- a/drivers/pinctrl/pinctrl-single.c +++ b/drivers/pinctrl/pinctrl-single.c @@ -1501,7 +1501,7 @@ static void pcs_free_resources(struct pcs_device *pcs) } \ } while (0); -static struct of_device_id pcs_of_match[]; +static const struct of_device_id pcs_of_match[]; static int pcs_add_gpio_func(struct device_node *node, struct pcs_device *pcs) { @@ -2000,7 +2000,7 @@ static const struct pcs_soc_data pinconf_single = { .flags = PCS_FEAT_PINCONF, }; -static struct of_device_id pcs_of_match[] = { +static const struct of_device_id pcs_of_match[] = { { .compatible = "ti,omap3-padconf", .data = &pinctrl_single_omap_wkup }, { .compatible = "ti,omap4-padconf", .data = &pinctrl_single_omap_wkup }, { .compatible = "ti,omap5-padconf", .data = &pinctrl_single_omap_wkup }, diff --git a/drivers/pinctrl/pinctrl-st.c b/drivers/pinctrl/pinctrl-st.c index 9e5ec00..8442202 100644 --- a/drivers/pinctrl/pinctrl-st.c +++ b/drivers/pinctrl/pinctrl-st.c @@ -1531,7 +1531,7 @@ static int st_gpiolib_register_bank(struct st_pinctrl *info, return 0; } -static struct of_device_id st_pctl_of_match[] = { +static const struct of_device_id st_pctl_of_match[] = { { .compatible = "st,stih415-sbc-pinctrl", .data = &stih415_sbc_data }, { .compatible = "st,stih415-rear-pinctrl", .data = &stih415_rear_data }, { .compatible = "st,stih415-left-pinctrl", .data = &stih415_left_data }, diff --git a/drivers/pinctrl/pinctrl-tz1090-pdc.c b/drivers/pinctrl/pinctrl-tz1090-pdc.c index fab6aaf..8a8911b 100644 --- a/drivers/pinctrl/pinctrl-tz1090-pdc.c +++ b/drivers/pinctrl/pinctrl-tz1090-pdc.c @@ -969,7 +969,7 @@ static int tz1090_pdc_pinctrl_remove(struct platform_device *pdev) return 0; } -static struct of_device_id tz1090_pdc_pinctrl_of_match[] = { +static const struct of_device_id tz1090_pdc_pinctrl_of_match[] = { { .compatible = "img,tz1090-pdc-pinctrl", }, { }, }; diff --git a/drivers/pinctrl/pinctrl-tz1090.c b/drivers/pinctrl/pinctrl-tz1090.c index 8bd73075..fc5594a5 100644 --- a/drivers/pinctrl/pinctrl-tz1090.c +++ b/drivers/pinctrl/pinctrl-tz1090.c @@ -1984,7 +1984,7 @@ static int tz1090_pinctrl_remove(struct platform_device *pdev) return 0; } -static struct of_device_id tz1090_pinctrl_of_match[] = { +static const struct of_device_id tz1090_pinctrl_of_match[] = { { .compatible = "img,tz1090-pinctrl", }, { }, }; diff --git a/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c b/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c index 24c5d88..37aa2a8 100644 --- a/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c +++ b/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c @@ -1019,7 +1019,7 @@ static int sun4i_a10_pinctrl_probe(struct platform_device *pdev) &sun4i_a10_pinctrl_data); } -static struct of_device_id sun4i_a10_pinctrl_match[] = { +static const struct of_device_id sun4i_a10_pinctrl_match[] = { { .compatible = "allwinner,sun4i-a10-pinctrl", }, {} }; diff --git a/drivers/pinctrl/sunxi/pinctrl-sun5i-a10s.c b/drivers/pinctrl/sunxi/pinctrl-sun5i-a10s.c index 45a351a..6367661 100644 --- a/drivers/pinctrl/sunxi/pinctrl-sun5i-a10s.c +++ b/drivers/pinctrl/sunxi/pinctrl-sun5i-a10s.c @@ -670,7 +670,7 @@ static int sun5i_a10s_pinctrl_probe(struct platform_device *pdev) &sun5i_a10s_pinctrl_data); } -static struct of_device_id sun5i_a10s_pinctrl_match[] = { +static const struct of_device_id sun5i_a10s_pinctrl_match[] = { { .compatible = "allwinner,sun5i-a10s-pinctrl", }, {} }; diff --git a/drivers/pinctrl/sunxi/pinctrl-sun5i-a13.c b/drivers/pinctrl/sunxi/pinctrl-sun5i-a13.c index 4bd2347..2bb07b3 100644 --- a/drivers/pinctrl/sunxi/pinctrl-sun5i-a13.c +++ b/drivers/pinctrl/sunxi/pinctrl-sun5i-a13.c @@ -388,7 +388,7 @@ static int sun5i_a13_pinctrl_probe(struct platform_device *pdev) &sun5i_a13_pinctrl_data); } -static struct of_device_id sun5i_a13_pinctrl_match[] = { +static const struct of_device_id sun5i_a13_pinctrl_match[] = { { .compatible = "allwinner,sun5i-a13-pinctrl", }, {} }; diff --git a/drivers/pinctrl/sunxi/pinctrl-sun6i-a31-r.c b/drivers/pinctrl/sunxi/pinctrl-sun6i-a31-r.c index 02174fa..9596b0a3 100644 --- a/drivers/pinctrl/sunxi/pinctrl-sun6i-a31-r.c +++ b/drivers/pinctrl/sunxi/pinctrl-sun6i-a31-r.c @@ -120,7 +120,7 @@ static int sun6i_a31_r_pinctrl_probe(struct platform_device *pdev) return ret; } -static struct of_device_id sun6i_a31_r_pinctrl_match[] = { +static const struct of_device_id sun6i_a31_r_pinctrl_match[] = { { .compatible = "allwinner,sun6i-a31-r-pinctrl", }, {} }; diff --git a/drivers/pinctrl/sunxi/pinctrl-sun6i-a31.c b/drivers/pinctrl/sunxi/pinctrl-sun6i-a31.c index 18038f0..022863a 100644 --- a/drivers/pinctrl/sunxi/pinctrl-sun6i-a31.c +++ b/drivers/pinctrl/sunxi/pinctrl-sun6i-a31.c @@ -922,7 +922,7 @@ static int sun6i_a31_pinctrl_probe(struct platform_device *pdev) &sun6i_a31_pinctrl_data); } -static struct of_device_id sun6i_a31_pinctrl_match[] = { +static const struct of_device_id sun6i_a31_pinctrl_match[] = { { .compatible = "allwinner,sun6i-a31-pinctrl", }, {} }; diff --git a/drivers/pinctrl/sunxi/pinctrl-sun6i-a31s.c b/drivers/pinctrl/sunxi/pinctrl-sun6i-a31s.c index 9b5a91f..d3725dc 100644 --- a/drivers/pinctrl/sunxi/pinctrl-sun6i-a31s.c +++ b/drivers/pinctrl/sunxi/pinctrl-sun6i-a31s.c @@ -794,7 +794,7 @@ static int sun6i_a31s_pinctrl_probe(struct platform_device *pdev) &sun6i_a31s_pinctrl_data); } -static struct of_device_id sun6i_a31s_pinctrl_match[] = { +static const struct of_device_id sun6i_a31s_pinctrl_match[] = { { .compatible = "allwinner,sun6i-a31s-pinctrl", }, {} }; diff --git a/drivers/pinctrl/sunxi/pinctrl-sun7i-a20.c b/drivers/pinctrl/sunxi/pinctrl-sun7i-a20.c index 6af6cc8..cf1ce0c 100644 --- a/drivers/pinctrl/sunxi/pinctrl-sun7i-a20.c +++ b/drivers/pinctrl/sunxi/pinctrl-sun7i-a20.c @@ -1045,7 +1045,7 @@ static int sun7i_a20_pinctrl_probe(struct platform_device *pdev) &sun7i_a20_pinctrl_data); } -static struct of_device_id sun7i_a20_pinctrl_match[] = { +static const struct of_device_id sun7i_a20_pinctrl_match[] = { { .compatible = "allwinner,sun7i-a20-pinctrl", }, {} }; diff --git a/drivers/pinctrl/sunxi/pinctrl-sun8i-a23-r.c b/drivers/pinctrl/sunxi/pinctrl-sun8i-a23-r.c index 327e03f..0562876 100644 --- a/drivers/pinctrl/sunxi/pinctrl-sun8i-a23-r.c +++ b/drivers/pinctrl/sunxi/pinctrl-sun8i-a23-r.c @@ -119,7 +119,7 @@ static int sun8i_a23_r_pinctrl_probe(struct platform_device *pdev) return ret; } -static struct of_device_id sun8i_a23_r_pinctrl_match[] = { +static const struct of_device_id sun8i_a23_r_pinctrl_match[] = { { .compatible = "allwinner,sun8i-a23-r-pinctrl", }, {} }; diff --git a/drivers/pinctrl/sunxi/pinctrl-sun8i-a23.c b/drivers/pinctrl/sunxi/pinctrl-sun8i-a23.c index 62695c9..55083d2 100644 --- a/drivers/pinctrl/sunxi/pinctrl-sun8i-a23.c +++ b/drivers/pinctrl/sunxi/pinctrl-sun8i-a23.c @@ -571,7 +571,7 @@ static int sun8i_a23_pinctrl_probe(struct platform_device *pdev) &sun8i_a23_pinctrl_data); } -static struct of_device_id sun8i_a23_pinctrl_match[] = { +static const struct of_device_id sun8i_a23_pinctrl_match[] = { { .compatible = "allwinner,sun8i-a23-pinctrl", }, {} }; diff --git a/drivers/pinctrl/sunxi/pinctrl-sun9i-a80.c b/drivers/pinctrl/sunxi/pinctrl-sun9i-a80.c index adb2942..1b580ba 100644 --- a/drivers/pinctrl/sunxi/pinctrl-sun9i-a80.c +++ b/drivers/pinctrl/sunxi/pinctrl-sun9i-a80.c @@ -729,7 +729,7 @@ static int sun9i_a80_pinctrl_probe(struct platform_device *pdev) &sun9i_a80_pinctrl_data); } -static struct of_device_id sun9i_a80_pinctrl_match[] = { +static const struct of_device_id sun9i_a80_pinctrl_match[] = { { .compatible = "allwinner,sun9i-a80-pinctrl", }, {} }; diff --git a/drivers/pinctrl/vt8500/pinctrl-vt8500.c b/drivers/pinctrl/vt8500/pinctrl-vt8500.c index cf8bbc9..ca946b3 100644 --- a/drivers/pinctrl/vt8500/pinctrl-vt8500.c +++ b/drivers/pinctrl/vt8500/pinctrl-vt8500.c @@ -478,7 +478,7 @@ static int vt8500_pinctrl_remove(struct platform_device *pdev) return wmt_pinctrl_remove(pdev); } -static struct of_device_id wmt_pinctrl_of_match[] = { +static const struct of_device_id wmt_pinctrl_of_match[] = { { .compatible = "via,vt8500-pinctrl" }, { /* sentinel */ }, }; diff --git a/drivers/pinctrl/vt8500/pinctrl-wm8505.c b/drivers/pinctrl/vt8500/pinctrl-wm8505.c index 3f9c32d..626fc7e 100644 --- a/drivers/pinctrl/vt8500/pinctrl-wm8505.c +++ b/drivers/pinctrl/vt8500/pinctrl-wm8505.c @@ -509,7 +509,7 @@ static int wm8505_pinctrl_remove(struct platform_device *pdev) return wmt_pinctrl_remove(pdev); } -static struct of_device_id wmt_pinctrl_of_match[] = { +static const struct of_device_id wmt_pinctrl_of_match[] = { { .compatible = "wm,wm8505-pinctrl" }, { /* sentinel */ }, }; diff --git a/drivers/pinctrl/vt8500/pinctrl-wm8650.c b/drivers/pinctrl/vt8500/pinctrl-wm8650.c index 4e80f98..8953aba 100644 --- a/drivers/pinctrl/vt8500/pinctrl-wm8650.c +++ b/drivers/pinctrl/vt8500/pinctrl-wm8650.c @@ -347,7 +347,7 @@ static int wm8650_pinctrl_remove(struct platform_device *pdev) return wmt_pinctrl_remove(pdev); } -static struct of_device_id wmt_pinctrl_of_match[] = { +static const struct of_device_id wmt_pinctrl_of_match[] = { { .compatible = "wm,wm8650-pinctrl" }, { /* sentinel */ }, }; diff --git a/drivers/pinctrl/vt8500/pinctrl-wm8750.c b/drivers/pinctrl/vt8500/pinctrl-wm8750.c index 47b52a7..c79053d 100644 --- a/drivers/pinctrl/vt8500/pinctrl-wm8750.c +++ b/drivers/pinctrl/vt8500/pinctrl-wm8750.c @@ -386,7 +386,7 @@ static int wm8750_pinctrl_remove(struct platform_device *pdev) return wmt_pinctrl_remove(pdev); } -static struct of_device_id wmt_pinctrl_of_match[] = { +static const struct of_device_id wmt_pinctrl_of_match[] = { { .compatible = "wm,wm8750-pinctrl" }, { /* sentinel */ }, }; diff --git a/drivers/pinctrl/vt8500/pinctrl-wm8850.c b/drivers/pinctrl/vt8500/pinctrl-wm8850.c index 8bbb38c..f232b16 100644 --- a/drivers/pinctrl/vt8500/pinctrl-wm8850.c +++ b/drivers/pinctrl/vt8500/pinctrl-wm8850.c @@ -365,7 +365,7 @@ static int wm8850_pinctrl_remove(struct platform_device *pdev) return wmt_pinctrl_remove(pdev); } -static struct of_device_id wmt_pinctrl_of_match[] = { +static const struct of_device_id wmt_pinctrl_of_match[] = { { .compatible = "wm,wm8850-pinctrl" }, { /* sentinel */ }, }; -- 2.1.0 ^ permalink raw reply related [flat|nested] 49+ messages in thread
* [PATCH 35/35 linux-next] pinctrl: constify of_device_id array 2015-03-16 19:59 ` [PATCH 35/35 linux-next] pinctrl: " Fabian Frederick @ 2015-03-17 4:03 ` Jean-Christophe PLAGNIOL-VILLARD 2015-03-17 8:15 ` Patrice Chotard ` (4 subsequent siblings) 5 siblings, 0 replies; 49+ messages in thread From: Jean-Christophe PLAGNIOL-VILLARD @ 2015-03-17 4:03 UTC (permalink / raw) To: linux-arm-kernel On 20:59 Mon 16 Mar , Fabian Frederick wrote: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > > Signed-off-by: Fabian Frederick <fabf@skynet.be> Acked-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com> Best Regards, J. > --- > drivers/pinctrl/bcm/pinctrl-bcm2835.c | 2 +- > drivers/pinctrl/mediatek/pinctrl-mt8135.c | 2 +- > drivers/pinctrl/mediatek/pinctrl-mt8173.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-armada-370.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-armada-375.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-armada-38x.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-armada-39x.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-armada-xp.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-kirkwood.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-orion.c | 2 +- > drivers/pinctrl/pinctrl-as3722.c | 2 +- > drivers/pinctrl/pinctrl-at91.c | 4 ++-- > drivers/pinctrl/pinctrl-palmas.c | 2 +- > drivers/pinctrl/pinctrl-single.c | 4 ++-- > drivers/pinctrl/pinctrl-st.c | 2 +- > drivers/pinctrl/pinctrl-tz1090-pdc.c | 2 +- > drivers/pinctrl/pinctrl-tz1090.c | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun5i-a10s.c | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun5i-a13.c | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun6i-a31-r.c | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun6i-a31.c | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun6i-a31s.c | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun7i-a20.c | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun8i-a23-r.c | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun8i-a23.c | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun9i-a80.c | 2 +- > drivers/pinctrl/vt8500/pinctrl-vt8500.c | 2 +- > drivers/pinctrl/vt8500/pinctrl-wm8505.c | 2 +- > drivers/pinctrl/vt8500/pinctrl-wm8650.c | 2 +- > drivers/pinctrl/vt8500/pinctrl-wm8750.c | 2 +- > drivers/pinctrl/vt8500/pinctrl-wm8850.c | 2 +- > 32 files changed, 34 insertions(+), 34 deletions(-) > > diff --git a/drivers/pinctrl/bcm/pinctrl-bcm2835.c b/drivers/pinctrl/bcm/pinctrl-bcm2835.c > index 9aa8a3f..4d08b85 100644 > --- a/drivers/pinctrl/bcm/pinctrl-bcm2835.c > +++ b/drivers/pinctrl/bcm/pinctrl-bcm2835.c > @@ -1051,7 +1051,7 @@ static int bcm2835_pinctrl_remove(struct platform_device *pdev) > return 0; > } > > -static struct of_device_id bcm2835_pinctrl_match[] = { > +static const struct of_device_id bcm2835_pinctrl_match[] = { > { .compatible = "brcm,bcm2835-gpio" }, > {} > }; > diff --git a/drivers/pinctrl/mediatek/pinctrl-mt8135.c b/drivers/pinctrl/mediatek/pinctrl-mt8135.c > index 1296d6d..82c4af4 100644 > --- a/drivers/pinctrl/mediatek/pinctrl-mt8135.c > +++ b/drivers/pinctrl/mediatek/pinctrl-mt8135.c > @@ -347,7 +347,7 @@ static int mt8135_pinctrl_probe(struct platform_device *pdev) > return mtk_pctrl_init(pdev, &mt8135_pinctrl_data); > } > > -static struct of_device_id mt8135_pctrl_match[] = { > +static const struct of_device_id mt8135_pctrl_match[] = { > { > .compatible = "mediatek,mt8135-pinctrl", > }, { > diff --git a/drivers/pinctrl/mediatek/pinctrl-mt8173.c b/drivers/pinctrl/mediatek/pinctrl-mt8173.c > index f07cafb..594f7b5 100644 > --- a/drivers/pinctrl/mediatek/pinctrl-mt8173.c > +++ b/drivers/pinctrl/mediatek/pinctrl-mt8173.c > @@ -427,7 +427,7 @@ static int mt8173_pinctrl_probe(struct platform_device *pdev) > return mtk_pctrl_init(pdev, &mt8173_pinctrl_data); > } > > -static struct of_device_id mt8173_pctrl_match[] = { > +static const struct of_device_id mt8173_pctrl_match[] = { > { > .compatible = "mediatek,mt8173-pinctrl", > }, { > diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-370.c b/drivers/pinctrl/mvebu/pinctrl-armada-370.c > index c4f51d0..42f930f 100644 > --- a/drivers/pinctrl/mvebu/pinctrl-armada-370.c > +++ b/drivers/pinctrl/mvebu/pinctrl-armada-370.c > @@ -379,7 +379,7 @@ static struct mvebu_mpp_mode mv88f6710_mpp_modes[] = { > > static struct mvebu_pinctrl_soc_info armada_370_pinctrl_info; > > -static struct of_device_id armada_370_pinctrl_of_match[] = { > +static const struct of_device_id armada_370_pinctrl_of_match[] = { > { .compatible = "marvell,mv88f6710-pinctrl" }, > { }, > }; > diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-375.c b/drivers/pinctrl/mvebu/pinctrl-armada-375.c > index cd7c8f5..ca1e757 100644 > --- a/drivers/pinctrl/mvebu/pinctrl-armada-375.c > +++ b/drivers/pinctrl/mvebu/pinctrl-armada-375.c > @@ -399,7 +399,7 @@ static struct mvebu_mpp_mode mv88f6720_mpp_modes[] = { > > static struct mvebu_pinctrl_soc_info armada_375_pinctrl_info; > > -static struct of_device_id armada_375_pinctrl_of_match[] = { > +static const struct of_device_id armada_375_pinctrl_of_match[] = { > { .compatible = "marvell,mv88f6720-pinctrl" }, > { }, > }; > diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-38x.c b/drivers/pinctrl/mvebu/pinctrl-armada-38x.c > index 7302f66..83bbcc7 100644 > --- a/drivers/pinctrl/mvebu/pinctrl-armada-38x.c > +++ b/drivers/pinctrl/mvebu/pinctrl-armada-38x.c > @@ -389,7 +389,7 @@ static struct mvebu_mpp_mode armada_38x_mpp_modes[] = { > > static struct mvebu_pinctrl_soc_info armada_38x_pinctrl_info; > > -static struct of_device_id armada_38x_pinctrl_of_match[] = { > +static const struct of_device_id armada_38x_pinctrl_of_match[] = { > { > .compatible = "marvell,mv88f6810-pinctrl", > .data = (void *) V_88F6810, > diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-39x.c b/drivers/pinctrl/mvebu/pinctrl-armada-39x.c > index 5963411..4249162 100644 > --- a/drivers/pinctrl/mvebu/pinctrl-armada-39x.c > +++ b/drivers/pinctrl/mvebu/pinctrl-armada-39x.c > @@ -362,7 +362,7 @@ static struct mvebu_mpp_mode armada_39x_mpp_modes[] = { > > static struct mvebu_pinctrl_soc_info armada_39x_pinctrl_info; > > -static struct of_device_id armada_39x_pinctrl_of_match[] = { > +static const struct of_device_id armada_39x_pinctrl_of_match[] = { > { > .compatible = "marvell,mv88f6920-pinctrl", > .data = (void *) V_88F6920, > diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-xp.c b/drivers/pinctrl/mvebu/pinctrl-armada-xp.c > index fc33761..7daedc1 100644 > --- a/drivers/pinctrl/mvebu/pinctrl-armada-xp.c > +++ b/drivers/pinctrl/mvebu/pinctrl-armada-xp.c > @@ -361,7 +361,7 @@ static struct mvebu_mpp_mode armada_xp_mpp_modes[] = { > > static struct mvebu_pinctrl_soc_info armada_xp_pinctrl_info; > > -static struct of_device_id armada_xp_pinctrl_of_match[] = { > +static const struct of_device_id armada_xp_pinctrl_of_match[] = { > { > .compatible = "marvell,mv78230-pinctrl", > .data = (void *) V_MV78230, > diff --git a/drivers/pinctrl/mvebu/pinctrl-kirkwood.c b/drivers/pinctrl/mvebu/pinctrl-kirkwood.c > index dbc673c..0f07dc5 100644 > --- a/drivers/pinctrl/mvebu/pinctrl-kirkwood.c > +++ b/drivers/pinctrl/mvebu/pinctrl-kirkwood.c > @@ -456,7 +456,7 @@ static struct mvebu_pinctrl_soc_info mv98dx4122_info = { > .ngpioranges = ARRAY_SIZE(mv88f628x_gpio_ranges), > }; > > -static struct of_device_id kirkwood_pinctrl_of_match[] = { > +static const struct of_device_id kirkwood_pinctrl_of_match[] = { > { .compatible = "marvell,88f6180-pinctrl", .data = &mv88f6180_info }, > { .compatible = "marvell,88f6190-pinctrl", .data = &mv88f6190_info }, > { .compatible = "marvell,88f6192-pinctrl", .data = &mv88f6192_info }, > diff --git a/drivers/pinctrl/mvebu/pinctrl-orion.c b/drivers/pinctrl/mvebu/pinctrl-orion.c > index 3a632ef..3b7122d 100644 > --- a/drivers/pinctrl/mvebu/pinctrl-orion.c > +++ b/drivers/pinctrl/mvebu/pinctrl-orion.c > @@ -211,7 +211,7 @@ static struct mvebu_pinctrl_soc_info mv88f5281_info = { > * There are multiple variants of the Orion SoCs, but in terms of pin > * muxing, they are identical. > */ > -static struct of_device_id orion_pinctrl_of_match[] = { > +static const struct of_device_id orion_pinctrl_of_match[] = { > { .compatible = "marvell,88f5181l-pinctrl", .data = &mv88f5181l_info }, > { .compatible = "marvell,88f5182-pinctrl", .data = &mv88f5182_info }, > { .compatible = "marvell,88f5281-pinctrl", .data = &mv88f5281_info }, > diff --git a/drivers/pinctrl/pinctrl-as3722.c b/drivers/pinctrl/pinctrl-as3722.c > index 169b1bf..db0571f 100644 > --- a/drivers/pinctrl/pinctrl-as3722.c > +++ b/drivers/pinctrl/pinctrl-as3722.c > @@ -625,7 +625,7 @@ static int as3722_pinctrl_remove(struct platform_device *pdev) > return 0; > } > > -static struct of_device_id as3722_pinctrl_of_match[] = { > +static const struct of_device_id as3722_pinctrl_of_match[] = { > { .compatible = "ams,as3722-pinctrl", }, > { }, > }; > diff --git a/drivers/pinctrl/pinctrl-at91.c b/drivers/pinctrl/pinctrl-at91.c > index 6f1eba3..6b32817 100644 > --- a/drivers/pinctrl/pinctrl-at91.c > +++ b/drivers/pinctrl/pinctrl-at91.c > @@ -1114,7 +1114,7 @@ static int at91_pinctrl_parse_functions(struct device_node *np, > return 0; > } > > -static struct of_device_id at91_pinctrl_of_match[] = { > +static const struct of_device_id at91_pinctrl_of_match[] = { > { .compatible = "atmel,sama5d3-pinctrl", .data = &sama5d3_ops }, > { .compatible = "atmel,at91sam9x5-pinctrl", .data = &at91sam9x5_ops }, > { .compatible = "atmel,at91rm9200-pinctrl", .data = &at91rm9200_ops }, > @@ -1690,7 +1690,7 @@ static struct gpio_chip at91_gpio_template = { > .ngpio = MAX_NB_GPIO_PER_BANK, > }; > > -static struct of_device_id at91_gpio_of_match[] = { > +static const struct of_device_id at91_gpio_of_match[] = { > { .compatible = "atmel,at91sam9x5-gpio", .data = &at91sam9x5_ops, }, > { .compatible = "atmel,at91rm9200-gpio", .data = &at91rm9200_ops }, > { /* sentinel */ } > diff --git a/drivers/pinctrl/pinctrl-palmas.c b/drivers/pinctrl/pinctrl-palmas.c > index 26461e3..2631df0 100644 > --- a/drivers/pinctrl/pinctrl-palmas.c > +++ b/drivers/pinctrl/pinctrl-palmas.c > @@ -987,7 +987,7 @@ static struct palmas_pinctrl_data tps80036_pinctrl_data = { > .num_pin_groups = ARRAY_SIZE(tps80036_pingroups), > }; > > -static struct of_device_id palmas_pinctrl_of_match[] = { > +static const struct of_device_id palmas_pinctrl_of_match[] = { > { .compatible = "ti,palmas-pinctrl", .data = &tps65913_pinctrl_data}, > { .compatible = "ti,tps65913-pinctrl", .data = &tps65913_pinctrl_data}, > { .compatible = "ti,tps80036-pinctrl", .data = &tps80036_pinctrl_data}, > diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c > index 69e8442..13b45f2 100644 > --- a/drivers/pinctrl/pinctrl-single.c > +++ b/drivers/pinctrl/pinctrl-single.c > @@ -1501,7 +1501,7 @@ static void pcs_free_resources(struct pcs_device *pcs) > } \ > } while (0); > > -static struct of_device_id pcs_of_match[]; > +static const struct of_device_id pcs_of_match[]; > > static int pcs_add_gpio_func(struct device_node *node, struct pcs_device *pcs) > { > @@ -2000,7 +2000,7 @@ static const struct pcs_soc_data pinconf_single = { > .flags = PCS_FEAT_PINCONF, > }; > > -static struct of_device_id pcs_of_match[] = { > +static const struct of_device_id pcs_of_match[] = { > { .compatible = "ti,omap3-padconf", .data = &pinctrl_single_omap_wkup }, > { .compatible = "ti,omap4-padconf", .data = &pinctrl_single_omap_wkup }, > { .compatible = "ti,omap5-padconf", .data = &pinctrl_single_omap_wkup }, > diff --git a/drivers/pinctrl/pinctrl-st.c b/drivers/pinctrl/pinctrl-st.c > index 9e5ec00..8442202 100644 > --- a/drivers/pinctrl/pinctrl-st.c > +++ b/drivers/pinctrl/pinctrl-st.c > @@ -1531,7 +1531,7 @@ static int st_gpiolib_register_bank(struct st_pinctrl *info, > return 0; > } > > -static struct of_device_id st_pctl_of_match[] = { > +static const struct of_device_id st_pctl_of_match[] = { > { .compatible = "st,stih415-sbc-pinctrl", .data = &stih415_sbc_data }, > { .compatible = "st,stih415-rear-pinctrl", .data = &stih415_rear_data }, > { .compatible = "st,stih415-left-pinctrl", .data = &stih415_left_data }, > diff --git a/drivers/pinctrl/pinctrl-tz1090-pdc.c b/drivers/pinctrl/pinctrl-tz1090-pdc.c > index fab6aaf..8a8911b 100644 > --- a/drivers/pinctrl/pinctrl-tz1090-pdc.c > +++ b/drivers/pinctrl/pinctrl-tz1090-pdc.c > @@ -969,7 +969,7 @@ static int tz1090_pdc_pinctrl_remove(struct platform_device *pdev) > return 0; > } > > -static struct of_device_id tz1090_pdc_pinctrl_of_match[] = { > +static const struct of_device_id tz1090_pdc_pinctrl_of_match[] = { > { .compatible = "img,tz1090-pdc-pinctrl", }, > { }, > }; > diff --git a/drivers/pinctrl/pinctrl-tz1090.c b/drivers/pinctrl/pinctrl-tz1090.c > index 8bd73075..fc5594a5 100644 > --- a/drivers/pinctrl/pinctrl-tz1090.c > +++ b/drivers/pinctrl/pinctrl-tz1090.c > @@ -1984,7 +1984,7 @@ static int tz1090_pinctrl_remove(struct platform_device *pdev) > return 0; > } > > -static struct of_device_id tz1090_pinctrl_of_match[] = { > +static const struct of_device_id tz1090_pinctrl_of_match[] = { > { .compatible = "img,tz1090-pinctrl", }, > { }, > }; > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c b/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c > index 24c5d88..37aa2a8 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c > @@ -1019,7 +1019,7 @@ static int sun4i_a10_pinctrl_probe(struct platform_device *pdev) > &sun4i_a10_pinctrl_data); > } > > -static struct of_device_id sun4i_a10_pinctrl_match[] = { > +static const struct of_device_id sun4i_a10_pinctrl_match[] = { > { .compatible = "allwinner,sun4i-a10-pinctrl", }, > {} > }; > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun5i-a10s.c b/drivers/pinctrl/sunxi/pinctrl-sun5i-a10s.c > index 45a351a..6367661 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun5i-a10s.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun5i-a10s.c > @@ -670,7 +670,7 @@ static int sun5i_a10s_pinctrl_probe(struct platform_device *pdev) > &sun5i_a10s_pinctrl_data); > } > > -static struct of_device_id sun5i_a10s_pinctrl_match[] = { > +static const struct of_device_id sun5i_a10s_pinctrl_match[] = { > { .compatible = "allwinner,sun5i-a10s-pinctrl", }, > {} > }; > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun5i-a13.c b/drivers/pinctrl/sunxi/pinctrl-sun5i-a13.c > index 4bd2347..2bb07b3 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun5i-a13.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun5i-a13.c > @@ -388,7 +388,7 @@ static int sun5i_a13_pinctrl_probe(struct platform_device *pdev) > &sun5i_a13_pinctrl_data); > } > > -static struct of_device_id sun5i_a13_pinctrl_match[] = { > +static const struct of_device_id sun5i_a13_pinctrl_match[] = { > { .compatible = "allwinner,sun5i-a13-pinctrl", }, > {} > }; > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun6i-a31-r.c b/drivers/pinctrl/sunxi/pinctrl-sun6i-a31-r.c > index 02174fa..9596b0a3 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun6i-a31-r.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun6i-a31-r.c > @@ -120,7 +120,7 @@ static int sun6i_a31_r_pinctrl_probe(struct platform_device *pdev) > return ret; > } > > -static struct of_device_id sun6i_a31_r_pinctrl_match[] = { > +static const struct of_device_id sun6i_a31_r_pinctrl_match[] = { > { .compatible = "allwinner,sun6i-a31-r-pinctrl", }, > {} > }; > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun6i-a31.c b/drivers/pinctrl/sunxi/pinctrl-sun6i-a31.c > index 18038f0..022863a 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun6i-a31.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun6i-a31.c > @@ -922,7 +922,7 @@ static int sun6i_a31_pinctrl_probe(struct platform_device *pdev) > &sun6i_a31_pinctrl_data); > } > > -static struct of_device_id sun6i_a31_pinctrl_match[] = { > +static const struct of_device_id sun6i_a31_pinctrl_match[] = { > { .compatible = "allwinner,sun6i-a31-pinctrl", }, > {} > }; > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun6i-a31s.c b/drivers/pinctrl/sunxi/pinctrl-sun6i-a31s.c > index 9b5a91f..d3725dc 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun6i-a31s.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun6i-a31s.c > @@ -794,7 +794,7 @@ static int sun6i_a31s_pinctrl_probe(struct platform_device *pdev) > &sun6i_a31s_pinctrl_data); > } > > -static struct of_device_id sun6i_a31s_pinctrl_match[] = { > +static const struct of_device_id sun6i_a31s_pinctrl_match[] = { > { .compatible = "allwinner,sun6i-a31s-pinctrl", }, > {} > }; > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun7i-a20.c b/drivers/pinctrl/sunxi/pinctrl-sun7i-a20.c > index 6af6cc8..cf1ce0c 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun7i-a20.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun7i-a20.c > @@ -1045,7 +1045,7 @@ static int sun7i_a20_pinctrl_probe(struct platform_device *pdev) > &sun7i_a20_pinctrl_data); > } > > -static struct of_device_id sun7i_a20_pinctrl_match[] = { > +static const struct of_device_id sun7i_a20_pinctrl_match[] = { > { .compatible = "allwinner,sun7i-a20-pinctrl", }, > {} > }; > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun8i-a23-r.c b/drivers/pinctrl/sunxi/pinctrl-sun8i-a23-r.c > index 327e03f..0562876 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun8i-a23-r.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun8i-a23-r.c > @@ -119,7 +119,7 @@ static int sun8i_a23_r_pinctrl_probe(struct platform_device *pdev) > return ret; > } > > -static struct of_device_id sun8i_a23_r_pinctrl_match[] = { > +static const struct of_device_id sun8i_a23_r_pinctrl_match[] = { > { .compatible = "allwinner,sun8i-a23-r-pinctrl", }, > {} > }; > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun8i-a23.c b/drivers/pinctrl/sunxi/pinctrl-sun8i-a23.c > index 62695c9..55083d2 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun8i-a23.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun8i-a23.c > @@ -571,7 +571,7 @@ static int sun8i_a23_pinctrl_probe(struct platform_device *pdev) > &sun8i_a23_pinctrl_data); > } > > -static struct of_device_id sun8i_a23_pinctrl_match[] = { > +static const struct of_device_id sun8i_a23_pinctrl_match[] = { > { .compatible = "allwinner,sun8i-a23-pinctrl", }, > {} > }; > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun9i-a80.c b/drivers/pinctrl/sunxi/pinctrl-sun9i-a80.c > index adb2942..1b580ba 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun9i-a80.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun9i-a80.c > @@ -729,7 +729,7 @@ static int sun9i_a80_pinctrl_probe(struct platform_device *pdev) > &sun9i_a80_pinctrl_data); > } > > -static struct of_device_id sun9i_a80_pinctrl_match[] = { > +static const struct of_device_id sun9i_a80_pinctrl_match[] = { > { .compatible = "allwinner,sun9i-a80-pinctrl", }, > {} > }; > diff --git a/drivers/pinctrl/vt8500/pinctrl-vt8500.c b/drivers/pinctrl/vt8500/pinctrl-vt8500.c > index cf8bbc9..ca946b3 100644 > --- a/drivers/pinctrl/vt8500/pinctrl-vt8500.c > +++ b/drivers/pinctrl/vt8500/pinctrl-vt8500.c > @@ -478,7 +478,7 @@ static int vt8500_pinctrl_remove(struct platform_device *pdev) > return wmt_pinctrl_remove(pdev); > } > > -static struct of_device_id wmt_pinctrl_of_match[] = { > +static const struct of_device_id wmt_pinctrl_of_match[] = { > { .compatible = "via,vt8500-pinctrl" }, > { /* sentinel */ }, > }; > diff --git a/drivers/pinctrl/vt8500/pinctrl-wm8505.c b/drivers/pinctrl/vt8500/pinctrl-wm8505.c > index 3f9c32d..626fc7e 100644 > --- a/drivers/pinctrl/vt8500/pinctrl-wm8505.c > +++ b/drivers/pinctrl/vt8500/pinctrl-wm8505.c > @@ -509,7 +509,7 @@ static int wm8505_pinctrl_remove(struct platform_device *pdev) > return wmt_pinctrl_remove(pdev); > } > > -static struct of_device_id wmt_pinctrl_of_match[] = { > +static const struct of_device_id wmt_pinctrl_of_match[] = { > { .compatible = "wm,wm8505-pinctrl" }, > { /* sentinel */ }, > }; > diff --git a/drivers/pinctrl/vt8500/pinctrl-wm8650.c b/drivers/pinctrl/vt8500/pinctrl-wm8650.c > index 4e80f98..8953aba 100644 > --- a/drivers/pinctrl/vt8500/pinctrl-wm8650.c > +++ b/drivers/pinctrl/vt8500/pinctrl-wm8650.c > @@ -347,7 +347,7 @@ static int wm8650_pinctrl_remove(struct platform_device *pdev) > return wmt_pinctrl_remove(pdev); > } > > -static struct of_device_id wmt_pinctrl_of_match[] = { > +static const struct of_device_id wmt_pinctrl_of_match[] = { > { .compatible = "wm,wm8650-pinctrl" }, > { /* sentinel */ }, > }; > diff --git a/drivers/pinctrl/vt8500/pinctrl-wm8750.c b/drivers/pinctrl/vt8500/pinctrl-wm8750.c > index 47b52a7..c79053d 100644 > --- a/drivers/pinctrl/vt8500/pinctrl-wm8750.c > +++ b/drivers/pinctrl/vt8500/pinctrl-wm8750.c > @@ -386,7 +386,7 @@ static int wm8750_pinctrl_remove(struct platform_device *pdev) > return wmt_pinctrl_remove(pdev); > } > > -static struct of_device_id wmt_pinctrl_of_match[] = { > +static const struct of_device_id wmt_pinctrl_of_match[] = { > { .compatible = "wm,wm8750-pinctrl" }, > { /* sentinel */ }, > }; > diff --git a/drivers/pinctrl/vt8500/pinctrl-wm8850.c b/drivers/pinctrl/vt8500/pinctrl-wm8850.c > index 8bbb38c..f232b16 100644 > --- a/drivers/pinctrl/vt8500/pinctrl-wm8850.c > +++ b/drivers/pinctrl/vt8500/pinctrl-wm8850.c > @@ -365,7 +365,7 @@ static int wm8850_pinctrl_remove(struct platform_device *pdev) > return wmt_pinctrl_remove(pdev); > } > > -static struct of_device_id wmt_pinctrl_of_match[] = { > +static const struct of_device_id wmt_pinctrl_of_match[] = { > { .compatible = "wm,wm8850-pinctrl" }, > { /* sentinel */ }, > }; > -- > 2.1.0 > ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 35/35 linux-next] pinctrl: constify of_device_id array 2015-03-16 19:59 ` [PATCH 35/35 linux-next] pinctrl: " Fabian Frederick 2015-03-17 4:03 ` Jean-Christophe PLAGNIOL-VILLARD @ 2015-03-17 8:15 ` Patrice Chotard 2015-03-17 8:34 ` Maxime Coquelin ` (3 subsequent siblings) 5 siblings, 0 replies; 49+ messages in thread From: Patrice Chotard @ 2015-03-17 8:15 UTC (permalink / raw) To: linux-arm-kernel Hi Fabian On 03/16/2015 08:59 PM, Fabian Frederick wrote: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > > Signed-off-by: Fabian Frederick <fabf@skynet.be> > --- ... > drivers/pinctrl/pinctrl-st.c | 2 +- > For this driver Acked-by: Patrice Chotard <patrice.chotard@st.com> Thanks ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 35/35 linux-next] pinctrl: constify of_device_id array 2015-03-16 19:59 ` [PATCH 35/35 linux-next] pinctrl: " Fabian Frederick 2015-03-17 4:03 ` Jean-Christophe PLAGNIOL-VILLARD 2015-03-17 8:15 ` Patrice Chotard @ 2015-03-17 8:34 ` Maxime Coquelin 2015-03-19 1:58 ` Hongzhou Yang ` (2 subsequent siblings) 5 siblings, 0 replies; 49+ messages in thread From: Maxime Coquelin @ 2015-03-17 8:34 UTC (permalink / raw) To: linux-arm-kernel On 03/16/2015 08:59 PM, Fabian Frederick wrote: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > > Signed-off-by: Fabian Frederick <fabf@skynet.be> > --- > drivers/pinctrl/bcm/pinctrl-bcm2835.c | 2 +- > drivers/pinctrl/mediatek/pinctrl-mt8135.c | 2 +- > drivers/pinctrl/mediatek/pinctrl-mt8173.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-armada-370.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-armada-375.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-armada-38x.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-armada-39x.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-armada-xp.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-kirkwood.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-orion.c | 2 +- > drivers/pinctrl/pinctrl-as3722.c | 2 +- > drivers/pinctrl/pinctrl-at91.c | 4 ++-- > drivers/pinctrl/pinctrl-palmas.c | 2 +- > drivers/pinctrl/pinctrl-single.c | 4 ++-- > drivers/pinctrl/pinctrl-st.c | 2 +- > For the pinctrl-st driver, you can add my: Acked-by: Maxime Coquelin <maxime.coquelin@st.com> Thanks! Maxime ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 35/35 linux-next] pinctrl: constify of_device_id array 2015-03-16 19:59 ` [PATCH 35/35 linux-next] pinctrl: " Fabian Frederick ` (2 preceding siblings ...) 2015-03-17 8:34 ` Maxime Coquelin @ 2015-03-19 1:58 ` Hongzhou Yang 2015-03-19 18:41 ` Fabian Frederick 2015-03-19 7:12 ` Lee Jones 2015-03-27 9:00 ` Linus Walleij 5 siblings, 1 reply; 49+ messages in thread From: Hongzhou Yang @ 2015-03-19 1:58 UTC (permalink / raw) To: linux-arm-kernel On Mon, 2015-03-16 at 20:59 +0100, Fabian Frederick wrote: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > > Signed-off-by: Fabian Frederick <fabf@skynet.be> > --- > drivers/pinctrl/bcm/pinctrl-bcm2835.c | 2 +- > drivers/pinctrl/mediatek/pinctrl-mt8135.c | 2 +- > drivers/pinctrl/mediatek/pinctrl-mt8173.c | 2 +- > For the pinctrl-mt81xx driver, Acked-by: Hongzhou Yang <hongzhou.yang@mediatek.com> By the way, Axel has sent same patch to linux-gpio. http://article.gmane.org/gmane.linux.kernel.gpio/6815 Thanks. Hongzhou ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 35/35 linux-next] pinctrl: constify of_device_id array 2015-03-19 1:58 ` Hongzhou Yang @ 2015-03-19 18:41 ` Fabian Frederick 2015-03-19 20:55 ` Hongzhou Yang 0 siblings, 1 reply; 49+ messages in thread From: Fabian Frederick @ 2015-03-19 18:41 UTC (permalink / raw) To: linux-arm-kernel > On 19 March 2015 at 02:58 Hongzhou Yang <hongzhou.yang@mediatek.com> wrote: > > > On Mon, 2015-03-16 at 20:59 +0100, Fabian Frederick wrote: > > of_device_id is always used as const. > > (See driver.of_match_table and open firmware functions) > > > > Signed-off-by: Fabian Frederick <fabf@skynet.be> > > --- > >? drivers/pinctrl/bcm/pinctrl-bcm2835.c? ? ? ?| 2 +- > >? drivers/pinctrl/mediatek/pinctrl-mt8135.c? ?| 2 +- > >? drivers/pinctrl/mediatek/pinctrl-mt8173.c? ?| 2 +- > >? > > For the pinctrl-mt81xx driver, > Acked-by: Hongzhou Yang <hongzhou.yang@mediatek.com> > > By the way, Axel has sent same patch to linux-gpio. > http://article.gmane.org/gmane.linux.kernel.gpio/6815 > > Thanks. > Hongzhou > Axel's version is better (it also improves display) but doesn't include bcm. I'll resend if necessary. Regards, Fabian ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 35/35 linux-next] pinctrl: constify of_device_id array 2015-03-19 18:41 ` Fabian Frederick @ 2015-03-19 20:55 ` Hongzhou Yang 0 siblings, 0 replies; 49+ messages in thread From: Hongzhou Yang @ 2015-03-19 20:55 UTC (permalink / raw) To: linux-arm-kernel On Thu, 2015-03-19 at 19:41 +0100, Fabian Frederick wrote: > > > On 19 March 2015 at 02:58 Hongzhou Yang <hongzhou.yang@mediatek.com> wrote: > > > > > > On Mon, 2015-03-16 at 20:59 +0100, Fabian Frederick wrote: > > > of_device_id is always used as const. > > > (See driver.of_match_table and open firmware functions) > > > > > > Signed-off-by: Fabian Frederick <fabf@skynet.be> > > > --- > > > drivers/pinctrl/bcm/pinctrl-bcm2835.c | 2 +- > > > drivers/pinctrl/mediatek/pinctrl-mt8135.c | 2 +- > > > drivers/pinctrl/mediatek/pinctrl-mt8173.c | 2 +- > > > > > > > For the pinctrl-mt81xx driver, > > Acked-by: Hongzhou Yang <hongzhou.yang@mediatek.com> > > > > By the way, Axel has sent same patch to linux-gpio. > > http://article.gmane.org/gmane.linux.kernel.gpio/6815 > > > > Thanks. > > Hongzhou > > > Axel's version is better (it also improves display) but doesn't > include bcm. I'll resend if necessary. > > Regards, > Fabian Hi Fabian, Sorry, it's my fault. Our patch only include pinctrl-mt8135.c and pinctrl-mt8173.c, bcm is not ours. Thank you. Yours, Hongzhou ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 35/35 linux-next] pinctrl: constify of_device_id array 2015-03-16 19:59 ` [PATCH 35/35 linux-next] pinctrl: " Fabian Frederick ` (3 preceding siblings ...) 2015-03-19 1:58 ` Hongzhou Yang @ 2015-03-19 7:12 ` Lee Jones 2015-03-27 9:00 ` Linus Walleij 5 siblings, 0 replies; 49+ messages in thread From: Lee Jones @ 2015-03-19 7:12 UTC (permalink / raw) To: linux-arm-kernel On Mon, 16 Mar 2015, Fabian Frederick wrote: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > > Signed-off-by: Fabian Frederick <fabf@skynet.be> > --- > drivers/pinctrl/bcm/pinctrl-bcm2835.c | 2 +- Acked-by: Lee Jones <lee@kernel.org> > drivers/pinctrl/mediatek/pinctrl-mt8135.c | 2 +- > drivers/pinctrl/mediatek/pinctrl-mt8173.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-armada-370.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-armada-375.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-armada-38x.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-armada-39x.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-armada-xp.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-kirkwood.c | 2 +- > drivers/pinctrl/mvebu/pinctrl-orion.c | 2 +- > drivers/pinctrl/pinctrl-as3722.c | 2 +- > drivers/pinctrl/pinctrl-at91.c | 4 ++-- > drivers/pinctrl/pinctrl-palmas.c | 2 +- > drivers/pinctrl/pinctrl-single.c | 4 ++-- > drivers/pinctrl/pinctrl-st.c | 2 +- > drivers/pinctrl/pinctrl-tz1090-pdc.c | 2 +- > drivers/pinctrl/pinctrl-tz1090.c | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun5i-a10s.c | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun5i-a13.c | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun6i-a31-r.c | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun6i-a31.c | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun6i-a31s.c | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun7i-a20.c | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun8i-a23-r.c | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun8i-a23.c | 2 +- > drivers/pinctrl/sunxi/pinctrl-sun9i-a80.c | 2 +- > drivers/pinctrl/vt8500/pinctrl-vt8500.c | 2 +- > drivers/pinctrl/vt8500/pinctrl-wm8505.c | 2 +- > drivers/pinctrl/vt8500/pinctrl-wm8650.c | 2 +- > drivers/pinctrl/vt8500/pinctrl-wm8750.c | 2 +- > drivers/pinctrl/vt8500/pinctrl-wm8850.c | 2 +- > 32 files changed, 34 insertions(+), 34 deletions(-) > > diff --git a/drivers/pinctrl/bcm/pinctrl-bcm2835.c b/drivers/pinctrl/bcm/pinctrl-bcm2835.c > index 9aa8a3f..4d08b85 100644 > --- a/drivers/pinctrl/bcm/pinctrl-bcm2835.c > +++ b/drivers/pinctrl/bcm/pinctrl-bcm2835.c > @@ -1051,7 +1051,7 @@ static int bcm2835_pinctrl_remove(struct platform_device *pdev) > return 0; > } > > -static struct of_device_id bcm2835_pinctrl_match[] = { > +static const struct of_device_id bcm2835_pinctrl_match[] = { > { .compatible = "brcm,bcm2835-gpio" }, > {} > }; > diff --git a/drivers/pinctrl/mediatek/pinctrl-mt8135.c b/drivers/pinctrl/mediatek/pinctrl-mt8135.c > index 1296d6d..82c4af4 100644 > --- a/drivers/pinctrl/mediatek/pinctrl-mt8135.c > +++ b/drivers/pinctrl/mediatek/pinctrl-mt8135.c > @@ -347,7 +347,7 @@ static int mt8135_pinctrl_probe(struct platform_device *pdev) > return mtk_pctrl_init(pdev, &mt8135_pinctrl_data); > } > > -static struct of_device_id mt8135_pctrl_match[] = { > +static const struct of_device_id mt8135_pctrl_match[] = { > { > .compatible = "mediatek,mt8135-pinctrl", > }, { > diff --git a/drivers/pinctrl/mediatek/pinctrl-mt8173.c b/drivers/pinctrl/mediatek/pinctrl-mt8173.c > index f07cafb..594f7b5 100644 > --- a/drivers/pinctrl/mediatek/pinctrl-mt8173.c > +++ b/drivers/pinctrl/mediatek/pinctrl-mt8173.c > @@ -427,7 +427,7 @@ static int mt8173_pinctrl_probe(struct platform_device *pdev) > return mtk_pctrl_init(pdev, &mt8173_pinctrl_data); > } > > -static struct of_device_id mt8173_pctrl_match[] = { > +static const struct of_device_id mt8173_pctrl_match[] = { > { > .compatible = "mediatek,mt8173-pinctrl", > }, { > diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-370.c b/drivers/pinctrl/mvebu/pinctrl-armada-370.c > index c4f51d0..42f930f 100644 > --- a/drivers/pinctrl/mvebu/pinctrl-armada-370.c > +++ b/drivers/pinctrl/mvebu/pinctrl-armada-370.c > @@ -379,7 +379,7 @@ static struct mvebu_mpp_mode mv88f6710_mpp_modes[] = { > > static struct mvebu_pinctrl_soc_info armada_370_pinctrl_info; > > -static struct of_device_id armada_370_pinctrl_of_match[] = { > +static const struct of_device_id armada_370_pinctrl_of_match[] = { > { .compatible = "marvell,mv88f6710-pinctrl" }, > { }, > }; > diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-375.c b/drivers/pinctrl/mvebu/pinctrl-armada-375.c > index cd7c8f5..ca1e757 100644 > --- a/drivers/pinctrl/mvebu/pinctrl-armada-375.c > +++ b/drivers/pinctrl/mvebu/pinctrl-armada-375.c > @@ -399,7 +399,7 @@ static struct mvebu_mpp_mode mv88f6720_mpp_modes[] = { > > static struct mvebu_pinctrl_soc_info armada_375_pinctrl_info; > > -static struct of_device_id armada_375_pinctrl_of_match[] = { > +static const struct of_device_id armada_375_pinctrl_of_match[] = { > { .compatible = "marvell,mv88f6720-pinctrl" }, > { }, > }; > diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-38x.c b/drivers/pinctrl/mvebu/pinctrl-armada-38x.c > index 7302f66..83bbcc7 100644 > --- a/drivers/pinctrl/mvebu/pinctrl-armada-38x.c > +++ b/drivers/pinctrl/mvebu/pinctrl-armada-38x.c > @@ -389,7 +389,7 @@ static struct mvebu_mpp_mode armada_38x_mpp_modes[] = { > > static struct mvebu_pinctrl_soc_info armada_38x_pinctrl_info; > > -static struct of_device_id armada_38x_pinctrl_of_match[] = { > +static const struct of_device_id armada_38x_pinctrl_of_match[] = { > { > .compatible = "marvell,mv88f6810-pinctrl", > .data = (void *) V_88F6810, > diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-39x.c b/drivers/pinctrl/mvebu/pinctrl-armada-39x.c > index 5963411..4249162 100644 > --- a/drivers/pinctrl/mvebu/pinctrl-armada-39x.c > +++ b/drivers/pinctrl/mvebu/pinctrl-armada-39x.c > @@ -362,7 +362,7 @@ static struct mvebu_mpp_mode armada_39x_mpp_modes[] = { > > static struct mvebu_pinctrl_soc_info armada_39x_pinctrl_info; > > -static struct of_device_id armada_39x_pinctrl_of_match[] = { > +static const struct of_device_id armada_39x_pinctrl_of_match[] = { > { > .compatible = "marvell,mv88f6920-pinctrl", > .data = (void *) V_88F6920, > diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-xp.c b/drivers/pinctrl/mvebu/pinctrl-armada-xp.c > index fc33761..7daedc1 100644 > --- a/drivers/pinctrl/mvebu/pinctrl-armada-xp.c > +++ b/drivers/pinctrl/mvebu/pinctrl-armada-xp.c > @@ -361,7 +361,7 @@ static struct mvebu_mpp_mode armada_xp_mpp_modes[] = { > > static struct mvebu_pinctrl_soc_info armada_xp_pinctrl_info; > > -static struct of_device_id armada_xp_pinctrl_of_match[] = { > +static const struct of_device_id armada_xp_pinctrl_of_match[] = { > { > .compatible = "marvell,mv78230-pinctrl", > .data = (void *) V_MV78230, > diff --git a/drivers/pinctrl/mvebu/pinctrl-kirkwood.c b/drivers/pinctrl/mvebu/pinctrl-kirkwood.c > index dbc673c..0f07dc5 100644 > --- a/drivers/pinctrl/mvebu/pinctrl-kirkwood.c > +++ b/drivers/pinctrl/mvebu/pinctrl-kirkwood.c > @@ -456,7 +456,7 @@ static struct mvebu_pinctrl_soc_info mv98dx4122_info = { > .ngpioranges = ARRAY_SIZE(mv88f628x_gpio_ranges), > }; > > -static struct of_device_id kirkwood_pinctrl_of_match[] = { > +static const struct of_device_id kirkwood_pinctrl_of_match[] = { > { .compatible = "marvell,88f6180-pinctrl", .data = &mv88f6180_info }, > { .compatible = "marvell,88f6190-pinctrl", .data = &mv88f6190_info }, > { .compatible = "marvell,88f6192-pinctrl", .data = &mv88f6192_info }, > diff --git a/drivers/pinctrl/mvebu/pinctrl-orion.c b/drivers/pinctrl/mvebu/pinctrl-orion.c > index 3a632ef..3b7122d 100644 > --- a/drivers/pinctrl/mvebu/pinctrl-orion.c > +++ b/drivers/pinctrl/mvebu/pinctrl-orion.c > @@ -211,7 +211,7 @@ static struct mvebu_pinctrl_soc_info mv88f5281_info = { > * There are multiple variants of the Orion SoCs, but in terms of pin > * muxing, they are identical. > */ > -static struct of_device_id orion_pinctrl_of_match[] = { > +static const struct of_device_id orion_pinctrl_of_match[] = { > { .compatible = "marvell,88f5181l-pinctrl", .data = &mv88f5181l_info }, > { .compatible = "marvell,88f5182-pinctrl", .data = &mv88f5182_info }, > { .compatible = "marvell,88f5281-pinctrl", .data = &mv88f5281_info }, > diff --git a/drivers/pinctrl/pinctrl-as3722.c b/drivers/pinctrl/pinctrl-as3722.c > index 169b1bf..db0571f 100644 > --- a/drivers/pinctrl/pinctrl-as3722.c > +++ b/drivers/pinctrl/pinctrl-as3722.c > @@ -625,7 +625,7 @@ static int as3722_pinctrl_remove(struct platform_device *pdev) > return 0; > } > > -static struct of_device_id as3722_pinctrl_of_match[] = { > +static const struct of_device_id as3722_pinctrl_of_match[] = { > { .compatible = "ams,as3722-pinctrl", }, > { }, > }; > diff --git a/drivers/pinctrl/pinctrl-at91.c b/drivers/pinctrl/pinctrl-at91.c > index 6f1eba3..6b32817 100644 > --- a/drivers/pinctrl/pinctrl-at91.c > +++ b/drivers/pinctrl/pinctrl-at91.c > @@ -1114,7 +1114,7 @@ static int at91_pinctrl_parse_functions(struct device_node *np, > return 0; > } > > -static struct of_device_id at91_pinctrl_of_match[] = { > +static const struct of_device_id at91_pinctrl_of_match[] = { > { .compatible = "atmel,sama5d3-pinctrl", .data = &sama5d3_ops }, > { .compatible = "atmel,at91sam9x5-pinctrl", .data = &at91sam9x5_ops }, > { .compatible = "atmel,at91rm9200-pinctrl", .data = &at91rm9200_ops }, > @@ -1690,7 +1690,7 @@ static struct gpio_chip at91_gpio_template = { > .ngpio = MAX_NB_GPIO_PER_BANK, > }; > > -static struct of_device_id at91_gpio_of_match[] = { > +static const struct of_device_id at91_gpio_of_match[] = { > { .compatible = "atmel,at91sam9x5-gpio", .data = &at91sam9x5_ops, }, > { .compatible = "atmel,at91rm9200-gpio", .data = &at91rm9200_ops }, > { /* sentinel */ } > diff --git a/drivers/pinctrl/pinctrl-palmas.c b/drivers/pinctrl/pinctrl-palmas.c > index 26461e3..2631df0 100644 > --- a/drivers/pinctrl/pinctrl-palmas.c > +++ b/drivers/pinctrl/pinctrl-palmas.c > @@ -987,7 +987,7 @@ static struct palmas_pinctrl_data tps80036_pinctrl_data = { > .num_pin_groups = ARRAY_SIZE(tps80036_pingroups), > }; > > -static struct of_device_id palmas_pinctrl_of_match[] = { > +static const struct of_device_id palmas_pinctrl_of_match[] = { > { .compatible = "ti,palmas-pinctrl", .data = &tps65913_pinctrl_data}, > { .compatible = "ti,tps65913-pinctrl", .data = &tps65913_pinctrl_data}, > { .compatible = "ti,tps80036-pinctrl", .data = &tps80036_pinctrl_data}, > diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c > index 69e8442..13b45f2 100644 > --- a/drivers/pinctrl/pinctrl-single.c > +++ b/drivers/pinctrl/pinctrl-single.c > @@ -1501,7 +1501,7 @@ static void pcs_free_resources(struct pcs_device *pcs) > } \ > } while (0); > > -static struct of_device_id pcs_of_match[]; > +static const struct of_device_id pcs_of_match[]; > > static int pcs_add_gpio_func(struct device_node *node, struct pcs_device *pcs) > { > @@ -2000,7 +2000,7 @@ static const struct pcs_soc_data pinconf_single = { > .flags = PCS_FEAT_PINCONF, > }; > > -static struct of_device_id pcs_of_match[] = { > +static const struct of_device_id pcs_of_match[] = { > { .compatible = "ti,omap3-padconf", .data = &pinctrl_single_omap_wkup }, > { .compatible = "ti,omap4-padconf", .data = &pinctrl_single_omap_wkup }, > { .compatible = "ti,omap5-padconf", .data = &pinctrl_single_omap_wkup }, > diff --git a/drivers/pinctrl/pinctrl-st.c b/drivers/pinctrl/pinctrl-st.c > index 9e5ec00..8442202 100644 > --- a/drivers/pinctrl/pinctrl-st.c > +++ b/drivers/pinctrl/pinctrl-st.c > @@ -1531,7 +1531,7 @@ static int st_gpiolib_register_bank(struct st_pinctrl *info, > return 0; > } > > -static struct of_device_id st_pctl_of_match[] = { > +static const struct of_device_id st_pctl_of_match[] = { > { .compatible = "st,stih415-sbc-pinctrl", .data = &stih415_sbc_data }, > { .compatible = "st,stih415-rear-pinctrl", .data = &stih415_rear_data }, > { .compatible = "st,stih415-left-pinctrl", .data = &stih415_left_data }, > diff --git a/drivers/pinctrl/pinctrl-tz1090-pdc.c b/drivers/pinctrl/pinctrl-tz1090-pdc.c > index fab6aaf..8a8911b 100644 > --- a/drivers/pinctrl/pinctrl-tz1090-pdc.c > +++ b/drivers/pinctrl/pinctrl-tz1090-pdc.c > @@ -969,7 +969,7 @@ static int tz1090_pdc_pinctrl_remove(struct platform_device *pdev) > return 0; > } > > -static struct of_device_id tz1090_pdc_pinctrl_of_match[] = { > +static const struct of_device_id tz1090_pdc_pinctrl_of_match[] = { > { .compatible = "img,tz1090-pdc-pinctrl", }, > { }, > }; > diff --git a/drivers/pinctrl/pinctrl-tz1090.c b/drivers/pinctrl/pinctrl-tz1090.c > index 8bd73075..fc5594a5 100644 > --- a/drivers/pinctrl/pinctrl-tz1090.c > +++ b/drivers/pinctrl/pinctrl-tz1090.c > @@ -1984,7 +1984,7 @@ static int tz1090_pinctrl_remove(struct platform_device *pdev) > return 0; > } > > -static struct of_device_id tz1090_pinctrl_of_match[] = { > +static const struct of_device_id tz1090_pinctrl_of_match[] = { > { .compatible = "img,tz1090-pinctrl", }, > { }, > }; > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c b/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c > index 24c5d88..37aa2a8 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c > @@ -1019,7 +1019,7 @@ static int sun4i_a10_pinctrl_probe(struct platform_device *pdev) > &sun4i_a10_pinctrl_data); > } > > -static struct of_device_id sun4i_a10_pinctrl_match[] = { > +static const struct of_device_id sun4i_a10_pinctrl_match[] = { > { .compatible = "allwinner,sun4i-a10-pinctrl", }, > {} > }; > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun5i-a10s.c b/drivers/pinctrl/sunxi/pinctrl-sun5i-a10s.c > index 45a351a..6367661 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun5i-a10s.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun5i-a10s.c > @@ -670,7 +670,7 @@ static int sun5i_a10s_pinctrl_probe(struct platform_device *pdev) > &sun5i_a10s_pinctrl_data); > } > > -static struct of_device_id sun5i_a10s_pinctrl_match[] = { > +static const struct of_device_id sun5i_a10s_pinctrl_match[] = { > { .compatible = "allwinner,sun5i-a10s-pinctrl", }, > {} > }; > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun5i-a13.c b/drivers/pinctrl/sunxi/pinctrl-sun5i-a13.c > index 4bd2347..2bb07b3 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun5i-a13.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun5i-a13.c > @@ -388,7 +388,7 @@ static int sun5i_a13_pinctrl_probe(struct platform_device *pdev) > &sun5i_a13_pinctrl_data); > } > > -static struct of_device_id sun5i_a13_pinctrl_match[] = { > +static const struct of_device_id sun5i_a13_pinctrl_match[] = { > { .compatible = "allwinner,sun5i-a13-pinctrl", }, > {} > }; > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun6i-a31-r.c b/drivers/pinctrl/sunxi/pinctrl-sun6i-a31-r.c > index 02174fa..9596b0a3 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun6i-a31-r.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun6i-a31-r.c > @@ -120,7 +120,7 @@ static int sun6i_a31_r_pinctrl_probe(struct platform_device *pdev) > return ret; > } > > -static struct of_device_id sun6i_a31_r_pinctrl_match[] = { > +static const struct of_device_id sun6i_a31_r_pinctrl_match[] = { > { .compatible = "allwinner,sun6i-a31-r-pinctrl", }, > {} > }; > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun6i-a31.c b/drivers/pinctrl/sunxi/pinctrl-sun6i-a31.c > index 18038f0..022863a 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun6i-a31.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun6i-a31.c > @@ -922,7 +922,7 @@ static int sun6i_a31_pinctrl_probe(struct platform_device *pdev) > &sun6i_a31_pinctrl_data); > } > > -static struct of_device_id sun6i_a31_pinctrl_match[] = { > +static const struct of_device_id sun6i_a31_pinctrl_match[] = { > { .compatible = "allwinner,sun6i-a31-pinctrl", }, > {} > }; > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun6i-a31s.c b/drivers/pinctrl/sunxi/pinctrl-sun6i-a31s.c > index 9b5a91f..d3725dc 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun6i-a31s.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun6i-a31s.c > @@ -794,7 +794,7 @@ static int sun6i_a31s_pinctrl_probe(struct platform_device *pdev) > &sun6i_a31s_pinctrl_data); > } > > -static struct of_device_id sun6i_a31s_pinctrl_match[] = { > +static const struct of_device_id sun6i_a31s_pinctrl_match[] = { > { .compatible = "allwinner,sun6i-a31s-pinctrl", }, > {} > }; > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun7i-a20.c b/drivers/pinctrl/sunxi/pinctrl-sun7i-a20.c > index 6af6cc8..cf1ce0c 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun7i-a20.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun7i-a20.c > @@ -1045,7 +1045,7 @@ static int sun7i_a20_pinctrl_probe(struct platform_device *pdev) > &sun7i_a20_pinctrl_data); > } > > -static struct of_device_id sun7i_a20_pinctrl_match[] = { > +static const struct of_device_id sun7i_a20_pinctrl_match[] = { > { .compatible = "allwinner,sun7i-a20-pinctrl", }, > {} > }; > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun8i-a23-r.c b/drivers/pinctrl/sunxi/pinctrl-sun8i-a23-r.c > index 327e03f..0562876 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun8i-a23-r.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun8i-a23-r.c > @@ -119,7 +119,7 @@ static int sun8i_a23_r_pinctrl_probe(struct platform_device *pdev) > return ret; > } > > -static struct of_device_id sun8i_a23_r_pinctrl_match[] = { > +static const struct of_device_id sun8i_a23_r_pinctrl_match[] = { > { .compatible = "allwinner,sun8i-a23-r-pinctrl", }, > {} > }; > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun8i-a23.c b/drivers/pinctrl/sunxi/pinctrl-sun8i-a23.c > index 62695c9..55083d2 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun8i-a23.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun8i-a23.c > @@ -571,7 +571,7 @@ static int sun8i_a23_pinctrl_probe(struct platform_device *pdev) > &sun8i_a23_pinctrl_data); > } > > -static struct of_device_id sun8i_a23_pinctrl_match[] = { > +static const struct of_device_id sun8i_a23_pinctrl_match[] = { > { .compatible = "allwinner,sun8i-a23-pinctrl", }, > {} > }; > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun9i-a80.c b/drivers/pinctrl/sunxi/pinctrl-sun9i-a80.c > index adb2942..1b580ba 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun9i-a80.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun9i-a80.c > @@ -729,7 +729,7 @@ static int sun9i_a80_pinctrl_probe(struct platform_device *pdev) > &sun9i_a80_pinctrl_data); > } > > -static struct of_device_id sun9i_a80_pinctrl_match[] = { > +static const struct of_device_id sun9i_a80_pinctrl_match[] = { > { .compatible = "allwinner,sun9i-a80-pinctrl", }, > {} > }; > diff --git a/drivers/pinctrl/vt8500/pinctrl-vt8500.c b/drivers/pinctrl/vt8500/pinctrl-vt8500.c > index cf8bbc9..ca946b3 100644 > --- a/drivers/pinctrl/vt8500/pinctrl-vt8500.c > +++ b/drivers/pinctrl/vt8500/pinctrl-vt8500.c > @@ -478,7 +478,7 @@ static int vt8500_pinctrl_remove(struct platform_device *pdev) > return wmt_pinctrl_remove(pdev); > } > > -static struct of_device_id wmt_pinctrl_of_match[] = { > +static const struct of_device_id wmt_pinctrl_of_match[] = { > { .compatible = "via,vt8500-pinctrl" }, > { /* sentinel */ }, > }; > diff --git a/drivers/pinctrl/vt8500/pinctrl-wm8505.c b/drivers/pinctrl/vt8500/pinctrl-wm8505.c > index 3f9c32d..626fc7e 100644 > --- a/drivers/pinctrl/vt8500/pinctrl-wm8505.c > +++ b/drivers/pinctrl/vt8500/pinctrl-wm8505.c > @@ -509,7 +509,7 @@ static int wm8505_pinctrl_remove(struct platform_device *pdev) > return wmt_pinctrl_remove(pdev); > } > > -static struct of_device_id wmt_pinctrl_of_match[] = { > +static const struct of_device_id wmt_pinctrl_of_match[] = { > { .compatible = "wm,wm8505-pinctrl" }, > { /* sentinel */ }, > }; > diff --git a/drivers/pinctrl/vt8500/pinctrl-wm8650.c b/drivers/pinctrl/vt8500/pinctrl-wm8650.c > index 4e80f98..8953aba 100644 > --- a/drivers/pinctrl/vt8500/pinctrl-wm8650.c > +++ b/drivers/pinctrl/vt8500/pinctrl-wm8650.c > @@ -347,7 +347,7 @@ static int wm8650_pinctrl_remove(struct platform_device *pdev) > return wmt_pinctrl_remove(pdev); > } > > -static struct of_device_id wmt_pinctrl_of_match[] = { > +static const struct of_device_id wmt_pinctrl_of_match[] = { > { .compatible = "wm,wm8650-pinctrl" }, > { /* sentinel */ }, > }; > diff --git a/drivers/pinctrl/vt8500/pinctrl-wm8750.c b/drivers/pinctrl/vt8500/pinctrl-wm8750.c > index 47b52a7..c79053d 100644 > --- a/drivers/pinctrl/vt8500/pinctrl-wm8750.c > +++ b/drivers/pinctrl/vt8500/pinctrl-wm8750.c > @@ -386,7 +386,7 @@ static int wm8750_pinctrl_remove(struct platform_device *pdev) > return wmt_pinctrl_remove(pdev); > } > > -static struct of_device_id wmt_pinctrl_of_match[] = { > +static const struct of_device_id wmt_pinctrl_of_match[] = { > { .compatible = "wm,wm8750-pinctrl" }, > { /* sentinel */ }, > }; > diff --git a/drivers/pinctrl/vt8500/pinctrl-wm8850.c b/drivers/pinctrl/vt8500/pinctrl-wm8850.c > index 8bbb38c..f232b16 100644 > --- a/drivers/pinctrl/vt8500/pinctrl-wm8850.c > +++ b/drivers/pinctrl/vt8500/pinctrl-wm8850.c > @@ -365,7 +365,7 @@ static int wm8850_pinctrl_remove(struct platform_device *pdev) > return wmt_pinctrl_remove(pdev); > } > > -static struct of_device_id wmt_pinctrl_of_match[] = { > +static const struct of_device_id wmt_pinctrl_of_match[] = { > { .compatible = "wm,wm8850-pinctrl" }, > { /* sentinel */ }, > }; ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 35/35 linux-next] pinctrl: constify of_device_id array 2015-03-16 19:59 ` [PATCH 35/35 linux-next] pinctrl: " Fabian Frederick ` (4 preceding siblings ...) 2015-03-19 7:12 ` Lee Jones @ 2015-03-27 9:00 ` Linus Walleij 5 siblings, 0 replies; 49+ messages in thread From: Linus Walleij @ 2015-03-27 9:00 UTC (permalink / raw) To: linux-arm-kernel On Mon, Mar 16, 2015 at 8:59 PM, Fabian Frederick <fabf@skynet.be> wrote: > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > > Signed-off-by: Fabian Frederick <fabf@skynet.be> Patch applied with the ACKs. Had to do some fixup on some MTK controllers, no big deal. Yours, Linus Walleij ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 32/35 linux-next] clk: constify of_device_id array 2015-03-16 19:59 ` [PATCH 32/35 linux-next] clk: " Fabian Frederick 2015-03-16 19:59 ` [PATCH 33/35 linux-next] mmc: " Fabian Frederick 2015-03-16 19:59 ` [PATCH 35/35 linux-next] pinctrl: " Fabian Frederick @ 2015-03-18 14:15 ` Michael Turquette 2015-03-22 10:34 ` Fabian Frederick 2 siblings, 1 reply; 49+ messages in thread From: Michael Turquette @ 2015-03-18 14:15 UTC (permalink / raw) To: linux-arm-kernel Quoting Fabian Frederick (2015-03-16 12:59:06) > of_device_id is always used as const. > (See driver.of_match_table and open firmware functions) > > Signed-off-by: Fabian Frederick <fabf@skynet.be> Acked-by: Michael Turquette <mturquette@linaro.org> > --- > drivers/clk/clk-palmas.c | 2 +- > drivers/clk/st/clkgen-fsyn.c | 2 +- > drivers/clk/st/clkgen-mux.c | 8 ++++---- > drivers/clk/st/clkgen-pll.c | 4 ++-- > drivers/clk/ti/clk-dra7-atl.c | 2 +- > drivers/clk/ti/clockdomain.c | 2 +- > drivers/clk/versatile/clk-vexpress-osc.c | 2 +- > 7 files changed, 11 insertions(+), 11 deletions(-) > > diff --git a/drivers/clk/clk-palmas.c b/drivers/clk/clk-palmas.c > index 8d45992..45a535a 100644 > --- a/drivers/clk/clk-palmas.c > +++ b/drivers/clk/clk-palmas.c > @@ -161,7 +161,7 @@ static struct palmas_clks_of_match_data palmas_of_clk32kgaudio = { > }, > }; > > -static struct of_device_id palmas_clks_of_match[] = { > +static const struct of_device_id palmas_clks_of_match[] = { > { > .compatible = "ti,palmas-clk32kg", > .data = &palmas_of_clk32kg, > diff --git a/drivers/clk/st/clkgen-fsyn.c b/drivers/clk/st/clkgen-fsyn.c > index af94ed8..a917c4c 100644 > --- a/drivers/clk/st/clkgen-fsyn.c > +++ b/drivers/clk/st/clkgen-fsyn.c > @@ -1057,7 +1057,7 @@ static struct clk * __init st_clk_register_quadfs_fsynth( > return clk; > } > > -static struct of_device_id quadfs_of_match[] = { > +static const struct of_device_id quadfs_of_match[] = { > { > .compatible = "st,stih416-quadfs216", > .data = &st_fs216c65_416 > diff --git a/drivers/clk/st/clkgen-mux.c b/drivers/clk/st/clkgen-mux.c > index 9a15ec3..fdcff10 100644 > --- a/drivers/clk/st/clkgen-mux.c > +++ b/drivers/clk/st/clkgen-mux.c > @@ -341,7 +341,7 @@ static struct clkgena_divmux_data st_divmux_c32odf3 = { > .fb_start_bit_idx = 24, > }; > > -static struct of_device_id clkgena_divmux_of_match[] = { > +static const struct of_device_id clkgena_divmux_of_match[] = { > { > .compatible = "st,clkgena-divmux-c65-hs", > .data = &st_divmux_c65hs, > @@ -479,7 +479,7 @@ static struct clkgena_prediv_data prediv_c32_data = { > .table = prediv_table16, > }; > > -static struct of_device_id clkgena_prediv_of_match[] = { > +static const struct of_device_id clkgena_prediv_of_match[] = { > { .compatible = "st,clkgena-prediv-c65", .data = &prediv_c65_data }, > { .compatible = "st,clkgena-prediv-c32", .data = &prediv_c32_data }, > {} > @@ -586,7 +586,7 @@ static struct clkgen_mux_data stih407_a9_mux_data = { > .width = 2, > }; > > -static struct of_device_id mux_of_match[] = { > +static const struct of_device_id mux_of_match[] = { > { > .compatible = "st,stih416-clkgenc-vcc-hd", > .data = &clkgen_mux_c_vcc_hd_416, > @@ -693,7 +693,7 @@ static struct clkgen_vcc_data st_clkgenf_vcc_416 = { > .lock = &clkgenf_lock, > }; > > -static struct of_device_id vcc_of_match[] = { > +static const struct of_device_id vcc_of_match[] = { > { .compatible = "st,stih416-clkgenc", .data = &st_clkgenc_vcc_416 }, > { .compatible = "st,stih416-clkgenf", .data = &st_clkgenf_vcc_416 }, > {} > diff --git a/drivers/clk/st/clkgen-pll.c b/drivers/clk/st/clkgen-pll.c > index 29769d7..d204ba8 100644 > --- a/drivers/clk/st/clkgen-pll.c > +++ b/drivers/clk/st/clkgen-pll.c > @@ -593,7 +593,7 @@ static struct clk * __init clkgen_odf_register(const char *parent_name, > return clk; > } > > -static struct of_device_id c32_pll_of_match[] = { > +static const struct of_device_id c32_pll_of_match[] = { > { > .compatible = "st,plls-c32-a1x-0", > .data = &st_pll3200c32_a1x_0, > @@ -708,7 +708,7 @@ err: > } > CLK_OF_DECLARE(clkgen_c32_pll, "st,clkgen-plls-c32", clkgen_c32_pll_setup); > > -static struct of_device_id c32_gpu_pll_of_match[] = { > +static const struct of_device_id c32_gpu_pll_of_match[] = { > { > .compatible = "st,stih415-gpu-pll-c32", > .data = &st_pll1200c32_gpu_415, > diff --git a/drivers/clk/ti/clk-dra7-atl.c b/drivers/clk/ti/clk-dra7-atl.c > index 59bb4b3..d86bc46 100644 > --- a/drivers/clk/ti/clk-dra7-atl.c > +++ b/drivers/clk/ti/clk-dra7-atl.c > @@ -294,7 +294,7 @@ static int of_dra7_atl_clk_remove(struct platform_device *pdev) > return 0; > } > > -static struct of_device_id of_dra7_atl_clk_match_tbl[] = { > +static const struct of_device_id of_dra7_atl_clk_match_tbl[] = { > { .compatible = "ti,dra7-atl", }, > {}, > }; > diff --git a/drivers/clk/ti/clockdomain.c b/drivers/clk/ti/clockdomain.c > index b4c5fac..35fe108 100644 > --- a/drivers/clk/ti/clockdomain.c > +++ b/drivers/clk/ti/clockdomain.c > @@ -52,7 +52,7 @@ static void __init of_ti_clockdomain_setup(struct device_node *node) > } > } > > -static struct of_device_id ti_clkdm_match_table[] __initdata = { > +static const struct of_device_id ti_clkdm_match_table[] __initconst = { > { .compatible = "ti,clockdomain" }, > { } > }; > diff --git a/drivers/clk/versatile/clk-vexpress-osc.c b/drivers/clk/versatile/clk-vexpress-osc.c > index 765f1e0..89c0609 100644 > --- a/drivers/clk/versatile/clk-vexpress-osc.c > +++ b/drivers/clk/versatile/clk-vexpress-osc.c > @@ -110,7 +110,7 @@ static int vexpress_osc_probe(struct platform_device *pdev) > return 0; > } > > -static struct of_device_id vexpress_osc_of_match[] = { > +static const struct of_device_id vexpress_osc_of_match[] = { > { .compatible = "arm,vexpress-osc", }, > {} > }; > -- > 2.1.0 > ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 32/35 linux-next] clk: constify of_device_id array 2015-03-18 14:15 ` [PATCH 32/35 linux-next] clk: " Michael Turquette @ 2015-03-22 10:34 ` Fabian Frederick 2015-03-27 7:19 ` Stephen Boyd 0 siblings, 1 reply; 49+ messages in thread From: Fabian Frederick @ 2015-03-22 10:34 UTC (permalink / raw) To: linux-arm-kernel > On 18 March 2015 at 15:15 Michael Turquette <mturquette@linaro.org> wrote: > > > Quoting Fabian Frederick (2015-03-16 12:59:06) > > of_device_id is always used as const. > > (See driver.of_match_table and open firmware functions) > > > > Signed-off-by: Fabian Frederick <fabf@skynet.be> > > Acked-by: Michael Turquette <mturquette@linaro.org> Thanks :) btw, something I forgot to mention in changelog is the __initdata -> __initconst for ti_clkdm_match_table[] I can send it again with a new changelog if necessary ... Regards, Fabian > > > --- > >? drivers/clk/clk-palmas.c? ? ? ? ? ? ? ? ?| 2 +- > >? drivers/clk/st/clkgen-fsyn.c? ? ? ? ? ? ?| 2 +- > >? drivers/clk/st/clkgen-mux.c? ? ? ? ? ? ? | 8 ++++---- > >? drivers/clk/st/clkgen-pll.c? ? ? ? ? ? ? | 4 ++-- > >? drivers/clk/ti/clk-dra7-atl.c? ? ? ? ? ? | 2 +- > >? drivers/clk/ti/clockdomain.c? ? ? ? ? ? ?| 2 +- > >? drivers/clk/versatile/clk-vexpress-osc.c | 2 +- > >? 7 files changed, 11 insertions(+), 11 deletions(-) > > > > diff --git a/drivers/clk/clk-palmas.c b/drivers/clk/clk-palmas.c > > index 8d45992..45a535a 100644 > > --- a/drivers/clk/clk-palmas.c > > +++ b/drivers/clk/clk-palmas.c > > @@ -161,7 +161,7 @@ static struct palmas_clks_of_match_data > > palmas_of_clk32kgaudio = { > >? ? ? ? ?}, > >? }; > >? > > -static struct of_device_id palmas_clks_of_match[] = { > > +static const struct of_device_id palmas_clks_of_match[] = { > >? ? ? ? ?{ > >? ? ? ? ? ? ? ? ?.compatible = "ti,palmas-clk32kg", > >? ? ? ? ? ? ? ? ?.data = &palmas_of_clk32kg, > > diff --git a/drivers/clk/st/clkgen-fsyn.c b/drivers/clk/st/clkgen-fsyn.c > > index af94ed8..a917c4c 100644 > > --- a/drivers/clk/st/clkgen-fsyn.c > > +++ b/drivers/clk/st/clkgen-fsyn.c > > @@ -1057,7 +1057,7 @@ static struct clk * __init > > st_clk_register_quadfs_fsynth( > >? ? ? ? ?return clk; > >? } > >? > > -static struct of_device_id quadfs_of_match[] = { > > +static const struct of_device_id quadfs_of_match[] = { > >? ? ? ? ?{ > >? ? ? ? ? ? ? ? ?.compatible = "st,stih416-quadfs216", > >? ? ? ? ? ? ? ? ?.data = &st_fs216c65_416 > > diff --git a/drivers/clk/st/clkgen-mux.c b/drivers/clk/st/clkgen-mux.c > > index 9a15ec3..fdcff10 100644 > > --- a/drivers/clk/st/clkgen-mux.c > > +++ b/drivers/clk/st/clkgen-mux.c > > @@ -341,7 +341,7 @@ static struct clkgena_divmux_data st_divmux_c32odf3 = { > >? ? ? ? ?.fb_start_bit_idx = 24, > >? }; > >? > > -static struct of_device_id clkgena_divmux_of_match[] = { > > +static const struct of_device_id clkgena_divmux_of_match[] = { > >? ? ? ? ?{ > >? ? ? ? ? ? ? ? ?.compatible = "st,clkgena-divmux-c65-hs", > >? ? ? ? ? ? ? ? ?.data = &st_divmux_c65hs, > > @@ -479,7 +479,7 @@ static struct clkgena_prediv_data prediv_c32_data = { > >? ? ? ? ?.table = prediv_table16, > >? }; > >? > > -static struct of_device_id clkgena_prediv_of_match[] = { > > +static const struct of_device_id clkgena_prediv_of_match[] = { > >? ? ? ? ?{ .compatible = "st,clkgena-prediv-c65", .data = &prediv_c65_data }, > >? ? ? ? ?{ .compatible = "st,clkgena-prediv-c32", .data = &prediv_c32_data }, > >? ? ? ? ?{} > > @@ -586,7 +586,7 @@ static struct clkgen_mux_data stih407_a9_mux_data = { > >? ? ? ? ?.width = 2, > >? }; > >? > > -static struct of_device_id mux_of_match[] = { > > +static const struct of_device_id mux_of_match[] = { > >? ? ? ? ?{ > >? ? ? ? ? ? ? ? ?.compatible = "st,stih416-clkgenc-vcc-hd", > >? ? ? ? ? ? ? ? ?.data = &clkgen_mux_c_vcc_hd_416, > > @@ -693,7 +693,7 @@ static struct clkgen_vcc_data st_clkgenf_vcc_416 = { > >? ? ? ? ?.lock = &clkgenf_lock, > >? }; > >? > > -static struct of_device_id vcc_of_match[] = { > > +static const struct of_device_id vcc_of_match[] = { > >? ? ? ? ?{ .compatible = "st,stih416-clkgenc", .data = &st_clkgenc_vcc_416 }, > >? ? ? ? ?{ .compatible = "st,stih416-clkgenf", .data = &st_clkgenf_vcc_416 }, > >? ? ? ? ?{} > > diff --git a/drivers/clk/st/clkgen-pll.c b/drivers/clk/st/clkgen-pll.c > > index 29769d7..d204ba8 100644 > > --- a/drivers/clk/st/clkgen-pll.c > > +++ b/drivers/clk/st/clkgen-pll.c > > @@ -593,7 +593,7 @@ static struct clk * __init clkgen_odf_register(const > > char *parent_name, > >? ? ? ? ?return clk; > >? } > >? > > -static struct of_device_id c32_pll_of_match[] = { > > +static const struct of_device_id c32_pll_of_match[] = { > >? ? ? ? ?{ > >? ? ? ? ? ? ? ? ?.compatible = "st,plls-c32-a1x-0", > >? ? ? ? ? ? ? ? ?.data = &st_pll3200c32_a1x_0, > > @@ -708,7 +708,7 @@ err: > >? } > >? CLK_OF_DECLARE(clkgen_c32_pll, "st,clkgen-plls-c32", clkgen_c32_pll_setup); > >? > > -static struct of_device_id c32_gpu_pll_of_match[] = { > > +static const struct of_device_id c32_gpu_pll_of_match[] = { > >? ? ? ? ?{ > >? ? ? ? ? ? ? ? ?.compatible = "st,stih415-gpu-pll-c32", > >? ? ? ? ? ? ? ? ?.data = &st_pll1200c32_gpu_415, > > diff --git a/drivers/clk/ti/clk-dra7-atl.c b/drivers/clk/ti/clk-dra7-atl.c > > index 59bb4b3..d86bc46 100644 > > --- a/drivers/clk/ti/clk-dra7-atl.c > > +++ b/drivers/clk/ti/clk-dra7-atl.c > > @@ -294,7 +294,7 @@ static int of_dra7_atl_clk_remove(struct platform_device > > *pdev) > >? ? ? ? ?return 0; > >? } > >? > > -static struct of_device_id of_dra7_atl_clk_match_tbl[] = { > > +static const struct of_device_id of_dra7_atl_clk_match_tbl[] = { > >? ? ? ? ?{ .compatible = "ti,dra7-atl", }, > >? ? ? ? ?{}, > >? }; > > diff --git a/drivers/clk/ti/clockdomain.c b/drivers/clk/ti/clockdomain.c > > index b4c5fac..35fe108 100644 > > --- a/drivers/clk/ti/clockdomain.c > > +++ b/drivers/clk/ti/clockdomain.c > > @@ -52,7 +52,7 @@ static void __init of_ti_clockdomain_setup(struct > > device_node *node) > >? ? ? ? ?} > >? } > >? > > -static struct of_device_id ti_clkdm_match_table[] __initdata = { > > +static const struct of_device_id ti_clkdm_match_table[] __initconst = { > >? ? ? ? ?{ .compatible = "ti,clockdomain" }, > >? ? ? ? ?{ } > >? }; > > diff --git a/drivers/clk/versatile/clk-vexpress-osc.c > > b/drivers/clk/versatile/clk-vexpress-osc.c > > index 765f1e0..89c0609 100644 > > --- a/drivers/clk/versatile/clk-vexpress-osc.c > > +++ b/drivers/clk/versatile/clk-vexpress-osc.c > > @@ -110,7 +110,7 @@ static int vexpress_osc_probe(struct platform_device > > *pdev) > >? ? ? ? ?return 0; > >? } > >? > > -static struct of_device_id vexpress_osc_of_match[] = { > > +static const struct of_device_id vexpress_osc_of_match[] = { > >? ? ? ? ?{ .compatible = "arm,vexpress-osc", }, > >? ? ? ? ?{} > >? }; > > -- > > 2.1.0 > > ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 32/35 linux-next] clk: constify of_device_id array 2015-03-22 10:34 ` Fabian Frederick @ 2015-03-27 7:19 ` Stephen Boyd 2015-03-31 19:05 ` Fabian Frederick 0 siblings, 1 reply; 49+ messages in thread From: Stephen Boyd @ 2015-03-27 7:19 UTC (permalink / raw) To: linux-arm-kernel On 03/22, Fabian Frederick wrote: > > > > On 18 March 2015 at 15:15 Michael Turquette <mturquette@linaro.org> wrote: > > > > > > Quoting Fabian Frederick (2015-03-16 12:59:06) > > > of_device_id is always used as const. > > > (See driver.of_match_table and open firmware functions) > > > > > > Signed-off-by: Fabian Frederick <fabf@skynet.be> > > > > Acked-by: Michael Turquette <mturquette@linaro.org> > > Thanks :) > > btw, something I forgot to mention in changelog is the __initdata -> __initconst > for ti_clkdm_match_table[] > > I can send it again with a new changelog if necessary ... > Did you want us to take this through clk-next? If so please resend with the new commit text. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project ^ permalink raw reply [flat|nested] 49+ messages in thread
* [PATCH 32/35 linux-next] clk: constify of_device_id array 2015-03-27 7:19 ` Stephen Boyd @ 2015-03-31 19:05 ` Fabian Frederick 0 siblings, 0 replies; 49+ messages in thread From: Fabian Frederick @ 2015-03-31 19:05 UTC (permalink / raw) To: linux-arm-kernel > On 27 March 2015 at 08:19 Stephen Boyd <sboyd@codeaurora.org> wrote: > > > On 03/22, Fabian Frederick wrote: > > > > > > > On 18 March 2015 at 15:15 Michael Turquette <mturquette@linaro.org> wrote: > > > > > > > > > Quoting Fabian Frederick (2015-03-16 12:59:06) > > > > of_device_id is always used as const. > > > > (See driver.of_match_table and open firmware functions) > > > > > > > > Signed-off-by: Fabian Frederick <fabf@skynet.be> > > > > > > Acked-by: Michael Turquette <mturquette@linaro.org> > > > > Thanks :) > > > > btw, something I forgot to mention in changelog is the __initdata -> > > __initconst > > for ti_clkdm_match_table[] > > > > I can send it again with a new changelog if necessary ... > > > > Did you want us to take this through clk-next? If so please > resend with the new commit text. Of course :) I hope V2 is ok. Regards, Fabian > > -- > Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, > a Linux Foundation Collaborative Project ^ permalink raw reply [flat|nested] 49+ messages in thread
end of thread, other threads:[~2015-03-31 19:05 UTC | newest] Thread overview: 49+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2015-03-16 19:17 [PATCH 00/35 linux-next] constify of_device_id array Fabian Frederick 2015-03-16 19:17 ` [PATCH 05/35 linux-next] tty: " Fabian Frederick 2015-03-16 19:20 ` Timur Tabi 2015-03-16 19:51 ` Peter Korsgaard 2015-03-17 8:10 ` Patrice Chotard 2015-03-16 19:17 ` [PATCH 06/35 linux-next] power: " Fabian Frederick 2015-03-17 11:38 ` Sudeep Holla 2015-03-20 12:32 ` Sebastian Reichel 2015-03-16 19:17 ` [PATCH 08/35 linux-next] dma: " Fabian Frederick 2015-03-18 16:43 ` Vinod Koul [not found] ` <1426533651-25586-1-git-send-email-fabf@skynet.be> 2015-03-16 19:20 ` [PATCH 11/35 linux-next] usb: gadget: " Fabian Frederick 2015-03-16 19:20 ` [PATCH 16/35 linux-next] video: " Fabian Frederick 2015-03-16 19:29 ` Timur Tabi 2015-03-16 19:20 ` [PATCH 17/35 linux-next] coresight-replicator: " Fabian Frederick 2015-03-16 19:54 ` [PATCH 22/35 linux-next] soc: ti: " Fabian Frederick 2015-03-16 19:54 ` [PATCH 23/35 linux-next] [media] " Fabian Frederick 2015-03-17 8:12 ` Patrice Chotard 2015-03-16 19:54 ` [PATCH 25/35 linux-next] PCI: " Fabian Frederick 2015-03-16 20:23 ` Sergei Shtylyov 2015-03-16 20:30 ` Fabian Frederick 2015-03-16 19:54 ` [PATCH 26/35 linux-next] hwmon: " Fabian Frederick 2015-03-17 3:55 ` Guenter Roeck 2015-03-16 19:54 ` [PATCH 27/35 linux-next] reset: sti: " Fabian Frederick 2015-03-17 8:13 ` Patrice Chotard 2015-03-17 8:33 ` Maxime Coquelin 2015-03-17 9:52 ` Philipp Zabel 2015-03-16 19:54 ` [PATCH 29/35 linux-next] gpu: " Fabian Frederick 2015-03-16 19:54 ` [PATCH 30/35 linux-next] devfreq: " Fabian Frederick 2015-03-16 19:54 ` [PATCH 31/35 linux-next] EDAC: " Fabian Frederick 2015-03-20 16:57 ` Borislav Petkov 2015-03-16 19:59 ` [PATCH 32/35 linux-next] clk: " Fabian Frederick 2015-03-16 19:59 ` [PATCH 33/35 linux-next] mmc: " Fabian Frederick 2015-03-17 9:49 ` Ulf Hansson 2015-03-20 8:56 ` Ulf Hansson 2015-03-21 20:09 ` Fabian Frederick 2015-03-23 10:06 ` Ulf Hansson 2015-03-16 19:59 ` [PATCH 35/35 linux-next] pinctrl: " Fabian Frederick 2015-03-17 4:03 ` Jean-Christophe PLAGNIOL-VILLARD 2015-03-17 8:15 ` Patrice Chotard 2015-03-17 8:34 ` Maxime Coquelin 2015-03-19 1:58 ` Hongzhou Yang 2015-03-19 18:41 ` Fabian Frederick 2015-03-19 20:55 ` Hongzhou Yang 2015-03-19 7:12 ` Lee Jones 2015-03-27 9:00 ` Linus Walleij 2015-03-18 14:15 ` [PATCH 32/35 linux-next] clk: " Michael Turquette 2015-03-22 10:34 ` Fabian Frederick 2015-03-27 7:19 ` Stephen Boyd 2015-03-31 19:05 ` Fabian Frederick
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).