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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAKohpokj0Up23YEDTuSOHDgqcCuM3Q21cwnkAz5Ya=JhPKM98A@mail.gmail.com>
Date:	Sat, 13 Oct 2012 19:40:53 +0530
From:	Viresh Kumar <viresh.kumar@...aro.org>
To:	Jean-Christophe PLAGNIOL-VILLARD <plagnioj@...osoft.com>
Cc:	andriy.shevchenko@...ux.intel.com, vinod.koul@...el.com,
	spear-devel@...t.st.com, linux-kernel@...r.kernel.org,
	linux-arm-kernel@...ts.infradead.org,
	devicetree-discuss@...ts.ozlabs.org
Subject: Re: [PATCH V2 3/3] ARM: SPEAr13xx: Pass DW DMAC platform data from DT

On 13 October 2012 19:38, Viresh Kumar <viresh.kumar@...aro.org> wrote:
> On 13 October 2012 17:52, Jean-Christophe PLAGNIOL-VILLARD
> <plagnioj@...osoft.com> wrote:
>> On 14:18 Sat 13 Oct     , Viresh Kumar wrote:
>>>    On Oct 13, 2012 12:16 PM, "Jean-Christophe PLAGNIOL-VILLARD"
>>>    <plagnioj@...osoft.com> wrote:
>>>    >
>>>    > On 22:42 Fri 12 Oct     , Viresh Kumar wrote:
>>>    > > On 12 October 2012 21:51, Jean-Christophe PLAGNIOL-VILLARD
>>>    > > <plagnioj@...osoft.com> 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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/