From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hiroshi Doyu Subject: Re: [PATCHv3 01/19] [HACK] of: dev_node has struct device pointer Date: Mon, 4 Nov 2013 07:42:48 +0100 Message-ID: <20131104.084248.48379321856602699.hdoyu@nvidia.com> References: <20131101094644.GG27864@ulmo.nvidia.com><20131101.115919.351108054879013006.hdoyu@nvidia.com><527420EF.1070102@wwwdotorg.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <527420EF.1070102-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: "swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org" Cc: "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Stephen Warren , "rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org" , "linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org" , "thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org" , "grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org" List-Id: devicetree@vger.kernel.org Stephen Warren wrote @ Fri, 1 Nov 2013 22:45:19 +0100: > On 11/01/2013 03:59 AM, Hiroshi Doyu wrote: > ... > > One idea is that, rather than inserting a hook(function) per > > subsystems in driver core, if we invent a new /special section/ which > > collects all hooks in sequence like initcalls, the subsystem just > > would declare a hook function for that special section. All hooks in > > that section can be called from driver core iterately. Then we > > wouldn't touch driver core code at all, once we insert a func to call > > hooks repeatedly. Looks simpler than tweaking bus notifier. > > That sounds like a lot of effort to avoid adding a simple function call > to the core's probe path. True. Can be implemented/revisited later if needed after we once have a simple function call, of_iommu_attach() in drvier core.