[U-Boot] [PATCH 2/4 v4] arm: socfpga: dts: Add Cadence QSPI DT node to socfpga.dtsi

[Dinh Nguyen <dinguyen@opensource.altera.com>]

+CC: Graham Moore

On 11/10/2014 04:47 AM, Pavel Machek wrote:
> On Fri 2014-11-07 18:26:05, Stefan Roese wrote:
>> Hi Simon,
>>
>> On 07.11.2014 18:21, Simon Glass wrote:
>>>> +               qspi: spi at ff705000 {
>>>> +                       compatible = "cadence,qspi";
>>>> +                       #address-cells = <1>;
>>>> +                       #size-cells = <0>;
>>>> +                       reg = <0xff705000 0x1000>,
>>>> +                               <0xffa00000 0x1000>;
>>>> +                       interrupts = <0 151 4>;
>>>> +                       clocks = <&qspi_clk>;
>>>> +                       ext-decoder = <0>;  /* external decoder */
>>>> +                       num-chipselect = <4>;
>>>> +                       fifo-depth = <128>;
>>>> +                       bus-num = <2>;
>>>> +                       status = "disabled";
>>>> +               };
>>>> +
>>>
>>> Is this the same binding as Linux? Can you please bring in the binding
>>> document too? Some of these feel like they should have a
>>> "manufacture," prefix.
>>
>> Yes, its from Linux. But not from kernel.org but from the Altera
>> Rocketboards.org version (v3.16). So the bindings might change if this
>> driver would be pushed to mainline Linux.
> 
> I guess it would be good to submit the binding to Linux for review, so
> that we don't end up with two incompatible bindings...
> 

Graham has already submitted an RFC, but have not gotten any feedback yet.

http://marc.info/?l=linux-kernel&m=141417787314186&w=2