From nobody Wed Feb 5 18:49:22 2025 Received: from mail-ed1-f98.google.com (mail-ed1-f98.google.com [209.85.208.98]) (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 E4BA11F37D2 for ; Thu, 16 Jan 2025 13:07:24 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.98 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1737032847; cv=none; b=sGTffoXq6DP9ddivwpuvQCCJH9ZSPuVrNOReDj4S37nBwH86rxaXGXb33ak7ZGxksq9m2CXic9PFP/KJV/6WgD7H+9Hj38H6cs047WEvEwk44PrXq0OTfmfPeq79D406AtCIb/jIFkZzd7v2v3Jzf3Q5/5mOkWbg1gnLwEV/xbE= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1737032847; c=relaxed/simple; bh=NLVUtRDWIyErhGq1btJXLWXuuVShqXuFKgAtujLSvrU=; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version; b=q7SWsZS0jtyX1Y/Qrk6t8yG9HXRd6iMT+zIW6Nal7dDJ5n0A0zvcflyEhmQH9v8RCNwS6Wi2iPtO2JVNwaV8CNi3fDeOLIa64cdHu5X+Etpudv09J4h16ImILX/05AoABxv07iQIYyECDjy7vEgM8edJ7xgCbZyqNOmCrk2hx68= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=raspberrypi.com; spf=fail smtp.mailfrom=raspberrypi.com; dkim=pass (2048-bit key) header.d=raspberrypi.com header.i=@raspberrypi.com header.b=n62c+lQH; arc=none smtp.client-ip=209.85.208.98 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=raspberrypi.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=raspberrypi.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=raspberrypi.com header.i=@raspberrypi.com header.b="n62c+lQH" Received: by mail-ed1-f98.google.com with SMTP id 4fb4d7f45d1cf-5da135d3162so1477042a12.3 for ; Thu, 16 Jan 2025 05:07:24 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=raspberrypi.com; s=google; t=1737032843; x=1737637643; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=7P+v3T+nrQTMibupPu9ktHQrDK2ERVyoiAaGUmlwr34=; b=n62c+lQH0MrrRQQFas4C5JMk8qJxJNQsTb29k1yJbyez43m40/zCmKip9Z52CPoh06 Y6hbrDbp1CT+nKJrIk5Pnp+XKVOPH0/wEPo3PfGFAhGpFjFnNVjPH+QYcBbSsvnEhUzz ZXSTuSQ5dfaEbVW348F/uJzt88TmAeB1BD37VDdDftvX6SyAcGFVHPlM30xXdASjh6hi L7Pe78yEvFkzxCr2ngEFqucOtGNAFiQq9ygFuyHRiJ8YLMFsoyuu/jMUrGuEmzAVYb1W O011O931wybJxn5Q3xGuPo24E94esoztJdwqd7pc5DJp0Q+fK8nWK4v2xcJLdJfLHrOp lClQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1737032843; x=1737637643; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=7P+v3T+nrQTMibupPu9ktHQrDK2ERVyoiAaGUmlwr34=; b=NlE5XKepRxyQLaUJiDIvKPbTi6dFOeClZYD3BGvVraltVd251RKLxkxuQx138lZWjz X2pEtsLPBpheO7mai8dJI0fLzar9N2J/12QfkCiJ88RuJKbM2FTqaOfcf+aI5YOeh1w9 7lB8Bu4sIxmvQBUcxrSynWqv4ZllK05XdMfi1s/sfpfl7QqsniSmsmsGMI2BXFanp8pf oVR1ClKXgSq6XOl9KgHyVjzvse29bh7Vbywej0fQyiIH13PKv0h6scYTOmY4G0Y7C6nL 56yNq3Y8E9kkK1mZpG3uzsLQV3HdPUgygNsImhEjVlVzOuXlxbZCL0gwJCT0soM7/Ved gJEQ== X-Forwarded-Encrypted: i=1; AJvYcCWDTV82nXYf5ygtwKdncyPPTqWedZz9udwDWS5MRUiijkpd7Un39i3Tg+mHzzrFJG5g3CJUU1iEIOk6ptA=@vger.kernel.org X-Gm-Message-State: AOJu0YwN1CY7Iy+SUz893l/r2vKIHJKeyaU5hrVl4mjtxXn4Wkso0i99 6FWbBYrpZcqAy3RSrhE2VrUT0X809kQQ2h4cP3DHvY3MBHf8GCc0nKnBDEb3P25v0YKHdbKDWU1 sddnc4JkAKkrZ1qXL6WPlgFk86p6oqFrX X-Gm-Gg: ASbGncsseRDcXAeTAxRxyrnQWRL5o88uOSa2TInjzxlfPBucid2e3NAV0sX0BmyDesg L5gxVTET/mVEbuEJeBWVQ62O01hHD17i6MMelKxIVfm2+4ZEKlcQ5r1hsO0irGnVi3M8cdEfTcf EB3a9cQf3W9Mc7ePn1UDL/qty2vGG7FApWtJqGoAvfciorlJ8x/Pn32ytRTCBvuOgTsX7Rh1fEv AtPPF8fm47L60zFU+AW+EnbLJ+6UJ6XbiOklVSH6YwHx8k4XLqLiQsnWE6FF2jE9ylOLF2E X-Google-Smtp-Source: AGHT+IGViDPhV57wZ6b9jVwuuHeCZhjQncFp1lN6cuhdiqlXIycDF1nRHxi+kNX1Yeo7TN1wCIAeIuZIX8jO X-Received: by 2002:a05:6402:3514:b0:5da:105b:86c2 with SMTP id 4fb4d7f45d1cf-5da105b87d5mr4665682a12.20.1737032843068; Thu, 16 Jan 2025 05:07:23 -0800 (PST) Received: from raspberrypi.com ([93.93.133.154]) by smtp-relay.gmail.com with ESMTPS id 4fb4d7f45d1cf-5d99efc9b94sm660964a12.28.2025.01.16.05.07.22 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 16 Jan 2025 05:07:23 -0800 (PST) X-Relaying-Domain: raspberrypi.com From: Naushir Patuck To: Raspberry Pi Kernel Maintenance , Mauro Carvalho Chehab , Florian Fainelli , Broadcom internal kernel review list , Ray Jui , Scott Branden Cc: linux-media@vger.kernel.org, linux-rpi-kernel@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, jacopo.mondi@ideasonboard.com, Dave Stevenson , Naushir Patuck Subject: [PATCH v1] media: uapi: pisp_be_config: Remove unused config fields Date: Thu, 16 Jan 2025 13:07:21 +0000 Message-Id: <20250116130721.3354550-1-naush@raspberrypi.com> X-Mailer: git-send-email 2.34.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" These fields should not be set by either the user or the kernel driver so remove them. Replace them with padding bytes to maintain uapi compatibility. Signed-off-by: Naushir Patuck --- .../linux/media/raspberrypi/pisp_be_config.h | 42 ++----------------- 1 file changed, 4 insertions(+), 38 deletions(-) diff --git a/include/uapi/linux/media/raspberrypi/pisp_be_config.h b/includ= e/uapi/linux/media/raspberrypi/pisp_be_config.h index cbeb714f4d61..82560db4da61 100644 --- a/include/uapi/linux/media/raspberrypi/pisp_be_config.h +++ b/include/uapi/linux/media/raspberrypi/pisp_be_config.h @@ -716,13 +716,6 @@ struct pisp_be_hog_buffer_config { /** * struct pisp_be_config - RaspberryPi PiSP Back End Processing configurat= ion * - * @input_buffer: Input buffer addresses - * @tdn_input_buffer: TDN input buffer addresses - * @stitch_input_buffer: Stitch input buffer addresses - * @tdn_output_buffer: TDN output buffer addresses - * @stitch_output_buffer: Stitch output buffer addresses - * @output_buffer: Output buffers addresses - * @hog_buffer: HOG buffer addresses * @global: Global PiSP configuration * @input_format: Input image format * @decompress: Decompress configuration @@ -761,28 +754,10 @@ struct pisp_be_hog_buffer_config { * @output_format: Output format configuration * @hog: HOG configuration * @axi: AXI bus configuration - * @lsc_extra: LSC extra info - * @cac_extra: CAC extra info - * @downscale_extra: Downscaler extra info - * @resample_extra: Resample extra info - * @crop: Crop configuration - * @hog_format: HOG format info - * @dirty_flags_bayer: Bayer enable dirty flags - * (:c:type:`pisp_be_bayer_enable`) - * @dirty_flags_rgb: RGB enable dirty flags - * (:c:type:`pisp_be_rgb_enable`) - * @dirty_flags_extra: Extra dirty flags */ struct pisp_be_config { - /* I/O configuration: */ - struct pisp_be_input_buffer_config input_buffer; - struct pisp_be_tdn_input_buffer_config tdn_input_buffer; - struct pisp_be_stitch_input_buffer_config stitch_input_buffer; - struct pisp_be_tdn_output_buffer_config tdn_output_buffer; - struct pisp_be_stitch_output_buffer_config stitch_output_buffer; - struct pisp_be_output_buffer_config - output_buffer[PISP_BACK_END_NUM_OUTPUTS]; - struct pisp_be_hog_buffer_config hog_buffer; + /* For backward compatibility */ + uint8_t pad0[112]; /* Processing configuration: */ struct pisp_be_global_config global; struct pisp_image_format_config input_format; @@ -823,17 +798,8 @@ struct pisp_be_config { output_format[PISP_BACK_END_NUM_OUTPUTS]; struct pisp_be_hog_config hog; struct pisp_be_axi_config axi; - /* Non-register fields: */ - struct pisp_be_lsc_extra lsc_extra; - struct pisp_be_cac_extra cac_extra; - struct pisp_be_downscale_extra - downscale_extra[PISP_BACK_END_NUM_OUTPUTS]; - struct pisp_be_resample_extra resample_extra[PISP_BACK_END_NUM_OUTPUTS]; - struct pisp_be_crop_config crop; - struct pisp_image_format_config hog_format; - __u32 dirty_flags_bayer; /* these use pisp_be_bayer_enable */ - __u32 dirty_flags_rgb; /* use pisp_be_rgb_enable */ - __u32 dirty_flags_extra; /* these use pisp_be_dirty_t */ + /* For backward compatibility */ + uint8_t pad1[84]; } __attribute__((packed)); =20 /** --=20 2.34.1