mplexity.
[1] https://lore.kernel.org/r/yhepsqgqoau1v...@pendragon.ideasonboard.com
Suggested-by: Laurent Pinchart
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/panel/panel-simple.c | 14 --
1 file changed, 14 deletions(-)
diff --git a/drivers/gpu/drm/panel/panel
Adding this link allows the panel code to do things like read the
EDID.
Signed-off-by: Douglas Anderson
---
(no changes since v1)
arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
b/arch/arm64/boot
e the fallback mode to use
(since the EDID reading was a bit broken).
Suggested-by: Andrzej Hajda
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 22 --
1 file changed, 22 deletions(-)
diff --git a/drivers/gpu/drm/bridge/ti-
As of commit 5186421cbfe2 ("drm: Introduce epoch counter to
drm_connector") the drm_get_edid() function calls
drm_connector_update_edid_property() for us. There's no reason for us
to call it again.
Signed-off-by: Douglas Anderson
---
As Laurent pointed out [1] this is actually a pre
c-bus" property.
I believe that this problem got worse, however, with commit
e814e688413a ("i2c: of: Try to find an I2C adapter matching the
parent"). Starting at that commit it would be even easier to
accidentally miss finding the adapter.
Signed-off-by: Douglas Anderson
---
This
use out of this cache.
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/panel/panel-simple.c | 17 ++---
1 file changed, 10 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/panel/panel-simple.c
b/drivers/gpu/drm/panel/panel-simple.c
index
-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/panel/panel-simple.c | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/panel/panel-simple.c
b/drivers/gpu/drm/panel/panel-simple.c
index c91e8aa108f7..40382c1be692 100644
--- a/drivers/gpu/drm/panel
ago
when we realized how inflexible it was.
Even if someone is using the bridge chip without the "refclk" they're
in no worse shape than they were before the (fairly recent) commit
58074b08c04a ("drm/bridge: ti-sn65dsi86: Read EDID blob over DDC").
Signed-off-by: Douglas Anders
of completion), but it seems simple enough
to just not even bother creating the bridge device until the AUX
channel probes. That's what we'll do.
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 82 ---
1 file changed, 63
Let's make the bridge use autosuspend with a 500ms delay. This is in
preparation for promoting DP AUX transfers to their own sub-driver so
that we're not constantly powering up and down the device as we
transfer all the chunks.
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers
No functional changes--this just makes the diffstat of a future change
easier to understand.
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 116 +-
1 file changed, 58 insertions(+), 58 deletions(-)
diff --git
's get rid of the old code to clean things
up.
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/panel/panel-simple.c | 24 +---
1 file changed, 5 insertions(+), 19 deletions(-)
diff --git a/drivers/gpu/drm/panel/panel-simple.c
b/drivers/gpu/drm/p
the "whole chip" stuff this is
actually quite easy now.
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 26 +-
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi86.
ty present in the bridge chip
for a backlight we'll end up with another chicken-and-egg
problem. If we allow the PWM to be a sub-driver too then it solves
this problem.
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/bridge/Kconfig| 1 +
drivers/gpu/drm
Let's:
- Set the drvdata as soon as it's allocated. This just sets up a
pointer so there's no downside here.
- Remove the useless call to i2c_set_clientdata() which is literally
the same thing as dev_set_drvdata().
No functional changes intended.
Signed-off-by: Douglas Anderson
Tiny cleanup for probe so we don't keep having to specify
">dev" or "pdata->dev". No functional changes intended.
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 26 --
1 file changed, 12 i
There's no devm_runtime_enable(), but it's easy to use
devm_add_action_or_reset() and means we don't need to worry about the
disable in our remove() routine or in error paths.
No functional changes intended by this change.
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu
Let's cleanup the debugfs code to:
- Check for errors.
- Use devm to manage freeing, which also means we don't need to store
a pointer in our structure.
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 32 +--
1 file
In preparation for splitting this driver into sub-drivers, let's
rename the main data structure so it's clear that it's holding data
for the whole device and not just the MIPI-eDP bridge part.
This is a no-op change.
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm
ad of just the MIPI to eDP
bridge part) included "sn65dsi86" somewhere in the name instead of
just "ti_sn_bridge".
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 84 +--
1 file changed, 42 insertions(+
If we just leave the detect() function as NULL then the upper layers
assume we're always connected. There's no reason for a stub.
Signed-off-by: Douglas Anderson
Reviewed-by: Andrzej Hajda
Reviewed-by: Laurent Pinchart
---
(no changes since v1)
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 12
We prepared the panel in pre_enable() so we should unprepare it in
post_disable() to match.
Signed-off-by: Douglas Anderson
Reviewed-by: Andrzej Hajda
Reviewed-by: Laurent Pinchart
---
Changes in v4:
- Reword commit mesage slightly.
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 4 ++--
1 file
mory this is a legit thing to assume these days and we
don't have to find some fallback to keep working if someone wants to
build their system without "PM".
Signed-off-by: Douglas Anderson
Link:
https://lore.kernel.org/r/20210402152701.v3.12.I9e8bd33b49c496745bfac58ea9ab418bd3b6f5ce
Let's make the remove() function strictly the reverse of the probe()
function so it's easier to reason about.
This patch was created by code inspection and should move us closer to
a proper remove.
Signed-off-by: Douglas Anderson
Reviewed-by: Andrzej Hajda
Reviewed-by: Laurent Pinchart
-off-by: Douglas Anderson
Reviewed-by: Robert Foss
Reviewed-by: Bjorn Andersson
Reviewed-by: Stephen Boyd
Reviewed-by: Laurent Pinchart
Reviewed-by: Andrzej Hajda
---
(no changes since v2)
Changes in v2:
- Removed 2nd paragraph in commit message.
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 11
A random comment inside a function had "/**" in front of it. That
doesn't make sense. Remove.
Signed-off-by: Douglas Anderson
Reviewed-by: Andrzej Hajda
Reviewed-by: Laurent Pinchart
---
(no changes since v1)
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 2 +-
1 file changed, 1 inser
The drm_bridge_chain_pre_enable() is not the proper opposite of
drm_bridge_chain_post_disable(). It continues along the chain to
_before_ the starting bridge. Let's fix that.
Fixes: 05193dc38197 ("drm/bridge: Make the bridge chain a double-linked list")
Signed-off-by: Douglas Anderson
in v2:
- Removed 2nd paragraph in commit message.
Douglas Anderson (27):
drm/bridge: Fix the stop condition of drm_bridge_chain_pre_enable()
drm/bridge: ti-sn65dsi86: Simplify refclk handling
drm/bridge: ti-sn65dsi86: Remove incorrectly tagged kerneldoc comment
drm/bridge: ti-sn65dsi86: Re
and we
don't have to find some fallback to keep working if someone wants to
build their system without "PM".
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/panel/Kconfig| 1 +
drivers/gpu/drm/panel/panel-simple.c | 93 +++
-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi86.c
b/drivers/gpu/drm/bridge/ti-sn65dsi86.c
index fb50f9f95b0f..3b61898cf9cb 100644
--- a/drivers/gpu/drm/bridge/ti
we realized how inflexible it was.
Even if someone is using the bridge chip without the "refclk" they're
in no worse shape than they were before the (fairly recent) commit
58074b08c04a ("drm/bridge: ti-sn65dsi86: Read EDID blob over DDC").
Signed-off-by: Douglas Anderson
---
(no ch
If the bridge (and panel) haven't been powered on then AUX transfers
just won't work. Let's just fail them instantly.
Signed-off-by: Douglas Anderson
---
If the patch ("drm/panel: panel-simple: Use runtime pm to avoid
excessive unprepare / prepare") is accepted then we could conside
fully off. Then we'll likely have to wait the minimum
time between power off and power on.
Fixes: 58074b08c04a ("drm/bridge: ti-sn65dsi86: Read EDID blob over DDC")
Signed-off-by: Douglas Anderson
---
Changes in v3:
- Rebased now that we're not moving EDID caching to the core.
- Separating ou
As of commit 5186421cbfe2 ("drm: Introduce epoch counter to
drm_connector") the drm_get_edid() function calls
drm_connector_update_edid_property() for us. There's no reason for us
to call it again.
Signed-off-by: Douglas Anderson
Reviewed-by: Andrzej Hajda
---
(no changes since v1)
If we just leave the detect() function as NULL then the upper layers
assume we're always connected. There's no reason for a stub.
Signed-off-by: Douglas Anderson
Reviewed-by: Andrzej Hajda
---
(no changes since v1)
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 12
1 file changed, 12
We prepared the panel in pre_enable() so we should unprepare it in
post_disable() to match.
This becomes important once we start using pre_enable() and
post_disable() to make sure things are powered on (and then off again)
when reading the EDID.
Signed-off-by: Douglas Anderson
Reviewed
A random comment inside a function had "/**" in front of it. That
doesn't make sense. Remove.
Signed-off-by: Douglas Anderson
Reviewed-by: Andrzej Hajda
---
(no changes since v1)
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Let's make the remove() function strictly the reverse of the probe()
function so it's easier to reason about.
This patch was created by code inspection and should move us closer to
a proper remove.
Signed-off-by: Douglas Anderson
Reviewed-by: Andrzej Hajda
---
Changes in v3:
- Removed "
-off-by: Douglas Anderson
Reviewed-by: Robert Foss
Reviewed-by: Bjorn Andersson
Reviewed-by: Stephen Boyd
Reviewed-by: Laurent Pinchart
Reviewed-by: Andrzej Hajda
---
(no changes since v2)
Changes in v2:
- Removed 2nd paragraph in commit message.
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 11
The drm_bridge_chain_pre_enable() is not the proper opposite of
drm_bridge_chain_post_disable(). It continues along the chain to
_before_ the starting bridge. Let's fix that.
Fixes: 05193dc38197 ("drm/bridge: Make the bridge chain a double-linked list")
Signed-off-by: Douglas Anderson
ssage.
- Rebased now that we're not moving EDID caching to the core.
- Separating out patch to block AUX channel when not powered.
- Added note about boot speed implications.
- ("Fail aux transfers right away if not powered") split out for v3.
Changes in v2:
- Removed 2nd paragraph in commit
and we
don't have to find some fallback to keep working if someone wants to
build their system without "PM".
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/panel/Kconfig| 1 +
drivers/gpu/drm/panel/panel-simple.c | 93 +++
we realized how inflexible it was.
Even if someone is using the bridge chip without the "refclk" they're
in no worse shape than they were before the (fairly recent) commit
58074b08c04a ("drm/bridge: ti-sn65dsi86: Read EDID blob over DDC").
Signed-off-by: Douglas Anderson
---
(no ch
ailing aux transfers if we're not powered.
- If the first read of the EDID fails we try again after powering.
Fixes: 58074b08c04a ("drm/bridge: ti-sn65dsi86: Read EDID blob over DDC")
Signed-off-by: Douglas Anderson
---
Depending on what people think of the other patches in this series,
some
Now that we have the patch ("drm/edid: Use the cached EDID in
drm_get_edid() if eDP") we no longer need to maintain our own
cache. Drop this code.
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 22 +-
1 file
-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi86.c
b/drivers/gpu/drm/bridge/ti-sn65dsi86.c
index 92498900c58d..20c3b13939c2 100644
--- a/drivers/gpu/drm/bridge/ti
the EDID once then we've got the EDID and we
should never need to read it again. For now we'll only use the cache
for eDP both because it's more important and extra safe.
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/drm_edid.c | 32
1 file
We prepared the panel in pre_enable() so we should unprepare it in
post_disable() to match.
This becomes important once we start using pre_enable() and
post_disable() to make sure things are powered on (and then off again)
when reading the EDID.
Signed-off-by: Douglas Anderson
---
(no changes
As of commit 5186421cbfe2 ("drm: Introduce epoch counter to
drm_connector") the drm_get_edid() function calls
drm_connector_update_edid_property() for us. There's no reason for us
to call it again.
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/bridge/ti-s
If we just leave the detect() function as NULL then the upper layers
assume we're always connected. There's no reason for a stub.
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 12
1 file changed, 12 deletions(-)
diff --git
The register() / attach() for MIPI happen in the bridge's
attach(). That means that the inverse belongs in the bridge's
detach().
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 15 +--
1 file changed, 9 insertions(+), 6 deletions
orphaning memory until better handling of the
EDID happens.
This patch was created by code inspection and should move us closer to
a proper remove.
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 15 ---
1 file changed, 8 insertions
A random comment inside a function had "/**" in front of it. That
doesn't make sense. Remove.
Signed-off-by: Douglas Anderson
---
(no changes since v1)
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/bridge/ti
-off-by: Douglas Anderson
Reviewed-by: Robert Foss
Reviewed-by: Bjorn Andersson
Reviewed-by: Stephen Boyd
Reviewed-by: Laurent Pinchart
---
Changes in v2:
- Removed 2nd paragraph in commit message.
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 11 +++
1 file changed, 3 insertions(+), 8
The drm_bridge_chain_pre_enable() is not the proper opposite of
drm_bridge_chain_post_disable(). It continues along the chain to
_before_ the starting bridge. Let's fix that.
Fixes: 05193dc38197 ("drm/bridge: Make the bridge chain a double-linked list")
Signed-off-by: Dougla
c74b876e68@changeid/
[2]
https://lore.kernel.org/r/161706912161.3012082.17313817257247946...@swboyd.mtv.corp.google.com/
Changes in v2:
- Removed 2nd paragraph in commit message.
Douglas Anderson (14):
drm/bridge: Fix the stop condition of drm_bridge_chain_pre_enable()
drm/bridge: ti-sn65dsi86
at might be
attached and other differences were handled by EDID. This patch
attempts to set a new precedent and avoid the need for the white lie.
Signed-off-by: Douglas Anderson
---
.../devicetree/bindings/display/panel/panel-simple.yaml | 4
1 file changed, 4 insertions(+)
diff --
As talked about in the previous patch ("dt-bindings: display: simple:
Add the panel on sc7180-trogdor-pompom") any number of panels might be
attached.
Let's stop specifying a single panel and just specify that we have a
"pompom panel".
Signed-off-by: Douglas Anderson
---
Be
delay" imposed by the
Starry panel.
Signed-off-by: Douglas Anderson
---
For overall discussion about the idea of defining a generic panel at
the board level, please respond in the bindings patch so we can keep
all discussion in one place.
drivers/gpu/drm/panel/panel-simple.c | 12
Match what's downstream for this board.
Cc: Srinivasa Rao Mandadapu
Cc: Ajit Pandey
Cc: Judy Hsiao
Cc: Cheng-Yi Chiang
Cc: Stephen Boyd
Cc: Matthias Kaehlcke
Signed-off-by: Douglas Anderson
---
This applies atop the patch (arm64: dts: qcom: sc7180-trogdor: Add
lpass dai link for I2S driver
Chiang
Cc: Stephen Boyd
Cc: Matthias Kaehlcke
Signed-off-by: Douglas Anderson
---
This applies atop the patch ("arm64: dts: qcom: Add sound node for
sc7180-trogdor-coachz") [1].
NOTE: downstream this property was present in each of the board
revisions. There's actually no longer
Match what's downstream for this board.
Cc: Srinivasa Rao Mandadapu
Cc: Ajit Pandey
Cc: Judy Hsiao
Cc: Cheng-Yi Chiang
Cc: Stephen Boyd
Cc: Matthias Kaehlcke
Signed-off-by: Douglas Anderson
---
This applies atop the patch (arm64: dts: qcom: sc7180-trogdor: Add
lpass dai link for I2S driver
This was present downstream. Add upstream too.
Cc: Srinivasa Rao Mandadapu
Cc: Ajit Pandey
Cc: Judy Hsiao
Cc: Cheng-Yi Chiang
Cc: Stephen Boyd
Cc: Matthias Kaehlcke
Signed-off-by: Douglas Anderson
---
This applies atop the patch ("arm64: dts: qcom: Add sound node for
sc7180-trogdor-c
c: Rob Clark
Fixes: 58fd7ae621e7 ("arm64: dts: qcom: sc7180: Update dts for DP phy inside
QMP phy")
Reported-by: Dmitry Baryshkov
Signed-off-by: Douglas Anderson
---
arch/arm64/boot/dts/qcom/sc7180.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/b
rogdor dts. Let's do it now.
Signed-off-by: Stephen Boyd
[dianders: updated desc]
Signed-off-by: Douglas Anderson
---
Sorry for missing this in my last batch. I think my eyes glazed over
everything "dp" related as not-done-yet even though this one is easy
to do now.
arch/arm64/boot/dts/qc
doesn't fit in 32-bits (the old code
would have truncated silently).
Signed-off-by: Douglas Anderson
---
I haven't done any more than compile-test this. Mostly I'm just
writing this patch because it helped provide inspiration for the
general API function.
drivers/soc/qcom/cpr.c | 43
t;big endian" version would require adding "big
endian" support to nvmem_shift_read_buffer_in_place().
Signed-off-by: Douglas Anderson
---
This is a logical follow-up to:
https://lore.kernel.org/r/20210227002603.3260599-1-diand...@chromium.org/
...but since it doesn't really share
Let's use the newly-added nvmem_cell_read_variable_le_u32() to future
proof ourselves a little bit.
Signed-off-by: Douglas Anderson
---
This is based on my previous patch ("drm/msm: Fix speed-bin support
not to access outside valid memory") which has already landed in
msm-next. In
r
with doing them before userspace requests anything.
Fixes: 58074b08c04a ("drm/bridge: ti-sn65dsi86: Read EDID blob over DDC")
Signed-off-by: Douglas Anderson
---
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 98 ---
1 file changed, 88 insertions(+), 10 deletions(-)
This patch is _only_ code motion to prepare for the patch
("drm/bridge: ti-sn65dsi86: Properly get the EDID, but only if
refclk") and make it easier to understand.
Signed-off-by: Douglas Anderson
---
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 196 +-
1 file c
: this makes things look a tad bit asymmetric now since we check
for NULL before clk_prepare_enable() but not for
clk_disable_unprepare(). This seemed OK to me. We already have to
check for NULL in the enable case anyway so why not avoid the extra
call?
Signed-off-by: Douglas Anderson
---
drivers
Kaehlcke
Cc: Stephen Boyd
Cc: Tzung-Bi Shih
Cc: Judy Hsiao
Signed-off-by: Douglas Anderson
Reviewed-by: Matthias Kaehlcke
---
NOTE: technically the "google,cros-ec-mkbp-proximity" binding hasn't
landed upstream yet. However it has a Reviewed-by from Rob and the
series in general sounds a
[1]
https://lore.kernel.org/r/20210115144345.v2.4.I6889e21811df6adaff5c5b8a8c80fda0669ab3a5@changeid
Cc: Stephen Boyd
Signed-off-by: Douglas Anderson
Reviewed-by: Matthias Kaehlcke
---
Changes in v2:
- Add link to Acked bindings.
- Proper compatible for the "correct" panel (added
m/c/2719075> for the
keyboard.
Cc: Philip Chen
Cc: Matthias Kaehlcke
Cc: Stephen Boyd
Cc: Tzung-Bi Shih
Cc: Judy Hsiao
Signed-off-by: Douglas Anderson
Reviewed-by: Matthias Kaehlcke
---
Changes in v2:
- Rebase atop commit 7822de1a2946 ("arm64:...1 for charger temperature")
-
From: Matthias Kaehlcke
Lazor rev3 and later use the 'lite' version of the SC7180 SoC.
Cc: Stephen Boyd
Signed-off-by: Matthias Kaehlcke
[dianders: Adjust commit message which referred to downstream history]
Signed-off-by: Douglas Anderson
---
(no changes since v1)
arch/arm64/boot/dts
From: Venkata Lakshmi Narayana Gubba
Removed voting for RPMH_RF_CLK2 which is not required as it is
getting managed by BT SoC through SW_CTRL line.
Cc: Matthias Kaehlcke
Signed-off-by: Venkata Lakshmi Narayana Gubba
Signed-off-by: Douglas Anderson
Reviewed-by: Matthias Kaehlcke
-by: Douglas Anderson
Reviewed-by: Matthias Kaehlcke
---
(no changes since v1)
arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor.dtsi | 4
1 file changed, 4 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor.dtsi
b/arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor.dtsi
index
no coachz upstream yet]
Signed-off-by: Douglas Anderson
Reviewed-by: Matthias Kaehlcke
---
(no changes since v1)
arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi | 4
1 file changed, 4 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
b/arch/arm64/boot/dts/qcom/sc7180
Since we're not using hogs, we rely on AP firmware to set all the
control pins correctly.
Cc: Stephen Boyd
Cc: Craig Hesling
Signed-off-by: Alexandru M Stan
[dianders: adjusted since coachz isn't upstream yet]
Reviewed-by; Matthias Kaehlcke
Signed-off-by: Douglas Anderson
---
(no changes
From: Stephen Boyd
Configure the pen to be a wakeup source only when the pen is ejected
instead of both when the pen is ejected and inserted. This corresponds
to wake source requirements.
Signed-off-by: Stephen Boyd
Signed-off-by: Douglas Anderson
Reviewed-by: Matthias Kaehlcke
Let's avoid a bit of duplication by pushing this up to the trogdor.dtsi
file.
Signed-off-by: Douglas Anderson
Reviewed-by: Matthias Kaehlcke
---
(no changes since v1)
.../boot/dts/qcom/sc7180-trogdor-lazor.dtsi | 21 ---
.../arm64/boot/dts/qcom/sc7180-trogdor-r1.dts | 19
In general pinconf belongs in board files, not SoC files. Move it to
the only current user (trogdor). Also adjust the drive strengths and
pulls.
Cc: V Sujith Kumar Reddy
Cc: Srinivasa Rao Mandadapu
Cc: Tzung-Bi Shih
Cc: Judy Hsiao
Signed-off-by: Douglas Anderson
Reviewed-by: Matthias
-by: Douglas Anderson
---
(no changes since v1)
arch/arm64/boot/dts/qcom/sc7180.dtsi | 23 ---
1 file changed, 16 insertions(+), 7 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi
b/arch/arm64/boot/dts/qcom/sc7180.dtsi
index 1ea3344ab62c..60248a6757d8 100644
after more boards are
supported.
Signed-off-by: Douglas Anderson
Reviewed-by: Matthias Kaehlcke
---
(no changes since v1)
.../boot/dts/qcom/sc7180-trogdor-lazor.dtsi | 14 ++
arch/arm64/boot/dts/qcom/sc7180-trogdor-r1.dts | 16
arch/arm64/boot/dts/qcom/sc7180-tr
r (1):
arm64: dts: qcom: sc7180: add GO_LAZOR variant property for lazor
Alexandru M Stan (1):
arm64: dts: qcom: sc7180-trogdor: Remove fp control pins in prep for
coachz
Douglas Anderson (6):
arm64: dts: qcom: Move sc7180 MI2S config to board files and make
pulldown
arm64: dts: qcom
nd the big case that will change is the Qualcomm "CPR" driver
which seems to handle the length properly (it could probably be
simplified now, actually).
Signed-off-by: Douglas Anderson
---
drivers/nvmem/core.c | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/drivers
rsions just in case there was
an instance where someone was using this API to read big endian data
on a big endian machine and it happened to be working because there
was no bit offset.
Signed-off-by: Douglas Anderson
---
I will freely admit that I always end up thinking in circles and
getting diz
(since
the value is 8 bits big) or the u32 version (just for fun). At the
moment you need to pick the accessor that exactly matches the length
the cell was specified as in the device tree. Hopefully future
patches to the nvmem subsystem will fix this.
Fixes: fe7952c629da ("drm/msm: Add speed-b
fying a length of "4" for the GPU
speed bin cells.
About applying the patches.
- GPU patch can land on its own. No need for the nvmem patches.
- nvmem patches can land on their own too.
- If the second nvmem patch lands without the first, however, it will
break the GPU patch.
Douglas Ande
From: Matthias Kaehlcke
Lazor rev3 and later use the 'lite' version of the SC7180 SoC.
Cc: Stephen Boyd
Signed-off-by: Matthias Kaehlcke
[dianders: Adjust commit message which referred to downstream history]
Signed-off-by: Douglas Anderson
---
arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor
-by: Douglas Anderson
---
arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor.dtsi | 4
1 file changed, 4 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor.dtsi
b/arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor.dtsi
index 5efa04d0d256..6b10b96173e8 100644
--- a/arch/arm64/boot
Kaehlcke
Cc: Stephen Boyd
Cc: Tzung-Bi Shih
Cc: Judy Hsiao
Signed-off-by: Douglas Anderson
---
NOTE: technically the "google,cros-ec-mkbp-proximity" binding hasn't
landed upstream yet. However it has a Reviewed-by from Rob and the
series in general sounds about ready to land. If necess
m/c/2719075> for the
keyboard.
Cc: Philip Chen
Cc: Matthias Kaehlcke
Cc: Stephen Boyd
Cc: Tzung-Bi Shih
Cc: Judy Hsiao
Signed-off-by: Douglas Anderson
---
arch/arm64/boot/dts/qcom/Makefile | 4 +
.../dts/qcom/sc7180-trogdor-pompom-r1-lte.dts | 14 +
.../dts/qcom/sc7180-t
e "innolux,n116bge" timings for now,
though.
- The wonky special dts just for "-rev4" arguably doesn't need to go
upstream since they weren't widely distributed, but since a few
people have them we might as well. If it ever causes problems we
can delete it.
Cc: Stephe
From: Venkata Lakshmi Narayana Gubba
Removed voting for RPMH_RF_CLK2 which is not required as it is
getting managed by BT SoC through SW_CTRL line.
Cc: Matthias Kaehlcke
Signed-off-by: Venkata Lakshmi Narayana Gubba
Signed-off-by: Douglas Anderson
---
arch/arm64/boot/dts/qcom/sc7180
Since we're not using hogs, we rely on AP firmware to set all the
control pins correctly.
Cc: Stephen Boyd
Cc: Craig Hesling
Signed-off-by: Alexandru M Stan
[dianders: adjusted since coachz isn't upstream yet]
Signed-off-by: Douglas Anderson
---
arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
no coachz upstream yet]
Signed-off-by: Douglas Anderson
---
arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi | 4
1 file changed, 4 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
index f3a99c801582..69bf600e1c9f 100644
after more boards are
supported.
Signed-off-by: Douglas Anderson
---
.../boot/dts/qcom/sc7180-trogdor-lazor.dtsi | 14 ++
arch/arm64/boot/dts/qcom/sc7180-trogdor-r1.dts | 16
arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi | 14 --
3 files c
In general pinconf belongs in board files, not SoC files. Move it to
the only current user (trogdor). Also adjust the drive strengths and
pulls.
Cc: V Sujith Kumar Reddy
Cc: Srinivasa Rao Mandadapu
Cc: Tzung-Bi Shih
Cc: Judy Hsiao
Signed-off-by: Douglas Anderson
---
This should replace
From: Stephen Boyd
Configure the pen to be a wakeup source only when the pen is ejected
instead of both when the pen is ejected and inserted. This corresponds
to wake source requirements.
Signed-off-by: Stephen Boyd
Signed-off-by: Douglas Anderson
---
arch/arm64/boot/dts/qcom/sc7180
1 - 100 of 1431 matches
Mail list logo