Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752179AbdCNPyA (ORCPT ); Tue, 14 Mar 2017 11:54:00 -0400 Received: from mail-qt0-f174.google.com ([209.85.216.174]:35154 "EHLO mail-qt0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750942AbdCNPxJ (ORCPT ); Tue, 14 Mar 2017 11:53:09 -0400 Subject: Re: [PATCH v7 2/2] backlight arcxcnn add support for ArcticSand devices To: Daniel Thompson , Olimpiu Dejeu , robh@kernel.org References: <1489429323-6103-1-git-send-email-olimpiu@arcticsand.com> Cc: lee.jones@linaro.org, linux-kernel@vger.kernel.org, linux-fbdev@vger.kernel.org, devicetree@vger.kernel.org, jingoohan1@gmail.com, joe@perches.com, medasaro@arcticsand.com From: Brian Dodge Message-ID: <61f6e116-6190-72f3-ca89-54c9a94a6ac2@arcticsand.com> Date: Tue, 14 Mar 2017 11:53:04 -0400 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1017 Lines: 29 Hi Daniel, Yes checkpatch.pl was telling us this. I didn't think we had the "authority" to modify that file. Is it OK if we put the vendor-prefixes.txt change in the first patch (device tree bindings) or so you think we should have an 0003 patch for just that? Brian On 03/14/2017 06:16 AM, Daniel Thompson wrote: > On 13/03/17 18:22, Olimpiu Dejeu wrote: >> backlight: Add support for Arctic Sand LED backlight driver chips >> This driver provides support for the Arctic Sand arc2c0608 chip, >> and provides a framework to support future devices. >> Signed-off-by: Olimpiu Dejeu > > Please could you also submit a patch to add arc to > ./Documentation/devicetree/bindings/vendor-prefixes.txt. > > checkpatch.pl should have been asking you to do since v1 but only if > you test the patch series against a clean kernel (tools such as aiaiai > can help with this). > > However, once that is attended to: > > Reviewed-by: Daniel Thompson > > > Daniel.