From: Vinod Koul <vkoul@kernel.org>
To: Nathan Chancellor <natechancellor@gmail.com>
Cc: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>,
Sanyog Kale <sanyog.r.kale@intel.com>,
clang-built-linux@googlegroups.com, alsa-devel@alsa-project.org,
linux-kernel@vger.kernel.org
Subject: Re: [alsa-devel] [PATCH] soundwire: Don't build sound.o without CONFIG_ACPI
Date: Wed, 14 Aug 2019 11:16:04 +0530 [thread overview]
Message-ID: <20190814054604.GW12733@vkoul-mobl.Dlink> (raw)
In-Reply-To: <20190814042428.GA125416@archlinux-threadripper>
On 13-08-19, 21:24, Nathan Chancellor wrote:
> On Wed, Aug 14, 2019 at 09:29:47AM +0530, Vinod Koul wrote:
> > On 13-08-19, 09:22, Pierre-Louis Bossart wrote:
> > > On 8/13/19 1:10 AM, Nathan Chancellor wrote:
> > > I am fine with the change, but we might as well rename the file acpi_slave.c
> > > then?
> >
> > Srini's change add support for DT for the same file, so It does not make
> > sense to rename. Yes this patch tries to fix a warn which is there due
> > to DT being not supported but with Srini's patches this warn should go
> > away as sdw_slave_add() will be invoked by the DT counterpart
> >
> > Sorry Nathan, we would have to live with the warn for few more days till
> > I apply Srini's changes. So I am not taking this (or v2) patch
> >
>
> That is fine as I can apply this locally. Could you point me to these
> patches so that I can take a look at them?
Here you go:
https://lore.kernel.org/lkml/20190808144504.24823-3-srinivas.kandagatla@linaro.org/
--
~Vinod
next prev parent reply other threads:[~2019-08-14 5:47 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-13 6:10 [PATCH] soundwire: Don't build sound.o without CONFIG_ACPI Nathan Chancellor
2019-08-13 14:22 ` [alsa-devel] " Pierre-Louis Bossart
2019-08-13 17:39 ` Nathan Chancellor
2019-08-14 3:59 ` Vinod Koul
2019-08-14 4:24 ` Nathan Chancellor
2019-08-14 5:46 ` Vinod Koul [this message]
2019-08-13 18:09 ` [PATCH v2] soundwire: Make slave.o depend on ACPI and rename to acpi_slave.o Nathan Chancellor
2019-08-13 20:37 ` Nick Desaulniers
2019-08-13 20:41 ` [alsa-devel] " Pierre-Louis Bossart
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=20190814054604.GW12733@vkoul-mobl.Dlink \
--to=vkoul@kernel.org \
--cc=alsa-devel@alsa-project.org \
--cc=clang-built-linux@googlegroups.com \
--cc=linux-kernel@vger.kernel.org \
--cc=natechancellor@gmail.com \
--cc=pierre-louis.bossart@linux.intel.com \
--cc=sanyog.r.kale@intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).