return -ENODEV;
> + }
> +
> + pdata = of_id->data;
> +
> dsi = devm_kzalloc(dev, sizeof(*dsi), GFP_KERNEL);
> if (!dsi)
> return -ENOMEM;
Hello
You could use of_device_get_match_data()
Regards
Corentin Labbe
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
On 23/08/2016 17:24, Sean Paul wrote:
> On Tue, Aug 16, 2016 at 9:33 AM, LABBE Corentin
> wrote:
>> of_match_device could return NULL, and so cause a NULL pointer
>> dereference later.
>>
>> For fixing this problem, we use of_device_get_match_data(), this will
>> simplify the code a little by
Le 09/12/2015 16:32, Jani Nikula a écrit :
> On Wed, 09 Dec 2015, LABBE Corentin wrote:
>> My latest commit introduce some case where a valid mode, could be
>> rejected.
>> simple_strtox functions stop at first non-digit character, but kstrtox not.
>> So args like "video=HDMI-A-1:720x480-16 at
Instead of rewriting write/readq, use linux/io-64-nonatomic-lo-hi.h which
already have them.
Signed-off-by: Corentin Labbe <clabbe.montj...@gmail.com>
---
include/drm/drm_os_linux.h | 14 +-
1 file changed, 1 insertion(+), 13 deletions(-)
diff --git a/include/drm/drm_os_lin
SH_LCD_MIPI_DSI is unused since commit 18b6562c243f ("fbdev: sh_mipi_dsi:
remove driver")
So no need to keep it.
Fixes: 18b6562c243f ("fbdev: sh_mipi_dsi: remove driver")
Signed-off-by: Corentin Labbe <clabbe.montj...@gmail.com>
---
drivers/video/Kconfig | 3 ---
1
All thoses files are not used by anybody.
Lets just remove them.
Signed-off-by: Corentin Labbe <cla...@baylibre.com>
---
drivers/gpu/drm/msm/adreno/a2xx.xml.h | 1845 -
drivers/gpu/drm/msm/dsi/mmss_cc.xml.h | 124 ---
drivers/gpu/drm/msm/hdmi/qfprom.xml.h
All thoses headers are not used by any source files.
Lets just remove them.
Signed-off-by: Corentin Labbe <cla...@baylibre.com>
---
.../drm/amd/include/asic_reg/uvd/uvd_4_0_sh_mask.h | 795 -
.../drm/amd/include/asic_reg/uvd/uvd_5_0_enum.h| 1211
.../d
All thoses headers are not used by any source files.
Lets just remove them.
Signed-off-by: Corentin Labbe <cla...@baylibre.com>
---
.../gpu/drm/amd/powerplay/inc/polaris10_ppsmc.h| 412 -
drivers/gpu/drm/amd/powerplay/inc/pp_feature.h | 67
2 files change
All thoses headers are not used by any source files.
Lets just remove them.
Signed-off-by: Corentin Labbe <cla...@baylibre.com>
---
.../gpu/drm/amd/include/asic_reg/vce/vce_1_0_d.h | 64 --
.../drm/amd/include/asic_reg/vce/vce_1_0_sh_mask.h | 99 --
2
All thoses headers are not used by any source files.
Lets just remove them.
Signed-off-by: Corentin Labbe <cla...@baylibre.com>
---
.../amd/include/asic_reg/sdma0/sdma0_4_0_default.h | 286 -
.../amd/include/asic_reg/sdma1/sdma1_4_0_default.h | 282 -
All thoses headers are not used by any source files.
Lets just remove them.
Signed-off-by: Corentin Labbe <cla...@baylibre.com>
---
.../drm/amd/include/asic_reg/dce/dce_11_2_enum.h | 6813
.../drm/amd/include/asic_reg/dce/dce_8_0_enum.h| 1117
2 files c
All thoses headers are not used by any source files.
Lets just remove them.
Signed-off-by: Corentin Labbe <cla...@baylibre.com>
---
.../gpu/drm/amd/include/asic_reg/gca/gfx_8_1_d.h | 2791 ---
.../drm/amd/include/asic_reg/gca/gfx_8_1_enum.h| 6808 --
.../drm/amd/include/asic_r
All thoses headers are not used by any source files.
Lets just remove them.
Signed-off-by: Corentin Labbe <cla...@baylibre.com>
---
.../amd/include/asic_reg/nbif/nbif_6_1_sh_mask.h | 10281 ---
1 file changed, 10281 deletions(-)
delete mode 100644 drivers/gpu/drm/amd/i
All thoses headers are not used by any source files.
Lets just remove them.
Signed-off-by: Corentin Labbe <cla...@baylibre.com>
---
.../drm/amd/include/asic_reg/oss/oss_2_4_enum.h| 1340 --
.../drm/amd/include/asic_reg/oss/oss_3_0_1_enum.h | 1464 ---
.
All thoses headers are not used by any source files.
Lets just remove them.
Signed-off-by: Corentin Labbe <cla...@baylibre.com>
---
.../drm/amd/include/asic_reg/umc/umc_6_0_default.h | 31 -
.../drm/amd/include/asic_reg/umc/umc_6_0_offset.h | 52 --
2
displayobject.h is not used by any source files.
Lets just remove them.
Signed-off-by: Corentin Labbe <cla...@baylibre.com>
---
drivers/gpu/drm/amd/include/displayobject.h | 249
1 file changed, 249 deletions(-)
delete mode 100644 drivers/gpu/drm/amd/i
All thoses headers are not used by any source files.
Lets just remove them.
Signed-off-by: Corentin Labbe <cla...@baylibre.com>
---
.../drm/amd/include/asic_reg/bif/bif_5_0_enum.h| 1198
.../drm/amd/include/asic_reg/bif/bif_5_1_enum.h| 1068 -
2
All thoses headers are not used by any source files.
Lets just remove them.
Signed-off-by: Corentin Labbe <cla...@baylibre.com>
---
.../gpu/drm/amd/include/asic_reg/smu/smu_6_0_d.h | 148 -
.../drm/amd/include/asic_reg/smu/smu_6_0_sh_mask.h | 715 ---
.../gpu/drm/amd/include/asic_r
All thoses headers are not used by any source files.
Lets just remove them.
Signed-off-by: Corentin Labbe <cla...@baylibre.com>
---
.../drm/amd/include/asic_reg/gmc/gmc_8_1_enum.h| 1198
.../drm/amd/include/asic_reg/gmc/gmc_8_2_enum.h| 1068 -
2
Hello
This patchset remove several headers which are not used by any source
file.
Regards
Change since v1:
- splited in multiple patchs
Change since v2
- Use --irreversible-delete for format-patch
Corentin Labbe (13):
drm/amd/include: remove unused asic_reg/oss headers
drm/amd/include
Since its inclusion in 2012 via commit bea8a429d91a ("drm/exynos: add rotator
ipp driver")
this header is not used by any source files and is empty.
Lets just remove it.
Signed-off-by: Corentin Labbe <cla...@baylibre.com>
---
drivers/gpu/drm/exynos/exynos_dr
The scheduler directory was removed via commit 1b1f42d8fde4 ("drm: move
amd_gpu_scheduler into common location")
Remove it from include path.
Signed-off-by: Corentin Labbe <cla...@baylibre.com>
---
drivers/gpu/drm/amd/amdgpu/Makefile | 1 -
1 file changed, 1 deletion(-)
diff
/amdgpu_device.o
cc1: warning: ../display/: No such file or directory [-Wmissing-include-dirs]
cc1: warning: ../display/include: No such file or directory
[-Wmissing-include-dirs]
[...]
This warning is shown for each file in amdgpu directory, so it spams a lot.
Signed-off-by: Corentin Labbe <
patch with JLawall's comments
Corentin Labbe (7):
powerpc: rename setbits32/clrbits32 to setbits_be32/clrbits_be32
include: add setbits_leXX/clrbits_leXX/clrsetbits_leXX in
linux/setbits.h
coccinelle: add xxxsetbits_leXX converting spatch
ata: ahci_sunxi: use xxxsetbitsi_le32 funct
.cocci
Signed-off-by: Corentin Labbe
---
scripts/add_new_include_in_source.py | 61 +++
scripts/coccinelle/misc/setbits32.cocci | 487 ++
.../coccinelle/misc/setbits32_relaxed.cocci | 487 ++
scripts/coccinelle/misc/setbits64.cocci | 487
This patch convert dwmac-sun8i driver to use all xxxsetbits_le32 functions.
Signed-off-by: Corentin Labbe
---
.../net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 62 +--
1 file changed, 16 insertions(+), 46 deletions(-)
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c
This patch convert meson DRM driver to use all xxxsetbits_le32 functions.
Signed-off-by: Corentin Labbe
Reviewed-by: Neil Armstrong
Tested-by: Neil Armstrong
---
drivers/gpu/drm/meson/meson_crtc.c | 14 +++---
drivers/gpu/drm/meson/meson_dw_hdmi.c | 33 +++--
drivers/gpu/drm
Since setbits32/clrbits32 work on be32, it's better to remove ambiguity on
the used data type.
Signed-off-by: Corentin Labbe
---
arch/powerpc/include/asm/fsl_lbc.h| 2 +-
arch/powerpc/include/asm/io.h | 4 +-
arch/powerpc/platforms/44x/canyonlands.c | 4
This patch convert meson stmmac glue driver to use all xxxsetbits_le32
functions.
Signed-off-by: Corentin Labbe
Reviewed-by: Neil Armstrong
---
.../ethernet/stmicro/stmmac/dwmac-meson8b.c | 56 ---
1 file changed, 22 insertions(+), 34 deletions(-)
diff --git a/drivers/net
This patch adds setbits_le32/clrbits_le32/clrsetbits_le32 and
setbits_le64/clrbits_le64/clrsetbits_le64 in linux/setbits.h header.
Signed-off-by: Corentin Labbe
---
include/linux/setbits.h | 84 +
1 file changed, 84 insertions(+)
create mode 100644
This patch converts ahci_sunxi to use xxxsetbits_le32 functions
Signed-off-by: Corentin Labbe
---
drivers/ata/ahci_sunxi.c | 62 +++-
1 file changed, 17 insertions(+), 45 deletions(-)
diff --git a/drivers/ata/ahci_sunxi.c b/drivers/ata/ahci_sunxi.c
index
This patch convert meson stmmac glue driver to use all xxxsetbits32 functions.
Signed-off-by: Corentin Labbe
---
.../net/ethernet/stmicro/stmmac/dwmac-meson8b.c| 56 +-
1 file changed, 22 insertions(+), 34 deletions(-)
diff --git a/drivers/net/ethernet/stmicro/stmmac
This patch adds setbits32/clrbits32/clrsetbits32 and
setbits64/clrbits64/clrsetbits64 in linux/setbits.h header.
Signed-off-by: Corentin Labbe
---
include/linux/setbits.h | 88 +
1 file changed, 88 insertions(+)
create mode 100644 include/linux
This patch convert meson DRM driver to use all xxxsetbits32 functions.
Signed-off-by: Corentin Labbe
---
drivers/gpu/drm/meson/meson_crtc.c | 14 ---
drivers/gpu/drm/meson/meson_dw_hdmi.c | 33 +
drivers/gpu/drm/meson/meson_plane.c | 13 ---
drivers/gpu/drm
Signed-off-by: Corentin Labbe
---
scripts/add_new_include_in_source.py| 61 +++
scripts/coccinelle/misc/setbits32.cocci | 487
scripts/coccinelle/misc/setbits32_relaxed.cocci | 487
scripts/coccinelle/misc/setbits64.cocci
Since setbits32/clrbits32 work on be32, it's better to remove ambiguity on
the used data type.
Signed-off-by: Corentin Labbe
---
arch/powerpc/include/asm/fsl_lbc.h | 2 +-
arch/powerpc/include/asm/io.h| 5 +-
arch/powerpc/platforms/44x/canyonlands.c
This patch convert dwmac-sun8i driver to use all xxxsetbits32 functions.
Signed-off-by: Corentin Labbe
---
drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 62 ++-
1 file changed, 16 insertions(+), 46 deletions(-)
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac
This patch converts ahci_sunxi to use xxxsetbits32 functions
Signed-off-by: Corentin Labbe
---
drivers/ata/ahci_sunxi.c | 51
1 file changed, 12 insertions(+), 39 deletions(-)
diff --git a/drivers/ata/ahci_sunxi.c b/drivers/ata/ahci_sunxi.c
Lawall for her help on the coccinelle
patch.
Regards
Changes since v1:
- renamed LE functions to _leXX
- updated coccinnelle patch with JLawall's comments
Corentin Labbe (7):
powerpc: rename setbits32/clrbits32 to setbits32_be/clrbits32_be
include: add
setbits32/clrbits32/clrsetbits32
tcm-sita.h is unused since commit 0d6fa53fd805 ("drm/omap: Use bitmaps for
TILER placement")
Let's remove it.
Signed-off-by: Corentin Labbe
---
drivers/gpu/drm/omapdrm/tcm-sita.h | 93 --
1 file changed, 93 deletions(-)
delete mode 100644 drive
Hello
When I rmmod sun4i_drm I got
[ 546.417886] Internal error: Oops: 17 [#1] SMP ARM
[ 547.024731] CPU: 0 PID: 18811 Comm: rmmod Not tainted
5.1.0-next-20190515-00100-gf33d93f7d2a0 #39
[ 547.033588] Hardware name: Allwinner sun7i (A20) Family
[ 547.038816] PC is at
On Wed, Aug 14, 2019 at 07:49:27PM +0200, Daniel Vetter wrote:
> On Wed, Aug 14, 2019 at 04:50:33PM +0200, Corentin Labbe wrote:
> > Hello
> >
> > Since lot of release (at least since 4.19), I hit the following error
> > message:
> > DMA-API: cacheline tr
Hello
Since lot of release (at least since 4.19), I hit the following error message:
DMA-API: cacheline tracking ENOMEM, dma-debug disabled
After hitting that, I try to check who is creating so many DMA mapping and see:
cat /sys/kernel/debug/dma-api/dump | cut -d' ' -f2 | sort | uniq -c
6
Converts display/faraday,tve200.txt to yaml.
Signed-off-by: Corentin Labbe
---
.../bindings/display/faraday,tve200.txt | 54 ---
.../bindings/display/faraday,tve200.yaml | 92 +++
2 files changed, 92 insertions(+), 54 deletions(-)
delete mode 100644
Hello
Since some days on next, nouveau fail to load:
[2.754087] nouveau :02:00.0: vgaarb: deactivate vga console
[2.761260] Console: switching to colour dummy device 80x25
[2.766888] nouveau :02:00.0: NVIDIA MCP77/MCP78 (0aa480a2)
[2.783954] nouveau :02:00.0: bios:
The address and reg adds no value to the port node, remove them.
Signed-off-by: Corentin Labbe
---
arch/arm/boot/dts/gemini-dlink-dir-685.dts | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/gemini-dlink-dir-685.dts
b/arch/arm/boot/dts/gemini-dlink-dir
dtb_check complains about #address-cells and #size-cells, so lets
remove them.
Signed-off-by: Corentin Labbe
---
arch/arm/boot/dts/gemini.dtsi | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/arm/boot/dts/gemini.dtsi b/arch/arm/boot/dts/gemini.dtsi
index fa708f5d0c72..34961e5bc7b2
Converts display/faraday,tve200.txt to yaml.
Signed-off-by: Corentin Labbe
---
Changes since v1:
- added two subsequent patchs fixing issue found when converting
- fixed all issues reported by Rob Herring
.../bindings/display/faraday,tve200.txt | 54 ---
.../bindings/display
From: Jean-Jacques Hiblot
This patch adds support for an "enable GPIO".
Signed-off-by: Jean-Jacques Hiblot
Signed-off-by: Corentin LABBE
---
drivers/video/backlight/led_bl.c | 13 +
1 file changed, 13 insertions(+)
diff --git a/drivers/video/backlight/led_bl.c b/dri
make dt_binding_check fail on lot of mediatek bindings due to missing
includes.
Signed-off-by: Corentin Labbe
---
2 files remains to be fixed, but their examples use some variable undefined, so
I cannot do anything:
- display/mediatek/mediatek,ethdr.yaml
- display/mediatek/mediatek,merge.yaml
50 matches
Mail list logo
