From mboxrd@z Thu Jan 1 00:00:00 1970 From: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= Subject: Re: [PATCHv1 1/7] dt-bindings: use isil prefix for Intersil Date: Tue, 16 Dec 2014 09:19:13 +0100 Message-ID: <20141216081913.GC13486@pengutronix.de> References: <8e8a6eff40822083c0510f6c2787639ee819ef45.1418688042.git.arno@natisbad.org> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: <8e8a6eff40822083c0510f6c2787639ee819ef45.1418688042.git.arno-LkuqDEemtHBg9hUCZPvPmw@public.gmane.org> Sender: linux-i2c-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Arnaud Ebalard Cc: Wolfram Sang , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Heiko Stuebner , Shawn Guo , Mark Brown , Jason Cooper , Florian Vaussard , Andrew Lunn , Hayato Suzuki , Antony Pavlov , linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Andrew Morton , Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= List-Id: devicetree@vger.kernel.org Hello Arnaud, On Tue, Dec 16, 2014 at 01:22:27AM +0100, Arnaud Ebalard wrote: > This patch fixes DT documentation to reference isil (NASDAQ symbol an= d > the most used prefix inside the kernel) for all Intersil products. >=20 > It reverts both 7c75c1d5e72b ("dt-bindings: Document deprecated devic= e > vendor name to fix related warning") and b2ea3f82e798 ("dt-bindings: > Document correct and deprecated vendor-prefix with device isl29028"). Unrelated to this patch I wonder if checkpatch could be improved to not warn about unknown vendor prefixes in compatibility strings that only exist for compatibility. This assumes that it's not necessary to also document deprecated vendor prefixes, not sure about this one. Thoughts? > Signed-off-by: Arnaud Ebalard > --- > Documentation/devicetree/bindings/i2c/trivial-devices.txt | 5 ++--- > Documentation/devicetree/bindings/vendor-prefixes.txt | 3 +-- I guess it's OK to keep the two changes in one patch, but formally they are maintained by different people. Should this patch be split? > 2 files changed, 3 insertions(+), 5 deletions(-) >=20 > diff --git a/Documentation/devicetree/bindings/i2c/trivial-devices.tx= t b/Documentation/devicetree/bindings/i2c/trivial-devices.txt > index 9f4e3824e71e..0aaff72065bf 100644 > --- a/Documentation/devicetree/bindings/i2c/trivial-devices.txt > +++ b/Documentation/devicetree/bindings/i2c/trivial-devices.txt > @@ -58,9 +58,8 @@ fsl,sgtl5000 SGTL5000: Ultra Low-Power Audio Codec > gmt,g751 G751: Digital Temperature Sensor and Thermal Watchdog with= Two-Wire Interface > infineon,slb9635tt Infineon SLB9635 (Soft-) I2C TPM (old protocol, m= ax 100khz) > infineon,slb9645tt Infineon SLB9645 I2C TPM (new protocol, max 400kh= z) > -isl,isl12057 Intersil ISL12057 I2C RTC Chip > -isil,isl29028 (deprecated, use isl) > -isl,isl29028 Intersil ISL29028 Ambient Light and Proximit= y Sensor > +isil,isl12057 Intersil ISL12057 I2C RTC Chip > +isil,isl29028 Intersil ISL29028 Ambient Light and Proximit= y Sensor Here is some tab vs space issue. The surrounding lines (in the context) all use tab. > maxim,ds1050 5 Bit Programmable, Pulse-Width Modulator > maxim,max1237 Low-Power, 4-/12-Channel, 2-Wire Serial, 12-Bit ADCs > maxim,max6625 9-Bit/12-Bit Temperature Sensors with I=B2C-Compatibl= e Serial Interface Best regards Uwe --=20 Pengutronix e.K. | Uwe Kleine-K=F6nig = | Industrial Linux Solutions | http://www.pengutronix.de/= |