On 05/01/2014 11:11 AM, Russell King - ARM Linux wrote: > On Thu, May 01, 2014 at 09:04:19AM +0200, Andrzej Hajda wrote: >> 2. You replace calls of component_add and component_del with calls >> to interface_tracker_ifup(dev, INTERFACE_TRACKER_TYPE_COMPONENT, >> &specific_component_ops), >> or interface_tracker_ifdown. >> Thats all for components. > How does the master get to decide which components are for it?
This is not a part of the framework. The master can construct the list of its components anyhow. Some examples: 1. List of device tree video interface port nodes pointed by master's remote-endpoint phandles. 2. List of device nodes pointed by supernode phandles. 3. Nodes pointed by other phandles in master's node. 4. Devices compatible with the list of drivers. This is for create list of objects. As interface type it should use the types of interface it expects at given nodes and is able to handle. Small issue: If the master creates list of devices and for particular interface type expects DT node as the object, translation is easy: dev->of_node. But if the situation is reverse kernel does not provide generic helper for translating of_node to device, however kernel have everything to provide such function if necessary. Other solution is to use only DT nodes for object identification, it will narrow the framework usage to DT architectures, but seems to be more flexible anyway - we can have more than one interface of given type per device, we can distinguish them by port node. > As > I see it, all masters see all components of a particular "type". > What if you have a system with two masters each of which are bound > to a set of unique components but some of the components are of a > the same type? The master receives notifications only about interfaces he has registered callback for. For example: interface_tracker_register(node, INTERFACE_TRACKER_TYPE_DRM_PANEL, cb); means that 'cb' callback will be called only if panel identifed by node is up or down. If the driver expect that at the 'node' there could be also component it can also listen for components: interface_tracker_register(node, INTERFACE_TRACKER_TYPE_COMPONENT, cb); Now 'cb' will be called if component or panel appears/disappears at node 'node'. so callback function can look like: void cb_func(struct interface_tracker_block *itb, const void *object, unsigned long type, bool on, void *data) { struct priv_struct *priv = container_of(itb, struct priv_struct, itb); switch(type) { case INTERFACE_TRACKER_TYPE_DRM_PANEL: handle_drm_panel(priv, data, on); break; case INTERFACE_TRACKER_TYPE_DRM_COMPONENT: handle_drm_component(priv, data, object, on); break; } } where handlers can look like: void handle_drm_panel(struct priv_struct *priv, struct drm_panel *panel, bool on); void handle_drm_component(struct priv_struct *priv, struct component_ops *ops, struct device *dev, bool on); > > How does the master know what "type" to use? > It should use type which it expects at the given node. Regards Andrzej -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html