linux-kernel - RE: [PATCH 7/7] mips: pistachio: Allow to enable the external timer based clocksource

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <72BC0C8BD7BB6F45988A99382E5FBAE5445286D0@hhmail02.hh.imgtec.org>
Date:	Fri, 22 May 2015 16:58:28 +0000
From:	James Hartley <James.Hartley@...tec.com>
To:	Andrew Bresticker <abrestic@...omium.org>,
	Ezequiel Garcia <Ezequiel.Garcia@...tec.com>
CC:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Linux-MIPS <linux-mips@...ux-mips.org>,
	Daniel Lezcano <daniel.lezcano@...aro.org>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
	James Hogan <James.Hogan@...tec.com>,
	Thomas Gleixner <tglx@...utronix.de>,
	"Damien Horsley" <Damien.Horsley@...tec.com>,
	Govindraj Raja <Govindraj.Raja@...tec.com>
Subject: RE: [PATCH 7/7] mips: pistachio: Allow to enable the external timer
 based clocksource



> -----Original Message-----
> From: abrestic@...gle.com [mailto:abrestic@...gle.com] On Behalf Of
> Andrew Bresticker
> Sent: 22 May 2015 17:50
> To: Ezequiel Garcia
> Cc: linux-kernel@...r.kernel.org; Linux-MIPS; Daniel Lezcano;
> devicetree@...r.kernel.org; James Hartley; James Hogan; Thomas Gleixner;
> Damien Horsley; Govindraj Raja
> Subject: Re: [PATCH 7/7] mips: pistachio: Allow to enable the external timer
> based clocksource
> 
> On Thu, May 21, 2015 at 2:43 PM, Ezequiel Garcia
> <ezequiel.garcia@...tec.com> wrote:
> > This commit introduces a new config, so the user can choose to enable
> > the General Purpose Timer based clocksource. This option is required
> > to have CPUFreq support.
> >
> > Signed-off-by: Ezequiel Garcia <ezequiel.garcia@...tec.com>
> > ---
> >  arch/mips/Kconfig           |  1 +
> >  arch/mips/pistachio/Kconfig | 13 +++++++++++++
> >  2 files changed, 14 insertions(+)
> >  create mode 100644 arch/mips/pistachio/Kconfig
> >
> > diff --git a/arch/mips/Kconfig b/arch/mips/Kconfig index
> > f501665..91f6ca0 100644
> > --- a/arch/mips/Kconfig
> > +++ b/arch/mips/Kconfig
> > @@ -934,6 +934,7 @@ source "arch/mips/jazz/Kconfig"
> >  source "arch/mips/jz4740/Kconfig"
> >  source "arch/mips/lantiq/Kconfig"
> >  source "arch/mips/lasat/Kconfig"
> > +source "arch/mips/pistachio/Kconfig"
> >  source "arch/mips/pmcs-msp71xx/Kconfig"
> >  source "arch/mips/ralink/Kconfig"
> >  source "arch/mips/sgi-ip27/Kconfig"
> > diff --git a/arch/mips/pistachio/Kconfig b/arch/mips/pistachio/Kconfig
> > new file mode 100644 index 0000000..97731ea
> > --- /dev/null
> > +++ b/arch/mips/pistachio/Kconfig
> > @@ -0,0 +1,13 @@
> > +config PISTACHIO_GPTIMER_CLKSRC
> > +       bool "Enable General Purpose Timer based clocksource"
> > +       depends on MACH_PISTACHIO
> > +       select CLKSRC_PISTACHIO
> > +       select MIPS_EXTERNAL_TIMER
> 
> Why not just select these in the MACH_PISTACHIO Kconfig entry?  Is there any
> harm in always having the Pistachio GPT enabled?

It does mean that there are less GPT's available for other users, and whilst I'm not aware of any use cases that currently require all 4, perhaps having the flexibility is worth it.

James