2015-05-25 20:46:56

by Shailendra Verma

[permalink] [raw]
Subject: [PATCH] staging:nvec:nvec - Fix for typo in comment to function tegra_nvec_remove().


Signed-off-by: Shailendra Verma <[email protected]>
---
drivers/staging/nvec/nvec.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/nvec/nvec.c b/drivers/staging/nvec/nvec.c
index 1bdc8d0..164634d 100644
--- a/drivers/staging/nvec/nvec.c
+++ b/drivers/staging/nvec/nvec.c
@@ -916,7 +916,7 @@ static int tegra_nvec_remove(struct platform_device *pdev)
nvec_unregister_notifier(nvec, &nvec->nvec_status_notifier);
cancel_work_sync(&nvec->rx_work);
cancel_work_sync(&nvec->tx_work);
- /* FIXME: needs check wether nvec is responsible for power off */
+ /* FIXME: needs check whether nvec is responsible for power off */
pm_power_off = NULL;

return 0;
--
1.7.9.5


2015-05-26 13:44:18

by Marc Dietrich

[permalink] [raw]
Subject: Re: [PATCH] staging:nvec:nvec - Fix for typo in comment to function tegra_nvec_remove().

Hi Shailendra,

you should fill the patch description with something, even if trivial, and
trim the subject line a bit, e.g.

---
staging: nvec: fix typo in comment (all lowercase, no full stop at the end)

Fix spelling error in comment in function tegra_nvec_remove.

S-o-b
-----

Thanks,

Marc

Am Dienstag, 26. Mai 2015, 02:16:41 schrieb Shailendra Verma:
> Signed-off-by: Shailendra Verma <[email protected]>
> ---
> drivers/staging/nvec/nvec.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/nvec/nvec.c b/drivers/staging/nvec/nvec.c
> index 1bdc8d0..164634d 100644
> --- a/drivers/staging/nvec/nvec.c
> +++ b/drivers/staging/nvec/nvec.c
> @@ -916,7 +916,7 @@ static int tegra_nvec_remove(struct platform_device
> *pdev) nvec_unregister_notifier(nvec, &nvec->nvec_status_notifier);
> cancel_work_sync(&nvec->rx_work);
> cancel_work_sync(&nvec->tx_work);
> - /* FIXME: needs check wether nvec is responsible for power off */
> + /* FIXME: needs check whether nvec is responsible for power off */
> pm_power_off = NULL;
>
> return 0;


Attachments:
signature.asc (473.00 B)
This is a digitally signed message part.