Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751907AbcDZMfD (ORCPT ); Tue, 26 Apr 2016 08:35:03 -0400 Received: from mail-by2on0064.outbound.protection.outlook.com ([207.46.100.64]:18752 "EHLO na01-by2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751082AbcDZMe7 (ORCPT ); Tue, 26 Apr 2016 08:34:59 -0400 Authentication-Results: the-dreams.de; dkim=none (message not signed) header.d=none;the-dreams.de; dmarc=none action=none header.from=caviumnetworks.com; Date: Tue, 26 Apr 2016 14:34:44 +0200 From: Jan Glauber To: Wolfram Sang CC: , , David Daney Subject: Re: [PATCH v7 03/15] i2c: octeon: Remove I2C_FUNC_SMBUS_QUICK support Message-ID: <20160426123444.GB9796@hardcore> References: <1461594824-7215-1-git-send-email-jglauber@cavium.com> <1461594824-7215-4-git-send-email-jglauber@cavium.com> <20160425221621.GI1550@katana> <20160426055845.GB5758@hardcore> <20160426064242.GC5758@hardcore> <20160426073620.GA1543@katana> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20160426073620.GA1543@katana> User-Agent: Mutt/1.5.21 (2010-09-15) X-Originating-IP: [88.66.108.80] X-ClientProxiedBy: AM2PR09CA0027.eurprd09.prod.outlook.com (10.161.22.165) To CO2PR07MB2584.namprd07.prod.outlook.com (10.166.201.23) X-MS-Office365-Filtering-Correlation-Id: 296b9116-b3f8-42fd-9228-08d36dcf2d3d X-Microsoft-Exchange-Diagnostics: 1;CO2PR07MB2584;2:HIi76+9LoOUY5HOuVmTTMRJcfff8cZIwkVai3pcCnfSa9LmbfHinAQFc1lWUUVHzxzixpQko9p4XyXduuks9d/MOFr87+o7WWay+2nLSv252b6kik+Fkdk9uGhutLozbhmbJg/WCDvlJTkN1CRwVul6DT64EAvUMB98hPLghZPnc0QiaI1qKFSULdOCvpXSt;3:nGulZFOHt0g/AqhFSPQF5uPvo73KUmCHT1oZk5aaGc1NKp5XSRQelnN9s+6DU/4zlFvPBKuSrkGj2UMOcYAj1J7iqMWVkvtt6VeMXWM4tc6M/1E8ioKchZbzEcURGPVv;25:p7VgDTXRW2hSXY0ufYWVkS7AW89LOwCPtTH2w/vEk+AqJxjcLH/Mdm7Mc34ALjg0XhmlgOswta2m1up52d8DwgZOu1NdD2TtNcNHqxgH/F9xYdkA7/AclEe2BxoBt39SvaYWZi8AQ0XGs3iCuBYiZJkZBTbFCB/B80r0bbBeqCBC1bhhZ68EFD94+ZoGqowo6JWEfue7cU1qBdVUtr1uOCNKWVgaFhb+Yne1aZNxteGrdd0VcCyMXk/CjrzzGAmliYREiUiTWS++ZDUDOPbcsGaseMdlIxOX2wsJGnXCcRaTtDTvE6NkcvtJ+wJAn0qPyRDWDnqO07b+r5X2aC+IsA== X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:CO2PR07MB2584; X-Microsoft-Exchange-Diagnostics: 1;CO2PR07MB2584;20:95Ft37/jUl3FloEs29g77JeCjGOv3vMaXNksMUxNSHwvFgx9ou2yYFiZKSRC66u6l6RdFyPtpJGrwi4kZw6PD4gwygkjpQDVA1JHDinDlDQLKEpOxbjiB5Cr001qskmYn6LNHdyXR42azqdTDSbWVaFaKT6qXNXOxl/yAMcR9DKTnCoEyxB+9Ce+XtbEQdyY6c+NB6OE85yP8OY0z8khkECv+wh3zv3QkBmDfDW3sToyjlW1wjw6UgJBOK3V/pNH4uBwGGCEW3yPAujJm0CL3P1W0XDDYNVdPNBZGFUiG17VxReVBbb0fiDtBx/9elCICeXDEL2iTTpIo9YKlHK0DBP0oeQ4UDKrx82Y6qvPedkr1KyPNenZ1a8sn2fznzt2Oxa/7A4zRGG3y9xkZK90v94Hgm4SJcBGXQ+4rRjWEIyTxWGIdBKgaiel4qJ6ja+gW2hM/lAOOZoBktk9loEdOi4x8tuuj43wzbW0Ix2h2Yt9EKfcWjglURi6VrnjNAY0yEpcR06vS6VE13oHJ7Hgbb4bHh4GKhxtUrfLgK+v3J92bWwPILr6HF35Yf1xDhd5O9lF4y5nBmY0Uiq6VgIMnB3OFsdwoyuj+8JgPW/LvtI= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(9101521072)(601004)(2401047)(5005006)(8121501046)(10201501046)(3002001);SRVR:CO2PR07MB2584;BCL:0;PCL:0;RULEID:;SRVR:CO2PR07MB2584; X-Microsoft-Exchange-Diagnostics: 1;CO2PR07MB2584;4:5yR62TnjJP3k7fkDVfU119NRIGQ5wa0WhNAE19w+nqOP9VONB2vEdzhEQ2SkV7xXMfj/Et6Vfv9L7RChNh7ookb8Ad+9obHwhb/qvYzDpZZAQJJUolEm+yAWtcPI9NoHA77FrL5tPP0MKK6xQ2WTP0FmD2XBOaWTFZhTudqw80IKsP/TiOQr0lt+WsqA2YRGnBjlMwUVYVGGuLcm9a/tCxIXtjShQQF9fklNBVckqg1lkuUBZrmMv2e1w1a9U+x+oWI5bAgBJNKwBFkEz4fQP3CpmsMF36D+FKEw07d3vjAWdm37NR8uR5xJ+4/23yoUgEUseV12TrpwuwSsdKeQF+t8RpirGXeeuAp/kDi+seD771CyOTQIVxxiHIPP6JOBHrtEUWFcDG1xWJzEUas7rw== X-Forefront-PRVS: 0924C6A0D5 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(4630300001)(6009001)(24454002)(83506001)(42186005)(66066001)(47776003)(86362001)(93886004)(110136002)(107886002)(4001350100001)(189998001)(33716001)(97756001)(4326007)(50986999)(54356999)(77096005)(5008740100001)(5004730100002)(9686002)(2906002)(3846002)(6116002)(586003)(23726003)(50466002)(92566002)(1076002)(2950100001)(1096002)(76176999)(81166005)(33656002)(4001430100002)(46406003)(15760500001);DIR:OUT;SFP:1101;SCL:1;SRVR:CO2PR07MB2584;H:hardcore;FPR:;SPF:None;MLV:sfv;LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;CO2PR07MB2584;23:EtpBKEYuykQ2nXng6cpcfgVOmgi7JoxF7svFjAXx0?= =?us-ascii?Q?bs3lWtXSMEF1r+dTATVdRDZWZNvW3D/stcZJeur7LVvfNrspL+lPvfGaP8jx?= =?us-ascii?Q?6kiTeBw7ZOno9dNDm15HMRuHBB4xmRdyhqZB3tUyp3qI3598Tyd+KYQr+R7C?= =?us-ascii?Q?vDah3/jiC/fQrtD5i8QvYVZ67DZQFyAFpTetSto4bJulAE4RWWqbNoEa6cGJ?= =?us-ascii?Q?T/edj+oGoE96Nyi9sejxpCfQ7geAUe++x4a74xCk2WNPW30IDLpJzenzVW/D?= =?us-ascii?Q?qTWW5COQse0eyslWggcOR3nqDHH4zQNPBtxrG4ndFAmHZOLOABC6Z2yste6l?= =?us-ascii?Q?d3JpGFNEqN34J+Lc6bmjnxnxux0Q5+rB2y8Ym6qXkRWo4TjgmKGh48bsjs2T?= =?us-ascii?Q?C9cPP3zP583niqypS+Q7Cu6gWUm8VJnqkhZgiL9CfrvOCGpeKk3C6UwEwtDf?= =?us-ascii?Q?kZfJRp9zeId5ipEe/dZDNpioFSufS8mFxLgH7whrT32gpz5CwiqwppdDP4uI?= =?us-ascii?Q?R4MafyLc2GzfDTJ+VR3k3rdjdn6BKFGDA+yjTKkDm4MCrSmtBtr12icKPBBn?= =?us-ascii?Q?eWMNrIpU9EgnG9qGwMxo5UbvJ26Yf6P1P+e0R/84SwhXydhHDdF0G2yzhzxF?= =?us-ascii?Q?41elyyu3vVxVwwC5fIIu6JYn8U9FrgkBMNBLQlNH3y1IOcKBIGR4elWcdO3e?= =?us-ascii?Q?pi97O0hpNkKtYohTpOZk88hGuz4ZtlTMdnABOD1SFa5oA+mLLovvQKIKGnXD?= =?us-ascii?Q?Zm3Z/0xROsFnuN2PZK36LXfGwjFH+1KWr19yaasMxH7RqYbw+Alfx/+WH/0i?= =?us-ascii?Q?oIWzDsA//+u+zpbdE2OdPRjEkQiEhdwZKNF/X3ytTeOhCWh/Kxg6DCDQLdw6?= =?us-ascii?Q?icQHK9aqWn8Q8A7LNB+ZztmPKntU0aSGEY0KCPO2/gMADW+IxP6940Z7UUM4?= =?us-ascii?Q?C0VVNc8KCFwkq5hVSPsQajugCtruAuB2Xg8Fix3eO88cWhzdJ/DUq7Z1ppb/?= =?us-ascii?Q?AJ556k3kEci/TT+NQ5YD3cV?= X-Microsoft-Exchange-Diagnostics: 1;CO2PR07MB2584;5:WuckPvMRts/Zg0NVCVJrGc7NrkAEvvxOv2YpA+ontXc/PFCKjTYFd1/C9/qMkv3ag/plDmAaKE5ApXGeoq/0iTTSLu2CB8TjZiFTdG40948H/Fgj42TqboqC8tQ+T9DKhIrucW46Miq/4vgNb6yHvQ==;24:oVAc11zBt5N3P6f13o6ZFsywBFib6Qeiay9BnTp0e3SBZ8qmG+FwPSf2HFDIILl7ePTUEdcGf9Pg1xJxUaWSf8vNzlY+1WeNDOPY/O0MEPs=;7:rApzu2rAQTAlpYXkoeZttYoviL+r9OGBfF56Y5mbqajEWmmAQJ2EbFq7Joau8yDYgiW/6MuR41zzJ38BM4rvoSyu9T98FxA62oXt94SBnc0CZDXw6g+oO5xTMoByuqorLoJi1GiSlgAHiKsdxLSauPEPDrhFQ5Zya7qtuNbjydulJjMrflquN1Km43rfK3mo SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Apr 2016 12:34:56.1539 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO2PR07MB2584 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1182 Lines: 36 On Tue, Apr 26, 2016 at 09:36:20AM +0200, Wolfram Sang wrote: > > > > Yes, I thought briefly about splitting SMBUS_QUICK into read-write > > > variants too. To me the question is if this feature is still used on modern > > > devices or if this is more a relict of the past. I don't know enough > > > about SMBUS to answer that. > > Well, note that there are zero-length messages in I2C allowed as well. > Not only in SMBUS. I mainly use the term SMBUS_QUICK because it covers > both cases. > > > > Checking on ThunderX: > ... > > Do all these other numbers make sense (although there are no > > devices)? > > It makes sense in a way that it shows SMBUS_QUICK_WRITE is broken :) It > doesn't react to ACK/NACK properly. So, what needs to be done: > > 1) remove SMBUS_QUICK as you did in this patch 2) move the length check > so it doesn't only check read messages but also write messages. That is > to prevent handling custom setup I2C messages with a length of 0 (which > is legal). I'd suggest to return -EOPNOTSUPP in this case. OK, I'll do that. Should I rebase the remaining patches or would you like to review them first ? :) Thanks, Jan > Thanks, > > Wolfram >