From mboxrd@z Thu Jan 1 00:00:00 1970 From: Javier Martinez Canillas Subject: Re: [PATCH v5 5/8] i2c: i2c-cbus-gpio: Add vendor prefix to retu node in example Date: Wed, 19 Apr 2017 15:13:19 -0400 Message-ID: References: <20170412172800.23035-1-javier@osg.samsung.com> <20170412172800.23035-6-javier@osg.samsung.com> <20170419185113.msqhjm7fzqfeyjlk@ninjato> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20170419185113.msqhjm7fzqfeyjlk@ninjato> Sender: linux-i2c-owner@vger.kernel.org To: Wolfram Sang Cc: linux-kernel@vger.kernel.org, Aaro Koskinen , devicetree@vger.kernel.org, Rob Herring , Tony Lindgren , Lee Jones , linux-i2c@vger.kernel.org, Mark Rutland List-Id: devicetree@vger.kernel.org Hello Wolfram, On 04/19/2017 02:51 PM, Wolfram Sang wrote: > On Wed, Apr 12, 2017 at 02:27:56PM -0300, Javier Martinez Canillas wrote: >> The example contains a device node for a retu device, but >> its compatible string doesn't have a vendor prefix. >> >> While being there, drop the -mfd suffix since isn't correct. >> >> Signed-off-by: Javier Martinez Canillas >> Acked-by: Rob Herring >> Acked-by: Aaro Koskinen >> Acked-by: Tony Lindgren > > Wouldn't it be nice if we fix the driver also so it actually matches the > below compatible? I can't find such a change in linux-next. > [snip] >> >> - retu-mfd: retu@1 { >> - compatible = "retu-mfd"; >> + retu: retu@1 { >> + compatible = "nokia,retu"; >> reg = <0x1>; >> }; You mean having a "nokia,retu" entry in a OF table? That's done by patch 3/8 in this series: http://www.spinics.net/lists/devicetree/msg173145.html Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America