<div dir="ltr">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.<div><br></div><div>checkpatch is also asked about updating MAINTAINERS, but looks like it should be covered under existing line:</div><div>F:<span class="Apple-tab-span" style="white-space:pre">        </span>drivers/*/*aspeed*</div><div>Does it just ask this every time you add a file?</div><div><br></div><div>changelog:</div><div> - Squashed driver commits as suggested by Joel</div><div> - Renamed a couple functions for consistency</div><div> - Fixed a couple style errors</div></div><br><div class="gmail_quote"><div dir="ltr">On Sun, Aug 28, 2016 at 10:35 PM Joel Stanley <<a href="mailto:joel@jms.id.au">joel@jms.id.au</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Brendan,<br>
<br>
On Sun, Aug 21, 2016 at 11:04 AM, Brendan Higgins<br>
<<a href="mailto:brendanhiggins@google.com" target="_blank">brendanhiggins@google.com</a>> wrote:<br>
> Rewrote most of master IRQ handler, xfer functions, and error recovery.<br>
> Minor refactor of probing functions. Added remove functions.<br>
<br>
Thanks for the patch. The diff is quite noisy given you're deleting<br>
lots of cruft from the vendor driver. It would be easier for us to<br>
review if you send the patch that adds the driver from scratch.<br>
<br>
One suggestion I did have is to rename the driver to use the aspeed_<br>
prefix for all functions so that it matches the drivers that have<br>
already gone upstream.<br>
<br>
Given our goal is immediate upstream submission, how about you send a<br>
complete patch that adds the driver to upstream (say 4.8-rc4?)? Send<br>
it to the i2c lists and cc myself, <a href="mailto:jk@ozlabs.org" target="_blank">jk@ozlabs.org</a>, and <a href="mailto:andrew@aj.id.au" target="_blank">andrew@aj.id.au</a>.<br>
<br>
I hope we can get this sent out for upstream review this week. Let me<br>
know if there's anything I can do to help achieve that.<br>
<br>
Cheers,<br>
<br>
Joel<br>
<br>
> ---<br>
>  drivers/i2c/busses/i2c-aspeed.c | 932 ++++++++++++++--------------------------<br>
>  1 file changed, 318 insertions(+), 614 deletions(-)<br>
</blockquote></div>