From: Rodolfo Giometti <giometti@enneenne.com>
To: Alexander Gordeev <lasaine@lvk.cs.msu.su>
Cc: linux-kernel@vger.kernel.org,
"Nikita V. Youshchenko" <yoush@cs.msu.su>,
linuxpps@ml.enneenne.com,
Andrew Morton <akpm@linux-foundation.org>,
Tejun Heo <tj@kernel.org>
Subject: Re: [PATCHv4 06/17] pps: convert printk/pr_* to dev_*
Date: Sat, 20 Nov 2010 16:49:40 +0100 [thread overview]
Message-ID: <20101120154940.GY13356@enneenne.com> (raw)
In-Reply-To: <ce13c55a2acb39be7ef5667467c765595df19835.1290087479.git.lasaine@lvk.cs.msu.su>
On Thu, Nov 18, 2010 at 07:00:59PM +0300, Alexander Gordeev wrote:
> Since we now have direct pointers to struct pps_device everywhere it's
> easy to use dev_* functions to print messages instead of plain printks.
> Where dev_* cannot be used printks are converted to pr_*.
>
> Signed-off-by: Alexander Gordeev <lasaine@lvk.cs.msu.su>
> ---
> drivers/pps/clients/pps-ktimer.c | 5 +++--
> drivers/pps/clients/pps-ldisc.c | 2 ++
> drivers/pps/kapi.c | 15 ++++++++-------
> drivers/pps/pps.c | 25 +++++++++++++------------
> 4 files changed, 26 insertions(+), 21 deletions(-)
>
> diff --git a/drivers/pps/clients/pps-ktimer.c b/drivers/pps/clients/pps-ktimer.c
> index 966ead1..2728469 100644
> --- a/drivers/pps/clients/pps-ktimer.c
> +++ b/drivers/pps/clients/pps-ktimer.c
> @@ -19,6 +19,7 @@
> * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
> */
>
> +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
Instead of redefining pr_fmt() in each file you may do it once into
include file pps_kernel.h...
> #include <linux/kernel.h>
> #include <linux/module.h>
> @@ -45,7 +46,7 @@ static void pps_ktimer_event(unsigned long ptr)
> /* First of all we get the time stamp... */
> pps_get_ts(&ts);
>
> - pr_info("PPS event at %lu\n", jiffies);
> + dev_info(pps->dev, "PPS event at %lu\n", jiffies);
>
> pps_event(pps, &ts, PPS_CAPTUREASSERT, NULL);
>
> @@ -94,7 +95,7 @@ static int __init pps_ktimer_init(void)
> pps = pps_register_source(&pps_ktimer_info,
> PPS_CAPTUREASSERT | PPS_OFFSETASSERT);
> if (pps == NULL) {
> - printk(KERN_ERR "cannot register ktimer source\n");
> + pr_err("cannot register PPS source\n");
> return -ENOMEM;
> }
>
> diff --git a/drivers/pps/clients/pps-ldisc.c b/drivers/pps/clients/pps-ldisc.c
> index 1950b15..30789fa 100644
> --- a/drivers/pps/clients/pps-ldisc.c
> +++ b/drivers/pps/clients/pps-ldisc.c
> @@ -19,6 +19,8 @@
> * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
> */
>
> +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
> +
> #include <linux/module.h>
> #include <linux/serial_core.h>
> #include <linux/tty.h>
> diff --git a/drivers/pps/kapi.c b/drivers/pps/kapi.c
> index 98d4012..fbb89c6 100644
> --- a/drivers/pps/kapi.c
> +++ b/drivers/pps/kapi.c
> @@ -19,6 +19,7 @@
> * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
> */
>
> +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
>
> #include <linux/kernel.h>
> #include <linux/module.h>
> @@ -80,20 +81,20 @@ struct pps_device *pps_register_source(struct pps_source_info *info,
>
> /* Sanity checks */
> if ((info->mode & default_params) != default_params) {
> - printk(KERN_ERR "pps: %s: unsupported default parameters\n",
> + pr_err("pps: %s: unsupported default parameters\n",
> info->name);
> err = -EINVAL;
> goto pps_register_source_exit;
> }
> if ((info->mode & (PPS_ECHOASSERT | PPS_ECHOCLEAR)) != 0 &&
> info->echo == NULL) {
> - printk(KERN_ERR "pps: %s: echo function is not defined\n",
> + pr_err("pps: %s: echo function is not defined\n",
> info->name);
> err = -EINVAL;
> goto pps_register_source_exit;
> }
> if ((info->mode & (PPS_TSFMT_TSPEC | PPS_TSFMT_NTPFP)) == 0) {
> - printk(KERN_ERR "pps: %s: unspecified time format\n",
> + pr_err("pps: %s: unspecified time format\n",
> info->name);
> err = -EINVAL;
> goto pps_register_source_exit;
> @@ -138,7 +139,7 @@ struct pps_device *pps_register_source(struct pps_source_info *info,
> if (id >= PPS_MAX_SOURCES) {
> spin_unlock_irq(&pps_idr_lock);
>
> - printk(KERN_ERR "pps: %s: too many PPS sources in the system\n",
> + pr_err("pps: %s: too many PPS sources in the system\n",
> info->name);
> err = -EBUSY;
> goto free_idr;
> @@ -150,12 +151,12 @@ struct pps_device *pps_register_source(struct pps_source_info *info,
> /* Create the char device */
> err = pps_register_cdev(pps);
> if (err < 0) {
> - printk(KERN_ERR "pps: %s: unable to create char device\n",
> + pr_err("%s: unable to create char device\n",
> info->name);
> goto free_idr;
> }
>
> - pr_info("new PPS source %s at ID %d\n", info->name, id);
> + dev_info(pps->dev, "new PPS source %s\n", info->name);
>
> return pps;
>
> @@ -168,7 +169,7 @@ kfree_pps:
> kfree(pps);
>
> pps_register_source_exit:
> - printk(KERN_ERR "pps: %s: unable to register source\n", info->name);
> + pr_err("%s: unable to register source\n", info->name);
>
> return NULL;
> }
> diff --git a/drivers/pps/pps.c b/drivers/pps/pps.c
> index 1922f27..9f7c2e8 100644
> --- a/drivers/pps/pps.c
> +++ b/drivers/pps/pps.c
> @@ -19,6 +19,7 @@
> * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
> */
>
> +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
>
> #include <linux/kernel.h>
> #include <linux/module.h>
> @@ -67,7 +68,7 @@ static long pps_cdev_ioctl(struct file *file,
>
> switch (cmd) {
> case PPS_GETPARAMS:
> - pr_debug("PPS_GETPARAMS: source %d\n", pps->id);
> + dev_dbg(pps->dev, "PPS_GETPARAMS\n");
>
> spin_lock_irq(&pps->lock);
>
> @@ -83,7 +84,7 @@ static long pps_cdev_ioctl(struct file *file,
> break;
>
> case PPS_SETPARAMS:
> - pr_debug("PPS_SETPARAMS: source %d\n", pps->id);
> + dev_dbg(pps->dev, "PPS_SETPARAMS\n");
>
> /* Check the capabilities */
> if (!capable(CAP_SYS_TIME))
> @@ -93,14 +94,14 @@ static long pps_cdev_ioctl(struct file *file,
> if (err)
> return -EFAULT;
> if (!(params.mode & (PPS_CAPTUREASSERT | PPS_CAPTURECLEAR))) {
> - pr_debug("capture mode unspecified (%x)\n",
> + dev_dbg(pps->dev, "capture mode unspecified (%x)\n",
> params.mode);
> return -EINVAL;
> }
>
> /* Check for supported capabilities */
> if ((params.mode & ~pps->info.mode) != 0) {
> - pr_debug("unsupported capabilities (%x)\n",
> + dev_dbg(pps->dev, "unsupported capabilities (%x)\n",
> params.mode);
> return -EINVAL;
> }
> @@ -113,7 +114,7 @@ static long pps_cdev_ioctl(struct file *file,
> /* Restore the read only parameters */
> if ((params.mode & (PPS_TSFMT_TSPEC | PPS_TSFMT_NTPFP)) == 0) {
> /* section 3.3 of RFC 2783 interpreted */
> - pr_debug("time format unspecified (%x)\n",
> + dev_dbg(pps->dev, "time format unspecified (%x)\n",
> params.mode);
> pps->params.mode |= PPS_TSFMT_TSPEC;
> }
> @@ -126,7 +127,7 @@ static long pps_cdev_ioctl(struct file *file,
> break;
>
> case PPS_GETCAP:
> - pr_debug("PPS_GETCAP: source %d\n", pps->id);
> + dev_dbg(pps->dev, "PPS_GETCAP\n");
>
> err = put_user(pps->info.mode, iuarg);
> if (err)
> @@ -138,7 +139,7 @@ static long pps_cdev_ioctl(struct file *file,
> struct pps_fdata fdata;
> unsigned int ev;
>
> - pr_debug("PPS_FETCH: source %d\n", pps->id);
> + dev_dbg(pps->dev, "PPS_FETCH\n");
>
> err = copy_from_user(&fdata, uarg, sizeof(struct pps_fdata));
> if (err)
> @@ -153,7 +154,7 @@ static long pps_cdev_ioctl(struct file *file,
> else {
> unsigned long ticks;
>
> - pr_debug("timeout %lld.%09d\n",
> + dev_dbg(pps->dev, "timeout %lld.%09d\n",
> (long long) fdata.timeout.sec,
> fdata.timeout.nsec);
> ticks = fdata.timeout.sec * HZ;
> @@ -171,7 +172,7 @@ static long pps_cdev_ioctl(struct file *file,
>
> /* Check for pending signals */
> if (err == -ERESTARTSYS) {
> - pr_debug("pending signal caught\n");
> + dev_dbg(pps->dev, "pending signal caught\n");
> return -EINTR;
> }
>
> @@ -240,7 +241,7 @@ int pps_register_cdev(struct pps_device *pps)
>
> err = cdev_add(&pps->cdev, devt, 1);
> if (err) {
> - printk(KERN_ERR "pps: %s: failed to add char device %d:%d\n",
> + pr_err("%s: failed to add char device %d:%d\n",
> pps->info.name, MAJOR(pps_devt), pps->id);
> return err;
> }
> @@ -282,14 +283,14 @@ static int __init pps_init(void)
>
> pps_class = class_create(THIS_MODULE, "pps");
> if (!pps_class) {
> - printk(KERN_ERR "pps: failed to allocate class\n");
> + pr_err("failed to allocate class\n");
> return -ENOMEM;
> }
> pps_class->dev_attrs = pps_attrs;
>
> err = alloc_chrdev_region(&pps_devt, 0, PPS_MAX_SOURCES, "pps");
> if (err < 0) {
> - printk(KERN_ERR "pps: failed to allocate char device region\n");
> + pr_err("failed to allocate char device region\n");
> goto remove_class;
> }
>
> --
> 1.7.2.3
>
Acked-by: Rodolfo Giometti <giometti@linux.it>
--
GNU/Linux Solutions e-mail: giometti@enneenne.com
Linux Device Driver giometti@linux.it
Embedded Systems phone: +39 349 2432127
UNIX programming skype: rodolfo.giometti
Freelance ICT Italia - Consulente ICT Italia - www.consulenti-ict.it
next prev parent reply other threads:[~2010-11-20 15:49 UTC|newest]
Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-11-18 16:00 [PATCHv4 00/17] pps: several fixes and improvements Alexander Gordeev
2010-11-18 16:00 ` [PATCHv4 01/17] pps: trivial fixes Alexander Gordeev
2010-11-18 16:00 ` [PATCHv4 02/17] pps: declare variables where they are used in switch Alexander Gordeev
2010-11-18 16:00 ` [PATCHv4 03/17] pps: fix race in PPS_FETCH handler Alexander Gordeev
2010-11-20 15:20 ` Rodolfo Giometti
2010-11-18 16:00 ` [PATCHv4 04/17] pps: unify timestamp gathering Alexander Gordeev
2010-11-18 16:00 ` [PATCHv4 05/17] pps: access pps device by direct pointer Alexander Gordeev
2010-11-20 15:44 ` Rodolfo Giometti
2010-11-20 22:33 ` Alexander Gordeev
2010-11-21 8:26 ` Rodolfo Giometti
2010-11-22 15:01 ` Alexander Gordeev
2010-11-21 14:12 ` Alan Cox
2010-11-22 14:55 ` Alexander Gordeev
2010-11-18 16:00 ` [PATCHv4 06/17] pps: convert printk/pr_* to dev_* Alexander Gordeev
2010-11-20 15:49 ` Rodolfo Giometti [this message]
2010-11-20 21:33 ` Alexander Gordeev
2010-11-20 21:42 ` Joe Perches
2010-11-20 22:38 ` Alexander Gordeev
2010-11-21 8:19 ` Rodolfo Giometti
2010-11-18 16:01 ` [PATCHv4 07/17] pps: move idr stuff to pps.c Alexander Gordeev
2010-11-20 15:51 ` Rodolfo Giometti
2010-11-18 16:01 ` [PATCHv4 08/17] pps: add async PPS event handler Alexander Gordeev
2010-11-20 16:08 ` Rodolfo Giometti
2010-11-20 23:23 ` Alexander Gordeev
2010-11-21 8:37 ` Rodolfo Giometti
2010-11-18 16:01 ` [PATCHv4 09/17] pps: don't disable interrupts when using spin locks Alexander Gordeev
2010-11-20 16:09 ` Rodolfo Giometti
2010-11-18 16:01 ` [PATCHv4 10/17] pps: use BUG_ON for kernel API safety checks Alexander Gordeev
2010-11-20 16:13 ` Rodolfo Giometti
2010-11-20 17:01 ` Joe Perches
2010-11-20 18:30 ` Rodolfo Giometti
2010-11-21 0:40 ` Alexander Gordeev
2010-11-21 1:18 ` Joe Perches
2010-11-21 8:42 ` Rodolfo Giometti
2010-11-21 0:13 ` Alexander Gordeev
2010-11-21 8:41 ` Rodolfo Giometti
2010-11-18 16:01 ` [PATCHv4 11/17] pps: simplify conditions a bit Alexander Gordeev
2010-11-20 16:15 ` Rodolfo Giometti
2010-11-18 16:01 ` [PATCHv4 12/17] pps: timestamp is always passed to dcd_change() Alexander Gordeev
2010-11-20 16:23 ` Rodolfo Giometti
2010-11-21 0:44 ` Alexander Gordeev
2010-11-21 8:42 ` Rodolfo Giometti
2010-11-18 16:01 ` [PATCHv4 13/17] ntp: add hardpps implementation Alexander Gordeev
2010-11-20 16:27 ` Rodolfo Giometti
2010-11-21 1:05 ` Alexander Gordeev
2010-11-18 16:01 ` [PATCHv4 14/17] pps: capture MONOTONIC_RAW timestamps as well Alexander Gordeev
2010-11-18 19:42 ` john stultz
2010-11-18 16:01 ` [PATCHv4 15/17] pps: add kernel consumer support Alexander Gordeev
2010-11-18 16:01 ` [PATCHv4 16/17] pps: add parallel port PPS client Alexander Gordeev
2010-11-18 16:01 ` [PATCHv4 17/17] pps: add parallel port PPS signal generator Alexander Gordeev
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=20101120154940.GY13356@enneenne.com \
--to=giometti@enneenne.com \
--cc=akpm@linux-foundation.org \
--cc=lasaine@lvk.cs.msu.su \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxpps@ml.enneenne.com \
--cc=tj@kernel.org \
--cc=yoush@cs.msu.su \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox