RE: [PATCH v1 1/1] libnvdimm: Don't use GUID APIs against raw buffer

2021-04-20 Thread Moore, Robert
-Original Message- From: Kaneda, Erik Sent: Monday, April 19, 2021 3:56 PM To: Williams, Dan J ; Andy Shevchenko ; Moore, Robert ; Wysocki, Rafael J Cc: linux-nvdimm ; Linux Kernel Mailing List ; Verma, Vishal L ; Jiang, Dave ; Weiny, Ira Subject: RE: [PATCH v1 1/1] libnvdimm

[PATCH v2] irqchip/xilinx: Expose Kconfig option for Zynq/ZynqMP

2021-04-19 Thread Robert Hancock
Previously the XILINX_INTC config option was hidden and only auto-selected on the MicroBlaze platform. However, this IP can also be used on the Zynq and ZynqMP platforms as a secondary cascaded controller. Allow this option to be user-enabled on those platforms. Signed-off-by: Robert Hancock

Re: [PATCH] irqchip/xilinx: Expose Kconfig option

2021-04-19 Thread Robert Hancock
On Mon, 2021-04-19 at 13:23 +0200, Michal Simek wrote: > Hi Marc and Robert, +Anirudha > > On 4/16/21 8:14 PM, Robert Hancock wrote: > > On Fri, 2021-04-16 at 18:53 +0100, Marc Zyngier wrote: > > > On Fri, 16 Apr 2021 17:05:49 +0100, > > > Robert Hancock wrote:

Re: [PATCH v4 2/3] drm: bridge: add it66121 driver

2021-04-19 Thread Robert Foss
Hey Neil, Thanks for the quick respin. Reviewed-by: Robert Foss On Mon, 19 Apr 2021 at 09:12, Neil Armstrong wrote: > > From: Phong LE > > This commit is a simple driver for bridge HMDI it66121. > The input format is RBG and there is no color conversion. > Audio, HD

[PATCH v3] drm/bridge/sii8620: fix dependency on extcon

2021-04-19 Thread Robert Foss
er. Fixes: 688838442147 ("drm/bridge/sii8620: use micro-USB cable detection logic to detect MHL") Signed-off-by: Robert Foss Reported-by: kernel test robot --- LKP reported issue: https://lore.kernel.org/lkml/202104040604.sste2cxf-...@intel.com/ Changes since v1: - Fix typo on comment Change

Re: [PATCH] irqchip/xilinx: Expose Kconfig option

2021-04-16 Thread Robert Hancock
On Fri, 2021-04-16 at 18:53 +0100, Marc Zyngier wrote: > On Fri, 16 Apr 2021 17:05:49 +0100, > Robert Hancock wrote: > > On Fri, 2021-04-16 at 14:41 +0100, Marc Zyngier wrote: > > > On Fri, 16 Apr 2021 00:32:50 +0100, > > > Robert Hancock wrote: > > > >

Re: [PATCH] irqchip/xilinx: Expose Kconfig option

2021-04-16 Thread Robert Hancock
On Fri, 2021-04-16 at 14:41 +0100, Marc Zyngier wrote: > On Fri, 16 Apr 2021 00:32:50 +0100, > Robert Hancock wrote: > > Previously the XILINX_INTC config option was hidden and only > > auto-selected on the MicroBlaze platform. However, this IP can also be > > used on oth

Re: [PATCH -next] drm: bridge: fix ANX7625 use of mipi_dsi_() functions

2021-04-16 Thread Robert Foss
Merged for 5.13 in drm-misc-next-fixes On Fri, 16 Apr 2021 at 10:41, Robert Foss wrote: > > Thanks Randy! > > Reviewed-by: Robert Foss > > On Thu, 15 Apr 2021 at 20:36, Randy Dunlap wrote: > > > > The Analogix DRM ANX7625 bridge driver uses mips_dsi_() functi

Re: [PATCH -next] drm: bridge: fix LONTIUM use of mipi_dsi_() functions

2021-04-16 Thread Robert Foss
fe ("drm/bridge: Introduce LT9611 DSI to HDMI bridge") > > Fixes: 0cbbd5b1a012 ("drm: bridge: add support for lontium LT9611UXC > > bridge") > > Fixes: 30e2ae943c26 ("drm/bridge: Introduce LT8912B DSI to HDMI bridge") > > Signed-off-by: R

Re: [PATCH -next] drm: bridge: fix ANX7625 use of mipi_dsi_() functions

2021-04-16 Thread Robert Foss
Thanks Randy! Reviewed-by: Robert Foss On Thu, 15 Apr 2021 at 20:36, Randy Dunlap wrote: > > The Analogix DRM ANX7625 bridge driver uses mips_dsi_() function > interfaces so it should select DRM_MIPI_DSI to prevent build errors. > > > ERROR: modpost: "mipi_dsi_atta

[PATCH] irqchip/xilinx: Expose Kconfig option

2021-04-15 Thread Robert Hancock
Previously the XILINX_INTC config option was hidden and only auto-selected on the MicroBlaze platform. However, this IP can also be used on other platforms. Allow this option to be user-enabled. Signed-off-by: Robert Hancock --- drivers/irqchip/Kconfig | 6 +- 1 file changed, 5 insertions

Re: [PATCH v3 2/3] drm: bridge: add it66121 driver

2021-04-14 Thread Robert Foss
On Wed, 14 Apr 2021 at 08:13, Neil Armstrong wrote: > > Hi Rob, > > Le 13/04/2021 à 22:21, Robert Foss a écrit : > > Hey Neil & Phong, > > > > Thanks for submitting this series! > > > >> + > >> +static const struct drm_

Re: [PATCH v3 2/3] drm: bridge: add it66121 driver

2021-04-13 Thread Robert Foss
Hey Neil & Phong, Thanks for submitting this series! > + > +static const struct drm_bridge_funcs it66121_bridge_funcs = { > + .attach = it66121_bridge_attach, > + .enable = it66121_bridge_enable, > + .disable = it66121_bridge_disable, > + .mode_set = it66121_bridge_mode_se

Re: [PATCH v3 3/3] MAINTAINERS: add it66121 HDMI bridge driver entry

2021-04-13 Thread Robert Foss
' #27: FILE: MAINTAINERS:9537: +F:drivers/gpu/drm/bridge/ite-it66121.c +T:git git://anongit.freedesktop.org/drm/drm-misc With this warning silenced, feel free to add my r-b. Reviewed-by: Robert Foss > > +F: Documentation/devicetree/bindings/display/bridge/ite,it66121.yaml > > + > > IVTV VIDEO4LINUX DRIVER > > M: Andy Walls > > L: linux-me...@vger.kernel.org > > >

Re: [PATCH v2] drm/bridge/sii8620: fix dependency on extcon

2021-04-13 Thread Robert Foss
Hey Randy, Thanks for looking at this! On Fri, 9 Apr 2021 at 18:38, Randy Dunlap wrote: > > On 4/8/21 6:07 AM, Robert Foss wrote: > > The DRM_SIL_SII8620 kconfig has a weak `imply` dependency > > on EXTCON, which causes issues when sii8620 is built > > as a builtin

Re: [PATCH v6 0/2] enable HDCP in Cadence MHDP bridge driver

2021-04-13 Thread Robert Foss
Merged series. https://cgit.freedesktop.org/drm/drm-misc/commit/?id=6a3608eae6d33a478a29348eb5e9ca330a528ae6 On Sat, 10 Apr 2021 at 20:14, Parshuram Thombare wrote: > > This patch series enables HDCP in Cadence MHDP DPI/DP bridge driver. > > Changes since v1: > - Move sapb reg block right after

Re: [PATCH 04/12] ARM: pxa: stargate2: Constify the software node

2021-04-10 Thread Robert Jarzmik
ed-off-by: Heikki Krogerus >> Cc: Jonathan Cameron >> Cc: Daniel Mack >> Cc: Haojian Zhuang >> Cc: Robert Jarzmik > > I like to apply it soon. Can we get an ack, please? Acked-by: Robert Jarzmik Cheers. -- Robert

Re: [PATCH] ARM: pxa: mainstone: avoid -Woverride-init warning

2021-04-10 Thread Robert Jarzmik
ote: in expansion of macro >> 'MAINSTONE_S0_CD_IRQ' >> 506 | [5] = MAINSTONE_S0_CD_IRQ, >> | ^~~ >> >> [...] > > Applied to arm/fixes. Too late, but ... Acked-by: Robert Jarzmik Cheers. -- Robert

Re: [PATCH -next] media: camss: ispif: Remove redundant dev_err call in msm_ispif_subdev_init()

2021-04-09 Thread Robert Foss
r = platform_get_resource_byname(pdev, IORESOURCE_MEM, res->reg[1]); > ispif->base_clk_mux = devm_ioremap_resource(dev, r); > - if (IS_ERR(ispif->base_clk_mux)) { > - dev_err(dev, "could not map memory\n"); > + if (IS_ERR(ispif->base_clk_mux)) > return PTR_ERR(ispif->base_clk_mux); > - } > > /* Interrupt */ > Reviewed-by: Robert Foss

Re: [PATCH 1/1] drm/bridge: anx7625: send DPCD command to downstream

2021-04-09 Thread Robert Foss
te_and(ctx, ctx->i2c.tx_p2_client, 0x08, 0x7f); > > ret |= anx7625_video_mute_control(ctx, 1); > + > + DRM_DEV_DEBUG_DRIVER(dev, "notify downstream enter into standby\n"); > + > + /* Downstream monitor enter into standby mode */ > + data = 2; > + ret |= anx7625_aux_dpcd_write(ctx, 0x00, 0x06, 0x00, 1, &data); > if (ret < 0) > DRM_DEV_ERROR(dev, "IO error : mute video fail\n"); > + > + return; > } Reviewed-by: Robert Foss

[PATCH v2] drm/bridge/sii8620: fix dependency on extcon

2021-04-08 Thread Robert Foss
ned-off-by: Robert Foss Reported-by: kernel test robot --- Changes since v1: - Fix typo on comment drivers/gpu/drm/bridge/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/bridge/Kconfig b/drivers/gpu/drm/bridge/Kconfig index 22a467abd3e9..2289b44613af 10

Re: [PATCH -next v3] drm/bridge: lt8912b: Add header file

2021-04-08 Thread Robert Foss
1: error: ‘GPIOD_OUT_HIGH’ > undeclared (first use in this function); did you mean ‘GPIOF_INIT_HIGH’? > gp_reset = devm_gpiod_get_optional(dev, "reset", GPIOD_OUT_HIGH); >^~ >

Re: [PATCH -next v2] drm/bridge: lt8912b: Add header file

2021-04-08 Thread Robert Foss
bridge/lontium-lt8912b.c > index 61491615bad0..79845b3b19e1 100644 > --- a/drivers/gpu/drm/bridge/lontium-lt8912b.c > +++ b/drivers/gpu/drm/bridge/lontium-lt8912b.c > @@ -3,6 +3,7 @@ > * Copyright (c) 2018, The Linux Foundation. All rights reserved. > */ > > +#inc

Re: [PATCH -next] drm/bridge: lt8912b: DRM_LONTIUM_LT8912B select GPIOLIB

2021-04-08 Thread Robert Foss
Hey Zhang, On Thu, 8 Apr 2021 at 09:10, zhangjianhua (E) wrote: > > Hello Robert > > I am sorry that I make a mistake about the compiling error of lt8912b, > > the reason is that lt8912b miss the header file . > > Although there are many files reference gpiod_

Re: [PATCH -next] drm/bridge: lt8912b: DRM_LONTIUM_LT8912B select GPIOLIB

2021-04-07 Thread Robert Foss
> Yes, you are right, there are many files reference > gpiod_set_value_cansleep() and > > devm_gpiod_get_optional(). How about add config dependencies for all > releated I think this is the way to go and roughly half of the drm bridge drivers seem to need this change. Do you mind submitting a ser

Re: [PATCH -next] drm/bridge: lt8912b: DRM_LONTIUM_LT8912B select GPIOLIB

2021-04-06 Thread Robert Foss
Hey Zhang On Tue, 6 Apr 2021 at 11:07, Zhang Jianhua wrote: > > If CONFIG_DRM_LONTIUM_LT8912B=y, the following errors will be seen while > compiling lontium-lt8912b.c > > drivers/gpu/drm/bridge/lontium-lt8912b.c: In function > ‘lt8912_hard_power_on’: > drivers/gpu/drm/bridge/lontium-lt8912b.c:252

Re: [PATCH] drm: bridge: rename the function drm_bridge_hpd_notify to drm_bridge_hpd_cb

2021-04-06 Thread Robert Foss
Reviewed-by: Robert Foss On Tue, 30 Mar 2021 at 14:28, Robert Foss wrote: > > Hey Dafna, > > Thanks for submitting a cleanup patch, it is much appreciated. It > looks good to me, feel free to add my r-b. > > I'm not going to merge this right away, but will let this p

[PATCH v1] drm/bridge/sii8620: fix dependency on extcon

2021-04-06 Thread Robert Foss
ned-off-by: Robert Foss Reported-by: kernel test robot --- LKP reported issue: https://lore.kernel.org/lkml/202104040604.sste2cxf-...@intel.com/ drivers/gpu/drm/bridge/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/bridge/Kconfig b/drivers/gpu/drm/brid

Re: [PATCH -next] media: camss: csid: Remove redundant dev_err call in msm_csid_subdev_init()

2021-04-06 Thread Robert Foss
Hi Yang, Thanks for the fix. Reviewed-by: Robert Foss On Fri, 2 Apr 2021 at 11:29, Yang Yingliang wrote: > > There is a error message within devm_ioremap_resource > already, so remove the dev_err call to avoid redundant > error message. > > Reported-by: Hulk Robot >

RE: [PATCH v2 3/3] hpsa: add an assert to prevent from __packed reintroduction

2021-04-02 Thread Elliott, Robert (Servers)
It looks like ia64 implements atomic_t as a 64-bit value and expects atomic_t to be 64-bit aligned, but does nothing to ensure that. For x86, atomic_t is a 32-bit value and atomic64_t is a 64-bit value, and the definition of atomic64_t is overridden in a way that ensures 64-bit (8 byte) alignment:

RE: [PATCH] ACPICA: Events: support fixed pcie wake event

2021-04-01 Thread Moore, Robert
Can you send/point us to the spec that this change is based upon? Thanks, Bob -Original Message- From: Jianmin Lv Sent: Thursday, April 01, 2021 5:06 AM To: Moore, Robert ; Kaneda, Erik ; Wysocki, Rafael J ; Len Brown Cc: linux-a...@vger.kernel.org; de...@acpica.org; linux-kernel

Re: [PATCH -next] media: camss: csiphy: Remove redundant dev_err call in msm_csiphy_subdev_init()

2021-04-01 Thread Robert Foss
Hey Yang, Thanks for the patch, feel free to add my r-b. Reviewed-by: Robert Foss On Thu, 1 Apr 2021 at 12:33, Yang Yingliang wrote: > > There is an error message within devm_ioremap_resource > already, so remove the dev_err call to avoid redundant > error message. > > Repor

Re: [PATCH v6 3/5] drm/bridge: anx7625: add MIPI DPI input feature support

2021-04-01 Thread Robert Foss
Hey Xin, This patch looks good to me, feel free to add my r-b. Reviewed-by: Robert Foss On Fri, 19 Mar 2021 at 07:34, Xin Ji wrote: > > Add MIPI rx DPI input support. > > Reported-by: kernel test robot > Signed-off-by: Xin Ji > --- > drivers/gpu/drm/bridge/ana

Re: [PATCH] drivers: gpu: drm: bridge: fix kconfig dependency on DRM_KMS_HELPER

2021-04-01 Thread Robert Foss
Hey Julian, Nice catch, this patch looks good to me. I've pushed it to drm-misc-next. https://cgit.freedesktop.org/drm/drm-misc/commit/?id=62066d3164467167fc27b2383f67d097e39bf176 On Mon, 22 Feb 2021 at 22:56, Julian Braha wrote: > > When DRM_TOSHIBA_TC358762 is enabled and DRM_KMS_HELPER is di

Re: [PATCH v6 5/5] drm/bridge: anx7625: add HDMI audio function

2021-04-01 Thread Robert Foss
Hey Xin, This patch looks good to me. Reviewed-by: Robert Foss On Fri, 19 Mar 2021 at 07:35, Xin Ji wrote: > > Add audio HDMI codec function support, enable it through device true flag > "analogix,audio-enable". > > Reported-by: kernel test robot > Signed-off-by:

Re: [PATCH v6 1/5] dt-bindings:drm/bridge:anx7625:add vendor define flags

2021-04-01 Thread Robert Foss
Hey Xin, This series no longer applies to drm-misc/drm-misc-next, please rebase it. On Wed, 24 Mar 2021 at 08:52, Xin Ji wrote: > > On Sun, Mar 21, 2021 at 02:00:38PM +0200, Laurent Pinchart wrote: > > Hi Xin, > > > > Thank you for the patch. > > > > On Fri, Mar 19, 2021 at 02:32:39PM +0800, Xin

[PATCH v1] arm64: dts: qcom: sm8350: Add support for PRNG EE

2021-04-01 Thread Robert Foss
RNG (Random Number Generator) in SM8350 features PRNG EE (Execution Environment), hence add devicetree support for it. Signed-off-by: Robert Foss --- arch/arm64/boot/dts/qcom/sm8350.dtsi | 7 +++ 1 file changed, 7 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/sm8350.dtsi b/arch

Re: [RESEND PATCH v4 1/2] dt-bindings: drm/bridge: anx7625: Add power supplies

2021-04-01 Thread Robert Foss
Hsin-Yi Wang > Reviewed-by: Rob Herring > Reviewed-by: Robert Foss > --- > v3->v4: rebase to drm-misc/for-linux-next > --- > .../bindings/display/bridge/analogix,anx7625.yaml | 15 +++ > 1 file changed, 15 insertions(+) > > diff --git > a/Documentation/device

Re: Candidate Linux ABI for Intel AMX and hypothetical new related features

2021-03-31 Thread Robert O'Callahan
For the record, the benefits of dynamic XCR0 for rr recording portability still apply. I guess it'd be useful for CRIU too. We would also benefit from anything that incentivizes increased support for CPUID faulting. Rob -- Su ot deraeppa sah dna Rehtaf eht htiw saw hcihw, efil lanrete eht uoy ot m

Re: [PATCH v2 1/1] i2c: drivers: Use generic definitions for bus frequencies (part 2)

2021-03-31 Thread Robert Foss
Hey Andy, This patch looks good to me. Reviewed-by: Robert Foss On Wed, 31 Mar 2021 at 12:46, Andy Shevchenko wrote: > > Since we have generic definitions for bus frequencies, let's use them. > > Cc: Wolfram Sang > Signed-off-by: Andy Shevchenko > Acked-by: Khalil Bla

Re: [PATCH v6 13/14] drm/bridge: imx: Add LDB support for i.MX8qm

2021-03-31 Thread Robert Foss
On Wed, 31 Mar 2021 at 08:41, Liu Ying wrote: > > Hi Robert, > > On Tue, 2021-03-30 at 12:05 +0200, Robert Foss wrote: > > Hey Liu, > > > > checkpatch --strict lists some nits for this patch with those and the > > below warning fixed, feel free to add my r-b.

Re: [PATCH] drm: bridge: rename the function drm_bridge_hpd_notify to drm_bridge_hpd_cb

2021-03-30 Thread Robert Foss
Hey Dafna, Thanks for submitting a cleanup patch, it is much appreciated. It looks good to me, feel free to add my r-b. I'm not going to merge this right away, but will let this patch soak for a while to let other people have a look at it. On Tue, 30 Mar 2021 at 13:52, Dafna Hirschfeld wrote: >

Re: [PATCH v2 20/20] drm/dp_mst: Convert drm_dp_mst_topology.c to drm_err()/drm_dbg*()

2021-03-30 Thread Robert Foss
Hey Lyude, This patch looks good, but I have one question below. With it addressed, feel free to add my r-b. Reviewed-by: Robert Foss > > -static bool drm_dp_sideband_parse_req(struct drm_dp_sideband_msg_rx *raw, > +static bool drm_dp_sideband_parse_req(const struct drm_dp_mst_top

Re: [PATCH v2 19/20] drm/dp_mst: Drop DRM_ERROR() on kzalloc() fail in drm_dp_mst_handle_up_req()

2021-03-30 Thread Robert Foss
Hey Lyude, This patch looks good to me. Reviewed-by: Robert Foss On Fri, 26 Mar 2021 at 21:40, Lyude Paul wrote: > > Checkpatch was complaining about this - there's no need for us to print > errors when kzalloc() fails, as kzalloc() will already WARN for us. So, > let&

Re: [PATCH v2 17/20] drm/dp: Convert drm_dp_helper.c to using drm_err/drm_dbg_*()

2021-03-30 Thread Robert Foss
Hey Lyude, Looks good to me. Reviewed-by: Robert Foss On Fri, 26 Mar 2021 at 21:40, Lyude Paul wrote: > > Now that we've added a back-pointer to drm_device to drm_dp_aux, made > drm_dp_aux available to any functions in drm_dp_helper.c which need to > print to the kernel log

Re: [PATCH v2 07/20] drm/print: Fixup DRM_DEBUG_KMS_RATELIMITED()

2021-03-30 Thread Robert Foss
Hey Lyude, I'm seeing no issues with this patch and the reasoning behind the patch is sound to me. Reviewed-by: Robert Foss On Fri, 26 Mar 2021 at 21:39, Lyude Paul wrote: > > Since we're about to move drm_dp_helper.c over to drm_dbg_*(), we'll want > to make s

Re: [PATCH v6 13/14] drm/bridge: imx: Add LDB support for i.MX8qm

2021-03-30 Thread Robert Foss
hich has already been sent with the following series to add Mixel combo PHY > found in i.MX8qxp: > https://www.spinics.net/lists/arm-kernel/msg879957.html > > v5->v6: > * No change. > > v4->v5: > * Link with the imx-ldb-helper object. (Robert) > * Correspondingly, re

Re: [PATCH v6 12/14] drm/bridge: imx: Add LDB support for i.MX8qxp

2021-03-30 Thread Robert Foss
Hey Liu, checkpatch --strict lists some nits and a warning. I think the kconfig warning can be ignored. With the rest fixed, feel free to add my r-b. Reviewed-by: Robert Foss On Wed, 17 Mar 2021 at 04:57, Liu Ying wrote: > > This patch adds a drm bridge driver for i.MX8qxp LVDS display

Re: [PATCH v6 12/14] drm/bridge: imx: Add LDB support for i.MX8qxp

2021-03-30 Thread Robert Foss
this patch depends on the patch 'phy: Add LVDS configuration > options', > which has already been sent with the following series to add Mixel combo PHY > found in i.MX8qxp: > https://www.spinics.net/lists/arm-kernel/msg879957.html > > v5->v6: > * No change. >

Re: [PATCH v6 14/14] MAINTAINERS: add maintainer for DRM bridge drivers for i.MX SoCs

2021-03-30 Thread Robert Foss
Hey Liu, Reviewed-by: Robert Foss On Wed, 17 Mar 2021 at 04:57, Liu Ying wrote: > > Add myself as the maintainer of DRM bridge drivers for i.MX SoCs. > > Signed-off-by: Liu Ying > --- > v5->v6: > * No change. > > v4->v5: > * No change. > > v3->

Re: [PATCH v6 10/14] drm/bridge: imx: Add LDB driver helper support

2021-03-30 Thread Robert Foss
Hey Liu, checkpatch --strict is listing some nits for this patch, with those fixed feel free to add my r-b. Reviewed-by: Robert Foss On Wed, 17 Mar 2021 at 04:57, Liu Ying wrote: > > This patch adds a helper to support LDB drm bridge drivers for > i.MX SoCs. Helper functions sup

Re: [PATCH v6 09/14] drm/bridge: imx: Add i.MX8qxp pixel link to DPI support

2021-03-30 Thread Robert Foss
Hey Liu, checkpatch --strict had some complaints, with those fixed feel free to add my r-b. Reviewed-by: Robert Foss On Wed, 17 Mar 2021 at 04:57, Liu Ying wrote: > > This patch adds a drm bridge driver for i.MX8qxp pixel link to display > pixel interface(PXL2DPI). The PXL2DPI inter

RE: [PATCH] ACPICA: Fix a typo

2021-03-26 Thread Moore, Robert
Please make a pull request for this on our github. Thanks, Bob -Original Message- From: Bhaskar Chowdhury Sent: Thursday, March 25, 2021 5:19 PM To: Moore, Robert ; Kaneda, Erik ; Wysocki, Rafael J ; l...@kernel.org; linux-a...@vger.kernel.org; de...@acpica.org; linux-kernel

Re: [PATCH v9 0/2] Add support of Lontium lt8912 MIPI to HDMI bridge

2021-03-26 Thread Robert Foss
Pushed: https://cgit.freedesktop.org/drm/drm-misc/commit/?id=3aa6031deefa9a2c056af2182af02d3dc5df1067 On Fri, 26 Mar 2021 at 13:20, Adrien Grassein wrote: > > Hi, > this patch set adds the support of the Lontium lt8912 MIPI to HDMI > bridge in the kernel. > > It's only support the video part, no

Re: [PATCH v5 1/2] dt-bindings: display: bridge: Add Chipone ICN6211 bindings

2021-03-25 Thread Robert Foss
al input and > produces RGB565, RGB666, RGB888 output format. > > Add dt-bingings for it. > > Signed-off-by: Jagan Teki > Reviewed-by: Robert Foss > Reviewed-by: Rob Herring > --- > Changes for v5: > - rebase drm-misc-next > - collect Rob, Robert review tags

Re: [PATCH] drm: bridge: convert sysfs sprintf/snprintf family to sysfs_emit

2021-03-25 Thread Robert Foss
Pushed to drm-misc-next https://cgit.freedesktop.org/drm/drm-misc/commit/?id=fffa69aa6b1c89853cd00dea969e4754633596d7 On Sun, 7 Feb 2021 at 10:12, Jiapeng Chong wrote: > > Fix the following coccicheck warning: > > drivers/gpu/drm/bridge/lontium-lt9611uxc.c:858:8-16: WARNING: use > scnprintf or s

[PATCH v2] MAINTAINERS: Update Maintainers of DRM Bridge Drivers

2021-03-25 Thread Robert Foss
-off-by: Robert Foss Acked-by: Neil Armstrong Acked-by: Maxime Ripard Acked-by: Laurent Pinchart Acked-by: Andrzej Hajda --- Add maintainer Acks. Submitted to follow proper dim workflow. MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index

Re: [PATCH] cxl/mem: Force array size of mem_commands[] to CXL_MEM_COMMAND_ID_MAX

2021-03-25 Thread Robert Richter
On 24.03.21 12:08:20, Dan Williams wrote: > On Wed, Mar 24, 2021 at 11:43 AM Ira Weiny wrote: > > Can't we use ARRAY_SIZE? > > An ARRAY_SIZE() check in cxl_validate_cmd_from_user() would work too, > but it wouldn't give the compiler protection that Robert mentio

[PATCH] cxl/mem: Force array size of mem_commands[] to CXL_MEM_COMMAND_ID_MAX

2021-03-24 Thread Robert Richter
This could lead then further to an out-of-bounds array access in cxl_validate_cmd_from_user(). Fix this by forcing the array size to CXL_MEM_COMMAND_ID_MAX. This also adds range checks for array items in mem_commands[] at compile time. Signed-off-by: Robert Richter --- drivers/cxl/mem.c | 2

[PATCH v3 2/2] arm64: dts: qcom: sm8350: Add thermal zones and throttling support

2021-03-24 Thread Robert Foss
sm8350 has 29 thermal sensors split across two tsens controllers. Add the thermal zones to expose them and wireup the cpus to throttle their frequencies on crossing passive temperature thresholds. Signed-off-by: Robert Foss Reviewed-by: Vinod Koul --- Changes since v1: - Vinod: Remove comment

[PATCH v3 1/2] dt-bindings: thermal: qcom-tsens: Add compatible for sm8350

2021-03-24 Thread Robert Foss
Add tsens bindings for sm8350. Signed-off-by: Robert Foss Reviewed-by: Vinod Koul --- Changes since v1: - Vinod: Remove comment Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/thermal/qcom

[PATCH v1] MAINTAINERS: Update Maintainers of DRM Bridge Drivers

2021-03-24 Thread Robert Foss
-off-by: Robert Foss --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 4b705ba51c54..16ace8f58649 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -5902,6 +5902,7 @@ F:drivers/gpu/drm/atmel-hlcdc/ DRM DRIVERS FOR BRIDGE CHIPS M: Andrzej

[PATCH v2 2/2] arm64: dts: qcom: sm8350: Add thermal zones and throttling support

2021-03-23 Thread Robert Foss
sm8350 has 29 thermal sensors split across two tsens controllers. Add the thermal zones to expose them and wireup the cpus to throttle their frequencies on crossing passive temperature thresholds. Signed-off-by: Robert Foss Reviewed-by: Vinod Koul --- Changes since v1: - Vinod: Remove comment

[PATCH v2 1/2] dt-bindings: thermal: qcom-tsens: Add compatible for sm8350

2021-03-23 Thread Robert Foss
Add tsens bindings for sm8350. Signed-off-by: Robert Foss Reviewed-by: Vinod Koul --- Changes since v2: - Vinod: Add r-b Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml b

Re: [PATCH v7 2/2] drm/bridge: Introduce LT8912B DSI to HDMI bridge

2021-03-23 Thread Robert Foss
Alright, I think the best way forward is to remove the polled HPD support. Don't forget to update flags and function pointers. With that, feel free to add my r-b. Reviewed-by: Robert Foss

Re: [PATCH v7 2/2] drm/bridge: Introduce LT8912B DSI to HDMI bridge

2021-03-23 Thread Robert Foss
> > Are you saying that pin 63 never is high, or that an irq/isr routine > > isn't getting executed? > > > > > When reading the value of the HPD pin, I always get 1 (and no > > > transition occurs when plugging / unplugging a cable). > > > The HPD IRQ is done on the HDMI connector driver [5]. > > >

Re: [PATCH v7 2/2] drm/bridge: Introduce LT8912B DSI to HDMI bridge

2021-03-23 Thread Robert Foss
> > > > > > > > > +static enum drm_connector_status lt8912_check_cable_status(struct > > > > > lt8912 *lt) > > > > > +{ > > > > > + int ret; > > > > > + unsigned int reg_val; > > > > > + > > > > > + ret = regmap_read(lt->regmap[I2C_MAIN], 0xC1, ®_val); > > > > > + if (ret)

Re: [PATCH v7 2/2] drm/bridge: Introduce LT8912B DSI to HDMI bridge

2021-03-23 Thread Robert Foss
On Tue, 23 Mar 2021 at 11:01, Adrien Grassein wrote: > > Hey Robert, > > Thanks for the update. > > Le mar. 23 mars 2021 à 10:10, Robert Foss a écrit : > > > > Hey Adrien, > > > > Sorry about the slow reply, but I just received the documentation from

Re: [PATCH v7 2/2] drm/bridge: Introduce LT8912B DSI to HDMI bridge

2021-03-23 Thread Robert Foss
Hey Adrien, Sorry about the slow reply, but I just received the documentation from the vendor. So let's dig in to the HPD issue. > +static enum drm_connector_status lt8912_check_cable_status(struct lt8912 *lt) > +{ > + int ret; > + unsigned int reg_val; > + > + ret = regmap_read

Re: [PATCH v2 2/2] arm64: dts: qcom: sm8350: Add thermal zones and throttling support

2021-03-23 Thread Robert Foss
Hey Konrad, Thanks for the review! On Mon, 22 Mar 2021 at 18:27, Konrad Dybcio wrote: > > Hi! > > > > + tsens0: thermal-sensor@c222000 { > > + compatible = "qcom,sm8350-tsens", "qcom,tsens-v2"; > > + reg = <0 0x0C263000 0 0x1ff>, /* TM */ > >

[PATCH v2 2/2] arm64: dts: qcom: sm8350: Add thermal zones and throttling support

2021-03-22 Thread Robert Foss
sm8350 has 29 thermal sensors split across two tsens controllers. Add the thermal zones to expose them and wireup the cpus to throttle their frequencies on crossing passive temperature thresholds. Signed-off-by: Robert Foss --- Changes since v1: - Vinod: Remove comment arch/arm64/boot/dts

[PATCH v2 1/2] dt-bindings: thermal: qcom-tsens: Add compatible for sm8350

2021-03-22 Thread Robert Foss
Add tsens bindings for sm8350. Signed-off-by: Robert Foss --- Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml

Re: [PATCH v6 2/5] drm/bridge: anx7625: fix not correct return value

2021-03-22 Thread Robert Foss
Hey Xin, This patch looks good to me. Reviewed-by: Robert Foss On Fri, 19 Mar 2021 at 07:33, Xin Ji wrote: > > At some time, the original code may return non zero value, force return 0 > if operation finished. > > Signed-off-by: Xin Ji > --- > drivers/gpu/drm/bridge/a

Re: [PATCH] drm: bridge: adv7511: Support I2S IEC958 encoded PCM format

2021-03-19 Thread Robert Foss
Hey Sia, Thanks for the patch, it looks good to me. Reviewed-by: Robert Foss On Fri, 12 Mar 2021 at 10:09, Sia Jee Heng wrote: > > Support IEC958 encoded PCM format for ADV7511 so that ADV7511 HDMI > audio driver can accept the IEC958 data from the I2S input. > > Signed-off-b

Re: [PATCH 13/30] drm/bridge/analogix/dp_core: Unregister DP AUX channel on error in analogix_dp_probe()

2021-03-19 Thread Robert Foss
Hey Lyude, Thanks for the patch, it looks good to me. Reviewed-by: Robert Foss On Fri, 19 Feb 2021 at 22:58, Lyude Paul wrote: > > Just another drive-by fix I noticed while going through the tree to cleanup > DP aux adapter registration - make sure we unregister the DP A

Re: [PATCH 12/30] drm/bridge/analogix/anx6345: Cleanup on errors in anx6345_bridge_attach()

2021-03-19 Thread Robert Foss
Hey Lyude, Thanks for the patch, it looks good to me. Reviewed-by: Robert Foss On Fri, 19 Feb 2021 at 22:58, Lyude Paul wrote: > > Another drive-by fix I found when fixing DP AUX adapter across the kernel > tree - make sure we don't leak resources (and by proxy-AUX adapters) on

Re: [PATCH 11/30] drm/bridge/analogix/anx6345: Don't link encoder until after connector registration

2021-03-19 Thread Robert Foss
Hey Lyude, Thanks for the patch, it looks good to me. Reviewed-by: Robert Foss On Fri, 19 Feb 2021 at 22:58, Lyude Paul wrote: > > Another case of linking an encoder to a connector after the connector's > been registered. The proper place to do this is before connector >

Re: [PATCH 10/30] drm/bridge/analogix/anx6345: Add missing drm_dp_aux_unregister() call

2021-03-19 Thread Robert Foss
Hey Lyude, Thanks for the patch, it looks good to me. Reviewed-by: Robert Foss On Fri, 19 Feb 2021 at 22:58, Lyude Paul wrote: > > Another driver I found that seems to forget to unregister it's DP AUX > device. Let's fix this by adding anx6345_bridge_detach(). > >

Re: [PATCH 09/30] drm/bridge/analogix/anx78xx: Cleanup on error in anx78xx_bridge_attach()

2021-03-19 Thread Robert Foss
Hey Lyude, Thanks for the patch, it looks good to me. Reviewed-by: Robert Foss On Fri, 19 Feb 2021 at 22:56, Lyude Paul wrote: > > Just another issue I noticed while correcting usages of > drm_dp_aux_init()/drm_dp_aux_register() around the tree. If any of the > steps in anx78xx_b

Re: [PATCH 08/30] drm/bridge/analogix/anx78xx: Setup encoder before registering connector

2021-03-19 Thread Robert Foss
Hey Lyude, Thanks for the patch, it looks good to me. Reviewed-by: Robert Foss On Fri, 19 Feb 2021 at 22:56, Lyude Paul wrote: > > Since encoder mappings for connectors are exposed to userspace, we should > be attaching the encoder before exposing the connector to userspace. Just a &

Re: [PATCH 07/30] drm/bridge/analogix/anx78xx: Add missing drm_dp_aux_unregister() call

2021-03-19 Thread Robert Foss
Hey Lyude, Thanks for the patch, it looks good to me. Reviewed-by: Robert Foss On Fri, 19 Feb 2021 at 22:56, Lyude Paul wrote: > > Surprisingly, this bridge actually registers it's AUX adapter at the > correct time already. Nice job! However, it does forget to actually > u

Re: [PATCH 04/30] drm/bridge/tc358767: Don't register DP AUX channel until bridge is attached

2021-03-19 Thread Robert Foss
Hey Lyude, This patch looks good to me. Reviewed-by: Robert Foss On Fri, 19 Feb 2021 at 22:56, Lyude Paul wrote: > > Since this is a bridge, we don't start out with a respective DRM device. > Likewise this means we don't have a connector, which also means that we &g

Re: [PATCH 30/30] drm/dp_mst: Convert drm_dp_mst_topology.c to drm_err()/drm_dbg*()

2021-03-19 Thread Robert Foss
Hey Lyude, Thanks for the patch On Fri, 19 Feb 2021 at 23:03, Lyude Paul wrote: > > And finally, convert all of the code in drm_dp_mst_topology.c over to using > drm_err() and drm_dbg*(). Note that this refactor would have been a lot > more complicated to have tried writing a coccinelle script f

Re: [PATCH 28/30] drm/dp: Convert drm_dp_helper.c to using drm_err/drm_dbg_*()

2021-03-19 Thread Robert Foss
Hey Lyude, Thanks for the patch. On Fri, 19 Feb 2021 at 23:03, Lyude Paul wrote: > > Now that we've added a back-pointer to drm_device to drm_dp_aux, made > drm_dp_aux available to any functions in drm_dp_helper.c which need to > print to the kernel log, and ensured all of our logging uses a con

Re: [PATCH 27/30] drm/dp_mst: Pass drm_dp_mst_topology_mgr to drm_dp_get_vc_payload_bw()

2021-03-19 Thread Robert Foss
Hey Lyude, Thanks for the patch. On Fri, 19 Feb 2021 at 23:03, Lyude Paul wrote: > > Since this is one of the few functions in drm_dp_mst_topology.c that > doesn't have any way of getting access to a drm_device, let's pass the > drm_dp_mst_topology_mgr down to this function so that it can use >

Re: [PATCH 18/30] drm/print: Fixup DRM_DEBUG_KMS_RATELIMITED()

2021-03-19 Thread Robert Foss
Hey Lyude, Thanks for the patch, On Fri, 19 Feb 2021 at 22:59, Lyude Paul wrote: > > Since we're about to move drm_dp_helper.c over to drm_dbg_*(), we'll want > to make sure that we can also add ratelimited versions of these macros in > order to retain some of the previous debugging output behav

Re: [PATCH v1 2/2] arm64: dts: qcom: sm8350: Add thermal zones and throttling support

2021-03-19 Thread Robert Foss
On Fri, 19 Mar 2021 at 11:49, Robert Foss wrote: > > sm8350 has 29 thermal sensors split across two tsens controllers. Add > the thermal zones to expose them and wireup the cpus to throttle their > frequencies on crossing passive temperature thresholds. > > Signed-o

[PATCH v1 2/2] arm64: dts: qcom: sm8350: Add thermal zones and throttling support

2021-03-19 Thread Robert Foss
sm8350 has 29 thermal sensors split across two tsens controllers. Add the thermal zones to expose them and wireup the cpus to throttle their frequencies on crossing passive temperature thresholds. Signed-off-by: Robert Foss --- arch/arm64/boot/dts/qcom/sm8350.dtsi | 829

[PATCH v1 1/2] dt-bindings: thermal: qcom-tsens: Add compatible for sm8350

2021-03-19 Thread Robert Foss
Add tsens bindings for sm8350. Signed-off-by: Robert Foss --- Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml

[PATCH v9 13/22] media: camss: Enable SDM845

2021-03-16 Thread Robert Foss
Enable support for SDM845 based Titan 170 ISPs. Signed-off-by: Robert Foss Reviewed-by: Andrey Konovalov --- Changes since v5: - Andrey: Add r-b drivers/media/platform/qcom/camss/camss.c | 17 + drivers/media/platform/qcom/camss/camss.h | 6 -- 2 files changed, 21

[PATCH v9 07/22] media: camss: Add support for VFE hardware version Titan 170

2021-03-16 Thread Robert Foss
CSI decoder receives from the sensor. For example will a sensor outputting YUV pixel format frames, only allow the VFE to dump those frames as they are received by the ISP to memory through the RDI interface. Signed-off-by: Robert Foss Reviewed-by: Andrey Konovalov --- Changes since v1

[PATCH v9 21/22] arm64: dts: sdm845-db845c: Configure regulators for camss node

2021-03-16 Thread Robert Foss
Add regulator to camss device tree node. Signed-off-by: Robert Foss Reviewed-by: Andrey Konovalov --- Changes since v5: - Andrey: Add r-b arch/arm64/boot/dts/qcom/sdm845-db845c.dts | 4 1 file changed, 4 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/sdm845-db845c.dts b/arch

[PATCH v9 14/22] dt-bindings: media: camss: Add qcom,msm8916-camss binding

2021-03-16 Thread Robert Foss
Add bindings for qcom,msm8916-camss in order to support the camera subsystem on MSM8916. Signed-off-by: Robert Foss Reviewed-by: Rob Herring --- Changes since v2: - Remove redundant descriptions - Add power domain description - Make clock-lanes a constant - Add max & minItems to

[PATCH v9 22/22] arm64: dts: sdm845-db845c: Enable ov8856 sensor and connect to ISP

2021-03-16 Thread Robert Foss
Enable camss & ov8856 DT nodes. Signed-off-by: Robert Foss Reviewed-by: Andrey Konovalov --- Changes since v5: - Andrey: Add r-b - Change CSI clock & data pins arch/arm64/boot/dts/qcom/sdm845-db845c.dts | 19 +-- 1 file changed, 17 insertions(+), 2 deletions(

[PATCH v9 20/22] arm64: dts: sdm845: Add CAMSS ISP node

2021-03-16 Thread Robert Foss
Add the camss dt node for sdm845. Signed-off-by: Robert Foss Reviewed-by: Andrey Konovalov --- Changes since v1: - Laurent: Fix subject - Laurent: Remove redundant regulator labels - Laurent: Remove empty line Changes since v3: - Fixed ordering of IRQs - Add newlines for better

[PATCH v9 19/22] media: dt-bindings: media: Remove qcom,camss documentation

2021-03-16 Thread Robert Foss
This documentation has been incorporated in dtschema dt-bindings for the devices supported by CAMSS and is no longer helpful. Signed-off-by: Robert Foss Reviewed-by: Rob Herring --- Changes since v4: - Rob: Added r-b .../devicetree/bindings/media/qcom,camss.txt | 236

[PATCH v9 18/22] MAINTAINERS: Change CAMSS documentation to use dtschema bindings

2021-03-16 Thread Robert Foss
Due to the complexity of describing multiple hardware generations in one document, switch to using separate dt-bindings. Signed-off-by: Robert Foss Reviewed-by: Rob Herring --- Changes since v4 - Rob: Added r-b MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH v9 17/22] dt-bindings: media: camss: Add qcom,sdm845-camss binding

2021-03-16 Thread Robert Foss
Add bindings for qcom,sdm845-camss in order to support the camera subsystem for SDM845. Signed-off-by: Robert Foss Reviewed-by: Rob Herring --- Changes since v2: - Rob: Removed extra empty lines at end of file - Rob: Add new line at end of file - Rob: Remove redundant descriptions - Rob

[PATCH v9 16/22] dt-bindings: media: camss: Add qcom,sdm660-camss binding

2021-03-16 Thread Robert Foss
Add bindings for qcom,sdm660-camss in order to support the camera subsystem on SDM630/660 and SDA variants. Signed-off-by: Robert Foss Reviewed-by: Rob Herring --- Changes since v2: - Rob: Add new line at end of file - Rob: Remove redundant descriptions - Rob: Add power domain description

[PATCH v9 15/22] dt-bindings: media: camss: Add qcom,msm8996-camss binding

2021-03-16 Thread Robert Foss
Add bindings for qcom,msm8996-camss in order to support the camera subsystem on MSM8996. Signed-off-by: Robert Foss Reviewed-by: Rob Herring --- Changes since v2 - Rob: Add new line at end of file - Rob: Remove redundant descriptions - Rob: Add power domain description - Rob: Make clock

  1   2   3   4   5   6   7   8   9   10   >