On 06/05/2013 05:24 PM, Sebastian Andrzej Siewior wrote:
> TI-adc reads a little better compared to tiadc. And if we add am335x to
> it then we have the same naming scheme as the tsc side.
> While here add an alias so the driver is loaded on demand.
Good, I also prefer this naming.

> 
> Signed-off-by: Sebastian Andrzej Siewior <[email protected]>
Acked-by: Jonathan Cameron <[email protected]>
> ---
>  drivers/iio/adc/ti_am335x_adc.c |    4 ++--
>  drivers/mfd/ti_am335x_tscadc.c  |    2 +-
>  2 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/iio/adc/ti_am335x_adc.c b/drivers/iio/adc/ti_am335x_adc.c
> index c6695a8..38f2415 100644
> --- a/drivers/iio/adc/ti_am335x_adc.c
> +++ b/drivers/iio/adc/ti_am335x_adc.c
> @@ -318,14 +318,14 @@ static const struct dev_pm_ops tiadc_pm_ops = {
>  
>  static struct platform_driver tiadc_driver = {
>       .driver = {
> -             .name   = "tiadc",
> +             .name   = "TI-am335x-adc",
>               .owner  = THIS_MODULE,
>               .pm     = TIADC_PM_OPS,
>       },
>       .probe  = tiadc_probe,
>       .remove = tiadc_remove,
>  };
> -
> +MODULE_ALIAS("platform:TI-am335x-adc");
>  module_platform_driver(tiadc_driver);
>  
>  MODULE_DESCRIPTION("TI ADC controller driver");
> diff --git a/drivers/mfd/ti_am335x_tscadc.c b/drivers/mfd/ti_am335x_tscadc.c
> index ba9235a..8bdab8d 100644
> --- a/drivers/mfd/ti_am335x_tscadc.c
> +++ b/drivers/mfd/ti_am335x_tscadc.c
> @@ -203,7 +203,7 @@ static    int ti_tscadc_probe(struct platform_device 
> *pdev)
>       if (adc_channels > 0) {
>               tscadc->adc_cell = tscadc->used_cells;
>               cell = &tscadc->cells[tscadc->used_cells++];
> -             cell->name = "tiadc";
> +             cell->name = "TI-am335x-adc";
>               cell->platform_data = &tscadc;
>               cell->pdata_size = sizeof(tscadc);
>       }
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to