On 2013년 02월 06일 17:51, Inki Dae wrote: > > >> -----Original Message----- >> From: Sachin Kamat [mailto:sachin.kamat@xxxxxxxxxx] >> Sent: Wednesday, February 06, 2013 5:03 PM >> To: Inki Dae >> Cc: linux-media@xxxxxxxxxxxxxxx; dri-devel@xxxxxxxxxxxxxxxxxxxxx; >> devicetree-discuss@xxxxxxxxxxxxxxxx; k.debski@xxxxxxxxxxx; >> s.nawrocki@xxxxxxxxxxx; kgene.kim@xxxxxxxxxxx; patches@xxxxxxxxxx; Ajay >> Kumar >> Subject: Re: [PATCH v2 2/2] drm/exynos: Add device tree based discovery >> support for G2D >> >> On 6 February 2013 13:02, Inki Dae <inki.dae@xxxxxxxxxxx> wrote: >>> >>> Looks good to me but please add document for it. >> >> Yes. I will. I was planning to send the bindings document patch along >> with the dt patches (adding node entries to dts files). >> Sylwester had suggested adding this to >> Documentation/devicetree/bindings/media/ which contains other media >> IPs. > > I think that it's better to go to gpu than media and we can divide Exynos > IPs into the bellow categories, > > Media : mfc > GPU : g2d, g3d, fimc, gsc > Video : fimd, hdmi, eDP, MIPI-DSI Hm, here is another considering point. Some device can be used as one of two sub-system. For example g2d can be used as V4L2 driver or DRM driver. And more specific case, multiple fimc/gsc deivces can be separately used as both drivers: two fimc devices are used as V4L2 driver and other devices are used as DRM driver. Current discussion, without change of build configuration, device can be only used as one driver. So I want to discuss about how we can bind device and driver just with dts configuration. IMO, there are two options. First, driver usage is set on configurable node. g2d: g2d { compatible = "samsung,exynos4212-g2d"; ... *subsystem = "v4l2"* or *subsystem = "drm"* }; Node name and type is just an example to describe. With this option, driver which is not matched with subsystem node should return with fail during its probing. Second, using dual compatible strings. g2d: g2d { *compatible = "samsung,exynos4212-v4l2-g2d"; or compatible = "samsung,exynos4212-v4l2-g2d";* ... }; String is just an example so don't mind if it is ugly. Actually, with this option, compatible string has non HW information. But this option does not need fail in probing. I'm not sure these options are fit to DT concept. Please let me know if anyone has idea. Best Regards, - Seung-Woo Kim > > And I think that the device-tree describes hardware so possibly, all > documents in .../bindings/drm/exynos/* should be moved to proper place also. > Please give me any opinions. > > Thanks, > Inki Dae > >> >>> >>> To other guys, >>> And is there anyone who know where this document should be added to? >>> I'm not sure that the g2d document should be placed in >>> Documentation/devicetree/bindings/gpu, media, drm/exynos or arm/exynos. >> At >>> least, this document should be shared with the g2d hw relevant drivers >> such >>> as v4l2 and drm. So is ".../bindings/gpu" proper place? >>> >> >> >> -- >> With warm regards, >> Sachin > > _______________________________________________ > dri-devel mailing list > dri-devel@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/dri-devel > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
