Received-SPF: pass (google.com: domain of linux-kernel+bounces-73598-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Date: Tue, 20 Feb 2024 14:32:27 -0500
From: Rodrigo Vivi <rodrigo.vivi@intel.com>
To: Zhi Wang <zhi.wang.linux@gmail.com>
CC: Jani Nikula <jani.nikula@linux.intel.com>, "Jiri Slaby (SUSE)"
	<jirislaby@kernel.org>, <linux-kernel@vger.kernel.org>,
	<intel-gfx@lists.freedesktop.org>, Joonas Lahtinen
	<joonas.lahtinen@linux.intel.com>, Tvrtko Ursulin
	<tvrtko.ursulin@linux.intel.com>, Zhenyu Wang <zhenyuw@linux.intel.com>,
	<intel-gvt-dev@lists.freedesktop.org>
Subject: Re: [PATCH 00/21] drm/i915: remove unused structure members
Message-ID: <ZdT-S7UlgZC-8Rp2@intel.com>
References: <20240216065326.6910-1-jirislaby@kernel.org>
 <87plww4ws7.fsf@intel.com>
 <20240216144026.00005144.zhi.wang.linux@gmail.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20240216144026.00005144.zhi.wang.linux@gmail.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?vx2Bbvs5A/HEkWgDcMUG1/oxBqUZQAAxADCuDJXmXd05qH3Boi4UW3SiKLHd?=
 =?us-ascii?Q?RQmVVpyQG0LSe/XiR8RNdddk9WxDxN6K2TjnmP4340Pue/dnAQ6MozBjNpwR?=
 =?us-ascii?Q?012JCdOJyA/9AUI2nep7Ef9mANc+9jdTaPp3cNcAcjRKCtA1oUnyQ+Gi6qrx?=
 =?us-ascii?Q?QGSKSppdQRazycacyREodIFu2ZsP7aEgpVLFZadlcmymnD0Q7aIAHeHN9YtC?=
 =?us-ascii?Q?1HcAeOXPYyer1b6Zf97cFc7ZV4j6vON6ba1q7TPnR0smkIxr5EFuCWKHmxm0?=
 =?us-ascii?Q?Jdh3lyTNpoO5eFBihHKJ7He4gmQzoHRp0NsSmy9JYAqQiKNN/SgJ8phg3AAz?=
 =?us-ascii?Q?eXyo92py9/aHE69B6yxmNio3DoZ8Ke3mAOWeLr4OpQTR2O66aAzCa7c/IqeC?=
 =?us-ascii?Q?PYL29j50OZ3bAiLT+9r75NB4QpMb0euyR/FtLc11tQ945pO4Fh24OZGBlE/r?=
 =?us-ascii?Q?nXVkYh2o43s/hmAYR3wpfvWKbPkLXAC+JQ8M/WCObCIKIGgUrHASpND2Nv85?=
 =?us-ascii?Q?9bFNVUqYpiTy5Yh4hMRFdFZnEd9pm//Q14Gk1rh1KV7wF9GIEtsGIpWFSqWU?=
 =?us-ascii?Q?FBqi2DLcOIGR+LOENusqowob8ZD3jTLIO6CG1b+rUkeJ4LRPrF0M84qs4yVW?=
 =?us-ascii?Q?XB1+om0iZjcLKbmhOjDpMFzXRmC4VxKwvxGauJsYQnJnSVrw6L3uBR94y9Ux?=
 =?us-ascii?Q?iXeM470n2Cl3DvHPcTplsoA8bafFG+ehwvVLhdpsq+R7jXG/GabnMH80Xqd5?=
 =?us-ascii?Q?wLIIyH3+kXtjb19fFayLCImyvAV2K/ayfBGHWdl53T2PhfD18QZFlFS4vM+J?=
 =?us-ascii?Q?Xxr1+UCHNIjiYrV2JaW0Y2RVTtV6wRlgxOqjjakB4rg9b2/ZZyK7uD2hSYFQ?=
 =?us-ascii?Q?N0pYO0Rde5gS7Xn844iU+Pc4I0kgcTZyOypd+QB4ZWnaCoDGt4SDJF8K2XxP?=
 =?us-ascii?Q?nCLFdXxw40/bTh3AlmoDnlQj9KMzn6+UJc2UaAcqPDLDYLPLuAevO2Zz/heR?=
 =?us-ascii?Q?VSAVa5hxg1Nfp6V+oVrHk8f50NPGTYWsOm6yt6quPBiauPqncOWSEH+lqP31?=
 =?us-ascii?Q?851ge7N+rpdArjIPrTmhnyVwDhizZuhUmQcQKsAxNA6veiPdPaRk4ZMG1Zp3?=
 =?us-ascii?Q?BqseYbgoRZcR3RJ2gBlLu2DxabjkD+hBZbLoYlgAna0Ie09YKKkWd2WyX5bX?=
 =?us-ascii?Q?nG9VZ2AwjIblsBAh5a4lOXaBhdD5ZkANpOdqqotw7nvosvbHRnpq+wvTWUHs?=
 =?us-ascii?Q?FikvleVrvwtrXg3gouFLhDn13k6rF/TW2YyZICW1BFXzRROUJD6lSVFyjUpX?=
 =?us-ascii?Q?hgubCShg3cnx/EoR9QglgrioZx3uXiKvdpCKRP3f/+y1yWghtJxxMjffrujC?=
 =?us-ascii?Q?O3ORFa86k4MPEkJ0ysYQEdt2dhYC2LebeFujNYpNoSTVQgatTRF31o6/aZQd?=
 =?us-ascii?Q?uRhpKPIZeCiBtG6ECw3RoZIVDbE+4yS6TTetou18n0TezVPA1EN5iLgflGqz?=
 =?us-ascii?Q?DvxClTJxosQOAhEj17rLbUiFzZNDBNc/nP/fHG2mWJy+FblxtfDOyi94wmYu?=
 =?us-ascii?Q?Z5AlmX1rewXUmYPO8NBP3Se87V0C1P0hpYMbmPZk?=
X-MS-Exchange-CrossTenant-Network-Message-Id: cfacf2c6-0d2e-48d2-ae7d-08dc324aaed3
X-MS-Exchange-CrossTenant-AuthSource: MN0PR11MB6059.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Feb 2024 19:32:32.4332
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: yFrJlCrK5Gw8FhwKFkrPZ+ybSJoRLiSKHTGzGjVT6qvoXN7gB5oYJhhS7EWBOYX8UgkVR6pQBYW8sE8/T7W0bQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR11MB6761
X-OriginatorOrg: intel.com

On Fri, Feb 16, 2024 at 02:40:26PM +0200, Zhi Wang wrote:
> On Fri, 16 Feb 2024 10:51:20 +0200
> Jani Nikula <jani.nikula@linux.intel.com> wrote:
> 
> > On Fri, 16 Feb 2024, "Jiri Slaby (SUSE)" <jirislaby@kernel.org> wrote:
> > > this series removes unused i915 structure members as found by
> > > clang-struct (and manually checked by me).
> > 
> > Thanks Jiri, good stuff!
> > 
> > Acked-by: Jani Nikula <jani.nikula@intel.com>
> > 
> > However, you may have overlooked that drivers/gpu/drm/i915/gvt/ is
> > maintained separately.
> > 
> > Cc: Zhenyu, Zhi, how do you want the gvt patches in this series
> > handled?
> > 
> 
> Many thanks for the clean-up patch Jiri! Jani, it would be easier
> for us that you can help to apply the patches through i915.

I pushed the entire series through drm-intel-next.
Thanks for the patches and acks.

> 
> Thanks,
> Zhi.
> 
> > 
> > BR,
> > Jani.
> > 
> > 
> > >
> > > Cc: intel-gfx@lists.freedesktop.org
> > > Cc: Jani Nikula <jani.nikula@linux.intel.com>
> > > Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
> > > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > > Cc: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
> > >
> > > Jiri Slaby (SUSE) (21):
> > >   drm/i915: remove unused intel_dvo_dev_ops hooks
> > >   drm/i915: remove structs intel_vgpu_pipe_format and
> > >     intel_vgpu_fb_format
> > >   drm/i915: remove intel_dsi::{port_bits,hs}
> > >   drm/i915: remove
> > >     intel_gvt_gtt::{mm_alloc_page_table,mm_free_page_table}
> > >   drm/i915: remove intel_gvt_mmio_info::{device,addr_range}
> > >   drm/i915: remove
> > > intel_vgpu_workload::{ring_context,restore_inhibit} drm/i915:
> > > remove intel_vbt_panel_data::edp::initialized drm/i915: remove
> > > intel_guc::ads_engine_usage_size drm/i915: remove
> > > i915_drm_client::id drm/i915: remove i915_perf_stream::size_exponent
> > >   drm/i915: remove intel_vgpu_gtt::active_ppgtt_mm_bitmap
> > >   drm/i915: remove intel_vgpu_fence::base
> > >   drm/i915: remove intel_vgpu_opregion::mapped
> > >   drm/i915: remove intel_vgpu::intx_trigger
> > >   drm/i915: remove gvt_mmio_block::device
> > >   drm/i915: remove intel_gvt_irq_info::warned
> > >   drm/i915: remove intel_gvt_event_info::policy
> > >   drm/i915: remove intel_gvt_irq::pending_events
> > >   drm/i915: remove execute_cb::signal
> > >   drm/i915: remove i915_vma::obj_hash
> > >   drm/i915: remove intel_memory_region_ops::flags
> > >
> > >  .../drm/i915/display/intel_display_types.h    |  1 -
> > >  drivers/gpu/drm/i915/display/intel_dsi.h      |  4 ---
> > >  drivers/gpu/drm/i915/display/intel_dvo_dev.h  | 25
> > > ------------------- drivers/gpu/drm/i915/gt/uc/intel_guc.h        |
> > >  2 -- drivers/gpu/drm/i915/gvt/fb_decoder.h         | 11 --------
> > >  drivers/gpu/drm/i915/gvt/gtt.h                |  3 ---
> > >  drivers/gpu/drm/i915/gvt/gvt.h                |  5 ----
> > >  drivers/gpu/drm/i915/gvt/interrupt.c          |  1 -
> > >  drivers/gpu/drm/i915/gvt/interrupt.h          |  2 --
> > >  drivers/gpu/drm/i915/gvt/mmio.h               |  2 --
> > >  drivers/gpu/drm/i915/gvt/scheduler.h          |  2 --
> > >  drivers/gpu/drm/i915/i915_drm_client.h        |  2 --
> > >  drivers/gpu/drm/i915/i915_perf_types.h        |  1 -
> > >  drivers/gpu/drm/i915/i915_request.c           |  1 -
> > >  drivers/gpu/drm/i915/i915_vma_types.h         |  1 -
> > >  drivers/gpu/drm/i915/intel_memory_region.h    |  2 --
> > >  16 files changed, 65 deletions(-)
> > 
>