On Thu, Nov 16, 2017 at 07:33:07PM +0100, Thomas Gleixner wrote: > +2. Style: > + > + The SPDX license identifier is added in form of a comment. The comment > + style depends on the file type: > + > + :: > + > + C source: // SPDX-License-Identifier: <SPDX License Expression> > + C header: /* SPDX-License-Identifier: <SPDX License Expression> */ > + ASM: /* SPDX-License-Identifier: <SPDX License Expression> */ > + scripts: # SPDX-License-Identifier: <SPDX License Expression> > + .rst: .. SPDX-License-Identifier: <SPDX License Expression> Minor nit, do you want a tab there on the .rst line or should it be all spaces like the other lines above it to make things line up properly? Otherwise, looks great to me: Reviewed-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html