From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kleber Sacilotto de Souza Subject: Re: [PATCHv5 0/2] Speed Cap fixes for ppc64 Date: Wed, 15 May 2013 10:12:10 -0300 Message-ID: <519389AA.8090707@linux.vnet.ibm.com> References: <1367620993-27037-1-git-send-email-klebers@linux.vnet.ibm.com> <1367622108.4389.129.camel@pasglop> <519380FF.4020703@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: linuxppc-dev-bounces+glppe-linuxppc-embedded-2=m.gmane.org@lists.ozlabs.org Sender: "Linuxppc-dev" To: Alex Deucher Cc: David Airlie , dri-devel@lists.freedesktop.org, Bjorn Helgaas , Jerome Glisse , Thadeu Lima de Souza Cascardo , Brian King , Alex Deucher , linuxppc-dev@lists.ozlabs.org List-Id: dri-devel@lists.freedesktop.org On 05/15/2013 09:58 AM, Alex Deucher wrote: > On Wed, May 15, 2013 at 8:35 AM, Kleber Sacilotto de Souza > wrote: >> On 05/06/2013 11:32 AM, Alex Deucher wrote: >>> >>> On Fri, May 3, 2013 at 7:01 PM, Benjamin Herrenschmidt >>> wrote: >>>> >>>> On Fri, 2013-05-03 at 19:43 -0300, Kleber Sacilotto de Souza wrote: >>>> >>>>> This patch series does: >>>>> 1. max_bus_speed is used to set the device to gen2 speeds >>>>> 2. on power there's no longer a conflict between the pseries call and >>>>> other >>>>> architectures, because the overwrite is done via a ppc_md hook >>>>> 3. radeon is using bus->max_bus_speed instead of >>>>> drm_pcie_get_speed_cap_mask >>>>> for gen2 capability detection >>>>> >>>>> The first patch consists of some architecture changes, such as adding a >>>>> hook on >>>>> powerpc for pci_root_bridge_prepare, so that pseries will initialize it >>>>> to a >>>>> function, while all other architectures get a NULL pointer. So that >>>>> whenever >>>>> pci_create_root_bus is called, we'll get max_bus_speed properly setup >>>>> from >>>>> OpenFirmware. >>>>> >>>>> The second patch consists of simple radeon changes not to call >>>>> drm_get_pcie_speed_cap_mask anymore. I assume that on x86 machines, >>>>> the max_bus_speed property will be properly set already. >>>> >>>> >>>> So I'm ok with the approach now and I might even put the powerpc patch >>>> in for 3.10 since arguably we are fixing a nasty bug (uninitialized >>>> max_bus_speed). >>>> >>>> David, what's your feeling about the radeon change ? It would be nice if >>>> that could go in soon for various distro targets :-) On the other hand >>>> I'm not going to be pushy if you are not comfortable with it. >>> >>> >>> FWIW, the radeon change looks fine to me. >>> >>> Alex >>> >> >> Hi David, >> >> Are you planning to accept the radeon patch? If yes, can we still expect it >> to make it for 3.10? >> >> As Ben mentioned, we have some distro targets to make and it would be nice >> to have an outlook of the upstream acceptance to start the conversations >> with those distros. > > Is the rest of this series already upstream? I don't see a problem > pulling it in if the rest of the patches are in Dave's tree. Dave, do > you want to grab this, or do you wan me to pull it in through my tree? > > Alex > The powerpc patch was applied by Ben and is already upstream. Thanks, -- Kleber Sacilotto de Souza IBM Linux Technology Center