Date: Wed, 23 Mar 2016 18:20:26 +0100 (CET)
From: Thomas Gleixner <tglx@linutronix.de>
To: Alex Thorlton <athorlton@sgi.com>
cc: linux-kernel@vger.kernel.org, Ingo Molnar <mingo@redhat.com>,
        "H. Peter Anvin" <hpa@zytor.com>, Hedi Berriche <hedi@sgi.com>,
        x86@kernel.org
Subject: Re: [PATCH 1/2] Disable UV BAU by default
In-Reply-To: <20160323161158.GC205791@stormcage.americas.sgi.com>
Message-ID: <alpine.DEB.2.11.1603231812500.3978@nanos>
References: <1458579852-37580-1-git-send-email-athorlton@sgi.com> <1458579852-37580-2-git-send-email-athorlton@sgi.com> <alpine.DEB.2.11.1603231223440.3978@nanos> <20160323161158.GC205791@stormcage.americas.sgi.com>
User-Agent: Alpine 2.11 (DEB 23 2013-08-11)
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 713
Lines: 20

On Wed, 23 Mar 2016, Alex Thorlton wrote:
> This was actually what I initially wrote, but we decided to go with the
> on/off switch instead, because, in the UV4 time-frame, we're hoping to
> get a few things changed so that we can default to having the bau *on*
> for the new UV4 systems.
> 
> I left that detail out of the original commit message, as I didn't
> figure our future (still tentative) plans were all that important to the
> community.  I can add that information to my commit message, if you
> would prefer to see it there.

Yes please.

Please make that parameter boolean type, so you can just use strtobool(). 

That supports "uvbau==[yYnN01]" and you can avoid the custom parser.

Thanks,

	tglx