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


Groups > linux.kernel > #1452933

Re: [PATCH v7 2/2] [media] atmel-isc: DT binding for Image Sensor Controller driver

From Rob Herring <robh@kernel.org>
Newsgroups linux.kernel
Subject Re: [PATCH v7 2/2] [media] atmel-isc: DT binding for Image Sensor Controller driver
Date 2016-08-01 04:00 +0200
Message-ID <s1aGl-82S-5@gated-at.bofh.it> (permalink)
References <s0b1M-20c-7@gated-at.bofh.it> <s0b1M-20c-29@gated-at.bofh.it> <s0nPk-1RL-13@gated-at.bofh.it> <s1adk-7Sh-5@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Sun, Jul 31, 2016 at 8:23 PM, Wu, Songjun <Songjun.Wu@microchip.com> wrote:
>
>
> On 7/30/2016 05:44, Rob Herring wrote:
>>
>> On Fri, Jul 29, 2016 at 03:54:08PM +0800, Songjun Wu wrote:
>>>
>>> DT binding documentation for ISC driver.
>>>
>>> Signed-off-by: Songjun Wu <songjun.wu@microchip.com>
>>> ---
>>>
>>> Changes in v7: None
>>> Changes in v6:
>>> - Add "iscck" and "gck" to clock-names.
>>>
>>> Changes in v5:
>>> - Add clock-output-names.
>>>
>>> Changes in v4:
>>> - Remove the isc clock nodes.
>>>
>>> Changes in v3:
>>> - Remove the 'atmel,sensor-preferred'.
>>> - Modify the isc clock node according to the Rob's remarks.
>>>
>>> Changes in v2:
>>> - Remove the unit address of the endpoint.
>>> - Add the unit address to the clock node.
>>> - Avoid using underscores in node names.
>>> - Drop the "0x" in the unit address of the i2c node.
>>> - Modify the description of 'atmel,sensor-preferred'.
>>> - Add the description for the ISC internal clock.
>>>
>>>  .../devicetree/bindings/media/atmel-isc.txt        | 65
>>> ++++++++++++++++++++++
>>>  1 file changed, 65 insertions(+)
>>>  create mode 100644 Documentation/devicetree/bindings/media/atmel-isc.txt
>>
>>
>> Please add acks when posting new versions.
>>
>> Rob
>>
> Hi Rob,
>
> Thank you for your reminder.
>
> Should I Add 'Acked-by: Rob Herring <robh@kernel.org>' behind
> 'Signed-off-by: Songjun Wu <songjun.wu@microchip.com>'?

Before or after is fine.

> Should I resend this patch?

Not just to add acks, but if you send v8, add the acks.

Rob

Back to linux.kernel | Previous | NextPrevious in thread | Find similar | Unroll thread


Thread

[PATCH v7 2/2] [media] atmel-isc: DT binding for Image Sensor Controller driver Songjun Wu <songjun.wu@microchip.com> - 2016-07-29 10:10 +0200
  Re: [PATCH v7 2/2] [media] atmel-isc: DT binding for Image Sensor  Controller driver Rob Herring <robh@kernel.org> - 2016-07-29 23:50 +0200
    Re: [PATCH v7 2/2] [media] atmel-isc: DT binding for Image Sensor  Controller driver "Wu, Songjun" <Songjun.Wu@microchip.com> - 2016-08-01 03:30 +0200
      Re: [PATCH v7 2/2] [media] atmel-isc: DT binding for Image Sensor  Controller driver Rob Herring <robh@kernel.org> - 2016-08-01 04:00 +0200

csiph-web