Content-Class: urn:content-classes:message
MIME-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: 8BIT
Subject: RE: Xilinx SYSACE driver and 8-bit attachment
Date: Wed, 7 Oct 2009 08:24:28 -0600
In-Reply-To: <fa686aa40910070634r4a8dcf74qdcbfb3bde3413109@mail.gmail.com>
Thread-Topic: Xilinx SYSACE driver and 8-bit attachment
Thread-Index: AcpHUwAhH5aNw0SuTEanW48+j2280gABkofA
References: <1d3f23370910070013u6307bc42hf7ab243b52d007df@mail.gmail.com> <fa686aa40910070634r4a8dcf74qdcbfb3bde3413109@mail.gmail.com>
From: John Linn <John.Linn@xilinx.com>
To: "Grant Likely" <grant.likely@secretlab.ca>,
       "John Williams" <john.williams@petalogix.com>
CC: "Linux Kernel list" <linux-kernel@vger.kernel.org>,
       "Michal Simek" <michal.simek@petalogix.com>,
       "David DeBonis" <ddeboni@xilinx.com>,
       "Stephen Neuendorffer" <stephenn@xilinx.com>,
       "devicetree-discuss" <devicetree-discuss@lists.ozlabs.org>
Message-ID: <838c3c32-27d9-4a0a-a197-c7227b911942@VA3EHSMHS014.ehs.local>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2213
Lines: 55

I'll look to see if something has changed.  It's not clear to me if it has ever been right.

It look like we normally use 16 bit mode as that's how BSB generates the system by default.

Thanks,
John

> -----Original Message-----
> From: glikely@secretlab.ca [mailto:glikely@secretlab.ca] On Behalf Of Grant Likely
> Sent: Wednesday, October 07, 2009 7:35 AM
> To: John Williams
> Cc: Linux Kernel list; Michal Simek; John Linn; David DeBonis; Stephen Neuendorffer; devicetree-
> discuss
> Subject: Re: Xilinx SYSACE driver and 8-bit attachment
> 
> [Added devicetree-discuss to cc: list]
> 
> On Wed, Oct 7, 2009 at 1:13 AM, John Williams
> <john.williams@petalogix.com> wrote:
> > Hi Grant (it's your driver :) and David D ( and your DTS generator :)
> [...]
> > if (of_find_property(op->node, "8-bit", NULL))
> [...]
> > doesn't match the properties generated by Xilinx's device tree generator:
> [...]
> > ??????????????????????? xlnx,mem-width = <0x8>;
> >
> > So, the question is, which should change?
> 
> Well, obviously its a device tree generator bug, the driver code is
> perfect.  :-P
> 
> In all seriousness though, the '8-bit' property has been present for a
> while now and needs to be retained to not break existing users.  It
> would be okay for the driver to be modified to *also* check for the
> xlnx,mem-width property, but the better solution is to modify the
> device tree generator.  Plus, the '8-bit' property is the documented
> binding in Documentation/powerpc/dts-bindings/xilinx.txt
> 
> Cheers,
> g.
> 
> --
> Grant Likely, B.Sc., P.Eng.
> Secret Lab Technologies Ltd.


This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/