| Message ID | 87fu06d91u.wl-kuninori.morimoto.gx@renesas.com (mailing list archive) |
|---|---|
| State | Not Applicable |
| Delegated to: | Geert Uytterhoeven |
| Headers | show |
| Series | convert to SPDX identifiers | expand |
Hi Morimoto-san, On Thursday, 26 July 2018 05:34:42 EEST Kuninori Morimoto wrote: > From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > --- > drivers/media/platform/soc_camera/soc_camera_platform.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) I have second thoughts about this one. Is it worth switching to SPDX as we're in the process of removing soc-camera from the kernel ? If it is, shouldn't you also address the other soc-camera source files ? I would personally prefer not touching soc-camera as it won't be there for much longer. > diff --git a/drivers/media/platform/soc_camera/soc_camera_platform.c > b/drivers/media/platform/soc_camera/soc_camera_platform.c index > ce00e90..6745a6e 100644 > --- a/drivers/media/platform/soc_camera/soc_camera_platform.c > +++ b/drivers/media/platform/soc_camera/soc_camera_platform.c > @@ -1,13 +1,10 @@ > +// SPDX-License-Identifier: GPL-2.0 > /* > * Generic Platform Camera Driver > * > * Copyright (C) 2008 Magnus Damm > * Based on mt9m001 driver, > * Copyright (C) 2008, Guennadi Liakhovetski <kernel@pengutronix.de> > - * > - * 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 <linux/init.h>
Em Thu, 26 Jul 2018 18:10:32 +0300 Laurent Pinchart <laurent.pinchart@ideasonboard.com> escreveu: > Hi Morimoto-san, > > On Thursday, 26 July 2018 05:34:42 EEST Kuninori Morimoto wrote: > > From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > > > > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > > --- > > drivers/media/platform/soc_camera/soc_camera_platform.c | 5 +---- > > 1 file changed, 1 insertion(+), 4 deletions(-) > > I have second thoughts about this one. Is it worth switching to SPDX as we're > in the process of removing soc-camera from the kernel ? If it is, shouldn't > you also address the other soc-camera source files ? I would personally prefer > not touching soc-camera as it won't be there for much longer. I'd say that, if the code there will be converted from soc_camera and will stay at the Kernel, the convert is a good thing to do, as it makes easier for the conversion, as it would mean one less thing to be concerned with. So, I'm inclined to apply this patch series. > > > diff --git a/drivers/media/platform/soc_camera/soc_camera_platform.c > > b/drivers/media/platform/soc_camera/soc_camera_platform.c index > > ce00e90..6745a6e 100644 > > --- a/drivers/media/platform/soc_camera/soc_camera_platform.c > > +++ b/drivers/media/platform/soc_camera/soc_camera_platform.c > > @@ -1,13 +1,10 @@ > > +// SPDX-License-Identifier: GPL-2.0 > > /* > > * Generic Platform Camera Driver > > * > > * Copyright (C) 2008 Magnus Damm > > * Based on mt9m001 driver, > > * Copyright (C) 2008, Guennadi Liakhovetski <kernel@pengutronix.de> > > - * > > - * 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 <linux/init.h> > Thanks, Mauro
Hi Mauro, On Thursday, 26 July 2018 22:18:49 EEST Mauro Carvalho Chehab wrote: > Em Thu, 26 Jul 2018 18:10:32 +0300 Laurent Pinchart escreveu: > > On Thursday, 26 July 2018 05:34:42 EEST Kuninori Morimoto wrote: > > > From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > > > > > > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > > > --- > > > > > > drivers/media/platform/soc_camera/soc_camera_platform.c | 5 +---- > > > 1 file changed, 1 insertion(+), 4 deletions(-) > > > > I have second thoughts about this one. Is it worth switching to SPDX as > > we're in the process of removing soc-camera from the kernel ? If it is, > > shouldn't you also address the other soc-camera source files ? I would > > personally prefer not touching soc-camera as it won't be there for much > > longer. > > I'd say that, if there are code there that will be converted and will stay > at the Kernel, the SPDX patchset is a good thing, as it makes easier for > the conversion, as it would mean one less thing to be concerned with. > > So, I'm inclined to apply this patch series. My comment referred to this patch only, not the whole series. The rest of the series is totally fine, only soc_camera_platform.c doesn't seem worth touching to me as it's going away. As far as I know the only remaining user is sh_mobile_ceu_camera.c, which isn't used by any platform anymore. It's just a matter of dropping it (I think Jacopo has submitted a patch already), and then removing the drivers/media/platform/soc_camera/ directory. I've taken the whole series in my tree and collected acks, so I'll submit a pull request when we will decide what to do with patch 01/11. > > > diff --git a/drivers/media/platform/soc_camera/soc_camera_platform.c > > > b/drivers/media/platform/soc_camera/soc_camera_platform.c index > > > ce00e90..6745a6e 100644 > > > --- a/drivers/media/platform/soc_camera/soc_camera_platform.c > > > +++ b/drivers/media/platform/soc_camera/soc_camera_platform.c > > > @@ -1,13 +1,10 @@ > > > +// SPDX-License-Identifier: GPL-2.0 > > > /* > > > * Generic Platform Camera Driver > > > * > > > * Copyright (C) 2008 Magnus Damm > > > * Based on mt9m001 driver, > > > * Copyright (C) 2008, Guennadi Liakhovetski <kernel@pengutronix.de> > > > - * > > > - * 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 <linux/init.h>
Hi Laurent, Mauro > > > > From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > > > > > > > > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > > > > --- > > > > > > > > drivers/media/platform/soc_camera/soc_camera_platform.c | 5 +---- > > > > 1 file changed, 1 insertion(+), 4 deletions(-) > > > > > > I have second thoughts about this one. Is it worth switching to SPDX as > > > we're in the process of removing soc-camera from the kernel ? If it is, > > > shouldn't you also address the other soc-camera source files ? I would > > > personally prefer not touching soc-camera as it won't be there for much > > > longer. > > > > I'd say that, if there are code there that will be converted and will stay > > at the Kernel, the SPDX patchset is a good thing, as it makes easier for > > the conversion, as it would mean one less thing to be concerned with. > > > > So, I'm inclined to apply this patch series. > > My comment referred to this patch only, not the whole series. The rest of the > series is totally fine, only soc_camera_platform.c doesn't seem worth touching > to me as it's going away. As far as I know the only remaining user is > sh_mobile_ceu_camera.c, which isn't used by any platform anymore. It's just a > matter of dropping it (I think Jacopo has submitted a patch already), and then > removing the drivers/media/platform/soc_camera/ directory. > > I've taken the whole series in my tree and collected acks, so I'll submit a > pull request when we will decide what to do with patch 01/11. I have no special opinion about it. If you think it is not needed, please drop it. soc_camera_platform is my (and Magnus's) driver of memories, almost 10 years ago. It is a littile bit lonely that it will disappear, ut there is no problem. Thanks soc_camera_platform Best regards --- Kuninori Morimoto
Hi Morimoto-san, On Friday, 27 July 2018 03:49:20 EEST Kuninori Morimoto wrote: > Hi Laurent, Mauro > > >>>> From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > >>>> > >>>> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > >>> --- > >>>> > >>>> drivers/media/platform/soc_camera/soc_camera_platform.c | 5 +---- > >>>> 1 file changed, 1 insertion(+), 4 deletions(-) > >>> > >>> I have second thoughts about this one. Is it worth switching to SPDX > >>> as we're in the process of removing soc-camera from the kernel ? If it > >>> is, shouldn't you also address the other soc-camera source files ? I > >>> would personally prefer not touching soc-camera as it won't be there > >>> for much longer. > >> > >> I'd say that, if there are code there that will be converted and will > >> stay at the Kernel, the SPDX patchset is a good thing, as it makes > >> easier for the conversion, as it would mean one less thing to be > >> concerned with. > >> > >> So, I'm inclined to apply this patch series. > > > > My comment referred to this patch only, not the whole series. The rest of > > the series is totally fine, only soc_camera_platform.c doesn't seem worth > > touching to me as it's going away. As far as I know the only remaining > > user is sh_mobile_ceu_camera.c, which isn't used by any platform anymore. > > It's just a matter of dropping it (I think Jacopo has submitted a patch > > already), and then removing the drivers/media/platform/soc_camera/ > > directory. > > > > I've taken the whole series in my tree and collected acks, so I'll submit > > a pull request when we will decide what to do with patch 01/11. > > I have no special opinion about it. > If you think it is not needed, please drop it. > > soc_camera_platform is my (and Magnus's) driver of memories, > almost 10 years ago. > It is a littile bit lonely that it will disappear, > ut there is no problem. > Thanks soc_camera_platform I'll bake a cake with 10 candles before removing it :-)
diff --git a/drivers/media/platform/soc_camera/soc_camera_platform.c b/drivers/media/platform/soc_camera/soc_camera_platform.c index ce00e90..6745a6e 100644 --- a/drivers/media/platform/soc_camera/soc_camera_platform.c +++ b/drivers/media/platform/soc_camera/soc_camera_platform.c @@ -1,13 +1,10 @@ +// SPDX-License-Identifier: GPL-2.0 /* * Generic Platform Camera Driver * * Copyright (C) 2008 Magnus Damm * Based on mt9m001 driver, * Copyright (C) 2008, Guennadi Liakhovetski <kernel@pengutronix.de> - * - * 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 <linux/init.h>