Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753537AbbKYQmz (ORCPT ); Wed, 25 Nov 2015 11:42:55 -0500 Received: from e28smtp06.in.ibm.com ([122.248.162.6]:38700 "EHLO e28smtp06.in.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753052AbbKYQmx (ORCPT ); Wed, 25 Nov 2015 11:42:53 -0500 X-IBM-Helo: d28dlp03.in.ibm.com X-IBM-MailFrom: hegdevasant@linux.vnet.ibm.com X-IBM-RcptTo: linux-kernel@vger.kernel.org;linux-leds@vger.kernel.org Message-ID: <5655E502.4070202@linux.vnet.ibm.com> Date: Wed, 25 Nov 2015 22:12:42 +0530 From: Vasant Hegde User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Jacek Anaszewski CC: Saurabh Sengar , rpurdie@rpsys.net, linux-leds@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] leds/powernv : removing NULL check References: <1448270896-23955-1-git-send-email-saurabh.truth@gmail.com> <5655C94F.4070504@linux.vnet.ibm.com> <5655D113.50206@samsung.com> In-Reply-To: <5655D113.50206@samsung.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-TM-AS-MML: disable X-Content-Scanned: Fidelis XPS MAILER x-cbid: 15112516-0021-0000-0000-000008B3C4FB Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 776 Lines: 31 On 11/25/2015 08:47 PM, Jacek Anaszewski wrote: > On 11/25/2015 03:44 PM, Vasant Hegde wrote: >> On 11/23/2015 02:58 PM, Saurabh Sengar wrote: >>> no need to explicitly check for pointer to be null, >>> of_prop_next_string anyways return NULL, if passed pointer is NULL >>> and hence loop will continue >> >> Thanks! Patch looks good. >> >> -Vasant >> >> > > I infer that you offer your: Yes.. Forgot to add Ack-by tag. > > Acked-by: Vasant Hegde > > Can I add it? :) Sure. Thanks! -Vasant -- 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/