All of lore.kernel.org
 help / color / mirror / Atom feed
* + mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char.patch added to -mm tree
@ 2009-04-15  0:09 akpm
  2009-04-18  6:28   ` KOSAKI Motohiro
  0 siblings, 1 reply; 7+ messages in thread
From: akpm @ 2009-04-15  0:09 UTC (permalink / raw)
  To: mm-commits; +Cc: den, dwmw2


The patch titled
     mtd: mtd in mtd_release is unused without CONFIG_MTD_CHAR
has been added to the -mm tree.  Its filename is
     mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char.patch

Before you just go and hit "reply", please:
   a) Consider who else should be cc'ed
   b) Prefer to cc a suitable mailing list as well
   c) Ideally: find the original patch on the mailing list and do a
      reply-to-all to that, adding suitable additional cc's

*** Remember to use Documentation/SubmitChecklist when testing your code ***

See http://userweb.kernel.org/~akpm/stuff/added-to-mm.txt to find
out what to do about this

The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/

------------------------------------------------------
Subject: mtd: mtd in mtd_release is unused without CONFIG_MTD_CHAR
From: "Denis V. Lunev" <den@openvz.org>

drivers/mtd/mtdcore.c: In function 'mtd_release':
drivers/mtd/mtdcore.c:51: warning: unused variable 'mtd'

Signed-off-by: Denis V. Lunev <den@openvz.org>
Cc: David Woodhouse <dwmw2@infradead.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---

 drivers/mtd/mtdcore.c |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff -puN drivers/mtd/mtdcore.c~mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char drivers/mtd/mtdcore.c
--- a/drivers/mtd/mtdcore.c~mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char
+++ a/drivers/mtd/mtdcore.c
@@ -48,11 +48,11 @@ static LIST_HEAD(mtd_notifiers);
  */
 static void mtd_release(struct device *dev)
 {
-	struct mtd_info *mtd = dev_to_mtd(dev);
+	dev_t index = MTD_DEVT(dev_to_mtd(dev));
 
 	/* remove /dev/mtdXro node if needed */
-	if (MTD_DEVT(mtd->index))
-		device_destroy(mtd_class, MTD_DEVT(mtd->index) + 1);
+	if (index)
+		device_destroy(mtd_class, index + 1);
 }
 
 static ssize_t mtd_type_show(struct device *dev,
_

Patches currently in -mm which might be from den@openvz.org are

pxafb-lcsr1-is-unused-without-config_fb_pxa_overlay.patch
pxafb-lcsr1-is-unused-without-config_fb_pxa_overlay-fix.patch
arm-compilation-fix-sys_oabi-compatc.patch
mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char.patch


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: + mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char.patch added to -mm tree
  2009-04-15  0:09 + mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char.patch added to -mm tree akpm
@ 2009-04-18  6:28   ` KOSAKI Motohiro
  0 siblings, 0 replies; 7+ messages in thread
From: KOSAKI Motohiro @ 2009-04-18  6:28 UTC (permalink / raw)
  To: linux-kernel, den, dwmw2, linux-mm, Andrew Morton; +Cc: kosaki.motohiro

> Signed-off-by: Denis V. Lunev <den@openvz.org>
> Cc: David Woodhouse <dwmw2@infradead.org>
> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
> ---
> 
>  drivers/mtd/mtdcore.c |    6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff -puN drivers/mtd/mtdcore.c~mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char drivers/mtd/mtdcore.c
> --- a/drivers/mtd/mtdcore.c~mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char
> +++ a/drivers/mtd/mtdcore.c
> @@ -48,11 +48,11 @@ static LIST_HEAD(mtd_notifiers);
>   */
>  static void mtd_release(struct device *dev)
>  {
> -	struct mtd_info *mtd = dev_to_mtd(dev);
> +	dev_t index = MTD_DEVT(dev_to_mtd(dev));
>  
>  	/* remove /dev/mtdXro node if needed */
> -	if (MTD_DEVT(mtd->index))
> -		device_destroy(mtd_class, MTD_DEVT(mtd->index) + 1);
> +	if (index)
> +		device_destroy(mtd_class, index + 1);
>  }

I get compile failure problem on mmotm-0414.

=====================
mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char.patch remove one
warnig if CONFIG_MTD_CHAR=n.

but it introduce one compile error if CONFIG_MTD_CHAR=y/m.

    drivers/mtd/mtdcore.c: In function ‘mtd_release’:
    drivers/mtd/mtdcore.c:51: error: invalid operands to binary * (have ‘struct mtd_info *’ and ‘int’)



Signed-off-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Cc: Denis V. Lunev <den@openvz.org>
Cc: David Woodhouse <dwmw2@infradead.org>
---
 drivers/mtd/mtdcore.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Index: b/drivers/mtd/mtdcore.c
===================================================================
--- a/drivers/mtd/mtdcore.c	2009-04-16 21:43:18.000000000 +0900
+++ b/drivers/mtd/mtdcore.c	2009-04-16 21:44:23.000000000 +0900
@@ -48,7 +48,7 @@ static LIST_HEAD(mtd_notifiers);
  */
 static void mtd_release(struct device *dev)
 {
-	dev_t index = MTD_DEVT(dev_to_mtd(dev));
+	dev_t index = MTD_DEVT(dev_to_mtd(dev)->index);
 
 	/* remove /dev/mtdXro node if needed */
 	if (index)




^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: + mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char.patch added to -mm tree
@ 2009-04-18  6:28   ` KOSAKI Motohiro
  0 siblings, 0 replies; 7+ messages in thread
From: KOSAKI Motohiro @ 2009-04-18  6:28 UTC (permalink / raw)
  To: linux-kernel, den, dwmw2, linux-mm, Andrew Morton; +Cc: kosaki.motohiro

> Signed-off-by: Denis V. Lunev <den@openvz.org>
> Cc: David Woodhouse <dwmw2@infradead.org>
> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
> ---
> 
>  drivers/mtd/mtdcore.c |    6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff -puN drivers/mtd/mtdcore.c~mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char drivers/mtd/mtdcore.c
> --- a/drivers/mtd/mtdcore.c~mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char
> +++ a/drivers/mtd/mtdcore.c
> @@ -48,11 +48,11 @@ static LIST_HEAD(mtd_notifiers);
>   */
>  static void mtd_release(struct device *dev)
>  {
> -	struct mtd_info *mtd = dev_to_mtd(dev);
> +	dev_t index = MTD_DEVT(dev_to_mtd(dev));
>  
>  	/* remove /dev/mtdXro node if needed */
> -	if (MTD_DEVT(mtd->index))
> -		device_destroy(mtd_class, MTD_DEVT(mtd->index) + 1);
> +	if (index)
> +		device_destroy(mtd_class, index + 1);
>  }

I get compile failure problem on mmotm-0414.

=====================
mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char.patch remove one
warnig if CONFIG_MTD_CHAR=n.

but it introduce one compile error if CONFIG_MTD_CHAR=y/m.

    drivers/mtd/mtdcore.c: In function ‘mtd_release’:
    drivers/mtd/mtdcore.c:51: error: invalid operands to binary * (have ‘struct mtd_info *’ and ‘int’)



Signed-off-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Cc: Denis V. Lunev <den@openvz.org>
Cc: David Woodhouse <dwmw2@infradead.org>
---
 drivers/mtd/mtdcore.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Index: b/drivers/mtd/mtdcore.c
===================================================================
--- a/drivers/mtd/mtdcore.c	2009-04-16 21:43:18.000000000 +0900
+++ b/drivers/mtd/mtdcore.c	2009-04-16 21:44:23.000000000 +0900
@@ -48,7 +48,7 @@ static LIST_HEAD(mtd_notifiers);
  */
 static void mtd_release(struct device *dev)
 {
-	dev_t index = MTD_DEVT(dev_to_mtd(dev));
+	dev_t index = MTD_DEVT(dev_to_mtd(dev)->index);
 
 	/* remove /dev/mtdXro node if needed */
 	if (index)



--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: + mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char.patch added to -mm tree
  2009-04-18  6:28   ` KOSAKI Motohiro
@ 2009-04-18 15:56     ` Denis V. Lunev
  -1 siblings, 0 replies; 7+ messages in thread
From: Denis V. Lunev @ 2009-04-18 15:56 UTC (permalink / raw)
  To: KOSAKI Motohiro; +Cc: linux-kernel, dwmw2, linux-mm, Andrew Morton

already fixed by Andrew by 
  mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char-fix

> ------------------------------------------------------
> Subject: mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char-fix
> From: Andrew Morton <akpm@linux-foundation.org>
> 
> Cc: David Woodhouse <dwmw2@infradead.org>
> Cc: Denis V. Lunev <den@openvz.org>
> Cc: Randy Dunlap <randy.dunlap@oracle.com>
> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
> ---
> 
>  drivers/mtd/mtdcore.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff -puN
drivers/mtd/mtdcore.c~mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char-fix drivers/mtd/mtdcore.c
> ---
a/drivers/mtd/mtdcore.c~mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char-fix
> +++ a/drivers/mtd/mtdcore.c
> @@ -48,7 +48,7 @@ static LIST_HEAD(mtd_notifiers);
>   */
>  static void mtd_release(struct device *dev)
>  {
> -     dev_t index = MTD_DEVT(dev_to_mtd(dev));
> +     dev_t index = MTD_DEVT(dev_to_mtd(dev)->index);
>  
>       /* remove /dev/mtdXro node if needed */
>       if (index)


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: + mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char.patch added to -mm tree
@ 2009-04-18 15:56     ` Denis V. Lunev
  0 siblings, 0 replies; 7+ messages in thread
From: Denis V. Lunev @ 2009-04-18 15:56 UTC (permalink / raw)
  To: KOSAKI Motohiro; +Cc: linux-kernel, dwmw2, linux-mm, Andrew Morton

already fixed by Andrew by 
  mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char-fix

> ------------------------------------------------------
> Subject: mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char-fix
> From: Andrew Morton <akpm@linux-foundation.org>
> 
> Cc: David Woodhouse <dwmw2@infradead.org>
> Cc: Denis V. Lunev <den@openvz.org>
> Cc: Randy Dunlap <randy.dunlap@oracle.com>
> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
> ---
> 
>  drivers/mtd/mtdcore.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff -puN
drivers/mtd/mtdcore.c~mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char-fix drivers/mtd/mtdcore.c
> ---
a/drivers/mtd/mtdcore.c~mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char-fix
> +++ a/drivers/mtd/mtdcore.c
> @@ -48,7 +48,7 @@ static LIST_HEAD(mtd_notifiers);
>   */
>  static void mtd_release(struct device *dev)
>  {
> -     dev_t index = MTD_DEVT(dev_to_mtd(dev));
> +     dev_t index = MTD_DEVT(dev_to_mtd(dev)->index);
>  
>       /* remove /dev/mtdXro node if needed */
>       if (index)

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: + mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char.patch added to -mm tree
  2009-04-18 15:56     ` Denis V. Lunev
@ 2009-04-19 12:39       ` KOSAKI Motohiro
  -1 siblings, 0 replies; 7+ messages in thread
From: KOSAKI Motohiro @ 2009-04-19 12:39 UTC (permalink / raw)
  To: Denis V. Lunev
  Cc: kosaki.motohiro, linux-kernel, dwmw2, linux-mm, Andrew Morton

> already fixed by Andrew by 
>   mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char-fix

Oh, thanks.

 - kosaki

> 
> > ------------------------------------------------------
> > Subject: mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char-fix
> > From: Andrew Morton <akpm@linux-foundation.org>
> > 
> > Cc: David Woodhouse <dwmw2@infradead.org>
> > Cc: Denis V. Lunev <den@openvz.org>
> > Cc: Randy Dunlap <randy.dunlap@oracle.com>
> > Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
> > ---
> > 
> >  drivers/mtd/mtdcore.c |    2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff -puN
> drivers/mtd/mtdcore.c~mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char-fix drivers/mtd/mtdcore.c
> > ---
> a/drivers/mtd/mtdcore.c~mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char-fix
> > +++ a/drivers/mtd/mtdcore.c
> > @@ -48,7 +48,7 @@ static LIST_HEAD(mtd_notifiers);
> >   */
> >  static void mtd_release(struct device *dev)
> >  {
> > -     dev_t index = MTD_DEVT(dev_to_mtd(dev));
> > +     dev_t index = MTD_DEVT(dev_to_mtd(dev)->index);
> >  
> >       /* remove /dev/mtdXro node if needed */
> >       if (index)
> 




^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: + mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char.patch added to -mm tree
@ 2009-04-19 12:39       ` KOSAKI Motohiro
  0 siblings, 0 replies; 7+ messages in thread
From: KOSAKI Motohiro @ 2009-04-19 12:39 UTC (permalink / raw)
  To: Denis V. Lunev
  Cc: kosaki.motohiro, linux-kernel, dwmw2, linux-mm, Andrew Morton

> already fixed by Andrew by 
>   mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char-fix

Oh, thanks.

 - kosaki

> 
> > ------------------------------------------------------
> > Subject: mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char-fix
> > From: Andrew Morton <akpm@linux-foundation.org>
> > 
> > Cc: David Woodhouse <dwmw2@infradead.org>
> > Cc: Denis V. Lunev <den@openvz.org>
> > Cc: Randy Dunlap <randy.dunlap@oracle.com>
> > Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
> > ---
> > 
> >  drivers/mtd/mtdcore.c |    2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff -puN
> drivers/mtd/mtdcore.c~mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char-fix drivers/mtd/mtdcore.c
> > ---
> a/drivers/mtd/mtdcore.c~mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char-fix
> > +++ a/drivers/mtd/mtdcore.c
> > @@ -48,7 +48,7 @@ static LIST_HEAD(mtd_notifiers);
> >   */
> >  static void mtd_release(struct device *dev)
> >  {
> > -     dev_t index = MTD_DEVT(dev_to_mtd(dev));
> > +     dev_t index = MTD_DEVT(dev_to_mtd(dev)->index);
> >  
> >       /* remove /dev/mtdXro node if needed */
> >       if (index)
> 



--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2009-04-19 12:41 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-04-15  0:09 + mtd-mtd-in-mtd_release-is-unused-without-config_mtd_char.patch added to -mm tree akpm
2009-04-18  6:28 ` KOSAKI Motohiro
2009-04-18  6:28   ` KOSAKI Motohiro
2009-04-18 15:56   ` Denis V. Lunev
2009-04-18 15:56     ` Denis V. Lunev
2009-04-19 12:39     ` KOSAKI Motohiro
2009-04-19 12:39       ` KOSAKI Motohiro

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.