From: kgene.kim@samsung.com (Kukjin Kim)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: S5P: Add PMU device
Date: Tue, 06 Jul 2010 12:12:37 +0900 [thread overview]
Message-ID: <009f01cb1cb9$188332d0$49899870$%kim@samsung.com> (raw)
In-Reply-To: <AANLkTik3WGvkUsvKHllgSlj9fJJy90y7L4me4tywCoE0@mail.gmail.com>
Kyungmin Park wrote:
>
> On Mon, Jul 5, 2010 at 9:57 PM, Kukjin Kim <kgene.kim@samsung.com> wrote:
> > Maurus Cuelenaere wrote:
> >>
> >> ?Op 05-07-10 03:46, Joonyoung Shim schreef:
> >> > This patch adds an initcall for the s5p platforms so that they
register
> >> > their PMU IRQs with the PMU framework in the Kernel.
> >> >
> >> > Signed-off-by: Joonyoung Shim <jy0922.shim@samsung.com>
> >> > ---
> >> > ?arch/arm/mach-s5p6442/include/mach/irqs.h | ? ?2 +-
> >> > ?arch/arm/mach-s5pc100/include/mach/irqs.h | ? ?1 +
> >> > ?arch/arm/mach-s5pv210/include/mach/irqs.h | ? ?1 +
> >> > ?arch/arm/plat-s5p/Makefile ? ? ? ? ? ? ? ?| ? ?1 +
> >> > ?arch/arm/plat-s5p/dev-pmu.c ? ? ? ? ? ? ? | ? 37
> >> +++++++++++++++++++++++++++++
> >> > ?5 files changed, 41 insertions(+), 1 deletions(-)
> >> > ?create mode 100644 arch/arm/plat-s5p/dev-pmu.c
> >>
> >> Wouldn't it be better if this was in plat-samsung? I can see that the
S3C6410
> >> datasheet mentions a PMU_IRQ_ENABLE bit in SYS_OTHERS so I suspect
> that it
> >> has
> >> the same functionality (even though there's no mention of which
interrupt this
> >> is).
> >>
> > Yes, I also found PMU_IRQ_ENABLE bit in System Others register of
S3C6410
> datasheet. But as your comments, could not found the interrupt number and
any
> description...Actually, need to check whether it's available or not.
> >
> > And S5P6440 has it.
> > So Joonyoung, it would be helpful if you could add 6440 PMUIRQ
(VIC1[23]) in
> this patch.
>
> It's another story. Can you explain the difference between 6440 and 6442?
> As I heard it's same chip and type is difference. If true, how about
> to delete the 6442 directory?
> It makes a single kernel simple.
>
I remember, already explained about that.
Hmm..Where did you hear wrong information that they are same? :-(
Absolutely, they are different !! not same chip.
...
But actually, I'm working on merge some S5P SoCs...
Thanks.
Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
next prev parent reply other threads:[~2010-07-06 3:12 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-07-05 1:46 [PATCH] ARM: S5P: Add PMU device Joonyoung Shim
2010-07-05 11:39 ` Maurus Cuelenaere
2010-07-05 12:57 ` Kukjin Kim
2010-07-06 0:50 ` Kyungmin Park
2010-07-06 3:12 ` Kukjin Kim [this message]
2010-07-06 4:09 ` Kyungmin Park
2010-07-06 1:48 ` Joonyoung Shim
2010-07-06 3:23 ` Kukjin Kim
2010-07-12 4:20 ` Kukjin Kim
2010-07-05 12:44 ` Kukjin Kim
2010-07-06 1:56 ` Joonyoung Shim
2010-07-06 3:43 ` Kukjin Kim
2010-07-06 7:09 ` Joonyoung Shim
2010-07-07 0:26 ` Kukjin Kim
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='009f01cb1cb9$188332d0$49899870$%kim@samsung.com' \
--to=kgene.kim@samsung.com \
--cc=linux-arm-kernel@lists.infradead.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