Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757669AbbEEKmx (ORCPT ); Tue, 5 May 2015 06:42:53 -0400 Received: from lists.s-osg.org ([54.187.51.154]:39530 "EHLO lists.s-osg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757456AbbEEKmo (ORCPT ); Tue, 5 May 2015 06:42:44 -0400 Date: Tue, 5 May 2015 07:42:38 -0300 From: Mauro Carvalho Chehab To: Rickard Strandqvist Cc: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] [media] usb: as102: as10x_cmd_cfg: Remove unused function Message-ID: <20150505074238.3c5df54a@recife.lan> In-Reply-To: <1421017510-26243-1-git-send-email-rickard_strandqvist@spectrumdigital.se> References: <1421017510-26243-1-git-send-email-rickard_strandqvist@spectrumdigital.se> Organization: Samsung X-Mailer: Claws Mail 3.11.1 (GTK+ 2.24.27; x86_64-redhat-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3341 Lines: 97 Em Mon, 12 Jan 2015 00:05:10 +0100 Rickard Strandqvist escreveu: > Remove the function as10x_cmd_eLNA_change_mode() that is not used anywhere. > > This was partially found by using a static code analysis program called cppcheck. The fix here should add support for it, as the DVB core now supports changing the LNA settings via DTV_LNA property, as described at: http://linuxtv.org/downloads/v4l-dvb-apis/FE_GET_SET_PROPERTY.html#DTV-LNA Regards, Mauro > > Signed-off-by: Rickard Strandqvist > --- > drivers/media/usb/as102/as10x_cmd.h | 1 - > drivers/media/usb/as102/as10x_cmd_cfg.c | 49 ------------------------------- > 2 files changed, 50 deletions(-) > > diff --git a/drivers/media/usb/as102/as10x_cmd.h b/drivers/media/usb/as102/as10x_cmd.h > index e06b84e..d87fc2f 100644 > --- a/drivers/media/usb/as102/as10x_cmd.h > +++ b/drivers/media/usb/as102/as10x_cmd.h > @@ -518,6 +518,5 @@ int as10x_cmd_get_context(struct as10x_bus_adapter_t *adap, > uint16_t tag, > uint32_t *pvalue); > > -int as10x_cmd_eLNA_change_mode(struct as10x_bus_adapter_t *adap, uint8_t mode); > int as10x_context_rsp_parse(struct as10x_cmd_t *prsp, uint16_t proc_id); > #endif > diff --git a/drivers/media/usb/as102/as10x_cmd_cfg.c b/drivers/media/usb/as102/as10x_cmd_cfg.c > index c87f2ca..74def1f 100644 > --- a/drivers/media/usb/as102/as10x_cmd_cfg.c > +++ b/drivers/media/usb/as102/as10x_cmd_cfg.c > @@ -130,55 +130,6 @@ out: > } > > /** > - * as10x_cmd_eLNA_change_mode - send eLNA change mode command to AS10x > - * @adap: pointer to AS10x bus adapter > - * @mode: mode selected: > - * - ON : 0x0 => eLNA always ON > - * - OFF : 0x1 => eLNA always OFF > - * - AUTO : 0x2 => eLNA follow hysteresis parameters > - * to be ON or OFF > - * > - * Return 0 on success or negative value in case of error. > - */ > -int as10x_cmd_eLNA_change_mode(struct as10x_bus_adapter_t *adap, uint8_t mode) > -{ > - int error; > - struct as10x_cmd_t *pcmd, *prsp; > - > - pcmd = adap->cmd; > - prsp = adap->rsp; > - > - /* prepare command */ > - as10x_cmd_build(pcmd, (++adap->cmd_xid), > - sizeof(pcmd->body.cfg_change_mode.req)); > - > - /* fill command */ > - pcmd->body.cfg_change_mode.req.proc_id = > - cpu_to_le16(CONTROL_PROC_ELNA_CHANGE_MODE); > - pcmd->body.cfg_change_mode.req.mode = mode; > - > - /* send command */ > - if (adap->ops->xfer_cmd) { > - error = adap->ops->xfer_cmd(adap, (uint8_t *) pcmd, > - sizeof(pcmd->body.cfg_change_mode.req) > - + HEADER_SIZE, (uint8_t *) prsp, > - sizeof(prsp->body.cfg_change_mode.rsp) > - + HEADER_SIZE); > - } else { > - error = AS10X_CMD_ERROR; > - } > - > - if (error < 0) > - goto out; > - > - /* parse response */ > - error = as10x_rsp_parse(prsp, CONTROL_PROC_ELNA_CHANGE_MODE_RSP); > - > -out: > - return error; > -} > - > -/** > * as10x_context_rsp_parse - Parse context command response > * @prsp: pointer to AS10x command response buffer > * @proc_id: id of the command -- 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/