From mboxrd@z Thu Jan 1 00:00:00 1970 From: Timur Tabi Subject: Re: [PATCH v2] ata: increase retry count but shorten duration for Calxeda controller Date: Thu, 30 May 2013 09:39:48 -0500 Message-ID: <51A764B4.9020302@calxeda.com> References: <1369923509-3246-1-git-send-email-mark.langsdorf@calxeda.com> <51A76197.9080708@calxeda.com> <51A7646B.6080203@cogentembedded.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: Received: from smtp181.iad.emailsrvr.com ([207.97.245.181]:38487 "EHLO smtp181.iad.emailsrvr.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932733Ab3E3Ojc (ORCPT ); Thu, 30 May 2013 10:39:32 -0400 In-Reply-To: <51A7646B.6080203@cogentembedded.com> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Sergei Shtylyov Cc: Mark Langsdorf , "tj@kernel.org" , "linux-kernel@vger.kernel.org" , "linux-ide@vger.kernel.org" On 05/30/2013 09:38 AM, Sergei Shtylyov wrote: >>> >> - const unsigned long *timing = sata_ehc_deb_timing(&link->eh_context); >>> >> + const unsigned long timing[] = { 5, 100, 500}; >> > You'll save space and time if you also make this array "static", >> > otherwise the compiler will build the array every time this function is >> > called. > No, *const* specifier is enough to not build this array every time. > It will be put into the .const section. Ok. Now that I think about it, that makes sense. -- Timur Tabi Software Developer at Calxeda