linux-next: Fixes tag needs some work in the phy-next tree

linux-next: Fixes tag needs some work in the phy-next tree

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 441 bytes --]

Hi all,

In commits

  5b679072af07 ("phy: usb: Fix clock imbalance for suspend/resume")
  219ec4e8f7b2 ("phy: usb: Use slow clock for wake enabled suspend")

Fixes tag

  Fixes: ae532b2b7aa5 ("usb: Add "wake on" functionality for newer Synopsis XHCI controllers")

has these problem(s):

  - Subject does not match target commit subject
    Just use
	git log -1 --format='Fixes: %h ("%s")'

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: linux-next: Fixes tag needs some work in the phy-next tree

[Justin Chen]

On Sun, Nov 6, 2022 at 1:12 PM Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> Hi all,
>
> In commits
>
>   5b679072af07 ("phy: usb: Fix clock imbalance for suspend/resume")
>   219ec4e8f7b2 ("phy: usb: Use slow clock for wake enabled suspend")
>
> Fixes tag
>
>   Fixes: ae532b2b7aa5 ("usb: Add "wake on" functionality for newer Synopsis XHCI controllers")
>
> has these problem(s):
>
>   - Subject does not match target commit subject
>     Just use
>         git log -1 --format='Fixes: %h ("%s")'
>
Apologies, I missed the "phy:" in the beginning. Should be this.
Fixes: ae532b2b7aa5 ("phy: usb: Add "wake on" functionality for newer
Synopsis XHCI controllers")

How should I fix this? Can this be fixed in place?

Thanks,
Justin
> --
> Cheers,
> Stephen Rothwell

Re: linux-next: Fixes tag needs some work in the phy-next tree

[Vinod Koul]

On 06-11-22, 17:10, Justin Chen wrote:
> On Sun, Nov 6, 2022 at 1:12 PM Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> >
> > Hi all,
> >
> > In commits
> >
> >   5b679072af07 ("phy: usb: Fix clock imbalance for suspend/resume")
> >   219ec4e8f7b2 ("phy: usb: Use slow clock for wake enabled suspend")
> >
> > Fixes tag
> >
> >   Fixes: ae532b2b7aa5 ("usb: Add "wake on" functionality for newer Synopsis XHCI controllers")
> >
> > has these problem(s):
> >
> >   - Subject does not match target commit subject

Fixed now

> >     Just use
> >         git log -1 --format='Fixes: %h ("%s")'
> >
> Apologies, I missed the "phy:" in the beginning. Should be this.
> Fixes: ae532b2b7aa5 ("phy: usb: Add "wake on" functionality for newer
> Synopsis XHCI controllers")

Use the script above to generate this and not manually
> 
> How should I fix this? Can this be fixed in place?

I have fixed it up

-- 
~Vinod

linux-next: Fixes tag needs some work in the phy-next tree

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 352 bytes --]

Hi all,

In commit

  3ef54ad3004a ("phy: usb: s2 WoL wakeup_count not incremented for USB->Eth devices")

Fixes tag

  Fixes: f1c0db40a3ad ("usb: Add "wake on" functionality")

has these problem(s):

  - Subject does not match target commit subject
    Just use
	git log -1 --format='Fixes: %h ("%s")'

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: linux-next: Fixes tag needs some work in the phy-next tree

[Vinod Koul]

On 07-11-22, 08:13, Stephen Rothwell wrote:
> Hi all,
> 
> In commit
> 
>   3ef54ad3004a ("phy: usb: s2 WoL wakeup_count not incremented for USB->Eth devices")

Thanks for reporting, fixed now

> 
> Fixes tag
> 
>   Fixes: f1c0db40a3ad ("usb: Add "wake on" functionality")
> 
> has these problem(s):
> 
>   - Subject does not match target commit subject
>     Just use
> 	git log -1 --format='Fixes: %h ("%s")'
> 
> -- 
> Cheers,
> Stephen Rothwell



-- 
~Vinod

linux-next: Fixes tag needs some work in the phy-next tree

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 410 bytes --]

Hi all,

In commit

  0b96a3349af1 ("dt-bindings: fix compilation error of the example in intel,lgm-emmc-phy.yaml")

Fixes tag

  Fixes: 5bc999108025 ("dt-bindings: phy: intel-emmc-phy: Add YAML

has these problem(s):

  - Subject has leading but no trailing parentheses
  - Subject has leading but no trailing quotes

Please keep Fixes tags on a single line.

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: linux-next: Fixes tag needs some work in the phy-next tree

[Kishon Vijay Abraham I]

On 28/01/20 1:35 am, Stephen Rothwell wrote:
> Hi all,
> 
> In commit
> 
>   0b96a3349af1 ("dt-bindings: fix compilation error of the example in intel,lgm-emmc-phy.yaml")
> 
> Fixes tag
> 
>   Fixes: 5bc999108025 ("dt-bindings: phy: intel-emmc-phy: Add YAML
> 
> has these problem(s):
> 
>   - Subject has leading but no trailing parentheses
>   - Subject has leading but no trailing quotes
> 
> Please keep Fixes tags on a single line.

Fixed this.

Thanks
Kishon