From: Hans de Goede <hdegoede@redhat.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/3] sunxi: Ippo_q8h: Enable OTG VBUS detection using AXP223
Date: Tue, 10 Mar 2015 12:08:40 +0100 [thread overview]
Message-ID: <54FED0B8.70900@redhat.com> (raw)
In-Reply-To: <1425887057-14614-1-git-send-email-wens@csie.org>
Hi,
On 09-03-15 08:44, Chen-Yu Tsai wrote:
> Hi Hans,
>
> This series fixes otg support on the A23 q8h tablets. It adds support
> for the AXP's (AXP221/223 for now) VBUS detection function.
>
> I've tested this with a USB wireless keyboard dongle, which works fine.
> More importantly, I'm using this and your mainline kernel musb work plus
> a few A23 dts patches to get musb working on my q8h tablet in host mode.
> I have an ethernet dongle plugged in, which works reasonably well.
Thanks for working on this, I've merged this into u-boot-sunxi/next
and I plan to send a pull-req for this to get it into v2015.4-rc# soon,
as this fixes a regression which I (deliberately) introduced.
It would be cool if you do similar patches for the mainline kernel, for
the mainline kernel I would really like to see the status bit exported
as part of the gpio-chip which we still need to add to the axp2xx code for
the axp2xx gpio pins, this way the existing kernel musb code can just use
it.
I also believe that Ian was right when he said that we should probably
also make these bits special gpio-s in u-boot rather then using custom
API-s for them. But this is something which we can fix later, for now
this fixes the regression and as such is good enough.
> Note that U-Boot never calls musb_shutdown() to do proper cleanup, like
> disabling VBUS, so the current state carries on into Linux.
Hmm true, I guess that is not necessarily a problem, but something which
we do need to keep in mind.
Regards,
Hans
next prev parent reply other threads:[~2015-03-10 11:08 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-09 7:44 [U-Boot] [PATCH 0/3] sunxi: Ippo_q8h: Enable OTG VBUS detection using AXP223 Chen-Yu Tsai
2015-03-09 7:44 ` [U-Boot] [PATCH 1/3] sunxi: axp221: Add VBUS detection support Chen-Yu Tsai
2015-03-09 7:44 ` [U-Boot] [PATCH 2/3] sunxi: musb: Support checking VBUS using AXP221 PMIC Chen-Yu Tsai
2015-03-09 7:44 ` [U-Boot] [PATCH 3/3] sunxi: Ippo_q8h defconfigs: Enable otg vbus detection using AXP223 PMIC Chen-Yu Tsai
2015-03-10 11:08 ` Hans de Goede [this message]
2015-03-10 11:49 ` [U-Boot] [PATCH 0/3] sunxi: Ippo_q8h: Enable OTG VBUS detection using AXP223 Chen-Yu Tsai
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=54FED0B8.70900@redhat.com \
--to=hdegoede@redhat.com \
--cc=u-boot@lists.denx.de \
/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