Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932148AbVKCJgA (ORCPT ); Thu, 3 Nov 2005 04:36:00 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932159AbVKCJgA (ORCPT ); Thu, 3 Nov 2005 04:36:00 -0500 Received: from penta.pentaserver.com ([66.45.247.194]:53709 "EHLO penta.pentaserver.com") by vger.kernel.org with ESMTP id S932148AbVKCJf7 (ORCPT ); Thu, 3 Nov 2005 04:35:59 -0500 Message-ID: <4369D6B3.60501@linuxtv.org> Date: Thu, 03 Nov 2005 13:21:55 +0400 From: Manu Abraham User-Agent: Mozilla Thunderbird 1.0.6-1.1.fc4 (X11/20050720) X-Accept-Language: en-us, en MIME-Version: 1.0 To: Andrew Morton CC: Michael Krufky , linux-kernel@vger.kernel.org, linux-dvb-maintainer@linuxtv.org Subject: Re: [PATCH 01/37] dvb: dst: Correcty Identify Tuner and Daughterboards References: <4367235B.40608@m1k.net> <20051103132412.6266ccf0.akpm@osdl.org> In-Reply-To: <20051103132412.6266ccf0.akpm@osdl.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Antivirus-Scanner: Clean mail though you should still use an Antivirus X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - penta.pentaserver.com X-AntiAbuse: Original Domain - vger.kernel.org X-AntiAbuse: Originator/Caller UID/GID - [0 0] / [47 12] X-AntiAbuse: Sender Address Domain - linuxtv.org X-Source: X-Source-Args: X-Source-Dir: Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1294 Lines: 45 Andrew Morton wrote: >Michael Krufky 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 .. ? Thanks, Manu - 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/