From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCH v8 00/20] OMAP2+: UART: Runtime adaptation + cleanup Date: Wed, 14 Dec 2011 10:56:21 -0800 Message-ID: <87r507oucq.fsf@ti.com> References: <1321005446-21741-1-git-send-email-govindraj.raja@ti.com> <87fwgow9vz.fsf@ti.com> <877h1znpdo.fsf@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: (Govindraj's message of "Wed, 14 Dec 2011 21:50:08 +0530") Sender: linux-serial-owner@vger.kernel.org To: Govindraj Cc: Tony Lindgren , linux-omap@vger.kernel.org, linux-serial@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Partha Basak , Vishwanath Sripathy , Rajendra Nayak , Santosh Shilimkar , "Govindraj.R" List-Id: linux-omap@vger.kernel.org Govindraj writes: > On Wed, Dec 14, 2011 at 8:59 PM, Kevin Hilman wrote: >> Govindraj writes: >> >>> On Wed, Dec 14, 2011 at 12:56 AM, Kevin Hilman wro= te: >>>> Govindraj writes: >>>> >>>> [...] >>>> >>>>> I have re-based this patch series against LO master >>>>> commit id: deee6d5359969a0ce4e2760cfd7b9f379bd5698a >>>>> >>>>> Same is available here [1] >>>>> >>>>> I have tested this patch series along with: >>>>> >>>>> 1.) Tero's V11 irq chaining series >>>>> =C2=A0 =C2=A0 =C2=A0http://www.spinics.net/lists/linux-omap/msg61= 445.html >>>>> =C2=A0 =C2=A0 (This patch series is used for uart wakeup handling= using >>>>> =C2=A0 =C2=A0 =C2=A0 prcm_irq chaining) >>>>> >>>>> 2.) Rajendra's hwmod change >>>>> =C2=A0 =C2=A0 =C2=A0http://www.spinics.net/lists/arm-kernel/msg14= 8632.html >>>>> =C2=A0 =C2=A0 =C2=A0(This patch handles init_no_idle flag setting >>>>> =C2=A0 =C2=A0 =C2=A0 without this patch there will be boot warnin= g however >>>>> =C2=A0 =C2=A0 =C2=A0 all pm features will work after boot up.) >>>>> >>>>> 3.) Vishwa's io daisy chain changes. >>>>> =C2=A0 =C2=A0 =C2=A0http://permalink.gmane.org/gmane.linux.ports.= arm.omap/65500 >>>>> =C2=A0 =C2=A0 =C2=A0(tested with and without this patch series pm= features works). >>>>> >>>>> Same combination of patches based on above commit id >>>>> used for testing is available here [2]. >>>> >>>> Please have a closer look at your branch. >>>> >>>> The second commit[1] commits the .rej file from a failed patch app= ly, >>>> so obviously doesn't do what was intended. >>>> >>> >>> Sorry my bad I have refreshed the uart runtime branch [1] >>> & test branch [2]. >> >> Can you add another patch which fixes these compiler warnings: >> >> /work/kernel/omap/pm/drivers/tty/serial/omap-serial.c: In function '= serial_omap_irq': >> /work/kernel/omap/pm/drivers/tty/serial/omap-serial.c:228:29: warnin= g: 'ch' may be used uninitialized in this function [-Wuninitialized] >> /work/kernel/omap/pm/drivers/tty/serial/omap-serial.c:176:16: note: = 'ch' was declared here >> /work/kernel/omap/pm/drivers/tty/serial/omap-serial.c:228:29: warnin= g: 'ch' may be used uninitialized in this function [-Wuninitialized] >> /work/kernel/omap/pm/drivers/tty/serial/omap-serial.c:176:16: note: = 'ch' was declared here >> > > Here is the patch [1] and pushed to > git://gitorious.org/runtime_3-0/runtime_3-0.git for_3_3/lo_rc4_uartru= ntime Thanks! Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-serial"= in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@ti.com (Kevin Hilman) Date: Wed, 14 Dec 2011 10:56:21 -0800 Subject: [PATCH v8 00/20] OMAP2+: UART: Runtime adaptation + cleanup In-Reply-To: (Govindraj's message of "Wed, 14 Dec 2011 21:50:08 +0530") References: <1321005446-21741-1-git-send-email-govindraj.raja@ti.com> <87fwgow9vz.fsf@ti.com> <877h1znpdo.fsf@ti.com> Message-ID: <87r507oucq.fsf@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Govindraj writes: > On Wed, Dec 14, 2011 at 8:59 PM, Kevin Hilman wrote: >> Govindraj writes: >> >>> On Wed, Dec 14, 2011 at 12:56 AM, Kevin Hilman wrote: >>>> Govindraj writes: >>>> >>>> [...] >>>> >>>>> I have re-based this patch series against LO master >>>>> commit id: deee6d5359969a0ce4e2760cfd7b9f379bd5698a >>>>> >>>>> Same is available here [1] >>>>> >>>>> I have tested this patch series along with: >>>>> >>>>> 1.) Tero's V11 irq chaining series >>>>> ? ? ?http://www.spinics.net/lists/linux-omap/msg61445.html >>>>> ? ? (This patch series is used for uart wakeup handling using >>>>> ? ? ? prcm_irq chaining) >>>>> >>>>> 2.) Rajendra's hwmod change >>>>> ? ? ?http://www.spinics.net/lists/arm-kernel/msg148632.html >>>>> ? ? ?(This patch handles init_no_idle flag setting >>>>> ? ? ? without this patch there will be boot warning however >>>>> ? ? ? all pm features will work after boot up.) >>>>> >>>>> 3.) Vishwa's io daisy chain changes. >>>>> ? ? ?http://permalink.gmane.org/gmane.linux.ports.arm.omap/65500 >>>>> ? ? ?(tested with and without this patch series pm features works). >>>>> >>>>> Same combination of patches based on above commit id >>>>> used for testing is available here [2]. >>>> >>>> Please have a closer look at your branch. >>>> >>>> The second commit[1] commits the .rej file from a failed patch apply, >>>> so obviously doesn't do what was intended. >>>> >>> >>> Sorry my bad I have refreshed the uart runtime branch [1] >>> & test branch [2]. >> >> Can you add another patch which fixes these compiler warnings: >> >> /work/kernel/omap/pm/drivers/tty/serial/omap-serial.c: In function 'serial_omap_irq': >> /work/kernel/omap/pm/drivers/tty/serial/omap-serial.c:228:29: warning: 'ch' may be used uninitialized in this function [-Wuninitialized] >> /work/kernel/omap/pm/drivers/tty/serial/omap-serial.c:176:16: note: 'ch' was declared here >> /work/kernel/omap/pm/drivers/tty/serial/omap-serial.c:228:29: warning: 'ch' may be used uninitialized in this function [-Wuninitialized] >> /work/kernel/omap/pm/drivers/tty/serial/omap-serial.c:176:16: note: 'ch' was declared here >> > > Here is the patch [1] and pushed to > git://gitorious.org/runtime_3-0/runtime_3-0.git for_3_3/lo_rc4_uartruntime Thanks! Kevin