From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-qa0-f48.google.com ([209.85.216.48]:57634 "EHLO mail-qa0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752326AbaJBBTZ (ORCPT ); Wed, 1 Oct 2014 21:19:25 -0400 Received: by mail-qa0-f48.google.com with SMTP id dc16so1197906qab.7 for ; Wed, 01 Oct 2014 18:19:25 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <1412201282.4285.232.camel@pasglop> References: <1412129393.4285.195.camel@pasglop> <1412149289.4285.204.camel@pasglop> <1412201282.4285.232.camel@pasglop> Date: Wed, 1 Oct 2014 21:19:24 -0400 Message-ID: Subject: Re: [PATCH 4/4] sounds/hda/radeon: Disable 64-bit DMA on radeon From: Alex Deucher To: Benjamin Herrenschmidt Cc: Takashi Iwai , Bjorn Helgaas , Dave Airlie , Brian King , Linux PCI , Yijing Wang , Anton Blanchard , linuxppc-dev@ozlabs.org Content-Type: text/plain; charset=UTF-8 Sender: linux-pci-owner@vger.kernel.org List-ID: On Wed, Oct 1, 2014 at 6:08 PM, Benjamin Herrenschmidt wrote: > On Wed, 2014-10-01 at 13:58 -0400, Alex Deucher wrote: >> Patch looks good. Audio DMAs are limited to 40 bits, same as the GPU >> side. I'm still waiting to hear back on the MSIs for audio, but they >> probably follow the GPU side, so I expect they should be fixed on Sea >> Islands as well. > > In the audio driver we don't have the "family names", just a list of > PCI IDs, do you happen to know which ones are unaffected ? The list of unaffected ones are: 0x1308 //kaveri 0x9840 //Kabini 0xaac0 //bonaire 0xaac8 //hawaii 0xaad8 //tonga Pus all newer asics. Alternatively, the list of older asics that are affected (some of these may not even support MSI64 in the first place): 0x1314 //palm 0x7919 //RS690 0x793b //RS600 0x7969 //RS740 0x960f //RS780 0x9646 //sumo 0x970f //RS880 0x9902 //trinity 0xaa00 //R600 0xaa08 //RV630 0xaa10 //RV610 0xaa18 //RV670 0xaa20 //RV635 0xaa28 //RV620 0xaa30 //RV770 0xaa38 //RV730 0xaa40 //RV710 0xaa48 //RV740 0xaa50 //cypress 0xaa58 //juniper 0xaa60 //redwood 0xaa68 //cedar 0xaa80 //cayman 0xaa88 //barts 0xaa90 //turks 0xaa98 //caicos 0xaaa0 //tahiti 0xaab0 //verde, pitcairn, oland It might be better to only set the msi32 flag on the above affected asics so we don't have to worry about new ones. Alex