* [PATCH 00/35 linux-next] constify of_device_id array @ 2015-03-16 19:17 Fabian Frederick [not found] ` <1426535685-25996-1-git-send-email-fabf@skynet.be> 0 siblings, 1 reply; 4+ messages in thread From: Fabian Frederick @ 2015-03-16 19:17 UTC (permalink / raw) To: linux-kernel Cc: linux-fbdev, kernel, linux-sh, linux-iio, linux-pci, dri-devel, virtualization, linux-ide, linux-mtd, Peter Meerwald, openipmi-developer, devel, Lars-Peter Clausen, lm-sensors, Sören Brinkmann, linux-serial, linux-input, linux-media, linux-pwm, linux-pm, Fabian Frederick, linux-gpio, linux-samsung-soc, linux-mediatek, linux-rpi-kernel, linux-tegra, Dan Williams 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] 4+ messages in thread
[parent not found: <1426535685-25996-1-git-send-email-fabf@skynet.be>]
* [PATCH 25/35 linux-next] PCI: constify of_device_id array [not found] ` <1426535685-25996-1-git-send-email-fabf@skynet.be> @ 2015-03-16 19:54 ` Fabian Frederick 2015-03-16 20:23 ` Sergei Shtylyov 0 siblings, 1 reply; 4+ 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] 4+ messages in thread
* Re: [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; 4+ 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] 4+ messages in thread
* Re: [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; 4+ 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] 4+ messages in thread
end of thread, other threads:[~2015-03-16 20:30 UTC | newest] Thread overview: 4+ 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 [not found] ` <1426535685-25996-1-git-send-email-fabf@skynet.be> 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
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).