On Fri, Sep 6, 2024 at 4:07 PM Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> wrote: > > On Fri, Sep 06, 2024 at 03:53:06PM +0200, Łukasz Bartosik wrote: > > On Fri, Sep 6, 2024 at 10:44 AM Tzung-Bi Shih <tzungbi@xxxxxxxxxx> wrote: > > > > > > On Tue, Sep 03, 2024 at 04:30:27PM +0000, Łukasz Bartosik wrote: > > > > From: Pavan Holla <pholla@xxxxxxxxxxxx> > > > > > > > > Define EC_FEATURE_UCSI_PPM to enable usage of the cros_ec_ucsi > > > > driver. Also, add any feature flags that are implemented by the EC > > > > but are missing in the kernel header. > > > > > > > > Signed-off-by: Pavan Holla <pholla@xxxxxxxxxxxx> > > > > > > Same as previous patch, it needs your S-o-b tag at the end. See [1]. > > > > > > [1]: https://www.kernel.org/doc/html/latest/process/submitting-patches.html#when-to-use-acked-by-cc-and-co-developed-by > > > > I have not added any modifications to this patch. I understand that my > > S-o-b tag is not needed in such a case. Is that not correct ? > > If you are forwarding on patches from someone else, yes, you HAVE TO > sign off on it as well saying that you are allowed to do the forwarding. > Thanks for the clarification Greg. I will add S-o-b then. Thanks, Lukasz > thanks, > > greg k-h >