2018-12-18 06:01:35

by Kuninori Morimoto

[permalink] [raw]
Subject: [PATCH][resend] drm: dw-hdmi-i2s: convert to SPDX identifiers


From: Kuninori Morimoto <[email protected]>

This patch updates license to use SPDX-License-Identifier
instead of verbose license text.

Signed-off-by: Kuninori Morimoto <[email protected]>
---
few weeks passed, nothing happen. I re-post this patch again.
I added Andrew on Cc

drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c b/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c
index 8f9c8a6..2228689 100644
--- a/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c
+++ b/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c
@@ -1,12 +1,9 @@
+// SPDX-License-Identifier: GPL-2.0
/*
* dw-hdmi-i2s-audio.c
*
* Copyright (c) 2017 Renesas Solutions Corp.
* Kuninori Morimoto <[email protected]>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License version 2 as
- * published by the Free Software Foundation.
*/
#include <drm/bridge/dw_hdmi.h>

--
2.7.4



2018-12-18 06:49:25

by Laurent Pinchart

[permalink] [raw]
Subject: Re: [PATCH][resend] drm: dw-hdmi-i2s: convert to SPDX identifiers

Hi Morimoto-san,

Thank you for the patch.

On Tuesday, 18 December 2018 08:00:24 EET Kuninori Morimoto wrote:
> From: Kuninori Morimoto <[email protected]>
>
> This patch updates license to use SPDX-License-Identifier
> instead of verbose license text.
>
> Signed-off-by: Kuninori Morimoto <[email protected]>

Reviewed-by: Laurent Pinchart <[email protected]>

> ---
> few weeks passed, nothing happen. I re-post this patch again.
> I added Andrew on Cc

The driver seems to be lacking a maintainer :-S

> drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c | 5 +----
> 1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c
> b/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c index
> 8f9c8a6..2228689 100644
> --- a/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c
> +++ b/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c
> @@ -1,12 +1,9 @@
> +// SPDX-License-Identifier: GPL-2.0
> /*
> * dw-hdmi-i2s-audio.c
> *
> * Copyright (c) 2017 Renesas Solutions Corp.
> * Kuninori Morimoto <[email protected]>
> - *
> - * This program is free software; you can redistribute it and/or modify
> - * it under the terms of the GNU General Public License version 2 as
> - * published by the Free Software Foundation.
> */
> #include <drm/bridge/dw_hdmi.h>


--
Regards,

Laurent Pinchart




2018-12-18 09:38:38

by Daniel Vetter

[permalink] [raw]
Subject: Re: [PATCH][resend] drm: dw-hdmi-i2s: convert to SPDX identifiers

On Tue, Dec 18, 2018 at 7:47 AM Laurent Pinchart
<[email protected]> wrote:
>
> Hi Morimoto-san,
>
> Thank you for the patch.
>
> On Tuesday, 18 December 2018 08:00:24 EET Kuninori Morimoto wrote:
> > From: Kuninori Morimoto <[email protected]>
> >
> > This patch updates license to use SPDX-License-Identifier
> > instead of verbose license text.
> >
> > Signed-off-by: Kuninori Morimoto <[email protected]>
>
> Reviewed-by: Laurent Pinchart <[email protected]>
>
> > ---
> > few weeks passed, nothing happen. I re-post this patch again.
> > I added Andrew on Cc
>
> The driver seems to be lacking a maintainer :-S

bridge drivers all have a fallback maintainer, but none of them are
cc'ed. It's maintained in drm-misc, so you could just push the patch
too :-) Especially since you're listed:

DRM DRIVERS FOR BRIDGE CHIPS
M: Archit Taneja <[email protected]>
M: Andrzej Hajda <[email protected]>
R: Laurent Pinchart <[email protected]>
S: Maintained
T: git git://anongit.freedesktop.org/drm/drm-misc
F: drivers/gpu/drm/bridge/


Cheers, Daniel

>
> > drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c | 5 +----
> > 1 file changed, 1 insertion(+), 4 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c
> > b/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c index
> > 8f9c8a6..2228689 100644
> > --- a/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c
> > +++ b/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c
> > @@ -1,12 +1,9 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > /*
> > * dw-hdmi-i2s-audio.c
> > *
> > * Copyright (c) 2017 Renesas Solutions Corp.
> > * Kuninori Morimoto <[email protected]>
> > - *
> > - * This program is free software; you can redistribute it and/or modify
> > - * it under the terms of the GNU General Public License version 2 as
> > - * published by the Free Software Foundation.
> > */
> > #include <drm/bridge/dw_hdmi.h>
>
>
> --
> Regards,
>
> Laurent Pinchart
>
>
>
> _______________________________________________
> dri-devel mailing list
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/dri-devel



--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch

2018-12-18 09:55:31

by Laurent Pinchart

[permalink] [raw]
Subject: Re: [PATCH][resend] drm: dw-hdmi-i2s: convert to SPDX identifiers

Hi Daniel,

On Tuesday, 18 December 2018 11:37:14 EET Daniel Vetter wrote:
> On Tue, Dec 18, 2018 at 7:47 AM Laurent Pinchart wrote:
> > On Tuesday, 18 December 2018 08:00:24 EET Kuninori Morimoto wrote:
> >> From: Kuninori Morimoto <[email protected]>
> >>
> >> This patch updates license to use SPDX-License-Identifier
> >> instead of verbose license text.
> >>
> >> Signed-off-by: Kuninori Morimoto <[email protected]>
> >
> > Reviewed-by: Laurent Pinchart <[email protected]>
> >
> >> ---
> >> few weeks passed, nothing happen. I re-post this patch again.
> >> I added Andrew on Cc
> >
> > The driver seems to be lacking a maintainer :-S
>
> bridge drivers all have a fallback maintainer, but none of them are
> cc'ed. It's maintained in drm-misc, so you could just push the patch
> too :-) Especially since you're listed:
>
> DRM DRIVERS FOR BRIDGE CHIPS
> M: Archit Taneja <[email protected]>
> M: Andrzej Hajda <[email protected]>
> R: Laurent Pinchart <[email protected]>

Note the R, not the M :-)

> S: Maintained
> T: git git://anongit.freedesktop.org/drm/drm-misc
> F: drivers/gpu/drm/bridge/
>
>
> Cheers, Daniel
>
> >> drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c | 5 +----
> >> 1 file changed, 1 insertion(+), 4 deletions(-)
> >>
> >> diff --git a/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c
> >> b/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c index
> >> 8f9c8a6..2228689 100644
> >> --- a/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c
> >> +++ b/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c
> >> @@ -1,12 +1,9 @@
> >> +// SPDX-License-Identifier: GPL-2.0
> >> /*
> >> * dw-hdmi-i2s-audio.c
> >> *
> >> * Copyright (c) 2017 Renesas Solutions Corp.
> >> * Kuninori Morimoto <[email protected]>
> >> - *
> >> - * This program is free software; you can redistribute it and/or modify
> >> - * it under the terms of the GNU General Public License version 2 as
> >> - * published by the Free Software Foundation.
> >> */
> >>
> >> #include <drm/bridge/dw_hdmi.h>

--
Regards,

Laurent Pinchart




2018-12-19 02:06:34

by Laurent Pinchart

[permalink] [raw]
Subject: Re: [PATCH][resend] drm: dw-hdmi-i2s: convert to SPDX identifiers

Hi Morimoto-san,

On Wednesday, 19 December 2018 02:16:55 EET Kuninori Morimoto wrote:
> Hi
>
> >>>> This patch updates license to use SPDX-License-Identifier
> >>>> instead of verbose license text.
> >>>>
> >>>> Signed-off-by: Kuninori Morimoto <[email protected]>
> >>>
> >>> Reviewed-by: Laurent Pinchart <[email protected]>
> >>>
> >>>> ---
> >>>> few weeks passed, nothing happen. I re-post this patch again.
> >>>> I added Andrew on Cc
> >>>
> >>> The driver seems to be lacking a maintainer :-S
> >>
> >> bridge drivers all have a fallback maintainer, but none of them are
> >> cc'ed. It's maintained in drm-misc, so you could just push the patch
> >> too :-) Especially since you're listed:
> >>
> >> DRM DRIVERS FOR BRIDGE CHIPS
> >> M: Archit Taneja <[email protected]>
> >> M: Andrzej Hajda <[email protected]>
> >> R: Laurent Pinchart <[email protected]>
>
> ??
> I double checked To and Cc.
> Original patch included Archit, Andrzej address,
> but these were suddenly removed on from Re: mail.
> (I added these address again on this mail).
> ... what's happening ??

I've noticed that several mailing lists sometimes drop addresses from Cc: in
the e-mail content (but still forwards mails to those addresses). I haven't
investigated why.

--
Regards,

Laurent Pinchart




2018-12-19 02:11:31

by Kuninori Morimoto

[permalink] [raw]
Subject: Re: [PATCH][resend] drm: dw-hdmi-i2s: convert to SPDX identifiers


Hi Laurent

> > >>>> few weeks passed, nothing happen. I re-post this patch again.
> > >>>> I added Andrew on Cc
> > >>>
> > >>> The driver seems to be lacking a maintainer :-S
> > >>
> > >> bridge drivers all have a fallback maintainer, but none of them are
> > >> cc'ed. It's maintained in drm-misc, so you could just push the patch
> > >> too :-) Especially since you're listed:
> > >>
> > >> DRM DRIVERS FOR BRIDGE CHIPS
> > >> M: Archit Taneja <[email protected]>
> > >> M: Andrzej Hajda <[email protected]>
> > >> R: Laurent Pinchart <[email protected]>
> >
> > ??
> > I double checked To and Cc.
> > Original patch included Archit, Andrzej address,
> > but these were suddenly removed on from Re: mail.
> > (I added these address again on this mail).
> > ... what's happening ??
>
> I've noticed that several mailing lists sometimes drop addresses from Cc: in
> the e-mail content (but still forwards mails to those addresses). I haven't
> investigated why.

Hmm...
I hope this mail is still reaching to Maintainer

Best regards
---
Kuninori Morimoto

2018-12-19 02:48:58

by Kuninori Morimoto

[permalink] [raw]
Subject: Re: [PATCH][resend] drm: dw-hdmi-i2s: convert to SPDX identifiers


Hi

> > >> This patch updates license to use SPDX-License-Identifier
> > >> instead of verbose license text.
> > >>
> > >> Signed-off-by: Kuninori Morimoto <[email protected]>
> > >
> > > Reviewed-by: Laurent Pinchart <[email protected]>
> > >
> > >> ---
> > >> few weeks passed, nothing happen. I re-post this patch again.
> > >> I added Andrew on Cc
> > >
> > > The driver seems to be lacking a maintainer :-S
> >
> > bridge drivers all have a fallback maintainer, but none of them are
> > cc'ed. It's maintained in drm-misc, so you could just push the patch
> > too :-) Especially since you're listed:
> >
> > DRM DRIVERS FOR BRIDGE CHIPS
> > M: Archit Taneja <[email protected]>
> > M: Andrzej Hajda <[email protected]>
> > R: Laurent Pinchart <[email protected]>

??
I double checked To and Cc.
Original patch included Archit, Andrzej address,
but these were suddenly removed on from Re: mail.
(I added these address again on this mail).
... what's happening ??


Best regards
---
Kuninori Morimoto

2018-12-19 10:07:14

by Andrzej Hajda

[permalink] [raw]
Subject: Re: [PATCH][resend] drm: dw-hdmi-i2s: convert to SPDX identifiers

Hi,


On 19.12.2018 01:56, Kuninori Morimoto wrote:
> Hi Laurent
>
>>>>>>> few weeks passed, nothing happen. I re-post this patch again.
>>>>>>> I added Andrew on Cc
>>>>>> The driver seems to be lacking a maintainer :-S
>>>>> bridge drivers all have a fallback maintainer, but none of them are
>>>>> cc'ed. It's maintained in drm-misc, so you could just push the patch
>>>>> too :-) Especially since you're listed:
>>>>>
>>>>> DRM DRIVERS FOR BRIDGE CHIPS
>>>>> M: Archit Taneja <[email protected]>
>>>>> M: Andrzej Hajda <[email protected]>
>>>>> R: Laurent Pinchart <[email protected]>
>>> ??
>>> I double checked To and Cc.
>>> Original patch included Archit, Andrzej address,
>>> but these were suddenly removed on from Re: mail.
>>> (I added these address again on this mail).
>>> ... what's happening ??
>> I've noticed that several mailing lists sometimes drop addresses from Cc: in
>> the e-mail content (but still forwards mails to those addresses). I haven't
>> investigated why.
> Hmm...
> I hope this mail is still reaching to Maintainer


Yes, I have just missed it. Adding drm/bridge in the subject will be
helpful next time.

Queued to drm-misc-next.


Regards

Andrzej


>
> Best regards
> ---
> Kuninori Morimoto
>
>