From: Eric Anholt <eric@anholt.net> To: dri-devel@lists.freedesktop.org, Archit Taneja <architt@codeaurora.org>, Andrzej Hajda <a.hajda@samsung.com>, Laurent Pinchart <Laurent.pinchart@ideasonboard.com>, Thierry Reding <thierry.reding@gmail.com> Cc: linux-kernel@vger.kernel.org, Eric Anholt <eric@anholt.net> Subject: [PATCH 6/8] drm: Allow DSI devices to be registered before the host registers. Date: Tue, 27 Jun 2017 12:58:37 -0700 [thread overview] Message-ID: <20170627195839.3338-7-eric@anholt.net> (raw) In-Reply-To: <20170627195839.3338-1-eric@anholt.net> When a mipi_dsi_host is registered, the DT is walked to find any child nodes with compatible strings. Those get registered as DSI devices, and most DSI panel drivers are mipi_dsi_drivers that attach to those nodes. There is one special case currently, the adv7533 bridge, where the bridge probes on I2C, and during the bridge attach step it looks up the mipi_dsi_host and registers the mipi_dsi_device (for its own stub mipi_dsi_driver). For the Raspberry Pi panel, though, we also need to attach on I2C (our control bus), but don't have a bridge driver. The lack of a bridge's attach() step like adv7533 uses means that we aren't able to delay the mipi_dsi_device creation until the mipi_dsi_host is present. To fix this, we extend mipi_dsi_device_register_full() to allow being called with a NULL host, which puts the device on a queue waiting for a host to appear. When a new host is registered, we fill in the host value and finish the device creation process. Signed-off-by: Eric Anholt <eric@anholt.net> --- drivers/gpu/drm/drm_mipi_dsi.c | 49 ++++++++++++++++++++++++++++++++---------- include/drm/drm_mipi_dsi.h | 3 +++ 2 files changed, 41 insertions(+), 11 deletions(-) diff --git a/drivers/gpu/drm/drm_mipi_dsi.c b/drivers/gpu/drm/drm_mipi_dsi.c index 1160a579e0dc..9cdd68a7dc0d 100644 --- a/drivers/gpu/drm/drm_mipi_dsi.c +++ b/drivers/gpu/drm/drm_mipi_dsi.c @@ -45,6 +45,13 @@ * subset of the MIPI DCS command set. */ +static DEFINE_MUTEX(host_lock); +static LIST_HEAD(host_list); +/* List of struct mipi_dsi_device which were registered while no host + * was available. + */ +static LIST_HEAD(unattached_device_list); + static int mipi_dsi_device_match(struct device *dev, struct device_driver *drv) { struct mipi_dsi_device *dsi = to_mipi_dsi_device(dev); @@ -138,10 +145,12 @@ static struct mipi_dsi_device *mipi_dsi_device_alloc(struct mipi_dsi_host *host) dsi->host = host; dsi->dev.bus = &mipi_dsi_bus_type; - dsi->dev.parent = host->dev; dsi->dev.type = &mipi_dsi_device_type; - device_initialize(&dsi->dev); + if (dsi->host) { + dsi->dev.parent = host->dev; + device_initialize(&dsi->dev); + } return dsi; } @@ -206,7 +215,7 @@ mipi_dsi_device_register_full(struct mipi_dsi_host *host, const struct mipi_dsi_device_info *info) { struct mipi_dsi_device *dsi; - struct device *dev = host->dev; + struct device *dev = host ? host->dev : NULL; int ret; if (!info) { @@ -230,11 +239,17 @@ mipi_dsi_device_register_full(struct mipi_dsi_host *host, dsi->channel = info->channel; strlcpy(dsi->name, info->type, sizeof(dsi->name)); - ret = mipi_dsi_device_add(dsi); - if (ret) { - dev_err(dev, "failed to add DSI device %d\n", ret); - kfree(dsi); - return ERR_PTR(ret); + if (!dsi->host) { + mutex_lock(&host_lock); + list_add(&dsi->list, &unattached_device_list); + mutex_unlock(&host_lock); + } else { + ret = mipi_dsi_device_add(dsi); + if (ret) { + dev_err(dev, "failed to add DSI device %d\n", ret); + kfree(dsi); + return ERR_PTR(ret); + } } return dsi; @@ -251,9 +266,6 @@ void mipi_dsi_device_unregister(struct mipi_dsi_device *dsi) } EXPORT_SYMBOL(mipi_dsi_device_unregister); -static DEFINE_MUTEX(host_lock); -static LIST_HEAD(host_list); - /** * of_find_mipi_dsi_host_by_node() - find the MIPI DSI host matching a * device tree node @@ -285,6 +297,7 @@ EXPORT_SYMBOL(of_find_mipi_dsi_host_by_node); int mipi_dsi_host_register(struct mipi_dsi_host *host) { struct device_node *node; + struct mipi_dsi_device *dsi, *temp; for_each_available_child_of_node(host->dev->of_node, node) { /* skip nodes without reg property */ @@ -295,6 +308,20 @@ int mipi_dsi_host_register(struct mipi_dsi_host *host) mutex_lock(&host_lock); list_add_tail(&host->list, &host_list); + + /* If any DSI devices were registered under our OF node, then + * connect our host to it and probe them now. + */ + list_for_each_entry_safe(dsi, temp, &unattached_device_list, list) { + if (of_get_parent(dsi->dev.of_node) == host->dev->of_node) { + dsi->host = host; + dsi->dev.parent = host->dev; + device_initialize(&dsi->dev); + + mipi_dsi_device_add(dsi); + list_del_init(&dsi->list); + } + } mutex_unlock(&host_lock); return 0; diff --git a/include/drm/drm_mipi_dsi.h b/include/drm/drm_mipi_dsi.h index 4fef19064b0f..699ea4acd5b6 100644 --- a/include/drm/drm_mipi_dsi.h +++ b/include/drm/drm_mipi_dsi.h @@ -178,6 +178,9 @@ struct mipi_dsi_device { unsigned int lanes; enum mipi_dsi_pixel_format format; unsigned long mode_flags; + + /* Entry on the unattached_device_list */ + struct list_head list; }; #define MIPI_DSI_MODULE_PREFIX "mipi-dsi:" -- 2.11.0
WARNING: multiple messages have this Message-ID (diff)
From: Eric Anholt <eric@anholt.net> To: dri-devel@lists.freedesktop.org, Archit Taneja <architt@codeaurora.org>, Andrzej Hajda <a.hajda@samsung.com>, Laurent Pinchart <Laurent.pinchart@ideasonboard.com>, Thierry Reding <thierry.reding@gmail.com> Cc: linux-kernel@vger.kernel.org Subject: [PATCH 6/8] drm: Allow DSI devices to be registered before the host registers. Date: Tue, 27 Jun 2017 12:58:37 -0700 [thread overview] Message-ID: <20170627195839.3338-7-eric@anholt.net> (raw) In-Reply-To: <20170627195839.3338-1-eric@anholt.net> When a mipi_dsi_host is registered, the DT is walked to find any child nodes with compatible strings. Those get registered as DSI devices, and most DSI panel drivers are mipi_dsi_drivers that attach to those nodes. There is one special case currently, the adv7533 bridge, where the bridge probes on I2C, and during the bridge attach step it looks up the mipi_dsi_host and registers the mipi_dsi_device (for its own stub mipi_dsi_driver). For the Raspberry Pi panel, though, we also need to attach on I2C (our control bus), but don't have a bridge driver. The lack of a bridge's attach() step like adv7533 uses means that we aren't able to delay the mipi_dsi_device creation until the mipi_dsi_host is present. To fix this, we extend mipi_dsi_device_register_full() to allow being called with a NULL host, which puts the device on a queue waiting for a host to appear. When a new host is registered, we fill in the host value and finish the device creation process. Signed-off-by: Eric Anholt <eric@anholt.net> --- drivers/gpu/drm/drm_mipi_dsi.c | 49 ++++++++++++++++++++++++++++++++---------- include/drm/drm_mipi_dsi.h | 3 +++ 2 files changed, 41 insertions(+), 11 deletions(-) diff --git a/drivers/gpu/drm/drm_mipi_dsi.c b/drivers/gpu/drm/drm_mipi_dsi.c index 1160a579e0dc..9cdd68a7dc0d 100644 --- a/drivers/gpu/drm/drm_mipi_dsi.c +++ b/drivers/gpu/drm/drm_mipi_dsi.c @@ -45,6 +45,13 @@ * subset of the MIPI DCS command set. */ +static DEFINE_MUTEX(host_lock); +static LIST_HEAD(host_list); +/* List of struct mipi_dsi_device which were registered while no host + * was available. + */ +static LIST_HEAD(unattached_device_list); + static int mipi_dsi_device_match(struct device *dev, struct device_driver *drv) { struct mipi_dsi_device *dsi = to_mipi_dsi_device(dev); @@ -138,10 +145,12 @@ static struct mipi_dsi_device *mipi_dsi_device_alloc(struct mipi_dsi_host *host) dsi->host = host; dsi->dev.bus = &mipi_dsi_bus_type; - dsi->dev.parent = host->dev; dsi->dev.type = &mipi_dsi_device_type; - device_initialize(&dsi->dev); + if (dsi->host) { + dsi->dev.parent = host->dev; + device_initialize(&dsi->dev); + } return dsi; } @@ -206,7 +215,7 @@ mipi_dsi_device_register_full(struct mipi_dsi_host *host, const struct mipi_dsi_device_info *info) { struct mipi_dsi_device *dsi; - struct device *dev = host->dev; + struct device *dev = host ? host->dev : NULL; int ret; if (!info) { @@ -230,11 +239,17 @@ mipi_dsi_device_register_full(struct mipi_dsi_host *host, dsi->channel = info->channel; strlcpy(dsi->name, info->type, sizeof(dsi->name)); - ret = mipi_dsi_device_add(dsi); - if (ret) { - dev_err(dev, "failed to add DSI device %d\n", ret); - kfree(dsi); - return ERR_PTR(ret); + if (!dsi->host) { + mutex_lock(&host_lock); + list_add(&dsi->list, &unattached_device_list); + mutex_unlock(&host_lock); + } else { + ret = mipi_dsi_device_add(dsi); + if (ret) { + dev_err(dev, "failed to add DSI device %d\n", ret); + kfree(dsi); + return ERR_PTR(ret); + } } return dsi; @@ -251,9 +266,6 @@ void mipi_dsi_device_unregister(struct mipi_dsi_device *dsi) } EXPORT_SYMBOL(mipi_dsi_device_unregister); -static DEFINE_MUTEX(host_lock); -static LIST_HEAD(host_list); - /** * of_find_mipi_dsi_host_by_node() - find the MIPI DSI host matching a * device tree node @@ -285,6 +297,7 @@ EXPORT_SYMBOL(of_find_mipi_dsi_host_by_node); int mipi_dsi_host_register(struct mipi_dsi_host *host) { struct device_node *node; + struct mipi_dsi_device *dsi, *temp; for_each_available_child_of_node(host->dev->of_node, node) { /* skip nodes without reg property */ @@ -295,6 +308,20 @@ int mipi_dsi_host_register(struct mipi_dsi_host *host) mutex_lock(&host_lock); list_add_tail(&host->list, &host_list); + + /* If any DSI devices were registered under our OF node, then + * connect our host to it and probe them now. + */ + list_for_each_entry_safe(dsi, temp, &unattached_device_list, list) { + if (of_get_parent(dsi->dev.of_node) == host->dev->of_node) { + dsi->host = host; + dsi->dev.parent = host->dev; + device_initialize(&dsi->dev); + + mipi_dsi_device_add(dsi); + list_del_init(&dsi->list); + } + } mutex_unlock(&host_lock); return 0; diff --git a/include/drm/drm_mipi_dsi.h b/include/drm/drm_mipi_dsi.h index 4fef19064b0f..699ea4acd5b6 100644 --- a/include/drm/drm_mipi_dsi.h +++ b/include/drm/drm_mipi_dsi.h @@ -178,6 +178,9 @@ struct mipi_dsi_device { unsigned int lanes; enum mipi_dsi_pixel_format format; unsigned long mode_flags; + + /* Entry on the unattached_device_list */ + struct list_head list; }; #define MIPI_DSI_MODULE_PREFIX "mipi-dsi:" -- 2.11.0 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2017-06-27 19:59 UTC|newest] Thread overview: 52+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-06-27 19:58 [PATCH 0/8] RPi touchscreen panel driver v4 Eric Anholt 2017-06-27 19:58 ` Eric Anholt 2017-06-27 19:58 ` [PATCH 1/8] drm/vc4: Fix DSI T_INIT timing Eric Anholt 2017-06-27 19:58 ` Eric Anholt 2017-06-29 9:02 ` Andrzej Hajda 2017-06-29 9:02 ` Andrzej Hajda 2017-06-27 19:58 ` [PATCH 2/8] drm/vc4: Fix misleading name of the continuous flag Eric Anholt 2017-06-27 19:58 ` Eric Anholt 2017-06-29 9:03 ` Andrzej Hajda 2017-06-29 9:03 ` Andrzej Hajda 2017-06-27 19:58 ` [PATCH 3/8] drm/vc4: Use drm_mode_vrefresh() in DSI fixup, in case vrefresh is 0 Eric Anholt 2017-06-27 19:58 ` Eric Anholt 2017-06-29 9:24 ` Andrzej Hajda 2017-06-29 9:24 ` Andrzej Hajda 2017-06-27 19:58 ` [PATCH 4/8] drm/bridge: Add a devm_ allocator for panel bridge Eric Anholt 2017-06-27 19:58 ` Eric Anholt 2017-06-29 9:30 ` Andrzej Hajda 2017-06-29 9:30 ` Andrzej Hajda 2017-06-27 19:58 ` [PATCH 5/8] drm/vc4: Delay DSI host registration until the panel has probed Eric Anholt 2017-06-27 19:58 ` Eric Anholt 2017-06-29 9:42 ` Andrzej Hajda 2017-06-29 9:42 ` Andrzej Hajda 2017-06-27 19:58 ` Eric Anholt [this message] 2017-06-27 19:58 ` [PATCH 6/8] drm: Allow DSI devices to be registered before the host registers Eric Anholt 2017-06-29 5:03 ` Archit Taneja 2017-06-29 5:03 ` Archit Taneja 2017-06-29 10:39 ` Andrzej Hajda 2017-06-29 10:39 ` Andrzej Hajda 2017-06-29 15:22 ` Eric Anholt 2017-06-29 15:22 ` Eric Anholt 2017-06-30 8:48 ` Andrzej Hajda 2017-06-30 8:48 ` Andrzej Hajda 2017-07-03 10:56 ` Archit Taneja 2017-07-03 10:56 ` Archit Taneja 2017-07-14 23:01 ` Eric Anholt 2017-07-14 23:01 ` Eric Anholt 2017-07-17 13:39 ` Archit Taneja 2017-07-17 13:39 ` Archit Taneja 2017-07-14 22:58 ` Eric Anholt 2017-07-14 22:58 ` Eric Anholt 2017-07-17 4:26 ` Archit Taneja 2017-07-17 4:26 ` Archit Taneja 2017-07-18 20:13 ` Eric Anholt 2017-07-18 20:13 ` Eric Anholt 2017-07-19 3:29 ` Archit Taneja 2017-07-19 3:29 ` Archit Taneja 2017-08-04 9:29 ` Boris Brezillon 2017-08-04 9:29 ` Boris Brezillon 2017-06-27 19:58 ` [PATCH 7/8] dt-bindings: Document the Raspberry Pi Touchscreen nodes Eric Anholt 2017-06-27 19:58 ` Eric Anholt 2017-06-27 19:58 ` [PATCH 8/8] drm/panel: Add support for the Raspberry Pi 7" Touchscreen Eric Anholt 2017-06-27 19:58 ` Eric Anholt
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=20170627195839.3338-7-eric@anholt.net \ --to=eric@anholt.net \ --cc=Laurent.pinchart@ideasonboard.com \ --cc=a.hajda@samsung.com \ --cc=architt@codeaurora.org \ --cc=dri-devel@lists.freedesktop.org \ --cc=linux-kernel@vger.kernel.org \ --cc=thierry.reding@gmail.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.