Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Toke =?utf-8?Q?H=C3=B8iland-J=C3=B8rgensen?= <toke@toke.dk>
To:     Dan Carpenter <dan.carpenter@linaro.org>
Cc:     Kalle Valo <kvalo@kernel.org>, linux-wireless@vger.kernel.org,
        kernel-janitors@vger.kernel.org, Arnd Bergmann <arnd@arndb.de>
Subject: Re: [PATCH] ath9k: use struct_group() to silence static checker
 warning
In-Reply-To: <6de44cd9-a9a0-4b76-a9b5-a3c37b97f9aa@moroto.mountain>
References: <6de44cd9-a9a0-4b76-a9b5-a3c37b97f9aa@moroto.mountain>
Date:   Mon, 03 Jul 2023 19:35:33 +0200
Message-ID: <877crgdhju.fsf@toke.dk>
MIME-Version: 1.0
Content-Type: text/plain
Precedence: bulk

Dan Carpenter <dan.carpenter@linaro.org> writes:

> We are deliberately copying both ba_high and ba_low so use a struct
> group to make that clear.  Otherwise static checkers like Smatch and
> Clang complain that we are copying beyond the end of the ba_low struct
> member.
>
> Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>

Thanks for the fix; however, an identical patch is already queued, here:
https://patchwork.kernel.org/project/linux-wireless/patch/20230620080855.396851-2-dmantipov@yandex.ru/

-Toke