| Message ID | 1411992293-7729-7-git-send-email-zhang.lyra@gmail.com |
|---|---|
| State | Needs Review / ACK, archived |
| Headers | show |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | warning | total: 1 errors, 0 warnings, 0 lines checked |
| robh/patch-applied | success |
On Monday 29 September 2014 20:04:53 zhang.lyra@gmail.com wrote: > diff --git a/Documentation/devicetree/bindings/arm/sprd.txt b/Documentation/devicetree/bindings/arm/sprd.txt > new file mode 100644 > index 0000000..b6dc143 > --- /dev/null > +++ b/Documentation/devicetree/bindings/arm/sprd.txt > @@ -0,0 +1,6 @@ > +Spreadtrum Platforms Device Tree Bindings > +---------------------------------------------------- > + > +Shark64 Board > +Required root node properties: > + - compatible = "sprd,shark64"; > Is Shark a board or an SOC? The subject line of the patch and the contents seem to disagree here. It's generally better to have the exact SoC product numbers in here, e.g. "sprd,sc1234", in case we have to apply fixups based on the SoC. You should also have a board specific string in addition to the SoC-specific one. Arnd -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Monday 29 September 2014 20:04:53 zhang.lyra@gmail.com wrote: > From: "chunyan.zhang" <chunyan.zhang@spreadtrum.com> > > Adds a devicetree binding documentation for Spreadtrum's Shark64 SoC. > > Signed-off-by: chunyan.zhang <chunyan.zhang@spreadtrum.com> One more thing: The email addresses look wrong, please use the form in which you normally write the names, such as Signed-off-by: Zhang Chunyan <chunyan.zhang@spreadtrum.com> You can set the user name that git will add using the 'git config --global user.name ...'. Also, I notice that the email from which the patch is sent does not match the last 'Signed-off-by' line. If Chunyan and Lyra are both your names, better pick one that you use for both the email and the signoff. If you are Lyra and are posting a patch from Chunyan, please add your own line below hers, like Signed-off-by: Zhang Chunyan <chunyan.zhang@spreadtrum.com> Signed-off-by: Zhang Lyra <zhang.lyra@gmail.com> so that you include everybody who was involved in sending the patch, but have your own list last. Arnd -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi, Arnd [use plain text pattern to send again] Thank you very much for your review in detail so carefully. I'll address these changes which you mentioned in this patch set in V2, but the days from tomorrow to 10.7 are Chinese holidays, I'm going to send PATCHv2 a few days later. I'm also sorry that make you confused about my name, I'll solve this problem in the next submit. Thanks, Chunyan 2014-09-29 21:59 GMT+08:00 Arnd Bergmann <arnd@arndb.de>: > On Monday 29 September 2014 20:04:53 zhang.lyra@gmail.com wrote: >> From: "chunyan.zhang" <chunyan.zhang@spreadtrum.com> >> >> Adds a devicetree binding documentation for Spreadtrum's Shark64 SoC. >> >> Signed-off-by: chunyan.zhang <chunyan.zhang@spreadtrum.com> > > One more thing: The email addresses look wrong, please use the form in > which you normally write the names, such as > > Signed-off-by: Zhang Chunyan <chunyan.zhang@spreadtrum.com> > > You can set the user name that git will add using the > 'git config --global user.name ...'. > > Also, I notice that the email from which the patch is sent does not > match the last 'Signed-off-by' line. If Chunyan and Lyra are both > your names, better pick one that you use for both the email and the > signoff. If you are Lyra and are posting a patch from Chunyan, please > add your own line below hers, like > > Signed-off-by: Zhang Chunyan <chunyan.zhang@spreadtrum.com> > Signed-off-by: Zhang Lyra <zhang.lyra@gmail.com> > > so that you include everybody who was involved in sending the > patch, but have your own list last. > > Arnd -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/Documentation/devicetree/bindings/arm/sprd.txt b/Documentation/devicetree/bindings/arm/sprd.txt new file mode 100644 index 0000000..b6dc143 --- /dev/null +++ b/Documentation/devicetree/bindings/arm/sprd.txt @@ -0,0 +1,6 @@ +Spreadtrum Platforms Device Tree Bindings +---------------------------------------------------- + +Shark64 Board +Required root node properties: + - compatible = "sprd,shark64";