Path: csiph.com!aioe.org!bofh.it!news.nic.it!robomod From: Paul Bolle Newsgroups: linux.kernel Subject: Re: [PATCH] driver/i2c/mux: Add register based mux i2c-mux-reg Date: Thu, 18 Jun 2015 10:10:01 +0200 Message-ID: References: X-Original-To: Alexander Sverdlin Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.10.4 (3.10.4-4.fc20) MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: robomod@news.nic.it List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Approved: robomod@news.nic.it Lines: 34 Organization: linux.* mail to news gateway X-Original-Cc: York Sun , wsa@the-dreams.de, linux-kernel@vger.kernel.org, linux-i2c@vger.kernel.org, Peter Korsgaard X-Original-Date: Thu, 18 Jun 2015 10:08:21 +0200 X-Original-Message-ID: <1434614901.2385.27.camel@x220> X-Original-References: <1434475692-4611-1-git-send-email-yorksun@freescale.com> <1434531244.2069.111.camel@x220> <55818BAB.5090605@nokia.com> <1434556990.2400.16.camel@x220> <558275F1.5020100@nokia.com> X-Original-Sender: linux-kernel-owner@vger.kernel.org Xref: aioe.org linux.kernel:1167655 Hi Alexander, On Thu, 2015-06-18 at 09:40 +0200, Alexander Sverdlin wrote: > On 17/06/15 18:03, ext Paul Bolle wrote: > >> You do not see the platform_device, because there are no users yet, put > >> > this MODULE_ALIAS() is perfectly fine, it will allow automatic module loading > >> > in non-DT case. > > Do you mean that it will allow automatic module loading once the patch > > that adds a struct platform_device with a "i2c-mux-reg" name lands? > > Any platform code which will register the platform_device will trigger uevent and > udevd will be able to find the module with this macro. This is a legacy alternative > to device-tree approach. That means I've correctly figured out the purpose of this MODULE_ALIAS("platform:" stuff. Because it might actually be documented somewhere but I managed to not stumble on that documentation. With that out of the way: am I right in thinking there's currently no platform code that triggers that uevent for "MODALIAS=platform:i2c-mux-reg"? Because if there's no struct platform_device taking care of that I think this MODULE_ALIAS() should not be added, not yet. Thanks, Paul Bolle -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/