From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751800AbcFNIT2 (ORCPT ); Tue, 14 Jun 2016 04:19:28 -0400 Received: from mail-bn1bon0086.outbound.protection.outlook.com ([157.56.111.86]:19980 "EHLO na01-bn1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751483AbcFNITM (ORCPT ); Tue, 14 Jun 2016 04:19:12 -0400 X-Greylist: delayed 905 seconds by postgrey-1.27 at vger.kernel.org; Tue, 14 Jun 2016 04:19:11 EDT Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Jan.Glauber@cavium.com; Date: Tue, 14 Jun 2016 10:03:51 +0200 From: Jan Glauber To: Wolfram Sang CC: , , David Daney , Subject: Re: [PATCH 2/3] i2c: octeon: Add retry logic after receiving STAT_RXADDR_NAK Message-ID: <20160614080351.GA4725@hardcore> References: <2cc75a97f920146edc7af389870e271a8a056dd1.1465304877.git.jglauber@cavium.com> <20160609201150.GH23522@katana> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20160609201150.GH23522@katana> User-Agent: Mutt/1.5.21 (2010-09-15) X-Originating-IP: [109.193.46.37] X-ClientProxiedBy: AM2PR07CA0036.eurprd07.prod.outlook.com (10.163.24.174) To BN3PR07MB2580.namprd07.prod.outlook.com (10.167.5.8) X-MS-Office365-Filtering-Correlation-Id: b7fe1e88-6640-42ca-1f1e-08d3942a7305 X-Microsoft-Exchange-Diagnostics: 1;BN3PR07MB2580;2:HskeicjIlwzVuI/axtCD0GesLsEzIkElz1WiC13lczqMkgW+MMpKI6Pd/mX7i8/UhlIr2qKnaipRRCWE+r6Z7+CTpkgmF5W2Xeyepv9W9L9YiTielX0PUqKrU0RMzEGfxEOlVlLqPgUO8/vvkzue8r4fvkb5cAeEJV+FjtMQo+CvbfCQhG5zTkKxsSlbaiep;3:tqjNX4DSQEezpA9WjpHJu/Xcub07zkKxGeGC9VwiluPgBpEU2/GbtcxWpmuFGKeIk6Z1NYPytGwN+XZ6Kaf6FCiTm7IjWOILVwRHF1MLJe6uCn3O2tjGptAyfhnTS6/4;25:rZER289MrqU/OaWJY5VBIJejRR+IX2J4Ny2gWxUpDPF/sAzm0L+u2RXqiMC+TWQXatXLSwXUN9Ug1lrnqYNcn2Gx2dUpOKvclmdI3c7aRAQ7x0XVGc/D8sy+qN/DxXvkONHCXJ/wKVbgumVJDftyfRGMHfXWAjBBPrxun7dOhXHPLzGm8T2xJ9ofkmcyLJ9s8al3tu8aZPZ0uK/2qkJ9EJSCodLdiP3XFYUIxRplX+CykUAgKMueIcEitJZbqtfbLNGpZGKQQ/Gt/MoHHGt7hGgKJod/u7n5TNRJeKULXr3ZsBqfIT89ielNquZfVjGB5MRlDPQI9c+4yhO/xe2/1HvoQFDCdaXTVnqkjX3Ph3eAjqW9h94T4FXvA3r5/KFYMbMNVWkIpZaPeNg42MPGSgBXOoKd/2hOxwLXto5prFA= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BN3PR07MB2580; X-Microsoft-Exchange-Diagnostics: 1;BN3PR07MB2580;20:wckNHGs7EVKiYLouLQblAxSKxaggbJ+8YJzyJZtbyeJ890qpsCnsOKPxaa1FQWo5OM1fnnXYykJYO11/e+fHg0YqoyuwYn1KG3vmLGdQMcayyjPVtxSW62geAfT88ohVd3Z8ZNHfGHEPdqKIYwI0DwNOO/J3tshsMBRKl1SXVz2IAdx7M5PPzzzoNi9eNb625yZmHrtm48MOKqU2StdZYmVryoK6G4JRfQ0SLJAnAiEchGSdkdHlWOw/ZJTKUqNSoDnutrYkRhtYeeOfwpe23OgTCxZLmclhxXPMqfIeGRjsTHf8s5zHo9ImHJJXq2yWNX3MdM3tMP7lPzBoS5lU14tFLS95esUaIm4KMQuDEpspI4ckwG3Jyxogp6O1JefnjCFbErt2HbDe1yicwWe/bFXaEpT1c4r1PDO2ozqYd1huqblhtK7L+4qDJfzLlkDnph95rx+UQ8R/6aF6AxUt1WhYnihegMRzPde2m3C6A9g0aLMnT/EpWoU5nbexX0Kr2dtR7/cWFUqY0vsFPnn6QOsd561+sILLovP67v3VOsNiOo6zHaA8aXRCIZH+6ab2Vt5gSywTggfsJWV86I7rcUgrN45SafGjUqxKwSVVAHY= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(2401047)(5005006)(8121501046)(10201501046)(3002001);SRVR:BN3PR07MB2580;BCL:0;PCL:0;RULEID:;SRVR:BN3PR07MB2580; X-Microsoft-Exchange-Diagnostics: 1;BN3PR07MB2580;4:IvLETRty33pitOyJVg9MdOQQAqNwgbVcbrZ8kWQFvJNutYgKB5ca0zAvayxf1OuZutMReBai3mkL1Jedewt2EaKpCHv5tjAvEuicgzMtUjbdbsPLtbIn6NyS4fnE3tuZyDSmRhkHR6cmeYCzBQd6BytidVEaue8buRvDdynhguT20DcOpeaum2msP5IaZXLEaRs+hD3K11kv6cM+T2KZjdE8UI9ciqaMmAHNP0eNrhYZCGcFhSsUQwCD0x2pEhJcQMYuQ1fsgc9zpZAGlaollnAVkzFw523HoOO3HM68gtjiMhZNkyDdhcy8gHUSCwpEfZ3+Ssg0b16NIxiH962Gc6VSGAyiaoe/RS06szcvs1wsj+4eQhljdrilh19gBsKA X-Forefront-PRVS: 09730BD177 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(4630300001)(6009001)(7916002)(189002)(24454002)(199003)(83506001)(4001350100001)(97756001)(50986999)(110136002)(68736007)(33716001)(76176999)(5004730100002)(107886002)(42186005)(105586002)(5008740100001)(97736004)(1076002)(4326007)(50466002)(3846002)(46406003)(77096005)(189998001)(6116002)(586003)(92566002)(2950100001)(81156014)(81166006)(8676002)(106356001)(66066001)(4001430100002)(2906002)(9686002)(23726003)(33656002)(47776003)(54356999)(101416001);DIR:OUT;SFP:1101;SCL:1;SRVR:BN3PR07MB2580;H:hardcore;FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:1;CAT:NONE;LANG:en;CAT:NONE; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;BN3PR07MB2580;23:j4vS5P27lVZKiex+Tm046y87t4yFd1XeUYKCKhBdk?= =?us-ascii?Q?c9lID02d7++Cp40fpWI4ek7WH0rwpZopFyAZ3sUh0nYmwBr0qQAOR9VSQYEl?= =?us-ascii?Q?q3PjG7bMjLFLdYMwA/F1BmFAksdIRKapl0oMvsl1C1ztnGNooxwIdgyAkZdS?= =?us-ascii?Q?YOXcRp3ieio/tYpxoEnzsfc763NYSvFWhZKZbtYmn1EYawOzRNMyuO5PkASN?= =?us-ascii?Q?HApkRpswUyLhOPOOJ4BuXVxjMvhO0Eq/K4+B0J+p5h0/21qj8msfCfbdDC95?= =?us-ascii?Q?sF+4PzCFff7WxKdXSxjJ8OLZDt2jjRhCwbU2ldfy+jIBl4awvzeZErZF+vQO?= =?us-ascii?Q?pX6mIapUTaieS7vUS895erH7+13luED5zkDKTCnlH7m+A3xOvKsA43Jl5Y3J?= =?us-ascii?Q?agZ21ng6453GZnpSQmPgEwp8+asm+lRsDei4JU06kwzyaLGiwjfuxtmxtERd?= =?us-ascii?Q?KWCVumNZALSFdOMuRXMUMJ3laG5YzHyaIZ2z8yN7NnTRnzXTBEBx8QjMFHnL?= =?us-ascii?Q?WuqHxCJjl5A55TEGSjojdbGK4UOaixPSpaOBu/bYAX9UjZFFS7SR53gJBaP9?= =?us-ascii?Q?sQoVi8iY1VcEV6E3B3VPreTM5D2l37qhpxL2T7Gcq3UbpPxO5dagiiSFeNYJ?= =?us-ascii?Q?UEXv41dGDQvfD/7ge/NmWBsP+xcNFKf6dBqWxnVZC6qupbQ6LeFVakH+c+Ya?= =?us-ascii?Q?vl5Ljr5X17UBoZ+dfJdGv670pdxTZs1EdGzo5XkuHNpve0uWhdJmEK9fVGEG?= =?us-ascii?Q?d1X5i5izGk9gVqIXuo6E9BzFSUm43vKITMz9Hb3opRcWodVJWguvYSC09Jk2?= =?us-ascii?Q?YNMCHOhXK9uUPjqpaftPcI3K2mE/pAzgb32Is88U3S0Iidd84kYQF6SV1Bb/?= =?us-ascii?Q?g1u2BHB3XtGoB2C2oZ5JAM4TINogZO4vHGC6SfUxoodOdQQCAtzb4Z91IDkk?= =?us-ascii?Q?WSszb9Kt8liZOezYAcn/TVzWXryqhBvfjydTbVK8x6zxcqKji7r0UWjkVZwr?= =?us-ascii?Q?ghpx8jtoOhnDNhJOHVLZ5frgPtL+itdZmsgzGogfuYtJNn9+z8zcSdLzN0eS?= =?us-ascii?Q?qFgRsFiuzWvMDF2lSY00TehzvEm2knomt6yXtV8dJ9SRT8Je0e7JOvZH8pL9?= =?us-ascii?Q?mc/9h332bMHaQNlgX4spF1AUquoq3xH?= X-Microsoft-Exchange-Diagnostics: 1;BN3PR07MB2580;6:ZioqSQOZgYhsYjaXGY9N5N7g7z6hdtjgftHMbCXWpvqIZ3T6diz17gTPbqoKqo/iBWmEoXbN7c9iFBKlLWK4594OlbnhZsCXaI2DnGv+vow6Ivt/4/xbVquasLMsltDZoObIPfjBFVIURUpP5fXXxucTM5+CsgLx7dzPUweYOofZSx93JOeQgOn6WUFSKZVgWGvhHasdSUUWlQFz2K79DbXXNslvfoaTtHdEfNdtk4f8sOOkZp0oZTsdhPYWdbYAWs9yQ8ow9m1lnp71PATr2Q==;5:Th3LqceFMuBQu+wUXt3OuUbNQrWlKBrpp1DB1RgEOin+zpVnnJXt5pfQHUwZujJ3qF3wfEaeY9vts6Vlov9iRMC5JUmO7EZIlmkEaZOJmIeRH4a94OoMbozfI3KxBlPvuxhsYGarTXk7J5Akyhz2LA==;24:wexR0GejtO8xwZnRpR4G6rKtRULB3PWusPOlpHuFCysWltSh3qhiLaI2xuW54fG95sf4JtWMylDwY5EjXlz3jOooJdPnGG6hQEwxn3H13o0=;7:1TBQ1G87UBZdEuwSpfcbUIVwIfkzrrTB+/7MF30CcbLwm+E1lm1Fney9/MZsn7VNHQHCj77pKN/wWxZyinfVftDpusm/GjIiiPAdzPJkm1MTSqSP9nEMjcsVDpIqQC3G78i2z4VKP0RrcmyKpJM9vAGvBbLvsmgwtdn9sbQyXQL64A67+Atd1eUaKb+OZyN5Fz5szWylcH3C43lJCE9E3Q== SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Jun 2016 08:04:02.0655 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN3PR07MB2580 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Jun 09, 2016 at 10:11:51PM +0200, Wolfram Sang wrote: > On Wed, Jun 08, 2016 at 08:51:18AM +0200, Jan Glauber wrote: > > The controller specification states that when receiving STAT_RXADDR_NAK > > the START should be sent again. Retry several times before finally > > failing with -ENXIO. > > > > Without this change the IPMI SSIF driver fails executing several commands > > like 'ipmitool fru' on ThunderX. > > Huh? Looks wrong to me. I'd say the client driver needs to retry. Only > that one knows if retrying is appropriate or a waste of time. > I've been debugging this and it turned out that there was an related issue with the clock setting. With that corrected this patch is not needed anymore, so you can drop it. I still see a huge number of RXADDR_NAK's after START but the ipmi_ssif driver retry logic seems to deal with that. thanks, Jan