linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
To: Kevin Hilman <khilman@kernel.org>
Cc: Andy Gross <agross@codeaurora.org>,
	linux-arm-msm <linux-arm-msm@vger.kernel.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>,
	Russell King <linux@arm.linux.org.uk>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	lkml <linux-kernel@vger.kernel.org>,
	Tyler Baker <tyler.baker@linaro.org>,
	Olof Johansson <olof@lixom.net>,
	Nicolas Dechesne <nicolas.dechesne@linaro.org>
Subject: Re: [PATCH 09/14] ARM: dts: ifc6410: Add missing pinctrl to gsbi7 uart
Date: Wed, 30 Sep 2015 14:22:39 +0100	[thread overview]
Message-ID: <560BE21F.8090901@linaro.org> (raw)
In-Reply-To: <CAMAWPa_7U-PBwsngU+qc5SAt4HcCvPwUGvGY9nPUu+pZtPA7Uw@mail.gmail.com>

Hi Kevin,

Thanks for reporting this issue.

On 25/09/15 19:54, Kevin Hilman wrote:
> Hi Srini,
>
> On Fri, Sep 18, 2015 at 5:31 AM, Srinivas Kandagatla
> <srinivas.kandagatla@linaro.org> wrote:
>> This patch adds missing 2pin uart pinctrl property to gsbi7 uart on
>> IFC6410.
>>
>> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
>
> kernelci.org found some new boot failures in -next on 8064[1].  I
> bisected it down to this patch, which landed in linux next in the form
> of commit d3a578ed4310 ARM: dts: ifc6410: Add missing pinctrl to gsbi7
> uart.
>
This issue is due to missing CONFIG_PINCTRL_APQ8064 Kconfig in 
multi_v7_defconfig.

In my tests I always enable this config which is why I could not spot 
this breakage.

This patch as it is the right thing to do however we need a fix to 
multi_v7_defconfig. I will send a patch for multi_v7_defconfig to 
address this and add few more missing configs too.


thanks,
--srini
> Kevin
>
> [1] http://kernelci.org/boot/qcom-apq8064-ifc6410/job/next/kernel/next-20150925/defconfig/multi_v7_defconfig/lab/lab-khilman/?_id=5605083c59b51466d26c3250
>

  reply	other threads:[~2015-09-30 13:23 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-18 12:29 [PATCH 00/14] ARM: dts: apq8064 dt cleanups and additions Srinivas Kandagatla
2015-09-18 12:30 ` [PATCH 01/14] ARM: dts: apq8064: remove redundant i2c pinctrl properties Srinivas Kandagatla
2015-09-18 12:30 ` [PATCH 02/14] ARM: dts: ifc6410: remove unnecessary eeprom label Srinivas Kandagatla
2015-09-18 12:30 ` [PATCH 03/14] ARM: dts: qs600: " Srinivas Kandagatla
2015-09-18 12:31 ` [PATCH 04/14] devicetree: Add hardware rng entry to qcom-apq8064.dtsi Srinivas Kandagatla
2015-10-26 21:46   ` Bjorn Andersson
2015-09-18 12:31 ` [PATCH 05/14] ARM: dts: apq8064: add pm8921 rtc Srinivas Kandagatla
2015-09-18 12:31 ` [PATCH 06/14] ARM: dts: apq8064: add pm8921 pwrkey support Srinivas Kandagatla
2015-09-18 12:31 ` [PATCH 07/14] ARM: dts: apq8064: Prefix the gsbi6 uart pins correctly Srinivas Kandagatla
2015-09-18 12:31 ` [PATCH 08/14] ARM: dts: apq8064: add missing gsbi7 uart pinctrl Srinivas Kandagatla
2015-09-18 12:31 ` [PATCH 09/14] ARM: dts: ifc6410: Add missing pinctrl to gsbi7 uart Srinivas Kandagatla
2015-09-25 18:54   ` Kevin Hilman
2015-09-30 13:22     ` Srinivas Kandagatla [this message]
2015-09-18 12:31 ` [PATCH 10/14] ARM: dts: qs600: Add missing pinctrl property for " Srinivas Kandagatla
2015-09-21  6:26   ` Igor Grinberg
2015-09-18 12:31 ` [PATCH 11/14] ARM: dts: ifc6410: Add pwrseq support for WLAN Srinivas Kandagatla
2015-09-18 12:32 ` [PATCH 12/14] ARM: dts: qs600: add pwrseq support to WLAN Srinivas Kandagatla
2015-09-21  6:27   ` Igor Grinberg
2015-09-18 12:32 ` [PATCH 13/14] ARM: dts: apq8064-ifc6410: add notify led support Srinivas Kandagatla
2015-09-18 12:32 ` [PATCH 14/14] ARM: dts: qs600: Add SD card detect support Srinivas Kandagatla
2015-09-21  6:31   ` Igor Grinberg
2015-09-20  2:52 ` [PATCH 00/14] ARM: dts: apq8064 dt cleanups and additions Andy Gross

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=560BE21F.8090901@linaro.org \
    --to=srinivas.kandagatla@linaro.org \
    --cc=agross@codeaurora.org \
    --cc=devicetree@vger.kernel.org \
    --cc=ijc+devicetree@hellion.org.uk \
    --cc=khilman@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=mark.rutland@arm.com \
    --cc=nicolas.dechesne@linaro.org \
    --cc=olof@lixom.net \
    --cc=pawel.moll@arm.com \
    --cc=robh+dt@kernel.org \
    --cc=tyler.baker@linaro.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 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).