Re: [PATCH 1/6] dt-bindings: display: Document NewVision NV3052C DT node

2020-07-27 Thread Maarten ter Huurne
On Monday, 27 July 2020 21:10:52 CEST Sam Ravnborg wrote: > > +description: | > > + This is a driver for 320x240 TFT panels, > > The binding describes the HW, not the driver. So please re-phrase this > part. > > This datasheet: > https://www.phoenixdisplay.com/wp-content/uploads/2019/05/NV3052C-

Re: [PATCH 04/15] clk: Add Ingenic jz4770 CGU driver

2017-06-08 Thread Maarten ter Huurne
On Wednesday 07 June 2017 13:59:43 Stephen Boyd wrote: > On 06/07, Paul Cercueil wrote: > > Add support for the clocks provided by the CGU in the Ingenic JZ4770 > > SoC. > > > > Signed-off-by: Paul Cercueil > > Signed-off-by: Maarten ter Huurne > > Signe

[PATCH] dma-debug: Avoid NULL dereference when checking sync

2016-10-31 Thread Maarten ter Huurne
check_sync() calls bucket_find_contain(), which in turn calls dma_get_max_seg_size(), which dereferences the device pointer. Signed-off-by: Maarten ter Huurne --- lib/dma-debug.c | 5 + 1 file changed, 5 insertions(+) diff --git a/lib/dma-debug.c b/lib/dma-debug.c index 8971370..84c6e88

[PATCH] MIPS: zboot: Add "uzImage.bin" target

2016-10-31 Thread Maarten ter Huurne
uzImage.bin is vmlinuz.bin wrapped in a legacy U-Boot image. Since the extraction code is inside the image, it does not depend on the boot loader to extract the kernel. Signed-off-by: Maarten ter Huurne --- arch/mips/Makefile | 4 arch/mips/boot/compressed/Makefile | 4

[PATCH v2] regmap: cache: Fix num_reg_defaults computation from reg_defaults_raw

2016-07-29 Thread Maarten ter Huurne
register cache initializes from the full array. Fixed it by excluding non-readable registers from the count as well. Signed-off-by: Maarten ter Huurne --- drivers/base/regmap/regcache.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/base/regmap/regcache.c b

[PATCH] regmap: cache: Fix num_reg_defaults computation from reg_defaults_raw

2016-07-29 Thread Maarten ter Huurne
register cache initializes from the full array. Fixed it by excluding non-readable registers from the count as well. Signed-off-by: Maarten ter Huurne --- drivers/base/regmap/regcache.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/base/regmap/regcache.c b

[PATCH 2/3] MIPS: JZ4740: Probe OHCI platform device via DT

2016-04-18 Thread Maarten ter Huurne
ed by default; boards that want to use it can override the "status" property. The mass-production Qi LB60 boards don't use the USB host controller. Signed-off-by: Maarten ter Huurne --- arch/mips/boot/dts/ingenic/jz4740.dtsi | 14 ++ arch/mips/include/asm/mach-jz

[PATCH 1/3] MIPS: JZ4740: Qi LB60: Remove support for AVT2 variant

2016-04-18 Thread Maarten ter Huurne
AVT2 was a prototype board of which about 5 were made, none of which are in use anymore. Signed-off-by: Maarten ter Huurne --- arch/mips/jz4740/board-qi_lb60.c | 52 ++-- 1 file changed, 2 insertions(+), 50 deletions(-) diff --git a/arch/mips/jz4740/board

[PATCH 3/3] USB: ohci-jz4740: Remove obsolete driver

2016-04-18 Thread Maarten ter Huurne
The ohci-platform driver can control the clock, while usb-nop-xceiv as the PHY can control the vbus regulator. So this JZ4740-specific glue is not needed anymore. Signed-off-by: Maarten ter Huurne --- drivers/usb/host/ohci-hcd.c| 5 - drivers/usb/host/ohci-jz4740.c | 245

[PATCH] mtd: nand: jz4740: Remove unused local variable

2016-04-18 Thread Maarten ter Huurne
Signed-off-by: Maarten ter Huurne --- drivers/mtd/nand/jz4740_nand.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/mtd/nand/jz4740_nand.c b/drivers/mtd/nand/jz4740_nand.c index 673ceb2..2f39ee1 100644 --- a/drivers/mtd/nand/jz4740_nand.c +++ b/drivers/mtd/nand/jz4740_nand.c

[PATCH v2 4/4] regulator: act8865: Configure register access for act8600

2016-03-19 Thread Maarten ter Huurne
This can be used to expose the act8600 registers via debugfs. Signed-off-by: Maarten ter Huurne --- drivers/regulator/act8865-regulator.c | 74 ++- 1 file changed, 73 insertions(+), 1 deletion(-) diff --git a/drivers/regulator/act8865-regulator.c b/drivers

[PATCH v2 2/4] regulator: act8865: Remove "too many regulators" error handler

2016-03-19 Thread Maarten ter Huurne
The error handler might have some value as a sanity check on the platform data, but the platform data could be broken in many other ways that are not checked for (unknown IDs, duplicate IDs), so I see no reason to perform only this specific check. Signed-off-by: Maarten ter Huurne --- drivers/

[PATCH v2 3/4] regulator: act8865: Pass of_node via act8865_regulator_data

2016-03-19 Thread Maarten ter Huurne
This makes the code easier to read and it avoids a dynamic memory allocation. Signed-off-by: Maarten ter Huurne --- drivers/regulator/act8865-regulator.c | 28 include/linux/regulator/act8865.h | 2 ++ 2 files changed, 14 insertions(+), 16 deletions(-) diff

[PATCH v2 1/4] regulator: act8865: Remove redundant dev lookups

2016-03-19 Thread Maarten ter Huurne
The local variable "dev" already contains a pointer to the device, so there is no need to take the address of "client->dev" again. Signed-off-by: Maarten ter Huurne --- drivers/regulator/act8865-regulator.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-

Re: [PATCH 7/7] regulator: act8865: Init at subsys level

2016-03-18 Thread Maarten ter Huurne
On Monday 29 February 2016 20:35:37 Mark Brown wrote: > On Sun, Feb 28, 2016 at 04:53:29PM +0100, Maarten ter Huurne wrote: > > Since the defined regulators are used in other drivers, we can avoid > > deferred probing by registering this driver sooner. > > No, don't p

[PATCH] jz4740-battery: Correct voltage change check

2016-02-28 Thread Maarten ter Huurne
The check is supposed to avoid redundant update notifications, so it should check for the difference between old and new voltage exceeding a threshold. Also make sure the result of a failed read is never stored. Signed-off-by: Maarten ter Huurne --- drivers/power/jz4740-battery.c | 2 +- 1

[PATCH 2/7] regulator: act8865: Remove redundant dev lookups

2016-02-28 Thread Maarten ter Huurne
The local variable "dev" already contains a pointer to the device, so there is no need to take the address of "client->dev" again. Signed-off-by: Maarten ter Huurne --- drivers/regulator/act8865-regulator.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-

[PATCH 4/7] regulator: act8865: Rename platform_data field to init_data

2016-02-28 Thread Maarten ter Huurne
Make the field name match its type. Signed-off-by: Maarten ter Huurne --- drivers/regulator/act8865-regulator.c | 4 ++-- include/linux/regulator/act8865.h | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/regulator/act8865-regulator.c b/drivers/regulator

[PATCH 7/7] regulator: act8865: Init at subsys level

2016-02-28 Thread Maarten ter Huurne
Since the defined regulators are used in other drivers, we can avoid deferred probing by registering this driver sooner. Signed-off-by: Maarten ter Huurne --- drivers/regulator/act8865-regulator.c | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/drivers/regulator

[PATCH 6/7] regulator: act8865: Specify fixed voltage of 3.3V for ACT8600's REG9

2016-02-28 Thread Maarten ter Huurne
actual output voltage of the hardware. Signed-off-by: Maarten ter Huurne --- drivers/regulator/act8865-regulator.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/regulator/act8865-regulator.c b/drivers/regulator/act8865-regulator.c index cdfe082..ac6956d 100644 --- a

[PATCH 5/7] regulator: act8865: Pass of_node via act8865_regulator_data

2016-02-28 Thread Maarten ter Huurne
This makes the code easier to read and it avoids a dynamic memory allocation. Note that the "too many regulators" error handler was broken prior to its removal in this commit, since it dereferenced pdata, which can be NULL in the non-DT case. Signed-off-by: Maarten ter Huurne --

[PATCH 3/7] regulator: act8865: Remove "static" from local variable

2016-02-28 Thread Maarten ter Huurne
There is no need to preserve its value between calls. I guess this was a copy-paste slip-up. Signed-off-by: Maarten ter Huurne --- drivers/regulator/act8865-regulator.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/regulator/act8865-regulator.c b/drivers/regulator

[PATCH 1/7] regulator: act8865: Expose act8600 registers via debugfs

2016-02-28 Thread Maarten ter Huurne
The read/write/volatile configuration is valid also when debugfs is not enabled, but it doesn't add any value then. Signed-off-by: Maarten ter Huurne --- drivers/regulator/act8865-regulator.c | 67 --- 1 file changed, 63 insertions(+), 4 deletions(-) diff

[PATCH] usb: phy: generic: Handle late registration of gadget

2016-02-28 Thread Maarten ter Huurne
It is possible for the VBUS detect GPIO interrupt to occur before nop_set_peripheral() is called, in which case otg->gadget is NULL. Signed-off-by: Maarten ter Huurne --- drivers/usb/phy/phy-generic.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/drivers/

[PATCH] dma-debug: Avoid NULL dereference when checking sync

2016-02-28 Thread Maarten ter Huurne
check_sync() calls bucket_find_contain(), which in turn calls dma_get_max_seg_size(), which dereferences the device pointer. Signed-off-by: Maarten ter Huurne --- lib/dma-debug.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/lib/dma-debug.c b/lib/dma-debug.c index 4a1515f..4960402

[PATCH] fbcon: Fix option parsing control flow in fb_console_setup

2014-10-09 Thread Maarten ter Huurne
nconditionally: strsep ensures *options != ','), causing any following option to be ignored, while for "vc:" the parse code would go on to parse further options within the same token, which could lead to invalid input being accepted. Signed-off-by: Maarten ter Huurne Acked-by: Paul

Re: [PATCH] fonts: Add 6x10 font

2014-10-08 Thread Maarten ter Huurne
On Tuesday 30 September 2014 12:41:46 Tomi Valkeinen wrote: > Hi, > > On 09/09/14 14:46, Maarten ter Huurne wrote: > > This font is suitable for framebuffer consoles on devices with a > > 320x240 screen, to get a reasonable number of characters (53x24) that > > a

[PATCH] MIPS: Removed declaration of obsolete arch_init_clk_ops()

2014-09-09 Thread Maarten ter Huurne
Signed-off-by: Maarten ter Huurne --- arch/mips/include/asm/clock.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/arch/mips/include/asm/clock.h b/arch/mips/include/asm/clock.h index 778e32d..4809c29 100644 --- a/arch/mips/include/asm/clock.h +++ b/arch/mips/include/asm/clock.h @@ -35,9

[PATCH] fonts: Add 6x10 font

2014-09-09 Thread Maarten ter Huurne
glyhps so they are more distinct and better fill the available space. Signed-off-by: Maarten ter Huurne --- include/linux/font.h |4 +- lib/fonts/Kconfig |9 + lib/fonts/Makefile|1 + lib/fonts/font_6x10.c | 3086 + lib/fonts

[PATCH] USB: musb: Avoid null pointer dereference in debug logging

2013-08-18 Thread Maarten ter Huurne
Since commit 511f3c53 (usb: gadget: udc-core: fix a regression during gadget driver unbinding) usb_gadget_remove_driver will pass NULL for the driver argument. Signed-off-by: Maarten ter Huurne --- drivers/usb/musb/musb_gadget.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff

[PATCH] USB: musb: Avoid null pointer dereference in debug logging

2013-08-18 Thread Maarten ter Huurne
Since commit 511f3c53 usb_gadget_remove_driver will pass NULL for the driver argument. Signed-off-by: Maarten ter Huurne --- drivers/usb/musb/musb_gadget.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/usb/musb/musb_gadget.c b/drivers/usb/musb/musb_gadget.c index

[PATCH] regmap: Implemented default cache sync operation

2013-06-02 Thread Maarten ter Huurne
This can be used for cache types for which syncing values one by one is equally efficient as syncing a range, such as the flat cache. Signed-off-by: Maarten ter Huurne --- drivers/base/regmap/regcache.c | 46 1 file changed, 42 insertions(+), 4

[PATCH] regmap: regcache-rbtree: Fixed node range check on sync

2013-05-31 Thread Maarten ter Huurne
A node starting before the minimum register is no reason to reject it, since its end could be in range. The check for the end already exists two lines lower, so we can just remove the incorrect check. Signed-off-by: Maarten ter Huurne --- drivers/base/regmap/regcache-rbtree.c |2 -- 1 file

[PATCH] regmap: regcache-rbtree: Fixed node range check on sync

2013-05-31 Thread Maarten ter Huurne
Hi, This patch fixes what I think is a bug in regcache_rbtree_sync(). It is something I noticed when reading the code, I don't have a test scenario for it. So please review it carefully. Bye, Maarten -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the

[PATCH] regmap: regcache-flat: Implemented sync operation

2013-05-31 Thread Maarten ter Huurne
Signed-off-by: Maarten ter Huurne --- drivers/base/regmap/regcache-flat.c | 32 1 file changed, 32 insertions(+) diff --git a/drivers/base/regmap/regcache-flat.c b/drivers/base/regmap/regcache-flat.c index d9762e4..1f18bec 100644 --- a/drivers/base/regmap

[PATCH] ext4: Fixed corruption when online resizing a fs with 1K block size

2013-05-06 Thread Maarten ter Huurne
Subtracting the number of the first data block places the superblock backups one block too early, corrupting the file system. When the block size is larger than 1K, the first data block is 0, so the subtraction has no effect and no corruption occurs. Signed-off-by: Maarten ter Huurne --- fs

Re: [PATCH 0/3] MIPS: JZ4740: Move PWM driver to PWM framework

2012-09-02 Thread Maarten ter Huurne
On Sunday 02 September 2012 21:27:53 Thierry Reding wrote: > On Sun, Sep 02, 2012 at 03:25:55PM +0200, Maarten ter Huurne wrote: > > I tested the "for-next" branch on the Dingoo A320 with the pwm-backlight > > driver. It didn't work at first, because the PWM number a

Re: [PATCH 0/3] MIPS: JZ4740: Move PWM driver to PWM framework

2012-09-02 Thread Maarten ter Huurne
On Sunday 02 September 2012 11:52:27 Thierry Reding wrote: > This small series fixes a build error due to a circular header > dependency, exports the timer API so it can be used outside of > the arch/mips/jz4740 tree and finally moves and converts the > JZ4740 PWM driver to the PWM framework. > >