[PATCH v4 02/10] ARM: DaVinci: base address and lpsc assignments for dm6467

[Rajashekhara, Sudhakar]

Hi,

There are many variants of DM644x like DM6443 and DM6446, but we have only DM6467, hence I was planning to use DM6467 instead of DM646x.

Regards, Sudhakar

________________________________________
From: Sergei Shtylyov [sshtylyov at ru.mvista.com]
Sent: Saturday, September 13, 2008 7:11 PM
To: Rajashekhara, Sudhakar
Cc: davinci-linux-open-source at linux.davincidsp.com
Subject: Re: [PATCH v4 02/10] ARM: DaVinci: base address and lpsc assignments for dm6467

Hello.

Rajashekhara, Sudhakar wrote:

> Your comment is valid but I feel keeping all hardware related stuffs like base address, LPSC numbers and IRQ numbers in one file say hardware.h is appropriate. I am planning to do this as part of my next set of updated patches. There I'll combine the DM644x hardware.h, psc.h and irq.h. File hardware_dm6467.h will have base address, LPSC assignments and IRQ assignments for DM6467.
>

   OK.

> Also, I got an offline comment saying I have used DM646x and DM6467 interchangeably. I'll change this and use DM6467 consistently all across the files.
>

   I think this will be a waste of space since the assortment of the SoC
peripherals seem to be the same across all DM646x, isn't it?
Hence I'm suggesting the opposite, keep DM646x in the #define's and call
the header hardware_dm646x.h.

WBR, Sergei