From: Laurent Pinchart <laurent.pinchart@ideasonboard.com> To: Sam Ravnborg <sam@ravnborg.org> Cc: Venkateshwar Rao Gannavarapu <venkateshwar.rao.gannavarapu@xilinx.com>, airlied@linux.ie, vgannava@xilinx.com, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: Re: [LINUX PATCH 2/2] drm: xlnx: dsi: driver for Xilinx DSI Tx subsystem Date: Mon, 13 Jun 2022 00:28:03 +0300 [thread overview] Message-ID: <YqZaY8Jiprc69TQZ@pendragon.ideasonboard.com> (raw) In-Reply-To: <Yn47YsSH4fn/wjKN@ravnborg.org> Hi Sam, One more comment. On Fri, May 13, 2022 at 01:05:06PM +0200, Sam Ravnborg wrote: > On Thu, May 12, 2022 at 07:23:13PM +0530, Venkateshwar Rao Gannavarapu wrote: > > The Xilinx MIPI DSI Tx Subsystem soft IP is used to display video > > data from AXI-4 stream interface. > > > > It supports upto 4 lanes, optional register interface for the DPHY > > and multiple RGB color formats. > > This is a MIPI-DSI host driver and provides DSI bus for panels. > > This driver also helps to communicate with its panel using panel > > framework. > > Thanks for submitting this driver. I have added a few comments in the > following that I hope you will find useful to improve the driver. > > > Signed-off-by: Venkateshwar Rao Gannavarapu <venkateshwar.rao.gannavarapu@xilinx.com> > > --- > > drivers/gpu/drm/xlnx/Kconfig | 14 ++ > > drivers/gpu/drm/xlnx/Makefile | 1 + > > drivers/gpu/drm/xlnx/xlnx_dsi.c | 456 ++++++++++++++++++++++++++++++++++++++++ > > 3 files changed, 471 insertions(+) > > create mode 100644 drivers/gpu/drm/xlnx/xlnx_dsi.c [snip] > > diff --git a/drivers/gpu/drm/xlnx/xlnx_dsi.c b/drivers/gpu/drm/xlnx/xlnx_dsi.c > > new file mode 100644 > > index 0000000..a5291f3 > > --- /dev/null > > +++ b/drivers/gpu/drm/xlnx/xlnx_dsi.c [snip] > > +static const struct drm_bridge_funcs xlnx_dsi_bridge_funcs = { > > + .mode_set = xlnx_dsi_bridge_mode_set, > > From the documentation of the mode_set operation: > * This is deprecated, do not use! > * New drivers shall set their mode in the > * &drm_bridge_funcs.atomic_enable operation. > > Please adjust accordingly. > > > + .atomic_enable = xlnx_dsi_bridge_enable, > > + .atomic_disable = xlnx_dsi_bridge_disable, > > + .attach = xlnx_dsi_bridge_attach, > > + .detach = xlnx_dsi_bridge_detach, > > +}; > > For a new bridge please implement all the mandatory atomic operations. > > You will need at least: > .atomic_get_output_bus_fmts = xlnx_dsi_bridge_get_output_bus_fmts, As this DSI encoder will never be the last bridge in the chain (there should always be a panel or another bridge afterwards), I think this function can be skipped. > .atomic_get_input_bus_fmts = xlnx_dsi_bridge_get_input_bus_fmts, > .atomic_duplicate_state = drm_atomic_helper_bridge_duplicate_state, > .atomic_destroy_state = drm_atomic_helper_bridge_destroy_state, > .atomic_reset = drm_atomic_helper_bridge_reset, > }; [snip] -- Regards, Laurent Pinchart
WARNING: multiple messages have this Message-ID (diff)
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com> To: Sam Ravnborg <sam@ravnborg.org> Cc: airlied@linux.ie, vgannava@xilinx.com, Venkateshwar Rao Gannavarapu <venkateshwar.rao.gannavarapu@xilinx.com>, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org Subject: Re: [LINUX PATCH 2/2] drm: xlnx: dsi: driver for Xilinx DSI Tx subsystem Date: Mon, 13 Jun 2022 00:28:03 +0300 [thread overview] Message-ID: <YqZaY8Jiprc69TQZ@pendragon.ideasonboard.com> (raw) In-Reply-To: <Yn47YsSH4fn/wjKN@ravnborg.org> Hi Sam, One more comment. On Fri, May 13, 2022 at 01:05:06PM +0200, Sam Ravnborg wrote: > On Thu, May 12, 2022 at 07:23:13PM +0530, Venkateshwar Rao Gannavarapu wrote: > > The Xilinx MIPI DSI Tx Subsystem soft IP is used to display video > > data from AXI-4 stream interface. > > > > It supports upto 4 lanes, optional register interface for the DPHY > > and multiple RGB color formats. > > This is a MIPI-DSI host driver and provides DSI bus for panels. > > This driver also helps to communicate with its panel using panel > > framework. > > Thanks for submitting this driver. I have added a few comments in the > following that I hope you will find useful to improve the driver. > > > Signed-off-by: Venkateshwar Rao Gannavarapu <venkateshwar.rao.gannavarapu@xilinx.com> > > --- > > drivers/gpu/drm/xlnx/Kconfig | 14 ++ > > drivers/gpu/drm/xlnx/Makefile | 1 + > > drivers/gpu/drm/xlnx/xlnx_dsi.c | 456 ++++++++++++++++++++++++++++++++++++++++ > > 3 files changed, 471 insertions(+) > > create mode 100644 drivers/gpu/drm/xlnx/xlnx_dsi.c [snip] > > diff --git a/drivers/gpu/drm/xlnx/xlnx_dsi.c b/drivers/gpu/drm/xlnx/xlnx_dsi.c > > new file mode 100644 > > index 0000000..a5291f3 > > --- /dev/null > > +++ b/drivers/gpu/drm/xlnx/xlnx_dsi.c [snip] > > +static const struct drm_bridge_funcs xlnx_dsi_bridge_funcs = { > > + .mode_set = xlnx_dsi_bridge_mode_set, > > From the documentation of the mode_set operation: > * This is deprecated, do not use! > * New drivers shall set their mode in the > * &drm_bridge_funcs.atomic_enable operation. > > Please adjust accordingly. > > > + .atomic_enable = xlnx_dsi_bridge_enable, > > + .atomic_disable = xlnx_dsi_bridge_disable, > > + .attach = xlnx_dsi_bridge_attach, > > + .detach = xlnx_dsi_bridge_detach, > > +}; > > For a new bridge please implement all the mandatory atomic operations. > > You will need at least: > .atomic_get_output_bus_fmts = xlnx_dsi_bridge_get_output_bus_fmts, As this DSI encoder will never be the last bridge in the chain (there should always be a panel or another bridge afterwards), I think this function can be skipped. > .atomic_get_input_bus_fmts = xlnx_dsi_bridge_get_input_bus_fmts, > .atomic_duplicate_state = drm_atomic_helper_bridge_duplicate_state, > .atomic_destroy_state = drm_atomic_helper_bridge_destroy_state, > .atomic_reset = drm_atomic_helper_bridge_reset, > }; [snip] -- Regards, Laurent Pinchart
next prev parent reply other threads:[~2022-06-12 21:28 UTC|newest] Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-05-12 13:53 [LINUX PATCH 0/2] Add Xilinx DSI-Tx DRM driver Venkateshwar Rao Gannavarapu 2022-05-12 13:53 ` Venkateshwar Rao Gannavarapu 2022-05-12 13:53 ` [LINUX PATCH 1/2] dt-bindings: display: xlnx: Add DSI 2.0 Tx subsystem documentation Venkateshwar Rao Gannavarapu 2022-05-12 13:53 ` Venkateshwar Rao Gannavarapu 2022-05-13 13:39 ` Laurent Pinchart 2022-05-13 13:39 ` Laurent Pinchart 2022-05-12 13:53 ` [LINUX PATCH 2/2] drm: xlnx: dsi: driver for Xilinx DSI Tx subsystem Venkateshwar Rao Gannavarapu 2022-05-12 13:53 ` Venkateshwar Rao Gannavarapu 2022-05-13 11:05 ` Sam Ravnborg 2022-05-13 11:05 ` Sam Ravnborg 2022-06-12 21:21 ` Laurent Pinchart 2022-06-12 21:21 ` Laurent Pinchart 2022-06-13 5:53 ` Sam Ravnborg 2022-06-13 5:53 ` Sam Ravnborg 2022-06-12 21:28 ` Laurent Pinchart [this message] 2022-06-12 21:28 ` Laurent Pinchart 2022-05-13 13:39 ` Laurent Pinchart 2022-05-13 13:39 ` Laurent Pinchart
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=YqZaY8Jiprc69TQZ@pendragon.ideasonboard.com \ --to=laurent.pinchart@ideasonboard.com \ --cc=airlied@linux.ie \ --cc=dri-devel@lists.freedesktop.org \ --cc=linux-kernel@vger.kernel.org \ --cc=sam@ravnborg.org \ --cc=venkateshwar.rao.gannavarapu@xilinx.com \ --cc=vgannava@xilinx.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.