From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Wed, 5 Dec 2012 17:30:19 +0100 From: Borislav Petkov To: "Ortiz, Lance E" Cc: "bhelgaas@google.com" , "lance_ortiz@hotmail.com" , "jiang.liu@huawei.com" , "tony.luck@intel.com" , "rostedt@goodmis.org" , "mchehab@redhat.com" , "linux-acpi@vger.kernel.org" , "linux-pci@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v7 2/3] aerdrv: Enhanced AER logging Message-ID: <20121205163019.GC28556@liondog.tnic> References: <20121204210305.4515.91100.stgit@grignak.americas.hpqcorp.net> <20121204210312.4515.13415.stgit@grignak.americas.hpqcorp.net> <20121205143202.GD2364@liondog.tnic> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 In-Reply-To: Sender: linux-acpi-owner@vger.kernel.org List-ID: On Wed, Dec 05, 2012 at 04:14:14PM +0000, Ortiz, Lance E wrote: > I removed the prefix argument because it was never used by its caller > and never set. The reason I added the prefix variable and set it to > NULL was to help in breaking up the patch and adding it would help the > intermittent patch build without changing too much code. I knew I was > actually going to use the variable in patch 3. No, the correct way to do that is to keep all changes that belong logically together in a single patch for ease of reviewing and avoid breakages. And in your case this should be pretty easy: simply move all the 'prefix' touching code to patch #3 and you're done, AFAICT. Thanks. -- Regards/Gruss, Boris.