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 07:29:07 -0800 Message-ID: <877h1znpdo.fsf@ti.com> References: <1321005446-21741-1-git-send-email-govindraj.raja@ti.com> <87fwgow9vz.fsf@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from na3sys009aog118.obsmtp.com ([74.125.149.244]:33109 "EHLO na3sys009aog118.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753859Ab1LNP3Q convert rfc822-to-8bit (ORCPT ); Wed, 14 Dec 2011 10:29:16 -0500 In-Reply-To: (Govindraj's message of "Wed, 14 Dec 2011 14:16:41 +0530") Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@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" 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 >>> =C2=A0 =C2=A0 =C2=A0http://www.spinics.net/lists/linux-omap/msg6144= 5.html >>> =C2=A0 =C2=A0 (This patch series is used for uart wakeup handling u= sing >>> =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/msg1486= 32.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 warning = 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.ar= m.omap/65500 >>> =C2=A0 =C2=A0 =C2=A0(tested with and without this patch series pm f= eatures 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 'ser= ial_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 Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-omap" i= n 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 07:29:07 -0800 Subject: [PATCH v8 00/20] OMAP2+: UART: Runtime adaptation + cleanup In-Reply-To: (Govindraj's message of "Wed, 14 Dec 2011 14:16:41 +0530") References: <1321005446-21741-1-git-send-email-govindraj.raja@ti.com> <87fwgow9vz.fsf@ti.com> Message-ID: <877h1znpdo.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 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 Kevin