Re: [PATCH 3/3] ARM: davinci: mark spi_board_info arrays as const
On Thursday 22 February 2018 10:13 PM, Arnd Bergmann wrote: > On Tue, Feb 20, 2018 at 5:24 PM, Arnd Bergmannwrote: >> Building with LTO revealed that three spi_board_info arrays are marked >> __initconst, but not const: >> >> arch/arm/mach-davinci/board-dm365-evm.c: In function 'dm365_evm_init': >> arch/arm/mach-davinci/board-dm365-evm.c:729:30: error: 'dm365_evm_spi_info' >> causes a section type conflict with 'dm646x_edma_device' >> static struct spi_board_info dm365_evm_spi_info[] __initconst = { >> ^ >> arch/arm/mach-davinci/dm646x.c:603:42: note: 'dm646x_edma_device' was >> declared here >> static const struct platform_device_info dm646x_edma_device __initconst = { >> >> This marks them const as well, as was originally intended. >> >> Signed-off-by: Arnd Bergmann > > I applied this to the fixes branch now. Thanks Arnd. FWIW now: Acked-by: Sekhar Nori Regards, Sekhar
Re: [PATCH 3/3] ARM: davinci: mark spi_board_info arrays as const
On Thursday 22 February 2018 10:13 PM, Arnd Bergmann wrote: > On Tue, Feb 20, 2018 at 5:24 PM, Arnd Bergmann wrote: >> Building with LTO revealed that three spi_board_info arrays are marked >> __initconst, but not const: >> >> arch/arm/mach-davinci/board-dm365-evm.c: In function 'dm365_evm_init': >> arch/arm/mach-davinci/board-dm365-evm.c:729:30: error: 'dm365_evm_spi_info' >> causes a section type conflict with 'dm646x_edma_device' >> static struct spi_board_info dm365_evm_spi_info[] __initconst = { >> ^ >> arch/arm/mach-davinci/dm646x.c:603:42: note: 'dm646x_edma_device' was >> declared here >> static const struct platform_device_info dm646x_edma_device __initconst = { >> >> This marks them const as well, as was originally intended. >> >> Signed-off-by: Arnd Bergmann > > I applied this to the fixes branch now. Thanks Arnd. FWIW now: Acked-by: Sekhar Nori Regards, Sekhar
Re: [PATCH 3/3] ARM: davinci: mark spi_board_info arrays as const
On Tue, Feb 20, 2018 at 5:24 PM, Arnd Bergmannwrote: > Building with LTO revealed that three spi_board_info arrays are marked > __initconst, but not const: > > arch/arm/mach-davinci/board-dm365-evm.c: In function 'dm365_evm_init': > arch/arm/mach-davinci/board-dm365-evm.c:729:30: error: 'dm365_evm_spi_info' > causes a section type conflict with 'dm646x_edma_device' > static struct spi_board_info dm365_evm_spi_info[] __initconst = { > ^ > arch/arm/mach-davinci/dm646x.c:603:42: note: 'dm646x_edma_device' was > declared here > static const struct platform_device_info dm646x_edma_device __initconst = { > > This marks them const as well, as was originally intended. > > Signed-off-by: Arnd Bergmann I applied this to the fixes branch now. Arnd
Re: [PATCH 3/3] ARM: davinci: mark spi_board_info arrays as const
On Tue, Feb 20, 2018 at 5:24 PM, Arnd Bergmann wrote: > Building with LTO revealed that three spi_board_info arrays are marked > __initconst, but not const: > > arch/arm/mach-davinci/board-dm365-evm.c: In function 'dm365_evm_init': > arch/arm/mach-davinci/board-dm365-evm.c:729:30: error: 'dm365_evm_spi_info' > causes a section type conflict with 'dm646x_edma_device' > static struct spi_board_info dm365_evm_spi_info[] __initconst = { > ^ > arch/arm/mach-davinci/dm646x.c:603:42: note: 'dm646x_edma_device' was > declared here > static const struct platform_device_info dm646x_edma_device __initconst = { > > This marks them const as well, as was originally intended. > > Signed-off-by: Arnd Bergmann I applied this to the fixes branch now. Arnd
[PATCH 3/3] ARM: davinci: mark spi_board_info arrays as const
Building with LTO revealed that three spi_board_info arrays are marked __initconst, but not const: arch/arm/mach-davinci/board-dm365-evm.c: In function 'dm365_evm_init': arch/arm/mach-davinci/board-dm365-evm.c:729:30: error: 'dm365_evm_spi_info' causes a section type conflict with 'dm646x_edma_device' static struct spi_board_info dm365_evm_spi_info[] __initconst = { ^ arch/arm/mach-davinci/dm646x.c:603:42: note: 'dm646x_edma_device' was declared here static const struct platform_device_info dm646x_edma_device __initconst = { This marks them const as well, as was originally intended. Signed-off-by: Arnd Bergmann--- arch/arm/mach-davinci/board-dm355-evm.c | 2 +- arch/arm/mach-davinci/board-dm355-leopard.c | 2 +- arch/arm/mach-davinci/board-dm365-evm.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/arch/arm/mach-davinci/board-dm355-evm.c b/arch/arm/mach-davinci/board-dm355-evm.c index e457f299cd44..d6b11907380c 100644 --- a/arch/arm/mach-davinci/board-dm355-evm.c +++ b/arch/arm/mach-davinci/board-dm355-evm.c @@ -368,7 +368,7 @@ static struct spi_eeprom at25640a = { .flags = EE_ADDR2, }; -static struct spi_board_info dm355_evm_spi_info[] __initconst = { +static const struct spi_board_info dm355_evm_spi_info[] __initconst = { { .modalias = "at25", .platform_data = , diff --git a/arch/arm/mach-davinci/board-dm355-leopard.c b/arch/arm/mach-davinci/board-dm355-leopard.c index be997243447b..fad9a5611a5d 100644 --- a/arch/arm/mach-davinci/board-dm355-leopard.c +++ b/arch/arm/mach-davinci/board-dm355-leopard.c @@ -217,7 +217,7 @@ static struct spi_eeprom at25640a = { .flags = EE_ADDR2, }; -static struct spi_board_info dm355_leopard_spi_info[] __initconst = { +static const struct spi_board_info dm355_leopard_spi_info[] __initconst = { { .modalias = "at25", .platform_data = , diff --git a/arch/arm/mach-davinci/board-dm365-evm.c b/arch/arm/mach-davinci/board-dm365-evm.c index e75741fb2c1d..e3780986d2a3 100644 --- a/arch/arm/mach-davinci/board-dm365-evm.c +++ b/arch/arm/mach-davinci/board-dm365-evm.c @@ -726,7 +726,7 @@ static struct spi_eeprom at25640 = { .flags = EE_ADDR2, }; -static struct spi_board_info dm365_evm_spi_info[] __initconst = { +static const struct spi_board_info dm365_evm_spi_info[] __initconst = { { .modalias = "at25", .platform_data = , -- 2.9.0
[PATCH 3/3] ARM: davinci: mark spi_board_info arrays as const
Building with LTO revealed that three spi_board_info arrays are marked __initconst, but not const: arch/arm/mach-davinci/board-dm365-evm.c: In function 'dm365_evm_init': arch/arm/mach-davinci/board-dm365-evm.c:729:30: error: 'dm365_evm_spi_info' causes a section type conflict with 'dm646x_edma_device' static struct spi_board_info dm365_evm_spi_info[] __initconst = { ^ arch/arm/mach-davinci/dm646x.c:603:42: note: 'dm646x_edma_device' was declared here static const struct platform_device_info dm646x_edma_device __initconst = { This marks them const as well, as was originally intended. Signed-off-by: Arnd Bergmann --- arch/arm/mach-davinci/board-dm355-evm.c | 2 +- arch/arm/mach-davinci/board-dm355-leopard.c | 2 +- arch/arm/mach-davinci/board-dm365-evm.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/arch/arm/mach-davinci/board-dm355-evm.c b/arch/arm/mach-davinci/board-dm355-evm.c index e457f299cd44..d6b11907380c 100644 --- a/arch/arm/mach-davinci/board-dm355-evm.c +++ b/arch/arm/mach-davinci/board-dm355-evm.c @@ -368,7 +368,7 @@ static struct spi_eeprom at25640a = { .flags = EE_ADDR2, }; -static struct spi_board_info dm355_evm_spi_info[] __initconst = { +static const struct spi_board_info dm355_evm_spi_info[] __initconst = { { .modalias = "at25", .platform_data = , diff --git a/arch/arm/mach-davinci/board-dm355-leopard.c b/arch/arm/mach-davinci/board-dm355-leopard.c index be997243447b..fad9a5611a5d 100644 --- a/arch/arm/mach-davinci/board-dm355-leopard.c +++ b/arch/arm/mach-davinci/board-dm355-leopard.c @@ -217,7 +217,7 @@ static struct spi_eeprom at25640a = { .flags = EE_ADDR2, }; -static struct spi_board_info dm355_leopard_spi_info[] __initconst = { +static const struct spi_board_info dm355_leopard_spi_info[] __initconst = { { .modalias = "at25", .platform_data = , diff --git a/arch/arm/mach-davinci/board-dm365-evm.c b/arch/arm/mach-davinci/board-dm365-evm.c index e75741fb2c1d..e3780986d2a3 100644 --- a/arch/arm/mach-davinci/board-dm365-evm.c +++ b/arch/arm/mach-davinci/board-dm365-evm.c @@ -726,7 +726,7 @@ static struct spi_eeprom at25640 = { .flags = EE_ADDR2, }; -static struct spi_board_info dm365_evm_spi_info[] __initconst = { +static const struct spi_board_info dm365_evm_spi_info[] __initconst = { { .modalias = "at25", .platform_data = , -- 2.9.0