From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH 2/2] of_pci: use of_property_read_u32_array() Date: Mon, 24 Jul 2017 10:01:59 -0500 Message-ID: References: <20170723194327.508593685@cogentembedded.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Sergei Shtylyov Cc: Frank Rowand , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: devicetree@vger.kernel.org On Sun, Jul 23, 2017 at 2:46 PM, Sergei Shtylyov wrote: > On 07/23/2017 10:41 PM, Sergei Shtylyov wrote: > >> of_pci_get_devfn() and of_pci_parse_bus_range() somehow didn't use >> of_property_read_u32_array() though it was long available, basically >> open-coding it. Using the modern DT API saves several LoCs/bytes and > > > Oops, cut&paste error! This patch apparently doesn't save any LoCs, but > it does save 32 bytes of the object code (ARM gcc 4.8.5). Do I need to > resend? No, I've applied both. Thanks. Rob -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html