LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [GIT PULL] qcom SoC changes for v4.1
@ 2015-03-16 21:03 Kumar Gala
  2015-04-02  0:36 ` Olof Johansson
  2015-04-02  8:37 ` Nicolas Dechesne
  0 siblings, 2 replies; 8+ messages in thread
From: Kumar Gala @ 2015-03-16 21:03 UTC (permalink / raw)
  To: arm
  Cc: Olof Johansson, Arnd Bergmann, linux-arm-msm, linux-kernel,
	linux-arm-kernel, agross, lina.iyer

The following changes since commit c517d838eb7d07bbe9507871fab3931deccff539:

   Linux 4.0-rc1 (2015-02-22 18:21:14 -0800)

are available in the git repository at:

   git://git.kernel.org/pub/scm/linux/kernel/git/galak/linux-qcom.git tags/qcom-soc-for-4.1

for you to fetch changes up to e5fdad68d47ed344832b7ca4e18b2e9708d8141e:

   soc: qcom: gsbi: Add support for ADM CRCI muxing (2015-03-11 15:18:39 -0500)

----------------------------------------------------------------
Qualcomm ARM Based SoC Updates for v4.1

* Merged the based Qualcomm SCM and SCM boot support
* Cleaned up SCM interface to only expose functional SCM APIs
* Moved Qualcomm SCM code into drivers/firmware
* Updated the SCM APIs for setting cpu cold and warm boot addresses
* Added support for ADM CRCI muxing

----------------------------------------------------------------
Andy Gross (1):
       soc: qcom: gsbi: Add support for ADM CRCI muxing

Kumar Gala (4):
       ARM: qcom: Merge scm and scm boot code together
       ARM: qcom: Cleanup scm interface to only export what is needed
       ARM: qcom: Prep scm code for move to drivers/firmware
       firmware: qcom: scm: Move the scm driver to drivers/firmware

Lina Iyer (3):
       firmware: qcom: scm: Clean cold boot entry to export only the API
       firmware: qcom: scm: Add qcom_scm_set_warm_boot_addr function
       firmware: qcom: scm: Support cpu power down through SCM

  .../devicetree/bindings/soc/qcom/qcom,gsbi.txt     |  30 +-
  MAINTAINERS                                        |   1 +
  arch/arm/Kconfig                                   |   2 +
  arch/arm/mach-qcom/Kconfig                         |   3 -
  arch/arm/mach-qcom/Makefile                        |   3 -
  arch/arm/mach-qcom/platsmp.c                       |  23 +-
  arch/arm/mach-qcom/scm-boot.c                      |  39 --
  arch/arm/mach-qcom/scm-boot.h                      |  26 --
  arch/arm/mach-qcom/scm.c                           | 326 --------------
  arch/arm/mach-qcom/scm.h                           |  25 --
  drivers/firmware/Kconfig                           |   4 +
  drivers/firmware/Makefile                          |   2 +
  drivers/firmware/qcom_scm.c                        | 494 +++++++++++++++++++++
  drivers/soc/qcom/Kconfig                           |   1 +
  drivers/soc/qcom/qcom_gsbi.c                       | 152 +++++++
  include/linux/qcom_scm.h                           |  28 ++
  16 files changed, 708 insertions(+), 451 deletions(-)
  delete mode 100644 arch/arm/mach-qcom/scm-boot.c
  delete mode 100644 arch/arm/mach-qcom/scm-boot.h
  delete mode 100644 arch/arm/mach-qcom/scm.c
  delete mode 100644 arch/arm/mach-qcom/scm.h
  create mode 100644 drivers/firmware/qcom_scm.c
  create mode 100644 include/linux/qcom_scm.h

--
Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] qcom SoC changes for v4.1
  2015-03-16 21:03 [GIT PULL] qcom SoC changes for v4.1 Kumar Gala
@ 2015-04-02  0:36 ` Olof Johansson
  2015-04-02  8:37 ` Nicolas Dechesne
  1 sibling, 0 replies; 8+ messages in thread
From: Olof Johansson @ 2015-04-02  0:36 UTC (permalink / raw)
  To: Kumar Gala
  Cc: arm, Arnd Bergmann, linux-arm-msm, linux-kernel,
	linux-arm-kernel, agross, lina.iyer

On Mon, Mar 16, 2015 at 04:03:12PM -0500, Kumar Gala wrote:
> The following changes since commit c517d838eb7d07bbe9507871fab3931deccff539:
> 
>   Linux 4.0-rc1 (2015-02-22 18:21:14 -0800)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/galak/linux-qcom.git tags/qcom-soc-for-4.1
> 
> for you to fetch changes up to e5fdad68d47ed344832b7ca4e18b2e9708d8141e:
> 
>   soc: qcom: gsbi: Add support for ADM CRCI muxing (2015-03-11 15:18:39 -0500)

Thanks. Merged into next/drivers.


-Olof

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] qcom SoC changes for v4.1
  2015-03-16 21:03 [GIT PULL] qcom SoC changes for v4.1 Kumar Gala
  2015-04-02  0:36 ` Olof Johansson
@ 2015-04-02  8:37 ` Nicolas Dechesne
  2015-04-02 19:25   ` Kumar Gala
  1 sibling, 1 reply; 8+ messages in thread
From: Nicolas Dechesne @ 2015-04-02  8:37 UTC (permalink / raw)
  To: Kumar Gala
  Cc: arm, Olof Johansson, Arnd Bergmann, linux-arm-msm, linux-kernel,
	linux-arm-kernel, Andy Gross, Lina Iyer, Tyler Baker

Andy, Kumar,

On Mon, Mar 16, 2015 at 10:03 PM, Kumar Gala <galak@codeaurora.org> wrote:
> Andy Gross (1):
>       soc: qcom: gsbi: Add support for ADM CRCI muxing

this commit seems to break the boot on IFC6410, it was initially
reported on kernelci.org, see report and bootlog [1].

running git bisect led me to:

e5fdad68d47ed344832b7ca4e18b2e9708d8141e is the first bad commit
commit e5fdad68d47ed344832b7ca4e18b2e9708d8141e
Author: Andy Gross <agross@codeaurora.org>
Date:   Mon Feb 9 16:01:06 2015 -0600

    soc: qcom: gsbi: Add support for ADM CRCI muxing

    This patch adds automatic configuration for the ADM CRCI muxing required to
    support DMA operations for GSBI clients.  The GSBI mode and
instance determine
    the correct TCSR ADM CRCI MUX value that must be programmed so that the DMA
    works properly.

    Signed-off-by: Andy Gross <agross@codeaurora.org>
    Signed-off-by: Kumar Gala <galak@codeaurora.org>


[1] http://kernelci.org/boot/all/job/arm-soc/kernel/v4.0-rc4-354-ga0690e6586df/

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] qcom SoC changes for v4.1
  2015-04-02  8:37 ` Nicolas Dechesne
@ 2015-04-02 19:25   ` Kumar Gala
  2015-04-02 19:43     ` Stephen Boyd
  0 siblings, 1 reply; 8+ messages in thread
From: Kumar Gala @ 2015-04-02 19:25 UTC (permalink / raw)
  To: Nicolas Dechesne
  Cc: arm, Olof Johansson, Arnd Bergmann, linux-arm-msm, linux-kernel,
	linux-arm-kernel, Andy Gross, Lina Iyer, Tyler Baker


On Apr 2, 2015, at 3:37 AM, Nicolas Dechesne <nicolas.dechesne@linaro.org> wrote:

> Andy, Kumar,
> 
> On Mon, Mar 16, 2015 at 10:03 PM, Kumar Gala <galak@codeaurora.org> wrote:
>> Andy Gross (1):
>>      soc: qcom: gsbi: Add support for ADM CRCI muxing
> 
> this commit seems to break the boot on IFC6410, it was initially
> reported on kernelci.org, see report and bootlog [1].
> 
> running git bisect led me to:
> 
> e5fdad68d47ed344832b7ca4e18b2e9708d8141e is the first bad commit
> commit e5fdad68d47ed344832b7ca4e18b2e9708d8141e
> Author: Andy Gross <agross@codeaurora.org>
> Date:   Mon Feb 9 16:01:06 2015 -0600
> 
>    soc: qcom: gsbi: Add support for ADM CRCI muxing
> 
>    This patch adds automatic configuration for the ADM CRCI muxing required to
>    support DMA operations for GSBI clients.  The GSBI mode and
> instance determine
>    the correct TCSR ADM CRCI MUX value that must be programmed so that the DMA
>    works properly.
> 
>    Signed-off-by: Andy Gross <agross@codeaurora.org>
>    Signed-off-by: Kumar Gala <galak@codeaurora.org>
> 
> 
> [1] http://kernelci.org/boot/all/job/arm-soc/kernel/v4.0-rc4-354-ga0690e6586df/

I think we need to associated DT updates.

- k

-- 
Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] qcom SoC changes for v4.1
  2015-04-02 19:25   ` Kumar Gala
@ 2015-04-02 19:43     ` Stephen Boyd
  2015-04-02 19:47       ` Nicolas Dechesne
  2015-04-03 16:22       ` Andy Gross
  0 siblings, 2 replies; 8+ messages in thread
From: Stephen Boyd @ 2015-04-02 19:43 UTC (permalink / raw)
  To: Kumar Gala, Nicolas Dechesne
  Cc: arm, Olof Johansson, Arnd Bergmann, linux-arm-msm, linux-kernel,
	linux-arm-kernel, Andy Gross, Lina Iyer, Tyler Baker

On 04/02/15 12:25, Kumar Gala wrote:
> On Apr 2, 2015, at 3:37 AM, Nicolas Dechesne <nicolas.dechesne@linaro.org> wrote:
>
>> Andy, Kumar,
>>
>> On Mon, Mar 16, 2015 at 10:03 PM, Kumar Gala <galak@codeaurora.org> wrote:
>>> Andy Gross (1):
>>>      soc: qcom: gsbi: Add support for ADM CRCI muxing
>> this commit seems to break the boot on IFC6410, it was initially
>> reported on kernelci.org, see report and bootlog [1].
>>
>> running git bisect led me to:
>>
>> e5fdad68d47ed344832b7ca4e18b2e9708d8141e is the first bad commit
>> commit e5fdad68d47ed344832b7ca4e18b2e9708d8141e
>> Author: Andy Gross <agross@codeaurora.org>
>> Date:   Mon Feb 9 16:01:06 2015 -0600
>>
>>    soc: qcom: gsbi: Add support for ADM CRCI muxing
>>
>>    This patch adds automatic configuration for the ADM CRCI muxing required to
>>    support DMA operations for GSBI clients.  The GSBI mode and
>> instance determine
>>    the correct TCSR ADM CRCI MUX value that must be programmed so that the DMA
>>    works properly.
>>
>>    Signed-off-by: Andy Gross <agross@codeaurora.org>
>>    Signed-off-by: Kumar Gala <galak@codeaurora.org>
>>
>>
>> [1] http://kernelci.org/boot/all/job/arm-soc/kernel/v4.0-rc4-354-ga0690e6586df/
> I think we need to associated DT updates.
>
>

What about this patch squashed on top? Just guessing but I suspect we
don't care about cell-index if we're not doing the tcsr stuff. Also, I
imagine we could get rid of cell-index entirely if we matched against
the address of the gsbi instead.

Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>

----8<-----

diff --git a/drivers/soc/qcom/qcom_gsbi.c b/drivers/soc/qcom/qcom_gsbi.c
index 09c669e70d63..ac7d71b6527d 100644
--- a/drivers/soc/qcom/qcom_gsbi.c
+++ b/drivers/soc/qcom/qcom_gsbi.c
@@ -139,7 +139,7 @@ static int gsbi_probe(struct platform_device *pdev)
 	void __iomem *base;
 	struct gsbi_info *gsbi;
 	int i;
-	u32 mask, gsbi_num;
+	u32 mask, gsbi_num = 0;
 	const struct crci_config *config = NULL;
 
 	gsbi = devm_kzalloc(&pdev->dev, sizeof(*gsbi), GFP_KERNEL);
@@ -166,16 +166,19 @@ static int gsbi_probe(struct platform_device *pdev)
 
 			of_node_put(tcsr_node);
 		}
-	}
 
-	if (of_property_read_u32(node, "cell-index", &gsbi_num)) {
-		dev_err(&pdev->dev, "missing cell-index\n");
-		return -EINVAL;
-	}
+		if (config) {
+			if (of_property_read_u32(node, "cell-index", &gsbi_num)) {
+				dev_err(&pdev->dev, "missing cell-index\n");
+				return -EINVAL;
+			}
+
+			if (gsbi_num < 1 || gsbi_num > MAX_GSBI) {
+				dev_err(&pdev->dev, "invalid cell-index\n");
+				return -EINVAL;
+			}
+		}
 
-	if (gsbi_num < 1 || gsbi_num > MAX_GSBI) {
-		dev_err(&pdev->dev, "invalid cell-index\n");
-		return -EINVAL;
 	}
 
 	if (of_property_read_u32(node, "qcom,mode", &gsbi->mode)) {

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] qcom SoC changes for v4.1
  2015-04-02 19:43     ` Stephen Boyd
@ 2015-04-02 19:47       ` Nicolas Dechesne
  2015-04-03 16:22       ` Andy Gross
  1 sibling, 0 replies; 8+ messages in thread
From: Nicolas Dechesne @ 2015-04-02 19:47 UTC (permalink / raw)
  To: Stephen Boyd
  Cc: Kumar Gala, arm, Olof Johansson, Arnd Bergmann, linux-arm-msm,
	linux-kernel, linux-arm-kernel, Andy Gross, Lina Iyer,
	Tyler Baker

On Thu, Apr 2, 2015 at 9:43 PM, Stephen Boyd <sboyd@codeaurora.org> wrote:
> What about this patch squashed on top? Just guessing but I suspect we
> don't care about cell-index if we're not doing the tcsr stuff. Also, I
> imagine we could get rid of cell-index entirely if we matched against
> the address of the gsbi instead.
>
> Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
>
> ----8<-----
>
> diff --git a/drivers/soc/qcom/qcom_gsbi.c b/drivers/soc/qcom/qcom_gsbi.c
> index 09c669e70d63..ac7d71b6527d 100644
> --- a/drivers/soc/qcom/qcom_gsbi.c
> +++ b/drivers/soc/qcom/qcom_gsbi.c
> @@ -139,7 +139,7 @@ static int gsbi_probe(struct platform_device *pdev)
>         void __iomem *base;
>         struct gsbi_info *gsbi;
>         int i;
> -       u32 mask, gsbi_num;
> +       u32 mask, gsbi_num = 0;
>         const struct crci_config *config = NULL;
>
>         gsbi = devm_kzalloc(&pdev->dev, sizeof(*gsbi), GFP_KERNEL);
> @@ -166,16 +166,19 @@ static int gsbi_probe(struct platform_device *pdev)
>
>                         of_node_put(tcsr_node);
>                 }
> -       }
>
> -       if (of_property_read_u32(node, "cell-index", &gsbi_num)) {
> -               dev_err(&pdev->dev, "missing cell-index\n");
> -               return -EINVAL;
> -       }
> +               if (config) {
> +                       if (of_property_read_u32(node, "cell-index", &gsbi_num)) {
> +                               dev_err(&pdev->dev, "missing cell-index\n");
> +                               return -EINVAL;
> +                       }
> +
> +                       if (gsbi_num < 1 || gsbi_num > MAX_GSBI) {
> +                               dev_err(&pdev->dev, "invalid cell-index\n");
> +                               return -EINVAL;
> +                       }
> +               }
>
> -       if (gsbi_num < 1 || gsbi_num > MAX_GSBI) {
> -               dev_err(&pdev->dev, "invalid cell-index\n");
> -               return -EINVAL;
>         }
>
>         if (of_property_read_u32(node, "qcom,mode", &gsbi->mode)) {


I think it would work, i cannot test right now, i can do it tomorrow
if you need it, but that's pretty much how i tested earlier today (i
had commented out the 2 statements you are putting moving here in the
new if statement.

I did also test with the associated DT patches, and it worked as well.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] qcom SoC changes for v4.1
  2015-04-02 19:43     ` Stephen Boyd
  2015-04-02 19:47       ` Nicolas Dechesne
@ 2015-04-03 16:22       ` Andy Gross
  2015-04-03 18:40         ` Stephen Boyd
  1 sibling, 1 reply; 8+ messages in thread
From: Andy Gross @ 2015-04-03 16:22 UTC (permalink / raw)
  To: Stephen Boyd
  Cc: Kumar Gala, Nicolas Dechesne, arm, Olof Johansson, Arnd Bergmann,
	linux-arm-msm, linux-kernel, linux-arm-kernel, Lina Iyer,
	Tyler Baker

On Thu, Apr 02, 2015 at 12:43:50PM -0700, Stephen Boyd wrote:
> On 04/02/15 12:25, Kumar Gala wrote:
> > On Apr 2, 2015, at 3:37 AM, Nicolas Dechesne <nicolas.dechesne@linaro.org> wrote:
> >
> >> Andy, Kumar,
> >>
> >> On Mon, Mar 16, 2015 at 10:03 PM, Kumar Gala <galak@codeaurora.org> wrote:
> >>> Andy Gross (1):
> >>>      soc: qcom: gsbi: Add support for ADM CRCI muxing
> >> this commit seems to break the boot on IFC6410, it was initially
> >> reported on kernelci.org, see report and bootlog [1].
> >>
> >> running git bisect led me to:
> >>
> >> e5fdad68d47ed344832b7ca4e18b2e9708d8141e is the first bad commit
> >> commit e5fdad68d47ed344832b7ca4e18b2e9708d8141e
> >> Author: Andy Gross <agross@codeaurora.org>
> >> Date:   Mon Feb 9 16:01:06 2015 -0600
> >>
> >>    soc: qcom: gsbi: Add support for ADM CRCI muxing
> >>
> >>    This patch adds automatic configuration for the ADM CRCI muxing required to
> >>    support DMA operations for GSBI clients.  The GSBI mode and
> >> instance determine
> >>    the correct TCSR ADM CRCI MUX value that must be programmed so that the DMA
> >>    works properly.
> >>
> >>    Signed-off-by: Andy Gross <agross@codeaurora.org>
> >>    Signed-off-by: Kumar Gala <galak@codeaurora.org>
> >>
> >>
> >> [1] http://kernelci.org/boot/all/job/arm-soc/kernel/v4.0-rc4-354-ga0690e6586df/
> > I think we need to associated DT updates.
> >
> >
> 
> What about this patch squashed on top? Just guessing but I suspect we
> don't care about cell-index if we're not doing the tcsr stuff. Also, I
> imagine we could get rid of cell-index entirely if we matched against
> the address of the gsbi instead.

Except that the GSBI5 base address changes from chip to chip.


-- 
Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] qcom SoC changes for v4.1
  2015-04-03 16:22       ` Andy Gross
@ 2015-04-03 18:40         ` Stephen Boyd
  0 siblings, 0 replies; 8+ messages in thread
From: Stephen Boyd @ 2015-04-03 18:40 UTC (permalink / raw)
  To: Andy Gross
  Cc: Kumar Gala, Nicolas Dechesne, arm, Olof Johansson, Arnd Bergmann,
	linux-arm-msm, linux-kernel, linux-arm-kernel, Lina Iyer,
	Tyler Baker

On 04/03, Andy Gross wrote:
> On Thu, Apr 02, 2015 at 12:43:50PM -0700, Stephen Boyd wrote:
> > On 04/02/15 12:25, Kumar Gala wrote:
> > >> [1] http://kernelci.org/boot/all/job/arm-soc/kernel/v4.0-rc4-354-ga0690e6586df/
> > > I think we need to associated DT updates.
> > >
> > >
> > 
> > What about this patch squashed on top? Just guessing but I suspect we
> > don't care about cell-index if we're not doing the tcsr stuff. Also, I
> > imagine we could get rid of cell-index entirely if we matched against
> > the address of the gsbi instead.
> 
> Except that the GSBI5 base address changes from chip to chip.
> 
> 

Yep, for the cell-index removal part I was thinking we would make
another table per SoC like we already have for the TCSR part. The
table would map the physical address to the GSBI number.

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2015-04-03 18:40 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-03-16 21:03 [GIT PULL] qcom SoC changes for v4.1 Kumar Gala
2015-04-02  0:36 ` Olof Johansson
2015-04-02  8:37 ` Nicolas Dechesne
2015-04-02 19:25   ` Kumar Gala
2015-04-02 19:43     ` Stephen Boyd
2015-04-02 19:47       ` Nicolas Dechesne
2015-04-03 16:22       ` Andy Gross
2015-04-03 18:40         ` Stephen Boyd

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).