All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sudeep Holla <sudeep.holla@arm.com>
To: Heiner Kallweit <hkallweit1@gmail.com>
Cc: Stephen Boyd <sboyd@codeaurora.org>,
	linux-clk@vger.kernel.org,
	kbuild test robot <fengguang.wu@intel.com>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH] clk: fix compile testing of scpi clk driver
Date: Mon, 4 Dec 2017 02:25:35 +0000	[thread overview]
Message-ID: <20171204022535.GE6870@e107533-lin> (raw)
In-Reply-To: <d01ed8fa-9f9e-adf6-3133-800433fef7bb@gmail.com>

On Wed, Nov 29, 2017 at 09:37:46PM +0100, Heiner Kallweit wrote:
> Am 29.11.2017 um 18:04 schrieb Stephen Boyd:
> > On 11/28, Heiner Kallweit wrote:
> >> We now have a hard dependency on ARM_SCPI_PROTOCOL and may end up with
> >> the following error:
> >> ERROR: "get_scpi_ops" [drivers/clk/clk-scpi.ko] undefined!
> >> Fix this by removing COMPILE_TEST from COMMON_CLK_SCPI dependencies.
> >>
> >> Fixes: f7fee8be06a6 "firmware: arm_scpi: remove default for get_scpi_ops if not reachable"
> > 
> > I don't see this commit anywhere. Is this a future issue?
> > 
> Sorry, yes it is and the patch was primarily addressed to Sudeep. See here:
> https://git.kernel.org/pub/scm/linux/kernel/git/sudeep.holla/linux.git/log/?h=for-next/scpi

I was trying to figure the build regression that the above commit would
cause as I could not find it myself but I was sure about the regression.

If I had pulled it officially ;), I would have responded to your original
patch. Sorry for the name of the brnach, I should have used something else
to avoid the confusion.

--
Regards,
Sudeep


WARNING: multiple messages have this Message-ID (diff)
From: sudeep.holla@arm.com (Sudeep Holla)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] clk: fix compile testing of scpi clk driver
Date: Mon, 4 Dec 2017 02:25:35 +0000	[thread overview]
Message-ID: <20171204022535.GE6870@e107533-lin> (raw)
In-Reply-To: <d01ed8fa-9f9e-adf6-3133-800433fef7bb@gmail.com>

On Wed, Nov 29, 2017 at 09:37:46PM +0100, Heiner Kallweit wrote:
> Am 29.11.2017 um 18:04 schrieb Stephen Boyd:
> > On 11/28, Heiner Kallweit wrote:
> >> We now have a hard dependency on ARM_SCPI_PROTOCOL and may end up with
> >> the following error:
> >> ERROR: "get_scpi_ops" [drivers/clk/clk-scpi.ko] undefined!
> >> Fix this by removing COMPILE_TEST from COMMON_CLK_SCPI dependencies.
> >>
> >> Fixes: f7fee8be06a6 "firmware: arm_scpi: remove default for get_scpi_ops if not reachable"
> > 
> > I don't see this commit anywhere. Is this a future issue?
> > 
> Sorry, yes it is and the patch was primarily addressed to Sudeep. See here:
> https://git.kernel.org/pub/scm/linux/kernel/git/sudeep.holla/linux.git/log/?h=for-next/scpi

I was trying to figure the build regression that the above commit would
cause as I could not find it myself but I was sure about the regression.

If I had pulled it officially ;), I would have responded to your original
patch. Sorry for the name of the brnach, I should have used something else
to avoid the confusion.

--
Regards,
Sudeep

  reply	other threads:[~2017-12-04  2:25 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-28  6:58 [PATCH] clk: fix compile testing of scpi clk driver Heiner Kallweit
2017-11-28  6:58 ` Heiner Kallweit
2017-11-29 17:04 ` Stephen Boyd
2017-11-29 17:04   ` Stephen Boyd
2017-11-29 20:37   ` Heiner Kallweit
2017-11-29 20:37     ` Heiner Kallweit
2017-12-04  2:25     ` Sudeep Holla [this message]
2017-12-04  2:25       ` Sudeep Holla
2017-12-04  2:25 ` Sudeep Holla
2017-12-04  2:25   ` Sudeep Holla

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=20171204022535.GE6870@e107533-lin \
    --to=sudeep.holla@arm.com \
    --cc=fengguang.wu@intel.com \
    --cc=hkallweit1@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-clk@vger.kernel.org \
    --cc=sboyd@codeaurora.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.