linux-kernel - Re: [PATCH 04/31] iio: light: cm3605: changing LED_* from enum led

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: <20220122163851.1aa52770@jic23-huawei>
Date:   Sat, 22 Jan 2022 16:38:51 +0000
From:   Jonathan Cameron <jic23@...nel.org>
To:     Luiz Sampaio <sampaio.ime@...il.com>
Cc:     Kevin Tsai <ktsai@...ellamicro.com>,
        Lars-Peter Clausen <lars@...afoo.de>,
        linux-kernel@...r.kernel.org, linux-iio@...r.kernel.org
Subject: Re: [PATCH 04/31] iio: light: cm3605: changing LED_* from enum
 led_brightness to actual value

On Fri, 21 Jan 2022 13:54:09 -0300
Luiz Sampaio <sampaio.ime@...il.com> wrote:

> The enum led_brightness, which contains the declaration of LED_OFF,
> LED_ON, LED_HALF and LED_FULL is obsolete, as the led class now supports
> max_brightness.

No sign-off and please always cc everyone on the cover letter of a series
including all mailing lists.  Otherwise we have no visibility of the
reasoning behind the change.  We all need to see the cover letter.
Sure I can go search for this patch on lore.kernel.org, but why
make it hard :)

Thanks,

Jonathan

> ---
>  drivers/iio/light/cm3605.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/iio/light/cm3605.c b/drivers/iio/light/cm3605.c
> index 50d34a98839c..f60db6cd9e6d 100644
> --- a/drivers/iio/light/cm3605.c
> +++ b/drivers/iio/light/cm3605.c
> @@ -238,7 +238,7 @@ static int cm3605_probe(struct platform_device *pdev)
>  
>  	/* Just name the trigger the same as the driver */
>  	led_trigger_register_simple("cm3605", &cm3605->led);
> -	led_trigger_event(cm3605->led, LED_FULL);
> +	led_trigger_event(cm3605->led, 255);
>  
>  	indio_dev->info = &cm3605_info;
>  	indio_dev->name = "cm3605";
> @@ -255,7 +255,7 @@ static int cm3605_probe(struct platform_device *pdev)
>  	return 0;
>  
>  out_remove_trigger:
> -	led_trigger_event(cm3605->led, LED_OFF);
> +	led_trigger_event(cm3605->led, 0);
>  	led_trigger_unregister_simple(cm3605->led);
>  out_disable_aset:
>  	gpiod_set_value_cansleep(cm3605->aset, 0);
> @@ -269,7 +269,7 @@ static int cm3605_remove(struct platform_device *pdev)
>  	struct iio_dev *indio_dev = platform_get_drvdata(pdev);
>  	struct cm3605 *cm3605 = iio_priv(indio_dev);
>  
> -	led_trigger_event(cm3605->led, LED_OFF);
> +	led_trigger_event(cm3605->led, 0);
>  	led_trigger_unregister_simple(cm3605->led);
>  	gpiod_set_value_cansleep(cm3605->aset, 0);
>  	iio_device_unregister(indio_dev);
> @@ -283,7 +283,7 @@ static int __maybe_unused cm3605_pm_suspend(struct device *dev)
>  	struct iio_dev *indio_dev = dev_get_drvdata(dev);
>  	struct cm3605 *cm3605 = iio_priv(indio_dev);
>  
> -	led_trigger_event(cm3605->led, LED_OFF);
> +	led_trigger_event(cm3605->led, 0);
>  	regulator_disable(cm3605->vdd);
>  
>  	return 0;
> @@ -298,7 +298,7 @@ static int __maybe_unused cm3605_pm_resume(struct device *dev)
>  	ret = regulator_enable(cm3605->vdd);
>  	if (ret)
>  		dev_err(dev, "failed to enable regulator in resume path\n");
> -	led_trigger_event(cm3605->led, LED_FULL);
> +	led_trigger_event(cm3605->led, 255);
>  
>  	return 0;
>  }