Received: by 2002:ac0:98c7:0:0:0:0:0 with SMTP id g7-v6csp5268582imd; Tue, 30 Oct 2018 14:48:21 -0700 (PDT) X-Google-Smtp-Source: AJdET5eIJxXzb1dwsBecHssSeh+22Y7Doh2FInKUNY5Fb/oNJ2HzilzhU8Ie5ZgkOCWP89nBjl5Z X-Received: by 2002:a63:f006:: with SMTP id k6mr410352pgh.259.1540936101503; Tue, 30 Oct 2018 14:48:21 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1540936101; cv=none; d=google.com; s=arc-20160816; b=olQyoJRPMkv1fXy0bUQmFJIdwgj+m7PP2R28FCXZ0QtTQc+kfIcXnlzLgSfIz2XsqW laqYLmHPE0vU0av8LL8784ajzmLQNxfSq0BLi5JlN8000hS464y3q/ne7064XdZpd6zY EzbTKLKn/9zEiYa3pjBk7cj9wIhtFG23wBDXgrDQM3ET2OFRxMO0vcp1jRDU8LfW6bMx Gs2192sqNXel+PjxbUXkk1lGukABEVz5oqTRsI/26ouw30XQp/Q0InSsjlV44CdsoQRD YCd3Xm+oiVjb26AcXRMiE/OhaTZ3b+mdKHjvX60sZEtTJFYAe/+S0GwaPmr3Ii7hCmS/ AqKg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=qt+zxYm1Hjtk8t5DvE019D4czZAbeBACkllR7mN1znU=; b=Vi5FC/LtcjK6e1s9OP+Q3fjHxYICX5nOxFT54b/RDMRSa2Dk1mQ4v7Cy++OvahJUsP +IffhrIRB4Y0RosiJiWmPjqON0kN+iZf7+ZbuIjWPV7FoTVJ1bbU+MoWTJ/NZNZXa2g7 F4tExXO3KcaHj/JjaLibzRZruNCNlx7CfQ8s2agV4HvY0bVfdDgNo2W5ld+tegCPxPHK gc1eJAfg5TOiJJOhVJkdN1WThb/BmEwe7KzAzNsFtNVMHuwjnbegxDfY6WSz7vB5fSaV RT2g/zMSKunZyMZthymyIloyE/EDLf2rp3iS5q7WeMyQttXQKUpIWg4ZC7q9YhNjyppT fKYQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=ZowDBYDr; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id bg7-v6si21413797plb.5.2018.10.30.14.48.03; Tue, 30 Oct 2018 14:48:21 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=ZowDBYDr; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727840AbeJaGmn (ORCPT + 99 others); Wed, 31 Oct 2018 02:42:43 -0400 Received: from mail-qk1-f195.google.com ([209.85.222.195]:40003 "EHLO mail-qk1-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726174AbeJaGmm (ORCPT ); Wed, 31 Oct 2018 02:42:42 -0400 Received: by mail-qk1-f195.google.com with SMTP id f18-v6so8283362qkm.7; Tue, 30 Oct 2018 14:47:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=qt+zxYm1Hjtk8t5DvE019D4czZAbeBACkllR7mN1znU=; b=ZowDBYDrn56gIyHC1bmQ30BlxUInui9uyNDqBeXviCEJE648OmNQ224IXcOW0wcimx xp+Wup8xn8svDeJu3Cbe4Ao8gjooCnfnM6NN4hO6TXXfFaytUT/vGoeXaAK1pSRXCCCs 34xc5XXjtYrir292o9liKHvKBkpSWI1eUW7B0SWOIwNQe+KgTipS86qotfTHetuxhi83 Ou+z5cPH6JOhkgMnjKkSQYcFiO+PD0/TIKI4S7KVuzFtRdPcmRFj0k+SiD6hau6DE9kj ZA4t0EbmIwq5Er7NxwAnRaFmyqn2Gnxmj5I+iho/oYOJFQdDY5GCFmQSGadhHI6j3XDf KWag== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=qt+zxYm1Hjtk8t5DvE019D4czZAbeBACkllR7mN1znU=; b=B/8CaxS7oTU7DWrY2g/g2VuGjSsfph1SXh97YlACIgfaROZ6R1jwX/iVBoz913XYIK 1Jeeo30WdPeFR84zvpVL9u7RDDBKwS0eLwfVJSkdPn2jTvj3hiYB5rbyY2UI/iOIyfm5 chxfE/7Zz2S8l5HgG6jd6qYG9+V1vzVwrdGiR8Ea/xd6Oa6FSORv/i449hXcU56aHO/D w8iY5MZycv56VUv9xaWoJb3F6l+sFn1YfPgb5UfLTyKLaeQMDXaiDw5IN9Z2JXZSZAcP VN5utjXSaG+Jz9qfUnz7TpBQqzs7DaZt5VuggYFoRYTkEKqenzdUKAIbfHBO5vRQzkLS 8zJA== X-Gm-Message-State: AGRZ1gJgunMjDoZl+qVJQFyFBuEOIDqUAByNKHGJjS51Xdu8MK4vXRCX d+bck8aggtfVeB+Cx9o+J1GiY5+s2qgoZ6zq2YQ= X-Received: by 2002:a37:d91:: with SMTP id 139mr447609qkn.26.1540936051869; Tue, 30 Oct 2018 14:47:31 -0700 (PDT) MIME-Version: 1.0 References: <20181027150910.GA2070@syn-yoga530> <20181030205624.GC13681@bhelgaas-glaptop.roam.corp.google.com> In-Reply-To: <20181030205624.GC13681@bhelgaas-glaptop.roam.corp.google.com> From: Tobias Thomer Date: Tue, 30 Oct 2018 22:47:20 +0100 Message-ID: Subject: Re: [PATCH v7] i2c: Add PCI and platform drivers for the AMD MP2 I2C controller To: helgaas@kernel.org Cc: syniurge@gmail.com, linux-i2c@vger.kernel.org, wsa@the-dreams.de, Nehal-bakulchandra.Shah@amd.com, Shyam-sundar.S-k@amd.com, sandeep.singh@amd.com, linux-kernel@vger.kernel.org, rjw@rjwysocki.net, lenb@kernel.org, linux-acpi@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org I've tested v7 on a Yoga 530 and it seems to work (sometimes for hours) but it randomly dies with a i2c read timeout [ 9562.237020] i2c_amd_plat_mp2 AMDI0011:00: i2c read timed out and reloading/removing of the module results in this: [ 9703.591021] pcie_mp2_amd 0000:03:00.7: length 0 in event doesn't match buffer length 64! [ 9703.591030] pcie_mp2_amd 0000:03:00.7: unexpected slave address 0 (expected: 2c)! [ 9703.863740] i2c_amd_plat_mp2 AMDI0011:01: i2c connection timed out [ 9707.596826] INFO: task irq/67-WCOM517E:340 blocked for more than 120 seconds. [ 9707.596838] Tainted: G OE 4.19.0-v7 #1 [ 9707.596841] "echo 0 > /proc/sys/kernel/hung_task_timeout_secs" disables this message. [ 9707.596845] irq/67-WCOM517E D 0 340 2 0x80000080 [ 9707.596851] Call Trace: [ 9707.596868] ? __schedule+0x29b/0x8b0 [ 9707.596873] schedule+0x32/0x90 [ 9707.596875] schedule_preempt_disabled+0x14/0x20 [ 9707.596879] __mutex_lock.isra.0+0x217/0x520 [ 9707.596884] ? __switch_to_asm+0x40/0x70 [ 9707.596887] ? __switch_to_asm+0x34/0x70 [ 9707.596890] ? __switch_to_asm+0x34/0x70 [ 9707.596898] amd_mp2_read+0x4f/0x17b [i2c_amd_pci_mp2] [ 9707.596912] i2c_amd_xfer+0x61/0x150 [i2c_amd_plat_mp2] [ 9707.596919] __i2c_transfer+0x142/0x480 [ 9707.596922] ? _raw_spin_unlock_irq+0x1d/0x30 [ 9707.596926] ? irq_forced_thread_fn+0x70/0x70 [ 9707.596929] i2c_transfer+0x51/0xc0 [ 9707.596932] i2c_transfer_buffer_flags+0x4c/0x70 [ 9707.596937] i2c_hid_irq+0x3c/0x130 [i2c_hid] [ 9707.596941] irq_thread_fn+0x1f/0x50 [ 9707.596944] irq_thread+0xf7/0x1a0 [ 9707.596946] ? irq_thread_check_affinity.part.3+0xa0/0xa0 [ 9707.596949] ? irq_thread_dtor+0xb0/0xb0 [ 9707.596953] kthread+0x112/0x130 [ 9707.596956] ? kthread_park+0x80/0x80 [ 9707.596959] ret_from_fork+0x22/0x40