* [PATCH] drm: logicvc: Fix uninitialized variable in probe @ 2022-06-14 12:08 Dan Carpenter 2022-06-24 14:35 ` Paul Kocialkowski 0 siblings, 1 reply; 7+ messages in thread From: Dan Carpenter @ 2022-06-14 12:08 UTC (permalink / raw) To: Paul Kocialkowski Cc: David Airlie, Daniel Vetter, Maxime Ripard, dri-devel, linux-kernel, kernel-janitors The "regmap" is supposed to be initialized to NULL but it's used without being initialized. Fixes: efeeaefe9be5 ("drm: Add support for the LogiCVC display controller") Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> --- drivers/gpu/drm/logicvc/logicvc_drm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/logicvc/logicvc_drm.c b/drivers/gpu/drm/logicvc/logicvc_drm.c index df1805cf0f95..0b983a33f9ff 100644 --- a/drivers/gpu/drm/logicvc/logicvc_drm.c +++ b/drivers/gpu/drm/logicvc/logicvc_drm.c @@ -298,7 +298,7 @@ static int logicvc_drm_probe(struct platform_device *pdev) struct logicvc_drm *logicvc; struct device *dev = &pdev->dev; struct drm_device *drm_dev; - struct regmap *regmap; + struct regmap *regmap = NULL; struct resource res; void __iomem *base; int irq; -- 2.35.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] drm: logicvc: Fix uninitialized variable in probe 2022-06-14 12:08 [PATCH] drm: logicvc: Fix uninitialized variable in probe Dan Carpenter @ 2022-06-24 14:35 ` Paul Kocialkowski 2022-06-24 14:37 ` Maxime Ripard 2022-06-24 15:25 ` Paul Kocialkowski 0 siblings, 2 replies; 7+ messages in thread From: Paul Kocialkowski @ 2022-06-24 14:35 UTC (permalink / raw) To: Dan Carpenter Cc: David Airlie, Daniel Vetter, Maxime Ripard, dri-devel, linux-kernel, kernel-janitors [-- Attachment #1: Type: text/plain, Size: 1183 bytes --] Hi Dan, On Tue 14 Jun 22, 15:08, Dan Carpenter wrote: > The "regmap" is supposed to be initialized to NULL but it's used > without being initialized. > > Fixes: efeeaefe9be5 ("drm: Add support for the LogiCVC display controller") > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Nice catch, thanks a lot! Acked-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> Cheers, Paul > --- > drivers/gpu/drm/logicvc/logicvc_drm.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/logicvc/logicvc_drm.c b/drivers/gpu/drm/logicvc/logicvc_drm.c > index df1805cf0f95..0b983a33f9ff 100644 > --- a/drivers/gpu/drm/logicvc/logicvc_drm.c > +++ b/drivers/gpu/drm/logicvc/logicvc_drm.c > @@ -298,7 +298,7 @@ static int logicvc_drm_probe(struct platform_device *pdev) > struct logicvc_drm *logicvc; > struct device *dev = &pdev->dev; > struct drm_device *drm_dev; > - struct regmap *regmap; > + struct regmap *regmap = NULL; > struct resource res; > void __iomem *base; > int irq; > -- > 2.35.1 > -- Paul Kocialkowski, Bootlin Embedded Linux and kernel engineering https://bootlin.com [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm: logicvc: Fix uninitialized variable in probe 2022-06-24 14:35 ` Paul Kocialkowski @ 2022-06-24 14:37 ` Maxime Ripard 2022-06-24 14:46 ` Paul Kocialkowski 2022-06-24 15:25 ` Paul Kocialkowski 1 sibling, 1 reply; 7+ messages in thread From: Maxime Ripard @ 2022-06-24 14:37 UTC (permalink / raw) To: Paul Kocialkowski Cc: Dan Carpenter, David Airlie, Daniel Vetter, dri-devel, linux-kernel, kernel-janitors Hi, On Fri, Jun 24, 2022 at 04:35:25PM +0200, Paul Kocialkowski wrote: > On Tue 14 Jun 22, 15:08, Dan Carpenter wrote: > > The "regmap" is supposed to be initialized to NULL but it's used > > without being initialized. > > > > Fixes: efeeaefe9be5 ("drm: Add support for the LogiCVC display controller") > > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> > > Nice catch, thanks a lot! > > Acked-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> Since you have the commit rights to drm-misc, you should apply it Maxime ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm: logicvc: Fix uninitialized variable in probe 2022-06-24 14:37 ` Maxime Ripard @ 2022-06-24 14:46 ` Paul Kocialkowski 2022-06-24 15:06 ` Maxime Ripard 2022-06-27 8:53 ` Maxime Ripard 0 siblings, 2 replies; 7+ messages in thread From: Paul Kocialkowski @ 2022-06-24 14:46 UTC (permalink / raw) To: Maxime Ripard Cc: Dan Carpenter, David Airlie, Daniel Vetter, dri-devel, linux-kernel, kernel-janitors [-- Attachment #1: Type: text/plain, Size: 900 bytes --] Hi, On Fri 24 Jun 22, 16:37, Maxime Ripard wrote: > Hi, > > On Fri, Jun 24, 2022 at 04:35:25PM +0200, Paul Kocialkowski wrote: > > On Tue 14 Jun 22, 15:08, Dan Carpenter wrote: > > > The "regmap" is supposed to be initialized to NULL but it's used > > > without being initialized. > > > > > > Fixes: efeeaefe9be5 ("drm: Add support for the LogiCVC display controller") > > > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> > > > > Nice catch, thanks a lot! > > > > Acked-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> > > Since you have the commit rights to drm-misc, you should apply it Absolutely, I'm on my way to doing that. Do I need to reply to the emails with a message indicating that I merged them or is using the tool sufficient? Thanks, Paul -- Paul Kocialkowski, Bootlin Embedded Linux and kernel engineering https://bootlin.com [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm: logicvc: Fix uninitialized variable in probe 2022-06-24 14:46 ` Paul Kocialkowski @ 2022-06-24 15:06 ` Maxime Ripard 2022-06-27 8:53 ` Maxime Ripard 1 sibling, 0 replies; 7+ messages in thread From: Maxime Ripard @ 2022-06-24 15:06 UTC (permalink / raw) To: Paul Kocialkowski Cc: Dan Carpenter, David Airlie, Daniel Vetter, dri-devel, linux-kernel, kernel-janitors On Fri, Jun 24, 2022 at 04:46:36PM +0200, Paul Kocialkowski wrote: > Hi, > > On Fri 24 Jun 22, 16:37, Maxime Ripard wrote: > > Hi, > > > > On Fri, Jun 24, 2022 at 04:35:25PM +0200, Paul Kocialkowski wrote: > > > On Tue 14 Jun 22, 15:08, Dan Carpenter wrote: > > > > The "regmap" is supposed to be initialized to NULL but it's used > > > > without being initialized. > > > > > > > > Fixes: efeeaefe9be5 ("drm: Add support for the LogiCVC display controller") > > > > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> > > > > > > Nice catch, thanks a lot! > > > > > > Acked-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> > > > > Since you have the commit rights to drm-misc, you should apply it > > Absolutely, I'm on my way to doing that. Giving your Acked-by is confusing then, because it's kind of implied if you apply it. > Do I need to reply to the emails with a message indicating that I > merged them or is using the tool sufficient? Sending an email is better, dim doesn't do that Maxime ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm: logicvc: Fix uninitialized variable in probe 2022-06-24 14:46 ` Paul Kocialkowski 2022-06-24 15:06 ` Maxime Ripard @ 2022-06-27 8:53 ` Maxime Ripard 1 sibling, 0 replies; 7+ messages in thread From: Maxime Ripard @ 2022-06-27 8:53 UTC (permalink / raw) To: Paul Kocialkowski Cc: Dan Carpenter, David Airlie, Daniel Vetter, dri-devel, linux-kernel, kernel-janitors On Fri, Jun 24, 2022 at 04:46:36PM +0200, Paul Kocialkowski wrote: > Hi, > > On Fri 24 Jun 22, 16:37, Maxime Ripard wrote: > > Hi, > > > > On Fri, Jun 24, 2022 at 04:35:25PM +0200, Paul Kocialkowski wrote: > > > On Tue 14 Jun 22, 15:08, Dan Carpenter wrote: > > > > The "regmap" is supposed to be initialized to NULL but it's used > > > > without being initialized. > > > > > > > > Fixes: efeeaefe9be5 ("drm: Add support for the LogiCVC display controller") > > > > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> > > > > > > Nice catch, thanks a lot! > > > > > > Acked-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> > > > > Since you have the commit rights to drm-misc, you should apply it > > Absolutely, I'm on my way to doing that. > > Do I need to reply to the emails with a message indicating that I merged them > or is using the tool sufficient? Going back on this, but it's fairly easy with b4 to generate the messages that you applied them. For example, I'm using: cat mail | b4 am -t --no-cover -o - -P _ | dim apply-branch drm-misc-fixes b4 am will retrieve the last version of the patch from lore (and only the current patch with --no-cover -P _), add the tags (-t) and output it on stdout (-o -) and pass it to dim apply-branch Then, you can do b4 ty -a && git send-email *.thanks && rm *.thanks And it will send a mail for every patch you applied in the current branch Maxime ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm: logicvc: Fix uninitialized variable in probe 2022-06-24 14:35 ` Paul Kocialkowski 2022-06-24 14:37 ` Maxime Ripard @ 2022-06-24 15:25 ` Paul Kocialkowski 1 sibling, 0 replies; 7+ messages in thread From: Paul Kocialkowski @ 2022-06-24 15:25 UTC (permalink / raw) To: Dan Carpenter Cc: David Airlie, Daniel Vetter, Maxime Ripard, dri-devel, linux-kernel, kernel-janitors [-- Attachment #1: Type: text/plain, Size: 1466 bytes --] Hi, On Fri 24 Jun 22, 16:35, Paul Kocialkowski wrote: > Hi Dan, > > On Tue 14 Jun 22, 15:08, Dan Carpenter wrote: > > The "regmap" is supposed to be initialized to NULL but it's used > > without being initialized. > > > > Fixes: efeeaefe9be5 ("drm: Add support for the LogiCVC display controller") > > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> > > Nice catch, thanks a lot! > > Acked-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> Pushed to drm-misc-next, thanks! Paul > Cheers, > > Paul > > > --- > > drivers/gpu/drm/logicvc/logicvc_drm.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/logicvc/logicvc_drm.c b/drivers/gpu/drm/logicvc/logicvc_drm.c > > index df1805cf0f95..0b983a33f9ff 100644 > > --- a/drivers/gpu/drm/logicvc/logicvc_drm.c > > +++ b/drivers/gpu/drm/logicvc/logicvc_drm.c > > @@ -298,7 +298,7 @@ static int logicvc_drm_probe(struct platform_device *pdev) > > struct logicvc_drm *logicvc; > > struct device *dev = &pdev->dev; > > struct drm_device *drm_dev; > > - struct regmap *regmap; > > + struct regmap *regmap = NULL; > > struct resource res; > > void __iomem *base; > > int irq; > > -- > > 2.35.1 > > > > -- > Paul Kocialkowski, Bootlin > Embedded Linux and kernel engineering > https://bootlin.com -- Paul Kocialkowski, Bootlin Embedded Linux and kernel engineering https://bootlin.com [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-06-27 8:53 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-06-14 12:08 [PATCH] drm: logicvc: Fix uninitialized variable in probe Dan Carpenter 2022-06-24 14:35 ` Paul Kocialkowski 2022-06-24 14:37 ` Maxime Ripard 2022-06-24 14:46 ` Paul Kocialkowski 2022-06-24 15:06 ` Maxime Ripard 2022-06-27 8:53 ` Maxime Ripard 2022-06-24 15:25 ` Paul Kocialkowski
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).