From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.10]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6F0961E1A3D; Thu, 3 Sep 2026 04:38:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=192.198.163.10 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788410327; cv=none; b=gcLmrA94QhXT27x6okFOI8RW7eqQYoc27bC0qETCgBLwxJdxzcjK5eKlICqtwn9s+qiRQnChAVBnAKzPF8bFqi/8IoZZxZlWY67ciRS8xl7vyIqHbGiBV0OlySer4mY7IBqWtFuAVJ+QtEGsCnwiDCPc5rqFYY2m6avgRBJqdeg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788410327; c=relaxed/simple; bh=5HcDPVvfU/QGdKBl23/4a6+keo0j28eZUB+Jw9j5WWM=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=P6JXs0LoLGTlNimtrQWUj2BwWjdFvvMJUprCI4mOHLzeRT1RAhDN2ffmorl/x9EjRPg3gv5Izc2rCe8nPTLzevJ/0mYa28htYOers78HgXOCGkAQBI1SGwJvOnKwIG3ZtrSiXH8giSE98+PXuO/QXVfODcnXcupBp6NPueHIMW0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=SEK2SxIL; arc=none smtp.client-ip=192.198.163.10 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="SEK2SxIL" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1788410326; x=1819946326; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=5HcDPVvfU/QGdKBl23/4a6+keo0j28eZUB+Jw9j5WWM=; b=SEK2SxILrfR0a0clhO9gdAaAivjtRSijj5s/MTZBUPSTpH8ZB+xW2cBN FDzej2suxx/5L08Z0fBF+hEyZwHte2TPDjRKib2k1jgkGpMw+RWETVowU D0S+sedL/UAmiGWjVozIaNrNZ9nIRZP6Cl1HcWffqEu46bduJSIWwDLD0 DGfAzvvaQnn7fwx4nJmBoVp4T3D0iIoSFevKACfCPzsqHfvRyZYqCIa4t hFrKJpqK1yGPn0WGdvWB1cTzmlvahqaswpiUoZFKCo880QAxydRmJOzpk vEP1tZQJtSwG3Bg5J4+ddHbSjMt1tYFTdS0MWUNbM8Z7sivPkjAt+tFvb g==; X-CSE-ConnectionGUID: v5hraVgcTtiytRi0Y4n2xQ== X-CSE-MsgGUID: jpcyvTLlTtGv4CapwyMSmg== X-IronPort-AV: E=McAfee;i="6800,10657,11894"; a="100236626" X-IronPort-AV: E=Sophos;i="6.25,258,1779174000"; d="scan'208";a="100236626" Received: from orviesa003.jf.intel.com ([10.64.159.143]) by fmvoesa104.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Sep 2026 21:38:45 -0700 X-CSE-ConnectionGUID: on50LPqaTlWvRvH42F2lxg== X-CSE-MsgGUID: TGzr+IolSGWZxWN/iANwOg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.25,258,1779174000"; d="scan'208";a="273103789" Received: from smoticic-mobl1.ger.corp.intel.com (HELO localhost) ([10.245.244.28]) by ORVIESA003-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Sep 2026 21:38:41 -0700 Date: Thu, 3 Sep 2026 07:38:39 +0300 From: Andy Shevchenko To: Maurizio Casciano Cc: mchehab@kernel.org, linux-media@vger.kernel.org, sakari.ailus@linux.intel.com, bingbu.cao@amd.com, jacopo.mondi@ideasonboard.com, nicholas@rothemail.net, andy@kernel.org, hansg@kernel.org, gregkh@linuxfoundation.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: [PATCH v6 14/16] media: atomisp: Use struct v4l2_area for padding Message-ID: References: Precedence: bulk X-Mailing-List: linux-media@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: Organization: Intel Finland Oy - BIC 0357606-4 - c/o Alberga Business Park, 6 krs, Bertel Jungin Aukio 5, 02600 Espoo On Tue, Sep 01, 2026 at 09:04:36PM +0200, Maurizio Casciano wrote: > The padding helper passes width and height as four separate scalar > arguments even though they form two logical dimensions. > > Pass the requested size and returned padding as struct v4l2_area values. > This makes the dimensions explicit and simplifies all three callers. Reviewed-by: Andy Shevchenko ... > dev_dbg(isp->dev, "s_mbus_fmt: ask %ux%u (padding %ux%u, dvs %ux%u)\n", > - ffmt.width, ffmt.height, asd->sink_pad_padding_w, asd->sink_pad_padding_h, > + ffmt.width, ffmt.height, asd->sink_pad_padding.width, > + asd->sink_pad_padding.height, Split logically ffmt.width, ffmt.height, asd->sink_pad_padding.width, asd->sink_pad_padding.height, > dvs_env_w, dvs_env_h); -- With Best Regards, Andy Shevchenko