Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1694166 > unrolled thread

Re: [PATCHv4 3/4] dt-bindings: mfd: da9052: support TSI as ADC

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCHv4 3/4] dt-bindings: mfd: da9052: support TSI as ADC Guenter Roeck <linux@roeck-us.net> - 2017-07-22 17:10 +0200
    Re: [PATCHv4 3/4] dt-bindings: mfd: da9052: support TSI as ADC Sebastian Reichel <sebastian.reichel@collabora.co.uk> - 2017-07-22 21:10 +0200

#1694166 — Re: [PATCHv4 3/4] dt-bindings: mfd: da9052: support TSI as ADC

On 07/17/2017 04:27 AM, Sebastian Reichel wrote:
> Hi Lee,
> 
> On Mon, Jul 17, 2017 at 11:49:45AM +0100, Lee Jones wrote:
>> On Fri, 30 Jun 2017, Sebastian Reichel wrote:
>>
>>> DA9052 allows using the touchscreen input pins as general purpose
>>> analogue input pin by wiring analogue inputs to X+, X-, Y+ and Y-
>>> and providing a reference voltage at TSIREF pin.
>>>
>>> Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.co.uk>
>>> ---
>>>   Documentation/devicetree/bindings/mfd/da9052-i2c.txt | 8 ++++++++
>>>   1 file changed, 8 insertions(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/mfd/da9052-i2c.txt b/Documentation/devicetree/bindings/mfd/da9052-i2c.txt
>>> index 9554292dc6cb..57fa74e65664 100644
>>> --- a/Documentation/devicetree/bindings/mfd/da9052-i2c.txt
>>> +++ b/Documentation/devicetree/bindings/mfd/da9052-i2c.txt
>>> @@ -4,6 +4,14 @@ Required properties:
>>>   - compatible : Should be "dlg,da9052", "dlg,da9053-aa",
>>>   			 "dlg,da9053-ab", or "dlg,da9053-bb"
>>>   
>>> +Optional properties:
>>> +- dlg,tsi-as-adc : Boolean, if set the X+, X-, Y+, Y- touchscreen
>>> +                    input lines are used as general purpose analogue
>>> +					input.
>>> +- tsiref-supply: Phandle to the regulator, which provides the reference
>>> +                 voltage for the TSIREF pin. Must be provided when the
>>> +			     touchscreen pins are used for ADC purposes.
>>> +
>>
>> DT Ack please.
> 
> There is a v5 of this series, which has Acked-by from Rob already
> and got positive feedback from Guenter Roeck for the hwmon parts:
> 
> http://lkml.iu.edu/hypermail/linux/kernel/1707.0/00518.html
> 
> (if you received v4, v5 should also be in your INBOX)
> 

Yes, but you dropped all the Acks in v6. That is supposed to mean that there
was a substantial change to the patch, and that it needs to be re-reviewed.
For the hwmon patches, I ran a diff against v5 and noticed that there was no
change between v5 and v6, yet you dropped my Acks. Do you really expect us
to do all that work on your behalf ?

Sorry if I sound a bit annoyed. That is because I _am_ a bit annoyed.
Please keep in mind that you are taking our time.

Thanks,
Guenter

[toc] | [next] | [standalone]

#1694191

[Multipart message — attachments visible in raw view] — view raw

Hi,

On Sat, Jul 22, 2017 at 08:09:08AM -0700, Guenter Roeck wrote:
> Yes, but you dropped all the Acks in v6. That is supposed to mean that there
> was a substantial change to the patch, and that it needs to be re-reviewed.
> For the hwmon patches, I ran a diff against v5 and noticed that there was no
> change between v5 and v6, yet you dropped my Acks. Do you really expect us
> to do all that work on your behalf ?
> 
> Sorry if I sound a bit annoyed. That is because I _am_ a bit annoyed.
> Please keep in mind that you are taking our time.

FWIW, I did not drop them, but forgot to add the ones I received in
PATCHv5. Mainly since I initially did not plan to resend the whole
patchset, but then decided otherwise.

Sorry anyways.

-- Sebastian

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web