From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mho-01-ewr.mailhop.org (mho-03-ewr.mailhop.org [204.13.248.66]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id EB6EF1A00C7 for ; Sun, 7 Sep 2014 23:32:39 +1000 (EST) Date: Sun, 7 Sep 2014 09:32:18 -0400 From: Jason Cooper To: Philipp Zabel Subject: Re: [RFC PATCH 0/6] Change vendor prefix for Intersil Corporation Message-ID: <20140907133218.GA30828@titan.lakedaemon.net> References: <1408524184-31928-1-git-send-email-p.zabel@pengutronix.de> <20140820121643.GU12769@titan.lakedaemon.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20140820121643.GU12769@titan.lakedaemon.net> Cc: Mark Rutland , devicetree@vger.kernel.org, Alessandro Zummo , Russell King , Pawel Moll , Ian Campbell , Rob Herring , Paul Mackerras , Kumar Gala , linuxppc-dev@lists.ozlabs.org, linux-arm-kernel@lists.infradead.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Wed, Aug 20, 2014 at 08:16:43AM -0400, Jason Cooper wrote: > Philipp, > > On Wed, Aug 20, 2014 at 10:42:58AM +0200, Philipp Zabel wrote: > > Hi, > > > > currently there is a wild mixture of isl, isil, and intersil > > compatibles in the kernel. I figure at this point it is still > > possible to change this to use "isil" everywhere without too > > much pain, but it might be preferred to keep the already > > documented "isl" prefix, even though it doesn't follow the > > convention to use the NASDAQ symbol where available. > > > > The current users of the "isil" prefix are the following drivers > > and device trees, so we could just as well change those instead: > > arch/arm/boot/dts/tegra20-seaboard.dts > > arch/arm/boot/dts/tegra20-ventana.dts > > arch/arm/boot/dts/tegra30-cardhu.dtsi > > arch/powerpc/boot/dts/p1022rdk.dts > > drivers/staging/iio/light/isl29018.c > > drivers/staging/iio/light/isl29028.c > > > > regards > > Philipp > > > > Philipp Zabel (6): > > of: Change vendor prefix for Intersil Corporation to isil > > Documentation: Add isl1208 and isl12022 to trivial-devices list > > ARM: mvebu: Change vendor prefix for Intersil Corporation to isil > > powerpc/85xx: Change vendor prefix for Intersil Corporation to isil > > rtc: rtc-isl12022: Change vendor prefix for Intersil Corporation to > > isil > > rtc: rtc-isl12057: Change vendor prefix for Intersil Corporation to > > isil > > > > Documentation/devicetree/bindings/i2c/trivial-devices.txt | 4 +++- > > Documentation/devicetree/bindings/vendor-prefixes.txt | 2 +- > > arch/arm/boot/dts/armada-370-netgear-rn102.dts | 2 +- > > arch/arm/boot/dts/armada-370-netgear-rn104.dts | 2 +- > > arch/arm/boot/dts/armada-xp-netgear-rn2120.dts | 2 +- > > arch/powerpc/boot/dts/ppa8548.dts | 2 +- > > drivers/rtc/rtc-isl12022.c | 3 ++- > > drivers/rtc/rtc-isl12057.c | 3 ++- > > 8 files changed, 12 insertions(+), 8 deletions(-) > > This looks good overall. My only nit is that I'd like to see the legacy > name(s) captured in the binding docs. > > I'll take the Armada dts changes through mvebu/arm-soc after a few days. Just to dot our i's and cross our t's, would you mind sending out a non-rfc version of this series? thx, Jason.