Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751675AbdFIUMq (ORCPT ); Fri, 9 Jun 2017 16:12:46 -0400 Received: from mail-wr0-f195.google.com ([209.85.128.195]:36723 "EHLO mail-wr0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751633AbdFIUMm (ORCPT ); Fri, 9 Jun 2017 16:12:42 -0400 Date: Fri, 9 Jun 2017 22:12:37 +0200 From: Ralph Sennhauser To: Linus Walleij Cc: "linux-gpio@vger.kernel.org" , Thierry Reding , Alexandre Courbot , Rob Herring , Mark Rutland , Jason Cooper , Andrew Lunn , Gregory Clement , Sebastian Hesselbarth , Russell King , Richard Genoud , "linux-arm-kernel@lists.infradead.org" , "linux-pwm@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH] gpio: mvebu: change compatible string for PWM support Message-ID: <20170609221237.311cfb5c@gmail.com> In-Reply-To: References: <20170601200820.22396-1-ralph.sennhauser@gmail.com> Organization: none X-Mailer: Claws Mail 3.13.2 (GTK+ 2.24.31; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1181 Lines: 33 On Fri, 9 Jun 2017 09:42:26 +0200 Linus Walleij wrote: > On Thu, Jun 1, 2017 at 10:08 PM, Ralph Sennhauser > wrote: > > > As it turns out more than just Armada 370 and XP support using GPIO > > lines as PWM lines. For example the Armada 38x family has the same > > hardware support. As such "marvell,armada-370-xp-gpio" for the > > compatible string is a misnomer. > > > > Change the compatible string to "marvell,armada-370-gpio" before the > > driver makes it out of the -rc stage. This also follows the > > practice of using only the first device family supported as part of > > the name. > > > > Also update the documentation and comments in the code accordingly. > > > > Fixes: 757642f9a584 ("gpio: mvebu: Add limited PWM support") > > Signed-off-by: Ralph Sennhauser > > Patch applied for next with the tags. Hi Linus, As Richard already correctly pointed out this needs to go into 4.12 as we can't really change the compatible string after 4.12 final was released. So this commit was pushed to the wrong branch. Sorry for not having been more explicit in my wording. Thanks Ralph