From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752123AbdK3SqX (ORCPT ); Thu, 30 Nov 2017 13:46:23 -0500 Received: from mail-eopbgr20108.outbound.protection.outlook.com ([40.107.2.108]:37743 "EHLO EUR02-VE1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751077AbdK3SqS (ORCPT ); Thu, 30 Nov 2017 13:46:18 -0500 Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=peda@axentia.se; Subject: Re: [PATCH v2 2/2] ARM: dts: at91: disable the nxp,se97b SMBUS timeout on the TSE-850 To: Alexandre Belloni , Guenter Roeck Cc: linux-kernel@vger.kernel.org, Rob Herring , Mark Rutland , Nicolas Ferre , Russell King , Jean Delvare , Ludovic Desroches , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-hwmon@vger.kernel.org References: <20171127163101.27859-1-peda@axentia.se> <20171127163101.27859-3-peda@axentia.se> <20171129205311.GA20551@roeck-us.net> <20171129205629.GO21126@piout.net> <20171130171638.GA17788@roeck-us.net> <20171130172606.GZ21126@piout.net> From: Peter Rosin Organization: Axentia Technologies AB Message-ID: Date: Thu, 30 Nov 2017 19:46:09 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: <20171130172606.GZ21126@piout.net> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [85.226.244.23] X-ClientProxiedBy: HE1PR0401CA0063.eurprd04.prod.outlook.com (2603:10a6:3:19::31) To VI1PR0202MB2560.eurprd02.prod.outlook.com (2603:10a6:801:6::11) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: e38eb95b-20c7-4f22-b601-08d53822a22d X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(5600026)(4604075)(4534020)(4602075)(4603075)(4627115)(201702281549075)(2017052603286);SRVR:VI1PR0202MB2560; X-Microsoft-Exchange-Diagnostics: 1;VI1PR0202MB2560;3:VtGW3z+yGSlYGBJZRnl3fbXgrOeD8AUcLY8udLOWOoY1jGlwAABZy0bta3sGyKykCpRtiP6y5qlH8jNT4JoaMRIFYjfDRDAgtkhLnlsS2rPSt3t+Vx01eASBKIkmcPBuarjYFIP1oUiVo8TlcGts/1slFjwJ/6KoXs9Wcu9jL4aZ3ie8RcQs7wrp/GaVkLMu4jHA5cYlx3Vty3isjXMQXz/hHfriVz2zptREiAmnDZxmEXJfj/ZlAkItR/0e5i/l;25:sMmVEf0dkFoACsqdMVfTPdQvkEqbXFSRKxC+zKVm196TuUPADTCZ9W9J6TzxVNk3qsWko9oM3EkWL78GNXAjE4zTznf91+PzlxERN1ParrCmz0w/DFE9IPvCfBJXrE3iqcy9pLfMk8M2HXEHvulvipmmFdS1MJoOe0Vqd/LNwYtB6gF+MtvV9rfpnrEIrngX0lJ0qIAA7pBaBkOnunaeHIitW0yL4C+uemC35enmI2d6koWEGYj3seduHL4xOJ/sVRWlaSVrJrb/x0IoCJexJ37jV8qFuvQ2c2T2C05GaEQobyKJA7sCeedBFUWunzL+r7Q2rj8BZD/AVKG7ZXeqjQ==;31:teOqGZQ7hJKNwQA2lXYQUiShj4moGhk3LusK6Dt6A2ZvD6VNZ/8wxPwd5ucA4Y26L2/LC1tTHROt1jBDxFnciRhSeFr/fuyoaSepOVgvQPdTqZz9P4llcFl1QYYZAB+AaFeMdp6tn70FGpC3qu97fHYgvc6qGT8mDOLHMYQKYnDVii0SQHAB5b+eDo7Jn3PCoK9i1KTc/cNxi+KRwcSKitLbX/kDxTPYI6ZBmjD5H5Y= X-MS-TrafficTypeDiagnostic: VI1PR0202MB2560: X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(6040450)(2401047)(5005006)(8121501046)(3002001)(3231022)(93006095)(93001095)(10201501046)(6041248)(20161123560025)(20161123562025)(20161123555025)(2016111802025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123564025)(20161123558100)(6043046)(6072148)(201708071742011);SRVR:VI1PR0202MB2560;BCL:0;PCL:0;RULEID:(100000803101)(100110400095);SRVR:VI1PR0202MB2560; X-Microsoft-Exchange-Diagnostics: 1;VI1PR0202MB2560;4:XGhjxOPCdElE5VCLPJ30VsN2bUS2DYdwXQW9p1z02iF8E0NiaTF1Ddx1m/cKVapRYxFp8b+EfxRaOKpeXwJe+hQyaz/gLHK9HwmPge8Fr2zk+QglLkKBS0oyFzhjSx6VMt+IpFQHbKSvu24bZfIYHhxjaZG7/4e0EQCg/UA8v11hIcpuVbNEjV0+EqCbtv6b2TinVgi00PPP74d/7vi0F/b3HSr079hdYI1a4d0Bfl3rL3aLX7KLfUnFbwbCIOrlXcAo8Ihiu492IQG5Nnd0Lg== X-Forefront-PRVS: 05079D8470 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10019020)(6009001)(6049001)(346002)(366004)(39830400002)(376002)(377424004)(199003)(189002)(24454002)(76104003)(110136005)(7416002)(8676002)(7736002)(5660300001)(81156014)(81166006)(33646002)(6246003)(8936002)(305945005)(101416001)(23676004)(2486003)(2906002)(52146003)(65956001)(106356001)(36916002)(66066001)(105586002)(31696002)(68736007)(65826007)(65806001)(4326008)(4001150100001)(53546010)(86362001)(3846002)(47776003)(97736004)(64126003)(16526018)(478600001)(6116002)(316002)(58126008)(6486002)(3260700006)(53936002)(229853002)(77096006)(16576012)(50466002)(25786009)(6666003)(31686004)(74482002)(117156002)(54906003)(189998001)(36756003)(2950100002)(52116002)(93886005)(83506002)(76176010)(54356010)(50986010)(230700001)(142933001)(42262002);DIR:OUT;SFP:1102;SCL:1;SRVR:VI1PR0202MB2560;H:[192.168.13.3];FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtWSTFQUjAyMDJNQjI1NjA7MjM6MStRWURYemJSN2s1eUVYemUvb3pxYTJJ?= =?utf-8?B?cUVzMUpwdzFGbnJwSGoyM1lyRFppeFFYbU8rakcxUzdjM3lTd1lUdmNYVklG?= =?utf-8?B?R3VNTzh3KzNvYVpJejZmcU5UeDd4bytvZUZUa0hLUVNsVmRXYzNVY1FMOHFZ?= =?utf-8?B?RUFwblZSVkFlZ0lYRFlxUTZVMGdRajVQZVdzVHg5SHBZb0dGU0JtRVJVUStS?= =?utf-8?B?RFZoZ3M3aldDRXp6eGovQ0lsMk0vUGYwNE1uZ1FHMFRLUy9CODhOUWVDdExH?= =?utf-8?B?UGFLSHlXZzdhM2MvNmxkZHJRd3BMcDJ3aFNxaEJ3NTNpd3ZrMExHdWdibFdZ?= =?utf-8?B?cTNpQkNCRDJVcU9tSzBkSDRkR0hPZjBhSGt2T3JXNDRuZjhYMlJGMEJ3WGhO?= =?utf-8?B?REJwNlNoRFZ4ekN0KzdISHZLNWUrWSt0cnpXNmIzZWpWK05TWnhtbHZKeDg5?= =?utf-8?B?a0dxR0pINXo3YmJJc2cybVcyL2VFanQ5R1ZEbGJ0cytaYjNsVmk2UzROVFBw?= =?utf-8?B?S0RwbEs4SENHVWNWZ05VSll2UU1mRDZEOURLWVlxT2cvQmZEWUVEaHA2UzVi?= =?utf-8?B?U0k5b1dhMXU5UHdZMW51L3k3ODZVRWQvMk9tRGx1OWdpZ3c3ZG5mK1QrZjRN?= =?utf-8?B?cHZPZzJoMDNnZ0ZvUjhVTFZ5SnFSYkZIQ3dUT3pHKzJ0L3RPSVJSYkNmcGZJ?= =?utf-8?B?MEhybjhqSytqUmMrR0hiVmlaL2c1TVpGRUcrbGdUZEs1VEI4SjdRd0RyZFZO?= =?utf-8?B?anMyR1FVVHdRUUl6TjdlN0hYZkk2L2Z5ZHRuQzFlNXpkdGUzUVY2NHhzZnp4?= =?utf-8?B?UUVsSmF0U2FoaHpGdW93VTB2R3VsVHpHL01lZnN5aGxBSlVVdHpuYWdFeHFz?= =?utf-8?B?OWRwYS94dXBKcXBacExWSk9FN2xkdU5WKzZTL0lpTmhnMkFvdGRrS2dqeXFk?= =?utf-8?B?YjB6Y01kOXZzL25mcGFoTGRuaFBLYlloc29UMUgvM254L2lFNjVUclFjR0RZ?= =?utf-8?B?OVp6eXpRWGk5MFJQZVpmR2dBY2Z3VUhNRm1kbFNabFQvQjBMT0d5SGpqRzJl?= =?utf-8?B?c3ZHZUN6YmRqWktXREJFVENSYUtkWWZaOUFwSGdtRHBPNXRwSnNqMGR0QXkv?= =?utf-8?B?cDR4OXJiby9CZEVDczBuMXJTWUJBS2Y3NC90ZHlycS9iVGFCNS9ZUzNsT29H?= =?utf-8?B?Sk1xRWtUNTNtM2ZROThiTU1tZUJCNEM4dVB5SitheEpNTVhHZE5jbFFaZHpR?= =?utf-8?B?MjBHQ05FMzRtbjVmSjhDZXUwK25oOUgrRHNyMC9oSTJ0RzFHYXRER3hoUjIx?= =?utf-8?B?YzFWeWpUQnozQWtRbDg5UUVqVENQVDJGaXdNRGJKK1NvUVFBd05COFN1c0lJ?= =?utf-8?B?MCtOSCtVeDU4Unk1WGJYb3JzVHNWNHpyTEk2UTVOSHhSMmQzdjlDSDMvRjNN?= =?utf-8?B?N3prZ0NYT3ZhRkNabVFyaElQTDBrdFVLLytnQ0d3SVBvNEVnbFZlbGE5Mk5F?= =?utf-8?B?Vkh3SGVyc2JJOHU3Tlp0dUlEZ1BiRTNIZTlNSGQ3aUFSa2VqTDVubEkzV25J?= =?utf-8?B?RllPTjNHaWhHU1BuN3lCWGdQYnRxellKZ0VBU0IyUTZpdlM1U0J6RjY2WmJC?= =?utf-8?B?U3pIQjlBRkZhYll4dktuNUg2cTROVkhTcTBWT1ZtWUpIMlR0R1N3M2c1SFpV?= =?utf-8?B?MnE0cFZuc1R6RHlhY0VSMHRZUEgzOXBQempubWVEemRvNHdGYkVWSGc3UmY5?= =?utf-8?B?cFNkVmg2bTFrSjhjemhoS0V5NjgweFUzUWVrTmhoZkJva09SUkxKUWkvOERz?= =?utf-8?B?UUdycHNiTUF0emhyU2JxVFJCSXlUWmMweWxYNmNoVUZQWUNDTGpaZWdBcGJF?= =?utf-8?B?ZStxYmxMYkVTeDgxdS93WFFTZFFBbUorUThnMStGTCtZTkU0MVBsbVljOU14?= =?utf-8?B?VjRLQTU5T0JzY1BncVdmbFFtRWxyY05wWEF0OW9CbXArQm5NKyttQzhraW1M?= =?utf-8?B?NjhYamZRdFFqL2RlY3NxMGZPOVFDVTJuaWl1dnROTHc0MmFkYXQzQ0hCWHow?= =?utf-8?B?ZDJMQm4zdXJ0emZRMUttcVlQZGlNeDdUSi9rc041NVpCbGU5dk4wNk0xSGhu?= =?utf-8?B?Qkh0QVV5OTBDbG4rcVVjcnZwK3V6NVFIeTVBZHlFSC9SUGhaNk50cXpGTnNR?= =?utf-8?B?NGN4bGUyOFkrY3diQnhZc3dwTkI1ZlhFNmFlR2pEMzQydTJ3MUxIYmIwWUE1?= =?utf-8?B?YXNNUW9HdHA5dXVXa0tyR0UyQnNtaHkxNjhwemh4WDZzZm5BZjNMYTV3PT0=?= X-Microsoft-Exchange-Diagnostics: 1;VI1PR0202MB2560;6:YU/Stup7cGqNCTQDRwEnEBd7kElOXRVQMOdagwkw0k0xmNidMiaCQeNwRpezlSKB03s1AhlKKomxwEIUWY687m3Cu/net1VozMPKitZUSqhfEQMEoADYoc/qGOpnV1IaaPBre8mW7d7g1JUgwt0UwlDbw76A3CUPX4njI0q+b2kocDI/0dIrhtNEVqdvD5lhL4U6N5PZ3c5SVUpLhI+/UUJUoab0UpvOM6rp0UTQSouBy+AVykfeihtqOkDHSqBxwfsCPPpaR5YSWCAhgy0mhlKpATM76PywR1AHfvCLC+4x5Nah39hDVHV+D7PC7zyMmTkCWe2oKpnU8+PLv7wCEIslpGd8oTwk7HG+i/iRxFs=;5:ZfRZ973tE4KZ4DP9nJLAEl04AUGCAJ57n02oSFopUvOe87x2QipfaFbzVJFn+SiAolLMH//qWTCUQKUFlDGZA5g4d3EJDYQLNAUEx4bM3bVo7NKcLiuDsq8VnJleY7scXw2KZwLbnhqSC6yW9DRhLBmD1IB82teYqjHBZSqTERg=;24:NmvTfTHKXUZJf9BBrizi5xeqrreHf8HUoR4/n68XL9x/739cwyOWjLOlDAneAQT3veK6l6kjR4jf6XrWHVfXWF4VFVPLF7KL2W/iy6FQU/Y=;7:9fZSbPCCVd8KxUR848CPk/MEwqQNq0lmLhcxTPb4ByssJHg2kWL5FqAkVcuECw0ck5qC85AajgmJB7H/deV4/bhA/DhzmaKfhO00VCsnLNacd/9rV53gNPPTCQpG+ZTOIyekMIJZ0w9EkOSo5tFo8j07mFIJBd72n9py55Et0+0i6p0fs7u3K+fe9OgHR6jRn0w6dWjjXQ6UrP74LFViP3DTDTYw6eqDBno95HW1VU7oqt0knaxLwn4TVzAdirDL SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: axentia.se X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Nov 2017 18:46:13.4835 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: e38eb95b-20c7-4f22-b601-08d53822a22d X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 4ee68585-03e1-4785-942a-df9c1871a234 X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0202MB2560 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2017-11-30 18:26, Alexandre Belloni wrote: > On 30/11/2017 at 09:16:38 -0800, Guenter Roeck wrote: >> On Wed, Nov 29, 2017 at 09:56:29PM +0100, Alexandre Belloni wrote: >>> On 29/11/2017 at 12:53:11 -0800, Guenter Roeck wrote: >>>> On Mon, Nov 27, 2017 at 05:31:01PM +0100, Peter Rosin wrote: >>>>> The I2C adapter driver is sometimes slow, causing the SCL line to >>>>> be stuck low for more than the stipulated SMBUS timeout of 25-35 ms. >>>>> This causes the client device to give up which in turn causes silent >>>>> corruption of data. So, disable the SMBUS timeout in the client device. >>>>> >>>>> Signed-off-by: Peter Rosin >>>> >>>> Acked-by: Guenter Roeck >>>> >>>> I assume this will be sent upstream through an arm tree. >>>> >>> >>> Yes, I'm applying it right now. >>> >> Are you going to apply the patch for 4.15, or queue it up for 4.16 ? >> I have been arguing with myself if this is a feature or a bug fix. >> So far I queued the driver change up for 4.16, but I am open to >> applying it to 4.15. Any thoughts ? >> > > I was wondering that myself. I'm open to have it as a fix in 4.15. Or > maybe Peter can send the series to stable if he needs it in 4.14. > > Peter, what do you think/want? TL;DR Either way is fine. I think it's a bugfix; it fixes real problems where the application misbehave due to faulty content when reading from an eeprom. I'm expecting to make a new release for the hw in question RSN and these are the only local patches. So, it would be nice if they made it to 4.14.x before my release happens. However, it's not like it's difficult to rebase the patches should that backport not happen or take too long. The badness started to happen much more frequently due to some timing difference affecting the i2c bus driver, but in theory it's a problem that has been there from the start. I have just not noticed it before... Cheers, Peter