From patchwork Thu Aug 13 23:59:18 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dustin Byford X-Patchwork-Id: 507235 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id DD8E8140216 for ; Fri, 14 Aug 2015 09:59:45 +1000 (AEST) Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (1024-bit key; unprotected) header.d=cumulusnetworks.com header.i=@cumulusnetworks.com header.b=CVwCUFLH; dkim-atps=neutral Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754619AbbHMX72 (ORCPT ); Thu, 13 Aug 2015 19:59:28 -0400 Received: from mail-pd0-f171.google.com ([209.85.192.171]:33964 "EHLO mail-pd0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753816AbbHMX70 (ORCPT ); Thu, 13 Aug 2015 19:59:26 -0400 Received: by pdbfa8 with SMTP id fa8so24726544pdb.1 for ; Thu, 13 Aug 2015 16:59:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cumulusnetworks.com; s=google; h=from:to:subject:date:message-id:in-reply-to:references; bh=Mywwlx1YRpUK1Cq/dRL/+KEWRa7xDwtFVqHi1UlOhiA=; b=CVwCUFLHChATM0gMB9uuBC+XN+AJBZKaRd7Yxi0Be8ikdmbwVjCRJerbH36ePf9jmC H46Los65ZBq+/T9dAmlTmQk3/cQAsDEmiynJ4epkWHUCW8xZxoIQBq0Zh73vdM/djhXv TgZIW4nptmB7/332zJs4mmO/h7wccCDygSt7U= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:subject:date:message-id:in-reply-to :references; bh=Mywwlx1YRpUK1Cq/dRL/+KEWRa7xDwtFVqHi1UlOhiA=; b=ftNNoAUzlsMQAkSAlNFvkSjmut5gM1SfQ8tpWTnPpivTWIfad0k+gzqn2w3M7rpTZV thfXbi4yjoAVZaYAVf3+HC+tGIbjcsMNBS9/DLaGkMDoO2oMJVGJkdi4K8LJUTQFi4Qy 5dGe4rj4V7ozhBy1jMmKGl8l2EuWRFrTptyAJH1XPnK4+cXFtA6h78wiihkOhLZsXnSM HopNviHTqFtlmKqvqR9sVeCBr1bQz9dcxqnrBsHNMs3W0ngsmDiZBbvjBm77eEGqh4c0 E+44ibRmmxFq9jNaAGzY4B51ZZY/xwRDUdTGU5TKd5SePoSlnNdied5vjwJq9ghggrFm rMYQ== X-Gm-Message-State: ALoCoQlGYkkDFPpQyh3eL545W6KimHBhbNVCFEEMHwbMpmhG7idsEgVnJCfhGp96KTakQfvkTlaM X-Received: by 10.70.123.100 with SMTP id lz4mr81650070pdb.161.1439510365879; Thu, 13 Aug 2015 16:59:25 -0700 (PDT) Received: from magnum.cumulusnetworks.com ([216.129.126.126]) by smtp.gmail.com with ESMTPSA id xp10sm3923336pac.34.2015.08.13.16.59.25 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 13 Aug 2015 16:59:25 -0700 (PDT) Received: by magnum.cumulusnetworks.com (Postfix, from userid 1016) id C89FC8A1D91; Thu, 13 Aug 2015 16:59:24 -0700 (PDT) From: Dustin Byford To: Wolfram Sang , linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [RFC PATCH 1/1] i2c: acpi: scan ACPI enumerated I2C mux channels Date: Thu, 13 Aug 2015 16:59:18 -0700 Message-Id: <1439510358-16664-2-git-send-email-dustin@cumulusnetworks.com> X-Mailer: git-send-email 2.1.4 In-Reply-To: <1439510358-16664-1-git-send-email-dustin@cumulusnetworks.com> References: <1439510358-16664-1-git-send-email-dustin@cumulusnetworks.com> Sender: linux-i2c-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-i2c@vger.kernel.org Set an ACPI companion for I2C mux channels enumerated through ACPI and ensure they are scanned for devices. Signed-off-by: Dustin Byford --- drivers/i2c/i2c-core.c | 10 ++++++++++ drivers/i2c/i2c-mux.c | 8 ++++++++ 2 files changed, 18 insertions(+) diff --git a/drivers/i2c/i2c-core.c b/drivers/i2c/i2c-core.c index c83e4d1..23cc8e9 100644 --- a/drivers/i2c/i2c-core.c +++ b/drivers/i2c/i2c-core.c @@ -171,8 +171,18 @@ static void acpi_i2c_register_devices(struct i2c_adapter *adap) if (!adap->dev.parent) return; + /* + * Determine where to start walking the ACPI namespace. The common + * case is to start at the adapter's parent device. However, in + * the case of a "virtual" I2C adapter created to represent a mux + * channel the parent dev (pointing to the mux device) does not + * have an ACPI handle. Walk starting at the adapter instead. + */ handle = ACPI_HANDLE(adap->dev.parent); if (!handle) + handle = ACPI_HANDLE(&adap->dev); + + if (!handle) return; status = acpi_walk_namespace(ACPI_TYPE_DEVICE, handle, 1, diff --git a/drivers/i2c/i2c-mux.c b/drivers/i2c/i2c-mux.c index 2ba7c0f..2731b99 100644 --- a/drivers/i2c/i2c-mux.c +++ b/drivers/i2c/i2c-mux.c @@ -25,6 +25,7 @@ #include #include #include +#include /* multiplexer per channel data */ struct i2c_mux_priv { @@ -173,6 +174,13 @@ struct i2c_adapter *i2c_add_mux_adapter(struct i2c_adapter *parent, } } + /* + * Now try to populate the mux adapter's ACPI node. + */ + if (has_acpi_companion(mux_dev)) + acpi_preset_companion(&priv->adap.dev, ACPI_COMPANION(mux_dev), + chan_id); + if (force_nr) { priv->adap.nr = force_nr; ret = i2c_add_numbered_adapter(&priv->adap);