From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751779AbbJLRAa (ORCPT ); Mon, 12 Oct 2015 13:00:30 -0400 Received: from mail-pa0-f50.google.com ([209.85.220.50]:34148 "EHLO mail-pa0-f50.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751219AbbJLRA2 (ORCPT ); Mon, 12 Oct 2015 13:00:28 -0400 Date: Mon, 12 Oct 2015 10:00:26 -0700 From: Dustin Byford To: Andy Shevchenko Cc: Mika Westerberg , Wolfram Sang , linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org, rjw@rjwysocki.net Subject: Re: [PATCH 1/2] i2c: scan entire ACPI namespace for I2C connections Message-ID: <20151012170026.GA5909@cumulusnetworks.com> References: <1439580693-5538-1-git-send-email-dustin@cumulusnetworks.com> <1444437707-19027-1-git-send-email-dustin@cumulusnetworks.com> <1444437707-19027-2-git-send-email-dustin@cumulusnetworks.com> <20151012104649.GW1506@lahna.fi.intel.com> <1444648839.8361.590.camel@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1444648839.8361.590.camel@linux.intel.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon Oct 12 14:20, Andy Shevchenko wrote: > On Mon, 2015-10-12 at 13:46 +0300, Mika Westerberg wrote: > > On Fri, Oct 09, 2015 at 05:41:46PM -0700, Dustin Byford wrote: > > > An I2cSerialBus connection resource descriptor may indicate a > > > ResourceSource (a string uniquely identifying the I2C bus > > > controller) > > > anywhere in the ACPI namespace. However, when enumerating > > > connections to a > > > I2C bus controller, i2c-core.c:acpi_i2c_register_devices() as only > > > searching devices that are descendants of the bus controller. > > > > > > This change corrects acpi_i2c_register_devices() to walk the entire > > > ACPI > > > namespace searching for I2C connections. > > > > > > Suggested-by: Mika Westerberg > > > Signed-off-by: Dustin Byford > > > > This patch is already included in Andy's latest series adding support > > for Intel Galileo here: > > > > https://patchwork.ozlabs.org/patch/527231/ > > > > Maybe we can add your Tested-by/Reviewed-by to that patch instead? > > Indeed, the Tested-by tag would be much appreciated! OK, that works for me. I didn't realize it had been submitted elsewhere or I wouldn't have sent the duplicate. Thanks. --Dustin