Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752621AbbBJIK3 (ORCPT ); Tue, 10 Feb 2015 03:10:29 -0500 Received: from ausc60ps301.us.dell.com ([143.166.148.206]:2814 "EHLO ausc60ps301.us.dell.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751652AbbBJIK1 (ORCPT ); Tue, 10 Feb 2015 03:10:27 -0500 DomainKey-Signature: s=smtpout; d=dell.com; c=nofws; q=dns; h=X-LoopCount0:X-IronPort-AV:Date:From:To:Cc:Subject: Message-ID:References:MIME-Version:Content-Type: Content-Disposition:In-Reply-To:User-Agent; b=VHAiFYsgPEtYlCdKy8nq+HmcZ4VyFw+QZHY8eS6vuhd1AZ1WcotGvE3L mxVoIl9/TkRV0V0DpM89TtqGu2yBkeWZNFn19jMKAA0ddHQG4HitLWH0y 5M0plQB4OSm/dMkVw6oFc/75jiMgQAJP1oLc+SlWl1fICLA8XwAhmnNOf M=; X-LoopCount0: from 10.94.54.68 X-IronPort-AV: E=Sophos;i="5.09,548,1418104800"; d="scan'208";a="637291248" Date: Tue, 10 Feb 2015 03:10:20 -0500 From: Parmeshwr Prasad To: "Lad, Prabhakar" Cc: "pjones@redhat.com" , Jean-Christophe Plagniol-Villard , Tomi Valkeinen , LFBDEV , LKML , "linux-efi@vger.kernel.org" Subject: Re: [PATCH2/2] Trivial patch in efifb.c to solve common indent issue and indent error Message-ID: <20150210081015.GB8438@linuxteamdev.amer.dell.com> References: <20150209125548.GA27277@linuxteamdev.amer.dell.com> <20150210080614.GA8438@linuxteamdev.amer.dell.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150210080614.GA8438@linuxteamdev.amer.dell.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 Content-Length: 19799 Lines: 462 On Tue, Feb 10, 2015 at 02:06:24AM -0600, Prasad, Parmeshwr wrote: This is secong patch. This is addressing following changes: 1- It has removed "quoted string split across lines" warning. 2- There was static initialization of request_mem_succeeded, which is removed. 3- Assignment in if condition, this is fixed. >From bf6328196341ea36ec79fba71b9b5c2c36d61043 Mon Sep 17 00:00:00 2001 From: Parmeshwr Prasad Date: Tue, 10 Feb 2015 02:49:26 -0500 Subject: [PATCH 2/2] Trivial patch: In this patch printk is replaced by pr_* macros and static initialization of request_mem_succeeded is removed and assignment in if condition is removed Signed-off-by: Parmeshwr Prasad --- drivers/video/fbdev/efifb.c | 42 +++++++++++++++++++++--------------------- 1 file changed, 21 insertions(+), 21 deletions(-) diff --git a/drivers/video/fbdev/efifb.c b/drivers/video/fbdev/efifb.c index d757d0f..133fbfb 100644 --- a/drivers/video/fbdev/efifb.c +++ b/drivers/video/fbdev/efifb.c @@ -17,8 +17,7 @@ #include