From: Sebastian Reichel <sre-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
To: Tony Lindgren <tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
Cc: "Benoît Cousson"
<bcousson-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>,
"Rob Herring" <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
"Mark Rutland" <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH 0/8] Droid4 DTS update
Date: Fri, 3 Mar 2017 12:01:44 +0100 [thread overview]
Message-ID: <20170303110144.seoxa6xwago7q5ir@earth> (raw)
In-Reply-To: <20170302014910.GM20572-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
[-- Attachment #1: Type: text/plain, Size: 2462 bytes --]
Hi,
On Wed, Mar 01, 2017 at 05:49:11PM -0800, Tony Lindgren wrote:
> * Tony Lindgren <tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org> [170301 17:45]:
> > * Sebastian Reichel <sre-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> [170301 17:10]:
> > > Hi,
> > >
> > > Tony, since your omap4-droid4 dts patches were not
> > > yet in a public repository and they would merge conflict
> > > with my patches I put them all together into a single
> > > patchset. I also added my Tested-By to your patches.
> >
> > OK
> >
> > > The mounting-matrix for the sensors and the touchscreen's
> > > rotation assume, that the phone's default rotation is vertical.
> > > I think that's sensible, because
> > >
> > > * Android on the phone uses this as default rotation
> > > (like almost? all other phones)
> > > * volume buttons make sense (in horizontal position one
> > > would expect, that left button means lower volume. I
> > > know at least Maemo swaps them)
> > >
> > > That should cover all mainline drivers.
> >
> > Heheheh cool looks like you configured compass and accelerometer :)
> >
> > I have a better touchscreen patch that also configures the wakeirq
> > that I did not post yet at http://muru.com/linux/d4/patches/ in the
> > updated set droid4-wip-next-20170217.tar.gz.
> >
> > I have few more interesting ones coming, I got the HDMI working
> > reliably over the weekend. I was planning on waiting until -rc1 to
> > post them but let's see if I can send few patches out before that.
>
> Oh and I have the poweroff-gpio patch and tmp105 patch.
>
> And the LMU backlight patch should be OK to go as the binding was
> already acked a while back.
>
> Some of the command mode LCD patches could be sent out too, and
> if the timing patch is OK then we can configure the LCD in the dts
> too. Have you had a chance to look at those related to what all I
> broke for n950?
I took your patchset and fixed it for N950. The main problem was,
that you dropped the fifo size update, which is required for OMAP3
(HW bug). Here is a branch based on recent master branch from
torvalds. I had a quick test with droid4 and dmesg still looked fine,
but I did not have the lm3532 patches in that branch and nothing is
visible without display backlight. Can you check if the branch works
for you on D4?
http://git.kernel.org/cgit/linux/kernel/git/sre/linux-n900.git/log/?h=n950-display-tony
-- Sebastian
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2017-03-03 11:01 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-02 1:08 [PATCH 0/8] Droid4 DTS update Sebastian Reichel
2017-03-02 1:08 ` [PATCH 1/8] ARM: dts: omap4-droid4: Add gpio-keys Sebastian Reichel
2017-03-02 1:08 ` [PATCH 2/8] ARM: dts: omap4-droid4: Configure regulators Sebastian Reichel
2017-03-02 1:08 ` [PATCH 3/8] ARM: dts: motorola-cpcap-mapphone: add rtc Sebastian Reichel
2017-03-02 1:08 ` [PATCH 4/8] ARM: dts: motorola-cpcap-mapphone: add pwrbutton Sebastian Reichel
2017-03-02 1:08 ` [PATCH 5/8] ARM: dts: omap4-droid4: Add compass Sebastian Reichel
2017-03-02 1:08 ` [PATCH 6/8] ARM: dts: omap4-droid4: Add accelerometer Sebastian Reichel
2017-03-02 1:08 ` [PATCH 7/8] ARM: dts: omap4-droid4: Add touchscreen Sebastian Reichel
2017-03-02 1:09 ` [PATCH 8/8] ARM: dts: omap4-droid4: Add keypad support Sebastian Reichel
[not found] ` <20170302010900.28532-9-sre-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2017-03-02 1:41 ` Tony Lindgren
[not found] ` <20170302010900.28532-1-sre-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2017-03-02 1:37 ` [PATCH 0/8] Droid4 DTS update Tony Lindgren
2017-03-02 1:49 ` Tony Lindgren
[not found] ` <20170302014910.GM20572-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2017-03-03 11:01 ` Sebastian Reichel [this message]
2017-03-03 16:28 ` Tony Lindgren
[not found] ` <20170303162824.GW20572-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2017-03-04 16:26 ` Tony Lindgren
[not found] ` <20170304162655.GZ20572-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2017-03-04 23:50 ` Sebastian Reichel
[not found] ` <20170302013707.GK20572-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2017-03-03 11:17 ` Sebastian Reichel
2017-03-03 16:31 ` Tony Lindgren
2017-03-17 11:48 ` Pavel Machek
2017-03-17 14:00 ` Tony Lindgren
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=20170303110144.seoxa6xwago7q5ir@earth \
--to=sre-dgejt+ai2ygdnm+yrofe0a@public.gmane.org \
--cc=bcousson-rdvid1DuHRBWk0Htik3J/w@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \
--cc=robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.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