Question on mpc52xx_common.c

[Matt Sealey]

Grant Likely wrote:
> (cc'ing the mailing list in my reply)
> 
> On Thu, Apr 3, 2008 at 10:27 AM, Detlev Zundel <dzu at denx.de> wrote:
>> I wrote,
>>
>>  > debugging a lite5200b kernel (linux-2.6-denx equivalent of 2.6.25-rc8) I
>>  > noticed that in mpc52xx_map_common_devices (mpc52xx_common.c:161) the
>>  > cdm module is tentatively mapped.  It seems strange to me that there is no
>>  > error checking here.  Is a FDT without an cdm entry usable at all?
> 
> Sure, why not?  If the firmware has already set it up correctly and no
> devices using it are in use, then the kernel should be okay.  :-)
> That said, I can't imagine choosing to not put the cdm node into the
> device tree.

*ahem* Efika.

-- 
Matt Sealey <matt at genesi-usa.com>
Genesi, Manager, Developer Relations