From: Chanho Park <chanho61.park@samsung.com>
To: 'Olof Johansson' <olof@lixom.net>
Cc: kgene.kim@samsung.com, linux-samsung-soc@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, linux@arm.linux.org.uk,
ben-linux@fluff.org, kyungmin.park@samsung.com,
sachin.kamat@linaro.org, thomas.abraham@linaro.org,
will.deacon@arm.com
Subject: RE: [PATCH v5 3/5] ARM: EXYNOS: Enable PMUs for exynos4
Date: Thu, 25 Oct 2012 10:41:46 +0900 [thread overview]
Message-ID: <005801cdb251$e52f7480$af8e5d80$@samsung.com> (raw)
In-Reply-To: <CAOesGMg5Nn5i=J8o3Rt8es3Tq_LZgVttuYSBbvYfmw1QTGwx0A@mail.gmail.com>
> -----Original Message-----
> From: Olof Johansson [mailto:olof@lixom.net]
> Sent: Thursday, October 25, 2012 2:15 AM
> To: Chanho Park
> Cc: kgene.kim@samsung.com; linux-samsung-soc@vger.kernel.org; linux-
> arm-kernel@lists.infradead.org; linux@arm.linux.org.uk; ben-linux@fluff.org;
> kyungmin.park@samsung.com; sachin.kamat@linaro.org;
> thomas.abraham@linaro.org; will.deacon@arm.com
> Subject: Re: [PATCH v5 3/5] ARM: EXYNOS: Enable PMUs for exynos4
>
> Hi,
>
> On Tue, Oct 23, 2012 at 10:34 PM, Chanho Park
> <chanho61.park@samsung.com> wrote:
> > This patch defines irq numbers of ARM performance monitoring unit for
> exynos4.
> > Firs of all, we need to fix IRQ_PMU correctly and to split pmu
> > initialization of exynos from plat-samsung for easily defining it.
> >
> > The number of CPU cores and PMU irq numbers are vary according to soc
> types.
> > So, we need to identify each soc type using soc_is_xxx function and to
> > define the pmu irqs dynamically. For example, the exynos4412 has 4 cpu
> cores and pmus.
>
> I wonder if it's worth doing this complexity on the non-DT case for exynos4?
>
> I wish there was more focus on the Samsung platforms for getting the DT
> support up to par with non-DT so you can avoid having to add new platform
> devices like these in the first place.
The DT support of exynos4 is under development.
And many of exynos4 developers still use non-dt boot-up method.
By this time arm-pmu of exynos did not work. IMO we should fix and support it
for non-dt users.
Thanks,
Best regards,
Chanho Park
>
>
> Thanks,
>
> -Olof
WARNING: multiple messages have this Message-ID (diff)
From: chanho61.park@samsung.com (Chanho Park)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v5 3/5] ARM: EXYNOS: Enable PMUs for exynos4
Date: Thu, 25 Oct 2012 10:41:46 +0900 [thread overview]
Message-ID: <005801cdb251$e52f7480$af8e5d80$@samsung.com> (raw)
In-Reply-To: <CAOesGMg5Nn5i=J8o3Rt8es3Tq_LZgVttuYSBbvYfmw1QTGwx0A@mail.gmail.com>
> -----Original Message-----
> From: Olof Johansson [mailto:olof at lixom.net]
> Sent: Thursday, October 25, 2012 2:15 AM
> To: Chanho Park
> Cc: kgene.kim at samsung.com; linux-samsung-soc at vger.kernel.org; linux-
> arm-kernel at lists.infradead.org; linux at arm.linux.org.uk; ben-linux at fluff.org;
> kyungmin.park at samsung.com; sachin.kamat at linaro.org;
> thomas.abraham at linaro.org; will.deacon at arm.com
> Subject: Re: [PATCH v5 3/5] ARM: EXYNOS: Enable PMUs for exynos4
>
> Hi,
>
> On Tue, Oct 23, 2012 at 10:34 PM, Chanho Park
> <chanho61.park@samsung.com> wrote:
> > This patch defines irq numbers of ARM performance monitoring unit for
> exynos4.
> > Firs of all, we need to fix IRQ_PMU correctly and to split pmu
> > initialization of exynos from plat-samsung for easily defining it.
> >
> > The number of CPU cores and PMU irq numbers are vary according to soc
> types.
> > So, we need to identify each soc type using soc_is_xxx function and to
> > define the pmu irqs dynamically. For example, the exynos4412 has 4 cpu
> cores and pmus.
>
> I wonder if it's worth doing this complexity on the non-DT case for exynos4?
>
> I wish there was more focus on the Samsung platforms for getting the DT
> support up to par with non-DT so you can avoid having to add new platform
> devices like these in the first place.
The DT support of exynos4 is under development.
And many of exynos4 developers still use non-dt boot-up method.
By this time arm-pmu of exynos did not work. IMO we should fix and support it
for non-dt users.
Thanks,
Best regards,
Chanho Park
>
>
> Thanks,
>
> -Olof
next prev parent reply other threads:[~2012-10-25 1:41 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-24 5:34 [PATCH v5 0/5] Add support to enable ARM PMU for EXYNOS Chanho Park
2012-10-24 5:34 ` Chanho Park
2012-10-24 5:34 ` [PATCH v5 1/5] ARM: EXYNOS: Add set_irq_affinity function for combiner_irq Chanho Park
2012-10-24 5:34 ` Chanho Park
2012-10-24 5:34 ` [PATCH v5 2/5] ARM: EXYNOS: Correct combined IRQs for exynos4 Chanho Park
2012-10-24 5:34 ` Chanho Park
2012-10-24 5:34 ` [PATCH v5 3/5] ARM: EXYNOS: Enable PMUs " Chanho Park
2012-10-24 5:34 ` Chanho Park
2012-10-24 17:14 ` Olof Johansson
2012-10-24 17:14 ` Olof Johansson
2012-10-25 1:41 ` Chanho Park [this message]
2012-10-25 1:41 ` Chanho Park
2012-10-25 14:41 ` Will Deacon
2012-10-25 14:41 ` Will Deacon
2012-10-26 4:05 ` Chanho Park
2012-10-26 4:05 ` Chanho Park
2012-10-26 9:36 ` Will Deacon
2012-10-26 9:36 ` Will Deacon
2012-10-24 5:34 ` [PATCH v5 4/5] ARM: EXYNOS: Add arm-pmu DT binding for exynos5250 Chanho Park
2012-10-24 5:34 ` Chanho Park
2012-10-24 5:34 ` [PATCH v5 5/5] ARM: EXYNOS: Add arm-pmu DT binding for exynos421x Chanho Park
2012-10-24 5:34 ` Chanho Park
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='005801cdb251$e52f7480$af8e5d80$@samsung.com' \
--to=chanho61.park@samsung.com \
--cc=ben-linux@fluff.org \
--cc=kgene.kim@samsung.com \
--cc=kyungmin.park@samsung.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=olof@lixom.net \
--cc=sachin.kamat@linaro.org \
--cc=thomas.abraham@linaro.org \
--cc=will.deacon@arm.com \
/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.