Date: Fri, 4 Nov 2005 01:24:15 +1100
From: Andrew Morton <akpm@osdl.org>
To: Manu Abraham <manu@linuxtv.org>
Cc: mkrufky@m1k.net, linux-kernel@vger.kernel.org,
       linux-dvb-maintainer@linuxtv.org
Subject: Re: [PATCH 01/37] dvb: dst: Correcty Identify Tuner and
 Daughterboards
Message-Id: <20051104012415.1ec337f3.akpm@osdl.org>
In-Reply-To: <4369D6B3.60501@linuxtv.org>
References: <4367235B.40608@m1k.net>
	<20051103132412.6266ccf0.akpm@osdl.org>
	<4369D6B3.60501@linuxtv.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1468
Lines: 45

Manu Abraham <manu@linuxtv.org> wrote:
>
> Andrew Morton wrote:
> 
> >Michael Krufky <mkrufky@m1k.net> wrote:
> >  
> >
> >> +static int dst_get_tuner_info(struct dst_state *state)
> >> +{
> >> +	u8 get_tuner_1[] = { 0x00, 0x13, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 };
> >> +	u8 get_tuner_2[] = { 0x00, 0x0b, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 };
> >> +
> >> +	get_tuner_1[7] = dst_check_sum(get_tuner_1, 7);
> >> +	get_tuner_2[7] = dst_check_sum(get_tuner_2, 7);
> >> +	if (state->type_flags & DST_TYPE_HAS_MULTI_FE) {
> >> +		if (dst_command(state, get_tuner_2, 8) < 0) {
> >> +			dprintk(verbose, DST_INFO, 1, "Unsupported Command");
> >> +			return -1;
> >> +		}
> >> +	} else {
> >> +		if (dst_command(state, get_tuner_1, 8) < 0) {
> >> +			dprintk(verbose, DST_INFO, 1, "Unsupported Command");
> >> +			return -1;
> >> +		}
> >> +	}
> >> +	memset(&state->board_info, '\0', 8);
> >> +	memcpy(&state->board_info, &state->rxbuffer, 8);
> >>    
> >>
> >
> >The memset is unneeded...
> >  
> >
> Hello Andrew,
> 
> I will have that changed in dvb-kernel CVS. Would you like me to send in 
> a patch for the same. Or you can have it changed .. ?
> 

There's certainly no rush ;)  Please just add it to the 2.6.16 to-do list.
-
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/