Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753112AbbHETSv (ORCPT ); Wed, 5 Aug 2015 15:18:51 -0400 Received: from mga09.intel.com ([134.134.136.24]:31318 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751536AbbHETSt convert rfc822-to-8bit (ORCPT ); Wed, 5 Aug 2015 15:18:49 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.15,619,1432623600"; d="scan'208";a="762702764" From: "Luck, Tony" To: Seth Jennings , Mauro Carvalho Chehab CC: Aristeu Rozanski , "linux-edac@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: RE: [PATCH] sb_edac: fix TAD presence check for sbridge_mci_bind_devs() Thread-Topic: [PATCH] sb_edac: fix TAD presence check for sbridge_mci_bind_devs() Thread-Index: AQHQz6rcM/CfsepUnUC29e/pw6y5O539xzYA Date: Wed, 5 Aug 2015 19:18:47 +0000 Message-ID: <3908561D78D1C84285E8C5FCA982C28F32AD864C@ORSMSX114.amr.corp.intel.com> References: <1438798561-10180-1-git-send-email-sjenning@redhat.com> In-Reply-To: <1438798561-10180-1-git-send-email-sjenning@redhat.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.22.254.140] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 749 Lines: 19 > In 7d375bff, NUM_CHANNELS was changed to 8 and the channel space was > renumerated to handle EN, EP, and EX configurations. > > The *_mci_bind_devs functions, except for sbridge_mci_bind_devs(), got a > new device presence check in the form of saw_chan_mask. However, > sbridge_mci_bind_devs() still uses the NUM_CHANNELS for loop. > This patch adapts the saw_chan_mask logic for sbridge_mci_bind_devs() as > well. Oops. Sorry I missed that. Looks good. Acked-by: Tony Luck -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/