Subject: Re: [PATCH] net: emac: emac gigabit ethernet controller driver
To: Arnd Bergmann <arnd@arndb.de>, Christopher Covington <cov@codeaurora.org>
Cc: Florian Fainelli <f.fainelli@gmail.com>,
        Gilad Avidov <gavidov@codeaurora.org>, netdev@vger.kernel.org,
        linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
        linux-arm-msm@vger.kernel.org, sdharia@codeaurora.org,
        shankerd@codeaurora.org, gregkh@linuxfoundation.org,
        vikrams@codeaurora.org
References: <1450138740-32562-1-git-send-email-gavidov@codeaurora.org>
 <566F6F3D.5050004@gmail.com> <567023F8.80302@codeaurora.org>
 <1594472.ixZvj6PYdz@wuerfel>
From: Timur Tabi <timur@codeaurora.org>
Message-ID: <56702EEC.8000506@codeaurora.org>
Date: Tue, 15 Dec 2015 09:17:00 -0600
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:42.0)
 Gecko/20100101 Firefox/42.0 SeaMonkey/2.39
MIME-Version: 1.0
In-Reply-To: <1594472.ixZvj6PYdz@wuerfel>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 782
Lines: 22

Arnd Bergmann wrote:
> We generally want to use readl/writel rather than the relaxed versions,
> unless it is in performance-critical code.

What about if we have 20+ writes in a row, for example, when 
initializing a part?  I've seen code like this:

	writel_relaxed(...);
	writel_relaxed(...);
	writel_relaxed(...);
	...
	writel(...); // HW now inited, so enable

-- 
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the
Code Aurora Forum, hosted by The Linux Foundation.
--
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/