[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <96950836-f512-c5fb-237f-3e1502764c1f@ti.com>
Date: Tue, 27 Sep 2016 11:14:50 +0300
From: Tomi Valkeinen <tomi.valkeinen@...com>
To: Julia Lawall <Julia.Lawall@...6.fr>
CC: <joe@...ches.com>, <kernel-janitors@...r.kernel.org>,
<linux-fbdev@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 04/26] matroxfb: constify local structures
On 11/09/16 16:05, Julia Lawall wrote:
> For structure types defined in the same file or local header files, find
> top-level static structure declarations that have the following
> properties:
> 1. Never reassigned.
> 2. Address never taken
> 3. Not passed to a top-level macro call
> 4. No pointer or array-typed field passed to a function or stored in a
> variable.
> Declare structures having all of these properties as const.
>
> Done using Coccinelle.
> Based on a suggestion by Joe Perches <joe@...ches.com>.
>
> Signed-off-by: Julia Lawall <Julia.Lawall@...6.fr>
>
> ---
> The semantic patch seems too long for a commit log, but is in the cover
> letter.
>
> drivers/video/fbdev/matrox/matroxfb_g450.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Thanks, queued for 4.9.
Tomi
Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)
Powered by blists - more mailing lists