All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Brown <davidb@codeaurora.org>
To: Daniel Walker <dwalker@codeaurora.org>
Cc: linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v2 01/11] msm: Add CPU queries
Date: Tue, 25 Jan 2011 11:17:25 -0800	[thread overview]
Message-ID: <8ya39ogg5dm.fsf@huya.qualcomm.com> (raw)
In-Reply-To: <1295976910.25496.1.camel@c-dwalke-linux.qualcomm.com> (Daniel Walker's message of "Tue, 25 Jan 2011 09:35:10 -0800")

On Tue, Jan 25 2011, Daniel Walker wrote:

> On Mon, 2011-01-24 at 16:57 -0800, David Brown wrote:
>> MSM7201 and MSM7601 are identical as far as Linux is concerned.  Same
>> goes for MSM8250 and MSM8650.  Our dev boards are a somewhat random mix
>> of the two, and it doesn't matter which one you use.
>> 
>> MSM8960 is a completely different chip, it just shares a similar name,
>> to other chips.
>
> If you break it down without the "x" then you can recreate the "x"
> variant with the actual numerical identifier .. For 8660/8960 you just
> would do as much of a unification as with the others.. You could still
> use 8x60 to identify those two, you just wouldn't use it as often.

There isn't any unification to do here.  There are two issues.  We make
some chips in groups where we have _identical_ silicon on the Linux
side, but happen to have different numbers.  That's what the X is
about.  There are other cases where we have completely different silicon
that happen to have similar numbers.

The whole point of the cpu_is tests is to tell us exactly which chip we
are on.  If there is _anything_ different between two chips, that query
needs to be different.  If there is any kind of unification, it can be
done at a higher level.  Otherwise, you have no way of making the
distinction that needs to be made.

David

-- 
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.

WARNING: multiple messages have this Message-ID (diff)
From: davidb@codeaurora.org (David Brown)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 01/11] msm: Add CPU queries
Date: Tue, 25 Jan 2011 11:17:25 -0800	[thread overview]
Message-ID: <8ya39ogg5dm.fsf@huya.qualcomm.com> (raw)
In-Reply-To: <1295976910.25496.1.camel@c-dwalke-linux.qualcomm.com> (Daniel Walker's message of "Tue, 25 Jan 2011 09:35:10 -0800")

On Tue, Jan 25 2011, Daniel Walker wrote:

> On Mon, 2011-01-24 at 16:57 -0800, David Brown wrote:
>> MSM7201 and MSM7601 are identical as far as Linux is concerned.  Same
>> goes for MSM8250 and MSM8650.  Our dev boards are a somewhat random mix
>> of the two, and it doesn't matter which one you use.
>> 
>> MSM8960 is a completely different chip, it just shares a similar name,
>> to other chips.
>
> If you break it down without the "x" then you can recreate the "x"
> variant with the actual numerical identifier .. For 8660/8960 you just
> would do as much of a unification as with the others.. You could still
> use 8x60 to identify those two, you just wouldn't use it as often.

There isn't any unification to do here.  There are two issues.  We make
some chips in groups where we have _identical_ silicon on the Linux
side, but happen to have different numbers.  That's what the X is
about.  There are other cases where we have completely different silicon
that happen to have similar numbers.

The whole point of the cpu_is tests is to tell us exactly which chip we
are on.  If there is _anything_ different between two chips, that query
needs to be different.  If there is any kind of unification, it can be
done at a higher level.  Otherwise, you have no way of making the
distinction that needs to be made.

David

-- 
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.

  reply	other threads:[~2011-01-25 19:17 UTC|newest]

Thread overview: 156+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-12-15  3:49 [PATCH 0/7] msm: Support for Qualcomm MSM8960 Stepan Moskovchenko
2010-12-15  3:49 ` Stepan Moskovchenko
2010-12-15  3:49 ` [PATCH 1/7] msm: io: I/O register definitions for MSM8960 Stepan Moskovchenko
2010-12-15  3:49   ` Stepan Moskovchenko
2010-12-15 15:31   ` Arnd Bergmann
2010-12-15 15:31     ` Arnd Bergmann
2010-12-15 15:35     ` David Brown
2010-12-15 15:35       ` David Brown
2010-12-15 16:40       ` Arnd Bergmann
2010-12-15 16:40         ` Arnd Bergmann
2010-12-15 22:03         ` David Brown
2010-12-15 22:03           ` David Brown
2010-12-15 22:37           ` Arnd Bergmann
2010-12-15 22:37             ` Arnd Bergmann
2010-12-17  0:16             ` David Brown
2010-12-17  0:16               ` David Brown
2010-12-24 13:29               ` Arnd Bergmann
2010-12-24 13:29                 ` Arnd Bergmann
2010-12-25 16:04                 ` Nicolas Pitre
2010-12-25 16:04                   ` Nicolas Pitre
2011-01-03  0:33                   ` Russell King - ARM Linux
2011-01-03  0:33                     ` Russell King - ARM Linux
2011-01-03  1:04                     ` Nicolas Pitre
2011-01-03  1:04                       ` Nicolas Pitre
2010-12-25 18:40                 ` David Brown
2010-12-25 18:40                   ` David Brown
2010-12-15  3:49 ` [PATCH 2/7] msm: Physical offset " Stepan Moskovchenko
2010-12-15  3:49   ` Stepan Moskovchenko
2010-12-15 13:17   ` Sergei Shtylyov
2010-12-15 13:17     ` Sergei Shtylyov
2010-12-15 13:55     ` David Brown
2010-12-15 13:55       ` David Brown
2010-12-15 14:40       ` Daniel Walker
2010-12-15 14:40         ` Daniel Walker
2010-12-15 15:38         ` David Brown
2010-12-15 15:38           ` David Brown
2010-12-15 17:53           ` Steve Muckle
2010-12-15 17:53             ` Steve Muckle
2010-12-15 17:55           ` Daniel Walker
2010-12-15 17:55             ` Daniel Walker
2010-12-15 18:07             ` Steve Muckle
2010-12-15 18:07               ` Steve Muckle
2010-12-15 18:21               ` Daniel Walker
2010-12-15 18:21                 ` Daniel Walker
2010-12-15  3:49 ` [PATCH 3/7] msm: irqs-8960: Interrupt map " Stepan Moskovchenko
2010-12-15  3:49   ` Stepan Moskovchenko
2010-12-15  3:49 ` [PATCH 4/7] msm: Board file for MSM8960 simulator Stepan Moskovchenko
2010-12-15  3:49   ` Stepan Moskovchenko
2010-12-15 15:36   ` Arnd Bergmann
2010-12-15 15:36     ` Arnd Bergmann
2010-12-15  3:49 ` [PATCH 5/7] msm: timer: Timer support for MSM8960 Stepan Moskovchenko
2010-12-15  3:49   ` Stepan Moskovchenko
2010-12-15  3:49 ` [PATCH 6/7] msm: Makefile cleanup Stepan Moskovchenko
2010-12-15  3:49   ` Stepan Moskovchenko
2010-12-15  3:49 ` [PATCH 7/7] msm: Build support for the MSM8960 target Stepan Moskovchenko
2010-12-15  3:49   ` Stepan Moskovchenko
2010-12-15 15:34   ` Arnd Bergmann
2010-12-15 15:34     ` Arnd Bergmann
2011-01-19 20:25 ` [PATCH v2 00/11] msm: Add initial support for MSM8960 David Brown
2011-01-19 20:25   ` [PATCH v2 01/11] msm: Add CPU queries David Brown
2011-01-19 20:25     ` David Brown
2011-01-25  0:09     ` Daniel Walker
2011-01-25  0:09       ` Daniel Walker
2011-01-25  0:20       ` David Brown
2011-01-25  0:20         ` David Brown
2011-01-25  0:23         ` Daniel Walker
2011-01-25  0:23           ` Daniel Walker
2011-01-25  0:57           ` David Brown
2011-01-25  0:57             ` David Brown
2011-01-25 17:35             ` Daniel Walker
2011-01-25 17:35               ` Daniel Walker
2011-01-25 19:17               ` David Brown [this message]
2011-01-25 19:17                 ` David Brown
2011-01-25 19:25                 ` Daniel Walker
2011-01-25 19:25                   ` Daniel Walker
2011-01-25 19:45                   ` David Brown
2011-01-25 19:45                     ` David Brown
2011-01-25 20:06                     ` Daniel Walker
2011-01-25 20:06                       ` Daniel Walker
2011-01-26  4:29                       ` Zhaohui Wang
2011-01-26  4:29                         ` Zhaohui Wang
2011-01-26  6:19                         ` David Brown
2011-01-26  6:19                           ` David Brown
2011-01-26  6:32                           ` Stepan Moskovchenko
2011-01-26  6:32                             ` Stepan Moskovchenko
2011-01-26  6:52                             ` David Brown
2011-01-26  6:52                               ` David Brown
2011-01-19 20:25   ` [PATCH v2 02/11] msm: Generalize timer register mappings David Brown
2011-01-19 20:25     ` David Brown
2011-01-24 22:36     ` Daniel Walker
2011-01-24 22:36       ` Daniel Walker
2011-01-24 22:44       ` David Brown
2011-01-24 22:44         ` David Brown
2011-01-24 22:57         ` Daniel Walker
2011-01-24 22:57           ` Daniel Walker
2011-01-24 23:19           ` David Brown
2011-01-24 23:19             ` David Brown
2011-01-26 22:12         ` Dima Zavin
2011-01-26 22:12           ` Dima Zavin
2011-01-26 22:12           ` Dima Zavin
2011-01-26 22:41           ` David Brown
2011-01-26 22:41             ` David Brown
2011-01-26 22:50             ` Dima Zavin
2011-01-26 22:50               ` Dima Zavin
2011-01-26 22:50               ` Dima Zavin
2011-01-26 22:46           ` Russell King - ARM Linux
2011-01-26 22:46             ` Russell King - ARM Linux
2011-01-25 18:12     ` Carl Vanderlip
2011-01-25 18:12       ` Carl Vanderlip
2011-01-25 19:18       ` David Brown
2011-01-25 19:18         ` David Brown
2011-01-19 20:25   ` [PATCH v2 03/11] msm: Generalize QGIC registers David Brown
2011-01-19 20:25     ` David Brown
2011-01-19 20:25   ` [PATCH v2 04/11] msm: io: I/O register definitions for MSM8960 David Brown
2011-01-19 20:25     ` David Brown
2011-01-24 20:49     ` Daniel Walker
2011-01-24 20:49       ` Daniel Walker
2011-01-24 21:31       ` David Brown
2011-01-24 21:31         ` David Brown
2011-01-24 21:36         ` Daniel Walker
2011-01-24 21:36           ` Daniel Walker
2011-01-19 20:25   ` [PATCH v2 05/11] msm: Physical offset " David Brown
2011-01-19 20:25     ` David Brown
2011-01-19 20:25     ` David Brown
2011-01-24 23:31     ` Russell King - ARM Linux
2011-01-24 23:31       ` Russell King - ARM Linux
2011-01-19 20:25   ` [PATCH v2 06/11] msm: irqs-8960: Interrupt map " David Brown
2011-01-19 20:25     ` David Brown
2011-01-24 22:45     ` Daniel Walker
2011-01-24 22:45       ` Daniel Walker
2011-01-24 22:49       ` David Brown
2011-01-24 22:49         ` David Brown
2011-01-19 20:25   ` [PATCH v2 07/11] msm: Add MSM 8960 cpu_is check David Brown
2011-01-19 20:25     ` David Brown
2011-01-24 20:46     ` Daniel Walker
2011-01-24 20:46       ` Daniel Walker
2011-01-24 21:30       ` David Brown
2011-01-24 21:30         ` David Brown
2011-01-24 21:35         ` Daniel Walker
2011-01-24 21:35           ` Daniel Walker
2011-01-24 22:13           ` David Brown
2011-01-24 22:13             ` David Brown
2011-01-24 22:16             ` Daniel Walker
2011-01-24 22:16               ` Daniel Walker
2011-01-19 20:25   ` [PATCH v2 08/11] msm: timer: Timer support for MSM8960 David Brown
2011-01-19 20:25     ` David Brown
2011-01-19 20:25   ` [PATCH v2 09/11] msm: Makefile cleanup David Brown
2011-01-19 20:25     ` David Brown
2011-01-19 20:25   ` [PATCH v2 10/11] msm: Support for the MSM8960 Simulator target David Brown
2011-01-19 20:25     ` David Brown
2011-01-19 20:25   ` [PATCH v2 11/11] msm: Support for the MSM8960 RUMI3 target David Brown
2011-01-19 20:25     ` David Brown
2011-01-24 23:28     ` Russell King - ARM Linux
2011-01-24 23:28       ` Russell King - ARM Linux
2011-01-19 22:40 ` [PATCH v2 00/11] msm: Add initial support for MSM8960 David Brown
2011-01-19 22:40   ` David Brown

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=8ya39ogg5dm.fsf@huya.qualcomm.com \
    --to=davidb@codeaurora.org \
    --cc=dwalker@codeaurora.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.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 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.