[PATCH v6 0/5] usb: host: dwc2: use driver model for PHY and CLOCK

[Patrick DELAUNAY]

Hi Marek,

> From: Marek Vasut <marex at denx.de>
> Sent: mardi 10 mars 2020 19:27
> 
> On 3/10/20 6:01 PM, Patrick DELAUNAY wrote:
> > Hi Marek,
> >
> >> From: Marek Vasut <marex at denx.de>
> >> Sent: mardi 10 mars 2020 16:20
> >> Subject: Re: [PATCH v6 0/5] usb: host: dwc2: use driver model for PHY
> >> and CLOCK
> >> Importance: High
> >>
> >> On 3/10/20 11:09 AM, Patrick Delaunay wrote:
> >>>
> >>> In this serie I update the DWC2 host driver to use the device tree
> >>> information and the associated PHY and CLOCK drivers when they are
> >>> availables.
> >>>
> >>> V6 is minor update of the first patch of the serie; I update some
> >>> clk stub to return success after Simon Goldschmidt remarks on V5:
> >>> http://patchwork.ozlabs.org/project/uboot/list/?series=162791
> >>
> >> Once Simon reviews them, I'll put them in usb/next . btw I'm not CCed
> >> on 1/5, would be helpful if you could keep the CC consistent, thanks
> >
> > Ok and thanks
> >
> > I used patman for the serie with default maintainers list + 3 reviewers in Serie-CC
> >     Series-cc: simon.k.r.goldschmidt at gmail.com
> >     Series-cc: b.galvani at gmail.com
> >     Series-cc: ley.foon.tan at intel.com
> >
> > But I forgot you. Sorry.
> 
> I'd expect get-maintainer script should tell you to add me, maybe patman needs
> fixing.

I will try to avoid this issue for my next serie...

FYI: I check and the CC list is build with (series.py::MakeCcFile)

1/ Get maitainer script

It give me for this patch

./scripts/get_maintainer.pl 0001-dm-clk-add-stub-when-CONFIG_CLK-is-deactivated.patch 
Peng Fan <peng.fan at nxp.com> (commit_signer:7/15=47%,authored:2/15=13%)
Lukasz Majewski <lukma at denx.de> (commit_signer:6/15=40%,authored:6/15=40%,added_lines:36/255=14%,removed_lines:1/17=6%)
Jean-Jacques Hiblot <jjhiblot at ti.com> (commit_signer:3/15=20%,authored:3/15=20%,added_lines:53/255=21%,removed_lines:3/17=18%)
Simon Glass <sjg at chromium.org> (commit_signer:3/15=20%)
Lokesh Vutla <lokeshvutla at ti.com> (commit_signer:1/15=7%)
Sekhar Nori <nsekhar at ti.com> (authored:1/15=7%)
Chunfeng Yun <chunfeng.yun at mediatek.com> (authored:1/15=7%,added_lines:40/255=16%)
Patrick Delaunay <patrick.delaunay at st.com> (added_lines:89/255=35%,removed_lines:13/17=76%)
Jagan Teki <jagan at amarulasolutions.com> (added_lines:15/255=6%)
u-boot at lists.denx.de (open list)

PS: same for "./scripts/get_maintainer.pl include/clk.h"

2/ tags (based on aliases of ./doc/git-mailrc)
   "dm" and "clk:"
	=> dm = sjg
	=> clk = unknown tag

   But no tag "usb" => marex

3/ Series-cc

So I don't think it is a not patman issue
and the maintainer list is build patch by patch in the serie...

I perhaps miss something in my configuration for the 'usb' serie.
 
For patman, I use 

$> git config sendemail.to uboot
$> git config sendemail.aliasesfile doc/git-mailrc
....
$> tools/patman/patman -c 5 -t -i

Perhaps I need to use the correct alias = "usb" for USB related topic....
(for all the serie and no more rely on "tags")

$> git config sendemail.to usb

In this case I have :

To: u-boot at lists.denx.de,
	Marek Vasut <marex at denx.de>
Cc: ley.foon.tan at intel.com,
	b.galvani at gmail.com,
	simon.k.r.goldschmidt at gmail.com,
	Patrick Delaunay <patrick.delaunay at st.com>,
	Chunfeng Yun <chunfeng.yun at mediatek.com>,
	Jagan Teki <jagan at amarulasolutions.com>,
	Jean-Jacques Hiblot <jjhiblot at ti.com>,
	Lukasz Majewski <lukma at denx.de>,
	Peng Fan <peng.fan at nxp.com>,
	Sekhar Nori <nsekhar at ti.com>,
	Simon Glass <sjg at chromium.org>,
	U-Boot STM32 <uboot-stm32 at st-md-mailman.stormreply.com>
Subject: [PATCH v6 1/5] dm: clk: add stub when CONFIG_CLK is deactivat

Regards

Patrick