From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f51.google.com (mail-ej1-f51.google.com [209.85.218.51]) (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 A760F3655D6 for ; Fri, 19 Jun 2026 22:19:11 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.51 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781907553; cv=none; b=fXOjskNugFIE49HWL6jKZU0/QGAOFfxI6yR+2FZUVGkb+El9RA6NTSVkQkrd+kzQikPAiodjcFBQx5vDxsnU1JnEpFvpwC9SVlDK/C8icKiaWolVWPd12LSMmyvHrXw1jKW1XBy/i/VFK6yb0fV0BIKA+EQv22YpHeJRVusawdI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781907553; c=relaxed/simple; bh=DdEzNGXBbAJlLKvgKj8MWaMVC+JVtLzMufbPuNq+nrI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=dwkeYKW+ecErqszTLtUpX9jiEXNi2TFEPQxwHdRLiN2ept3/JjXhGEElJVj06oKKwlTkKN5Z76WR6gCFZ3aIDIjGTz7IOYEiMmjUcAtve3oQ7vvAJCgiwuF9A3MAtFdR24sznC0NDYIun7VtLQ2sJzsc3EdnXwrnM/ZPC2ve6Q4= 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=CMykFLnm; arc=none smtp.client-ip=209.85.218.51 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="CMykFLnm" Received: by mail-ej1-f51.google.com with SMTP id a640c23a62f3a-c0868ca8738so247910166b.0 for ; Fri, 19 Jun 2026 15:19:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1781907550; x=1782512350; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=QNj5JOnCEdVFcw3tVgjzQE7ie9IkNhVYMnie8GeBSUc=; b=CMykFLnmJFeYCNdiuoSiQSb8sH/yAen/3ZVwPKsufJnP2jRbqd+BmFFnAWrF8OYcR5 3TPZHhbkDpE3Hbuf4dg6yr/WlX3f19zgLaB5HvU+PeBMyf9tNqPaoXaPuMSFKRNanho4 y0M4RTbDXQ50qZEjnKClpLlK6VdTFR2zaKGodwTO75PhJq21C1wTLkn04ejQvLoOvhp7 nabc/qf1L1ilYGWq5z/rdoEFSLsZmVcox24muN1rz80t8H7MnZcYVe7dctlcuetjShG9 9kJV0A7hwW+X0mKdYNSDYOCYpVuMcPWJhKb0ySfXKuyrg5sdKWh++V9mQa0/m+cyJVhV kBTw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1781907550; x=1782512350; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=QNj5JOnCEdVFcw3tVgjzQE7ie9IkNhVYMnie8GeBSUc=; b=CVlA0of9MfB85duUnGmaqcWQZwoqhHWcktThKXJui4js/+LDB83x1axekiYQp8lgpi 7QgeBwmCS5UUR6mwjlNZc3vofdafN4woeNOxPs0oJ0qcpWNRbepO+Ej1oPNM/nMpuNVQ k5TUtDEqSWFJ6ko6zn5vLnanXW03bdGQfR++OzvwG54v+j5kO8hV9mz9pKr1eyTny1OL Nq3TO6mn2sIbt8ZD3FeqCGPdL7PH42GyqEoN9u7YRkorwcGil/6k+RNgogMH45e01wvp cft8ZrBy+ZkLkqjCJizeBuhzzuPezXMt59hR1Wi1dQkSOaOY0LQSLMO2MbjT5xQzWUom JuaQ== X-Forwarded-Encrypted: i=1; AFNElJ+kR1dr5wlhFnVsdWlWYrcbsaTYPS0u8n8goGpsNi9ZPNBPiKEvJKJUcTJi4sErn5ptDQwwSfLY4NQnB3E=@vger.kernel.org X-Gm-Message-State: AOJu0Yz4JCfAgLLONzX32sEzTpxBRgjHKeXJMs/YJBSj9KfjivZj+1Qh v2bVcJSZOiX1nhwQa/icUXACQJ1DCS9IATzvbPOV2E1wkOCK0A7UR+8H X-Gm-Gg: AfdE7ckpJgHr6fwieEg2bjvAmtOFlli1/F9Nocrq+fvXzhUX5CLO1qGULobo0F82hmA wGn5ZxZwVpu0xOX4qq9k15fCz/bi47PklmZFeMqobQVt/Knij0QoU0G404BzR0an2UHGIrPZJ9K 3gPuE9iS77aeypa3n9l39AZU0Ya2DSLTCnOT2dLwqdJQprsXteMXkqOfZQe5gKLGIr+1O8H4EI3 JeCZvBit1Im/tk8o9mSxX4KJU5AJpNfNRVluFvx4rchUur4D402mqNojk3ds3+UR8gJnxLR6GnB 1mMKJoFXT0CVbQHeEAoPiF1qQGd2+Y63ZxgRc7kiD7Du0M/q+/YPbDLoyic2Nvpp2L9oZ+Yb6MU a5mocX2khl0ROsQBdZcSsXSsDWI+UUkGh1mySR2fjJ+Vww+oj5c3r182JH0iMZmRa1aHYT0vmrK RTbs8RqG2t4r/5bpDqMLwmHdXUoQ3y4GQ8Fs4zf4E1UdP0tDo= X-Received: by 2002:a17:907:934d:b0:bfe:ed06:5a15 with SMTP id a640c23a62f3a-c098bd62af7mr321921166b.51.1781907549959; Fri, 19 Jun 2026 15:19:09 -0700 (PDT) Received: from adr.. (212-8-253-73.hosted-by-worldstream.net. [212.8.253.73]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-c0c5e49b77bsm28513066b.2.2026.06.19.15.19.05 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 19 Jun 2026 15:19:09 -0700 (PDT) From: =?UTF-8?q?Andr=C3=A9=20Moreira?= To: mchehab@kernel.org, gregkh@linuxfoundation.org Cc: linux-media@vger.kernel.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, =?UTF-8?q?Andr=C3=A9=20Moreira?= Subject: [PATCH 2/2] staging: media: av7110: refactor av7110_start_feed to reduce indentation Date: Fri, 19 Jun 2026 19:15:24 -0300 Message-ID: <20260619221524.51814-2-andrem.33333@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260619221524.51814-1-andrem.33333@gmail.com> References: <20260619221524.51814-1-andrem.33333@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The av7110_start_feed function contains heavily nested if-statements, causing excessive indentation and violating code style guidelines. Refactor the logic inside the DMX_MEMORY_FE case by inverting the conditional check to break early, reducing the indentation level. Signed-off-by: André Moreira --- drivers/staging/media/av7110/av7110.c | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/staging/media/av7110/av7110.c b/drivers/staging/media/av7110/av7110.c index e77be16f442c..2e32acf897ff 100644 --- a/drivers/staging/media/av7110/av7110.c +++ b/drivers/staging/media/av7110/av7110.c @@ -942,16 +942,16 @@ static int av7110_start_feed(struct dvb_demux_feed *feed) (feed->pes_type <= DMX_PES_PCR)) { switch (demux->dmx.frontend->source) { case DMX_MEMORY_FE: - if (feed->ts_type & TS_DECODER) - if (feed->pes_type < 2 && - !(demux->pids[0] & 0x8000) && - !(demux->pids[1] & 0x8000)) { - dvb_ringbuffer_flush_spinlock_wakeup(&av7110->avout); - dvb_ringbuffer_flush_spinlock_wakeup(&av7110->aout); - ret = av7110_av_start_play(av7110, RP_AV); - if (!ret) - demux->playing = 1; - } + if (feed->pes_type >= 2 || + (demux->pids[0] & 0x8000) || + (demux->pids[1] & 0x8000)) + break; + + dvb_ringbuffer_flush_spinlock_wakeup(&av7110->avout); + dvb_ringbuffer_flush_spinlock_wakeup(&av7110->aout); + ret = av7110_av_start_play(av7110, RP_AV); + if (!ret) + demux->playing = 1; break; default: ret = dvb_feed_start_pid(feed); -- 2.43.0