linux-kernel - Re: [PATCH] reset: sti: reset-syscfg: fix struct description warnings

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] [day] [month] [year] [list]

Message-ID: <295ebff6-e00c-878e-8fbd-f3d9883ed192@st.com>
Date:   Thu, 19 Nov 2020 07:56:45 +0000
From:   Patrice CHOTARD <patrice.chotard@...com>
To:     Alain Volmat <avolmat@...com>,
        Philipp Zabel <p.zabel@...gutronix.de>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] reset: sti: reset-syscfg: fix struct description warnings

Hi Alain

On 8/31/20 10:38 PM, Alain Volmat wrote:
> Fix formating of struct description to avoid warning highlighted
> by W=1 compilation.
>
> Fixes: e5d76075d930 ("drivers: reset: STi SoC system configuration reset controller support")
> Signed-off-by: Alain Volmat <avolmat@...com>
> ---
>  drivers/reset/sti/reset-syscfg.c | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/reset/sti/reset-syscfg.c b/drivers/reset/sti/reset-syscfg.c
> index 91215bb88f62..99b63035fe72 100644
> --- a/drivers/reset/sti/reset-syscfg.c
> +++ b/drivers/reset/sti/reset-syscfg.c
> @@ -17,7 +17,7 @@
>  #include "reset-syscfg.h"
>  
>  /**
> - * Reset channel regmap configuration
> + * struct syscfg_reset_channel - Reset channel regmap configuration
>   *
>   * @reset: regmap field for the channel's reset bit.
>   * @ack: regmap field for the channel's ack bit (optional).
> @@ -28,8 +28,9 @@ struct syscfg_reset_channel {
>  };
>  
>  /**
> - * A reset controller which groups together a set of related reset bits, which
> - * may be located in different system configuration registers.
> + * struct syscfg_reset_controller - A reset controller which groups together
> + * a set of related reset bits, which may be located in different system
> + * configuration registers.
>   *
>   * @rst: base reset controller structure.
>   * @active_low: are the resets in this controller active low, i.e. clearing

Reviewed-by: Patrice Chotard <patrice.chotard@...com>

Thanks

Patrice