[PATCH 2/2] i2c: aspeed: cleanup driver

Joel Stanley joel at jms.id.au
Wed Aug 31 08:38:21 AEST 2016


On Wed, Aug 31, 2016 at 7:54 AM, Brendan Higgins
<brendanhiggins at google.com> wrote:
>>> Did you send a new version?
>
> Yeah, the subject is: [PATCH v2] i2c: aspeed: added driver for Aspeed I2C
> I sent it --in-reply-to=<your previous message ID>, does it not show as a
> child of the thread in your email client?

I can see it in the list archives, but it never arrived in my inbox.
I'm a little surprised given that I'm using Gmail.

> Sorry if changing the commit message title screwed things up for you, but I
> felt that the clean up title was no longer fitting.

Nope, that's fine.

When you send it next with the bindings document, feel free to reset
the clock and drop the v2 or whatever and send it as a fresh patch.

Cheers,

Joel

>
> On Tue, Aug 30, 2016 at 3:15 PM Joel Stanley <joel at jms.id.au> wrote:
>>
>> On Wed, Aug 31, 2016 at 7:14 AM, Brendan Higgins
>> <brendanhiggins at google.com> wrote:
>> > Oh, and one more note: I moved out the eeprom quirk patch from this
>> > patch
>> > set because I think it makes sense as a different submission upstream.
>>
>> That makes sense to me.
>>
>> >
>> > On Tue, Aug 30, 2016 at 11:59 AM Brendan Higgins
>> > <brendanhiggins at google.com>
>> > wrote:
>> >>
>> >> checkpatch seems to be upset about my device tree identifiers being
>> >> undocumented; do we have a coordinated effort for this? I did not even
>> >> see a
>> >> directory for aspeed under arm.
>>
>> As this is a driver, the binding should be added in
>> Documentation/devicetree/bindings/i2c/. Use one of the recently added
>> existing drivers as a template. Add the binding in a separate commit,
>> coming before the addition of the driver. When you send it upstream
>> you will cc the device tree maintainers, who will ack your bindings.
>> The driver maintainer will then merge the two patches into their tree.
>>
>> >>
>> >> checkpatch is also asked about updating MAINTAINERS, but looks like it
>> >> should be covered under existing line:
>> >> F: drivers/*/*aspeed*
>> >> Does it just ask this every time you add a file?
>>
>> Yeah, I think it just complains for the sake of it.
>>
>> >>
>> >> changelog:
>> >>  - Squashed driver commits as suggested by Joel
>> >>  - Renamed a couple functions for consistency
>> >>  - Fixed a couple style errors
>>
>> Did you send a new version?
>>
>> Cheers,
>>
>> Joel


More information about the openbmc mailing list