diff mbox

Input: zet6223 - Export OF device ID as module aliases

Message ID 20170220200855.9951-1-javier@osg.samsung.com (mailing list archive)
State Accepted
Headers show

Commit Message

Javier Martinez Canillas Feb. 20, 2017, 8:08 p.m. UTC
The I2C core always reports a MODALIAS of the form i2c:<foo> even if the
device was registered via OF, this means that exporting the OF device ID
table device aliases in the module is not needed. But in order to change
how the core reports modaliases to user-space, it's better to export it.

Before this patch:

$ modinfo drivers/input/touchscreen/zet6223.ko | grep alias
alias:          i2c:zet6223

After this patch:

$ modinfo drivers/input/touchscreen/zet6223.ko | grep alias
alias:          of:N*T*Czeitec,zet6223C*
alias:          of:N*T*Czeitec,zet6223
alias:          i2c:zet6223

Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
---

 drivers/input/touchscreen/zet6223.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Dmitry Torokhov Feb. 20, 2017, 11:28 p.m. UTC | #1
On Mon, Feb 20, 2017 at 05:08:55PM -0300, Javier Martinez Canillas wrote:
> The I2C core always reports a MODALIAS of the form i2c:<foo> even if the
> device was registered via OF, this means that exporting the OF device ID
> table device aliases in the module is not needed. But in order to change
> how the core reports modaliases to user-space, it's better to export it.
> 
> Before this patch:
> 
> $ modinfo drivers/input/touchscreen/zet6223.ko | grep alias
> alias:          i2c:zet6223
> 
> After this patch:
> 
> $ modinfo drivers/input/touchscreen/zet6223.ko | grep alias
> alias:          of:N*T*Czeitec,zet6223C*
> alias:          of:N*T*Czeitec,zet6223
> alias:          i2c:zet6223
> 
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>

Applied, thank you.

> ---
> 
>  drivers/input/touchscreen/zet6223.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/input/touchscreen/zet6223.c b/drivers/input/touchscreen/zet6223.c
> index e7fb00b511ec..19ffcc7b886c 100644
> --- a/drivers/input/touchscreen/zet6223.c
> +++ b/drivers/input/touchscreen/zet6223.c
> @@ -245,6 +245,7 @@ static const struct of_device_id zet6223_of_match[] = {
>  	{ .compatible = "zeitec,zet6223" },
>  	{ }
>  };
> +MODULE_DEVICE_TABLE(of, zet6223_of_match);
>  
>  static const struct i2c_device_id zet6223_id[] = {
>  	{ "zet6223", 0},
> -- 
> 2.9.3
>
diff mbox

Patch

diff --git a/drivers/input/touchscreen/zet6223.c b/drivers/input/touchscreen/zet6223.c
index e7fb00b511ec..19ffcc7b886c 100644
--- a/drivers/input/touchscreen/zet6223.c
+++ b/drivers/input/touchscreen/zet6223.c
@@ -245,6 +245,7 @@  static const struct of_device_id zet6223_of_match[] = {
 	{ .compatible = "zeitec,zet6223" },
 	{ }
 };
+MODULE_DEVICE_TABLE(of, zet6223_of_match);
 
 static const struct i2c_device_id zet6223_id[] = {
 	{ "zet6223", 0},