* [PATCH] S5PC110: Add OneNAND interrupt number @ 2010-07-27 10:39 Kyungmin Park 2010-08-05 4:32 ` Artem Bityutskiy 0 siblings, 1 reply; 4+ messages in thread From: Kyungmin Park @ 2010-07-27 10:39 UTC (permalink / raw) To: linux-arm-kernel To use OneNAND interrupt at drivers, also extend DMA address to cover fully. Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com> --- diff --git a/arch/arm/mach-s5pv210/dev-onenand.c b/arch/arm/mach-s5pv210/dev-onenand.c index 34997b7..f8ede33 100644 --- a/arch/arm/mach-s5pv210/dev-onenand.c +++ b/arch/arm/mach-s5pv210/dev-onenand.c @@ -27,9 +27,14 @@ static struct resource s5pc110_onenand_resources[] = { }, [1] = { .start = S5PC110_PA_ONENAND_DMA, - .end = S5PC110_PA_ONENAND_DMA + SZ_2K - 1, + .end = S5PC110_PA_ONENAND_DMA + SZ_8K - 1, .flags = IORESOURCE_MEM, }, + [2] = { + .start = IRQ_ONENAND_AUDI, + .end = IRQ_ONENAND_AUDI, + .flags = IORESOURCE_IRQ, + }, }; struct platform_device s5pc110_device_onenand = { ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH] S5PC110: Add OneNAND interrupt number 2010-07-27 10:39 [PATCH] S5PC110: Add OneNAND interrupt number Kyungmin Park @ 2010-08-05 4:32 ` Artem Bityutskiy 2010-08-05 5:51 ` Kyungmin Park 0 siblings, 1 reply; 4+ messages in thread From: Artem Bityutskiy @ 2010-08-05 4:32 UTC (permalink / raw) To: linux-arm-kernel On Tue, 2010-07-27 at 19:39 +0900, Kyungmin Park wrote: > To use OneNAND interrupt at drivers, also extend DMA address to cover fully. > > Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com> > --- > diff --git a/arch/arm/mach-s5pv210/dev-onenand.c b/arch/arm/mach-s5pv210/dev-onenand.c > index 34997b7..f8ede33 100644 > --- a/arch/arm/mach-s5pv210/dev-onenand.c > +++ b/arch/arm/mach-s5pv210/dev-onenand.c > @@ -27,9 +27,14 @@ static struct resource s5pc110_onenand_resources[] = { > }, > [1] = { > .start = S5PC110_PA_ONENAND_DMA, > - .end = S5PC110_PA_ONENAND_DMA + SZ_2K - 1, > + .end = S5PC110_PA_ONENAND_DMA + SZ_8K - 1, > .flags = IORESOURCE_MEM, > }, > + [2] = { > + .start = IRQ_ONENAND_AUDI, > + .end = IRQ_ONENAND_AUDI, > + .flags = IORESOURCE_IRQ, > + }, > }; You do not expect this to go through the MTD tree, right? -- Best Regards, Artem Bityutskiy (????? ????????) ^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] S5PC110: Add OneNAND interrupt number 2010-08-05 4:32 ` Artem Bityutskiy @ 2010-08-05 5:51 ` Kyungmin Park 2010-08-05 6:15 ` Kukjin Kim 0 siblings, 1 reply; 4+ messages in thread From: Kyungmin Park @ 2010-08-05 5:51 UTC (permalink / raw) To: linux-arm-kernel On Thu, Aug 5, 2010 at 1:32 PM, Artem Bityutskiy <dedekind1@gmail.com> wrote: > On Tue, 2010-07-27 at 19:39 +0900, Kyungmin Park wrote: >> To use OneNAND interrupt at drivers, also extend DMA address to cover fully. >> >> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com> >> --- >> diff --git a/arch/arm/mach-s5pv210/dev-onenand.c b/arch/arm/mach-s5pv210/dev-onenand.c >> index 34997b7..f8ede33 100644 >> --- a/arch/arm/mach-s5pv210/dev-onenand.c >> +++ b/arch/arm/mach-s5pv210/dev-onenand.c >> @@ -27,9 +27,14 @@ static struct resource s5pc110_onenand_resources[] = { >> ? ? ? }, >> ? ? ? [1] = { >> ? ? ? ? ? ? ? .start ?= S5PC110_PA_ONENAND_DMA, >> - ? ? ? ? ? ? .end ? ?= S5PC110_PA_ONENAND_DMA + SZ_2K - 1, >> + ? ? ? ? ? ? .end ? ?= S5PC110_PA_ONENAND_DMA + SZ_8K - 1, >> ? ? ? ? ? ? ? .flags ?= IORESOURCE_MEM, >> ? ? ? }, >> + ? ? [2] = { >> + ? ? ? ? ? ? .start ?= IRQ_ONENAND_AUDI, >> + ? ? ? ? ? ? .end ? ?= IRQ_ONENAND_AUDI, >> + ? ? ? ? ? ? .flags ?= IORESOURCE_IRQ, >> + ? ? }, >> ?}; > > You do not expect this to go through the MTD tree, right? After this patch, I did a lot of works but failed to persuade the Kukjin. At this time. it's okay only this patch to mtd tree. Kukjin, are you okay? Since It's only affect the c110. Thank you, Kyungmin Park ^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] S5PC110: Add OneNAND interrupt number 2010-08-05 5:51 ` Kyungmin Park @ 2010-08-05 6:15 ` Kukjin Kim 0 siblings, 0 replies; 4+ messages in thread From: Kukjin Kim @ 2010-08-05 6:15 UTC (permalink / raw) To: linux-arm-kernel Kyungmin Park wrote: > > On Thu, Aug 5, 2010 at 1:32 PM, Artem Bityutskiy <dedekind1@gmail.com> wrote: > > On Tue, 2010-07-27 at 19:39 +0900, Kyungmin Park wrote: > >> To use OneNAND interrupt at drivers, also extend DMA address to cover fully. > >> > >> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com> > >> --- > >> diff --git a/arch/arm/mach-s5pv210/dev-onenand.c b/arch/arm/mach- > s5pv210/dev-onenand.c > >> index 34997b7..f8ede33 100644 > >> --- a/arch/arm/mach-s5pv210/dev-onenand.c > >> +++ b/arch/arm/mach-s5pv210/dev-onenand.c > >> @@ -27,9 +27,14 @@ static struct resource s5pc110_onenand_resources[] = { > >> ? ? ? }, > >> ? ? ? [1] = { > >> ? ? ? ? ? ? ? .start ?= S5PC110_PA_ONENAND_DMA, > >> - ? ? ? ? ? ? .end ? ?= S5PC110_PA_ONENAND_DMA + SZ_2K - 1, > >> + ? ? ? ? ? ? .end ? ?= S5PC110_PA_ONENAND_DMA + SZ_8K - 1, > >> ? ? ? ? ? ? ? .flags ?= IORESOURCE_MEM, > >> ? ? ? }, > >> + ? ? [2] = { > >> + ? ? ? ? ? ? .start ?= IRQ_ONENAND_AUDI, > >> + ? ? ? ? ? ? .end ? ?= IRQ_ONENAND_AUDI, > >> + ? ? ? ? ? ? .flags ?= IORESOURCE_IRQ, > >> + ? ? }, > >> ?}; > > > > You do not expect this to go through the MTD tree, right? > > After this patch, I did a lot of works but failed to persuade the Kukjin. Maybe I missed this single patch...looks ok itself. > At this time. it's okay only this patch to mtd tree. > I think, it's better this patch is going to upstream through ARM tree. Artem, I will apply in my tree. Thanks. > Kukjin, are you okay? > Since It's only affect the c110. > Thanks. Best regards, Kgene. -- Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer, SW Solution Development Team, Samsung Electronics Co., Ltd. ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2010-08-05 6:15 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2010-07-27 10:39 [PATCH] S5PC110: Add OneNAND interrupt number Kyungmin Park 2010-08-05 4:32 ` Artem Bityutskiy 2010-08-05 5:51 ` Kyungmin Park 2010-08-05 6:15 ` Kukjin Kim
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).