From: "Giedrius Statkevičius" <giedrius.statkevicius@gmail.com>
To: lidza.louina@gmail.com, markh@compro.net
Cc: gregkh@linuxfoundation.org,
driverdev-devel@linuxdriverproject.org,
devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 1/7] dgnc: remove unused dgnc_ioctl_name() command
Date: Tue, 17 Mar 2015 16:35:00 +0200 [thread overview]
Message-ID: <55083B94.6020309@gmail.com> (raw)
In-Reply-To: <1426254995-28359-1-git-send-email-giedrius.statkevicius@gmail.com>
Greg, you probably missed these :) Could you look at these when you have
some time?
On 2015.03.13 15:56, Giedrius Statkevičius wrote:
> dgnc_ioctl_name() is never used anywhere so remove it
>
> Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@gmail.com>
> ---
> v2: Forgot to include this patch in the first version. Without this some
> of the other patches in this set will fail.
>
> drivers/staging/dgnc/dgnc_utils.c | 52 ---------------------------------------
> drivers/staging/dgnc/dgnc_utils.h | 1 -
> 2 files changed, 53 deletions(-)
>
> diff --git a/drivers/staging/dgnc/dgnc_utils.c b/drivers/staging/dgnc/dgnc_utils.c
> index 80b5133..f76de82 100644
> --- a/drivers/staging/dgnc/dgnc_utils.c
> +++ b/drivers/staging/dgnc/dgnc_utils.c
> @@ -16,55 +16,3 @@ int dgnc_ms_sleep(ulong ms)
> schedule_timeout((ms * HZ) / 1000);
> return signal_pending(current);
> }
> -
> -/*
> - * dgnc_ioctl_name() : Returns a text version of each ioctl value.
> - */
> -char *dgnc_ioctl_name(int cmd)
> -{
> - switch (cmd) {
> -
> - case TCGETA: return "TCGETA";
> - case TCGETS: return "TCGETS";
> - case TCSETA: return "TCSETA";
> - case TCSETS: return "TCSETS";
> - case TCSETAW: return "TCSETAW";
> - case TCSETSW: return "TCSETSW";
> - case TCSETAF: return "TCSETAF";
> - case TCSETSF: return "TCSETSF";
> - case TCSBRK: return "TCSBRK";
> - case TCXONC: return "TCXONC";
> - case TCFLSH: return "TCFLSH";
> - case TIOCGSID: return "TIOCGSID";
> -
> - case TIOCGETD: return "TIOCGETD";
> - case TIOCSETD: return "TIOCSETD";
> - case TIOCGWINSZ: return "TIOCGWINSZ";
> - case TIOCSWINSZ: return "TIOCSWINSZ";
> -
> - case TIOCMGET: return "TIOCMGET";
> - case TIOCMSET: return "TIOCMSET";
> - case TIOCMBIS: return "TIOCMBIS";
> - case TIOCMBIC: return "TIOCMBIC";
> -
> - /* from digi.h */
> - case DIGI_SETA: return "DIGI_SETA";
> - case DIGI_SETAW: return "DIGI_SETAW";
> - case DIGI_SETAF: return "DIGI_SETAF";
> - case DIGI_SETFLOW: return "DIGI_SETFLOW";
> - case DIGI_SETAFLOW: return "DIGI_SETAFLOW";
> - case DIGI_GETFLOW: return "DIGI_GETFLOW";
> - case DIGI_GETAFLOW: return "DIGI_GETAFLOW";
> - case DIGI_GETA: return "DIGI_GETA";
> - case DIGI_GEDELAY: return "DIGI_GEDELAY";
> - case DIGI_SEDELAY: return "DIGI_SEDELAY";
> - case DIGI_GETCUSTOMBAUD: return "DIGI_GETCUSTOMBAUD";
> - case DIGI_SETCUSTOMBAUD: return "DIGI_SETCUSTOMBAUD";
> - case TIOCMODG: return "TIOCMODG";
> - case TIOCMODS: return "TIOCMODS";
> - case TIOCSDTR: return "TIOCSDTR";
> - case TIOCCDTR: return "TIOCCDTR";
> -
> - default: return "unknown";
> - }
> -}
> diff --git a/drivers/staging/dgnc/dgnc_utils.h b/drivers/staging/dgnc/dgnc_utils.h
> index cebf601..1164c3a 100644
> --- a/drivers/staging/dgnc/dgnc_utils.h
> +++ b/drivers/staging/dgnc/dgnc_utils.h
> @@ -2,6 +2,5 @@
> #define __DGNC_UTILS_H
>
> int dgnc_ms_sleep(ulong ms);
> -char *dgnc_ioctl_name(int cmd);
>
> #endif
>
--
Thanks,
Giedrius
next prev parent reply other threads:[~2015-03-17 14:35 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-13 13:56 [PATCH v2 1/7] dgnc: remove unused dgnc_ioctl_name() command Giedrius Statkevičius
2015-03-13 13:56 ` [PATCH v2 2/7] dgnc: clean up digi.h Giedrius Statkevičius
2015-03-13 13:56 ` [PATCH v2 3/7] dgnc: get rid of dpacompat.h, move remaining stuff to digi.h Giedrius Statkevičius
2015-03-13 13:56 ` [PATCH v2 4/7] dgnc: use linux/types.h instead of dgnc_types.h Giedrius Statkevičius
2015-03-13 13:56 ` [PATCH v2 5/7] dgnc: remove old 2.4-2.6 compat kernel defines Giedrius Statkevičius
2015-03-13 13:56 ` [PATCH v2 6/7] dgnc: remove unused stuff from dgnc_cls.h Giedrius Statkevičius
2015-03-13 13:56 ` [PATCH v2 7/7] dgnc: Clean up dgnc_sysfs.h Giedrius Statkevičius
2015-03-17 14:35 ` Giedrius Statkevičius [this message]
2015-03-17 14:48 ` [PATCH v2 1/7] dgnc: remove unused dgnc_ioctl_name() command Greg KH
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=55083B94.6020309@gmail.com \
--to=giedrius.statkevicius@gmail.com \
--cc=devel@driverdev.osuosl.org \
--cc=driverdev-devel@linuxdriverproject.org \
--cc=gregkh@linuxfoundation.org \
--cc=lidza.louina@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=markh@compro.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.