Because it looks neater.
Signed-off-by: Peter Rosin
---
drivers/i2c/algos/i2c-algo-bit.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/i2c/algos/i2c-algo-bit.c b/drivers/i2c/algos/i2c-algo-bit.c
index 3df0efd69ae3..4a34f311e1ff 100644
--- a/drivers/i2c/algos/i2c
Because it looks neater.
Signed-off-by: Peter Rosin
---
drivers/i2c/algos/i2c-algo-pcf.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/i2c/algos/i2c-algo-pcf.c b/drivers/i2c/algos/i2c-algo-pcf.c
index 270d84bfc2c6..bb60c1bd1f2d 100644
--- a/drivers/i2c/algos
Because it looks neater.
Signed-off-by: Peter Rosin
---
drivers/i2c/busses/i2c-axxia.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/i2c/busses/i2c-axxia.c b/drivers/i2c/busses/i2c-axxia.c
index 13f07482ec68..f70b097fd567 100644
--- a/drivers/i2c/busses/i2c
Because it looks neater.
Signed-off-by: Peter Rosin
---
drivers/i2c/busses/i2c-efm32.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/i2c/busses/i2c-efm32.c b/drivers/i2c/busses/i2c-efm32.c
index aa336ba89aa3..5f2bab878b2c 100644
--- a/drivers/i2c/busses/i2c-efm32
Because it looks neater.
Signed-off-by: Peter Rosin
---
drivers/i2c/busses/i2c-hix5hd2.c | 9 ++---
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/drivers/i2c/busses/i2c-hix5hd2.c b/drivers/i2c/busses/i2c-hix5hd2.c
index bb68957d3da5..399b64c4c620 100644
--- a/drivers/i2c
Because it looks neater.
Also restructure debug output for resends, since that code as a
result is only handling debug output.
Signed-off-by: Peter Rosin
---
drivers/i2c/busses/i2c-stu300.c | 22 +-
1 file changed, 13 insertions(+), 9 deletions(-)
diff --git a/drivers/i2c
Because it looks neater.
Signed-off-by: Peter Rosin
---
drivers/i2c/busses/i2c-ocores.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/i2c/busses/i2c-ocores.c b/drivers/i2c/busses/i2c-ocores.c
index 8c42ca7107b2..a540916689b2 100644
--- a/drivers/i2c/busses/i2c
Because it looks neater.
Signed-off-by: Peter Rosin
---
drivers/i2c/busses/i2c-xiic.c | 11 ++-
1 file changed, 2 insertions(+), 9 deletions(-)
diff --git a/drivers/i2c/busses/i2c-xiic.c b/drivers/i2c/busses/i2c-xiic.c
index c80527816ad0..1818c3b5c8d7 100644
--- a/drivers/i2c/busses
Because it looks neater.
i2c_imx_dma_write and i2c_imx_write are always called with a
write in msgs->flags, and i2c_imx_read with a read.
Signed-off-by: Peter Rosin
---
drivers/i2c/busses/i2c-imx.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/i2c/bus
Because it looks neater.
qup_i2c_write_tx_fifo_v1 is always called with a write in
qup->msg->flags.
Signed-off-by: Peter Rosin
---
drivers/i2c/busses/i2c-qup.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/i2c/busses/i2c-qup.c b/drivers/i2c/busses/i2c-qup.c
Because it looks neater.
Signed-off-by: Peter Rosin
---
drivers/i2c/busses/i2c-pasemi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/i2c/busses/i2c-pasemi.c b/drivers/i2c/busses/i2c-pasemi.c
index df1dbc92a024..55fd5c6f3cca 100644
--- a/drivers/i2c/busses/i2c
Because it looks neater.
Signed-off-by: Peter Rosin
---
drivers/i2c/busses/i2c-rcar.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/i2c/busses/i2c-rcar.c b/drivers/i2c/busses/i2c-rcar.c
index c6915b835396..17b91ddbe013 100644
--- a/drivers/i2c/busses/i2c-rcar.c
Because it looks neater.
Signed-off-by: Peter Rosin
---
drivers/i2c/busses/i2c-riic.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/i2c/busses/i2c-riic.c b/drivers/i2c/busses/i2c-riic.c
index 95c2f1ce3cad..5f1fca7880b1 100644
--- a/drivers/i2c/busses/i2c
Because it looks neater.
Signed-off-by: Peter Rosin
---
drivers/i2c/busses/i2c-kempld.c | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/drivers/i2c/busses/i2c-kempld.c b/drivers/i2c/busses/i2c-kempld.c
index e879190b5d1d..1c874aaa0447 100644
--- a/drivers/i2c/busses
Because it looks neater.
Signed-off-by: Peter Rosin
---
drivers/i2c/busses/i2c-mxs.c | 9 +++--
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/drivers/i2c/busses/i2c-mxs.c b/drivers/i2c/busses/i2c-mxs.c
index e617bd600794..f62ae3d42232 100644
--- a/drivers/i2c/busses/i2c
Because it looks neater.
Signed-off-by: Peter Rosin
---
drivers/i2c/busses/i2c-imx-lpi2c.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/i2c/busses/i2c-imx-lpi2c.c
b/drivers/i2c/busses/i2c-imx-lpi2c.c
index e6da2c7a9a3e..159d23211600 100644
--- a/drivers/i2c
Because it looks neater.
Signed-off-by: Peter Rosin
---
drivers/i2c/busses/i2c-emev2.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/i2c/busses/i2c-emev2.c b/drivers/i2c/busses/i2c-emev2.c
index d2e84480fbe9..ba9b6ea48a31 100644
--- a/drivers/i2c/busses/i2c-emev2.c
Because it looks neater.
pch_i2c_writebytes is always called with a write in msgs->flags,
and pch_i2c_readbytes with a read.
Signed-off-by: Peter Rosin
---
drivers/i2c/busses/i2c-eg20t.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/i2c/busses/i2c-eg20t.
Because it looks neater and allows factoring out some code
that is now common between if and else.
Signed-off-by: Peter Rosin
---
drivers/i2c/busses/i2c-diolan-u2c.c | 11 ---
1 file changed, 4 insertions(+), 7 deletions(-)
diff --git a/drivers/i2c/busses/i2c-diolan-u2c.c
b/drivers
Because it looks neater.
Signed-off-by: Peter Rosin
---
drivers/i2c/busses/i2c-aspeed.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/i2c/busses/i2c-aspeed.c b/drivers/i2c/busses/i2c-aspeed.c
index 7d4aeb4465b3..60e4d0e939a3 100644
--- a/drivers/i2c/busses/i2c
Because it looks neater.
Signed-off-by: Peter Rosin
---
drivers/i2c/algos/i2c-algo-pca.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/i2c/algos/i2c-algo-pca.c b/drivers/i2c/algos/i2c-algo-pca.c
index e370804ec8bc..883a290f6a4d 100644
--- a/drivers/i2c/algos
Hi!
The nice little inline i2c_8bit_addr_from_msg is not getting
enough use. This series improves the situation and drops a
bunch of lines in the process.
I have only compile-tested (that part fine, at least over here).
Cheers,
Peter
Peter Rosin (21):
i2c: algo: bit: make use of
On 2018-05-14 18:11, Joe Perches wrote:
> On Mon, 2018-05-14 at 16:53 +0200, Peter Rosin wrote:
>> Hi!
>>
>> The nice little inline i2c_8bit_addr_from_msg is not getting
>> enough use. This series improves the situation and drops a
>> bunch of lines in the proces
On 2018-05-14 18:59, Andy Shevchenko wrote:
> On Mon, May 14, 2018 at 5:53 PM, Peter Rosin wrote:
>> Because it looks neater.
>
>> drivers/i2c/algos/i2c-algo-pcf.c | 5 +
>
> I guess for such trivial no functional change patches you may do them
> one per fol
On 2018-05-10 13:17, Wolfram Sang wrote:
> On Sat, May 05, 2018 at 07:57:10AM -0500, Wenwen Wang wrote:
>> In i2c_smbus_xfer_emulated(), there are two buffers: msgbuf0 and msgbuf1,
>> which are used to save a series of messages, as mentioned in the comment.
>> According to the value of the variable
On 2018-05-14 18:28, Daniel Vetter wrote:
> On Fri, May 11, 2018 at 09:37:47AM +0200, Peter Rosin wrote:
>> On 2018-05-10 10:10, Andrzej Hajda wrote:
>>> On 04.05.2018 15:52, Peter Rosin wrote:
>>>> If the bridge supplier is unbound, this will bring the bridge con
, instead going from 5 to 8, we are now
going from 5 to 3.
Peter Rosin (10):
tpm/tpm_i2c_infineon: switch to i2c_lock_bus(..., I2C_LOCK_SEGMENT)
i2c: mux: pca9541: switch to i2c_lock_bus(..., I2C_LOCK_SEGMENT)
input: rohm_bu21023: switch to i2c_lock_bus(..., I2C_LOCK_SEGMENT)
media: af
Calling the __i2c_smbus_xfer wrapper in __i2c_mux_smbus_xfer provides
retries and thus makes the parent-locked case consistent with the both
mux-locked (i2c_mux_smbus_xfer) and the I2C transfer cases.
Signed-off-by: Peter Rosin
---
drivers/i2c/i2c-mux.c | 4 ++--
1 file changed, 2 insertions
to last argument (size) of
the i2c_smbus_xfer declaration to protocol and remove the surplus extern
marker.
Signed-off-by: Peter Rosin
---
drivers/i2c/i2c-core-smbus.c | 28
include/linux/i2c.h | 11 ---
2 files changed, 28 insertions(+), 11 deletions
This simplifies the code, and you get retries for free if the adapter
does not support ->master_xfer.
Signed-off-by: Peter Rosin
---
drivers/i2c/muxes/i2c-mux-pca9541.c | 69 ++---
1 file changed, 11 insertions(+), 58 deletions(-)
diff --git a/drivers/i2c/mu
This simplifies the code, and you get retries for free if the adapter
does not support ->master_xfer.
Signed-off-by: Peter Rosin
---
drivers/i2c/muxes/i2c-mux-pca954x.c | 27 ---
1 file changed, 4 insertions(+), 23 deletions(-)
diff --git a/drivers/i2c/muxes/i2c-
This simplifies the code, and you get retries for free if the adapter
does not support ->master_xfer.
Signed-off-by: Peter Rosin
---
drivers/i2c/muxes/i2c-mux-mlxcpld.c | 28
1 file changed, 4 insertions(+), 24 deletions(-)
diff --git a/drivers/i2c/muxes/i2c-
from Akinobu Mita could also benefit, as indicated by Wolfram [1].
Cheers,
Peter
[1] https://patchwork.ozlabs.org/patch/928386/#1936016
Peter Rosin (5):
i2c: smbus: add unlocked __i2c_smbus_xfer variant
i2c: mux: mlxcpld: make use of __i2c_smbus_xfer
i2c: mux: pca9541: make use of
If DMA safe memory was allocated, but the subsequent I2C transfer
fails the memory is leaked. Plug this leak.
Fixes: 8a77821e74d6 ("i2c: smbus: use DMA safe buffers for emulated SMBus
transactions")
Signed-off-by: Peter Rosin
---
drivers/i2c/i2c-core-smbus.c | 14 +-
1 fi
Kees Cook wrote:
> This is a stab at providing three new helpers for allocation size
> calculation:
>
> struct_size(), array_size(), and array3_size().
>
> These are implemented on top of Rasmus's overflow checking functions. The
> existing allocators are adjusted to use the more efficient overfl
Locking the root adapter for __i2c_transfer will deadlock if the
device sits behind a mux-locked I2C mux. Switch to the finer-grained
i2c_lock_segment. If the device does not sit behind a mux-locked mux,
the two locking variants are equivalent.
Signed-off-by: Peter Rosin
---
drivers/mfd
vice_get_match_data(dev);
> +if (!e) e = e1;
>
> @@
> identifier r.i;
> @@
> - const struct of_device_id *i;
> ... when != i
> //
>
> Signed-off-by: Julia Lawall
Reviewed-by: Peter Rosin
Cheers,
Peter
>
> ---
> drivers/iio/potentiometer/max
Drop the boilerplate license text.
Signed-off-by: Peter Rosin
---
drivers/iio/multiplexer/iio-mux.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/iio/multiplexer/iio-mux.c
b/drivers/iio/multiplexer/iio-mux.c
index e1f44cecdef4..0422ef57914c 100644
--- a
The file is GPL v2 only.
Signed-off-by: Peter Rosin
---
drivers/iio/potentiometer/mcp4018.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/iio/potentiometer/mcp4018.c
b/drivers/iio/potentiometer/mcp4018.c
index 320a7c929777..5ab74a839982 100644
--- a/drivers/iio
Drop the boilerplate license text.
Signed-off-by: Peter Rosin
---
drivers/iio/dac/dpot-dac.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/iio/dac/dpot-dac.c b/drivers/iio/dac/dpot-dac.c
index aaa2103d7c2b..a791d0a09d3b 100644
--- a/drivers/iio/dac/dpot-dac.c
Drop the boilerplate license text.
Signed-off-by: Peter Rosin
---
drivers/iio/adc/envelope-detector.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/iio/adc/envelope-detector.c
b/drivers/iio/adc/envelope-detector.c
index 4ebda8ab54fe..2f2b563c1162 100644
--- a
Drop the boilerplate license text and use the correct MODULE_LICENSE.
Signed-off-by: Peter Rosin
---
drivers/iio/potentiometer/mcp4531.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/drivers/iio/potentiometer/mcp4531.c
b/drivers/iio/potentiometer/mcp4531.c
index
Hi!
I found a couple of discrepancies in the license annotations in
some of the files I have added to the drivers/iio/ directory.
This cleans up the trivialities that I found. While at it, convert
the remaining IIO files that are "mine" to SPDX license identifiers.
Cheers,
Peter
Pete
The file is GPL v2 or later.
Signed-off-by: Peter Rosin
---
drivers/mux/adgs1408.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Hi Mircea!
I just noticed a discrepancy in the license annotations in your
driver. The SPDX license identifier says GPL-2.0-or-later, and
I assume that is
Convert to // comments in the leading comment, drop the boilerplate
license text and use the correct MODULE_LICENSE.
Signed-off-by: Peter Rosin
---
sound/soc/atmel/tse850-pcm5142.c | 78 +++-
1 file changed, 36 insertions(+), 42 deletions(-)
diff --git a
The file is GPL v2 only.
Signed-off-by: Peter Rosin
---
drivers/mux/adgs1408.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
On 2018-08-20 12:16, Caprioru, Mircea wrote:
> Peter Rosin wrote:
>> I just noticed a discrepancy in the license annotations in your
>> driver. T
need to stir the bowl...
Cheers,
Peter
>
> Regards,
> Mircea
>
> -Original Message-
> From: Peter Rosin [mailto:p...@axentia.se]
> Sent: Monday, August 20, 2018 1:38 PM
> To: linux-kernel@vger.kernel.org
> Cc: Peter Rosin ; Caprioru, Mircea
>
> Subject
On 2018-12-13 21:19, Fabrizio Castro wrote:
> Document RZ/G2E (R8A774C0) SoC bindings.
>
> Signed-off-by: Fabrizio Castro
> ---
> Documentation/devicetree/bindings/i2c/i2c-sh_mobile.txt | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/
On 2018-12-17 10:12, Krzysztof Hałasa wrote:
> 90ad2cbe88c22d0215225ab9594eeead0eb24fde changed the i.MX I2C bus driver
This reference should ideally be in a fixes-tag, below...
> to use a notifier whenever the base clock ("ipg" - 66 MHz peripheral
> clock) rate changes.
>
> Unfortunately one ca
This chip is compatible with "lvds-encoder".
Signed-off-by: Peter Rosin
---
Documentation/devicetree/bindings/display/bridge/lvds-transmitter.txt | 1 +
1 file changed, 1 insertion(+)
diff --git
a/Documentation/devicetree/bindings/display/bridge/lvds-transmitter.txt
b/Doc
Hi!
These patches are useful for me. Please consider merging.
Cheers,
Peter
Peter Rosin (3):
dt-bindings: display: bridge: lvds-transmitter: add ti,ds90c187
dt-bindings: display: bridge: lvds-transmitter: add pwdn-gpios
drm/bridge: add pwdn-gpios support to the lvds-encoder
.../bindings
Optionally power down the LVDS-encoder when it is not in use.
Signed-off-by: Peter Rosin
---
drivers/gpu/drm/bridge/lvds-encoder.c | 34 ++
1 file changed, 34 insertions(+)
diff --git a/drivers/gpu/drm/bridge/lvds-encoder.c
b/drivers/gpu/drm/bridge/lvds
Add optional property to specify a power-down GPIO.
The pwdn-gpios name is already in use by the thine,thc63lvdm83d
binding, so go with that.
Signed-off-by: Peter Rosin
---
Documentation/devicetree/bindings/display/bridge/lvds-transmitter.txt | 3 +++
1 file changed, 3 insertions(+)
diff --git
On 2018-12-19 10:12, Andrzej Hajda wrote:
> On 19.12.2018 00:19, Peter Rosin wrote:
>> Add optional property to specify a power-down GPIO.
>> The pwdn-gpios name is already in use by the thine,thc63lvdm83d
>> binding, so go with that.
>>
>> Signed-off-by: Pete
On 2018-12-19 12:38, Laurent Pinchart wrote:
> Hello,
>
> On Wednesday, 19 December 2018 11:57:32 EET Peter Rosin wrote:
>> On 2018-12-19 10:12, Andrzej Hajda wrote:
>>> On 19.12.2018 00:19, Peter Rosin wrote:
>>>> Add optional property to specify a power-do
From: Peter Rosin
DS90C185 has a shutdown pin which does not fit in the lvds-transmitter
binding, which is meant to be generic.
The sister chip DS90C187 is similar to DS90C185, describe it here as well.
Signed-off-by: Peter Rosin
---
.../bindings/display/bridge/lvds-transmitter.txt | 8
From: Peter Rosin
Drop #address-cells and #size-cells from the root node in the
example, they are unused.
Signed-off-by: Peter Rosin
---
Documentation/devicetree/bindings/display/bridge/lvds-transmitter.txt | 2 --
1 file changed, 2 deletions(-)
diff --git
a/Documentation/devicetree
From: Peter Rosin
Optionally power down the LVDS-encoder when it is not in use.
Signed-off-by: Peter Rosin
---
drivers/gpu/drm/bridge/lvds-encoder.c | 34 ++
1 file changed, 34 insertions(+)
diff --git a/drivers/gpu/drm/bridge/lvds-encoder.c
b/drivers/gpu/drm
From: Peter Rosin
Hi!
I'm not sure if I should have added the texas chips to the lvds_encoder_match
list in the driver, right next to the thine,thc63lvdm83d entry, but ended
up not doing that. That can always be added later, it needed...
Changes since v1:
- fork out the bindings for the
On 2018-12-10 22:03, Wolfram Sang wrote:
> Because the adapter will be set up before every transaction anyhow, we
> just need to mark it as adapted to the I2C core.
mark it as "suspended", not "adapted"?
Cheers,
Peter
>
> Signed-off-by: Hiromitsu Yamasaki
> Signed-off-by: Wolfram Sang
> ---
>
On 2018-12-10 22:02, Wolfram Sang wrote:
> Some drivers open code handling of suspended adapters. It should be
> handled by the core, though, to ensure generic handling. This patch adds
> the flag and an accessor function.
>
> Signed-off-by: Wolfram Sang
> ---
> drivers/i2c/i2c-core-base.c | 1 +
On 2018-12-21 15:50, Wolfram Sang wrote:
>
I think this might be as simple as adding:
if (WARN_ON(adap->dev.parent->power.is_suspended))
return -ESHUTDOWN;
>
> Peter, I think this should work for muxes, too, or? The i2c_transfer()
> call to the mux will not be re
On 2019-01-06 10:33, Geert Uytterhoeven wrote:
> Hi Peter,
>
> On Mon, Nov 26, 2018 at 10:59 PM Peter Rosin wrote:
>> If there are extra logos (CONFIG_FB_LOGO_EXTRA) the heights of these
>> extra logos are not considered when centering the first logo vertically.
>>
&g
On 2019-01-07 09:40, Geert Uytterhoeven wrote:
> Hi Peter,
>
> On Mon, Jan 7, 2019 at 9:26 AM Peter Rosin wrote:
>> On 2019-01-06 10:33, Geert Uytterhoeven wrote:
>>> On Mon, Nov 26, 2018 at 10:59 PM Peter Rosin wrote:
>>>> If there are extra logos (CONFIG
On 2019-01-07 09:59, Peter Rosin wrote:
> On 2019-01-07 09:40, Geert Uytterhoeven wrote:
>> Hi Peter,
>>
>> On Mon, Jan 7, 2019 at 9:26 AM Peter Rosin wrote:
>>> On 2019-01-06 10:33, Geert Uytterhoeven wrote:
>>>> On Mon, Nov 26, 2018 at 10:59 PM Peter Ro
On 2019-01-07 10:11, Geert Uytterhoeven wrote:
> Hi Peter,
>
> On Mon, Jan 7, 2019 at 10:03 AM Peter Rosin wrote:
>> On 2019-01-07 09:59, Peter Rosin wrote:
>>> On 2019-01-07 09:40, Geert Uytterhoeven wrote:
>>>> On Mon, Jan 7, 2019 at 9:26 AM Peter Rosin wro
sages, but preserve the current behaviour that also happen to
make the plane disappear with the above call.
Signed-off-by: Peter Rosin
---
drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 14 ++
1 file changed, 10 insertions(+), 4 deletions(-)
Side note, when comp
On 2019-01-02 22:48, Sowjanya Komatineni wrote:
> Tegra194 supports maximum 64K Bytes transfer per packet.
> Tegra186 and prior supports maximum 4K Bytes transfer per packet.
>
> This patch fixes this payload difference between Tegra194 and prior
> tegra chipsets.
>
> Signed-off-by: Sowjanya Koma
On 2019-01-02 23:39, Sowjanya Komatineni wrote:
>>> +static const struct i2c_adapter_quirks tegra194_i2c_quirks = {
>>> + .flags = I2C_AQ_NO_ZERO_LEN,
>>
>> This is not mentioned in the commit message and the flag is not present in
>> the old struct tegra_i2c_quirks above. Are you trying to snea
On 2018-12-27 22:27, Rob Herring wrote:
> On Wed, Dec 19, 2018 at 02:04:47PM +0100, Peter Rosin wrote:
>> From: Peter Rosin
>>
>> DS90C185 has a shutdown pin which does not fit in the lvds-transmitter
>> binding, which is meant to be generic.
>>
>> The sist
le in order
to avoid clutter in the generic lvds-transmitter binding.
- added a patch to remove some surplus stuff in the generic lvds-transmitter
binding.
Cheers,
Peter
Peter Rosin (5):
dt-bindings: display: bridge: fork out ti,ds90c185 from
lvds-transmitter
dt-bindings: display: bridge:
Drop #address-cells and #size-cells from the root node in the
example, they are unused.
Reviewed-by: Rob Herring
Signed-off-by: Peter Rosin
---
Documentation/devicetree/bindings/display/bridge/lvds-transmitter.txt | 2 --
1 file changed, 2 deletions(-)
diff --git
a/Documentation/devicetree
Optionally power down the LVDS-encoder when it is not in use.
Signed-off-by: Peter Rosin
---
drivers/gpu/drm/bridge/lvds-encoder.c | 34 ++
1 file changed, 34 insertions(+)
diff --git a/drivers/gpu/drm/bridge/lvds-encoder.c
b/drivers/gpu/drm/bridge/lvds
DS90C185 has a shutdown pin which does not fit in the lvds-transmitter
binding, which is meant to be generic.
The sister chip DS90C187 is similar to DS90C185, describe it here as well.
Signed-off-by: Peter Rosin
---
.../bindings/display/bridge/lvds-transmitter.txt | 8 +---
.../bindings
Make the code easier to read and modify.
Signed-off-by: Peter Rosin
---
drivers/gpu/drm/bridge/lvds-encoder.c | 19 +--
1 file changed, 9 insertions(+), 10 deletions(-)
diff --git a/drivers/gpu/drm/bridge/lvds-encoder.c
b/drivers/gpu/drm/bridge/lvds-encoder.c
index
.
Signed-off-by: Peter Rosin
---
Documentation/devicetree/bindings/display/bridge/thine,thc63lvdm83d.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/Documentation/devicetree/bindings/display/bridge/thine,thc63lvdm83d.txt
b/Documentation/devicetree/bindings/display/bridge
On 2018-10-08 19:35, Gustavo A. R. Silva wrote:
> In preparation to enabling -Wimplicit-fallthrough, mark switch cases
> where we are expecting to fall through.
The way I see it, it is pretty well marked up as is. So, this paragraph
is not describing the change.
>
> Notice that in this particula
On 2018-08-20 12:11, Peter Rosin wrote:
> The file is GPL v2 or later.
>
> Signed-off-by: Peter Rosin
Applied to the for-next branch.
Cheers,
Peter
On 2018-10-01 13:41, Peter Rosin wrote:
> Just drop the "linux" part of the path, it was never correct.
>
> Reported-by: Joe Perches
> Fixes: 256ac0375098 ("dt-bindings: document devicetree bindings for
> mux-controllers and gpio-mux")
> Signed-off-by:
h use.
So, the patch needs some reworking. Sorry for not noticing this
earlier.
Cheers,
Peter
> Cc: Peter Rosin
> Signed-off-by: Luca Ceresoli
>
> ---
>
> Changes v2 -> v3:
> - fix coding style of comment when moving it (Peter Rosin)
> - move the 'algo' m
On 2018-10-05 13:43, Wolfram Sang wrote:
> On Wed, Oct 03, 2018 at 05:50:22PM +0200, Luca Ceresoli wrote:
>> From: Luca Ceresoli
>>
>> struct i2c_client has a direct pointer to the adapter, no need to dig
>> it out of the struct device tree.
>>
>> Signed-off-by: Luca Ceresoli
>
> I like such cle
On 2018-10-16 13:01, Gustavo A. R. Silva wrote:
> On 10/14/18 7:01 PM, Jonathan Cameron wrote:
>> Done the first of the above...
>
> Thank you, Jonathan.
Indeed. I meant to respond earlier, but then forgot... Thank you!
Cheers,
Peter
Hi Greg,
A couple of tiny fixlets for the mux subsystem. Please pick them up
for 4.20.
Cheers,
Peter
Peter Rosin (2):
mux: adgs1408: use the correct MODULE_LICENSE
MAINTAINERS: use the correct location for dt-bindings includes for mux
MAINTAINERS| 2 +-
drivers/mux/adgs1408.c
The file is GPL v2 or later.
Acked-by: Mircea Caprioru
Signed-off-by: Peter Rosin
---
drivers/mux/adgs1408.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mux/adgs1408.c b/drivers/mux/adgs1408.c
index 0f7cf54e3234..89096f10f4c4 100644
--- a/drivers/mux/adgs1408.c
Just drop the "linux" part of the path, it was never correct.
Reported-by: Joe Perches
Fixes: 256ac0375098 ("dt-bindings: document devicetree bindings for
mux-controllers and gpio-mux")
Signed-off-by: Peter Rosin
---
MAINTAINERS | 2 +-
1 file changed, 1 insertion(+),
On 2018-10-13 14:38, Jonathan Cameron wrote:
> On Mon, 8 Oct 2018 20:42:41 +
> Peter Rosin wrote:
>
>> On 2018-10-08 19:35, Gustavo A. R. Silva wrote:
>>> In preparation to enabling -Wimplicit-fallthrough, mark switch cases
>>> where we are expecting to fall
On 2018-11-02 13:45, Yangtao Li wrote:
> Fix multiple instances of a misspelled "Calculates".
Yes, it sure is hard to spell "Calcluated," it's almost impossible to
get it completely right and not introduce another typo somewhere... :-)
Cheers,
Peter
>
> Signed-off-by: Yangtao Li
> Reviewed-by:
On 2018-09-15 12:52, YueHaibing wrote:
> Drop call to of_match_device, which is subsumed by the subsequent
> call to of_device_get_match_data. The code becomes simpler, and a
> temporary variable can be dropped.
>
> Found by coccinelle.
>
> Signed-off-by: YueHaibing
On 2018-09-15 12:54, YueHaibing wrote:
> Drop call to of_match_device, which is subsumed by the subsequent
> call to of_device_get_match_data. The code becomes simpler, and a
> temporary variable can be dropped.
>
> Found by coccinelle.
>
> Signed-off-by: YueHaibing
On September 16, 2018 11:36:57 AM GMT+02:00, Jonathan Cameron
wrote:
>On Sat, 15 Sep 2018 13:35:06 +0200
>Peter Rosin wrote:
>
>> On 2018-09-15 12:52, YueHaibing wrote:
>> > Drop call to of_match_device, which is subsumed by the subsequent
>> > call to of
On 2018-07-17 15:42, Mircea Caprioru wrote:
> This patch adds basic support for Analog Device ADGS1408/09 SPI mux
> controller.
>
> The device is probed and set to a disabled state. It uses the new mux
> controller framework.
>
> Signed-off-by: Mircea Caprioru
> ---
> Changelog V1 -> V2
> - remo
From: Peter Rosin
Signed-off-by: Peter Rosin
---
Documentation/devicetree/bindings/sound/pcm512x.txt |3 ++-
sound/soc/codecs/pcm512x-i2c.c |4
sound/soc/codecs/pcm512x-spi.c |4
3 files changed, 10 insertions(+), 1 deletion
From: Peter Rosin
When the codec is connected using i2c, it will only auto-increment
register addresses if msb (0x80) of the register address byte is set.
Signed-off-by: Peter Rosin
---
sound/soc/codecs/pcm512x-i2c.c |7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a
Sylvain Rochet wrote:
> Hello Nicolas,
>
> On Mon, Jan 26, 2015 at 02:34:38PM +0100, Nicolas Ferre wrote:
> > Le 26/01/2015 11:36, Sylvain Rochet a écrit :
> > >
> > > I think we should explain we are dealing with an errata here, this
> > > is not obvious at first sight, the patch summary may find
From: Peter Rosin
Signed-off-by: Peter Rosin
---
sound/soc/codecs/pcm512x.c |2 +-
sound/soc/codecs/pcm512x.h |6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/sound/soc/codecs/pcm512x.c b/sound/soc/codecs/pcm512x.c
index 30c673cdc12e..355a8543c8b1 100644
--- a
From: Peter Rosin
Use register field names from the seemingly compatible PCM5242 datasheet,
as the PCM512x and PCM514x datasheets are severly lacking.
Signed-off-by: Peter Rosin
---
sound/soc/codecs/pcm512x.c | 452 ++--
sound/soc/codecs/pcm512x.h
From: Peter Rosin
Using the PLL in master mode requires using an external connection
between one of the GPIO pins (configured as PLL/4 output) and the
SCK pin. It also requires the external clock to be fed to some other
GPIO pin instead of the SCK pin.
This is described for the PCM5122 chip in
From: Peter Rosin
Signed-off-by: Peter Rosin
---
sound/soc/codecs/pcm512x.c | 13 +++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/sound/soc/codecs/pcm512x.c b/sound/soc/codecs/pcm512x.c
index 2a50c58a2fb1..511d98b3afa4 100644
--- a/sound/soc/codecs/pcm512x.c
From: Peter Rosin
Hi!
This series implements BCLK master modes for the pcm512x driver. It has
only been tested with the pcm5142 chip, but they are from the same family
and should be compatible. I have mainly used the spec for the newer
pcm5242 chip (also from the same family) as it fills in a
From: Peter Rosin
Add helper functions to allow drivers to specify several disjoint
ranges for a variable. In particular, there is a codec (PCM512x) that
has a hole in its supported range of rates, due to PLL and divider
restrictions.
Signed-off-by: Peter Rosin
---
include/sound/pcm.h | 12
301 - 400 of 1862 matches
Mail list logo