Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   Aleksandar Markovic <Aleksandar.Markovic@mips.com>
To:     James Hogan <James.Hogan@mips.com>,
        Aleksandar Markovic <aleksandar.markovic@rt-rk.com>
CC:     "linux-mips@linux-mips.org" <linux-mips@linux-mips.org>,
        Miodrag Dinic <Miodrag.Dinic@mips.com>,
        Goran Ferenc <Goran.Ferenc@mips.com>,
        "David S. Miller" <davem@davemloft.net>,
        Douglas Leung <Douglas.Leung@mips.com>,
        "Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Mauro Carvalho Chehab <mchehab@kernel.org>,
        Paul Burton <Paul.Burton@mips.com>,
        Petar Jovanovic <Petar.Jovanovic@mips.com>,
        Raghu Gandham <Raghu.Gandham@mips.com>,
        "Ralf Baechle" <ralf@linux-mips.org>,
        Randy Dunlap <rdunlap@infradead.org>
Subject: RE: [PATCH v9 3/3] MIPS: ranchu: Add Ranchu as a new generic-based
 board
Thread-Topic: [PATCH v9 3/3] MIPS: ranchu: Add Ranchu as a new generic-based
 board
Thread-Index: AQHTXhfOlk9EmIq0B0Sx4ZGVYt9VOaMWg9yAgABte/g=
Date:   Thu, 16 Nov 2017 12:45:28 +0000
Message-ID: <BD3A5F1946F2E540A31AF2CE969BAEEEC19ED4@MIPSMAIL01.mipstec.com>
References: <1510753368-16453-1-git-send-email-aleksandar.markovic@rt-rk.com>
 <1510753368-16453-4-git-send-email-aleksandar.markovic@rt-rk.com>,<20171115215910.GB27409@jhogan-linux.mipstec.com>
In-Reply-To: <20171115215910.GB27409@jhogan-linux.mipstec.com>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 8BIT
MIME-Version: 1.0
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

> From: James Hogan
> 
> Hi Aleksandar,
> 
> On Wed, Nov 15, 2017 at 02:42:20PM +0100, Aleksandar Markovic wrote:
> > diff --git a/arch/mips/configs/generic/board-ranchu.config b/arch/mips/configs/generic/board-ranchu.config
> > new file mode 100644
> > index 0000000..fee9ad4
> > --- /dev/null
> > +++ b/arch/mips/configs/generic/board-ranchu.config
> > @@ -0,0 +1,30 @@
> > +CONFIG_VIRT_BOARD_RANCHU=y
> 
> I presume its valid for Ranchu support to be enabled in MIPS32 / MIPS64,
> and R1 / R2 / R6 kernels? (that's fine if so, just making sure there's
> no need for a require comment).

Yes, Ranchu virtual machine, and Android emulator for that matter, support
all combinations you mentioned.

> 
> > diff --git a/arch/mips/generic/board-ranchu.c b/arch/mips/generic/board-ranchu.c
> > new file mode 100644
> > index 0000000..0efc555
> > --- /dev/null
> > +++ b/arch/mips/generic/board-ranchu.c
> > @@ -0,0 +1,85 @@
> 
> ...
> 
> > +static __init unsigned int ranchu_measure_hpt_freq(void)
> > +{
> 
> ...
> 
> > +     count += 5000;  /* round */
> > +     count -= count % 10000;
> 
> A comment to explain the purpose of the rounding would be helpful. I
> presume its there just to get a more accurate value since the frequency
> will always be a round value in practice.

An appropriate comment will be added in v10, but v10 will be submitted
no sooner than next week, to provide enough time for other people wanting
to review this and other patches.

> 
> Either way this patch looks good to me:
> Reviewed-by: James Hogan <jhogan@kernel.org>
> 
> Thanks!
> James

I appreciate your review!

Aleksandar

From 1584180968011046439@xxx Thu Nov 16 00:38:21 +0000 2017
X-GM-THRID: 1584140115256825008
X-Gmail-Labels: Inbox,Category Forums,HistoricalUnread