Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S262169AbUCBW3s (ORCPT ); Tue, 2 Mar 2004 17:29:48 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S262228AbUCBW3s (ORCPT ); Tue, 2 Mar 2004 17:29:48 -0500 Received: from fw.osdl.org ([65.172.181.6]:15579 "EHLO mail.osdl.org") by vger.kernel.org with ESMTP id S262169AbUCBW3p (ORCPT ); Tue, 2 Mar 2004 17:29:45 -0500 Date: Tue, 2 Mar 2004 14:35:47 -0800 (PST) From: Linus Torvalds To: "Wojciech 'Sas' Cieciwa" cc: linux-kernel , Andrew Morton Subject: Re: [PATCH] ppc32: macserial.c missing variable declaration In-Reply-To: Message-ID: References: <1078263053.21573.176.camel@gaston> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 797 Lines: 24 On Wed, 3 Mar 2004, Wojciech 'Sas' Cieciwa wrote: > { > struct mac_serial * info = (struct mac_serial *)tty->driver_data; > unsigned char control, status; > + unsigned int cmd; > unsigned long flags; > > #ifdef CONFIG_KGDB This can't be right. Those variables are never initialized anywhere. The usage of 'cmd' should either be removed entirely, or it should be passed in as an argument, it looks like. In the meantime, it's better to have code that doesn't compile than code that compiles but can't possibly do anything sane. Linus - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/