From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ot1-f51.google.com (mail-ot1-f51.google.com [209.85.210.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 77542405C21 for ; Mon, 20 Jul 2026 11:49:27 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.51 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784548169; cv=none; b=iTVzNqibqWWvaDF04Ye+5Gqc4YywhZA3oTwfbnQCH1TeDKa14LEbSZdhjYuH/g0euydZ2PgmlKAEdA6nwl5u2vIfsZusPijBrYHU5iU4DTHpy+uDsw7vHjAvlYUrz8vQM3Rp5/pDHs1RAqSrgDFPv3BLg0FzpeXj4vdA499y8lU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784548169; c=relaxed/simple; bh=kpQgwtqvNqqfcKxqM9LsZoSGqgZW8nCIf1+4VaTHB2c=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=cppUtuP/Y3/pTJUKqoNQLxtXopTBZD/2rakJzArL7czbHsTgJuh11+R+U17jajoNrdB29gjvU6WcfUWVHwkwvuFddt2tG1ocGbUWhRZuEQQsNs1SZ7KJJDvqEtVu314xOhfbPjEoupGSkK6FHAddZD8eGS6lZW/FBwY4rhKBzxM= 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=DHuoHJ5g; arc=none smtp.client-ip=209.85.210.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="DHuoHJ5g" Received: by mail-ot1-f51.google.com with SMTP id 46e09a7af769-7e9ecb1e13bso3669555a34.2 for ; Mon, 20 Jul 2026 04:49:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1784548166; x=1785152966; darn=vger.kernel.org; h=in-reply-to:content-disposition:content-type:mime-version :references:message-id:subject:cc:to:from:date:from:to:cc:subject :date:message-id:reply-to:content-type; bh=FWzkp2Wnqm0l9stmCGpc16u0gXSQJW1RvOaJ/1MTuh8=; b=DHuoHJ5gX2m2cmkhFxdRAmTK68daXhmBpsQAaN83hJD/tdpiI5XM+vJMFs8MKuo3x6 o5/JOaIaxgIfHS5Deyf1pLrbP7HMkV54G1gyFpe45frkXvlFkoyNfF77dggt95oYSwp0 v65VkkDqHa1YVFy9gQZB6zTvLxT9R8Hl9zNElXhXFK1UmBNydNKJxtpSp05PTCmcIndN tQKYsZCNTqHRsbmicf4qBIjbDekeo1f9D3JnSlEJ74lrahZFsMDYG+fRwngAi7OLjCau /rfOSBmO+QhQbidkO+jjWpLLQyrQ6WbxSay51+ZUSZIX9g/1z3rXjyUvdGvqSdxYcsdz eLJg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1784548166; x=1785152966; h=in-reply-to:content-disposition:content-type: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 :content-type; bh=FWzkp2Wnqm0l9stmCGpc16u0gXSQJW1RvOaJ/1MTuh8=; b=ZjboRJ8K1PnTepVZ4r4/wyMke8mxFkM4OG4jRBKGyyZIRb9V4DhgfW17spvB/bAU+9 cmA0dubgYejCr7LI1PCGSymkojHQibeUrZJuzYdP5J/YipiiXNMQH3NrMyAaGMZaa8SM t3ArAXvpwAq6BL/XCJpfJo8p2eUaW7xFGLJciJxK0xZvWkf8eGk80LREsqTPiuZS/1Ib xrsU2zlXml3XufdT53M31v1firV3qocvhjlIK0DT6IgKaprYHL8ZPLHE+hXkwy3kd+2g F40ux519mkoEVUoN5HUvexitHZWMVlrZSVSZ7nGJqoodWiZi8veZWqLERbTGMxO6isC/ A9zQ== X-Forwarded-Encrypted: i=1; AHgh+RqMK025NV/DscFOgvsIsIJyir83yn6kzoo10/4syIA4pqyJk5crtFvHMhTOZnFMun0NBbAiPNTbk69fsCI=@vger.kernel.org X-Gm-Message-State: AOJu0YwdYJE4VXWnA+SrHaY5cQsH+HH1zqP23/pIRetUKitJd3RPb6WN hQSFVbgitGSOZrcZO0M+AO9AK4n9BtiYExkZtyqA4ZEcjwCIQRAOOo4r X-Gm-Gg: AfdE7cly5HjWIuwP5u39hsYXO/sMJjXO9LbbKDPEhpQBWOZQfxWMLBlnyqCmBmy2oh7 0mrOo4oNmgt8j9XjQbglhWCBwUqiEchEyiNW7/Wm0p60cP8mWpsClkpLuns/84V/ykn41/oSvw6 N2orvKdnqzPtzqy1CgfmKTYcoMLNoLg2tFaMA8RXJdBXL0EnKi1Zkn9ePEGulYuOzTl8KRsmC3q xymS19wW7cO1jeUrkqjwFkNEOSeyMAjnsWdA8dHj4kG/hboZf0ljaEu59C+DOZ/SLXRRH2pAHfL i4ipX381nq4H73NdMQxMAnUDDHdmCXMPGYvG8Fdbj6zW9TV9Y29FqtML3/E7jbOGfWeaPlpY7RW ZK2FRRPoisKYRfPYemZp1XJjtezaQwiNL3CjiXZFrgPYn/bOEbX3caca1zNmmN9lP6AnWXg== X-Received: by 2002:a05:6830:4412:b0:7d7:ea9f:c0f9 with SMTP id 46e09a7af769-7ed9a604debmr6712118a34.0.1784548166307; Mon, 20 Jul 2026 04:49:26 -0700 (PDT) Received: from localhost ([74.80.182.90]) by smtp.gmail.com with ESMTPSA id 46e09a7af769-7edad94e86csm8007715a34.7.2026.07.20.04.49.23 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 20 Jul 2026 04:49:25 -0700 (PDT) Date: Mon, 20 Jul 2026 14:49:19 +0300 From: Dan Carpenter To: Prajval Chaudhary Cc: andy@kernel.org, hansg@kernel.org, mchehab@kernel.org, sakari.ailus@linux.intel.com, gregkh@linuxfoundation.org, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, linux-staging@lists.linux.dev Subject: Re: [PATCH v3] staging: media: atomisp: Remove pad_w and pad_h module parameters Message-ID: References: <20260720112807.6348-1-prajvalchaudhari733@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: <20260720112807.6348-1-prajvalchaudhari733@gmail.com> On Mon, Jul 20, 2026 at 04:58:07PM +0530, Prajval Chaudhary wrote: > Hi Andy, Dan, Hans, > > Thank you for the review and catching this. > > In this v3, I have completely dropped the changes to the debug > parameters to keep the staging debug facilities fully intact as requested. > > This patch now strictly focuses on removing the 'pad_w' and 'pad_h' > module parameters and converting them into internal driver constants. > > Remove the 'pad_w' and 'pad_h' module parameters to prevent userspace > from overriding internal ISP hardware padding boundaries. Convert them > into internal driver constants instead. > > This partially addresses the 'Remove unnecessary/unwanted module > parameters' task in the driver's TODO file, while keeping the debug > facilities intact as requested by maintainers. > > Signed-off-by: Prajval Chaudhary > --- No. The real answer requires is a bit complicated and requires some thought. regards, dan carpenter