From: Takashi Iwai <tiwai@suse.de>
To: Mauro Carvalho Chehab <mchehab@s-opensource.com>
Cc: alsa-devel@alsa-project.org, Jonathan Corbet <corbet@lwn.net>,
Linux Doc Mailing List <linux-doc@vger.kernel.org>,
linux-kernel@vger.kernel.org,
Mauro Carvalho Chehab <mchehab@infradead.org>
Subject: Re: [PATCH 10/13] sound: fix the comments that refers to kernel-doc
Date: Tue, 16 May 2017 12:50:55 +0200 [thread overview]
Message-ID: <s5hvap1f52o.wl-tiwai@suse.de> (raw)
In-Reply-To: <20170516074653.187d499f@vento.lan>
On Tue, 16 May 2017 12:46:53 +0200,
Mauro Carvalho Chehab wrote:
>
> Em Mon, 15 May 2017 11:04:26 +0200
> Takashi Iwai <tiwai@suse.de> escreveu:
>
> > On Sun, 14 May 2017 17:38:44 +0200,
> > Mauro Carvalho Chehab wrote:
> > >
> > > The markup inside the #if 0 comment actually refers to a
> > > kernel-doc markup. As we're getting rid of DocBook update it.
> > >
> > > Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
> >
> > I guess you prefer taking it from your tree? Feel free to take my
> > ack:
> > Reviewed-by: Takashi Iwai <tiwai@suse.de>
>
> In the specific case of this patch, it doesn't matter much from what
> tree it would be merged, as no other patch depends on it ;)
>
> So, if you prefer, you can merge it directly on your tree. Otherwise,
> I'll keep it on my pile of patches to be send to docs -next.
OK, since the PCM stuff is being changed for 4.13, may be it's safer
that I take this patch in sound git tree.
thanks,
Takashi
>
>
> >
> >
> > thanks,
> >
> > Takashi
> >
> > > ---
> > > include/sound/pcm.h | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/include/sound/pcm.h b/include/sound/pcm.h
> > > index 361749e60799..bbf97d4c4c17 100644
> > > --- a/include/sound/pcm.h
> > > +++ b/include/sound/pcm.h
> > > @@ -1054,7 +1054,7 @@ int snd_pcm_format_unsigned(snd_pcm_format_t format);
> > > int snd_pcm_format_linear(snd_pcm_format_t format);
> > > int snd_pcm_format_little_endian(snd_pcm_format_t format);
> > > int snd_pcm_format_big_endian(snd_pcm_format_t format);
> > > -#if 0 /* just for DocBook */
> > > +#if 0 /* just for kernel-doc */
> > > /**
> > > * snd_pcm_format_cpu_endian - Check the PCM format is CPU-endian
> > > * @format: the format to check
> > > --
> > > 2.9.3
> > >
> > >
>
>
>
> Thanks,
> Mauro
>
WARNING: multiple messages have this Message-ID (diff)
From: Takashi Iwai <tiwai@suse.de>
To: Mauro Carvalho Chehab <mchehab@s-opensource.com>
Cc: "Linux Doc Mailing List" <linux-doc@vger.kernel.org>,
<alsa-devel@alsa-project.org>,
"Mauro Carvalho Chehab" <mchehab@infradead.org>,
"Jonathan Corbet" <corbet@lwn.net>,
"Jaroslav Kysela" <perex@perex.cz>,
<linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 10/13] sound: fix the comments that refers to kernel-doc
Date: Tue, 16 May 2017 12:50:55 +0200 [thread overview]
Message-ID: <s5hvap1f52o.wl-tiwai@suse.de> (raw)
In-Reply-To: <20170516074653.187d499f@vento.lan>
On Tue, 16 May 2017 12:46:53 +0200,
Mauro Carvalho Chehab wrote:
>
> Em Mon, 15 May 2017 11:04:26 +0200
> Takashi Iwai <tiwai@suse.de> escreveu:
>
> > On Sun, 14 May 2017 17:38:44 +0200,
> > Mauro Carvalho Chehab wrote:
> > >
> > > The markup inside the #if 0 comment actually refers to a
> > > kernel-doc markup. As we're getting rid of DocBook update it.
> > >
> > > Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
> >
> > I guess you prefer taking it from your tree? Feel free to take my
> > ack:
> > Reviewed-by: Takashi Iwai <tiwai@suse.de>
>
> In the specific case of this patch, it doesn't matter much from what
> tree it would be merged, as no other patch depends on it ;)
>
> So, if you prefer, you can merge it directly on your tree. Otherwise,
> I'll keep it on my pile of patches to be send to docs -next.
OK, since the PCM stuff is being changed for 4.13, may be it's safer
that I take this patch in sound git tree.
thanks,
Takashi
>
>
> >
> >
> > thanks,
> >
> > Takashi
> >
> > > ---
> > > include/sound/pcm.h | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/include/sound/pcm.h b/include/sound/pcm.h
> > > index 361749e60799..bbf97d4c4c17 100644
> > > --- a/include/sound/pcm.h
> > > +++ b/include/sound/pcm.h
> > > @@ -1054,7 +1054,7 @@ int snd_pcm_format_unsigned(snd_pcm_format_t format);
> > > int snd_pcm_format_linear(snd_pcm_format_t format);
> > > int snd_pcm_format_little_endian(snd_pcm_format_t format);
> > > int snd_pcm_format_big_endian(snd_pcm_format_t format);
> > > -#if 0 /* just for DocBook */
> > > +#if 0 /* just for kernel-doc */
> > > /**
> > > * snd_pcm_format_cpu_endian - Check the PCM format is CPU-endian
> > > * @format: the format to check
> > > --
> > > 2.9.3
> > >
> > >
>
>
>
> Thanks,
> Mauro
>
next prev parent reply other threads:[~2017-05-16 10:50 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-05-14 15:38 [PATCH 00/13] Get rid of DocBook Mauro Carvalho Chehab
2017-05-14 15:38 ` [PATCH 01/13] docs-rst: convert lsm from DocBook to ReST Mauro Carvalho Chehab
2017-05-14 15:38 ` [PATCH 02/13] docs: remove DocBook from the building system Mauro Carvalho Chehab
2017-05-14 15:38 ` [PATCH 03/13] docs: update old references for DocBook from the documentation Mauro Carvalho Chehab
2017-05-14 15:38 ` Mauro Carvalho Chehab
2017-05-14 15:38 ` Mauro Carvalho Chehab
2017-05-15 2:07 ` SeongJae Park
2017-05-15 2:07 ` SeongJae Park
2017-05-15 2:07 ` SeongJae Park
2017-05-14 15:38 ` [PATCH 04/13] MAINTAINERS: update old references for DocBook directory Mauro Carvalho Chehab
2017-05-14 15:38 ` [PATCH 05/13] ata: update references for libata documentation Mauro Carvalho Chehab
2017-05-14 15:38 ` [PATCH 06/13] ia64, scsi: update references for the device-io book Mauro Carvalho Chehab
2017-05-14 15:38 ` Mauro Carvalho Chehab
2017-05-14 15:38 ` [PATCH 07/13] irq: update genericirq book location Mauro Carvalho Chehab
2017-05-14 15:38 ` [PATCH 08/13] fs: update location of filesystems documentation Mauro Carvalho Chehab
2017-05-14 15:38 ` [PATCH 09/13] lib: update location of kgdb documentation Mauro Carvalho Chehab
2017-05-14 15:38 ` [PATCH 10/13] sound: fix the comments that refers to kernel-doc Mauro Carvalho Chehab
2017-05-15 9:04 ` Takashi Iwai
2017-05-15 9:04 ` Takashi Iwai
2017-05-16 10:46 ` Mauro Carvalho Chehab
2017-05-16 10:46 ` Mauro Carvalho Chehab
2017-05-16 10:50 ` Takashi Iwai [this message]
2017-05-16 10:50 ` Takashi Iwai
2017-05-16 10:55 ` Mauro Carvalho Chehab
2017-05-16 10:55 ` Mauro Carvalho Chehab
2017-05-14 15:38 ` [PATCH 11/13] fs: fix the location of the kernel-api book Mauro Carvalho Chehab
2017-05-14 15:38 ` [PATCH 12/13] usb: fix the comment with regards to DocBook Mauro Carvalho Chehab
2017-05-14 15:38 ` [PATCH 13/13] docs-rst: get rid of Documentation/sphinx/tmplcvt script Mauro Carvalho Chehab
2017-05-14 21:05 ` [PATCH 00/13] Get rid of DocBook Kees Cook
2017-05-14 22:00 ` Mauro Carvalho Chehab
2017-05-15 14:34 ` Kees Cook
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=s5hvap1f52o.wl-tiwai@suse.de \
--to=tiwai@suse.de \
--cc=alsa-devel@alsa-project.org \
--cc=corbet@lwn.net \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mchehab@infradead.org \
--cc=mchehab@s-opensource.com \
/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.