RE: [PATCH 03/28] drm/writeback: Define function to get drm_connector from writeback

2025-08-01 Thread Kandpal, Suraj
> Subject: Re: [PATCH 03/28] drm/writeback: Define function to get > drm_connector from writeback > > > > > > > > > > > > Now that drm_connector may not always be embedded within > > > > > > drm_writeback_connector, let's define a

Re: [PATCH 03/28] drm/writeback: Define function to get drm_connector from writeback

2025-08-01 Thread Jani Nikula
On Fri, 01 Aug 2025, Dmitry Baryshkov wrote: > Yes. Basically, that's why I suggest refacoring drm_writeback_connector > to mvoe drm_connector_writeback into drm_connector itself (like it's done > for drm_connector_hdmi). Ah, sorry, I think I missed that portion in my post-vacation email catch-up

Re: [PATCH 03/28] drm/writeback: Define function to get drm_connector from writeback

2025-08-01 Thread Dmitry Baryshkov
On Fri, Aug 01, 2025 at 02:57:48PM +0300, Jani Nikula wrote: > On Fri, 01 Aug 2025, Dmitry Baryshkov > wrote: > > Thinking in OOP terms, the encoder is just a field in the struct. > > drm_connector is a base class for drm_writeback_connector. By making it > > optional, you are definitely semi-bre

Re: [PATCH 03/28] drm/writeback: Define function to get drm_connector from writeback

2025-08-01 Thread Jani Nikula
On Fri, 01 Aug 2025, Dmitry Baryshkov wrote: > Thinking in OOP terms, the encoder is just a field in the struct. > drm_connector is a base class for drm_writeback_connector. By making it > optional, you are definitely semi-breaking the abstraction. The trouble is, in OOP terms, drm_connector is t

Re: [PATCH 03/28] drm/writeback: Define function to get drm_connector from writeback

2025-08-01 Thread Dmitry Baryshkov
On Fri, Aug 01, 2025 at 05:18:47AM +, Kandpal, Suraj wrote: > > Please tune your mail client to insert smaller quotation headers. This is > > just > > useless. > > > > > > > > > > > Now that drm_connector may not always be embedded within > > > > > drm_writeback_connector, let's define a func

RE: [PATCH 03/28] drm/writeback: Define function to get drm_connector from writeback

2025-07-31 Thread Kandpal, Suraj
> Please tune your mail client to insert smaller quotation headers. This is just > useless. > > > > > > > > Now that drm_connector may not always be embedded within > > > > drm_writeback_connector, let's define a function which either uses > > > > the writeback helper hook that returns the drm_con

Re: [PATCH 03/28] drm/writeback: Define function to get drm_connector from writeback

2025-07-27 Thread Dmitry Baryshkov
@lists.freedesktop.org; intel- > > g...@lists.freedesktop.org; Nautiyal, Ankit K ; > > Murthy, Arun R ; Shankar, Uma > > > > Subject: Re: [PATCH 03/28] drm/writeback: Define function to get > > drm_connector from writeback Please tune your mail client to insert smaller quotation head

RE: [PATCH 03/28] drm/writeback: Define function to get drm_connector from writeback

2025-07-26 Thread Kandpal, Suraj
Uma > > Subject: Re: [PATCH 03/28] drm/writeback: Define function to get > drm_connector from writeback > > On Fri, Jul 25, 2025 at 10:33:44AM +0530, Suraj Kandpal wrote: > > Now that drm_connector may not always be embedded within > > drm_writeback_connector, let's

Re: [PATCH 03/28] drm/writeback: Define function to get drm_connector from writeback

2025-07-26 Thread Dmitry Baryshkov
On Fri, Jul 25, 2025 at 10:33:44AM +0530, Suraj Kandpal wrote: > Now that drm_connector may not always be embedded within > drm_writeback_connector, let's define a function which either uses > the writeback helper hook that returns the drm_connector associated > with the drm_writeback_connector or

[PATCH 03/28] drm/writeback: Define function to get drm_connector from writeback

2025-07-24 Thread Suraj Kandpal
Now that drm_connector may not always be embedded within drm_writeback_connector, let's define a function which either uses the writeback helper hook that returns the drm_connector associated with the drm_writeback_connector or just return the drm_connector embedded inside drm_writeback_connector i