From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bhadram Varka Subject: Re: [PATCH] net: stmmac: Use mutex instead of spinlock Date: Fri, 25 May 2018 20:06:47 +0530 Message-ID: References: <20180524140907.24197-1-thierry.reding@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit Cc: Giuseppe Cavallaro , Alexandre Torgue , Jon Hunter , , To: Thierry Reding , "David S. Miller" Return-path: In-Reply-To: <20180524140907.24197-1-thierry.reding@gmail.com> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org Hi, On 5/24/2018 7:39 PM, Thierry Reding wrote: > From: Thierry Reding > > Some drivers, such as DWC EQOS on Tegra, need to perform operations that > can sleep under this lock (clk_set_rate() in tegra_eqos_fix_speed()) for > proper operation. Since there is no need for this lock to be a spinlock, > convert it to a mutex instead. > > Fixes: e6ea2d16fc61 ("net: stmmac: dwc-qos: Add Tegra186 support") > Reported-by: Jon Hunter > Signed-off-by: Thierry Reding > --- Tested on P3310 Tegra186 platform. Tested-by: Bhadram Varka -- Bhadram