From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Shevchenko Subject: Re: [PATCHv2 0/4] ACPI / LPSS: Solution for two issues seen on Asus T100 Date: Fri, 16 May 2014 10:04:16 +0300 Message-ID: <1400223856.3703.44.camel@smile.fi.intel.com> References: <1400161226-24067-1-git-send-email-heikki.krogerus@linux.intel.com> <5374D0A8.2090407@linux.intel.com> <1400165610.3703.41.camel@smile.fi.intel.com> <5374E472.9030001@linux.intel.com> <20140515161148.GM1674@lahna.fi.intel.com> <53754DF4.10302@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: Received: from mga09.intel.com ([134.134.136.24]:18368 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753894AbaEPHEY (ORCPT ); Fri, 16 May 2014 03:04:24 -0400 In-Reply-To: <53754DF4.10302@linux.intel.com> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: "Li, Aubrey" Cc: Mika Westerberg , Heikki Krogerus , "Rafael J. Wysocki" , Mike Turquette , Jin Yao , linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org On Fri, 2014-05-16 at 07:29 +0800, Li, Aubrey wrote: > On 2014/5/16 0:11, Mika Westerberg wrote: > > On Thu, May 15, 2014 at 11:59:46PM +0800, Li, Aubrey wrote: > >> On 2014/5/15 22:53, Andy Shevchenko wrote: > >>> On Thu, 2014-05-15 at 22:35 +0800, Li, Aubrey wrote: > >>>> On 2014/5/15 21:40, Heikki Krogerus wrote: > >>>>> Changes since v1: > >>>>> - now using do_div() in clk_fd_recalc_rate() as suggested by Andy > >>>>> - NULL checks for clk_name allocation in acpi_lpss.c > >>>>> > >>>>> This combines two patch sets for LPSS that I had already send for > >>>>> review separately. They conflicted with each other. > >>>>> > >>>>> The first two patches will fix a problem were the context of the > >>>>> private LPSS registers is lost when entering D3. The last two will add > >>>>> support for the M/N dividers on LPSS by adding a new basic clock type > >>>>> for fractional dividers. The UART driver needs support for it in order > >>>>> to get clock rates that suit the requested baud rates. > >>>> > >>>> The major issue in my mind is, this proposal makes a couple between I2C > >>>> designware, HSUART, or probably DMA driver as well with LPSS driver. > >>> > >>> acpi_lpss driver creates platform devices for each found and enumerated > >>> device. > >> > >>> If there no acpi_lpss enabled the drivers work as supposed without it. > >> > >> This is not true. > > > > The drivers work fine on non-LPSS platform. If you make them depend on > > acpi_lpss, you break that. > > > > People don't know the relationship between LPSS driver and I2C/HSUART, > there is nowhere to describe that. If LPSS driver is unchecked, they > will encounter a weird issue which is very hard to figure out what's > going on. Besides this discussion is off the topic, I could say that LPSS drivers are kinda optional (we won't enforce user to use them) even on some systems where they are present. Relationship is provided by the proper kernel configuration. > > Thanks, > -Aubrey > > > -- Andy Shevchenko Intel Finland Oy