Re: [APPLIED] [PATCH 2/2] OMAP: Zoom2: Fix file system loading issue
* Kevin Hilman [090618 17:12]: > Tony Lindgren writes: > > > * Pandita, Vikram [090617 19:26]: > >> > >> > >> >-Original Message- > >> >From: Kevin Hilman [mailto:khil...@deeprootsystems.com] > >> >Sent: Wednesday, June 17, 2009 10:55 AM > >> >To: Tony Lindgren; Pandita, Vikram > >> >Cc: linux-omap@vger.kernel.org > >> >Subject: Re: [APPLIED] [PATCH 2/2] OMAP: Zoom2: Fix file system loading > >> >issue > >> > > >> >Tony Lindgren writes: > >> > > >> >> This patch has been applied to the linux-omap > >> >> by youw fwiendly patch wobot. > >> >> > >> >> Branch in linux-omap: omap-fixes > >> >> > >> >> Initial commit ID (Likely to change): > >> >> f84eca35d44fd64cdde542f12d08eb04ca534954 > >> >> > >> >> PatchWorks > >> >> http://patchwork.kernel.org/patch/29668/ > >> >> > >> >> Git (Likely to change, and takes a while to get mirrored) > >> >> http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap- > >> >2.6.git;a=commit;h=f84eca35d44fd64cdde542f12d08eb04ca534954 > >> > > >> >Sorry, I didn't get to this beforethe merge, but $SUBJECT for this > >> >patch is wrong and not terribly helpful. It refers to the symptom, bu > >> >tnot the root cause. > >> > > >> >It should be something like: > >> > > >> >OMAP3: Zoom2: pass IRQ triggering flags for serial driver > >> > >> Yes that should have been the subject and > >> is put that way for the 8250 driver: > >> http://lists.arm.linux.org.uk/lurker/attach/1...@20090612.173251.776f9eda.attach > >> > >> It's learning for next time. > > > > Will revert. I guess we now need two 8250 patches upstream? One for the > > omap register define, and one for the interrupt flags? > > Yes, both of these are pending on LKML, and still no word from Alan. > > To get it upstream, I defined wake-up register locally in > mach-omap2/serial.c until the patch gets merged, but for the interrupt > flags there isn't an easy solution. OK, so it's the irqflags that we still need. I took a look and commented that we should rather have irqflags field in struct uart_port instead.. So let's just wait and see until we have some ack from Alan. Regards, Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [APPLIED] [PATCH 2/2] OMAP: Zoom2: Fix file system loading issue
"Pandita, Vikram" writes: >> >>Will revert. I guess we now need two 8250 patches upstream? One for the >>omap register define, and one for the interrupt flags? > > Would be very helpful if you could host these 8250 patches on l-o > till we get someone to respond to these. This is a critical > dependency. FYI... I have these staged in the 'pm-upstream/backports' branch of my linux-omap-pm repo, since they are needed for PM testing on Zoom2. Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [APPLIED] [PATCH 2/2] OMAP: Zoom2: Fix file system loading issue
Tony Lindgren writes: > * Pandita, Vikram [090617 19:26]: >> >> >> >-Original Message- >> >From: Kevin Hilman [mailto:khil...@deeprootsystems.com] >> >Sent: Wednesday, June 17, 2009 10:55 AM >> >To: Tony Lindgren; Pandita, Vikram >> >Cc: linux-omap@vger.kernel.org >> >Subject: Re: [APPLIED] [PATCH 2/2] OMAP: Zoom2: Fix file system loading >> >issue >> > >> >Tony Lindgren writes: >> > >> >> This patch has been applied to the linux-omap >> >> by youw fwiendly patch wobot. >> >> >> >> Branch in linux-omap: omap-fixes >> >> >> >> Initial commit ID (Likely to change): >> >> f84eca35d44fd64cdde542f12d08eb04ca534954 >> >> >> >> PatchWorks >> >> http://patchwork.kernel.org/patch/29668/ >> >> >> >> Git (Likely to change, and takes a while to get mirrored) >> >> http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap- >> >2.6.git;a=commit;h=f84eca35d44fd64cdde542f12d08eb04ca534954 >> > >> >Sorry, I didn't get to this beforethe merge, but $SUBJECT for this >> >patch is wrong and not terribly helpful. It refers to the symptom, bu >> >tnot the root cause. >> > >> >It should be something like: >> > >> >OMAP3: Zoom2: pass IRQ triggering flags for serial driver >> >> Yes that should have been the subject and >> is put that way for the 8250 driver: >> http://lists.arm.linux.org.uk/lurker/attach/1...@20090612.173251.776f9eda.attach >> >> It's learning for next time. > > Will revert. I guess we now need two 8250 patches upstream? One for the > omap register define, and one for the interrupt flags? Yes, both of these are pending on LKML, and still no word from Alan. To get it upstream, I defined wake-up register locally in mach-omap2/serial.c until the patch gets merged, but for the interrupt flags there isn't an easy solution. Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [APPLIED] [PATCH 2/2] OMAP: Zoom2: Fix file system loading issue
>-Original Message- >From: Tony Lindgren [mailto:t...@atomide.com] >Sent: Thursday, June 18, 2009 12:27 AM >To: Pandita, Vikram >Cc: Kevin Hilman; linux-omap@vger.kernel.org >Subject: Re: [APPLIED] [PATCH 2/2] OMAP: Zoom2: Fix file system loading issue > >* Pandita, Vikram [090617 19:26]: >> >> >> >-Original Message- >> >From: Kevin Hilman [mailto:khil...@deeprootsystems.com] >> >Sent: Wednesday, June 17, 2009 10:55 AM >> >To: Tony Lindgren; Pandita, Vikram >> >Cc: linux-omap@vger.kernel.org >> >Subject: Re: [APPLIED] [PATCH 2/2] OMAP: Zoom2: Fix file system loading >> >issue >> > >> >Tony Lindgren writes: >> > >> >> This patch has been applied to the linux-omap >> >> by youw fwiendly patch wobot. >> >> >> >> Branch in linux-omap: omap-fixes >> >> >> >> Initial commit ID (Likely to change): >> >> f84eca35d44fd64cdde542f12d08eb04ca534954 >> >> >> >> PatchWorks >> >> http://patchwork.kernel.org/patch/29668/ >> >> >> >> Git (Likely to change, and takes a while to get mirrored) >> >> http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap- >> >2.6.git;a=commit;h=f84eca35d44fd64cdde542f12d08eb04ca534954 >> > >> >Sorry, I didn't get to this beforethe merge, but $SUBJECT for this >> >patch is wrong and not terribly helpful. It refers to the symptom, bu >> >tnot the root cause. >> > >> >It should be something like: >> > >> >OMAP3: Zoom2: pass IRQ triggering flags for serial driver >> >> Yes that should have been the subject and >> is put that way for the 8250 driver: >> http://lists.arm.linux.org.uk/lurker/attach/1...@20090612.173251.776f9eda.attach >> >> It's learning for next time. > >Will revert. I guess we now need two 8250 patches upstream? One for the >omap register define, and one for the interrupt flags? Would be very helpful if you could host these 8250 patches on l-o till we get someone to respond to these. This is a critical dependency. > >Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [APPLIED] [PATCH 2/2] OMAP: Zoom2: Fix file system loading issue
* Pandita, Vikram [090617 19:26]: > > > >-Original Message- > >From: Kevin Hilman [mailto:khil...@deeprootsystems.com] > >Sent: Wednesday, June 17, 2009 10:55 AM > >To: Tony Lindgren; Pandita, Vikram > >Cc: linux-omap@vger.kernel.org > >Subject: Re: [APPLIED] [PATCH 2/2] OMAP: Zoom2: Fix file system loading issue > > > >Tony Lindgren writes: > > > >> This patch has been applied to the linux-omap > >> by youw fwiendly patch wobot. > >> > >> Branch in linux-omap: omap-fixes > >> > >> Initial commit ID (Likely to change): > >> f84eca35d44fd64cdde542f12d08eb04ca534954 > >> > >> PatchWorks > >> http://patchwork.kernel.org/patch/29668/ > >> > >> Git (Likely to change, and takes a while to get mirrored) > >> http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap- > >2.6.git;a=commit;h=f84eca35d44fd64cdde542f12d08eb04ca534954 > > > >Sorry, I didn't get to this beforethe merge, but $SUBJECT for this > >patch is wrong and not terribly helpful. It refers to the symptom, bu > >tnot the root cause. > > > >It should be something like: > > > >OMAP3: Zoom2: pass IRQ triggering flags for serial driver > > Yes that should have been the subject and > is put that way for the 8250 driver: > http://lists.arm.linux.org.uk/lurker/attach/1...@20090612.173251.776f9eda.attach > > It's learning for next time. Will revert. I guess we now need two 8250 patches upstream? One for the omap register define, and one for the interrupt flags? Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [APPLIED] [PATCH 2/2] OMAP: Zoom2: Fix file system loading issue
>-Original Message- >From: Kevin Hilman [mailto:khil...@deeprootsystems.com] >Sent: Wednesday, June 17, 2009 10:55 AM >To: Tony Lindgren; Pandita, Vikram >Cc: linux-omap@vger.kernel.org >Subject: Re: [APPLIED] [PATCH 2/2] OMAP: Zoom2: Fix file system loading issue > >Tony Lindgren writes: > >> This patch has been applied to the linux-omap >> by youw fwiendly patch wobot. >> >> Branch in linux-omap: omap-fixes >> >> Initial commit ID (Likely to change): >> f84eca35d44fd64cdde542f12d08eb04ca534954 >> >> PatchWorks >> http://patchwork.kernel.org/patch/29668/ >> >> Git (Likely to change, and takes a while to get mirrored) >> http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap- >2.6.git;a=commit;h=f84eca35d44fd64cdde542f12d08eb04ca534954 > >Sorry, I didn't get to this beforethe merge, but $SUBJECT for this >patch is wrong and not terribly helpful. It refers to the symptom, bu >tnot the root cause. > >It should be something like: > >OMAP3: Zoom2: pass IRQ triggering flags for serial driver Yes that should have been the subject and is put that way for the 8250 driver: http://lists.arm.linux.org.uk/lurker/attach/1...@20090612.173251.776f9eda.attach It's learning for next time. > >Kevin > > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [APPLIED] [PATCH 2/2] OMAP: Zoom2: Fix file system loading issue
Tony Lindgren writes: > This patch has been applied to the linux-omap > by youw fwiendly patch wobot. > > Branch in linux-omap: omap-fixes > > Initial commit ID (Likely to change): f84eca35d44fd64cdde542f12d08eb04ca534954 > > PatchWorks > http://patchwork.kernel.org/patch/29668/ > > Git (Likely to change, and takes a while to get mirrored) > http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap-2.6.git;a=commit;h=f84eca35d44fd64cdde542f12d08eb04ca534954 Sorry, I didn't get to this beforethe merge, but $SUBJECT for this patch is wrong and not terribly helpful. It refers to the symptom, bu tnot the root cause. It should be something like: OMAP3: Zoom2: pass IRQ triggering flags for serial driver Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [APPLIED] [PATCH 2/2] OMAP: Zoom2: Fix file system loading issue
"Pandita, Vikram" writes: >>This patch has been applied to the linux-omap >>by youw fwiendly patch wobot. >> >>Branch in linux-omap: omap-fixes >> >>Initial commit ID (Likely to change): f84eca35d44fd64cdde542f12d08eb04ca534954 >> >>PatchWorks >>http://patchwork.kernel.org/patch/29668/ > > This patch breaks the zoom2 build. > ... > arch/arm/mach-omap2/board-zoom-debugboard.c:87: error: > 'UPF_IRQ_TRIG_HIGH' undeclared here (not in a function) > make[1]: *** [arch/arm/mach-omap2/board-zoom-debugboard.o] Error 1 > make: *** [arch/arm/mach-omap2] Error 2 > ... > > There is a dependency on 8250 patch: > http://lists.arm.linux.org.uk/lurker/attach/1...@20090612.173251.776f9eda.attach > > > how do you want to address this issue? > Tony, IMO, We should probably stage the 8250 UPF_* change in l-o as well while we wait for the linux-serial lag. Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [APPLIED] [PATCH 2/2] OMAP: Zoom2: Fix file system loading issue
Tony >-Original Message- >From: linux-omap-ow...@vger.kernel.org >[mailto:linux-omap-ow...@vger.kernel.org] On Behalf Of Tony >Lindgren >Sent: Wednesday, June 17, 2009 8:57 AM >To: linux-omap@vger.kernel.org >Subject: [APPLIED] [PATCH 2/2] OMAP: Zoom2: Fix file system loading issue > >This patch has been applied to the linux-omap >by youw fwiendly patch wobot. > >Branch in linux-omap: omap-fixes > >Initial commit ID (Likely to change): f84eca35d44fd64cdde542f12d08eb04ca534954 > >PatchWorks >http://patchwork.kernel.org/patch/29668/ This patch breaks the zoom2 build. ... arch/arm/mach-omap2/board-zoom-debugboard.c:87: error: 'UPF_IRQ_TRIG_HIGH' undeclared here (not in a function) make[1]: *** [arch/arm/mach-omap2/board-zoom-debugboard.o] Error 1 make: *** [arch/arm/mach-omap2] Error 2 ... There is a dependency on 8250 patch: http://lists.arm.linux.org.uk/lurker/attach/1...@20090612.173251.776f9eda.attach how do you want to address this issue? > >Git (Likely to change, and takes a while to get mirrored) >http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap- >2.6.git;a=commit;h=f84eca35d44fd64cdde542f12d08eb04ca534954 > > >-- >To unsubscribe from this list: send the line "unsubscribe linux-omap" in >the body of a message to majord...@vger.kernel.org >More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[APPLIED] [PATCH 2/2] OMAP: Zoom2: Fix file system loading issue
This patch has been applied to the linux-omap by youw fwiendly patch wobot. Branch in linux-omap: omap-fixes Initial commit ID (Likely to change): f84eca35d44fd64cdde542f12d08eb04ca534954 PatchWorks http://patchwork.kernel.org/patch/29668/ Git (Likely to change, and takes a while to get mirrored) http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap-2.6.git;a=commit;h=f84eca35d44fd64cdde542f12d08eb04ca534954 -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html