All of lore.kernel.org
 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
>

WARNING: multiple messages have this Message-ID (diff)
From: srinivas.kandagatla@linaro.org (Srinivas Kandagatla)
To: linux-arm-kernel@lists.infradead.org
Subject: [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:22 UTC|newest]

Thread overview: 48+ 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:29 ` Srinivas Kandagatla
2015-09-18 12:29 ` 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   ` Srinivas Kandagatla
2015-09-18 12:30 ` [PATCH 02/14] ARM: dts: ifc6410: remove unnecessary eeprom label Srinivas Kandagatla
2015-09-18 12:30   ` Srinivas Kandagatla
2015-09-18 12:30 ` [PATCH 03/14] ARM: dts: qs600: " Srinivas Kandagatla
2015-09-18 12:30   ` Srinivas Kandagatla
2015-09-18 12:31 ` [PATCH 04/14] devicetree: Add hardware rng entry to qcom-apq8064.dtsi Srinivas Kandagatla
2015-09-18 12:31   ` Srinivas Kandagatla
     [not found]   ` <1442579472-20102-1-git-send-email-srinivas.kandagatla-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2015-10-26 21:46     ` Bjorn Andersson
2015-10-26 21:46       ` Bjorn Andersson
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   ` Srinivas Kandagatla
2015-09-18 12:31 ` [PATCH 06/14] ARM: dts: apq8064: add pm8921 pwrkey support Srinivas Kandagatla
2015-09-18 12:31   ` 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   ` 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   ` Srinivas Kandagatla
2015-09-18 12:31 ` [PATCH 09/14] ARM: dts: ifc6410: Add missing pinctrl to gsbi7 uart Srinivas Kandagatla
2015-09-18 12:31   ` Srinivas Kandagatla
2015-09-25 18:54   ` Kevin Hilman
2015-09-25 18:54     ` Kevin Hilman
2015-09-25 18:54     ` Kevin Hilman
2015-09-30 13:22     ` Srinivas Kandagatla [this message]
2015-09-30 13:22       ` Srinivas Kandagatla
2015-09-30 13:22       ` Srinivas Kandagatla
2015-09-18 12:31 ` [PATCH 10/14] ARM: dts: qs600: Add missing pinctrl property for " Srinivas Kandagatla
2015-09-18 12:31   ` Srinivas Kandagatla
2015-09-21  6:26   ` Igor Grinberg
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:31   ` Srinivas Kandagatla
2015-09-18 12:32 ` [PATCH 12/14] ARM: dts: qs600: add pwrseq support to WLAN Srinivas Kandagatla
2015-09-18 12:32   ` Srinivas Kandagatla
2015-09-21  6:27   ` Igor Grinberg
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   ` Srinivas Kandagatla
2015-09-18 12:32 ` [PATCH 14/14] ARM: dts: qs600: Add SD card detect support Srinivas Kandagatla
2015-09-18 12:32   ` Srinivas Kandagatla
2015-09-21  6:31   ` Igor Grinberg
2015-09-21  6:31     ` Igor Grinberg
2015-09-20  2:52 ` [PATCH 00/14] ARM: dts: apq8064 dt cleanups and additions Andy Gross
2015-09-20  2:52   ` 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 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.