From: simran rai <ssimran-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>
To: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>,
kbuild test robot <lkp-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: kbuild-all-JC7UmRfGjtg@public.gmane.org,
Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org>,
Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
Ian Campbell
<ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>,
Kumar Gala <galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>,
Ray Jui <rjui-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>,
Scott Branden <sbranden-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>,
Liam Girdwood <lgirdwood-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
Mark Brown <broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
Jaroslav Kysela <perex-/Fr2/VpizcU@public.gmane.org>,
Takashi Iwai <tiwai-IBi9RG/b67k@public.gmane.org>,
Lori Hikichi <lhikichi-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
bcm-kernel-feedback-list-dY08KVG/lbpWk0Htik3J/w@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Arun Parameswaran <arunp-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>,
alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw@public.gmane.org
Subject: Re: [PATCH 2/2] sound: soc: Add Cygnus audio driver
Date: Fri, 30 Oct 2015 16:36:05 -0700 [thread overview]
Message-ID: <5633FEE5.7050101@broadcom.com> (raw)
In-Reply-To: <3878425.UJmmiCl69s@wuerfel>
Hi Arnd,
On 10/30/2015 4:28 PM, Arnd Bergmann wrote:
> On Saturday 31 October 2015 07:20:17 kbuild test robot wrote:
>> > 785 dev_dbg(rtd->cpu_dai->dev, "%s: size 0x%x @ 0x%p\n",
>> 786 __func__, size, buf->area);
>>
> Both reports are for the same bug: the "0x%x" should be "0x%zx".
>
> There is another bug in the same line: "0x%p" should be "%pK",
> to avoid the duplicate "0x" and to not leak the pointer when
> kptr_restrict is set.
>
> Arnd
I will fix both of these in the next patchset, early next week. If there
are any other comments/suggestions, please let me know. Thanks for the
feedback.
Simran
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: ssimran@broadcom.com (simran rai)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/2] sound: soc: Add Cygnus audio driver
Date: Fri, 30 Oct 2015 16:36:05 -0700 [thread overview]
Message-ID: <5633FEE5.7050101@broadcom.com> (raw)
In-Reply-To: <3878425.UJmmiCl69s@wuerfel>
Hi Arnd,
On 10/30/2015 4:28 PM, Arnd Bergmann wrote:
> On Saturday 31 October 2015 07:20:17 kbuild test robot wrote:
>> > 785 dev_dbg(rtd->cpu_dai->dev, "%s: size 0x%x @ 0x%p\n",
>> 786 __func__, size, buf->area);
>>
> Both reports are for the same bug: the "0x%x" should be "0x%zx".
>
> There is another bug in the same line: "0x%p" should be "%pK",
> to avoid the duplicate "0x" and to not leak the pointer when
> kptr_restrict is set.
>
> Arnd
I will fix both of these in the next patchset, early next week. If there
are any other comments/suggestions, please let me know. Thanks for the
feedback.
Simran
WARNING: multiple messages have this Message-ID (diff)
From: simran rai <ssimran@broadcom.com>
To: Arnd Bergmann <arnd@arndb.de>, kbuild test robot <lkp@intel.com>
Cc: <kbuild-all@01.org>, Rob Herring <robh+dt@kernel.org>,
Pawel Moll <pawel.moll@arm.com>,
Mark Rutland <mark.rutland@arm.com>,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
Kumar Gala <galak@codeaurora.org>, Ray Jui <rjui@broadcom.com>,
Scott Branden <sbranden@broadcom.com>,
Liam Girdwood <lgirdwood@gmail.com>,
Mark Brown <broonie@kernel.org>, Jaroslav Kysela <perex@perex.cz>,
Takashi Iwai <tiwai@suse.com>,
Lori Hikichi <lhikichi@broadcom.com>,
<devicetree@vger.kernel.org>,
<linux-arm-kernel@lists.infradead.org>,
<bcm-kernel-feedback-list@broadcom.com>,
<linux-kernel@vger.kernel.org>,
Arun Parameswaran <arunp@broadcom.com>,
<alsa-devel@alsa-project.org>
Subject: Re: [PATCH 2/2] sound: soc: Add Cygnus audio driver
Date: Fri, 30 Oct 2015 16:36:05 -0700 [thread overview]
Message-ID: <5633FEE5.7050101@broadcom.com> (raw)
In-Reply-To: <3878425.UJmmiCl69s@wuerfel>
Hi Arnd,
On 10/30/2015 4:28 PM, Arnd Bergmann wrote:
> On Saturday 31 October 2015 07:20:17 kbuild test robot wrote:
>> > 785 dev_dbg(rtd->cpu_dai->dev, "%s: size 0x%x @ 0x%p\n",
>> 786 __func__, size, buf->area);
>>
> Both reports are for the same bug: the "0x%x" should be "0x%zx".
>
> There is another bug in the same line: "0x%p" should be "%pK",
> to avoid the duplicate "0x" and to not leak the pointer when
> kptr_restrict is set.
>
> Arnd
I will fix both of these in the next patchset, early next week. If there
are any other comments/suggestions, please let me know. Thanks for the
feedback.
Simran
next prev parent reply other threads:[~2015-10-30 23:36 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-30 18:54 [PATCH 0/2] Add audio support for Broadcom Cygnus SoC Simran Rai
2015-10-30 18:54 ` Simran Rai
2015-10-30 18:54 ` Simran Rai
2015-10-30 18:54 ` [PATCH 1/2] dt-bindings: sound: Add DT bindings for Broadcom Cygnus audio Simran Rai
2015-10-30 18:54 ` Simran Rai
2015-10-30 18:54 ` Simran Rai
2015-10-30 18:54 ` [PATCH 2/2] sound: soc: Add Cygnus audio driver Simran Rai
2015-10-30 18:54 ` Simran Rai
2015-10-30 18:54 ` Simran Rai
2015-10-30 19:59 ` kbuild test robot
2015-10-30 19:59 ` kbuild test robot
2015-10-30 19:59 ` kbuild test robot
[not found] ` <1446231297-10648-3-git-send-email-ssimran-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>
2015-10-30 23:20 ` kbuild test robot
2015-10-30 23:20 ` kbuild test robot
2015-10-30 23:20 ` kbuild test robot
2015-10-30 23:28 ` Arnd Bergmann
2015-10-30 23:28 ` Arnd Bergmann
2015-10-30 23:36 ` simran rai [this message]
2015-10-30 23:36 ` simran rai
2015-10-30 23:36 ` simran rai
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=5633FEE5.7050101@broadcom.com \
--to=ssimran-dy08kvg/lbpwk0htik3j/w@public.gmane.org \
--cc=alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw@public.gmane.org \
--cc=arnd-r2nGTMty4D4@public.gmane.org \
--cc=arunp-dY08KVG/lbpWk0Htik3J/w@public.gmane.org \
--cc=bcm-kernel-feedback-list-dY08KVG/lbpWk0Htik3J/w@public.gmane.org \
--cc=broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \
--cc=ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org \
--cc=kbuild-all-JC7UmRfGjtg@public.gmane.org \
--cc=lgirdwood-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=lhikichi-dY08KVG/lbpWk0Htik3J/w@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=lkp-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
--cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \
--cc=pawel.moll-5wv7dgnIgG8@public.gmane.org \
--cc=perex-/Fr2/VpizcU@public.gmane.org \
--cc=rjui-dY08KVG/lbpWk0Htik3J/w@public.gmane.org \
--cc=robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=sbranden-dY08KVG/lbpWk0Htik3J/w@public.gmane.org \
--cc=tiwai-IBi9RG/b67k@public.gmane.org \
/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.