Re: [PATCH] hwmon: tmp103: use SIMPLE_DEV_PM_OPS helper macro

Hello Guenter, Heiko, On Mon, Apr 24, 2017 at 9:28 AM, Heiko Schocher wrote: > Hello Guenter, Rahul, > > Hmm.. I like this patch too, but have also no idea, what is preffered. > > Looking into drivers/hwmon > > pollux:linux hs [master] $ grep -lr __maybe_unused drivers/hwmon/ > drivers/hwmon/tmp1

[PATCH] hwmon: tmp103: use SIMPLE_DEV_PM_OPS helper macro

Replace ifdefs with SIMPLE_DEV_PM_OPS helper macro. Signed-off-by: Rahul Bedarkar --- drivers/hwmon/tmp103.c | 17 - 1 file changed, 4 insertions(+), 13 deletions(-) diff --git a/drivers/hwmon/tmp103.c b/drivers/hwmon/tmp103.c index d0bb28b..7f85b14 100644 --- a/drivers/hwmon

[PATCH] power: supply: generic-adc-battery: use SIMPLE_DEV_PM_OPS helper macro

Replace ifdefs with SIMPLE_DEV_PM_OPS helper macro. Signed-off-by: Rahul Bedarkar --- drivers/power/supply/generic-adc-battery.c | 17 - 1 file changed, 4 insertions(+), 13 deletions(-) diff --git a/drivers/power/supply/generic-adc-battery.c b/drivers/power/supply/generic-adc

[PATCH] MAINTAINERS: Update email-id of Rahul Bedarkar

I'm no longer with Imagination Technologies. I am still interested in maintaining or reviewing DTS patches for Ci40 if any. Update email-id to an active one. Signed-off-by: Rahul Bedarkar --- I'm not sure via which tree this should get in. Since this entry was merged via MIPS tree, I

[PATCH] dt-bindings: input: rotary-encoder: fix typo

s/rollove/rollover/ Signed-off-by: Rahul Bedarkar --- Documentation/devicetree/bindings/input/rotary-encoder.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/input/rotary-encoder.txt b/Documentation/devicetree/bindings/input/rotary

[RFC 5/6] mtd: spi-nor: add support to read/write user OTP

. Then this framework will convert it to actual physical address while reading/writing OTP area. _get_user_prot_info hook is likely to be specific to manufacturer as it involves reading status register to get status of lock bits for OTP. Signed-off-by: Rahul Bedarkar Cc: David Woodhouse Cc: Brian

[RFC 6/6] mtd: spi-nor: enable OTP support for s25fl016k

s25fl016k has 3 OTP areas of 256 bytes each. First bank starts at 0x1000, second at 0x2000 and third at 0x3000. When OTP area is unlocked, it can be rewritten. Lock bit for first, second and third area is bit 3, 4, 5 of Status Register 2. Signed-off-by: Rahul Bedarkar Cc: David Woodhouse Cc

[RFC 3/6] mtd: m25p80: don't pass spi_nor to helper methods

Helper methods m25p_addr2cmd, m25p80_rx_nbits and m25p_cmdsz accepts spi_nor. But with upcoming implementation of read_xfer and write_xfer, we need to pass addr_width and flash_read from cfg. Signed-off-by: Rahul Bedarkar Cc: David Woodhouse Cc: Brian Norris Cc: Boris Brezillon Cc: Marek

[RFC 4/6] mtd: m25p80: implement read_xfer and write_xfer

Implement read_xfer and write_xfer interfaces provided by SPI-NOR. These will be used in upcoming OTP support in SPI-NOR to read/write OTP area. Signed-off-by: Rahul Bedarkar Cc: David Woodhouse Cc: Brian Norris Cc: Boris Brezillon Cc: Marek Vasut Cc: Richard Weinberger Cc: Cyrille Pitchen

[RFC 2/6] mtd: spi-nor: change return value of read_xfer and write_xfer

ned-off-by: Rahul Bedarkar Cc: David Woodhouse Cc: Brian Norris Cc: Boris Brezillon Cc: Marek Vasut Cc: Richard Weinberger Cc: Cyrille Pitchen --- include/linux/mtd/spi-nor.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/linux/mtd/spi-nor.h b/include/linux/mtd

[RFC 1/6] Revert "mtd: spi-nor: remove unused read_xfer/write_xfer hooks"

This reverts commit 79c452adb159dc9abc507ea13faec8d115a78758. This was removed because hooks were never used by any driver. But with upcoming OTP support in SPI-NOR, m25p80 driver will implement it and will be used for reading/writing OTP area. Signed-off-by: Rahul Bedarkar Cc: David Woodhouse

[RFC 0/6] Add user OTP support in SPI-NOR

on. Cc: David Woodhouse Cc: Brian Norris Cc: Boris Brezillon Cc: Marek Vasut Cc: Richard Weinberger Cc: Cyrille Pitchen Rahul Bedarkar (6): Revert "mtd: spi-nor: remove unused read_xfer/write_xfer hooks" mtd: spi-nor: change return value of read_xfer and write_xfer mtd: m25p8

[PATCH v2 2/2] MIPS: DTS: img: add device tree for Marduk board

board with following peripherals: * PWM based heartbeat LED * GPIO based buttons * SPI NOR flash on SPI1 * UART0 and UART1 * SD card * Ethernet * USB * PWM * ADC * I2C Signed-off-by: Rahul Bedarkar --- Changes in v2: - Correct RAM size. It is 256MB instead of 128MB. - Rename nodes pwm_leds -&

[PATCH v2 1/2] MIPS: DTS: add base device tree for Pistachio SoC

Timer * USB * IR * Interrupt Controller Signed-off-by: Rahul Bedarkar --- Changes in v2: - No change --- MAINTAINERS | 2 +- arch/mips/boot/dts/img/pistachio.dtsi | 924 ++ 2 files changed, 925 insertions(+), 1 deletion(-) create mode 1

Re: [PATCH 2/2] MIPS: DTS: img: add device tree for Marduk board

Hi, On Monday 10 October 2016 07:51 PM, Rob Herring wrote: + + memory { Is 0 the actual base, or that gets filled in by bootloader? If the formet, add unit address. Bootloader (uboot) can override or fixup memory node. But with version of bootloader I tested with, base address is har

[PATCH 2/2] MIPS: DTS: img: add device tree for Marduk board

board with following peripherals: * PWM based heartbeat LED * GPIO based buttons * SPI NOR flash on SPI1 * UART0 and UART1 * SD card * Ethernet * USB * PWM * ADC * I2C Signed-off-by: Rahul Bedarkar --- .../bindings/mips/img/pistachio-marduk.txt | 10 ++ MAINTA

[PATCH 1/2] MIPS: DTS: add base device tree for Pistachio SoC

Timer * USB * IR * Interrupt Controller Signed-off-by: Rahul Bedarkar --- MAINTAINERS | 2 +- arch/mips/boot/dts/img/pistachio.dtsi | 924 ++ 2 files changed, 925 insertions(+), 1 deletion(-) create mode 100644 arch/mips/boot/dt

Re: [PATCH] char: misc: use pr_err instead of printk

On Sun, Oct 12, 2014 at 12:05 AM, Joe Perches wrote: > On Sun, 2014-10-12 at 00:01 +0530, Rahul Bedarkar wrote: > [] >> diff --git a/drivers/char/misc.c b/drivers/char/misc.c > [] >> @@ -280,12 +280,12 @@ static int __init misc_init(void) >> >> err = -

[PATCH] char: misc: fix coding style issues

This patch fixes checkpatch.pl error ERROR: space required after that ',' (ctx:VxV) Signed-off-by: Rahul Bedarkar --- drivers/char/misc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/char/misc.c b/drivers/char/misc.c index a6b96b1..47c9b04 100644 ---

[PATCH] char: misc: use pr_err instead of printk

Signed-off-by: Rahul Bedarkar --- drivers/char/misc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/char/misc.c b/drivers/char/misc.c index 75b6de9..a6b96b1 100644 --- a/drivers/char/misc.c +++ b/drivers/char/misc.c @@ -280,12 +280,12 @@ static int __init

[PATCH] char: misc: EXPORT_SYMBOL should immediately follow its function

This patch fixes checkpatch.pl warning WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable Signed-off-by: Rahul Bedarkar --- drivers/char/misc.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/char/misc.c b/drivers/char/misc.c index 3111e86

[PATCH] char: misc: add missing blank line after declarations

Signed-off-by: Rahul Bedarkar --- drivers/char/misc.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/char/misc.c b/drivers/char/misc.c index cb32ea9..3111e86 100644 --- a/drivers/char/misc.c +++ b/drivers/char/misc.c @@ -186,6 +186,7 @@ int misc_register(struct miscdevice *misc

[PATCH] char: misc: remove trailing whitespaces

Signed-off-by: Rahul Bedarkar --- drivers/char/misc.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/char/misc.c b/drivers/char/misc.c index b4cf821..cb32ea9 100644 --- a/drivers/char/misc.c +++ b/drivers/char/misc.c @@ -117,14 +117,14 @@ static int

[PATCH] char: misc: fix coding style issues

This patch fixes checkpatch.pl error ERROR: "foo * bar" should be "foo *bar" Signed-off-by: Rahul Bedarkar --- drivers/char/misc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/char/misc.c b/drivers/char/misc.c index ffa97d2..b4cf821 10064

[PATCH] staging: dgap: remove unused variable 'orig_count'

This patch fixes sparse warning warning: variable ‘orig_count’ set but not used [-Wunused-but-set-variable] Signed-off-by: Rahul Bedarkar --- drivers/staging/dgap/dgap.c | 8 1 file changed, 8 deletions(-) diff --git a/drivers/staging/dgap/dgap.c b/drivers/staging/dgap/dgap.c index

[PATCH] staging: bcm: fix sparse warning in module_param

This patch fixes sparse warning in module_param warning: pointer targets in return differ in signedness [-Wpointer-sign] Signed-off-by: Rahul Bedarkar --- drivers/staging/bcm/InterfaceInit.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/bcm/InterfaceInit.c

[PATCH] Documentation: kmemleak: correct spelling mistake

Signed-off-by: Rahul Bedarkar --- Documentation/kmemleak.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/kmemleak.txt b/Documentation/kmemleak.txt index b772418..f4f033c 100644 --- a/Documentation/kmemleak.txt +++ b/Documentation/kmemleak.txt @@ -122,7

[PATCH] staging: octeon: fix coding style

This patch fixes checkpatch.pl warning. Add blank line after declaration. Signed-off-by: Rahul Bedarkar --- drivers/staging/octeon/ethernet-xaui.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/octeon/ethernet-xaui.c b/drivers/staging/octeon/ethernet-xaui.c index 419f8c3