From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760312AbXH2Kml (ORCPT ); Wed, 29 Aug 2007 06:42:41 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753195AbXH2Kmb (ORCPT ); Wed, 29 Aug 2007 06:42:31 -0400 Received: from srv5.dvmed.net ([207.36.208.214]:41088 "EHLO mail.dvmed.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752589AbXH2Kma (ORCPT ); Wed, 29 Aug 2007 06:42:30 -0400 Message-ID: <46D54D93.5020100@garzik.org> Date: Wed, 29 Aug 2007 06:42:27 -0400 From: Jeff Garzik User-Agent: Thunderbird 2.0.0.5 (X11/20070719) MIME-Version: 1.0 To: Roland Dreier , Divy Le Ray CC: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Steve Wise Subject: Re: [PATCH 2.6.23 0/2] cxgb3 - Fix dev->priv usage References: <46D4F12C.7090409@chelsio.com> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Score: -4.3 (----) X-Spam-Report: SpamAssassin version 3.1.9 on srv5.dvmed.net summary: Content analysis details: (-4.3 points, 5.0 required) Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Roland Dreier wrote: > Looks OK to me but I would just roll up the second patch into the > first patch and let Jeff merge it as one commit. There's no point in > creating an intermediate tree that doesn't build -- it just breaks git > bisect for no useful purpose. Agreed -- this needs to be in a single patch. Jeff