From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933557AbbCRTj5 (ORCPT ); Wed, 18 Mar 2015 15:39:57 -0400 Received: from smtp.infotech.no ([82.134.31.41]:43871 "EHLO smtp.infotech.no" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932715AbbCRTjz (ORCPT ); Wed, 18 Mar 2015 15:39:55 -0400 Message-ID: <5509D46E.2020409@interlog.com> Date: Wed, 18 Mar 2015 15:39:26 -0400 From: Douglas Gilbert Reply-To: dgilbert@interlog.com User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 MIME-Version: 1.0 To: Greg Kroah-Hartman , Nicolas Ferre CC: Alan Stern , Boris Brezillon , Jean-Christophe Plagniol-Villard , Alexandre Belloni , Felipe Balbi , linux-usb@vger.kernel.org, Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Robert Nelson Subject: Re: [PATCH 1/5] USB: ehci-atmel: rework clk handling References: <55095E86.1060201@atmel.com> <20150318122854.GA18070@kroah.com> In-Reply-To: <20150318122854.GA18070@kroah.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 15-03-18 08:28 AM, Greg Kroah-Hartman wrote: > On Wed, Mar 18, 2015 at 12:16:22PM +0100, Nicolas Ferre wrote: >> Le 17/03/2015 20:01, Alan Stern a écrit : >>> On Tue, 17 Mar 2015, Boris Brezillon wrote: >>> >>>> The EHCI IP only needs the UTMI/UPLL (uclk) and the peripheral (iclk) >>>> clocks to work properly. Remove the useless system clock (fclk). >>>> >>>> Avoid calling set_rate on the fixed rate UTMI/IPLL clock and remove >>>> useless IS_ENABLED(CONFIG_COMMON_CLK) tests (all at91 platforms have been >>>> moved to the CCF). >>>> >>>> This patch also fixes a bug introduced by 3440ef1 (ARM: at91/dt: fix USB >>>> high-speed clock to select UTMI), which was leaving the usb clock >>>> uninitialized and preventing the OHCI driver from setting the usb clock >>>> rate to 48MHz. >>>> This bug was caused by several things: >>>> 1/ usb clock drivers set the CLK_SET_RATE_GATE flag, which means the rate >>>> cannot be changed once the clock is prepared >>>> 2/ The EHCI driver was retrieving and preparing/enabling the uhpck >>>> clock which was in turn preparing its parent clock (the usb clock), >>>> thus preventing any rate change because of 1/ >>>> >>>> Signed-off-by: Boris Brezillon >>> >>> Acked-by: Alan Stern >> >> Acked-by: Nicolas Ferre >> and: >> Fixes: 3440ef169100 ("ARM: at91/dt: fix USB high-speed clock to select UTMI") >> >> Alan, Greg, >> >> Can you please take this patch (only this patch 1/5 of the series) as a fix >> for the 4.0-rc? It would solve the issue that we see on at91sam9x5/at91sam9n12. >> I'll take care of the rest of the series for 4.1. >> >> If you want me to take it of to re-send the patch, tell me. > > I'll queue it up now, thanks. You can take the rest of the series :) Hi, I reported this error to Nicolas based on tests with lk 4.0-rc4. With the same USB WiFi dongle and platform this patch fixes my problem. Tested-by: Douglas Gilbert