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

From: Viresh Kumar
Date: Sat Oct 13 2012 - 10:10:47 EST


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 why do you keep the AUXDATA
>
> Because the compact flash driver expects it via platform data and not
> via DT. yes, it
> can be changed, but that wasn't the target of this patchset. If it has
> to be done, then
> it has to be done separately 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/