Re: [PATCH] staging: gasket: Add descriptions for module parameters

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Jul 08, 2018 at 08:30:50PM +0300, Kamal Heib wrote:
> This patch add description for the module parameters and remove the task
> to do it from the TODO file.
> 
> Signed-off-by: Kamal Heib <kamalheib1@xxxxxxxxx>
> ---
>  drivers/staging/gasket/TODO          | 1 -
>  drivers/staging/gasket/apex_driver.c | 8 ++++++++
>  2 files changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/gasket/TODO b/drivers/staging/gasket/TODO
> index 0d8ee9602c80..c524dba7945f 100644
> --- a/drivers/staging/gasket/TODO
> +++ b/drivers/staging/gasket/TODO
> @@ -5,7 +5,6 @@ staging directory.
>  - Remove static function declarations.
>  - Document sysfs files with Documentation/ABI/ entries.
>  - Use misc interface instead of major number for driver version description.
> -- Add descriptions of module_param's
>  - Remove gasket-specific logging functions.
>  - apex_get_status() should actually check status.
>  - Static functions don't need kernel doc formatting, can be simplified.
> diff --git a/drivers/staging/gasket/apex_driver.c b/drivers/staging/gasket/apex_driver.c
> index 395256704428..21deef38807a 100644
> --- a/drivers/staging/gasket/apex_driver.c
> +++ b/drivers/staging/gasket/apex_driver.c
> @@ -329,9 +329,17 @@ static int allow_hw_clock_gating = 1;
>  static int bypass_top_level;
>  
>  module_param(allow_power_save, int, 0644);
> +MODULE_PARM_DESC(allow_power_save,
> +		 "Allows device to enter power save upon driver close()");
> +
>  module_param(allow_sw_clock_gating, int, 0644);
> +MODULE_PARM_DESC(allow_sw_clock_gating, "Allows SW based clock gating");
> +
>  module_param(allow_hw_clock_gating, int, 0644);
> +MODULE_PARM_DESC(allow_hw_clock_gating, "Allows HW based clock gating");
> +
>  module_param(bypass_top_level, int, 0644);
> +MODULE_PARM_DESC(bypass_top_level, "Act as if only GCB is instantiated");

Are you sure these options are even needed?  "Modern" drivers do not
need kernel module options as everything should "just work"
automatically.  Why are these options on a per-module basis and not a
per-device basis?

So I'd prefer to not take this patch and have the root problem here
fixed instead.

thanks,

greg k-h
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux