From mboxrd@z Thu Jan 1 00:00:00 1970 From: Linus Walleij Subject: Re: [PATCH] pinctrl: samsung: Allow grouping multiple pinmux/pinconf nodes Date: Mon, 25 Nov 2013 15:46:12 +0100 Message-ID: References: <1384881034-10520-1-git-send-email-t.figa@samsung.com> <528BB789.5020201@wwwdotorg.org> <7242109.um7KlGhuI2@amdc1227> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Return-path: In-Reply-To: <7242109.um7KlGhuI2@amdc1227> Sender: linux-samsung-soc-owner@vger.kernel.org To: Tomasz Figa Cc: Stephen Warren , "linux-arm-kernel@lists.infradead.org" , linux-samsung-soc , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Doug Anderson , Thomas Abraham , Kukjin Kim , =?ISO-8859-1?Q?Heiko_St=FCbner?= , Kyungmin Park , Marek Szyprowski List-Id: devicetree@vger.kernel.org On Wed, Nov 20, 2013 at 3:00 PM, Tomasz Figa wrote: > Stephen: >> Is the lifetime of the string "returned" by >> of_property_read_string_index() really so short that you must copy the >> string? I'd be tempted just to store the pointer, although perhaps you >> need to get() the node so that's safe. > > Right. I have done it the copy-less way in other places, but missed this > one. Thanks. So I guess I'm waiting for a new version of this patch, right? Yours, Linus Walleij