* [PATCH 1/3] mtd: st_spi_fsm: Make of_device_id array const
@ 2014-05-07 8:46 Jingoo Han
2014-05-07 8:49 ` [PATCH 2/3] mtd: nand: pxa3xx: " Jingoo Han
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Jingoo Han @ 2014-05-07 8:46 UTC (permalink / raw)
To: 'Brian Norris'
Cc: linux-mtd, 'Lee Jones', 'Jingoo Han',
'David Woodhouse'
Make of_device_id array const, because all OF functions
handle it as const.
Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
drivers/mtd/devices/st_spi_fsm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mtd/devices/st_spi_fsm.c b/drivers/mtd/devices/st_spi_fsm.c
index f97fe14..d252514 100644
--- a/drivers/mtd/devices/st_spi_fsm.c
+++ b/drivers/mtd/devices/st_spi_fsm.c
@@ -2058,7 +2058,7 @@ static int stfsm_remove(struct platform_device *pdev)
return mtd_device_unregister(&fsm->mtd);
}
-static struct of_device_id stfsm_match[] = {
+static const struct of_device_id stfsm_match[] = {
{ .compatible = "st,spi-fsm", },
{},
};
--
1.7.10.4
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/3] mtd: nand: pxa3xx: Make of_device_id array const
2014-05-07 8:46 [PATCH 1/3] mtd: st_spi_fsm: Make of_device_id array const Jingoo Han
@ 2014-05-07 8:49 ` Jingoo Han
2014-05-07 12:49 ` Jason Cooper
2014-05-07 8:50 ` [PATCH 3/3] mtd: orion_nand: " Jingoo Han
2014-05-12 23:21 ` [PATCH 1/3] mtd: st_spi_fsm: " Brian Norris
2 siblings, 1 reply; 7+ messages in thread
From: Jingoo Han @ 2014-05-07 8:49 UTC (permalink / raw)
To: 'Brian Norris'
Cc: linux-mtd, 'Jingoo Han', 'David Woodhouse',
'Jason Cooper', 'Ezequiel Garcia'
Make of_device_id array const, because all OF functions
handle it as const.
Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
drivers/mtd/nand/pxa3xx_nand.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
index 7588fe2..940e6c7 100644
--- a/drivers/mtd/nand/pxa3xx_nand.c
+++ b/drivers/mtd/nand/pxa3xx_nand.c
@@ -337,7 +337,7 @@ static struct nand_ecclayout ecc_layout_4KB_bch8bit = {
/* convert nano-seconds to nand flash controller clock cycles */
#define ns2cycle(ns, clk) (int)((ns) * (clk / 1000000) / 1000)
-static struct of_device_id pxa3xx_nand_dt_ids[] = {
+static const struct of_device_id pxa3xx_nand_dt_ids[] = {
{
.compatible = "marvell,pxa3xx-nand",
.data = (void *)PXA3XX_NAND_VARIANT_PXA,
--
1.7.10.4
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 3/3] mtd: orion_nand: Make of_device_id array const
2014-05-07 8:46 [PATCH 1/3] mtd: st_spi_fsm: Make of_device_id array const Jingoo Han
2014-05-07 8:49 ` [PATCH 2/3] mtd: nand: pxa3xx: " Jingoo Han
@ 2014-05-07 8:50 ` Jingoo Han
2014-05-07 13:41 ` Andrew Lunn
2014-05-12 23:21 ` [PATCH 1/3] mtd: st_spi_fsm: " Brian Norris
2 siblings, 1 reply; 7+ messages in thread
From: Jingoo Han @ 2014-05-07 8:50 UTC (permalink / raw)
To: 'Brian Norris'
Cc: linux-mtd, 'Jingoo Han', 'David Woodhouse',
'Andrew Lunn'
Make of_device_id array const, because all OF functions
handle it as const.
Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
drivers/mtd/nand/orion_nand.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mtd/nand/orion_nand.c b/drivers/mtd/nand/orion_nand.c
index dd7fe81..471b4df 100644
--- a/drivers/mtd/nand/orion_nand.c
+++ b/drivers/mtd/nand/orion_nand.c
@@ -214,7 +214,7 @@ static int orion_nand_remove(struct platform_device *pdev)
}
#ifdef CONFIG_OF
-static struct of_device_id orion_nand_of_match_table[] = {
+static const struct of_device_id orion_nand_of_match_table[] = {
{ .compatible = "marvell,orion-nand", },
{},
};
--
1.7.10.4
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 2/3] mtd: nand: pxa3xx: Make of_device_id array const
2014-05-07 8:49 ` [PATCH 2/3] mtd: nand: pxa3xx: " Jingoo Han
@ 2014-05-07 12:49 ` Jason Cooper
2014-05-07 19:37 ` 'Ezequiel Garcia'
0 siblings, 1 reply; 7+ messages in thread
From: Jason Cooper @ 2014-05-07 12:49 UTC (permalink / raw)
To: Jingoo Han
Cc: linux-mtd, 'Brian Norris', 'David Woodhouse',
'Ezequiel Garcia'
On Wed, May 07, 2014 at 05:49:13PM +0900, Jingoo Han wrote:
> Make of_device_id array const, because all OF functions
> handle it as const.
>
> Signed-off-by: Jingoo Han <jg1.han@samsung.com>
> ---
> drivers/mtd/nand/pxa3xx_nand.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Acked-by: Jason Cooper <jason@lakedaemon.net>
thx,
Jason.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 3/3] mtd: orion_nand: Make of_device_id array const
2014-05-07 8:50 ` [PATCH 3/3] mtd: orion_nand: " Jingoo Han
@ 2014-05-07 13:41 ` Andrew Lunn
0 siblings, 0 replies; 7+ messages in thread
From: Andrew Lunn @ 2014-05-07 13:41 UTC (permalink / raw)
To: Jingoo Han
Cc: linux-mtd, 'Brian Norris', 'David Woodhouse',
'Andrew Lunn'
On Wed, May 07, 2014 at 05:50:23PM +0900, Jingoo Han wrote:
> Make of_device_id array const, because all OF functions
> handle it as const.
>
> Signed-off-by: Jingoo Han <jg1.han@samsung.com>
> ---
> drivers/mtd/nand/orion_nand.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mtd/nand/orion_nand.c b/drivers/mtd/nand/orion_nand.c
> index dd7fe81..471b4df 100644
> --- a/drivers/mtd/nand/orion_nand.c
> +++ b/drivers/mtd/nand/orion_nand.c
> @@ -214,7 +214,7 @@ static int orion_nand_remove(struct platform_device *pdev)
> }
>
> #ifdef CONFIG_OF
> -static struct of_device_id orion_nand_of_match_table[] = {
> +static const struct of_device_id orion_nand_of_match_table[] = {
> { .compatible = "marvell,orion-nand", },
> {},
> };
Acked-by: Andrew Lunn <andrew@lunn.ch>
Andrew
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/3] mtd: nand: pxa3xx: Make of_device_id array const
2014-05-07 12:49 ` Jason Cooper
@ 2014-05-07 19:37 ` 'Ezequiel Garcia'
0 siblings, 0 replies; 7+ messages in thread
From: 'Ezequiel Garcia' @ 2014-05-07 19:37 UTC (permalink / raw)
To: Jingoo Han, Jason Cooper
Cc: linux-mtd, 'Brian Norris', 'David Woodhouse'
On 07 May 08:49 AM, Jason Cooper wrote:
> On Wed, May 07, 2014 at 05:49:13PM +0900, Jingoo Han wrote:
> > Make of_device_id array const, because all OF functions
> > handle it as const.
> >
> > Signed-off-by: Jingoo Han <jg1.han@samsung.com>
> > ---
> > drivers/mtd/nand/pxa3xx_nand.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
>
> Acked-by: Jason Cooper <jason@lakedaemon.net>
>
Acked-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
--
Ezequiel García, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/3] mtd: st_spi_fsm: Make of_device_id array const
2014-05-07 8:46 [PATCH 1/3] mtd: st_spi_fsm: Make of_device_id array const Jingoo Han
2014-05-07 8:49 ` [PATCH 2/3] mtd: nand: pxa3xx: " Jingoo Han
2014-05-07 8:50 ` [PATCH 3/3] mtd: orion_nand: " Jingoo Han
@ 2014-05-12 23:21 ` Brian Norris
2 siblings, 0 replies; 7+ messages in thread
From: Brian Norris @ 2014-05-12 23:21 UTC (permalink / raw)
To: Jingoo Han; +Cc: linux-mtd, 'Lee Jones', 'David Woodhouse'
On Wed, May 07, 2014 at 05:46:59PM +0900, Jingoo Han wrote:
> Make of_device_id array const, because all OF functions
> handle it as const.
>
> Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Pushed all 3 to l2-mtd.git. Thanks!
Brian
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2014-05-12 23:21 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-05-07 8:46 [PATCH 1/3] mtd: st_spi_fsm: Make of_device_id array const Jingoo Han
2014-05-07 8:49 ` [PATCH 2/3] mtd: nand: pxa3xx: " Jingoo Han
2014-05-07 12:49 ` Jason Cooper
2014-05-07 19:37 ` 'Ezequiel Garcia'
2014-05-07 8:50 ` [PATCH 3/3] mtd: orion_nand: " Jingoo Han
2014-05-07 13:41 ` Andrew Lunn
2014-05-12 23:21 ` [PATCH 1/3] mtd: st_spi_fsm: " Brian Norris
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).