{"id":816653,"url":"http://patchwork.ozlabs.org/api/covers/816653/?format=json","web_url":"http://patchwork.ozlabs.org/project/ubuntu-kernel/cover/20170921050357.28558-1-kai.heng.feng@canonical.com/","project":{"id":15,"url":"http://patchwork.ozlabs.org/api/projects/15/?format=json","name":"Ubuntu Kernel","link_name":"ubuntu-kernel","list_id":"kernel-team.lists.ubuntu.com","list_email":"kernel-team@lists.ubuntu.com","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<20170921050357.28558-1-kai.heng.feng@canonical.com>","list_archive_url":null,"date":"2017-09-21T05:03:56","name":"[0/1,SRU,Xenial] Support setting I2C_TIMEOUT via ioctl for i2c-designware","submitter":{"id":70328,"url":"http://patchwork.ozlabs.org/api/people/70328/?format=json","name":"Kai-Heng Feng","email":"kai.heng.feng@canonical.com"},"mbox":"http://patchwork.ozlabs.org/project/ubuntu-kernel/cover/20170921050357.28558-1-kai.heng.feng@canonical.com/mbox/","series":[{"id":4287,"url":"http://patchwork.ozlabs.org/api/series/4287/?format=json","web_url":"http://patchwork.ozlabs.org/project/ubuntu-kernel/list/?series=4287","date":"2017-09-21T05:03:56","name":"Support setting I2C_TIMEOUT via ioctl for i2c-designware","version":1,"mbox":"http://patchwork.ozlabs.org/series/4287/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/covers/816653/comments/","headers":{"Return-Path":"<kernel-team-bounces@lists.ubuntu.com>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.ubuntu.com\n\t(client-ip=91.189.94.19; helo=huckleberry.canonical.com;\n\tenvelope-from=kernel-team-bounces@lists.ubuntu.com;\n\treceiver=<UNKNOWN>)","Received":["from huckleberry.canonical.com (huckleberry.canonical.com\n\t[91.189.94.19])\n\tby ozlabs.org (Postfix) with ESMTP id 3xyPfK6MF6z9t3C;\n\tThu, 21 Sep 2017 15:04:09 +1000 (AEST)","from localhost ([127.0.0.1] helo=huckleberry.canonical.com)\n\tby huckleberry.canonical.com with esmtp (Exim 4.86_2)\n\t(envelope-from <kernel-team-bounces@lists.ubuntu.com>)\n\tid 1dutec-0004kQ-7E; Thu, 21 Sep 2017 05:04:02 +0000","from youngberry.canonical.com ([91.189.89.112])\n\tby huckleberry.canonical.com with esmtps\n\t(TLS1.0:DHE_RSA_AES_128_CBC_SHA1:128)\n\t(Exim 4.86_2) (envelope-from <kai.heng.feng@canonical.com>)\n\tid 1duteb-0004kG-0W\n\tfor kernel-team@lists.ubuntu.com; Thu, 21 Sep 2017 05:04:01 +0000","from [175.41.48.77] (helo=localhost)\n\tby youngberry.canonical.com with esmtpsa\n\t(TLS1.0:RSA_AES_256_CBC_SHA1:32)\n\t(Exim 4.76) (envelope-from <kai.heng.feng@canonical.com>)\n\tid 1dutea-0000sn-Ad\n\tfor kernel-team@lists.ubuntu.com; Thu, 21 Sep 2017 05:04:00 +0000"],"From":"Kai-Heng Feng <kai.heng.feng@canonical.com>","To":"kernel-team@lists.ubuntu.com","Subject":"[PATCH 0/1 SRU Xenial] Support setting I2C_TIMEOUT via ioctl for\n\ti2c-designware","Date":"Thu, 21 Sep 2017 13:03:56 +0800","Message-Id":"<20170921050357.28558-1-kai.heng.feng@canonical.com>","X-Mailer":"git-send-email 2.14.1","MIME-Version":"1.0","X-BeenThere":"kernel-team@lists.ubuntu.com","X-Mailman-Version":"2.1.20","Precedence":"list","List-Id":"Kernel team discussions <kernel-team.lists.ubuntu.com>","List-Unsubscribe":"<https://lists.ubuntu.com/mailman/options/kernel-team>,\n\t<mailto:kernel-team-request@lists.ubuntu.com?subject=unsubscribe>","List-Archive":"<https://lists.ubuntu.com/archives/kernel-team>","List-Post":"<mailto:kernel-team@lists.ubuntu.com>","List-Help":"<mailto:kernel-team-request@lists.ubuntu.com?subject=help>","List-Subscribe":"<https://lists.ubuntu.com/mailman/listinfo/kernel-team>,\n\t<mailto:kernel-team-request@lists.ubuntu.com?subject=subscribe>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"kernel-team-bounces@lists.ubuntu.com","Sender":"\"kernel-team\" <kernel-team-bounces@lists.ubuntu.com>"},"content":"BugLink: https://bugs.launchpad.net/bugs/1718578\n\n[Impact]\nThe timeout value in i2c-designware is hardcoded. Userspace tool may\nwant to change the timeout value via ioctl I2C_TIMEOUT, but it doesn't\nwork for i2c-designware.\n\n[Test Case]\nChange the timeout value via ioctl I2C_TIMEOUT. Nothing happened.\nWith the patch, timeout value for i2c-designware will be updated\ncorrectly.\n\n[Fix]\nThe timeout value now uses adap->timeout instead of hardcoded HZ.\n\n[Regression Potential]\nMinimal. The change only applies to a specific driver, i2c-designware.\nAlso, it has minimal code change.\n\nWeifeng Voon (1):\n  i2c: designware: Use transfer timeout from ioctl I2C_TIMEOUT\n\n drivers/i2c/busses/i2c-designware-core.c | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)"}