Re: [linux-sunxi] [PATCH v2 6/8] ARM: dts: sun8i: a83t: Move mmc1 pinctrl setting to dtsi file

2017-10-18 Thread Joonas Kylmälä
Hi, Chen-Yu Tsai: > mmc1 only has 1 possible pinmux setting. What if someone is using the MMC with bus width 1 and then using the remaining 3 pins for something else? Joonas

Re: [linux-sunxi] [PATCH v2 6/8] ARM: dts: sun8i: a83t: Move mmc1 pinctrl setting to dtsi file

2017-10-19 Thread Joonas Kylmälä
Chen-Yu Tsai: >> Chen-Yu Tsai: >>> mmc1 only has 1 possible pinmux setting. >> >> What if someone is using the MMC with bus width 1 and then using the >> remaining 3 pins for something else? > > I would very much like to see such a design. Currently the devices > we see all follow Allwinner's refe

Re: [linux-sunxi] [PATCH 5/9] media: platform: Add Sunxi Cedrus decoder driver

2018-03-12 Thread Joonas Kylmälä
Paul Kocialkowski: > diff --git a/drivers/media/platform/sunxi-cedrus/sunxi_cedrus_regs.h > b/drivers/media/platform/sunxi-cedrus/sunxi_cedrus_regs.h > new file mode 100644 > index ..7384daa94737 > --- /dev/null > +++ b/drivers/media/platform/sunxi-cedrus/sunxi_cedrus_regs.h > @@ -0,0

Re: [linux-sunxi] [PATCH 2/9] media: videobuf2-v4l2: Copy planes when needed in request qbuf

2018-03-12 Thread Joonas Kylmälä
Paul Kocialkowski: > diff --git a/drivers/media/common/videobuf2/videobuf2-v4l2.c > b/drivers/media/common/videobuf2/videobuf2-v4l2.c > index 0627c3339572..c14528d4a518 100644 > --- a/drivers/media/common/videobuf2/videobuf2-v4l2.c > +++ b/drivers/media/common/videobuf2/videobuf2-v4l2.c > @@ -592,

Re: [linux-sunxi] [PATCH 5/9] media: platform: Add Sunxi Cedrus decoder driver

2018-03-12 Thread Joonas Kylmälä
Paul Kocialkowski: > diff --git a/drivers/media/platform/sunxi-cedrus/sunxi_cedrus.c > b/drivers/media/platform/sunxi-cedrus/sunxi_cedrus.c > new file mode 100644 > index ..88624035e0e3 > --- /dev/null > +++ b/drivers/media/platform/sunxi-cedrus/sunxi_cedrus.c > @@ -0,0 +1,313 @@ > +/*

Re: [linux-sunxi] [PATCH v3 15/16] ARM: dts: sun8i: h3: Enable HDMI output on H3 boards

2018-03-05 Thread Joonas Kylmälä
Jernej Skrabec: > +&hdmi_out { > + hdmi_out_con: endpoint { > + remote-endpoint = <&hdmi_con_in>; > + }; > +}; This node is added to all the DTS files you enabled HDMI on. Is it something that could be instead put to the DTSI file? Joonas

Re: [linux-sunxi] [PATCH v3 15/16] ARM: dts: sun8i: h3: Enable HDMI output on H3 boards

2018-03-05 Thread Joonas Kylmälä
Jernej Škrabec: > I guess that would mean also including connector node (hdmi_con_in) in DTSI, > since it is referenced inside. However, not all boards have HDMI connector, > so > I didn't include it in DTSI. You're absolutely right on this. I wish there was someway to get rid of this duplicati

Preferring cursor plane over overlay plane

2018-03-26 Thread Joonas Kylmälä
Hi DRM subsystem developers, I ran into this patch where overlay plane was switched to cursor plane because there was no proper cursor plane available on the display hardware: . Can we discuss whether to have a policy of using a normal plane for cursor plane in

Re: [PATCH] kconfig: fix missing choice values in auto.conf

2019-07-12 Thread Joonas Kylmälä
Hi, thanks a lot for this patch! I tested this and it fixes what is says in the commit message: Tested-by: Joonas Kylmälä Masahiro Yamada: > Since commit 00c864f8903d ("kconfig: allow all config targets to write > auto.conf if missing"), Kconfig creates include/config/