diff for duplicates of <87lj6uj1zu.fsf@deeprootsystems.com> diff --git a/a/1.txt b/N1/1.txt index 7953737..b576232 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -10,14 +10,14 @@ Govindraj <govindraj.ti@gmail.com> writes: >>>> boards, specifically 3630-based boards. >>>> >>>> First, due to the current UART idle code base, you need to enable all ->>>> OMAP UARTs 36xx. Enabling less than all OMAP UARTs will break the ->>>> current idle code. As we discussed, the next phase we will move the +>>>> OMAP UARTs 36xx. ?Enabling less than all OMAP UARTs will break the +>>>> current idle code. ?As we discussed, the next phase we will move the >>>> idle management from this serial.c hackery into the omap-serial driver ->>>> iteself. Until then, you need to call omap_serial_init() on ->>>> Zoom2/Zoom3. Patch below[1] +>>>> iteself. ?Until then, you need to call omap_serial_init() on +>>>> Zoom2/Zoom3. ?Patch below[1] >>>> >>>> Also, you previously had a patch that updated omap_uart_idle_init() to ->>>> handle 36xx and specifically UART4. Without that, struct +>>>> handle 36xx and specifically UART4. ?Without that, struct >>>> omap_uart_state is not setup correctly for UART4, and thus cannot be >>>> properly idled on 3630. >>> @@ -29,18 +29,18 @@ Govindraj <govindraj.ti@gmail.com> writes: >>> Is this still needed if we have initialized all uarts? >>> This patch might not needed if we have initialized all uarts right? >> ->> Right. We don't need the above patchwork patch if all UARTs are +>> Right. ?We don't need the above patchwork patch if all UARTs are >> initialized. >> >> The other patch I was referring to was the one that added UART4 support ->> to omap_uart_idle_init() (added the wk_en, wk_st, padconf etc.) I had a +>> to omap_uart_idle_init() (added the wk_en, wk_st, padconf etc.) ?I had a >> pending request for you to drop the muxmode from that patch, but the >> rest of it was still needed. >> >>>> ->>>> Also, it's been a while since I tested this on OMAP2. Please re-test on ->>>> OMAP2 with the whole series. Also, please report here the other ->>>> platforms this was tested on. The final needs to be tested on OMAP2, 3 +>>>> Also, it's been a while since I tested this on OMAP2. ?Please re-test on +>>>> OMAP2 with the whole series. ?Also, please report here the other +>>>> platforms this was tested on. ?The final needs to be tested on OMAP2, 3 >>>> and 4 before merge. >>> >>> Yes Sure, @@ -96,8 +96,3 @@ else (padconf, wk_en, wk_st) was fine. Please add this back to the series. 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 diff --git a/a/content_digest b/N1/content_digest index eb6e267..c0c24ef 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -3,15 +3,10 @@ "ref\0AANLkTik-+RC7y=dJnDdxzrqMFnTzRV-j8s5FtmMy8ZhE@mail.gmail.com\0" "ref\087wrqg2xog.fsf@deeprootsystems.com\0" "ref\0AANLkTinqiknLaBmMj17fi31TsZ+eaSkCcgd-wbQvMpz-@mail.gmail.com\0" - "From\0Kevin Hilman <khilman@deeprootsystems.com>\0" - "Subject\0Re: [PATCH 00/11] OMAP: Serial: Add omap-serial driver with platform support\0" + "From\0khilman@deeprootsystems.com (Kevin Hilman)\0" + "Subject\0[PATCH 00/11] OMAP: Serial: Add omap-serial driver with platform support\0" "Date\0Tue, 21 Sep 2010 13:05:57 -0700\0" - "To\0Govindraj <govindraj.ti@gmail.com>\0" - "Cc\0Govindraj.R <govindraj.raja@ti.com>" - linux-omap@vger.kernel.org - linux-arm-kernel@lists.infradead.org - linux-serial@vger.kernel.org - " Tony Lindgren <tony@atomide.com>\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "Govindraj <govindraj.ti@gmail.com> writes:\n" @@ -26,14 +21,14 @@ ">>>> boards, specifically 3630-based boards.\n" ">>>>\n" ">>>> First, due to the current UART idle code base, you need to enable all\n" - ">>>> OMAP UARTs 36xx. \302\240Enabling less than all OMAP UARTs will break the\n" - ">>>> current idle code. \302\240As we discussed, the next phase we will move the\n" + ">>>> OMAP UARTs 36xx. ?Enabling less than all OMAP UARTs will break the\n" + ">>>> current idle code. ?As we discussed, the next phase we will move the\n" ">>>> idle management from this serial.c hackery into the omap-serial driver\n" - ">>>> iteself. \302\240Until then, you need to call omap_serial_init() on\n" - ">>>> Zoom2/Zoom3. \302\240Patch below[1]\n" + ">>>> iteself. ?Until then, you need to call omap_serial_init() on\n" + ">>>> Zoom2/Zoom3. ?Patch below[1]\n" ">>>>\n" ">>>> Also, you previously had a patch that updated omap_uart_idle_init() to\n" - ">>>> handle 36xx and specifically UART4. \302\240Without that, struct\n" + ">>>> handle 36xx and specifically UART4. ?Without that, struct\n" ">>>> omap_uart_state is not setup correctly for UART4, and thus cannot be\n" ">>>> properly idled on 3630.\n" ">>>\n" @@ -45,18 +40,18 @@ ">>> Is this still needed if we have initialized all uarts?\n" ">>> This patch might not needed if we have initialized all uarts right?\n" ">>\n" - ">> Right. \302\240We don't need the above patchwork patch if all UARTs are\n" + ">> Right. ?We don't need the above patchwork patch if all UARTs are\n" ">> initialized.\n" ">>\n" ">> The other patch I was referring to was the one that added UART4 support\n" - ">> to omap_uart_idle_init() (added the wk_en, wk_st, padconf etc.) \302\240I had a\n" + ">> to omap_uart_idle_init() (added the wk_en, wk_st, padconf etc.) ?I had a\n" ">> pending request for you to drop the muxmode from that patch, but the\n" ">> rest of it was still needed.\n" ">>\n" ">>>>\n" - ">>>> Also, it's been a while since I tested this on OMAP2. \302\240Please re-test on\n" - ">>>> OMAP2 with the whole series. \302\240Also, please report here the other\n" - ">>>> platforms this was tested on. \302\240The final needs to be tested on OMAP2, 3\n" + ">>>> Also, it's been a while since I tested this on OMAP2. ?Please re-test on\n" + ">>>> OMAP2 with the whole series. ?Also, please report here the other\n" + ">>>> platforms this was tested on. ?The final needs to be tested on OMAP2, 3\n" ">>>> and 4 before merge.\n" ">>>\n" ">>> Yes Sure,\n" @@ -111,11 +106,6 @@ "\n" "Please add this back to the series.\n" "\n" - "Kevin\n" - "\n" - "--\n" - "To unsubscribe from this list: send the line \"unsubscribe linux-serial\" in\n" - "the body of a message to majordomo@vger.kernel.org\n" - More majordomo info at http://vger.kernel.org/majordomo-info.html + Kevin -1849072cb0a897013a049b6b2552ac78211d9ff0d96ab1334b9eb8dcb9d39f6e +2308d4a5c891f4d45600cdddc137a9eedbd250568672b8794c009418d0c99590
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.