Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933421Ab3CMO17 (ORCPT ); Wed, 13 Mar 2013 10:27:59 -0400 Received: from multi.imgtec.com ([194.200.65.239]:23762 "EHLO multi.imgtec.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932101Ab3CMO16 (ORCPT ); Wed, 13 Mar 2013 10:27:58 -0400 Message-ID: <51408CE8.70406@imgtec.com> Date: Wed, 13 Mar 2013 14:27:52 +0000 From: Markos Chandras User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130311 Thunderbird/17.0.3 MIME-Version: 1.0 To: Chris Ball CC: Seungwon Jeon , , , "'Jaehoon Chung'" Subject: Re: [PATCH 1/3] dw_mmc: Don't loop when handling an interrupt References: <1363085593-22214-1-git-send-email-markos.chandras@imgtec.com> <002501ce1ff6$38e3c470$aaab4d50$%jun@samsung.com> <87a9q7fk9l.fsf@octavius.laptop.org> In-Reply-To: <87a9q7fk9l.fsf@octavius.laptop.org> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [192.168.154.58] X-SEF-Processed: 7_3_0_01181__2013_03_13_14_27_52 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1053 Lines: 34 On 03/13/2013 02:26 PM, Chris Ball wrote: > Hi, > > On Wed, Mar 13 2013, Seungwon Jeon wrote: >> On Tuesday, March 12, 2013, Markos Chandras wrote: >>> There is no reason to loop when handling an interrupt. The "if" clauses >>> will handle all of them sequentially. This also eliminates the extra loop >>> we used to take with no pending interrupts and we ended up breaking out >>> of the while loop. >>> >>> Signed-off-by: Markos Chandras >>> Cc: Seungwon Jeon >> >> Acked-by: Seungwon Jeon >> >> You can add my ack to other patches. >> However, could you change the subject prefix to 'mmc: dw_mmc' and resend? > > It's okay, I'll take care of that when applying this time. Thanks! > > - Chris. > Hi Chris, Ok thank you Markos -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/