linux-kernel - Re: [PATCH V6 1/8] backlight: qcom-wled: Rename pm8941-wled.c to qcom-wled.c

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <3cbc14db-9d6d-f60f-eb92-4b4d80d3774d@ti.com>
Date:   Tue, 1 Oct 2019 10:03:13 -0500
From:   Dan Murphy <dmurphy@...com>
To:     Kiran Gunda <kgunda@...eaurora.org>, <bjorn.andersson@...aro.org>,
        <jingoohan1@...il.com>, <lee.jones@...aro.org>,
        <b.zolnierkie@...sung.com>, <dri-devel@...ts.freedesktop.org>,
        <daniel.thompson@...aro.org>, <jacek.anaszewski@...il.com>,
        <pavel@....cz>, <robh+dt@...nel.org>, <mark.rutland@....com>,
        <linux-leds@...r.kernel.org>, <devicetree@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>, Andy Gross <agross@...nel.org>,
        <linux-fbdev@...r.kernel.org>, <linux-arm-msm@...r.kernel.org>
Subject: Re: [PATCH V6 1/8] backlight: qcom-wled: Rename pm8941-wled.c to
 qcom-wled.c

Kiran

On 9/30/19 1:39 AM, Kiran Gunda wrote:
> pm8941-wled.c driver is supporting the WLED peripheral
> on pm8941. Rename it to qcom-wled.c so that it can support
> WLED on multiple PMICs.
>
> Signed-off-by: Kiran Gunda <kgunda@...eaurora.org>
> Reviewed-by: Bjorn Andersson <bjorn.andersson@...aro.org>
> Acked-by: Rob Herring <robh@...nel.org>
> Acked-by: Daniel Thompson <daniel.thompson@...aro.org>
> Acked-by: Pavel Machek <pavel@....cz>
> ---
>   .../bindings/leds/backlight/{pm8941-wled.txt => qcom-wled.txt}    | 2 +-

Instead of renaming this file would it be more maintainable to indicate 
in the pm8941-wled.txt

to reference the qcom-wled.txt file for complete description?

I will let Rob comment on maintainability.

Dan

>   drivers/video/backlight/Kconfig                                   | 8 ++++----
>   drivers/video/backlight/Makefile                                  | 2 +-
>   drivers/video/backlight/{pm8941-wled.c => qcom-wled.c}            | 0
>   4 files changed, 6 insertions(+), 6 deletions(-)
>   rename Documentation/devicetree/bindings/leds/backlight/{pm8941-wled.txt => qcom-wled.txt} (95%)
>   rename drivers/video/backlight/{pm8941-wled.c => qcom-wled.c} (100%)
>
> diff --git a/Documentation/devicetree/bindings/leds/backlight/pm8941-wled.txt b/Documentation/devicetree/bindings/leds/backlight/qcom-wled.txt
> similarity index 95%
> rename from Documentation/devicetree/bindings/leds/backlight/pm8941-wled.txt
> rename to Documentation/devicetree/bindings/leds/backlight/qcom-wled.txt
> index e5b294d..fb39e32 100644
> --- a/Documentation/devicetree/bindings/leds/backlight/pm8941-wled.txt
> +++ b/Documentation/devicetree/bindings/leds/backlight/qcom-wled.txt
> @@ -1,4 +1,4 @@
> -Binding for Qualcomm PM8941 WLED driver
> +Binding for Qualcomm Technologies, Inc. WLED driver
>   
>   Required properties:
>   - compatible: should be "qcom,pm8941-wled"
> diff --git a/drivers/video/backlight/Kconfig b/drivers/video/backlight/Kconfig
> index 8b081d6..6ff3176 100644
> --- a/drivers/video/backlight/Kconfig
> +++ b/drivers/video/backlight/Kconfig
> @@ -284,12 +284,12 @@ config BACKLIGHT_TOSA
>   	  If you have an Sharp SL-6000 Zaurus say Y to enable a driver
>   	  for its backlight
>   
> -config BACKLIGHT_PM8941_WLED
> -	tristate "Qualcomm PM8941 WLED Driver"
> +config BACKLIGHT_QCOM_WLED
> +	tristate "Qualcomm PMIC WLED Driver"
>   	select REGMAP
>   	help
> -	  If you have the Qualcomm PM8941, say Y to enable a driver for the
> -	  WLED block.
> +	  If you have the Qualcomm PMIC, say Y to enable a driver for the
> +	  WLED block. Currently it supports PM8941 and PMI8998.
>   
>   config BACKLIGHT_SAHARA
>   	tristate "Tabletkiosk Sahara Touch-iT Backlight Driver"
> diff --git a/drivers/video/backlight/Makefile b/drivers/video/backlight/Makefile
> index 63c507c..6f87770 100644
> --- a/drivers/video/backlight/Makefile
> +++ b/drivers/video/backlight/Makefile
> @@ -48,8 +48,8 @@ obj-$(CONFIG_BACKLIGHT_OMAP1)		+= omap1_bl.o
>   obj-$(CONFIG_BACKLIGHT_OT200)		+= ot200_bl.o
>   obj-$(CONFIG_BACKLIGHT_PANDORA)		+= pandora_bl.o
>   obj-$(CONFIG_BACKLIGHT_PCF50633)	+= pcf50633-backlight.o
> -obj-$(CONFIG_BACKLIGHT_PM8941_WLED)	+= pm8941-wled.o
>   obj-$(CONFIG_BACKLIGHT_PWM)		+= pwm_bl.o
> +obj-$(CONFIG_BACKLIGHT_QCOM_WLED)	+= qcom-wled.o
>   obj-$(CONFIG_BACKLIGHT_SAHARA)		+= kb3886_bl.o
>   obj-$(CONFIG_BACKLIGHT_SKY81452)	+= sky81452-backlight.o
>   obj-$(CONFIG_BACKLIGHT_TOSA)		+= tosa_bl.o
> diff --git a/drivers/video/backlight/pm8941-wled.c b/drivers/video/backlight/qcom-wled.c
> similarity index 100%
> rename from drivers/video/backlight/pm8941-wled.c
> rename to drivers/video/backlight/qcom-wled.c