From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from shards.monkeyblade.net (shards.monkeyblade.net [184.105.139.130]) by lists.ozlabs.org (Postfix) with ESMTP id 3rzXlk5ZbqzDqcn for ; Wed, 27 Jul 2016 08:26:26 +1000 (AEST) Date: Tue, 26 Jul 2016 15:26:24 -0700 (PDT) Message-Id: <20160726.152624.1129190524778208462.davem@davemloft.net> To: dan.carpenter@oracle.com Cc: benh@kernel.crashing.org, paulus@samba.org, mpe@ellerman.id.au, linux-ide@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, kernel-janitors@vger.kernel.org Subject: Re: [patch] ide: missing break statement in set_timings_mdma() From: David Miller In-Reply-To: <20160714104802.GI18175@mwanda> References: <20160714104802.GI18175@mwanda> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Dan Carpenter Date: Thu, 14 Jul 2016 13:48:02 +0300 > There was clearly supposed to be a break statement here. Currently we > use the k2 ata timings instead of sh ata ones we intended. Probably no > one has this hardware anymore so it likely doesn't make a difference > beyond the static checker warning. > > Signed-off-by: Dan Carpenter Applied.