[PATCH] [V3] Xilinx : Framebuffer Driver: Add PLB support and cleanup DCR
Grant Likely
grant.likely at secretlab.ca
Thu Apr 16 02:03:22 EST 2009
On Wed, Apr 15, 2009 at 9:58 AM, Stephen Neuendorffer
<stephen.neuendorffer at xilinx.com> wrote:
>> > - rc = of_address_to_resource(op->node, 0, &res);
>> > - if (rc) {
>> > - dev_err(&op->dev, "invalid address\n");
>> > - return rc;
>> > + /*
>> > + * To check whether the core is connected directly to DCR or PLB
>> > + * interface and initialize the tft_access accordingly.
>> > + */
>> > + p = (u32 *)of_get_property(op->node, "xlnx,dcr-splb-slave-if", NULL);
>>
>> Hmmm. This binding is undocumented. It would be better to make the
>> decision on the presence/absence of the dcr-reg and/or reg properties.
>
> For backward compatibility with the 'old' way, the device tree generator for this core has both dcr-reg and reg properties (where the reg has been translated back through the bridge).
So, what is in the regs and dcr-regs properties when DCR is used?
How about when MMIO is used?
g.
--
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
More information about the Linuxppc-dev
mailing list