On 04/04/2017 10:59 AM, Daniel Vetter wrote:
> On Tue, Mar 21, 2017 at 04:25:40PM +0100, Neil Armstrong wrote:
>> This patch adds support for optional components connected through the
>> Device Tree endpoints scheme.
>>
>> Signed-off-by: Neil Armstrong <narmstrong@xxxxxxxxxxxx>
>> ---
>> drivers/gpu/drm/meson/meson_drv.c | 113 +++++++++++++++++++++++++++++++++-----
>> 1 file changed, 99 insertions(+), 14 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/meson/meson_drv.c b/drivers/gpu/drm/meson/meson_drv.c
>> index bc562a0..a2e9f56 100644
>> --- a/drivers/gpu/drm/meson/meson_drv.c
>> +++ b/drivers/gpu/drm/meson/meson_drv.c
>> @@ -24,6 +24,7 @@
>> #include <linux/module.h>
>> #include <linux/mutex.h>
>> #include <linux/platform_device.h>
>> +#include <linux/component.h>
>> #include <linux/of_graph.h>
>>
>> #include <drm/drmP.h>
>> @@ -150,9 +151,9 @@ static bool meson_vpu_has_available_connectors(struct device *dev)
>> .max_register = 0x1000,
>> };
>>
>> -static int meson_drv_probe(struct platform_device *pdev)
>> +static int meson_drv_bind(struct device *dev)
>> {
>> - struct device *dev = &pdev->dev;
>> + struct platform_device *pdev = to_platform_device(dev);
>> struct meson_drm *priv;
>> struct drm_device *drm;
>> struct resource *res;
>> @@ -215,6 +216,15 @@ static int meson_drv_probe(struct platform_device *pdev)
>>
>> drm_vblank_init(drm, 1);
>> drm_mode_config_init(drm);
>> + drm->mode_config.max_width = 3840;
>> + drm->mode_config.max_height = 2160;
>> + drm->mode_config.funcs = &meson_mode_config_funcs;
>> +
>> + /* Hardware Initialization */
>> +
>> + meson_venc_init(priv);
>> + meson_vpp_init(priv);
>> + meson_viu_init(priv);
>>
>> /* Encoder Initialization */
>>
>> @@ -222,11 +232,11 @@ static int meson_drv_probe(struct platform_device *pdev)
>> if (ret)
>> goto free_drm;
>>
>> - /* Hardware Initialization */
>> -
>> - meson_venc_init(priv);
>> - meson_vpp_init(priv);
>> - meson_viu_init(priv);
>> + ret = component_bind_all(drm->dev, drm);
>> + if (ret) {
>> + dev_err(drm->dev, "Couldn't bind all components\n");
>> + goto free_drm;
>> + }
>>
>> ret = meson_plane_create(priv);
>> if (ret)
>> @@ -241,9 +251,6 @@ static int meson_drv_probe(struct platform_device *pdev)
>> goto free_drm;
>>
>> drm_mode_config_reset(drm);
>> - drm->mode_config.max_width = 8192;
>> - drm->mode_config.max_height = 8192;
>> - drm->mode_config.funcs = &meson_mode_config_funcs;
>>
>> priv->fbdev = drm_fbdev_cma_init(drm, 32,
>> drm->mode_config.num_connector);
>> @@ -268,9 +275,9 @@ static int meson_drv_probe(struct platform_device *pdev)
>> return ret;
>> }
>>
>> -static int meson_drv_remove(struct platform_device *pdev)
>> +static void meson_drv_unbind(struct device *dev)
>> {
>> - struct drm_device *drm = dev_get_drvdata(&pdev->dev);
>> + struct drm_device *drm = dev_get_drvdata(dev);
>> struct meson_drm *priv = drm->dev_private;
>>
>> drm_dev_unregister(drm);
>> @@ -280,9 +287,88 @@ static int meson_drv_remove(struct platform_device *pdev)
>> drm_vblank_cleanup(drm);
>> drm_dev_unref(drm);
>>
>> - return 0;
>> }
>>
>> +static const struct component_master_ops meson_drv_master_ops = {
>> + .bind = meson_drv_bind,
>> + .unbind = meson_drv_unbind,
>> +};
>> +
>> +static int compare_of(struct device *dev, void *data)
>> +{
>> + DRM_DEBUG_DRIVER("Comparing of node %s with %s\n",
>> + of_node_full_name(dev->of_node),
>> + of_node_full_name(data));
>> +
>> + return dev->of_node == data;
>> +}
>> +
>> +/* Possible connectors nodes to ignore */
>> +static const struct of_device_id connectors_match[] = {
>> + { .compatible = "composite-video-connector" },
>> + { .compatible = "svideo-connector" },
>> + { .compatible = "hdmi-connector" },
>> + { .compatible = "dvi-connector" },
>> + {}
>> +};
>> +
>> +static int meson_probe_remote(struct platform_device *pdev,
>> + struct component_match **match,
>> + struct device_node *parent,
>> + struct device_node *remote)
>> +{
>> + struct device_node *ep, *remote_node;
>> + int count = 1;
>> +
>> + /* If node is a connector, return and do not add to match table */
>> + if (of_match_node(connectors_match, remote))
>> + return 1;
>> +
>> + component_match_add(&pdev->dev, match, compare_of, remote);
>> +
>> + for_each_endpoint_of_node(remote, ep) {
>> + remote_node = of_graph_get_remote_port_parent(ep);
>> + if (!remote_node ||
>> + remote_node == parent || /* Ignore parent endpoint */
>> + !of_device_is_available(remote_node))
>> + continue;
>> +
>> + count += meson_probe_remote(pdev, match, remote, remote_node);
>> +
>> + of_node_put(remote_node);
>> + }
>> +
>> + return count;
>> +}
>> +
>> +static int meson_drv_probe(struct platform_device *pdev)
>> +{
>> + struct component_match *match = NULL;
>> + struct device_node *np = pdev->dev.of_node;
>> + struct device_node *ep, *remote;
>> + int count = 0;
>> +
>> + for_each_endpoint_of_node(np, ep) {
>> + remote = of_graph_get_remote_port_parent(ep);
>> + if (!remote || !of_device_is_available(remote))
>> + continue;
>> +
>> + count += meson_probe_remote(pdev, &match, np, remote);
>> + }
>> +
>> + /* If some endpoints were found, initialize the nodes */
>> + if (count) {
>
> What if you have more than 1 endpoint described, but module load order
> conspires against you to only initialize 1 at first?
As I understood, the component_bind_all() will fail until all modules are loaded
even in wrong order, no ?
I tested with multiple combinations and it worked like a charm... maybe I missed
something here !
Thanks,
Neil
> -Daniel
>
>> + dev_info(&pdev->dev, "Queued %d outputs on vpu\n", count);
>> +
>> + return component_master_add_with_match(&pdev->dev,
>> + &meson_drv_master_ops,
>> + match);
>> + }
>> +
>> + /* If no output endpoints were available, simply bail out */
>> + return 0;
>> +};
>> +
>> static const struct of_device_id dt_match[] = {
>> { .compatible = "amlogic,meson-gxbb-vpu" },
>> { .compatible = "amlogic,meson-gxl-vpu" },
>> @@ -293,7 +379,6 @@ static int meson_drv_remove(struct platform_device *pdev)
>>
>> static struct platform_driver meson_drm_platform_driver = {
>> .probe = meson_drv_probe,
>> - .remove = meson_drv_remove,
>> .driver = {
>> .name = "meson-drm",
>> .of_match_table = dt_match,
>> --
>> 1.9.1
>>
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel@xxxxxxxxxxxxxxxxxxxxx
>> https://lists.freedesktop.org/mailman/listinfo/dri-devel
>
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel
