From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Wed, 13 Jan 2010 02:30:37 +0100 From: Linus =?utf-8?Q?L=C3=BCssing?= Message-ID: <20100113013037.GA10176@Sellars> References: <20091231033752.GA18781@Sellars> <1263185876-24214-1-git-send-email-linus.luessing@web.de> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <1263185876-24214-1-git-send-email-linus.luessing@web.de> Sender: linus.luessing@web.de Subject: Re: [B.A.T.M.A.N.] [PATCH] batman-adv: atomic variable for vis-srv activation Reply-To: The list for a Better Approach To Mobile Ad-hoc Networking List-Id: The list for a Better Approach To Mobile Ad-hoc Networking List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: b.a.t.m.a.n@lists.open-mesh.net I now decided to use a more general variable name and as Marek suggested the checks against the given define-values. This should keep the structure of several "vis-modes" in general, as it had been done before by Simon, instead of just "on" and "off". The checkpatch-script from the batman-adv linux git branch showed me 0 errors/warnings for this one. - So far, just had the time to test this patch with my laptop and home-server only. But switching as well as receiving/displaying the vis-stuff worked fine for those (didn't have a closer look with tcpdump/wireshark for this vis-type part though). Cheers, Linus