* [PATCH 1/4] perf tools: remove duplicate const qualifier
@ 2016-04-25 9:47 Eric Engestrom
2016-04-25 9:47 ` [PATCH 2/4] drivers: firmware: psci: " Eric Engestrom
` (4 more replies)
0 siblings, 5 replies; 19+ messages in thread
From: Eric Engestrom @ 2016-04-25 9:47 UTC (permalink / raw)
To: linux-kernel
Cc: Eric Engestrom, Peter Zijlstra, Ingo Molnar,
Arnaldo Carvalho de Melo, David Ahern, Adrian Hunter
Signed-off-by: Eric Engestrom <eric.engestrom@imgtec.com>
---
tools/perf/util/thread.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/perf/util/thread.c b/tools/perf/util/thread.c
index dfd00c6..de2036d 100644
--- a/tools/perf/util/thread.c
+++ b/tools/perf/util/thread.c
@@ -233,7 +233,7 @@ void thread__find_cpumode_addr_location(struct thread *thread,
struct addr_location *al)
{
size_t i;
- const u8 const cpumodes[] = {
+ const u8 cpumodes[] = {
PERF_RECORD_MISC_USER,
PERF_RECORD_MISC_KERNEL,
PERF_RECORD_MISC_GUEST_USER,
--
2.7.4
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH 2/4] drivers: firmware: psci: remove duplicate const qualifier
2016-04-25 9:47 [PATCH 1/4] perf tools: remove duplicate const qualifier Eric Engestrom
@ 2016-04-25 9:47 ` Eric Engestrom
2016-04-25 10:33 ` Jisheng Zhang
2016-04-25 9:47 ` [PATCH 3/4] dmaengine: pxa_dma: " Eric Engestrom
` (3 subsequent siblings)
4 siblings, 1 reply; 19+ messages in thread
From: Eric Engestrom @ 2016-04-25 9:47 UTC (permalink / raw)
To: linux-kernel
Cc: Eric Engestrom, Mark Rutland, Lorenzo Pieralisi, linux-arm-kernel
Signed-off-by: Eric Engestrom <eric.engestrom@imgtec.com>
---
drivers/firmware/psci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/firmware/psci.c b/drivers/firmware/psci.c
index 11bfee8..0ab477ba 100644
--- a/drivers/firmware/psci.c
+++ b/drivers/firmware/psci.c
@@ -563,7 +563,7 @@ static int __init psci_0_1_init(struct device_node *np)
return err;
}
-static const struct of_device_id const psci_of_match[] __initconst = {
+static const struct of_device_id psci_of_match[] __initconst = {
{ .compatible = "arm,psci", .data = psci_0_1_init},
{ .compatible = "arm,psci-0.2", .data = psci_0_2_init},
{ .compatible = "arm,psci-1.0", .data = psci_0_2_init},
--
2.7.4
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH 3/4] dmaengine: pxa_dma: remove duplicate const qualifier
2016-04-25 9:47 [PATCH 1/4] perf tools: remove duplicate const qualifier Eric Engestrom
2016-04-25 9:47 ` [PATCH 2/4] drivers: firmware: psci: " Eric Engestrom
@ 2016-04-25 9:47 ` Eric Engestrom
2016-04-25 19:18 ` Robert Jarzmik
2016-04-26 16:30 ` Vinod Koul
2016-04-25 9:47 ` [PATCH 4/4] ARM: " Eric Engestrom
` (2 subsequent siblings)
4 siblings, 2 replies; 19+ messages in thread
From: Eric Engestrom @ 2016-04-25 9:47 UTC (permalink / raw)
To: linux-kernel
Cc: Eric Engestrom, Daniel Mack, Haojian Zhuang, Robert Jarzmik,
Vinod Koul, linux-arm-kernel, dmaengine
Signed-off-by: Eric Engestrom <eric.engestrom@imgtec.com>
---
drivers/dma/pxa_dma.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/pxa_dma.c b/drivers/dma/pxa_dma.c
index 77c1c44..315901d 100644
--- a/drivers/dma/pxa_dma.c
+++ b/drivers/dma/pxa_dma.c
@@ -1321,7 +1321,7 @@ static int pxad_init_phys(struct platform_device *op,
return 0;
}
-static const struct of_device_id const pxad_dt_ids[] = {
+static const struct of_device_id pxad_dt_ids[] = {
{ .compatible = "marvell,pdma-1.0", },
{}
};
--
2.7.4
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH 4/4] ARM: remove duplicate const qualifier
2016-04-25 9:47 [PATCH 1/4] perf tools: remove duplicate const qualifier Eric Engestrom
2016-04-25 9:47 ` [PATCH 2/4] drivers: firmware: psci: " Eric Engestrom
2016-04-25 9:47 ` [PATCH 3/4] dmaengine: pxa_dma: " Eric Engestrom
@ 2016-04-25 9:47 ` Eric Engestrom
2016-04-25 9:52 ` Viresh Kumar
2016-04-25 9:57 ` Alexandre Belloni
2016-04-25 21:11 ` [PATCH 1/4] perf tools: " Arnaldo Carvalho de Melo
2016-04-27 15:32 ` [tip:perf/core] perf tools: Remove " tip-bot for Eric Engestrom
4 siblings, 2 replies; 19+ messages in thread
From: Eric Engestrom @ 2016-04-25 9:47 UTC (permalink / raw)
To: linux-kernel
Cc: Eric Engestrom, Nicolas Ferre, Alexandre Belloni,
Jean-Christophe Plagniol-Villard, Russell King, Florian Fainelli,
Ray Jui, Scott Branden, Alexander Shiyan, Krzysztof Halasa,
Sekhar Nori, Kevin Hilman, Roland Stigge, Shawn Guo,
Sascha Hauer, Rajendra Nayak, Paul Walmsley, Tony Lindgren,
Viresh Kumar, Shiraz Hashim, Gregory Fong, Krzysztof Kozlowski,
Nicolas Pitre, Lee Jones, Linus Walleij, linux-arm-kernel,
bcm-kernel-feedback-list, linux-omap, spear-devel
Signed-off-by: Eric Engestrom <eric.engestrom@imgtec.com>
---
arch/arm/mach-at91/pm.c | 2 +-
arch/arm/mach-bcm/bcm_kona_smc.c | 2 +-
arch/arm/mach-clps711x/board-autcpu12.c | 2 +-
arch/arm/mach-cns3xxx/core.c | 2 +-
arch/arm/mach-davinci/da8xx-dt.c | 2 +-
arch/arm/mach-lpc32xx/phy3250.c | 2 +-
arch/arm/mach-mxs/mach-mxs.c | 2 +-
arch/arm/mach-omap2/prm_common.c | 2 +-
arch/arm/mach-omap2/vc.c | 2 +-
arch/arm/mach-spear/time.c | 2 +-
10 files changed, 10 insertions(+), 10 deletions(-)
diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
index f062701..9ef8604 100644
--- a/arch/arm/mach-at91/pm.c
+++ b/arch/arm/mach-at91/pm.c
@@ -318,7 +318,7 @@ static void at91sam9_sdram_standby(void)
at91_ramc_write(1, AT91_SDRAMC_LPR, saved_lpr1);
}
-static const struct of_device_id const ramc_ids[] __initconst = {
+static const struct of_device_id ramc_ids[] __initconst = {
{ .compatible = "atmel,at91rm9200-sdramc", .data = at91rm9200_standby },
{ .compatible = "atmel,at91sam9260-sdramc", .data = at91sam9_sdram_standby },
{ .compatible = "atmel,at91sam9g45-ddramc", .data = at91_ddr_standby },
diff --git a/arch/arm/mach-bcm/bcm_kona_smc.c b/arch/arm/mach-bcm/bcm_kona_smc.c
index cf3f865..a55a7ec 100644
--- a/arch/arm/mach-bcm/bcm_kona_smc.c
+++ b/arch/arm/mach-bcm/bcm_kona_smc.c
@@ -33,7 +33,7 @@ struct bcm_kona_smc_data {
unsigned result;
};
-static const struct of_device_id const bcm_kona_smc_ids[] __initconst = {
+static const struct of_device_id bcm_kona_smc_ids[] __initconst = {
{.compatible = "brcm,kona-smc"},
{.compatible = "bcm,kona-smc"}, /* deprecated name */
{},
diff --git a/arch/arm/mach-clps711x/board-autcpu12.c b/arch/arm/mach-clps711x/board-autcpu12.c
index ba3d7d1..4fdfd92 100644
--- a/arch/arm/mach-clps711x/board-autcpu12.c
+++ b/arch/arm/mach-clps711x/board-autcpu12.c
@@ -160,7 +160,7 @@ static struct platform_device autcpu12_mmgpio_pdev __initdata = {
},
};
-static const struct gpio const autcpu12_gpios[] __initconst = {
+static const struct gpio autcpu12_gpios[] __initconst = {
{ AUTCPU12_DPOT_CS, GPIOF_OUT_INIT_HIGH, "DPOT CS" },
{ AUTCPU12_DPOT_CLK, GPIOF_OUT_INIT_LOW, "DPOT CLK" },
{ AUTCPU12_DPOT_UD, GPIOF_OUT_INIT_LOW, "DPOT UD" },
diff --git a/arch/arm/mach-cns3xxx/core.c b/arch/arm/mach-cns3xxx/core.c
index 9b1dc22..e17a0e02 100644
--- a/arch/arm/mach-cns3xxx/core.c
+++ b/arch/arm/mach-cns3xxx/core.c
@@ -346,7 +346,7 @@ static struct usb_ohci_pdata cns3xxx_usb_ohci_pdata = {
.power_off = csn3xxx_usb_power_off,
};
-static const struct of_dev_auxdata const cns3xxx_auxdata[] __initconst = {
+static const struct of_dev_auxdata cns3xxx_auxdata[] __initconst = {
{ "intel,usb-ehci", CNS3XXX_USB_BASE, "ehci-platform", &cns3xxx_usb_ehci_pdata },
{ "intel,usb-ohci", CNS3XXX_USB_OHCI_BASE, "ohci-platform", &cns3xxx_usb_ohci_pdata },
{ "cavium,cns3420-ahci", CNS3XXX_SATA2_BASE, "ahci", NULL },
diff --git a/arch/arm/mach-davinci/da8xx-dt.c b/arch/arm/mach-davinci/da8xx-dt.c
index c4b5808..3d92c93 100644
--- a/arch/arm/mach-davinci/da8xx-dt.c
+++ b/arch/arm/mach-davinci/da8xx-dt.c
@@ -20,7 +20,7 @@
#define DA8XX_NUM_UARTS 3
-static const struct of_device_id const da8xx_irq_match[] __initconst = {
+static const struct of_device_id da8xx_irq_match[] __initconst = {
{ .compatible = "ti,cp-intc", .data = cp_intc_of_init, },
{ }
};
diff --git a/arch/arm/mach-lpc32xx/phy3250.c b/arch/arm/mach-lpc32xx/phy3250.c
index b2f9e22..1ae7230 100644
--- a/arch/arm/mach-lpc32xx/phy3250.c
+++ b/arch/arm/mach-lpc32xx/phy3250.c
@@ -159,7 +159,7 @@ static struct lpc32xx_mlc_platform_data lpc32xx_mlc_data = {
.dma_filter = pl08x_filter_id,
};
-static const struct of_dev_auxdata const lpc32xx_auxdata_lookup[] __initconst = {
+static const struct of_dev_auxdata lpc32xx_auxdata_lookup[] __initconst = {
OF_DEV_AUXDATA("arm,pl022", 0x20084000, "dev:ssp0", NULL),
OF_DEV_AUXDATA("arm,pl022", 0x2008C000, "dev:ssp1", NULL),
OF_DEV_AUXDATA("arm,pl110", 0x31040000, "dev:clcd", &lpc32xx_clcd_data),
diff --git a/arch/arm/mach-mxs/mach-mxs.c b/arch/arm/mach-mxs/mach-mxs.c
index f1ea470..19eeaec 100644
--- a/arch/arm/mach-mxs/mach-mxs.c
+++ b/arch/arm/mach-mxs/mach-mxs.c
@@ -282,7 +282,7 @@ static void __init apx4devkit_init(void)
#define TX28_FEC_PHY_RESET MXS_GPIO_NR(4, 13)
#define TX28_FEC_nINT MXS_GPIO_NR(4, 5)
-static const struct gpio const tx28_gpios[] __initconst = {
+static const struct gpio tx28_gpios[] __initconst = {
{ ENET0_MDC__GPIO_4_0, GPIOF_OUT_INIT_LOW, "GPIO_4_0" },
{ ENET0_MDIO__GPIO_4_1, GPIOF_OUT_INIT_LOW, "GPIO_4_1" },
{ ENET0_RX_EN__GPIO_4_2, GPIOF_OUT_INIT_LOW, "GPIO_4_2" },
diff --git a/arch/arm/mach-omap2/prm_common.c b/arch/arm/mach-omap2/prm_common.c
index 5b2f513..f1ca947 100644
--- a/arch/arm/mach-omap2/prm_common.c
+++ b/arch/arm/mach-omap2/prm_common.c
@@ -713,7 +713,7 @@ static struct omap_prcm_init_data scrm_data __initdata = {
};
#endif
-static const struct of_device_id const omap_prcm_dt_match_table[] __initconst = {
+static const struct of_device_id omap_prcm_dt_match_table[] __initconst = {
#ifdef CONFIG_SOC_AM33XX
{ .compatible = "ti,am3-prcm", .data = &am3_prm_data },
#endif
diff --git a/arch/arm/mach-omap2/vc.c b/arch/arm/mach-omap2/vc.c
index 2028167f..d76b1e5 100644
--- a/arch/arm/mach-omap2/vc.c
+++ b/arch/arm/mach-omap2/vc.c
@@ -559,7 +559,7 @@ struct i2c_init_data {
u8 hsscll_12;
};
-static const struct i2c_init_data const omap4_i2c_timing_data[] __initconst = {
+static const struct i2c_init_data omap4_i2c_timing_data[] __initconst = {
{
.load = 50,
.loadbits = 0x3,
diff --git a/arch/arm/mach-spear/time.c b/arch/arm/mach-spear/time.c
index 9ccffc1..aaaa678 100644
--- a/arch/arm/mach-spear/time.c
+++ b/arch/arm/mach-spear/time.c
@@ -204,7 +204,7 @@ static void __init spear_clockevent_init(int irq)
setup_irq(irq, &spear_timer_irq);
}
-static const struct of_device_id const timer_of_match[] __initconst = {
+static const struct of_device_id timer_of_match[] __initconst = {
{ .compatible = "st,spear-timer", },
{ },
};
--
2.7.4
^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH 4/4] ARM: remove duplicate const qualifier
2016-04-25 9:47 ` [PATCH 4/4] ARM: " Eric Engestrom
@ 2016-04-25 9:52 ` Viresh Kumar
2016-04-25 9:57 ` Alexandre Belloni
1 sibling, 0 replies; 19+ messages in thread
From: Viresh Kumar @ 2016-04-25 9:52 UTC (permalink / raw)
To: Eric Engestrom
Cc: linux-kernel, Nicolas Ferre, Alexandre Belloni,
Jean-Christophe Plagniol-Villard, Russell King, Florian Fainelli,
Ray Jui, Scott Branden, Alexander Shiyan, Krzysztof Halasa,
Sekhar Nori, Kevin Hilman, Roland Stigge, Shawn Guo,
Sascha Hauer, Rajendra Nayak, Paul Walmsley, Tony Lindgren,
Viresh Kumar, Shiraz Hashim, Gregory Fong, Krzysztof Kozlowski,
Nicolas Pitre, Lee Jones, Linus Walleij, linux-arm-kernel,
bcm-kernel-feedback-list, linux-omap, spear-devel
On 25-04-16, 10:47, Eric Engestrom wrote:
> diff --git a/arch/arm/mach-spear/time.c b/arch/arm/mach-spear/time.c
> index 9ccffc1..aaaa678 100644
> --- a/arch/arm/mach-spear/time.c
> +++ b/arch/arm/mach-spear/time.c
> @@ -204,7 +204,7 @@ static void __init spear_clockevent_init(int irq)
> setup_irq(irq, &spear_timer_irq);
> }
>
> -static const struct of_device_id const timer_of_match[] __initconst = {
> +static const struct of_device_id timer_of_match[] __initconst = {
> { .compatible = "st,spear-timer", },
> { },
> };
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
--
viresh
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 4/4] ARM: remove duplicate const qualifier
2016-04-25 9:47 ` [PATCH 4/4] ARM: " Eric Engestrom
2016-04-25 9:52 ` Viresh Kumar
@ 2016-04-25 9:57 ` Alexandre Belloni
2016-04-25 10:39 ` Eric Engestrom
2016-04-25 10:59 ` Sylvain Lemieux
1 sibling, 2 replies; 19+ messages in thread
From: Alexandre Belloni @ 2016-04-25 9:57 UTC (permalink / raw)
To: Eric Engestrom
Cc: linux-kernel, Nicolas Ferre, Jean-Christophe Plagniol-Villard,
Russell King, Florian Fainelli, Ray Jui, Scott Branden,
Alexander Shiyan, Krzysztof Halasa, Sekhar Nori, Kevin Hilman,
Roland Stigge, Shawn Guo, Sascha Hauer, Rajendra Nayak,
Paul Walmsley, Tony Lindgren, Viresh Kumar, Shiraz Hashim,
Gregory Fong, Krzysztof Kozlowski, Nicolas Pitre, Lee Jones,
Linus Walleij, linux-arm-kernel, bcm-kernel-feedback-list,
linux-omap, spear-devel
Hi,
this requires a commit message.
Also, you should think about separating those changes in multiple patches
to ease inclusion in the kernel.
On 25/04/2016 at 10:47:57 +0100, Eric Engestrom wrote :
> Signed-off-by: Eric Engestrom <eric.engestrom@imgtec.com>
> ---
> arch/arm/mach-at91/pm.c | 2 +-
> arch/arm/mach-bcm/bcm_kona_smc.c | 2 +-
> arch/arm/mach-clps711x/board-autcpu12.c | 2 +-
> arch/arm/mach-cns3xxx/core.c | 2 +-
> arch/arm/mach-davinci/da8xx-dt.c | 2 +-
> arch/arm/mach-lpc32xx/phy3250.c | 2 +-
> arch/arm/mach-mxs/mach-mxs.c | 2 +-
> arch/arm/mach-omap2/prm_common.c | 2 +-
> arch/arm/mach-omap2/vc.c | 2 +-
> arch/arm/mach-spear/time.c | 2 +-
> 10 files changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
> index f062701..9ef8604 100644
> --- a/arch/arm/mach-at91/pm.c
> +++ b/arch/arm/mach-at91/pm.c
> @@ -318,7 +318,7 @@ static void at91sam9_sdram_standby(void)
> at91_ramc_write(1, AT91_SDRAMC_LPR, saved_lpr1);
> }
>
> -static const struct of_device_id const ramc_ids[] __initconst = {
> +static const struct of_device_id ramc_ids[] __initconst = {
> { .compatible = "atmel,at91rm9200-sdramc", .data = at91rm9200_standby },
> { .compatible = "atmel,at91sam9260-sdramc", .data = at91sam9_sdram_standby },
> { .compatible = "atmel,at91sam9g45-ddramc", .data = at91_ddr_standby },
> diff --git a/arch/arm/mach-bcm/bcm_kona_smc.c b/arch/arm/mach-bcm/bcm_kona_smc.c
> index cf3f865..a55a7ec 100644
> --- a/arch/arm/mach-bcm/bcm_kona_smc.c
> +++ b/arch/arm/mach-bcm/bcm_kona_smc.c
> @@ -33,7 +33,7 @@ struct bcm_kona_smc_data {
> unsigned result;
> };
>
> -static const struct of_device_id const bcm_kona_smc_ids[] __initconst = {
> +static const struct of_device_id bcm_kona_smc_ids[] __initconst = {
> {.compatible = "brcm,kona-smc"},
> {.compatible = "bcm,kona-smc"}, /* deprecated name */
> {},
> diff --git a/arch/arm/mach-clps711x/board-autcpu12.c b/arch/arm/mach-clps711x/board-autcpu12.c
> index ba3d7d1..4fdfd92 100644
> --- a/arch/arm/mach-clps711x/board-autcpu12.c
> +++ b/arch/arm/mach-clps711x/board-autcpu12.c
> @@ -160,7 +160,7 @@ static struct platform_device autcpu12_mmgpio_pdev __initdata = {
> },
> };
>
> -static const struct gpio const autcpu12_gpios[] __initconst = {
> +static const struct gpio autcpu12_gpios[] __initconst = {
> { AUTCPU12_DPOT_CS, GPIOF_OUT_INIT_HIGH, "DPOT CS" },
> { AUTCPU12_DPOT_CLK, GPIOF_OUT_INIT_LOW, "DPOT CLK" },
> { AUTCPU12_DPOT_UD, GPIOF_OUT_INIT_LOW, "DPOT UD" },
> diff --git a/arch/arm/mach-cns3xxx/core.c b/arch/arm/mach-cns3xxx/core.c
> index 9b1dc22..e17a0e02 100644
> --- a/arch/arm/mach-cns3xxx/core.c
> +++ b/arch/arm/mach-cns3xxx/core.c
> @@ -346,7 +346,7 @@ static struct usb_ohci_pdata cns3xxx_usb_ohci_pdata = {
> .power_off = csn3xxx_usb_power_off,
> };
>
> -static const struct of_dev_auxdata const cns3xxx_auxdata[] __initconst = {
> +static const struct of_dev_auxdata cns3xxx_auxdata[] __initconst = {
> { "intel,usb-ehci", CNS3XXX_USB_BASE, "ehci-platform", &cns3xxx_usb_ehci_pdata },
> { "intel,usb-ohci", CNS3XXX_USB_OHCI_BASE, "ohci-platform", &cns3xxx_usb_ohci_pdata },
> { "cavium,cns3420-ahci", CNS3XXX_SATA2_BASE, "ahci", NULL },
> diff --git a/arch/arm/mach-davinci/da8xx-dt.c b/arch/arm/mach-davinci/da8xx-dt.c
> index c4b5808..3d92c93 100644
> --- a/arch/arm/mach-davinci/da8xx-dt.c
> +++ b/arch/arm/mach-davinci/da8xx-dt.c
> @@ -20,7 +20,7 @@
>
> #define DA8XX_NUM_UARTS 3
>
> -static const struct of_device_id const da8xx_irq_match[] __initconst = {
> +static const struct of_device_id da8xx_irq_match[] __initconst = {
> { .compatible = "ti,cp-intc", .data = cp_intc_of_init, },
> { }
> };
> diff --git a/arch/arm/mach-lpc32xx/phy3250.c b/arch/arm/mach-lpc32xx/phy3250.c
> index b2f9e22..1ae7230 100644
> --- a/arch/arm/mach-lpc32xx/phy3250.c
> +++ b/arch/arm/mach-lpc32xx/phy3250.c
> @@ -159,7 +159,7 @@ static struct lpc32xx_mlc_platform_data lpc32xx_mlc_data = {
> .dma_filter = pl08x_filter_id,
> };
>
> -static const struct of_dev_auxdata const lpc32xx_auxdata_lookup[] __initconst = {
> +static const struct of_dev_auxdata lpc32xx_auxdata_lookup[] __initconst = {
> OF_DEV_AUXDATA("arm,pl022", 0x20084000, "dev:ssp0", NULL),
> OF_DEV_AUXDATA("arm,pl022", 0x2008C000, "dev:ssp1", NULL),
> OF_DEV_AUXDATA("arm,pl110", 0x31040000, "dev:clcd", &lpc32xx_clcd_data),
> diff --git a/arch/arm/mach-mxs/mach-mxs.c b/arch/arm/mach-mxs/mach-mxs.c
> index f1ea470..19eeaec 100644
> --- a/arch/arm/mach-mxs/mach-mxs.c
> +++ b/arch/arm/mach-mxs/mach-mxs.c
> @@ -282,7 +282,7 @@ static void __init apx4devkit_init(void)
> #define TX28_FEC_PHY_RESET MXS_GPIO_NR(4, 13)
> #define TX28_FEC_nINT MXS_GPIO_NR(4, 5)
>
> -static const struct gpio const tx28_gpios[] __initconst = {
> +static const struct gpio tx28_gpios[] __initconst = {
> { ENET0_MDC__GPIO_4_0, GPIOF_OUT_INIT_LOW, "GPIO_4_0" },
> { ENET0_MDIO__GPIO_4_1, GPIOF_OUT_INIT_LOW, "GPIO_4_1" },
> { ENET0_RX_EN__GPIO_4_2, GPIOF_OUT_INIT_LOW, "GPIO_4_2" },
> diff --git a/arch/arm/mach-omap2/prm_common.c b/arch/arm/mach-omap2/prm_common.c
> index 5b2f513..f1ca947 100644
> --- a/arch/arm/mach-omap2/prm_common.c
> +++ b/arch/arm/mach-omap2/prm_common.c
> @@ -713,7 +713,7 @@ static struct omap_prcm_init_data scrm_data __initdata = {
> };
> #endif
>
> -static const struct of_device_id const omap_prcm_dt_match_table[] __initconst = {
> +static const struct of_device_id omap_prcm_dt_match_table[] __initconst = {
> #ifdef CONFIG_SOC_AM33XX
> { .compatible = "ti,am3-prcm", .data = &am3_prm_data },
> #endif
> diff --git a/arch/arm/mach-omap2/vc.c b/arch/arm/mach-omap2/vc.c
> index 2028167f..d76b1e5 100644
> --- a/arch/arm/mach-omap2/vc.c
> +++ b/arch/arm/mach-omap2/vc.c
> @@ -559,7 +559,7 @@ struct i2c_init_data {
> u8 hsscll_12;
> };
>
> -static const struct i2c_init_data const omap4_i2c_timing_data[] __initconst = {
> +static const struct i2c_init_data omap4_i2c_timing_data[] __initconst = {
> {
> .load = 50,
> .loadbits = 0x3,
> diff --git a/arch/arm/mach-spear/time.c b/arch/arm/mach-spear/time.c
> index 9ccffc1..aaaa678 100644
> --- a/arch/arm/mach-spear/time.c
> +++ b/arch/arm/mach-spear/time.c
> @@ -204,7 +204,7 @@ static void __init spear_clockevent_init(int irq)
> setup_irq(irq, &spear_timer_irq);
> }
>
> -static const struct of_device_id const timer_of_match[] __initconst = {
> +static const struct of_device_id timer_of_match[] __initconst = {
> { .compatible = "st,spear-timer", },
> { },
> };
> --
> 2.7.4
>
--
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 2/4] drivers: firmware: psci: remove duplicate const qualifier
2016-04-25 9:47 ` [PATCH 2/4] drivers: firmware: psci: " Eric Engestrom
@ 2016-04-25 10:33 ` Jisheng Zhang
0 siblings, 0 replies; 19+ messages in thread
From: Jisheng Zhang @ 2016-04-25 10:33 UTC (permalink / raw)
To: Eric Engestrom
Cc: linux-kernel, Mark Rutland, Lorenzo Pieralisi, linux-arm-kernel
Hi Eric,
On Mon, 25 Apr 2016 10:47:55 +0100 Eric Engestrom wrote:
> Signed-off-by: Eric Engestrom <eric.engestrom@imgtec.com>
> ---
> drivers/firmware/psci.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/firmware/psci.c b/drivers/firmware/psci.c
> index 11bfee8..0ab477ba 100644
> --- a/drivers/firmware/psci.c
> +++ b/drivers/firmware/psci.c
> @@ -563,7 +563,7 @@ static int __init psci_0_1_init(struct device_node *np)
> return err;
> }
>
> -static const struct of_device_id const psci_of_match[] __initconst = {
> +static const struct of_device_id psci_of_match[] __initconst = {
A previous resend patch ;)
http://lists.infradead.org/pipermail/linux-arm-kernel/2016-March/417068.html
And IIRC, the maintainer has merged it.
> { .compatible = "arm,psci", .data = psci_0_1_init},
> { .compatible = "arm,psci-0.2", .data = psci_0_2_init},
> { .compatible = "arm,psci-1.0", .data = psci_0_2_init},
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 4/4] ARM: remove duplicate const qualifier
2016-04-25 9:57 ` Alexandre Belloni
@ 2016-04-25 10:39 ` Eric Engestrom
2016-04-25 11:38 ` Arnd Bergmann
2016-04-25 10:59 ` Sylvain Lemieux
1 sibling, 1 reply; 19+ messages in thread
From: Eric Engestrom @ 2016-04-25 10:39 UTC (permalink / raw)
To: Alexandre Belloni
Cc: linux-kernel, Nicolas Ferre, Jean-Christophe Plagniol-Villard,
Russell King, Florian Fainelli, Ray Jui, Scott Branden,
Alexander Shiyan, Krzysztof Halasa, Sekhar Nori, Kevin Hilman,
Roland Stigge, Shawn Guo, Sascha Hauer, Rajendra Nayak,
Paul Walmsley, Tony Lindgren, Viresh Kumar, Shiraz Hashim,
Gregory Fong, Krzysztof Kozlowski, Nicolas Pitre, Lee Jones,
Linus Walleij, linux-arm-kernel, bcm-kernel-feedback-list,
linux-omap, spear-devel
On Mon, Apr 25, 2016 at 11:57:15AM +0200, Alexandre Belloni wrote:
> Hi,
>
> this requires a commit message.
OK? This seems rather pointless in this case (and even more so for the
typo fix I sent yesterday), but I guess you have some general rule to
enforce. Would this do then?
"The second `const` is unnecessary, and is simply ignored by most compilers."
> Also, you should think about separating those changes in multiple patches
> to ease inclusion in the kernel.
I'll resend patch #4 as multiple patches. What the way proper to convey
that *multiple* new patches replace *one* old one? Is it enough to just
send those as a reply to this one?
Cheers
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 4/4] ARM: remove duplicate const qualifier
2016-04-25 9:57 ` Alexandre Belloni
2016-04-25 10:39 ` Eric Engestrom
@ 2016-04-25 10:59 ` Sylvain Lemieux
1 sibling, 0 replies; 19+ messages in thread
From: Sylvain Lemieux @ 2016-04-25 10:59 UTC (permalink / raw)
To: Eric Engestrom
Cc: Alexandre Belloni, Nicolas Pitre, Krzysztof Kozlowski,
Tony Lindgren, Linus Walleij, Nicolas Ferre, Lee Jones,
Roland Stigge, Florian Fainelli, Russell King, Alexander Shiyan,
Kevin Hilman, Viresh Kumar, bcm-kernel-feedback-list,
Jean-Christophe Plagniol-Villard, spear-devel, Ray Jui,
Rajendra Nayak, Sekhar Nori, Krzysztof Halasa, Gregory Fong,
linux-omap, linux-arm-kernel, Paul Walmsley, Scott Branden,
Shiraz Hashim, linux-kernel, Sascha Hauer, Shawn Guo, vz
Hi Eric,
On Mon, 2016-04-25 at 11:57 +0200, Alexandre Belloni wrote:
> Hi,
>
> this requires a commit message.
>
> Also, you should think about separating those changes in multiple patches
> to ease inclusion in the kernel.
>
> On 25/04/2016 at 10:47:57 +0100, Eric Engestrom wrote :
> > Signed-off-by: Eric Engestrom <eric.engestrom@imgtec.com>
> > ---
> > arch/arm/mach-at91/pm.c | 2 +-
> > arch/arm/mach-bcm/bcm_kona_smc.c | 2 +-
> > arch/arm/mach-clps711x/board-autcpu12.c | 2 +-
> > arch/arm/mach-cns3xxx/core.c | 2 +-
> > arch/arm/mach-davinci/da8xx-dt.c | 2 +-
> > arch/arm/mach-lpc32xx/phy3250.c | 2 +-
> > arch/arm/mach-mxs/mach-mxs.c | 2 +-
> > arch/arm/mach-omap2/prm_common.c | 2 +-
> > arch/arm/mach-omap2/vc.c | 2 +-
> > arch/arm/mach-spear/time.c | 2 +-
> > 10 files changed, 10 insertions(+), 10 deletions(-)
> >
...
> > diff --git a/arch/arm/mach-lpc32xx/phy3250.c b/arch/arm/mach-lpc32xx/phy3250.c
> > index b2f9e22..1ae7230 100644
> > --- a/arch/arm/mach-lpc32xx/phy3250.c
> > +++ b/arch/arm/mach-lpc32xx/phy3250.c
> > @@ -159,7 +159,7 @@ static struct lpc32xx_mlc_platform_data lpc32xx_mlc_data = {
> > .dma_filter = pl08x_filter_id,
> > };
> >
> > -static const struct of_dev_auxdata const lpc32xx_auxdata_lookup[] __initconst = {
> > +static const struct of_dev_auxdata lpc32xx_auxdata_lookup[] __initconst = {
> > OF_DEV_AUXDATA("arm,pl022", 0x20084000, "dev:ssp0", NULL),
> > OF_DEV_AUXDATA("arm,pl022", 0x2008C000, "dev:ssp1", NULL),
> > OF_DEV_AUXDATA("arm,pl110", 0x31040000, "dev:clcd", &lpc32xx_clcd_data),
A patch was already submitted for the LPC32xx:
http://thread.gmane.org/gmane.linux.ports.arm.kernel/493312/
...
> > --
> > 2.7.4
> >
>
Sylvain
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 4/4] ARM: remove duplicate const qualifier
2016-04-25 10:39 ` Eric Engestrom
@ 2016-04-25 11:38 ` Arnd Bergmann
2016-04-25 12:58 ` Eric Engestrom
0 siblings, 1 reply; 19+ messages in thread
From: Arnd Bergmann @ 2016-04-25 11:38 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Eric Engestrom, Alexandre Belloni, Nicolas Pitre,
Krzysztof Kozlowski, Tony Lindgren, Linus Walleij, Nicolas Ferre,
Lee Jones, Roland Stigge, Florian Fainelli, Russell King,
Alexander Shiyan, Kevin Hilman, Viresh Kumar,
bcm-kernel-feedback-list, Jean-Christophe Plagniol-Villard,
spear-devel, Ray Jui, Rajendra Nayak, Sekhar Nori,
Krzysztof Halasa, Gregory Fong, linux-omap, Paul Walmsley,
Scott Branden, Shiraz Hashim, linux-kernel, Sascha Hauer,
Shawn Guo
On Monday 25 April 2016 11:39:11 Eric Engestrom wrote:
> On Mon, Apr 25, 2016 at 11:57:15AM +0200, Alexandre Belloni wrote:
> > Hi,
> >
> > this requires a commit message.
>
> OK? This seems rather pointless in this case (and even more so for the
> typo fix I sent yesterday), but I guess you have some general rule to
> enforce. Would this do then?
> "The second `const` is unnecessary, and is simply ignored by most compilers."
Maybe say you you found it (llvm, sparse, coccinelle?), and why this
is causing a problem for anyone. If it's just unnecessary but not
harmful, I'd probably ignore the patch.
> > Also, you should think about separating those changes in multiple patches
> > to ease inclusion in the kernel.
>
> I'll resend patch #4 as multiple patches. What the way proper to convey
> that *multiple* new patches replace *one* old one? Is it enough to just
> send those as a reply to this one?
That's fine. Having multiple patches here is slightly better than just
one, but I'd also be fine with just taking the large patch because it
is an identical trivial change in multiple files.
Arnd
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 4/4] ARM: remove duplicate const qualifier
2016-04-25 11:38 ` Arnd Bergmann
@ 2016-04-25 12:58 ` Eric Engestrom
2016-04-25 14:12 ` Nicolas Pitre
0 siblings, 1 reply; 19+ messages in thread
From: Eric Engestrom @ 2016-04-25 12:58 UTC (permalink / raw)
To: Arnd Bergmann
Cc: linux-arm-kernel, Alexandre Belloni, Nicolas Pitre,
Krzysztof Kozlowski, Tony Lindgren, Linus Walleij, Nicolas Ferre,
Lee Jones, Roland Stigge, Florian Fainelli, Russell King,
Alexander Shiyan, Kevin Hilman, Viresh Kumar,
bcm-kernel-feedback-list, Jean-Christophe Plagniol-Villard,
spear-devel, Ray Jui, Rajendra Nayak, Sekhar Nori,
Krzysztof Halasa, Gregory Fong, linux-omap, Paul Walmsley,
Scott Branden, Shiraz Hashim, linux-kernel, Sascha Hauer,
Shawn Guo
On Mon, Apr 25, 2016 at 01:38:18PM +0200, Arnd Bergmann wrote:
> Maybe say you you found it (llvm, sparse, coccinelle?), and why this
> is causing a problem for anyone. If it's just unnecessary but not
> harmful, I'd probably ignore the patch.
$ grep -rE '(^|\W)const(\s+\w+)+\s+const\s'
I just happened to notice some unnecessary const in our internal code,
so I grep'ed for it in a couple big OSS projects to see how common it
was. Since I found only a few, I decided to remove them, but like I said
it just gets ignored by all the compilers I know, so there's absolutely
no harm in leaving this dead code around.
> That's fine. Having multiple patches here is slightly better than just
> one, but I'd also be fine with just taking the large patch because it
> is an identical trivial change in multiple files.
So, do you want me to split them up? I'd prefer not having to (slightly
less work for me :P), but it doesn't really make much of a difference
either way.
Cheers
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 4/4] ARM: remove duplicate const qualifier
2016-04-25 12:58 ` Eric Engestrom
@ 2016-04-25 14:12 ` Nicolas Pitre
2016-04-25 14:37 ` Eric Engestrom
0 siblings, 1 reply; 19+ messages in thread
From: Nicolas Pitre @ 2016-04-25 14:12 UTC (permalink / raw)
To: Eric Engestrom
Cc: Arnd Bergmann, linux-arm-kernel, Alexandre Belloni,
Krzysztof Kozlowski, Tony Lindgren, Linus Walleij, Nicolas Ferre,
Lee Jones, Roland Stigge, Florian Fainelli, Russell King,
Alexander Shiyan, Kevin Hilman, Viresh Kumar,
bcm-kernel-feedback-list, Jean-Christophe Plagniol-Villard,
spear-devel, Ray Jui, Rajendra Nayak, Sekhar Nori,
Krzysztof Halasa, Gregory Fong, linux-omap, Paul Walmsley,
Scott Branden, Shiraz Hashim, linux-kernel, Sascha Hauer,
Shawn Guo
On Mon, 25 Apr 2016, Eric Engestrom wrote:
> On Mon, Apr 25, 2016 at 01:38:18PM +0200, Arnd Bergmann wrote:
> > Maybe say you you found it (llvm, sparse, coccinelle?), and why this
> > is causing a problem for anyone. If it's just unnecessary but not
> > harmful, I'd probably ignore the patch.
>
> $ grep -rE '(^|\W)const(\s+\w+)+\s+const\s'
>
> I just happened to notice some unnecessary const in our internal code,
> so I grep'ed for it in a couple big OSS projects to see how common it
> was. Since I found only a few, I decided to remove them, but like I said
> it just gets ignored by all the compilers I know, so there's absolutely
> no harm in leaving this dead code around.
Beware.
I added many of those exactly because gcc did not ignore them when
compiling with LTO where the lack of a const qualifier to qualify the
actual array content, and not only the reference to that content,
generated build errors due to section mismatches from the __initconst
annotation.
So this is a NAK from me unless you may confirm that LTO builds are
unaffected by your changes.
Nicolas
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 4/4] ARM: remove duplicate const qualifier
2016-04-25 14:12 ` Nicolas Pitre
@ 2016-04-25 14:37 ` Eric Engestrom
2016-04-26 10:12 ` Krzysztof Hałasa
2016-04-27 20:01 ` Nicolas Pitre
0 siblings, 2 replies; 19+ messages in thread
From: Eric Engestrom @ 2016-04-25 14:37 UTC (permalink / raw)
To: Nicolas Pitre
Cc: Arnd Bergmann, linux-arm-kernel, Alexandre Belloni,
Krzysztof Kozlowski, Tony Lindgren, Linus Walleij, Nicolas Ferre,
Lee Jones, Roland Stigge, Florian Fainelli, Russell King,
Alexander Shiyan, Kevin Hilman, Viresh Kumar,
bcm-kernel-feedback-list, Jean-Christophe Plagniol-Villard,
spear-devel, Ray Jui, Rajendra Nayak, Sekhar Nori,
Krzysztof Halasa, Gregory Fong, linux-omap, Paul Walmsley,
Scott Branden, Shiraz Hashim, linux-kernel, Sascha Hauer,
Shawn Guo
On Mon, Apr 25, 2016 at 10:12:16AM -0400, Nicolas Pitre wrote:
> Beware.
>
> I added many of those exactly because gcc did not ignore them when
> compiling with LTO where the lack of a const qualifier to qualify the
> actual array content, and not only the reference to that content,
> generated build errors due to section mismatches from the __initconst
> annotation.
I believe the first `const` does that, without the need for a second.
> So this is a NAK from me unless you may confirm that LTO builds are
> unaffected by your changes.
I can't confirm it (haven't tried), and don't care enough to do it :]
I guess I'm just dropping the patch then. Like I said, it can't hurt to
leave them in.
Cheers,
Eric
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 3/4] dmaengine: pxa_dma: remove duplicate const qualifier
2016-04-25 9:47 ` [PATCH 3/4] dmaengine: pxa_dma: " Eric Engestrom
@ 2016-04-25 19:18 ` Robert Jarzmik
2016-04-26 16:30 ` Vinod Koul
1 sibling, 0 replies; 19+ messages in thread
From: Robert Jarzmik @ 2016-04-25 19:18 UTC (permalink / raw)
To: Eric Engestrom
Cc: linux-kernel, Daniel Mack, Haojian Zhuang, Vinod Koul,
linux-arm-kernel, dmaengine
Eric Engestrom <eric.engestrom@imgtec.com> writes:
> Signed-off-by: Eric Engestrom <eric.engestrom@imgtec.com>
> ---
> drivers/dma/pxa_dma.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Acked-by: Robert Jarzmik <robert.jarzmik@free.fr>
--
Robert
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 1/4] perf tools: remove duplicate const qualifier
2016-04-25 9:47 [PATCH 1/4] perf tools: remove duplicate const qualifier Eric Engestrom
` (2 preceding siblings ...)
2016-04-25 9:47 ` [PATCH 4/4] ARM: " Eric Engestrom
@ 2016-04-25 21:11 ` Arnaldo Carvalho de Melo
2016-04-27 15:32 ` [tip:perf/core] perf tools: Remove " tip-bot for Eric Engestrom
4 siblings, 0 replies; 19+ messages in thread
From: Arnaldo Carvalho de Melo @ 2016-04-25 21:11 UTC (permalink / raw)
To: Eric Engestrom
Cc: linux-kernel, Peter Zijlstra, Ingo Molnar, David Ahern, Adrian Hunter
Em Mon, Apr 25, 2016 at 10:47:54AM +0100, Eric Engestrom escreveu:
> Signed-off-by: Eric Engestrom <eric.engestrom@imgtec.com>
Thanks, applied.
- Arnaldo
> ---
> tools/perf/util/thread.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/perf/util/thread.c b/tools/perf/util/thread.c
> index dfd00c6..de2036d 100644
> --- a/tools/perf/util/thread.c
> +++ b/tools/perf/util/thread.c
> @@ -233,7 +233,7 @@ void thread__find_cpumode_addr_location(struct thread *thread,
> struct addr_location *al)
> {
> size_t i;
> - const u8 const cpumodes[] = {
> + const u8 cpumodes[] = {
> PERF_RECORD_MISC_USER,
> PERF_RECORD_MISC_KERNEL,
> PERF_RECORD_MISC_GUEST_USER,
> --
> 2.7.4
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 4/4] ARM: remove duplicate const qualifier
2016-04-25 14:37 ` Eric Engestrom
@ 2016-04-26 10:12 ` Krzysztof Hałasa
2016-04-27 20:01 ` Nicolas Pitre
1 sibling, 0 replies; 19+ messages in thread
From: Krzysztof Hałasa @ 2016-04-26 10:12 UTC (permalink / raw)
To: Eric Engestrom
Cc: Nicolas Pitre, Arnd Bergmann, linux-arm-kernel,
Alexandre Belloni, Krzysztof Kozlowski, Tony Lindgren,
Linus Walleij, Nicolas Ferre, Lee Jones, Roland Stigge,
Florian Fainelli, Russell King, Alexander Shiyan, Kevin Hilman,
Viresh Kumar, bcm-kernel-feedback-list,
Jean-Christophe Plagniol-Villard, spear-devel, Ray Jui,
Rajendra Nayak, Sekhar Nori, Gregory Fong, linux-omap,
Paul Walmsley, Scott Branden, Shiraz Hashim, linux-kernel,
Sascha Hauer, Shawn Guo
Eric Engestrom <eric.engestrom@imgtec.com> writes:
> I can't confirm it (haven't tried), and don't care enough to do it :]
> I guess I'm just dropping the patch then. Like I said, it can't hurt to
> leave them in.
Actually it may hurt (a little bit) - it makes the code less readable.
--
Krzysztof Halasa
Industrial Research Institute for Automation and Measurements PIAP
Al. Jerozolimskie 202, 02-486 Warsaw, Poland
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 3/4] dmaengine: pxa_dma: remove duplicate const qualifier
2016-04-25 9:47 ` [PATCH 3/4] dmaengine: pxa_dma: " Eric Engestrom
2016-04-25 19:18 ` Robert Jarzmik
@ 2016-04-26 16:30 ` Vinod Koul
1 sibling, 0 replies; 19+ messages in thread
From: Vinod Koul @ 2016-04-26 16:30 UTC (permalink / raw)
To: Eric Engestrom
Cc: linux-kernel, Daniel Mack, Haojian Zhuang, Robert Jarzmik,
linux-arm-kernel, dmaengine
On Mon, Apr 25, 2016 at 10:47:56AM +0100, Eric Engestrom wrote:
> Signed-off-by: Eric Engestrom <eric.engestrom@imgtec.com>
> ---
> drivers/dma/pxa_dma.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/dma/pxa_dma.c b/drivers/dma/pxa_dma.c
> index 77c1c44..315901d 100644
> --- a/drivers/dma/pxa_dma.c
> +++ b/drivers/dma/pxa_dma.c
> @@ -1321,7 +1321,7 @@ static int pxad_init_phys(struct platform_device *op,
> return 0;
> }
>
> -static const struct of_device_id const pxad_dt_ids[] = {
> +static const struct of_device_id pxad_dt_ids[] = {
> { .compatible = "marvell,pdma-1.0", },
> {}
> };
Applied, thanks
--
~Vinod
^ permalink raw reply [flat|nested] 19+ messages in thread
* [tip:perf/core] perf tools: Remove duplicate const qualifier
2016-04-25 9:47 [PATCH 1/4] perf tools: remove duplicate const qualifier Eric Engestrom
` (3 preceding siblings ...)
2016-04-25 21:11 ` [PATCH 1/4] perf tools: " Arnaldo Carvalho de Melo
@ 2016-04-27 15:32 ` tip-bot for Eric Engestrom
4 siblings, 0 replies; 19+ messages in thread
From: tip-bot for Eric Engestrom @ 2016-04-27 15:32 UTC (permalink / raw)
To: linux-tip-commits
Cc: mingo, adrian.hunter, dsahern, linux-kernel, peterz, tglx, acme,
hpa, eric.engestrom
Commit-ID: 3b556bced46aa6b1873da7faa18eff235e896adc
Gitweb: http://git.kernel.org/tip/3b556bced46aa6b1873da7faa18eff235e896adc
Author: Eric Engestrom <eric.engestrom@imgtec.com>
AuthorDate: Mon, 25 Apr 2016 10:47:54 +0100
Committer: Arnaldo Carvalho de Melo <acme@redhat.com>
CommitDate: Mon, 25 Apr 2016 18:12:25 -0300
perf tools: Remove duplicate const qualifier
Signed-off-by: Eric Engestrom <eric.engestrom@imgtec.com>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1461577678-29517-1-git-send-email-eric.engestrom@imgtec.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
tools/perf/util/thread.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/perf/util/thread.c b/tools/perf/util/thread.c
index dfd00c6..de2036d 100644
--- a/tools/perf/util/thread.c
+++ b/tools/perf/util/thread.c
@@ -233,7 +233,7 @@ void thread__find_cpumode_addr_location(struct thread *thread,
struct addr_location *al)
{
size_t i;
- const u8 const cpumodes[] = {
+ const u8 cpumodes[] = {
PERF_RECORD_MISC_USER,
PERF_RECORD_MISC_KERNEL,
PERF_RECORD_MISC_GUEST_USER,
^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH 4/4] ARM: remove duplicate const qualifier
2016-04-25 14:37 ` Eric Engestrom
2016-04-26 10:12 ` Krzysztof Hałasa
@ 2016-04-27 20:01 ` Nicolas Pitre
1 sibling, 0 replies; 19+ messages in thread
From: Nicolas Pitre @ 2016-04-27 20:01 UTC (permalink / raw)
To: Eric Engestrom
Cc: Arnd Bergmann, linux-arm-kernel, Alexandre Belloni,
Krzysztof Kozlowski, Tony Lindgren, Linus Walleij, Nicolas Ferre,
Lee Jones, Roland Stigge, Florian Fainelli, Russell King,
Alexander Shiyan, Kevin Hilman, Viresh Kumar,
bcm-kernel-feedback-list, Jean-Christophe Plagniol-Villard,
spear-devel, Ray Jui, Rajendra Nayak, Sekhar Nori,
Krzysztof Halasa, Gregory Fong, linux-omap, Paul Walmsley,
Scott Branden, Shiraz Hashim, linux-kernel, Sascha Hauer,
Shawn Guo
On Mon, 25 Apr 2016, Eric Engestrom wrote:
> On Mon, Apr 25, 2016 at 10:12:16AM -0400, Nicolas Pitre wrote:
> > Beware.
> >
> > I added many of those exactly because gcc did not ignore them when
> > compiling with LTO where the lack of a const qualifier to qualify the
> > actual array content, and not only the reference to that content,
> > generated build errors due to section mismatches from the __initconst
> > annotation.
>
> I believe the first `const` does that, without the need for a second.
>
> > So this is a NAK from me unless you may confirm that LTO builds are
> > unaffected by your changes.
>
> I can't confirm it (haven't tried), and don't care enough to do it :]
> I guess I'm just dropping the patch then. Like I said, it can't hurt to
> leave them in.
I just tested LTO with your patch applied and everything appears to be
fine. The extra const are indeed redundant here. So you have an ACK
from me.
Acked-by: Nicolas Pitre <nico@linaro.org>
Nicolas
^ permalink raw reply [flat|nested] 19+ messages in thread
end of thread, other threads:[~2016-04-27 20:03 UTC | newest]
Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-04-25 9:47 [PATCH 1/4] perf tools: remove duplicate const qualifier Eric Engestrom
2016-04-25 9:47 ` [PATCH 2/4] drivers: firmware: psci: " Eric Engestrom
2016-04-25 10:33 ` Jisheng Zhang
2016-04-25 9:47 ` [PATCH 3/4] dmaengine: pxa_dma: " Eric Engestrom
2016-04-25 19:18 ` Robert Jarzmik
2016-04-26 16:30 ` Vinod Koul
2016-04-25 9:47 ` [PATCH 4/4] ARM: " Eric Engestrom
2016-04-25 9:52 ` Viresh Kumar
2016-04-25 9:57 ` Alexandre Belloni
2016-04-25 10:39 ` Eric Engestrom
2016-04-25 11:38 ` Arnd Bergmann
2016-04-25 12:58 ` Eric Engestrom
2016-04-25 14:12 ` Nicolas Pitre
2016-04-25 14:37 ` Eric Engestrom
2016-04-26 10:12 ` Krzysztof Hałasa
2016-04-27 20:01 ` Nicolas Pitre
2016-04-25 10:59 ` Sylvain Lemieux
2016-04-25 21:11 ` [PATCH 1/4] perf tools: " Arnaldo Carvalho de Melo
2016-04-27 15:32 ` [tip:perf/core] perf tools: Remove " tip-bot for Eric Engestrom
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).