Re: [PATCH 3/3] dt-bindings: Fix errors in 'if' schemas

2021-02-03 Thread Maxime Ripard
s as it applies to all entries. 'contains' is the correct schema > to use in the case of multiple entries. > > Cc: Herbert Xu > Cc: "David S. Miller" > Cc: Maxime Ripard > Cc: Chen-Yu Tsai > Cc: Eric Anholt > Cc: Nicolas Saenz Julienne > Cc: Florian Fainelli &g

Re: [PATCH] drm/vc4: remove unneeded variable: "ret"

2021-02-02 Thread Maxime Ripard
On Tue, Feb 02, 2021 at 04:23:38AM -0800, Bernard Zhao wrote: > remove unneeded variable: "ret". > > Signed-off-by: Bernard Zhao Applied, thanks maxime signature.asc Description: PGP signature

Re: [PATCH v4] ARM: dts: sun7i: a20: bananapro: Fix ethernet phy-mode

2021-02-01 Thread Maxime Ripard
On Mon, Feb 01, 2021 at 01:08:03PM +0100, Hermann Lauer wrote: > On Thu, Jan 28, 2021 at 03:59:37PM +0100, Maxime Ripard wrote: > > On Thu, Jan 28, 2021 at 12:18:42PM +0100, hermann.la...@uni-heidelberg.de > > wrote: > > > BPi Pro needs TX and RX delay for Gbit to work

Re: arm: sunxi: &83t: WARNING: CPU: 2 PID: 57 at drivers/thermal/thermal_core.c:563 thermal_zone_device_update

2021-01-31 Thread Maxime Ripard
On Sun, Jan 31, 2021 at 01:33:32PM +0800, Chen-Yu Tsai wrote: > Hi, > > On Sun, Jan 31, 2021 at 12:54 AM Corentin Labbe > wrote: > > > > Hello > > > > When booting next-20210128, I got the following warning on by bpim3 > > 6.148421] [ cut here ] > > [6.153145]

Re: [PATCH v2] ARM: dts: sun5i: Add dts for inet86v_rev2

2021-01-28 Thread Maxime Ripard
Hi, On Sun, Jan 24, 2021 at 08:39:03PM +0100, Alexandre GRIVEAUX wrote: > Add Inet 86V Rev 2 support, based upon Inet 86VS. > > The Inet 86V use SL1536 touchpanel controller, the Inet 86VS a GSL1680, > which make them both incompatible. > > Missing things: > - Accelerometer (MXC6225X) > -

Re: [PATCH v4] ARM: dts: sun7i: a20: bananapro: Fix ethernet phy-mode

2021-01-28 Thread Maxime Ripard
On Thu, Jan 28, 2021 at 12:18:42PM +0100, hermann.la...@uni-heidelberg.de wrote: > BPi Pro needs TX and RX delay for Gbit to work reliable and avoid high > packet loss rates. The realtek phy driver overrides the settings of the > pull ups for the delays, so fix this for BananaPro. > > Fix the

Re: [PATCH 2/2] soc: sunxi: mbus: Remove DE2 display engine compatibles

2021-01-28 Thread Maxime Ripard
On Wed, Jan 27, 2021 at 02:03:03PM +0100, Paul Kocialkowski wrote: > Hi, > > On Fri 15 Jan 21, 18:58, Paul Kocialkowski wrote: > > The DE2 display engine hardware takes physical addresses that do not > > need PHYS_BASE subtracted. As a result, they should not be present > > on the mbus driver

Re: [PATCH v5 16/20] net: stmmac: dwmac-sun8i: Prepare for second EMAC clock register

2021-01-28 Thread Maxime Ripard
add a parameter > to our syscon phandle. The driver will use this value as an index into > the regmap, so that we can address more than the first register, if > needed. > > Signed-off-by: Andre Przywara Acked-by: Maxime Ripard Maxime signature.asc Description: PGP signature

Re: [PATCH v5 12/20] dt-bindings: rtc: sun6i: Add H616 compatible string

2021-01-28 Thread Maxime Ripard
ob Herring Acked-by: Maxime Ripard maxime signature.asc Description: PGP signature

Re: [PATCH v5 15/20] dt-bindings: net: sun8i-emac: Add H616 compatible string

2021-01-28 Thread Maxime Ripard
evices together. > > Signed-off-by: Andre Przywara Acked-by: Maxime Ripard Maxime signature.asc Description: PGP signature

Re: [PATCH v5 06/20] mfd: axp20x: Allow AXP chips without interrupt lines

2021-01-28 Thread Maxime Ripard
is > registration, to avoid the driver to bail out completely. > > Signed-off-by: Andre Przywara Acked-by: Maxime Ripard Maxime signature.asc Description: PGP signature

Re: [PATCH v5 04/20] dt-bindings: mfd: axp20x: Add AXP305 compatible (plus optional IRQ)

2021-01-28 Thread Maxime Ripard
to the CPU, so make the "interrupts" property optional. > > Signed-off-by: Andre Przywara Acked-by: Maxime Ripard Maxime signature.asc Description: PGP signature

Re: [PATCH v2 00/15] drm/vc4: hdmi: Add CEC support for the BCM2711

2021-01-28 Thread Maxime Ripard
Hi! On Mon, Jan 25, 2021 at 10:03:44PM +0100, Nicolas Saenz Julienne wrote: > Hi, > > On Mon, 2021-01-11 at 15:22 +0100, Maxime Ripard wrote: > > Hi, > > > > Here's a series introducing the CEC support for the BCM2711 found on the > > RaspberryPi4. > >

Re: [PATCH v2 08/11] drm: Rename plane->state variables in atomic update and disable

2021-01-26 Thread Maxime Ripard
Hi Ville, On Fri, Jan 22, 2021 at 02:15:07PM +0200, Ville Syrjälä wrote: > On Thu, Jan 21, 2021 at 05:35:33PM +0100, Maxime Ripard wrote: > > Some drivers are storing the plane->state pointer in atomic_update and > > atomic_disable in a variable simply called state, whil

Re: [PATCH 12/21] clk: sunxi: clk-sun6i-ar100: Demote non-conformant kernel-doc header

2021-01-26 Thread Maxime Ripard
ot; > Cc: Michael Turquette > Cc: Stephen Boyd > Cc: Maxime Ripard > Cc: Chen-Yu Tsai > Cc: Jernej Skrabec > Cc: Boris BREZILLON > Cc: linux-...@vger.kernel.org > Cc: linux-arm-ker...@lists.infradead.org > Signed-off-by: Lee Jones > --- > drivers/clk/sunxi/

Re: [PATCH] pinctrl: sunxi: fix use-after-free in sunxi_pmx_free()

2021-01-26 Thread Maxime Ripard
On Tue, Jan 26, 2021 at 04:03:29PM +0100, Linus Walleij wrote: > On Tue, Jan 26, 2021 at 7:31 AM liu xiang wrote: > > > > Liu can you make a patch to Kconfig to just select REGULATOR? > > > Possibly even the specific regulator driver this SoC is using > > > if it is very specific for this

Re: [PATCH v4 00/21] arm64: sunxi: Initial Allwinner H616 SoC support

2021-01-26 Thread Maxime Ripard
Hi, On Mon, Jan 25, 2021 at 03:17:50PM +, Andre Przywara wrote: > Hi, > > an update from the v3 last week, to add support for the Allwinner H616 > SoC. Still based on the (updated) sunxi/for-next branch. > I am omitting the MMC and pinctrl patches now, as they have been taken > by Ulf and

Re: [PATCH v5 00/10] sunxi: Support IRQ wakeup from deep sleep

2021-01-25 Thread Maxime Ripard
On Sat, Jan 23, 2021 at 12:26:26AM -0600, Samuel Holland wrote: > On 1/22/21 4:47 AM, Maxime Ripard wrote: > > On Thu, Jan 21, 2021 at 07:33:54PM -0600, Samuel Holland wrote: > >> On 1/21/21 2:35 PM, Marc Zyngier wrote: > >>> On Sun, 17 Jan 2021 23:50:

Re: [PATCH v2 06/11] drm: Use state helper instead of plane state pointer in atomic_check

2021-01-25 Thread Maxime Ripard
Hi Ville, On Fri, Jan 22, 2021 at 02:07:22PM +0200, Ville Syrjälä wrote: > On Thu, Jan 21, 2021 at 05:35:31PM +0100, Maxime Ripard wrote: > > Many drivers reference the plane->state pointer in order to get the > > current plane state in their atomic_check hook, which would be

Re: [PATCH v5 00/10] sunxi: Support IRQ wakeup from deep sleep

2021-01-22 Thread Maxime Ripard
On Thu, Jan 21, 2021 at 07:33:54PM -0600, Samuel Holland wrote: > On 1/21/21 2:35 PM, Marc Zyngier wrote: > > On Sun, 17 Jan 2021 23:50:30 -0600, Samuel Holland wrote: > >> Allwinner sun6i/sun8i/sun50i SoCs (A31 and newer) have two interrupt > >> controllers: GIC and R_INTC. GIC does not support

[PATCH v2 06/11] drm: Use state helper instead of plane state pointer in atomic_check

2021-01-21 Thread Maxime Ripard
ne->state + old_plane_state ... } @ include depends on adds_old_state || replaces_old_state @ @@ #include @ no_include depends on !include && (adds_old_state || replaces_old_state) @ @@ + #include #include Signed-off-by: Maxime Ripard --- drivers/gpu/drm/imx/ipuv3-plane.c

[PATCH v2 11/11] drm/todo: Remove the drm_atomic_state todo item

2021-01-21 Thread Maxime Ripard
. Signed-off-by: Maxime Ripard --- Changes from v1: - New patch --- Documentation/gpu/todo.rst | 46 -- 1 file changed, 46 deletions(-) diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst index 009d8e6c7e3c..609794108f5a 100644

[PATCH v2 10/11] drm: Use state helper instead of the plane state pointer

2021-01-21 Thread Maxime Ripard
e Signed-off-by: Maxime Ripard --- drivers/gpu/drm/arc/arcpgu_crtc.c | 4 +++- drivers/gpu/drm/arm/hdlcd_crtc.c| 3 ++- drivers/gpu/drm/arm/malidp_planes.c | 3 ++- drivers/gpu/drm/armada/armada_overlay.c | 3 ++- drivers/gpu/drm/armada/armada_plane.c

[PATCH v2 09/11] drm/atomic: Pass the full state to planes atomic disable and update

2021-01-21 Thread Maxime Ripard
t;state + state ...+> } Signed-off-by: Maxime Ripard --- Changes from v1: - Reintroduce the old_plane_state check in zynqmp_disp_crtc_atomic_disable --- drivers/gpu/drm/arc/arcpgu_crtc.c | 2 +- drivers/gpu/drm/arm/display/komeda/komeda_plane.c | 2 +- drive

Re: [PATCH v3 02/21] mmc: sunxi: add support for A100 mmc controller

2021-01-21 Thread Maxime Ripard
Hi Andre, On Mon, Jan 18, 2021 at 03:52:28PM +, Andre Przywara wrote: > On Mon, 18 Jan 2021 14:28:54 +0100 > Maxime Ripard wrote: > > Hi Maxime, > > > On Mon, Jan 18, 2021 at 02:08:29AM +, Andre Przywara wrote: > > > From: Yangtao Li > > > >

Re: [PATCH] pinctrl: sunxi: fix use-after-free in sunxi_pmx_free()

2021-01-21 Thread Maxime Ripard
Hi, On Tue, Jan 19, 2021 at 02:29:08PM +0800, Liu Xiang wrote: > When CONFIG_REGULATOR is not set, sunxi_pmx_request() always return > success. Even a group of pins call sunxi_pmx_request(), the refcount > is only 1. This can cause a use-after-free warning in sunxi_pmx_free(). > To solve this

[PATCH v2 08/11] drm: Rename plane->state variables in atomic update and disable

2021-01-21 Thread Maxime Ripard
c(struct drm_plane *plane, struct drm_plane_state *old_pstate) { <... - state + new_pstate ...> } Signed-off-by: Maxime Ripard --- drivers/gpu/drm/arm/malidp_planes.c | 34 +++--- drivers/gpu/drm/armada/armada_overlay.c | 104 +-

[PATCH v2 05/11] drm: Use the state pointer directly in planes atomic_check

2021-01-21 Thread Maxime Ripard
ane_state = drm_atomic_get_new_plane_state(state, plane); <... - plane_state->state + state ...> } Reviewed-by: Laurent Pinchart Signed-off-by: Maxime Ripard --- Changes from v1: - Fixed the formatting in zynqmp_disp --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 2 +- drivers/gpu/drm/

[PATCH v2 07/11] drm: Store new plane state in a variable for atomic_update and disable

2021-01-21 Thread Maxime Ripard
s; @@ func(struct drm_plane *plane, struct drm_plane_state *old_s) { + struct drm_plane_state *new_s = plane->state; <+... - plane->state + new_s ...+> } Reviewed-by: Laurent Pinchart Signed-off-by: Maxime Ripard --- Changes from v1: - Wrapping cha

[PATCH v2 02/11] drm: Rename plane atomic_check state names

2021-01-21 Thread Maxime Ripard
} Reviewed-by: Laurent Pinchart Signed-off-by: Maxime Ripard --- Changes from v1: - Updated the variable name in the comment in omapdrm --- .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 13 +++--- .../gpu/drm/arm/display/komeda/komeda_plane.c | 11 ++--- drivers/gpu/drm/arm/hdlcd_crtc.c

[PATCH v2 04/11] drm/atomic: Pass the full state to planes atomic_check

2021-01-21 Thread Maxime Ripard
@ no_include depends on !include && adds_new_state @ @@ + #include #include Reviewed-by: Laurent Pinchart Signed-off-by: Maxime Ripard --- Changes from v1: - Rewording and removal of a coccinelle rule suggested by Laurent --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c |

[PATCH v2 01/11] drm/atomic: Pass the full state to planes async atomic check and update

2021-01-21 Thread Maxime Ripard
<+... - plane_state->state + state ...+> } Acked-by: Thomas Zimmermann Signed-off-by: Maxime Ripard --- Changes from v1: - Updated the comment according to Thomas suggestions --- .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 8 ++- drivers/gpu/drm/drm_atomic_helper.c

[PATCH v2 03/11] drm/atmel-hlcdc: Rename custom plane state variable

2021-01-21 Thread Maxime Ripard
Subsequent reworks will pass the global atomic state in the function prototype, and atomic_check and atomic_update already have such a variable already. Let's change them to ease the rework. Acked-by: Sam Ravnborg Signed-off-by: Maxime Ripard --- .../gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c

Re: [PATCH 10/10] drm: Use state helper instead of the plane state pointer

2021-01-20 Thread Maxime Ripard
Hi Laurent, On Fri, Jan 15, 2021 at 11:20:21PM +0200, Laurent Pinchart wrote: > Hi Maxime, > > Thank you for the patch. > > On Fri, Jan 15, 2021 at 01:57:02PM +0100, Maxime Ripard wrote: > > Many drivers reference the plane->state pointer in order to get the

Re: [PATCH 18/20] clk: sunxi: clk-mod0: Demote non-conformant kernel-doc header

2021-01-20 Thread Maxime Ripard
gt; Cc: Michael Turquette > Cc: Stephen Boyd > Cc: Maxime Ripard > Cc: Chen-Yu Tsai > Cc: Jernej Skrabec > Cc: linux-...@vger.kernel.org > Cc: linux-arm-ker...@lists.infradead.org > Signed-off-by: Lee Jones Applied all three sunxi patches Maxime signature.asc Description: PGP signature

Re: [PATCH v3] clk: sunxi-ng: h6: Fix clock divider range on some clocks

2021-01-20 Thread Maxime Ripard
On Mon, Jan 18, 2021 at 12:09:12AM +, Andre Przywara wrote: > While comparing clocks between the H6 and H616, some of the M factor > ranges were found to be wrong: the manual says they are only covering > two bits [1:0], but our code had "5" in the number-of-bits field. > > By writing 0xff

Re: [PATCH] drm/vc4: Unify PCM card's driver_name

2021-01-18 Thread Maxime Ripard
On Fri, Jan 15, 2021 at 08:12:09PM +0100, Nicolas Saenz Julienne wrote: > User-space ALSA matches a card's driver name against an internal list of > aliases in order to select the correct configuration for the system. > When the driver name isn't defined, the match is performed against the >

Re: [PATCH 03/10] drm/atmel-hlcdc: Rename custom plane state variable

2021-01-18 Thread Maxime Ripard
Hi Sam On Fri, Jan 15, 2021 at 09:43:24PM +0100, Sam Ravnborg wrote: > On Fri, Jan 15, 2021 at 01:56:55PM +0100, Maxime Ripard wrote: > > Subsequent reworks will pass the global atomic state in the function > > prototype, and atomic_check and atomic_update already have such a >

Re: [PATCH v3 08/21] clk: sunxi-ng: Add support for the Allwinner H616 CCU

2021-01-18 Thread Maxime Ripard
gned-off-by: Andre Przywara Acked-by: Maxime Ripard Maxime signature.asc Description: PGP signature

Re: [PATCH v3 14/21] dt-bindings: usb: Add H616 compatible string

2021-01-18 Thread Maxime Ripard
On Mon, Jan 18, 2021 at 02:08:41AM +, Andre Przywara wrote: > The H616 has four PHYs as the H3, along with their respective clock > gates and resets, so the property description is identical. > > However the PHYs itself need some special bits, so we need a new > compatible string for it. > >

Re: [PATCH v3 15/21] dt-bindings: usb: sunxi-musb: Add H616 compatible string

2021-01-18 Thread Maxime Ripard
On Mon, Jan 18, 2021 at 02:08:42AM +, Andre Przywara wrote: > The H616 MUSB peripheral is presumably compatible to the H3 one. > > Signed-off-by: Andre Przywara Acked-by: Maxime Ripard Maxime signature.asc Description: PGP signature

Re: [PATCH v3 16/21] phy: sun4i-usb: Add support for the H616 USB PHY

2021-01-18 Thread Maxime Ripard
NK1 register like the A100. > > Name all those properties in a new config struct and assign a new > compatible name to it. > > Signed-off-by: Andre Przywara Acked-by: Maxime Ripard Maxime signature.asc Description: PGP signature

Re: [PATCH v3 13/21] phy: sun4i-usb: Rework HCI PHY (aka. "pmu_unk1") handling

2021-01-18 Thread Maxime Ripard
ter. > Let's put proper names to those "unknown" variables and symbols. > > While we are at it, generalise the existing code by allowing a bitmap > of bits to clear, to cover newer SoCs: The A100 and H616 use a different > bit for the SIDDQ control. > > Sign

Re: [PATCH v3 04/21] pinctrl: sunxi: Add support for the Allwinner H616 pin controller

2021-01-18 Thread Maxime Ripard
On Mon, Jan 18, 2021 at 02:08:31AM +, Andre Przywara wrote: > Port A is used for an internal connection to some analogue circuitry > which looks like an AC200 IP (as in the H6), though this is not > mentioned in the manual. > > Signed-off-by: Andre Przywara Acked-by: Maxime

Re: [PATCH v3 07/21] clk: sunxi-ng: Add support for the Allwinner H616 R-CCU

2021-01-18 Thread Maxime Ripard
On Mon, Jan 18, 2021 at 02:08:34AM +, Andre Przywara wrote: > The clocks itself are identical to the H6 R-CCU, it's just that the H616 > has not all of them implemented (or connected). > > Signed-off-by: Andre Przywara Acked-by: Maxime Ripard Maxime signature.asc Desc

Re: [PATCH v3 06/21] dt-bindings: clk: sunxi-ccu: Add compatible string for Allwinner H616

2021-01-18 Thread Maxime Ripard
On Mon, Jan 18, 2021 at 02:08:33AM +, Andre Przywara wrote: > Signed-off-by: Andre Przywara > Acked-by: Rob Herring Acked-by: Maxime Ripard Maxime signature.asc Description: PGP signature

Re: [PATCH v3 03/21] dt-bindings: pinctrl: Add Allwinner H616 compatible strings

2021-01-18 Thread Maxime Ripard
On Mon, Jan 18, 2021 at 02:08:30AM +, Andre Przywara wrote: > A new SoC, a new compatible string. > Also we were too miserly with just allowing seven interrupt banks. > > Signed-off-by: Andre Przywara Acked-by: Maxime Ripard Maxime signature.asc Description: PGP signature

Re: [PATCH v3 02/21] mmc: sunxi: add support for A100 mmc controller

2021-01-18 Thread Maxime Ripard
Hi, On Mon, Jan 18, 2021 at 02:08:29AM +, Andre Przywara wrote: > From: Yangtao Li > > This patch adds support for A100 MMC controller, which use word address > for internal dma. > > Signed-off-by: Yangtao Li > Signed-off-by: Andre Przywara We should also disable the timings setup in

Re: [PATCH v5 04/16] media: sun6i-csi: Stop using the deprecated fwnode endpoint parser

2021-01-18 Thread Maxime Ripard
. > > Since this was the last user of the helper, it should now be safe to > remove. > > Signed-off-by: Paul Kocialkowski Acked-by: Maxime Ripard Maxime signature.asc Description: PGP signature

Re: [PATCH] ARM: dts: sun8i: h2-plus: bananapi-m2-zero: Increase BT UART speed

2021-01-18 Thread Maxime Ripard
On Sat, Jan 16, 2021 at 11:37:10AM +0100, Jernej Skrabec wrote: > Bluetooth module on BananaPi M2 Zero can also be used for streaming > audio. However, for that case higher UART speed is required. > > Add a max-speed property. > > Signed-off-by: Jernej Skrabec Applied, thanks Maxime

Re: [PATCH] ARM: dts: sunxi: bananapi-m2-plus: Increase BT UART speed

2021-01-18 Thread Maxime Ripard
On Sat, Jan 16, 2021 at 11:52:28AM +0100, Jernej Skrabec wrote: > Bluetooth module on BananaPi M2 Plus can also be used for streaming > audio. However, for that case higher UART speed is required. > > Add a max-speed property. > > Signed-off-by: Jernej Skrabec Applied, thanks Maxime

Re: [PATCH 02/10] drm: Rename plane atomic_check state names

2021-01-15 Thread Maxime Ripard
Hi, On Fri, Jan 15, 2021 at 02:46:36PM +0100, Thomas Zimmermann wrote: > Hi > > Am 15.01.21 um 13:56 schrieb Maxime Ripard: > > diff --git a/drivers/gpu/drm/imx/ipuv3-plane.c > > b/drivers/gpu/drm/imx/ipuv3-plane.c > > index 8a4235d9d9f1..2cb09e9d9306 100644 > >

[PATCH 09/10] drm/atomic: Pass the full state to planes atomic disable and update

2021-01-15 Thread Maxime Ripard
.. - plane_state->state + state ...+> } Signed-off-by: Maxime Ripard --- drivers/gpu/drm/arc/arcpgu_crtc.c | 2 +- .../gpu/drm/arm/display/komeda/komeda_plane.c | 2 +- drivers/gpu/drm/arm/hdlcd_crtc.c | 2 +- drivers/gpu/drm/arm/malidp_planes.c |

[PATCH 08/10] drm: Rename plane->state variables in atomic update and disable

2021-01-15 Thread Maxime Ripard
c(struct drm_plane *plane, struct drm_plane_state *old_pstate) { <... - state + new_pstate ...> } Signed-off-by: Maxime Ripard --- drivers/gpu/drm/arm/malidp_planes.c | 34 +++--- drivers/gpu/drm/armada/armada_overlay.c | 104 +-

[PATCH 07/10] drm: Store new plane state in a variable for atomic_update and disable

2021-01-15 Thread Maxime Ripard
s; @@ func(struct drm_plane *plane, struct drm_plane_state *old_s) { + struct drm_plane_state *new_s = plane->state; <+... - plane->state + new_s ...+> } Signed-off-by: Maxime Ripard --- drivers/gpu/drm/arc/arcpgu_crtc.c | 7 ++-- dr

[PATCH 10/10] drm: Use state helper instead of the plane state pointer

2021-01-15 Thread Maxime Ripard
e Signed-off-by: Maxime Ripard --- drivers/gpu/drm/arc/arcpgu_crtc.c | 4 +++- drivers/gpu/drm/arm/hdlcd_crtc.c| 3 ++- drivers/gpu/drm/arm/malidp_planes.c | 3 ++- drivers/gpu/drm/armada/armada_overlay.c | 3 ++- drivers/gpu/drm/armada/armada_plane.c

[PATCH 06/10] drm: Use state helper instead of plane state pointer in atomic_check

2021-01-15 Thread Maxime Ripard
ne->state + old_plane_state ... } @ include depends on adds_old_state || replaces_old_state @ @@ #include @ no_include depends on !include && (adds_old_state || replaces_old_state) @ @@ + #include #include Signed-off-by: Maxime Ripard --- drivers/gpu/drm/imx/ipuv3-plane.c

[PATCH 01/10] drm/atomic: Pass the full state to planes async atomic check and update

2021-01-15 Thread Maxime Ripard
<+... - plane_state->state + state ...+> } Signed-off-by: Maxime Ripard --- .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 8 ++- drivers/gpu/drm/drm_atomic_helper.c | 4 +- drivers/gpu/drm/mediatek/mtk_drm_plane.c | 26 + drivers/gpu/drm/msm/disp/md

[PATCH 02/10] drm: Rename plane atomic_check state names

2021-01-15 Thread Maxime Ripard
Signed-off-by: Maxime Ripard --- .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 13 +++--- .../gpu/drm/arm/display/komeda/komeda_plane.c | 11 ++--- drivers/gpu/drm/arm/hdlcd_crtc.c | 18 drivers/gpu/drm/arm/malidp_planes.c | 36 drivers/gp

[PATCH 05/10] drm: Use the state pointer directly in planes atomic_check

2021-01-15 Thread Maxime Ripard
ane_state = drm_atomic_get_new_plane_state(state, plane); <... - plane_state->state + state ...> } Signed-off-by: Maxime Ripard --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 2 +- drivers/gpu/drm/arm/display/komeda/komeda_plane.c | 2 +- drivers/gpu/drm/arm/hdlcd_crtc.c | 2 +

[PATCH 04/10] drm/atomic: Pass the full state to planes atomic_check

2021-01-15 Thread Maxime Ripard
!include && adds_new_state @ @@ + #include #include Signed-off-by: Maxime Ripard --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 4 +++- drivers/gpu/drm/arm/display/komeda/komeda_plane.c | 4 +++- drivers/gpu/drm/arm/hdlcd_crtc.c | 4 +++- drivers/gpu/drm/arm

[PATCH 03/10] drm/atmel-hlcdc: Rename custom plane state variable

2021-01-15 Thread Maxime Ripard
Subsequent reworks will pass the global atomic state in the function prototype, and atomic_check and atomic_update already have such a variable already. Let's change them to ease the rework. Signed-off-by: Maxime Ripard --- .../gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 118

Re: [PATCH v4 00/10] sunxi: Support IRQ wakeup from deep sleep

2021-01-14 Thread Maxime Ripard
ith appropriate firmware and configuration, this series allows waking > from (and it has been tested with) the RTC, NMI/PMIC (power button, A/C > plug, etc.), all GPIO ports (button, lid switch, modem, etc.), LRADC, > and UARTs. I have tested this patch set on the H3, A64, H5, and H6 So

Re: [PATCH v2 02/21] dt-bindings: pinctrl: Add Allwinner H616 compatible strings

2021-01-14 Thread Maxime Ripard
On Thu, Jan 14, 2021 at 12:45:12AM +, Andre Przywara wrote: > On Mon, 14 Dec 2020 10:37:28 +0100 > Maxime Ripard wrote: > > > On Fri, Dec 11, 2020 at 01:19:15AM +, Andre Przywara wrote: > > > A new SoC, a new compatible string. > > > Also we were too

Re: [PATCH v6] drm/sun4i: tcon: fix inverted DCLK polarity

2021-01-14 Thread Maxime Ripard
ls polarity. > > Fixes: 88bc4178568b ("drm: Use new > DRM_BUS_FLAG_*_(DRIVE|SAMPLE)_(POS|NEG)EDGE flags") > Suggested-by: Maxime Ripard > Signed-off-by: Giulio Benetti > --- > V2->V3: > - squash 2 patches into 1 > V3->V4: > - add SUN4I_TCON0_IO_POL_DCLK_POSITIVE t

Re: [PATCH] mmc: sunxi-mmc: Ensure host is suspended during system sleep

2021-01-13 Thread Maxime Ripard
> Signed-off-by: Samuel Holland Acked-by: Maxime Ripard Thanks! Maxime signature.asc Description: PGP signature

Re: [PATCH v3] drm/sun4i: tcon: fix inverted DCLK polarity

2021-01-13 Thread Maxime Ripard
ls polarity. > > Fixes: 88bc4178568b ("drm: Use new > DRM_BUS_FLAG_*_(DRIVE|SAMPLE)_(POS|NEG)EDGE flags") > Suggested-by: Maxime Ripard > Signed-off-by: Giulio Benetti > --- > drivers/gpu/drm/sun4i/sun4i_tcon.c | 20 +--- > drivers/gpu/drm/sun4i/sun4i_tcon.h

Re: [PATCH 0/3] drm/sun4i: de2/de3: CSC improvements

2021-01-13 Thread Maxime Ripard
On Sun, Jan 10, 2021 at 09:19:44PM +0100, Jernej Skrabec wrote: > This short series reworks CSC handling to remove duplicated constants > (patch 1 and 2) and adds BT2020 encoding to DE3 (patch 3). > > Please take a look. Applied, thanks Maxime signature.asc Description: PGP signature

Re: [PATCH v3] drm/sun4i: de2: Reimplement plane z position setting logic

2021-01-13 Thread Maxime Ripard
Hi, On Wed, Jan 06, 2021 at 09:46:30PM +0100, Jernej Skrabec wrote: > From: Roman Stratiienko > > To set blending channel order register software needs to know state and > position of each channel, which impossible at plane commit stage. > > Move this procedure to atomic_flush stage, where all

Re: [PATCH] arm64: dts: allwinner: h6: PineH64 model B: Add bluetooth

2021-01-13 Thread Maxime Ripard
On Sun, Jan 10, 2021 at 10:16:06PM +0100, Jernej Skrabec wrote: > PineH64 model B has wifi+bt combo module. Wifi is already supported, so > lets add also bluetooth node. > > Signed-off-by: Jernej Skrabec Applied, thanks Maxime signature.asc Description: PGP signature

Re: [PATCH v2 1/7] ASoC: dt-bindings: sun8i-codec: Increase #sound-dai-cells

2021-01-13 Thread Maxime Ripard
Hi, On Wed, Jan 13, 2021 at 12:06:09AM -0600, Samuel Holland wrote: > Increase sound-dai-cells to 1 to allow using the DAIs in the codec > corresponding to AIF2 and AIF3. > > The generic ASoC OF code supports a #sound-dai-cells value of 0 or 1 > with no impact to the driver, so this is a

Re: [PATCH v2 0/3] PinePhone volume key (LRADC) wakeup support

2021-01-13 Thread Maxime Ripard
.../input/allwinner,sun4i-a10-lradc-keys.yaml | 2 ++ > .../dts/allwinner/sun50i-a64-pinephone.dtsi | 1 + > drivers/input/keyboard/sun4i-lradc-keys.c | 22 +++ > 3 files changed, 21 insertions(+), 4 deletions(-) Acked-by: Maxime Ripard I've applied patch 3 Maxime signature.asc Description: PGP signature

Re: [PATCH 0/4] media: sunxi-cir: Cleanup and power management

2021-01-13 Thread Maxime Ripard
On Tue, Jan 12, 2021 at 10:51:28PM -0600, Samuel Holland wrote: > This series cleans up some dead code in the sunxi-cir driver and adds > system power management hooks. Acked-by: Maxime Ripard Thanks! Maxime signature.asc Description: PGP signature

Re: [PATCH v2] ARM: dts: sun5i: add A10s/A13 mali gpu support fallback

2021-01-13 Thread Maxime Ripard
On Fri, Jan 08, 2021 at 11:38:19AM +0100, Sergio Sota wrote: > The A10s/A13 mali gpu was not defined in device tree > Add A10 mali gpu as a fallback for A10s/A13 > Tested with Olimex-A13-SOM / Olimex-A13-OlinuXino-MICRO > "kmscube" 3d cube on screen (60fps / 10%cpu) > Versions: Lima:1.1.0 EGL:1.4

Re: [PATCH v2 2/2] drm/sun4i: tcon: improve DCLK polarity handling

2021-01-11 Thread Maxime Ripard
On Fri, Jan 08, 2021 at 03:34:52PM +0100, Giulio Benetti wrote: > Hi, > > On 1/8/21 10:23 AM, Maxime Ripard wrote: > > Hi, > > > > Thanks for those patches > > > > On Thu, Jan 07, 2021 at 03:30:32AM +0100, Giulio Benetti wrote: > > > From: G

[PATCH v2 06/15] drm/vc4: hdmi: Compute the CEC clock divider from the clock rate

2021-01-11 Thread Maxime Ripard
t HSM clock rate depending on pixel rate") Reviewed-by: Dave Stevenson Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/vc4/vc4_hdmi.c b/drivers/gpu/drm/vc4/vc4_hdmi.c index c3a301396aad..50

[PATCH v2 05/15] drm/vc4: hdmi: Restore cec physical address on reconnect

2021-01-11 Thread Maxime Ripard
assert. Fixes: 15b4511a4af6 ("drm/vc4: add HDMI CEC support") Signed-off-by: Dom Cobley Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 24 ++-- 1 file changed, 18 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/vc4/vc4_hdmi.c b/drive

[PATCH v2 11/15] drm/vc4: hdmi: Remove cec_available flag

2021-01-11 Thread Maxime Ripard
From: Dom Cobley Now that our HDMI controller supports CEC for the BCM2711, let's remove that flag. Reviewed-by: Dave Stevenson Signed-off-by: Dom Cobley Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 4 drivers/gpu/drm/vc4/vc4_hdmi.h | 3 --- 2 files changed, 7

[PATCH v2 10/15] drm/vc4: hdmi: Support BCM2711 CEC interrupt setup

2021-01-11 Thread Maxime Ripard
on an external irqchip. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 42 ++ drivers/gpu/drm/vc4/vc4_hdmi.h | 7 ++ 2 files changed, 39 insertions(+), 10 deletions(-) diff --git a/drivers/gpu/drm/vc4/vc4_hdmi.c b/drivers/gpu/drm/vc4/vc4_hdmi.c

[PATCH v2 12/15] drm/vc4: hdmi: Don't register the CEC adapter if there's no interrupts

2021-01-11 Thread Maxime Ripard
Stevenson Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/gpu/drm/vc4/vc4_hdmi.c b/drivers/gpu/drm/vc4/vc4_hdmi.c index 5a606b6f2917..eee9751009c2 100644 --- a/drivers/gpu/drm/vc4/vc4_hdmi.c +++ b/drivers/gpu/drm/vc4

[PATCH v2 15/15] ARM: dts: bcm2711: Add the CEC interrupt controller

2021-01-11 Thread Maxime Ripard
The CEC and hotplug interrupts go through an interrupt controller shared between the two HDMI controllers. Let's add that interrupt controller and the interrupts for both HDMI controllers Reviewed-by: Florian Fainelli Signed-off-by: Maxime Ripard --- arch/arm/boot/dts/bcm2711.dtsi | 18

[PATCH v2 13/15] dt-binding: display: bcm2711-hdmi: Add CEC and hotplug interrupts

2021-01-11 Thread Maxime Ripard
The CEC and hotplug interrupts were missing when that binding was introduced, let's add them in now that we've figured out how it works. Signed-off-by: Maxime Ripard --- .../bindings/display/brcm,bcm2711-hdmi.yaml | 20 ++- 1 file changed, 19 insertions(+), 1 deletion(-) diff

[PATCH v2 14/15] ARM: dts: bcm2711: Add the BSC interrupt controller

2021-01-11 Thread Maxime Ripard
The BSC controllers used for the HDMI DDC have an interrupt controller shared between both instances. Let's add it to avoid polling. Reviewed-by: Florian Fainelli Signed-off-by: Maxime Ripard --- arch/arm/boot/dts/bcm2711.dtsi | 12 1 file changed, 12 insertions(+) diff --git

[PATCH v2 08/15] drm/vc4: hdmi: Introduce a CEC clock

2021-01-11 Thread Maxime Ripard
While the BCM2835 had the CEC clock derived from the HSM clock, the BCM2711 has a dedicated parent clock for it. Let's introduce a separate clock for it so that we can handle both cases. Reviewed-by: Dave Stevenson Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 9

[PATCH v2 07/15] drm/vc4: hdmi: Update the CEC clock divider on HSM rate change

2021-01-11 Thread Maxime Ripard
. Fixes: cd4cb49dc5bb ("drm/vc4: hdmi: Adjust HSM clock rate depending on pixel rate") Reviewed-by: Dave Stevenson Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 39 +- 1 file changed, 29 insertions(+), 10 deletions(-) diff --git a/drive

[PATCH v2 09/15] drm/vc4: hdmi: Split the interrupt handlers

2021-01-11 Thread Maxime Ripard
the code as much as possible, yet still allowing to register independant handlers. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 86 +- 1 file changed, 65 insertions(+), 21 deletions(-) diff --git a/drivers/gpu/drm/vc4/vc4_hdmi.c b/drivers/gpu/drm/vc4

[PATCH v2 03/15] drm/vc4: hdmi: Fix register offset with longer CEC messages

2021-01-11 Thread Maxime Ripard
READ macro is now taking an enum, and the offset doesn't increment by 4 but 1 now. Divide the index by 4 to fix this. Fixes: 311e305fdb4e ("drm/vc4: hdmi: Implement a register layout abstraction") Reviewed-by: Dave Stevenson Signed-off-by: Dom Cobley Signed-off-by: Maxime Ripard ---

[PATCH v2 04/15] drm/vc4: hdmi: Fix up CEC registers

2021-01-11 Thread Maxime Ripard
;) Reviewed-by: Dave Stevenson Signed-off-by: Dom Cobley Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi_regs.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/vc4/vc4_hdmi_regs.h b/drivers/gpu/drm/vc4/vc4_hdmi_regs.h index 401863cb8c98.

[PATCH v2 01/15] ARM: bcm: Select BRCMSTB_L2_IRQ for bcm2835

2021-01-11 Thread Maxime Ripard
The BCM2711 has a number of instances of interrupt controllers handled by the driver behind the BRCMSTB_L2_IRQ Kconfig option (irq-brcmstb-l2). Let's select that driver as part of the ARCH_BCM2835 Kconfig option. Signed-off-by: Maxime Ripard --- arch/arm/mach-bcm/Kconfig| 1 + arch/arm64

[PATCH v2 00/15] drm/vc4: hdmi: Add CEC support for the BCM2711

2021-01-11 Thread Maxime Ripard
registers drm/vc4: hdmi: Restore cec physical address on reconnect drm/vc4: hdmi: Remove cec_available flag Maxime Ripard (10): ARM: bcm: Select BRCMSTB_L2_IRQ for bcm2835 drm/vc4: hdmi: Compute the CEC clock divider from the clock rate drm/vc4: hdmi: Update the CEC clock divider on HSM rate

[PATCH v2 02/15] drm/vc4: hdmi: Move hdmi reset to bind

2021-01-11 Thread Maxime Ripard
registers are in a separate block Fixes: 9045e91a476b ("drm/vc4: hdmi: Add reset callback") Reviewed-by: Dave Stevenson Signed-off-by: Dom Cobley Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a

Re: [PATCH 05/15] drm/vc4: hdmi: Restore cec physical address on reconnect

2021-01-11 Thread Maxime Ripard
Hi Dave, Thanks for your review On Fri, Dec 18, 2020 at 02:45:54PM +, Dave Stevenson wrote: > On Fri, 18 Dec 2020 at 14:21, Dave Stevenson > wrote: > > > > Hi Maxime & Dom > > > > On Thu, 10 Dec 2020 at 13:47, Maxime Ripard wrote: > > > >

Re: [PATCH v4 05/15] media: sun6i-csi: Only configure the interface data width for parallel

2021-01-08 Thread Maxime Ripard
by: Kévin L'hôpital > Signed-off-by: Paul Kocialkowski Acked-by: Maxime Ripard Thanks! Maxime signature.asc Description: PGP signature

Re: [PATCH v4 13/15] media: sunxi: Add support for the A83T MIPI CSI-2 controller

2021-01-08 Thread Maxime Ripard
onfiguration. > > Signed-off-by: Paul Kocialkowski There's a couple of checkpatch --strict warnings here as well Once fixed, Acked-by: Maxime Ripard Thanks! Maxime signature.asc Description: PGP signature

Re: [PATCH v4 09/15] media: sunxi: Add support for the A31 MIPI CSI-2 controller

2021-01-08 Thread Maxime Ripard
d 10-bit Bayer formats are currently supported. > While up to 4 internal channels to the CSI controller exist, only one > is currently supported by this implementation. > > Signed-off-by: Paul Kocialkowski CHECK: Alignment should match open parenthesis :) Once fixed, Acked-by: Maxime

Re: [PATCH v4 08/15] dt-bindings: media: Add A31 MIPI CSI-2 bindings documentation

2021-01-08 Thread Maxime Ripard
On Thu, Dec 31, 2020 at 03:29:41PM +0100, Paul Kocialkowski wrote: > This introduces YAML bindings documentation for the A31 MIPI CSI-2 > controller. > > Signed-off-by: Paul Kocialkowski Reviewed-by: Maxime Ripard Thanks! Maxime signature.asc Description: PGP signature

Re: [PATCH v4 07/15] media: sun6i-csi: Add support for MIPI CSI-2 bridge input

2021-01-08 Thread Maxime Ripard
not be linked in > the fwnode graph unless they have a sensor subdev attached. > > Signed-off-by: Paul Kocialkowski Acked-by: Maxime Ripard Maxime signature.asc Description: PGP signature

Re: [PATCH v4 06/15] dt-bindings: media: sun6i-a31-csi: Add MIPI CSI-2 input port

2021-01-08 Thread Maxime Ripard
ated as fwnode port 0. > > Note that additional ports may be added in the future, especially to > support feeding the CSI controller's output to the ISP. > > Signed-off-by: Paul Kocialkowski > Reviewed-by: Rob Herring Reviewed-by: Maxime Ripard Maxime

Re: [PATCH v3 01/10] dt-bindings: irq: sun6i-r: Split the binding from sun7i-nmi

2021-01-08 Thread Maxime Ripard
Hi Samuel, Thanks a lot for working on this I'm fine with the rest of the work, but I have a couple of questions On Sun, Jan 03, 2021 at 04:30:52AM -0600, Samuel Holland wrote: > The R_INTC in the A31 and newer sun8i/sun50i SoCs has additional > functionality compared to the sun7i/sun9i NMI

Re: [PATCH v2 2/2] drm/sun4i: tcon: improve DCLK polarity handling

2021-01-08 Thread Maxime Ripard
efore. So let's handle DCLK polarity by adding > SUN4I_TCON0_IO_POL_DCLK_POSITIVE as bit 26 and activating according to > bus_flags the same way is done for all the other signals. > > Cc: Maxime Ripard Suggested-by would be nice here :) > Signed-off-by: Giulio Benetti > --- >

<    1   2   3   4   5   6   7   8   9   10   >