Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
In-Reply-To: <20180213124810.oyf4ec5bpbnsnkqy@earth.universe>
References: <7655f39d44e804c9ed3209db475567abc250f642.1518169282.git.baolin.wang@linaro.org>
 <20180209123044.vjvkb3yfmk4gqfiv@earth.universe> <20180213124810.oyf4ec5bpbnsnkqy@earth.universe>
From:   Baolin Wang <baolin.wang@linaro.org>
Date:   Wed, 14 Feb 2018 10:14:46 +0800
Message-ID: <CAMz4ku+z6PBKQWfadhpi2yGeeM+=ZnPVAbdKc5+3XyvUvAZzAQ@mail.gmail.com>
Subject: Re: [PATCH v2] power: reset: Add Spreadtrum SC27xx PMIC power off support
To:     Sebastian Reichel <sre@kernel.org>
Cc:     Arnd Bergmann <arnd@arndb.de>, Mark Brown <broonie@kernel.org>,
        LKML <linux-kernel@vger.kernel.org>,
        Linux PM list <linux-pm@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Hi Sebastian,

On 13 February 2018 at 20:48, Sebastian Reichel <sre@kernel.org> wrote:
> Hi Baolin,
>
> On Fri, Feb 09, 2018 at 01:30:44PM +0100, Sebastian Reichel wrote:
>> On Fri, Feb 09, 2018 at 05:43:38PM +0800, Baolin Wang wrote:
>> > On Spreadtrum platform, we need power off system through external SC27xx
>> > series PMICs including the SC2720, SC2721, SC2723, SC2730 and SC2731 chips.
>> > Thus this patch adds SC27xx series PMICs power-off support.
>> >
>> > Signed-off-by: Baolin Wang <baolin.wang@linaro.org>
>>
>> Thanks, queued. My public -next branch is waiting for 4.16-rc1,
>> though.
>
> I just dropped this patch. It does not compile as a module:
>
> ERROR: "freeze_secondary_cpus" [drivers/power/reset/sc27xx-poweroff.ko] undefined!
>
> There is no EXPORT_SYMBOL for freeze_secondary_cpus, so you either
> need to add that or make the new driver built-in only.

I am sorry for the building error, and I will  make the driver
built-in only in next version. Thanks.

-- 
Baolin.wang
Best Regards