Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933330AbbHJWIT (ORCPT ); Mon, 10 Aug 2015 18:08:19 -0400 Received: from mail-yk0-f182.google.com ([209.85.160.182]:32875 "EHLO mail-yk0-f182.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932741AbbHJWIR (ORCPT ); Mon, 10 Aug 2015 18:08:17 -0400 MIME-Version: 1.0 In-Reply-To: <20150810124510.GC1262@ulmo.nvidia.com> References: <1439207923-30812-1-git-send-email-srinivas.kandagatla@linaro.org> <1439207962-30860-1-git-send-email-srinivas.kandagatla@linaro.org> <20150810124510.GC1262@ulmo.nvidia.com> From: Rob Herring Date: Mon, 10 Aug 2015 17:07:57 -0500 Message-ID: Subject: Re: [PATCH RFC 1/5] drm/msm/hdmi: deprecate non standard gpio properties. To: Thierry Reding Cc: Srinivas Kandagatla , dri-devel , Rob Clark , David Airlie , "linux-kernel@vger.kernel.org" , Rob Herring , linux-arm-msm , "devicetree@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3889 Lines: 91 On Mon, Aug 10, 2015 at 7:45 AM, Thierry Reding wrote: > On Mon, Aug 10, 2015 at 12:59:22PM +0100, Srinivas Kandagatla wrote: >> This patch updates the bindings to discourage the usage of non standard >> gpio properites, this will help in projects focused on upstreaming. > > That last part is an odd comment to make in the commit message of a > patch submitted upstream... > >> These deprecated properties are still supported but will be remove over >> the time. > > You can't ever remove them because you can't ever be sure that people > won't be using an old DTB. It is only an ABI if anyone notices... > >> >> Signed-off-by: Srinivas Kandagatla >> --- >> Documentation/devicetree/bindings/drm/msm/hdmi.txt | 22 +++++++++++++++++----- >> 1 file changed, 17 insertions(+), 5 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/drm/msm/hdmi.txt b/Documentation/devicetree/bindings/drm/msm/hdmi.txt >> index c43aa53..acba581 100644 >> --- a/Documentation/devicetree/bindings/drm/msm/hdmi.txt >> +++ b/Documentation/devicetree/bindings/drm/msm/hdmi.txt >> @@ -11,15 +11,27 @@ Required properties: >> - interrupts: The interrupt signal from the hdmi block. >> - clocks: device clocks >> See ../clocks/clock-bindings.txt for details. >> -- qcom,hdmi-tx-ddc-clk-gpio: ddc clk pin >> -- qcom,hdmi-tx-ddc-data-gpio: ddc data pin >> -- qcom,hdmi-tx-hpd-gpio: hpd pin >> +- qcom,hdmi-tx-ddc-clk-gpios: ddc clk pin >> +- qcom,hdmi-tx-ddc-data-gpios: ddc data pin The driver doesn't appear to do anything other than set these gpios high. Presumably you would ultimately do a bit-bang i2c bus for these? At that point, aren't you going to want to use the i2c-gpio binding? I think this binding has bigger issues like why is it not using the hdmi-connector binding which has a standard "hpd-gpios" property already. I can't really single out this binding though. There's a lot of crap when it comes to DRM related bindings. >> +- qcom,hdmi-tx-hpd-gpios: hpd pin >> - core-vdda-supply: phandle to supply regulator >> - hdmi-mux-supply: phandle to mux regulator >> >> +- qcom,hdmi-tx-ddc-clk-gpio: (deprecated) use >> + "qcom,hdmi-tx-ddc-clk-gpios" instead >> +- qcom,hdmi-tx-ddc-data-gpio: (deprecated) use >> + "qcom,hdmi-tx-ddc-data-gpios" instead >> +- qcom,hdmi-tx-hpd-gpio: (deprecated) use >> + "qcom,hdmi-tx-hpd-gpios" instead >> + >> Optional properties: >> -- qcom,hdmi-tx-mux-en-gpio: hdmi mux enable pin >> -- qcom,hdmi-tx-mux-sel-gpio: hdmi mux select pin >> +- qcom,hdmi-tx-mux-en-gpios: hdmi mux enable pin >> +- qcom,hdmi-tx-mux-sel-gpios: hdmi mux select pin >> + >> +- qcom,hdmi-tx-mux-en-gpio: (deprecated) use "qcom,hdmi-tx-mux-en-gpios" >> + instead >> +- qcom,hdmi-tx-mux-sel-gpio: (deprecated) use "qcom,hdmi-tx-mux-sel-gpio" >> + instead >> - pinctrl-names: the pin control state names; should contain "default" >> - pinctrl-0: the default pinctrl state (active) >> - pinctrl-1: the "sleep" pinctrl state > > I don't see much use in listing that these properties are deprecated. We > already have code to catch the deprecated names, so having them in the > binding will at best be distracting. > > Anyway, I don't know if there's been any advice on this from the device > tree bindings maintainers, so adding devicetree@vger.kernel.org for > visibility. If they need to be maintained, then marking them deprecated is perfectly fine IMO. Also, if the maintainers of platforms using this are okay with it, you can just switch it. Given there are no in tree dts files using this, it can be argued that there is no ABI. Rob -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/