Re: [PATCH V2 3/3] ARM: SPEAr13xx: Pass DW DMAC platform data from DT

[Viresh Kumar]

On 14 October 2012 15:31, Jean-Christophe PLAGNIOL-VILLARD
<plagnioj@xxxxxxxxxxxx> wrote:
> On 19:40 Sat 13 Oct     , Viresh Kumar wrote:
>> On 13 October 2012 19:38, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
>> > On 13 October 2012 17:52, Jean-Christophe PLAGNIOL-VILLARD
>> > <plagnioj@xxxxxxxxxxxx> wrote:
>> >> On 14:18 Sat 13 Oct     , Viresh Kumar wrote:
>> >>>    On Oct 13, 2012 12:16 PM, "Jean-Christophe PLAGNIOL-VILLARD"
>> >>>    <plagnioj@xxxxxxxxxxxx> wrote:
>> >>>    >
>> >>>    > On 22:42 Fri 12 Oct     , Viresh Kumar wrote:
>> >>>    > > On 12 October 2012 21:51, Jean-Christophe PLAGNIOL-VILLARD
>> >>>    > > <plagnioj@xxxxxxxxxxxx> wrote:
>> >>>    > > >> >> +     OF_DEV_AUXDATA("arasan,cf-spear1340", MCIF_CF_BASE, NULL,
>> >>>    "cf"),
>> >>>    > > >> > ?/
>> >>>    > > >>
>> >>>    > > >> Sorry. can't get it :(
>> >>>    > > > what is the "cf" as paltfrom data
>> >>>    > >
>> >>>    > > This is dma bus_id string, which matches with what is passed from dtb.
>> >>>    > so pass if via dtb too
>> >>>
>> >>>    Yes. Already passed in 13xx.dtsi.
>>
>> Probably some confusion here. What i meant to say here is, dmac's
>> DT slave info has a node for cf and cf driver expects this string to come
>> via platform data currently.
>
> so use a phandle to connect them

The purpose of this patchset wasn't to fix how CF driver sends its filter
routine and its parameter. CF driver would be fixed later by ST guys.

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