From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1032829AbdAHJo1 (ORCPT ); Sun, 8 Jan 2017 04:44:27 -0500 Received: from saturn.retrosnub.co.uk ([178.18.118.26]:42864 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S938775AbdAHJnj (ORCPT ); Sun, 8 Jan 2017 04:43:39 -0500 Subject: Re: [PATCH 19/19] staging: iio: isl29028: remove legacy device tree binding To: Brian Masney , linux-iio@vger.kernel.org References: <1480817983-32359-1-git-send-email-masneyb@onstation.org> <1480817983-32359-20-git-send-email-masneyb@onstation.org> Cc: gregkh@linuxfoundation.org, devel@driverdev.osuosl.org, knaack.h@gmx.de, lars@metafoo.de, pmeerw@pmeerw.net, linux-kernel@vger.kernel.org, ldewangan@nvidia.com, devicetree@vger.kernel.org, Rob Herring , Mark Rutland From: Jonathan Cameron Message-ID: <927a9b4b-31e5-0f63-4526-97936a578648@kernel.org> Date: Sun, 4 Dec 2016 11:55:05 +0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: <1480817983-32359-20-git-send-email-masneyb@onstation.org> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 04/12/16 02:19, Brian Masney wrote: > The isl29028 staging driver contains a legacy binding of "isl,isl29028" > that is marked as legacy and to not use. There are no in tree references > to that binding. This patch removes the legacy binding in preparation > for moving the driver out of staging. Otherwise, we will have to support > this binding indefinitely. Leave this be. Given it exists there almost certainly device trees out there using it. Lets not break them just to save on one line of code. Supporting legacy bindings isn't exactly costly! I'm happy with all the patches I haven't commented on. Just can't apply them because they have dependencies on early patches that I have raised questions on. Another generally nice bit of cleanup. Thanks, Jonathan > > Signed-off-by: Brian Masney > Cc: devicetree@vger.kernel.org > Cc: Rob Herring > Cc: Mark Rutland > --- > drivers/staging/iio/light/isl29028.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/staging/iio/light/isl29028.c b/drivers/staging/iio/light/isl29028.c > index ae6896f..6be9aa7 100644 > --- a/drivers/staging/iio/light/isl29028.c > +++ b/drivers/staging/iio/light/isl29028.c > @@ -611,7 +611,6 @@ static const struct i2c_device_id isl29028_id[] = { > MODULE_DEVICE_TABLE(i2c, isl29028_id); > > static const struct of_device_id isl29028_of_match[] = { > - { .compatible = "isl,isl29028", }, /* for backward compat., don't use */ > { .compatible = "isil,isl29028", }, > { }, > }; >