Re: [PATCH v3 2/5] w1: ds2433: remove unused definition

From: Krzysztof Kozlowski
Date: Sat Dec 02 2023 - 07:02:51 EST


On 30/11/2023 14:52, marc.ferland@xxxxxxxxx wrote:
> From: Marc Ferland <marc.ferland@xxxxxxxxxxxx>
>
> W1_F23_TIME isn't used anywhere, get rid of it.
>
> Signed-off-by: Marc Ferland <marc.ferland@xxxxxxxxxxxx>
> ---
> drivers/w1/slaves/w1_ds2433.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/w1/slaves/w1_ds2433.c b/drivers/w1/slaves/w1_ds2433.c
> index 9f21fd98f799..e18523ef8c45 100644
> --- a/drivers/w1/slaves/w1_ds2433.c
> +++ b/drivers/w1/slaves/w1_ds2433.c
> @@ -30,8 +30,6 @@
> #define W1_PAGE_BITS 5
> #define W1_PAGE_MASK 0x1F
>
> -#define W1_F23_TIME 300

Same for W1_PAGE_COUNT and, so is there a point to keep it?

Best regards,
Krzysztof