linux-kernel - Re: [PATCH] docs: w1: Remove w1_gpio_platform

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: <b81edf65-45bd-4412-b373-16732a0d1abd@infradead.org>
Date: Wed, 6 Mar 2024 13:33:36 -0800
From: Randy Dunlap <rdunlap@...radead.org>
To: Ayush Singh <ayushdevel1325@...il.com>, linux-doc@...r.kernel.org
Cc: linux-kernel@...r.kernel.org, krzysztof.kozlowski@...aro.org,
 corbet@....net
Subject: Re: [PATCH] docs: w1: Remove w1_gpio_platform_data



On 3/6/24 11:46, Ayush Singh wrote:
> `linux/w1-gpio.h` was removed in a previous patch and should no longer be
> needed for using w1 gpio.
> 
> - Removal Patch: https://lore.kernel.org/all/cover.1701727212.git.u.kleine-koenig@pengutronix.de/
> 
> Signed-off-by: Ayush Singh <ayushdevel1325@...il.com>

Possibly add:
Fixes: 9c0a5b3f9e55 ("w1: gpio: Don't use platform data for driver data")

Reviewed-by: Randy Dunlap <rdunlap@...radead.org>
Thanks.

> ---
>  Documentation/w1/masters/w1-gpio.rst | 6 ------
>  1 file changed, 6 deletions(-)
> 
> diff --git a/Documentation/w1/masters/w1-gpio.rst b/Documentation/w1/masters/w1-gpio.rst
> index 15236605503b..5d7eebc291bb 100644
> --- a/Documentation/w1/masters/w1-gpio.rst
> +++ b/Documentation/w1/masters/w1-gpio.rst
> @@ -20,7 +20,6 @@ Example (mach-at91)
>  ::
>  
>    #include <linux/gpio/machine.h>
> -  #include <linux/w1-gpio.h>
>  
>    static struct gpiod_lookup_table foo_w1_gpiod_table = {
>  	.dev_id = "w1-gpio",
> @@ -30,14 +29,9 @@ Example (mach-at91)
>  	},
>    };
>  
> -  static struct w1_gpio_platform_data foo_w1_gpio_pdata = {
> -	.ext_pullup_enable_pin	= -EINVAL,
> -  };
> -
>    static struct platform_device foo_w1_device = {
>  	.name			= "w1-gpio",
>  	.id			= -1,
> -	.dev.platform_data	= &foo_w1_gpio_pdata,
>    };
>  
>    ...

-- 
#Randy