[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAB=NE6U2-4uc90p5Os1kUp6QWChMcjarCMR7af0g=XUbGEXjGg@mail.gmail.com>
Date: Fri, 29 Mar 2013 00:29:44 -0700
From: "Luis R. Rodriguez" <mcgrof@...not-panic.com>
To: Julia Lawall <julia.lawall@...6.fr>
Cc: Jesse Barnes <jbarnes@...tuousgeek.org>,
florianschandinat <FlorianSchandinat@....de>,
linux-fbdev <linux-fbdev@...r.kernel.org>,
dri-devel@...ts.freedesktop.org,
"backports@...r.kernel.org" <backports@...r.kernel.org>,
cocci@...teme.lip6.fr,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"rodrigo.vivi" <rodrigo.vivi@...il.com>,
Daniel Vetter <daniel.vetter@...ll.ch>,
"rafael.j.wysocki" <rafael.j.wysocki@...el.com>
Subject: Re: [PATCH] compat/compat-drivers/linux-next: fb skip_vt_switch
On Thu, Mar 28, 2013 at 11:21 PM, Julia Lawall <julia.lawall@...6.fr> wrote:
>> > I looked in today's linux-next, and there seems to be only one
>> > initialization of this field, to true, and one test of this field. So
>> > perhaps the case for setting the field to false just isn't needed.
>>
>> Oh sorry now I get what you mean! I thought about this -- and yes I
>> decided to not add a bool false setting for the structure field given
>> that the assumption is this would not be something dynamic, and data
>> structure would be kzalloc()'d so by default they are 0.
>
> What do you do about the test, though?
Return the value.
Luis
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists