From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754504AbdLNX7K (ORCPT ); Thu, 14 Dec 2017 18:59:10 -0500 Received: from mail-cys01nam02on0074.outbound.protection.outlook.com ([104.47.37.74]:64648 "EHLO NAM02-CY1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754291AbdLNX7E (ORCPT ); Thu, 14 Dec 2017 18:59:04 -0500 Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=andrew.cooks@opengear.com; Subject: Re: [PATCH v2 1/3] i2c: piix4: Fix SMBus port selection for AMD Family 16h chips To: jdelvare@suse.com, linux-i2c@vger.kernel.org Cc: platypus-sw@opengear.com, Wolfram Sang , open list References: <3f87f37117e6662661f16ea8fdbbdffff5d2e414.1513215039.git.andrew.cooks@opengear.com> From: Andrew Cooks Message-ID: Date: Fri, 15 Dec 2017 09:58:44 +1000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 MIME-Version: 1.0 In-Reply-To: <3f87f37117e6662661f16ea8fdbbdffff5d2e414.1513215039.git.andrew.cooks@opengear.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [59.167.150.161] X-ClientProxiedBy: ME1PR01CA0119.ausprd01.prod.outlook.com (2603:10c6:200:19::28) To BY1PR15MB0023.namprd15.prod.outlook.com (2a01:111:e400:5079::25) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 302b3ee0-c550-4a2d-f5fc-08d5434ea5d1 X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(4534020)(4602075)(4627115)(201703031133081)(201702281549075)(5600026)(4604075)(2017052603307);SRVR:BY1PR15MB0023; X-Microsoft-Exchange-Diagnostics: 1;BY1PR15MB0023;3:PmrfARs1UhO7JSwes73MwWcnXqVBIUquZMSNqR7snlO20ii7tJi2NRAEJtxMLrl3DsCRVX7uhvwZJDzeoJ/m2clNp016IDy8CW9v8aaL9XwkoVr7l5bT5qOJ9cxdIawLVWDbqmRDx11tirGwlLnHjX36zny5f8Cr8coLEGmz7+2kQf85qzdGNhUpK1BVRdYqoJUclvslkgmCsxuCBuBqXZ6csFTG78eRJ+IZm4ueI/B9DlwEcf+qdXd9Weq4mzI8;25:dSxvqcMjXNfmzs9OzwdGwngw6Hjx/CM4aiXYVdDf2XRH+B0JunHXfg28DhR4SwNxIfv5Lrls5bBNQqYPfGYAn2FyWSokP16vi0fS7sbAZ/BvyCmPxxAK0c4whLX6YTgbx3SPs0Gka+cz8roB303OGD6ItDRVIFapp2vt+mwuRA8RY5p56dqd9/Y5ObtXuSFp/KHL6x5nFukZmkx1AzrBgq5dLQS/emRznL+R42YYRV5pJuuFjEZX4SIXo6Xyk94mt89tH0/YS82tG789ahLxxLMnZqHkrTapUXitDyZHx6aN1HqS0+lkflwbWdq+1VPcZd/YTffP5AiSJKrVDXo/tHE6i7AeyCFMgWeXc4R656A=;31:69CqGCmZO7q5x81RE2QcYN2vznXS9yhRegkjbg1bSSTkskgx+MJgbrJTG4NPxGBL41REbq6Sssya8KjR3P8p5JwtghZSBnZ/f6Q4bBh8chRdRJfqYA6Trh5N2pNwioWlrCp+aiYNl5ULdoJK4HdSqmr8R+oUR9Jwb/eptaiMXxGWr3Rg3bh7brEm8VknCTItqrt6K9Vibr53FVcBBSf3ZdzVmbwBwkdKVpXSn1/0T5s= X-MS-TrafficTypeDiagnostic: BY1PR15MB0023: X-Microsoft-Exchange-Diagnostics: 1;BY1PR15MB0023;20:7Ga9BwuskOw9irAsIjm9URBdqghhgN8suWNZRBSaeRmsRAglHnryO9pApRXZtaSk5uX9QRKQJSeTXVFUe7zYjTAwRZt+MdAFIfnfTHdVNT2vtzEbrC99Jmotgj2nGXYiqtmiatMjTLvX/GGa99P7TE3ww5cnZ7uJoH+KfNJMKUY=;4:UjaD4O7B6EknQVVdpiv6XDtiOUAX/S/yBlTw2K28bZJnlUKYVzW0IfQHmmNcTGz4Eu5ukt4UKspHcD1Bfmw+HsWC1sbVEB7sbjJBlAMxtbZyzAr+nK+0XmOh6AWRKzs5Kyyt7oUBePw+5FiNharLtW8S34gBKQEKyC4bBe5QazVu0LU53azaP2eohErDMmkgut3Mjc7Lx44DhHM/CLobELq3/Uyyoggo3yhIEueqJ+kAIGVXIOYC2+E8bzOXS7j6dV5xLluXtIRpo8qDJ/f+9FbnKlqIJTca9xffIPG88X01CMseknHw9vOjWPfG3qnB X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(767451399110); X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(6040450)(2401047)(5005006)(8121501046)(93006095)(93001095)(3002001)(10201501046)(3231023)(6041248)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123558100)(20161123564025)(20161123560025)(20161123555025)(20161123562025)(6072148)(201708071742011);SRVR:BY1PR15MB0023;BCL:0;PCL:0;RULEID:(100000803101)(100110400095);SRVR:BY1PR15MB0023; X-Forefront-PRVS: 05214FD68E X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(6049001)(346002)(39840400004)(396003)(366004)(376002)(199004)(189003)(24454002)(66066001)(64126003)(117156002)(316002)(229853002)(65956001)(47776003)(65806001)(105586002)(230700001)(23676004)(58126008)(83506002)(52116002)(76176011)(52146003)(54906003)(50466002)(97736004)(16526018)(68736007)(6306002)(106356001)(8936002)(2486003)(4326008)(6246003)(966005)(6486002)(31696002)(16576012)(478600001)(8676002)(305945005)(31686004)(575784001)(2950100002)(2906002)(77096006)(36756003)(6666003)(3846002)(25786009)(86362001)(6116002)(7736002)(81156014)(53546011)(386003)(81166006)(5660300001)(53936002)(65826007);DIR:OUT;SFP:1101;SCL:1;SRVR:BY1PR15MB0023;H:[192.168.254.67];FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtCWTFQUjE1TUIwMDIzOzIzOitieGtxTWxXVlJkUWZoQjJ1Kys0MS8wT2ZF?= =?utf-8?B?SHNGc2M5d3hIOVFXcnk3Y1hSQjFsSXA0cytLMkJSN1p0VVpzVUNmY1l6V0N4?= =?utf-8?B?dytiNkVudG9BQUJEWXhCU21UWkdiUm9xRlJDZTM5YnM4b1VweWwza0VrKytZ?= =?utf-8?B?TzFnKzladmtEM3RKQmpyVzRvTDBvWC9FaVRJRDJpVDI2VnNIWkduemVoanVC?= =?utf-8?B?REduSWdOTUMyRUlSd2V0Z0tXbW5lcmZlTHVJR2pEUXlZdVNFdmxSWlZQNTRE?= =?utf-8?B?NTZnUDdadG9BTWZHMThwRzQ4OTU0VTM4Ylo2clJhSmtHbVNmdHlWb3o2bk90?= =?utf-8?B?K2M4aGxHTDNFaVQ0cW82SkcyVkkyZHV4UkZ3TEt4OXVybGRIRjByME5ubE9s?= =?utf-8?B?ZzhRQTdMaUx5em9GbnZjNHNTVHdPUS9oamN3NjBVb2lXbm1NN3NNeXo4VmQv?= =?utf-8?B?a2VmQWNmY1B2ZDE0WGlBeGJTYmRFOUt0RVNwOWV6ckdGSHEzNnV5SjhMSURt?= =?utf-8?B?UlRnWUw2dDZYNXJqM2JiVW5xS3N3VUNLZnNxMTNvcDQweDQvemJjSWh2eVE2?= =?utf-8?B?bkRrN0NFRzh0MW9iOSt6dVBicFB3aGxqSlF1bkJhVWhLdFd4K0h5VnJ4QWlt?= =?utf-8?B?ZnRBV3F1UGxTMUZYeWRKK1h6MDF6M2d1OHQwcUV4eGlQTGYwRklNQjM4MDdw?= =?utf-8?B?d2tTMDNNSmFERktMWlRVTkFvSWg4bWdpTUg3VCt0WXQ5NEJGalhZTWNPbEJZ?= =?utf-8?B?LytBRzJlbzB1Zm5NQzA5c1VpdldSbDhMQlRYWHIyazZsTzFGcXpFa3lKZXNE?= =?utf-8?B?M2MrYTE3RzRxMWZZbEFFNWRmSERaU1kweitDb2w3blY4Z0dBckdYVGpGVEpa?= =?utf-8?B?RlRacFZ5VGpMOXNJdHJJWkV2QTMrQTRDUElmM1hFUlhVYmJpREN5QUhXS3Nl?= =?utf-8?B?a0I2VUdraEhpSURRa1p1UmM5RHcvUWNTYTR3TEI5Wk1UVlZJcGxTYnZibmRE?= =?utf-8?B?TnZGai9yUGFoNFVIa2c2QS80S3F0UVloRFVrRkRxaXQ1YWZzR3Ntakp4aVY2?= =?utf-8?B?SkowckxHcTd3aG54S2U2SVB1Yld5QlU4YktkdWtDWm9UamUxeGk3NXFDTURh?= =?utf-8?B?akZBRzlvaHdkNENpejVqNStJNW5ERnNMbmtyNGRBRG1iS3BxQXNWcWpOalI1?= =?utf-8?B?OE9wNXF3a294TTBqRWFRVFlQZld2ZEo0ejhYUlZId2tEeUZzdDVMbjhQYS90?= =?utf-8?B?VXdnbjBJZlRwM0pjaUUxcE1oYzU5QTd4eGRhTnpCdERMTXpIcHhzVFhRemlC?= =?utf-8?B?ZkRaU2I4TlcvTlQ2T0RYMVpHbkpXMm42Q2VmRzYvMUxYeEhaeCswUkE4TmNu?= =?utf-8?B?QTVGdHBseUUxSVV5aWxFNk5wdkxaOUpmY2xlTkNIck0zS2FRQXBsMEZSc0Vt?= =?utf-8?B?cXp0ZkE1TzZCbUlMTkVHeGdsU2YxR3IxRFdxQ1FZTEJLODZRZFN0ZnpyeHZR?= =?utf-8?B?UDdmOVY2ZVBvcGQwdEFQcmxRMWhCMGNtemFNY2dtUU5oYkNTOXU1cmpiU0Zs?= =?utf-8?B?bDhXNG14LzRBTEtMS1duU2ppQmhjeUNpeUx0bm9QNVMxeENDNUQ1RUd0aVBN?= =?utf-8?B?K01Jc0k4aFM4aFhpKzd1YWZPODFiQlp4b3VaQVJxY3hNOXprMzNWeFpmdTZP?= =?utf-8?B?eFhFZlVjWkxBNS92U1ZjRUoyT2pMNTdPUmhvNkh3UjlreDZJTm1PcmR2Wk1v?= =?utf-8?B?d2ZGZkxva3VsR2VoSjZtNXlsSjdLaFRGU1lQWWFFYWhtZGlGVExYY0V4U3NG?= =?utf-8?B?ZHZwR3VXUEpVZUJ6MGZ3aU9Rd2UwazZIaTl0L2xRUmZMMjExTEZTbm5wa2NP?= =?utf-8?Q?N4KD6vCgYeMCzuQKalTVVqIFq6vFFbad?= X-Microsoft-Exchange-Diagnostics: 1;BY1PR15MB0023;6:gu9FPjCAs915Lti3Ngq3MpFlNoAWMQs4OlKpVjhRU6qIzkt2slbuPgDtm3Pf8hHoXvUPDQgRgJnudaQmXCnby+1Yudd6aVacPZ1chb9Vh+RYcWzw0IDY6PyoZdyGHhM+l+8gsc3CYbqp2QX/ORlHDwSsLwO3XNm4IteVmLpT67zs5WSvlZOsJeJyYRqQliapGh+2qq1YZ1JvnlBGqlHynRyXlRIbSCRYW/emDyEcX/9xK3XmJkB3Bhq/DkRdKGC68Urbb6JKBC9gBR5RpKr3KoxVTCutoH0ubUmzoDyJkRUPNV/Cm7o3XnGEbY3178DLRlYgadpVJEn9QluRjZllS5+Tws27H7JkqSsuSknMHdU=;5:fd7h1UuBBUlWj6pAWrs+oiGQ+kvtLkJ7Hk5oUzsUrOBYHk+K539TBiGPp/BdMZWbPk7j4t25Cl/RGD1u29WKZKM/jCsvyc7DWE0roT2ra00dUsWDxGfl55wGAeHDbwvrUkpYy5ssSrNBWT0suAT12+NHau6JI7YbEFSXhySMq1Y=;24:3YBKhYxhLR5YdQ7EYlJ+ApwsjNtE4J7rFZ8P2KwoOZRsUIHUknvJyDyRmmFREU8nEHX9Z5eJxPTp+AeUOiBlnq3U8GfxkLecSsqo+C+v58o=;7:XOQgK6AOU73fGxXbVDw0VvLKGRc6IZWbPTpdvv5KZTiHMr2Dc3oOUo+hvJ/XgsYD+V6Ye9UblqxZagAmt8f5eA62Dv1RaUWVc2YVqbUSOxGEh3B0/LnrV4Ca16q5dCpy5KMWxt16Hj/xpbhNjD1jNKouRI0oJ+D2y1/CTbEvgyl9O9Cvns0joYj2mTT6HzjinJUEhl1xIDT4oggOG9jXnMgmtzy4uw74khhqotTTipvDFe+o8/kjVDcMpGHdKduI SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: opengear.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Dec 2017 23:58:59.2553 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 302b3ee0-c550-4a2d-f5fc-08d5434ea5d1 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: a6251c26-d21f-4164-a225-1f4eaebf5f9a X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY1PR15MB0023 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Jean On 14/12/17 13:11, Andrew Cooks wrote: > HUDSON2 SMBus controller has the same port selection register as > described and fixed in commit 0fe16195f89173652cf111d7b384941b00c5aabd > ("i2c: piix4: Fix SMBus port selection for AMD Family 17h chips") > > The SMBus port selection register is common to multiple Families and > models, as documented in AMD's publicly available BIOS and Kernel > Developer Guides: > > - 50742 -- Family 15h Model 60h-6Fh (PCI_DEVICE_ID_AMD_KERNCZ_SMBUS) > - 55072 -- Family 15h Model 70h-7Fh (PCI_DEVICE_ID_AMD_KERNCZ_SMBUS) > - 52740 -- Family 16h Model 30h-3Fh (PCI_DEVICE_ID_AMD_HUDSON2_SMBUS) > > The following additional public AMD BKDG documents were checked and do > not share the same port selection register: > > - 42301 -- Family 15h Model 00h-0Fh doesn't mention any > - 42300 -- Family 15h Model 10h-1Fh doesn't mention any > - 49125 -- Family 15h Model 30h-3Fh doesn't mention any > > - 48751 -- Family 16h Model 00h-0Fh uses the previously supported > index register SB800_PIIX4_PORT_IDX_ALT at 0x2e I just noticed that commit 6befa3fde65fe437f588da490c07a114393ce229 ("i2c: piix4: Support alternative port selection register") also fixed the port selection for Hudson2, and this patch is incorrect. Unfortunately the AMD naming and PCI Device IDs aren't particularly helpful here. In both the 51192 Bolton Register Reference Guide[1] and 52740 BKDG for Family 16h Model 30h-3Fh[2], the PCI Device ID for the SMBus controller is 0x780b, but the location of the SmBus0Sel port selection bits are different. We might be able to distinguish between them using the Revision ID in PCI register 0x08 (in addition to the PCI Device ID, of course). Bolton [1] is Revision 0x15 Family 16h Model 30h (Hudson2?)[2] is Revision 0x1F Family 15h Model 70h [3] uses 0x4A Family 15h Model 60h [4] uses 0x4A Thoughts? Andrew [1] http://support.amd.com/TechDocs/51192_Bolton_FCH_RRG.pdf [2] http://support.amd.com/TechDocs/52740_16h_Models_30h-3Fh_BKDG.pdf [3] http://support.amd.com/TechDocs/55072_AMD_Family_15h_Models_70h-7Fh_BKDG.pdf [4] http://support.amd.com/TechDocs/50742_15h_Models_60h-6Fh_BKDG.pdf