lkml.org 
[lkml]   [2011]   [Feb]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [RFC PATCH 02/10] MIPS: Octeon: Add device tree source files.
On 02/23/2011 03:49 PM, David Gibson wrote:
> On Wed, Feb 23, 2011 at 11:06:30AM -0800, David Daney wrote:
>> On 02/22/2011 04:07 PM, David Gibson wrote:
>>> On Tue, Feb 22, 2011 at 12:57:46PM -0800, David Daney wrote:
>>>> Signed-off-by: David Daney<ddaney@caviumnetworks.com>
>>>> ---
>>>> arch/mips/cavium-octeon/.gitignore | 2 +
>>>> arch/mips/cavium-octeon/Makefile | 13 ++
>>>> arch/mips/cavium-octeon/octeon_3xxx.dts | 314 +++++++++++++++++++++++++++++++
>>>> arch/mips/cavium-octeon/octeon_68xx.dts | 99 ++++++++++
>>>> 4 files changed, 428 insertions(+), 0 deletions(-)
>>>> create mode 100644 arch/mips/cavium-octeon/.gitignore
>>>> create mode 100644 arch/mips/cavium-octeon/octeon_3xxx.dts
>>>> create mode 100644 arch/mips/cavium-octeon/octeon_68xx.dts
>>>>
>>>> diff --git a/arch/mips/cavium-octeon/.gitignore b/arch/mips/cavium-octeon/.gitignore
>>>> new file mode 100644
>>>> index 0000000..39c9686
>>>> --- /dev/null
>>>> +++ b/arch/mips/cavium-octeon/.gitignore
>>>> @@ -0,0 +1,2 @@
>>>> +*.dtb.S
>>>
>>> .dtb.S?
>>
>> I think I have the correct .gitignore syntax.
>
> What I meant was, where are you generating .dtb.S files that you need
> to ignore them?
>

They are a byproduct of $(call cmd,dtc).

Normally make removes them automatically, but if you abort at just the
right time, they can be left around.

If it is objectionable, I can just remove that .gitignore bit.


>>>> + compatible = "octeon,octeon";
>>>
>>> There's no model number at all for this board?
>>
>>
>> I think it should be:
>>
>> compatible = "octeon,octeon-3860";
>
> That looks better.
>
> Also, the part before the comma is generally the vendor, so I would
> have expected cavium,XXX throughout rather than octeon,XXX.

OK, I will do that instead.


[...]
>>>> + device_type = "network";
>>>> + model = "mgmt";
>>>> + reg =<0x10700 0x00100000 0x0 0x100>, /* MIX */
>>>> +<0x11800 0xE0000000 0x0 0x300>, /* AGL */
>>>> +<0x11800 0xE0000400 0x0 0x400>, /* AGL_SHARED */
>>>> +<0x11800 0xE0002000 0x0 0x8>; /* AGL_PRT_CTL */
>>>> + unit-number =<0>;
>>>
>>> What is this 'unit-number' property for?
>>
>> The AGL_SHARED register bank is shared among all the octeon-5230-mii
>> devices. the 'unit-number' indicates the bit-field index that this
>> device should use within those registers.
>
> Ok. 'cell-index' is the normal property name for this sort of
> purpose.

Thanks, I will use 'cell-index'.

David Daney




\
 
 \ /
  Last update: 2011-02-24 02:59    [W:1.056 / U:0.016 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site