From nobody Thu Apr 2 12:33:35 2026 Received: from mail-pf1-f181.google.com (mail-pf1-f181.google.com [209.85.210.181]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 67241340A57 for ; Sat, 28 Mar 2026 16:51:03 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.181 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774716665; cv=none; b=FXZU/DJFa8YXybiLPfQImodeYxX5W55QI22PxyoHL2IEITdPAxwzzx8I7dyZ2gt+c5N1k5X3P7hrG0vSJBtn4Wp8m0IHbaXr9NVHq/pOXxYqU35ktYdDhcG02gMRsoVEYOOYqnTduQi9WW2BlC206MslPeLgoJgXnIK5jtGbnio= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774716665; c=relaxed/simple; bh=hgaAezUx7HYwGeej5DS+OEaboq9DyEUxZaRPG2f3esk=; h=From:To:Subject:Date:Message-Id:MIME-Version; b=T0RYY9jENk9ZinPPeOYZVKUFQ5QxkzzmLd1tlbvTCnDwyIRyaWjZvkK5fY7R2is0hiVF7wpY2HQE3GCYddNre83udJpLpz4IieDw6KdYuXZPAwO6GHcTFGSQPNKnrZno7jRj2B4tgCvWi1KoM/b+V6DhSV6yTIKbdnaMBn4TLdA= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=Axog/Kjy; arc=none smtp.client-ip=209.85.210.181 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="Axog/Kjy" Received: by mail-pf1-f181.google.com with SMTP id d2e1a72fcca58-82ae379000fso1154456b3a.1 for ; Sat, 28 Mar 2026 09:51:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1774716663; x=1775321463; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:to :from:from:to:cc:subject:date:message-id:reply-to; bh=dSW/eu4jxnxe0i2Wn72/Upo9NTFnOfqHiEesQY10C0g=; b=Axog/KjyOTC0U1BM5rBaMjwJudv3PG11w6ZLSMMcOYQ4OLG96bJOyJG22hDGwoDqX9 VUKyUBEZ+hvmz+7fEYwDUZ9NY4eAPgqk455LTWSzBaR6AfKxhLwAZKYfF9MCxxpBf98r iATak1WYlu6ljPy2lXdGwKVzFrcfHOMSR/0fhiIUblpCTdtLEUhdMaTdVwJwurPZFV/w 0ZBZR0uBMsrLPPgBK4ElCiZ/wM0+XVj0uN8bPH38xLNW3+jyACoqfX9zC/dGExA0a5MW gpiDDAedoxQ16C83e9lMt5fMVLG857bA78fP3JktYjZNJfJXwLo1qVG86VXN2WvBV5r9 BRLg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1774716663; x=1775321463; h=content-transfer-encoding:mime-version:message-id:date:subject:to :from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=dSW/eu4jxnxe0i2Wn72/Upo9NTFnOfqHiEesQY10C0g=; b=kKvsno3VHVcIj1WFXqcP4qlOU6x9xT151S7buKxAkdFXWMtZluwRk+9EwxRUzc4/Tf 03c9sHz6TVXFVkkQQFwokAnCIfezvzq+YBzSfUY6gEkIjeUku43vT9aHhm5fraPCa0e4 4Wzu9sa4gql/WkTqFzRtF9mjkvJygcrO2Wil90/D9w4BMhOAie0l+ecNB9Uaut7u1iiI VIfk+TVA8RSEo/tyRZWCyLX4D2N4/BLpvYTaTkCV18adOYcPUkzZDny6P7oONFB39hCK hodyKNSU9cPnuZFVuF6sDXlJDq7L5X3r8PmWfvsqXlx8sVOFEsjlOR4HMLs1J7K+4ZnX ItOw== X-Forwarded-Encrypted: i=1; AJvYcCXcDPZIg/LMsCgYrTNQvr7Y6YFTpOBBSJoiw7xOZXJtjxzLJpqeBLbP1vP2GvjXn+Ua4q698JB9jI7wUCY=@vger.kernel.org X-Gm-Message-State: AOJu0YywCfFuzmqhFQzp9pLkGAQNUKyVHMmCq80eyGnWCUNMO//bmH5w eL/p+ifBTZDMhRg9XXjYVQRNLNi3Y20TI33WG9UJ338Ew2W62o+OxdhO X-Gm-Gg: ATEYQzy0CkIcUaH5KJVTIaqZzSVKKmvmsL0UCw2YtwrF0tCH36hBTR/itvm4lBKS/WF ES7/iZsJT/bAo2JSoC4ny+KWj0HRqLDPmOk/MHJjqD3dywNsgoOGCuRl5ws4pZ7qpGddw8vxycX Kmul1gaXEa3lsIJxEl/HgQuDl+KHg5oYy6GDZSOeqK5JxbztQASylimSmXDnWaPIZ+y0FsP5Irl Ybs4OVawCkf2uUbN1w3W+e0r2yghGSxmmpTaymmnKUj3XFmpDXZVWXJoXMvdQEBqgZMBplbNd3z sRA8//Otyjx9GvVicZiscN8/+dOJ4iYRhTQGzPqX9SkUY70bIy5X5/1aBAnE23Qcqt97NeS32Xg lcuy3oVB3d3a0DZ+GXXpfD/WsXx49slGrfSpTjFS9K0kQfogVBCQLojw7UDte5VBYzro7vKDluQ mkptwGgEaye40UGKZM/323U0+EDIX3nIciHchI/Q== X-Received: by 2002:a05:6a00:428d:b0:82c:6b16:c894 with SMTP id d2e1a72fcca58-82c869c156bmr7912625b3a.20.1774716662453; Sat, 28 Mar 2026 09:51:02 -0700 (PDT) Received: from LIB-LAP-0152.lan ([2409:40f2:2088:4b61:5f5e:fb2e:946a:d345]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-82ca8466d89sm2612513b3a.22.2026.03.28.09.50.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 28 Mar 2026 09:51:02 -0700 (PDT) From: Chethan C To: Mauro Carvalho Chehab , Greg Kroah-Hartman , Petr Mladek , Kees Cook , Andy Shevchenko , Hans Verkuil , Chethan C , Osama Albahrani , Laurent Pinchart , linux-media@vger.kernel.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Subject: [PATCH v7] staging: media: av7110: fix coding style Date: Sat, 28 Mar 2026 22:20:34 +0530 Message-Id: <20260328165049.899917-1-mail.chethanc@gmail.com> X-Mailer: git-send-email 2.25.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Fix indentation and alignment issues reported by checkpatch.pl. Rename enums av7110_rec_play_state, av7110_type_rec_play_format, and av7110_encoder_command to follow kernel naming style. Rename wssData and wssMode to wss_data and wss_mode to avoid camelCase identifiers. Signed-off-by: Chethan C --- v7: - Remove unintended whitespace-only change in wait_event_interruptible() block v6: - Rename wssMode - Group av7110_fw_cmd parameters logically v5: - Limit lines to 80 columns - Add trailing commas to enums av7110_rec_play_state, av7110_type_rec_play_format, av7110_encoder_command - Remove unnecessary cast v4: - Fix coding style issues reported by checkpatch.pl v3: - Fix coding style and alignment issues reported by checkpatch.pl - Rename av7110_pid_command, av7110_type_rec_play_format, av7110_encoder_command to follow kernel naming style v2: - Fix indentation issues reported by checkpatch.pl - Rename av7110_rec_play_state to follow kernel naming style --- drivers/staging/media/av7110/av7110.c | 31 +++++--- drivers/staging/media/av7110/av7110.h | 4 +- drivers/staging/media/av7110/av7110_av.c | 89 ++++++++++++++++------- drivers/staging/media/av7110/av7110_ca.c | 3 +- drivers/staging/media/av7110/av7110_hw.h | 71 +++++++++--------- drivers/staging/media/av7110/av7110_ir.c | 4 +- drivers/staging/media/av7110/av7110_v4l.c | 22 +++--- 7 files changed, 134 insertions(+), 90 deletions(-) diff --git a/drivers/staging/media/av7110/av7110.c b/drivers/staging/media/= av7110/av7110.c index 607992100baf..dd62038dec56 100644 --- a/drivers/staging/media/av7110/av7110.c +++ b/drivers/staging/media/av7110/av7110.c @@ -121,19 +121,22 @@ static void init_av7110_av(struct av7110 *av7110) if (ret < 0) pr_err("cannot set internal volume to maximum:%d\n", ret); =20 - ret =3D av7110_fw_cmd(av7110, COMTYPE_ENCODER, SetMonitorType, - 1, (u16)av7110->display_ar); + ret =3D av7110_fw_cmd(av7110, COMTYPE_ENCODER, + AV7110_SET_MONITOR_TYPE, 1, av7110->display_ar); if (ret < 0) pr_err("unable to set aspect ratio\n"); - ret =3D av7110_fw_cmd(av7110, COMTYPE_ENCODER, SetPanScanType, - 1, av7110->display_panscan); + ret =3D av7110_fw_cmd(av7110, COMTYPE_ENCODER, + AV7110_SET_PANSCAN_TYPE, 1, + av7110->display_panscan); if (ret < 0) pr_err("unable to set pan scan\n"); =20 - ret =3D av7110_fw_cmd(av7110, COMTYPE_ENCODER, SetWSSConfig, 2, 2, wss_cf= g_4_3); + ret =3D av7110_fw_cmd(av7110, COMTYPE_ENCODER, + AV7110_SET_WSS_CONFIG, 2, 2, wss_cfg_4_3); if (ret < 0) pr_err("unable to configure 4:3 wss\n"); - ret =3D av7110_fw_cmd(av7110, COMTYPE_ENCODER, SetWSSConfig, 2, 3, wss_cf= g_16_9); + ret =3D av7110_fw_cmd(av7110, COMTYPE_ENCODER, + AV7110_SET_WSS_CONFIG, 2, 3, wss_cfg_16_9); if (ret < 0) pr_err("unable to configure 16:9 wss\n"); =20 @@ -717,7 +720,8 @@ static inline int SetPIDs(struct av7110 *av7110, u16 vp= id, u16 apid, u16 ttpid, if (av7110->audiostate.bypass_mode) aflags |=3D 0x8000; =20 - return av7110_fw_cmd(av7110, COMTYPE_PIDFILTER, MultiPID, 6, + return av7110_fw_cmd(av7110, COMTYPE_PIDFILTER, + AV7110_MULTI_PID, 6, pcrpid, vpid, apid, ttpid, subpid, aflags); } =20 @@ -785,7 +789,7 @@ static int StartHWFilter(struct dvb_demux_filter *dvbdm= xfilter) av7110_p2t_init(&av7110->p2t_filter[dvbdmxfilter->index], dvbdmxfeed); } =20 - buf[0] =3D (COMTYPE_PID_FILTER << 8) + AddPIDFilter; + buf[0] =3D (COMTYPE_PID_FILTER << 8) + AV7110_ADD_PID_FILTER; buf[1] =3D 16; buf[2] =3D dvbdmxfeed->pid; buf[3] =3D mode; @@ -828,7 +832,7 @@ static int StopHWFilter(struct dvb_demux_filter *dvbdmx= filter) =20 av7110->handle2filter[handle] =3D NULL; =20 - buf[0] =3D (COMTYPE_PID_FILTER << 8) + DelPIDFilter; + buf[0] =3D (COMTYPE_PID_FILTER << 8) + AV7110_DEL_PID_FILTER; buf[1] =3D 1; buf[2] =3D handle; ret =3D av7110_fw_request(av7110, buf, 3, answ, 2); @@ -873,7 +877,8 @@ static int dvb_feed_start_pid(struct dvb_demux_feed *dv= bdmxfeed) =20 if (dvbdmxfeed->pes_type < 2 && npids[0]) if (av7110->fe_synced) { - ret =3D av7110_fw_cmd(av7110, COMTYPE_PIDFILTER, Scan, 0); + ret =3D av7110_fw_cmd(av7110, COMTYPE_PIDFILTER, + AV7110_SCAN, 0); if (ret) return ret; } @@ -1911,11 +1916,13 @@ static int av7110_fe_lock_fix(struct av7110 *av7110= , enum fe_status status) av7110->pids[DMX_PES_TELETEXT], 0, av7110->pids[DMX_PES_PCR]); if (!ret) - ret =3D av7110_fw_cmd(av7110, COMTYPE_PIDFILTER, Scan, 0); + ret =3D av7110_fw_cmd(av7110, COMTYPE_PIDFILTER, + AV7110_SCAN, 0); } else { ret =3D SetPIDs(av7110, 0, 0, 0, 0, 0); if (!ret) { - ret =3D av7110_fw_cmd(av7110, COMTYPE_PID_FILTER, FlushTSQueue, 0); + ret =3D av7110_fw_cmd(av7110, COMTYPE_PID_FILTER, + AV7110_FLUSH_TS_QUEUE, 0); if (!ret) ret =3D av7110_wait_msgstate(av7110, GPMQBusy); } diff --git a/drivers/staging/media/av7110/av7110.h b/drivers/staging/media/= av7110/av7110.h index b584754f4be0..797dda437cbe 100644 --- a/drivers/staging/media/av7110/av7110.h +++ b/drivers/staging/media/av7110/av7110.h @@ -243,8 +243,8 @@ struct av7110 { struct dvb_video_events video_events; video_size_t video_size; =20 - u16 wssMode; - u16 wssData; + u16 wss_mode; + u16 wss_data; =20 struct infrared ir; =20 diff --git a/drivers/staging/media/av7110/av7110_av.c b/drivers/staging/med= ia/av7110/av7110_av.c index 2993ac43c49c..22f1335d371b 100644 --- a/drivers/staging/media/av7110/av7110_av.c +++ b/drivers/staging/media/av7110/av7110_av.c @@ -111,7 +111,7 @@ int av7110_av_start_record(struct av7110 *av7110, int a= v, =20 if (av7110->playing || (av7110->rec_mode & av)) return -EBUSY; - av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, __Stop, 0); + av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, AV7110_REC_PLAY_STOP, 0); dvbdmx->recording =3D 1; av7110->rec_mode |=3D av; =20 @@ -121,7 +121,9 @@ int av7110_av_start_record(struct av7110 *av7110, int a= v, dvbdmx->pesfilter[0]->pid, dvb_filter_pes2ts_cb, (void *)dvbdmx->pesfilter[0]); - ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, __Record, 2, AudioPES, 0= ); + ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, + AV7110_REC_PLAY_RECORD, 2, + AV7110_AUDIO_PES, 0); break; =20 case RP_VIDEO: @@ -129,7 +131,9 @@ int av7110_av_start_record(struct av7110 *av7110, int a= v, dvbdmx->pesfilter[1]->pid, dvb_filter_pes2ts_cb, (void *)dvbdmx->pesfilter[1]); - ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, __Record, 2, VideoPES, 0= ); + ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, + AV7110_REC_PLAY_RECORD, 2, + AV7110_VIDEO_PES, 0); break; =20 case RP_AV: @@ -141,7 +145,9 @@ int av7110_av_start_record(struct av7110 *av7110, int a= v, dvbdmx->pesfilter[1]->pid, dvb_filter_pes2ts_cb, (void *)dvbdmx->pesfilter[1]); - ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, __Record, 2, AV_PES, 0); + ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, + AV7110_REC_PLAY_RECORD, 2, + AV7110_AV_PES, 0); break; } return ret; @@ -158,7 +164,7 @@ int av7110_av_start_play(struct av7110 *av7110, int av) if (av7110->playing & av) return -EBUSY; =20 - av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, __Stop, 0); + av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, AV7110_REC_PLAY_STOP, 0); =20 if (av7110->playing =3D=3D RP_NONE) { av7110_ipack_reset(&av7110->ipack[0]); @@ -168,15 +174,21 @@ int av7110_av_start_play(struct av7110 *av7110, int a= v) av7110->playing |=3D av; switch (av7110->playing) { case RP_AUDIO: - ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, __Play, 2, AudioPES, 0); + ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, + AV7110_REC_PLAY_PLAY, 2, + AV7110_AUDIO_PES, 0); break; case RP_VIDEO: - ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, __Play, 2, VideoPES, 0); + ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, + AV7110_REC_PLAY_PLAY, 2, + AV7110_VIDEO_PES, 0); av7110->sinfo =3D 0; break; case RP_AV: av7110->sinfo =3D 0; - ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, __Play, 2, AV_PES, 0); + ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, + AV7110_REC_PLAY_PLAY, 2, + AV7110_AV_PES, 0); break; } return ret; @@ -190,15 +202,19 @@ int av7110_av_stop(struct av7110 *av7110, int av) =20 if (!(av7110->playing & av) && !(av7110->rec_mode & av)) return 0; - av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, __Stop, 0); + av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, AV7110_REC_PLAY_STOP, 0); if (av7110->playing) { av7110->playing &=3D ~av; switch (av7110->playing) { case RP_AUDIO: - ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, __Play, 2, AudioPES, 0); + ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, + AV7110_REC_PLAY_PLAY, 2, + AV7110_AUDIO_PES, 0); break; case RP_VIDEO: - ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, __Play, 2, VideoPES, 0); + ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, + AV7110_REC_PLAY_PLAY, 2, + AV7110_VIDEO_PES, 0); break; case RP_NONE: ret =3D av7110_set_vidmode(av7110, av7110->vidmode); @@ -208,10 +224,14 @@ int av7110_av_stop(struct av7110 *av7110, int av) av7110->rec_mode &=3D ~av; switch (av7110->rec_mode) { case RP_AUDIO: - ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, __Record, 2, AudioPES, = 0); + ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, + AV7110_REC_PLAY_RECORD, 2, + AV7110_AUDIO_PES, 0); break; case RP_VIDEO: - ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, __Record, 2, VideoPES, = 0); + ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, + AV7110_REC_PLAY_RECORD, 2, + AV7110_VIDEO_PES, 0); break; case RP_NONE: break; @@ -325,7 +345,8 @@ int av7110_set_vidmode(struct av7110 *av7110, enum av71= 10_video_mode mode) =20 dprintk(2, "av7110:%p\n", av7110); =20 - ret =3D av7110_fw_cmd(av7110, COMTYPE_ENCODER, LoadVidCode, 1, mode); + ret =3D av7110_fw_cmd(av7110, COMTYPE_ENCODER, + AV7110_LOAD_VID_CODE, 1, mode); =20 if (!ret && !av7110->playing) { ret =3D ChangePIDs(av7110, av7110->pids[DMX_PES_VIDEO], @@ -333,7 +354,8 @@ int av7110_set_vidmode(struct av7110 *av7110, enum av71= 10_video_mode mode) av7110->pids[DMX_PES_TELETEXT], 0, av7110->pids[DMX_PES_PCR]); if (!ret) - ret =3D av7110_fw_cmd(av7110, COMTYPE_PIDFILTER, Scan, 0); + ret =3D av7110_fw_cmd(av7110, COMTYPE_PIDFILTER, + AV7110_SCAN, 0); } return ret; } @@ -1168,7 +1190,8 @@ static int dvb_video_ioctl(struct file *file, } if (av7110->videostate.stream_source =3D=3D VIDEO_SOURCE_MEMORY) { if (av7110->playing =3D=3D RP_AV) { - ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, __Stop, 0); + ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, + AV7110_REC_PLAY_STOP, 0); if (ret) break; av7110->playing &=3D ~RP_VIDEO; @@ -1184,7 +1207,8 @@ static int dvb_video_ioctl(struct file *file, case VIDEO_FREEZE: av7110->videostate.play_state =3D VIDEO_FREEZED; if (av7110->playing & RP_VIDEO) - ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, __Pause, 0); + ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, + AV7110_REC_PLAY_PAUSE, 0); else ret =3D vidcom(av7110, AV_VIDEO_CMD_FREEZE, 1); if (!ret) @@ -1193,7 +1217,8 @@ static int dvb_video_ioctl(struct file *file, =20 case VIDEO_CONTINUE: if (av7110->playing & RP_VIDEO) - ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, __Continue, 0); + ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, + AV7110_REC_PLAY_CONTINUE, 0); if (!ret) ret =3D vidcom(av7110, AV_VIDEO_CMD_PLAY, 0); if (!ret) { @@ -1248,8 +1273,9 @@ static int dvb_video_ioctl(struct file *file, if (ret < 0) break; av7110->videostate.display_format =3D format; - ret =3D av7110_fw_cmd(av7110, COMTYPE_ENCODER, SetPanScanType, - 1, av7110->display_panscan); + ret =3D av7110_fw_cmd(av7110, COMTYPE_ENCODER, + AV7110_SET_PANSCAN_TYPE, 1, + av7110->display_panscan); break; } =20 @@ -1259,8 +1285,8 @@ static int dvb_video_ioctl(struct file *file, break; } av7110->display_ar =3D arg; - ret =3D av7110_fw_cmd(av7110, COMTYPE_ENCODER, SetMonitorType, - 1, (u16)arg); + ret =3D av7110_fw_cmd(av7110, COMTYPE_ENCODER, + AV7110_SET_MONITOR_TYPE, 1, arg); break; =20 #ifdef CONFIG_COMPAT @@ -1291,7 +1317,8 @@ static int dvb_video_ioctl(struct file *file, //note: arg is ignored by firmware if (av7110->playing & RP_VIDEO) ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, - __Scan_I, 2, AV_PES, 0); + AV7110_REC_PLAY_SCAN_I, 2, + AV7110_AV_PES, 0); else ret =3D vidcom(av7110, AV_VIDEO_CMD_FFWD, arg); if (!ret) { @@ -1303,7 +1330,9 @@ static int dvb_video_ioctl(struct file *file, case VIDEO_SLOWMOTION: if (av7110->playing & RP_VIDEO) { if (av7110->trickmode !=3D TRICK_SLOW) - ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, __Slow, 2, 0, 0); + ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, + AV7110_REC_PLAY_SLOW, 2, + 0, 0); if (!ret) ret =3D vidcom(av7110, AV_VIDEO_CMD_SLOW, arg); } else { @@ -1329,15 +1358,18 @@ static int dvb_video_ioctl(struct file *file, av7110_ipack_reset(&av7110->ipack[1]); if (av7110->playing =3D=3D RP_AV) { ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, - __Play, 2, AV_PES, 0); + AV7110_REC_PLAY_PLAY, 2, + AV7110_AV_PES, 0); if (ret) break; if (av7110->trickmode =3D=3D TRICK_FAST) ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, - __Scan_I, 2, AV_PES, 0); + AV7110_REC_PLAY_SCAN_I, 2, + AV7110_AV_PES, 0); if (av7110->trickmode =3D=3D TRICK_SLOW) { ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, - __Slow, 2, 0, 0); + AV7110_REC_PLAY_SLOW, 2, + 0, 0); if (!ret) ret =3D vidcom(av7110, AV_VIDEO_CMD_SLOW, arg); } @@ -1483,7 +1515,8 @@ static int dvb_audio_ioctl(struct file *file, av7110_ipack_reset(&av7110->ipack[0]); if (av7110->playing =3D=3D RP_AV) ret =3D av7110_fw_cmd(av7110, COMTYPE_REC_PLAY, - __Play, 2, AV_PES, 0); + AV7110_REC_PLAY_PLAY, 2, + AV7110_AV_PES, 0); break; =20 case AUDIO_SET_ID: diff --git a/drivers/staging/media/av7110/av7110_ca.c b/drivers/staging/med= ia/av7110/av7110_ca.c index 63d9c97a5190..24876c8fdc8c 100644 --- a/drivers/staging/media/av7110/av7110_ca.c +++ b/drivers/staging/media/av7110/av7110_ca.c @@ -307,7 +307,8 @@ static int dvb_ca_ioctl(struct file *file, unsigned int= cmd, void *parg) mutex_unlock(&av7110->ioctl_mutex); return -EINVAL; } - av7110_fw_cmd(av7110, COMTYPE_PIDFILTER, SetDescr, 5, + av7110_fw_cmd(av7110, COMTYPE_PIDFILTER, + AV7110_SET_DESCR, 5, (descr->index << 8) | descr->parity, (descr->cw[0] << 8) | descr->cw[1], (descr->cw[2] << 8) | descr->cw[3], diff --git a/drivers/staging/media/av7110/av7110_hw.h b/drivers/staging/med= ia/av7110/av7110_hw.h index d4579f411c56..3afe474da871 100644 --- a/drivers/staging/media/av7110/av7110_hw.h +++ b/drivers/staging/media/av7110/av7110_hw.h @@ -21,12 +21,12 @@ enum av7110_bootstate { }; =20 enum av7110_type_rec_play_format { - RP_None, - AudioPES, - AudioMp2, - AudioPCM, - VideoPES, - AV_PES + AV7110_RP_NONE, + AV7110_AUDIO_PES, + AV7110_AUDIO_MP2, + AV7110_AUDIO_PCM, + AV7110_VIDEO_PES, + AV7110_AV_PES, }; =20 enum av7110_osd_palette_type { @@ -112,19 +112,19 @@ enum av7110_osd_command { }; =20 enum av7110_pid_command { - MultiPID, - VideoPID, - AudioPID, - InitFilt, - FiltError, - NewVersion, - CacheError, - AddPIDFilter, - DelPIDFilter, - Scan, - SetDescr, - SetIR, - FlushTSQueue + AV7110_MULTI_PID, + AV7110_VIDEO_PID, + AV7110_AUDIO_PID, + AV7110_INIT_FILT, + AV7110_FILT_ERROR, + AV7110_NEW_VERSION, + AV7110_CACHE_ERROR, + AV7110_ADD_PID_FILTER, + AV7110_DEL_PID_FILTER, + AV7110_SCAN, + AV7110_SET_DESCR, + AV7110_SET_IR, + AV7110_FLUSH_TS_QUEUE, }; =20 enum av7110_mpeg_command { @@ -158,24 +158,24 @@ enum av7110_request_command { }; =20 enum av7110_encoder_command { - SetVidMode, - SetTestMode, - LoadVidCode, - SetMonitorType, - SetPanScanType, - SetFreezeMode, - SetWSSConfig + AV7110_SET_VID_MODE, + AV7110_SET_TEST_MODE, + AV7110_LOAD_VID_CODE, + AV7110_SET_MONITOR_TYPE, + AV7110_SET_PANSCAN_TYPE, + AV7110_SET_FREEZE_MODE, + AV7110_SET_WSS_CONFIG, }; =20 enum av7110_rec_play_state { - __Record, - __Stop, - __Play, - __Pause, - __Slow, - __FF_IP, - __Scan_I, - __Continue + AV7110_REC_PLAY_RECORD, + AV7110_REC_PLAY_STOP, + AV7110_REC_PLAY_PLAY, + AV7110_REC_PLAY_PAUSE, + AV7110_REC_PLAY_SLOW, + AV7110_REC_PLAY_FF_IP, + AV7110_REC_PLAY_SCAN_I, + AV7110_REC_PLAY_CONTINUE, }; =20 enum av7110_fw_cmd_misc { @@ -452,7 +452,8 @@ static inline int SendDAC(struct av7110 *av7110, u8 add= r, u8 data) =20 static inline int av7710_set_video_mode(struct av7110 *av7110, int mode) { - return av7110_fw_cmd(av7110, COMTYPE_ENCODER, SetVidMode, 1, mode); + return av7110_fw_cmd(av7110, COMTYPE_ENCODER, + AV7110_SET_VID_MODE, 1, mode); } =20 static inline int vidcom(struct av7110 *av7110, u32 com, u32 arg) diff --git a/drivers/staging/media/av7110/av7110_ir.c b/drivers/staging/med= ia/av7110/av7110_ir.c index 68b3979ba5f2..a033844d21b2 100644 --- a/drivers/staging/media/av7110/av7110_ir.c +++ b/drivers/staging/media/av7110/av7110_ir.c @@ -71,8 +71,8 @@ int av7110_set_ir_config(struct av7110 *av7110) { dprintk(4, "ir config =3D %08x\n", av7110->ir.ir_config); =20 - return av7110_fw_cmd(av7110, COMTYPE_PIDFILTER, SetIR, 1, - av7110->ir.ir_config); + return av7110_fw_cmd(av7110, COMTYPE_PIDFILTER, + AV7110_SET_IR, 1, av7110->ir.ir_config); } =20 static int change_protocol(struct rc_dev *rcdev, u64 *rc_type) diff --git a/drivers/staging/media/av7110/av7110_v4l.c b/drivers/staging/me= dia/av7110/av7110_v4l.c index 200a7a29ea31..aed7581fa8a5 100644 --- a/drivers/staging/media/av7110/av7110_v4l.c +++ b/drivers/staging/media/av7110/av7110_v4l.c @@ -556,7 +556,7 @@ static int vidioc_g_fmt_sliced_vbi_out(struct file *fil= e, void *fh, if (FW_VERSION(av7110->arm_app) < 0x2623) return -EINVAL; memset(&f->fmt.sliced, 0, sizeof(f->fmt.sliced)); - if (av7110->wssMode) { + if (av7110->wss_mode) { f->fmt.sliced.service_set =3D V4L2_SLICED_WSS_625; f->fmt.sliced.service_lines[0][23] =3D V4L2_SLICED_WSS_625; } @@ -596,14 +596,14 @@ static int vidioc_s_fmt_sliced_vbi_out(struct file *f= ile, void *fh, return -EINVAL; if (f->fmt.sliced.service_set & V4L2_SLICED_WSS_625) { /* WSS controlled by userspace */ - av7110->wssMode =3D 1; - av7110->wssData =3D 0; + av7110->wss_mode =3D 1; + av7110->wss_data =3D 0; } else { /* WSS controlled by firmware */ - av7110->wssMode =3D 0; - av7110->wssData =3D 0; + av7110->wss_mode =3D 0; + av7110->wss_data =3D 0; return av7110_fw_cmd(av7110, COMTYPE_ENCODER, - SetWSSConfig, 1, 0); + AV7110_SET_WSS_CONFIG, 1, 0); } return 0; } @@ -616,17 +616,19 @@ static ssize_t av7110_vbi_write(struct file *file, co= nst char __user *data, size int rc; =20 dprintk(2, "\n"); - if (FW_VERSION(av7110->arm_app) < 0x2623 || !av7110->wssMode || count != =3D sizeof(d)) + if (FW_VERSION(av7110->arm_app) < 0x2623 || + !av7110->wss_mode || count !=3D sizeof(d)) return -EINVAL; if (copy_from_user(&d, data, count)) return -EFAULT; if ((d.id !=3D 0 && d.id !=3D V4L2_SLICED_WSS_625) || d.field !=3D 0 || d= .line !=3D 23) return -EINVAL; if (d.id) - av7110->wssData =3D ((d.data[1] << 8) & 0x3f00) | d.data[0]; + av7110->wss_data =3D ((d.data[1] << 8) & 0x3f00) | d.data[0]; else - av7110->wssData =3D 0x8000; - rc =3D av7110_fw_cmd(av7110, COMTYPE_ENCODER, SetWSSConfig, 2, 1, av7110-= >wssData); + av7110->wss_data =3D 0x8000; + rc =3D av7110_fw_cmd(av7110, COMTYPE_ENCODER, + AV7110_SET_WSS_CONFIG, 2, 1, av7110->wss_data); return (rc < 0) ? rc : count; } =20 --=20 2.25.1