From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f52.google.com (mail-wr1-f52.google.com [209.85.221.52]) (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 0BC9D372B58 for ; Thu, 19 Mar 2026 09:02:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.52 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773910973; cv=none; b=GeOz1c1Bp6C2yoFud8+qlbdEbzm/XcDDJy5LGPtct/jYRpt4BPA6KCo2vqJZabQ6NsUX3gXJr8ieAmeNqCdFIii0gmykTZPXyP/rZTPxmcxdB3x13ZxnDhaqgj6evZeprDQUn2qGVOGOUlQPqWrP51IaI8SmUedhN+JEzsYlj8U= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773910973; c=relaxed/simple; bh=75nM3lYRQU7IZ2qAXZ2X20vZw1P7LFmoQkBFEZVgVAA=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=m2Yt1VHt+tvT598EkhMf9Im1HWKELpBso3cSgQwiKM+fgw7G4hj+7nq/x8X5qRXerEBQLb1xMp40s0khvGTUa4PNq1myh5vTLpkvBv3vWJqZS7JT8ITLf4DOPXfnrHhsTA9TGYGN+QDGkr5JU/oAspsW4RvRGYSMxSklVW/xak0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=y/7Xs+Gq; arc=none smtp.client-ip=209.85.221.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="y/7Xs+Gq" Received: by mail-wr1-f52.google.com with SMTP id ffacd0b85a97d-439d8df7620so272814f8f.0 for ; Thu, 19 Mar 2026 02:02:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1773910969; x=1774515769; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=kDHkOh/+rJs2mQxrqKz06BUKYb3btvvpq78iMwVqDVQ=; b=y/7Xs+Gq5jYiGf7PC8Q7c7IqOvopdkKNPW+yGjax4cPo89Y7ve1SxUce+lG2Tt2ZAn hnrPfksmMuunFwNMGvIL0G5+6lyFgR9VDaGKJjFjDTVNXdD9PJABXWgblFSQmGnWuPKX BTGIODYhzTWlvhig+pZE7DAm2PHaTiQoizmHQV0cG6/rSVuyM3Ly/mPuo4fujFaDNp1a CPN+XQw999J8PKb2wCP2JUg2c8mD4UP7s/fVlcZmAl4FnB6yZ3zmg37jsRAbtGpg10ed 7MVTX6KuDpmIzP7KNeBhYY39jBSgwVvcV3oE+MC3k/+FF4TEH1fa6bDTIQDAmy2PYKun nzKw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1773910969; x=1774515769; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=kDHkOh/+rJs2mQxrqKz06BUKYb3btvvpq78iMwVqDVQ=; b=qGBmvUF3khQo/njQheHPUyITN1B5igQI+mwc2lUfAHz8vutfEUvZfy3CQTI6uIwPFn md4QMUh4hI3nWyXX0dKTwrldEW0wOJzvmHBE1aI8W0T0KxyXiGDZssOLt7MC178G9SPI rgdnbiUrdDLxRSQg9YkXxO/Isw2/+MqigXlheM57ApfLmhn4zLCCRWm9RYQ3F0DLfNJe Q5rNm2aE9e4ovX/vLCfV6Vq6qKs5f/TwmzdYgTBlc0XN+hXe2syMYirejlpUTSZEMIaf FZC05djJq6UTjQ0QPGRUtltkzt7/e2I64S+PzVnnb40zbMFkD7FkwDv/xSJq1RkAttAn xIrA== X-Forwarded-Encrypted: i=1; AJvYcCVppAuBq/C71diD82t8Eq4Mh7LSQPAAUjZBJxUPRc8Qq+LvhB4UbDojqZfoao2tV73T5uJuElPslvTgXKo=@vger.kernel.org X-Gm-Message-State: AOJu0YylSxc0rdF5NdJ+P713ttSERTfGVpAOECtMXzezSxmBsWxX35nw WnEZqNnCQ+kNbIYgE2ECO+673tG8AJ+60Iz0AkTv3OmgnfUqhNtHOtnI/vaG/Oaa0uPhINK2guA 7yxkP X-Gm-Gg: ATEYQzy4Mqpa4LnvFcJgV12ID5zI0tOS6eib3P6IleqkGO+kWlhOAsW/dBjYW4RHxmg 60Nm+WnPSvIm3czB+2BIR6ndj9zT0dla5U6rYkGLt6ueupouq2JLfL44OlRN+2p6I101PLcBr20 WHQiHQxtVlahd4w53iI6VKDgCBuHttRnE5b9CBaMt/8B3O/Rvcl0w1dSziEQDuOogwp47iDW/jN azh25azqzUxJjsuIyLxBUwtitHPb/nfrQvQavbZyGFqaMAnhq+dPWd3QyEXCumFHgS8KzYTH3Rq QmCKzihZMaPbqOa7Suhducd/zGDekkn6bo5mFq1jSaAeUVuvbzDbJzlhxBJrHORHpvvHuwlB4nj bBSYUKs+04IJkkmOUPqi1C8vooshocS8yDnt+QoNMtt/A7D7yV4mLPcl/8EYGl+plByM+TmjhFj lDT6wFCH0H5Oys1ADZDjb9wdNfwMYU X-Received: by 2002:a05:6000:2006:b0:439:b629:42d7 with SMTP id ffacd0b85a97d-43b527c6c7emr11231724f8f.46.1773910968290; Thu, 19 Mar 2026 02:02:48 -0700 (PDT) Received: from localhost ([196.207.164.177]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-43b5184b8b9sm15002809f8f.1.2026.03.19.02.02.46 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 19 Mar 2026 02:02:47 -0700 (PDT) Date: Thu, 19 Mar 2026 12:02:44 +0300 From: Dan Carpenter To: Sudarshan Srinivasan Cc: mchehab@kernel.org, gregkh@linuxfoundation.org, linux-media@vger.kernel.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: [PATCH] staging: media: av7110: coding style fixes: too many tabs Message-ID: References: <20260318191149.109616-1-ss22.kern.dev@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=us-ascii Content-Disposition: inline In-Reply-To: <20260318191149.109616-1-ss22.kern.dev@gmail.com> On Thu, Mar 19, 2026 at 12:41:47AM +0530, Sudarshan Srinivasan wrote: > This patch fixes below warning reported by checkpatch.pl > WARNING: Too many leading tabs - consider code refactoring > > Signed-off-by: Sudarshan Srinivasan > --- > drivers/staging/media/av7110/av7110.c | 32 ++++++++++++++++++--------- > 1 file changed, 22 insertions(+), 10 deletions(-) > > diff --git a/drivers/staging/media/av7110/av7110.c b/drivers/staging/media/av7110/av7110.c > index 607992100baf..970343c4ec51 100644 > --- a/drivers/staging/media/av7110/av7110.c > +++ b/drivers/staging/media/av7110/av7110.c > @@ -932,6 +932,27 @@ static int dvb_feed_stop_pid(struct dvb_demux_feed *dvbdmxfeed) > return ret; > } > > +static int handle_ts_memory_fe(struct dvb_demux_feed *feed, > + struct dvb_demux *demux, > + struct av7110 *av7110) > +{ > + int ret = 0; > + > + if (feed->pes_type >= 2) > + return 0; > + > + if ((demux->pids[0] & 0x8000) || (demux->pids[1] & 0x8000)) > + return 0; > + > + dvb_ringbuffer_flush_spinlock_wakeup(&av7110->avout); > + dvb_ringbuffer_flush_spinlock_wakeup(&av7110->aout); > + > + ret = av7110_av_start_play(av7110, RP_AV); > + demux->playing = (ret) ? true : false; Keep this as: if (!ret) demux->playing = true; 1: It matches the original code. Don't make unnecessary style changes. 2: The original better style. The old code said, if there were no errors set playing to true. The new code is ambiguous. You might think ret is a bool or something. 3: How do we know that the playing wasn't true at the start? This could potentially introduce a behavior change. That wasn't mentioned in the commit message. > + > + return ret; > +} regards, dan carpenter