From: Pavan Kondeti <pkondeti@codeaurora.org>
To: Daniel Walker <dwalker@codeaurora.org>
Cc: Denis 'GNUtoo' Carikli <GNUtoo@no-log.org>,
linux-arm-msm@vger.kernel.org,
Greg Kroah-Hartman <gregkh@suse.de>
Subject: Re: msm gadget compilation failure with trout
Date: Mon, 31 Jan 2011 09:17:33 +0530 [thread overview]
Message-ID: <4D4630D5.6030704@codeaurora.org> (raw)
In-Reply-To: <1296234900.32421.2.camel@m0nster>
On 1/28/2011 10:45 PM, Daniel Walker wrote:
> On Fri, 2011-01-28 at 12:31 +0530, Pavan Kondeti wrote:
>> Hi Denis,
>>
>> On 1/28/2011 2:56 AM, Daniel Walker wrote:
>>> On Thu, 2011-01-27 at 22:12 +0100, Denis 'GNUtoo' Carikli wrote:
>>>> Hi,
>>>>
>>>> I've that:
>>>> drivers/usb/gadget/ci13xxx_msm.c: In function
>>>> 'ci13xxx_msm_notify_event':
>>>> drivers/usb/gadget/ci13xxx_msm.c:42:3: error: 'USB_AHBBURST' undeclared
>>>> (first use in this function)
>>>> drivers/usb/gadget/ci13xxx_msm.c:42:3: note: each undeclared identifier
>>>> is reported only once for each function it appears in
>>>> drivers/usb/gadget/ci13xxx_msm.c:43:3: error: 'USB_AHBMODE' undeclared
>>>> (first use in this function)
>>>> make[2]: *** [drivers/usb/gadget/ci13xxx_msm.o] Error 1
>>>> make[1]: *** [drivers/usb/gadget] Error 2
>>>> make[1]: *** Waiting for unfinished jobs....
>>>>
>>>> here's my setup:
>>>> * linux-next/master
>>>> ( git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git ) at
>>>> 47db3d1b0b3d69c6e977158ac354a972b81677bd
>>>> * trout(also known as htc dream or G1) board
>>>>
>>>> Should I include my defconfig?
>>>> Should I include more details?
>>>> Am I using the gadget wrong driver?
>>>>
>>
>> AHBMODE register offset is different for trout and AHBBURST register is
>> not present in trout.
>>
>> MSM gadget driver is supported only for QSD8x50 and MSM7x30 boards. USB
>> support is not added for trout which uses an external PHY unlike the
>> other targets. Please disable USB support in your defconfig.
>
> If it's not supported don't allow it to be compiled, but you need to
> make some sort of fix for this.
>
Agreed. I will submit a patch to fix this.
> btw, if it's not support why is there a 7X01 define used in the header ?
>
Sorry. It should not have been there. It was a copy-paste mistake.
--
Sent by a consultant of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.
prev parent reply other threads:[~2011-01-31 3:47 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-01-27 21:12 msm gadget compilation failure with trout Denis 'GNUtoo' Carikli
2011-01-27 21:26 ` Daniel Walker
2011-01-28 7:01 ` Pavan Kondeti
2011-01-28 17:15 ` Daniel Walker
2011-01-31 3:47 ` Pavan Kondeti [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4D4630D5.6030704@codeaurora.org \
--to=pkondeti@codeaurora.org \
--cc=GNUtoo@no-log.org \
--cc=dwalker@codeaurora.org \
--cc=gregkh@suse.de \
--cc=linux-arm-msm@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).