Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754185Ab1C2T0H (ORCPT ); Tue, 29 Mar 2011 15:26:07 -0400 Received: from mail1-out1.atlantis.sk ([80.94.52.55]:45965 "EHLO mail.atlantis.sk" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752309Ab1C2T0F (ORCPT ); Tue, 29 Mar 2011 15:26:05 -0400 From: Ondrej Zary To: Hans Verkuil Subject: Re: [RFC PATCH 1/3] tea575x-tuner: various improvements Date: Tue, 29 Mar 2011 21:25:55 +0200 User-Agent: KMail/1.9.10 Cc: "Takashi Iwai" , jirislaby@gmail.com, alsa-devel@alsa-project.org, "Kernel development list" , linux-media@vger.kernel.org References: <201103121919.05657.linux@rainbow-software.org> <201103252240.16051.linux@rainbow-software.org> <201103261119.31897.hverkuil@xs4all.nl> In-Reply-To: <201103261119.31897.hverkuil@xs4all.nl> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <201103292125.58561.linux@rainbow-software.org> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2325 Lines: 73 On Saturday 26 March 2011 11:19:31 Hans Verkuil wrote: > On Friday, March 25, 2011 22:40:12 Ondrej Zary wrote: > > On Tuesday 22 March 2011 20:02:30 Hans Verkuil wrote: > > > BTW, can you run the v4l2-compliance utility for the two boards that > > > use this radio tuner? > > > > > > This utility is part of the v4l-utils repository > > > (http://git.linuxtv.org/v4l-utils.git). > > > > > > Run as 'v4l2-compliance -r /dev/radioX -v2'. > > > > > > I'm sure there will be some errors/warnings (warnings regarding > > > G/S_PRIORITY are to be expected). But I can use it to make a patch for > > > 2.6.40 that fixes any issues. > > > > The output is the same for both fm801 and es1968 (see below). Seems that > > there are 4 errors: > > 1. multiple-open does not work > > 2. something bad with s_frequency > > 3. input functions are present > > 4. no extended controls > > Thanks for testing! Some comments are below... > > > Running on 2.6.38 > > > > Driver Info: > > Driver name : tea575x-tuner > > Card type : TEA5757 > > Bus info : PCI > > Driver version: 0.0.2 > > Capabilities : 0x00050000 > > Tuner > > Radio > > > > Compliance test for device /dev/radio0 (not using libv4l2): > > > > Required ioctls: > > test VIDIOC_QUERYCAP: OK > > > > Allow for multiple opens: > > test second radio open: FAIL > > I will fix this. Once 2.6.39-rc1 is released I can make a patch fixing > this. > > > Debug ioctls: > > test VIDIOC_DBG_G_CHIP_IDENT: Not Supported > > test VIDIOC_DBG_G/S_REGISTER: Not Supported > > test VIDIOC_LOG_STATUS: Not Supported > > > > Input ioctls: > > test VIDIOC_G/S_TUNER: OK > > fail: set rangehigh+1 frequency did not return EINVAL > > test VIDIOC_G/S_FREQUENCY: FAIL > > Hmm, S_FREQUENCY apparently fails to check for valid frequency values. > Can you take a quick look at the code? The driver code is OK. But there is a bug in v4l2-test-input-output.cpp at line 214: if (ret) return fail("set rangehigh+1 frequency did not return EINVAL\n"); There should be "if (ret != EINVAL)" instead of "if (ret)". -- Ondrej Zary -- 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/