* [PATCH] sim/microblaze/interp.c: Use long int format instead of int to avoid compiling warnings @ 2014-07-01 0:15 Chen Gang 2014-07-01 0:43 ` Michael Eager 0 siblings, 1 reply; 4+ messages in thread From: Chen Gang @ 2014-07-01 0:15 UTC (permalink / raw) To: eager; +Cc: vapier, cgen, binutils, gdb-patches The related warnings (cross compile microblaze-linux-elf): ../../../binutils-gdb/sim/microblaze/interp.c: In function ‘microblaze_extract_unsigned_integer’: ../../../binutils-gdb/sim/microblaze/interp.c:53:6: warning: format ‘%d’ expects argument of type ‘int’, but argument 2 has type ‘long unsigned int’ [-Wformat] ../../../binutils-gdb/sim/microblaze/interp.c: In function ‘sim_size’: ../../../binutils-gdb/sim/microblaze/interp.c:329:4: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘long unsigned int’ [-Wformat] ../../../binutils-gdb/sim/microblaze/interp.c: In function ‘set_initial_gprs’: ../../../binutils-gdb/sim/microblaze/interp.c:360:7: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘long unsigned int’ [-Wformat] ../../../binutils-gdb/sim/microblaze/interp.c:360:7: warning: format ‘%x’ expects argument of type ‘unsigned int’, but argument 4 has type ‘long unsigned int’ [-Wformat] Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> --- sim/ChangeLog | 5 +++++ sim/microblaze/interp.c | 6 +++--- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/sim/ChangeLog b/sim/ChangeLog index 35d0920..03c244b 100644 --- a/sim/ChangeLog +++ b/sim/ChangeLog @@ -1,3 +1,8 @@ +2014-07-01 Chen Gang <gang.chen.5i5j@gmail.com> + + * sim/microblaze/interp.c: Use long int format instead of int + format to avoid compiling warnings. + 2014-03-12 Nick Clifton <nickc@redhat.com> * MAINTAINERS: Add myself as the maintainer for the MSP430. diff --git a/sim/microblaze/interp.c b/sim/microblaze/interp.c index cc61278..e92a1d0 100644 --- a/sim/microblaze/interp.c +++ b/sim/microblaze/interp.c @@ -50,7 +50,7 @@ microblaze_extract_unsigned_integer (unsigned char *addr, int len) if (len > (int) sizeof (unsigned long)) printf ("That operation is not available on integers of more than " - "%d bytes.", sizeof (unsigned long)); + "%ld bytes.", sizeof (unsigned long)); /* Start at the most significant end of the integer, and work towards the least significant. */ @@ -325,7 +325,7 @@ sim_size (int size) { if (issue_messages) fprintf (stderr, - "Not enough VM for simulation of %d bytes of RAM\n", + "Not enough VM for simulation of %ld bytes of RAM\n", CPU.msize); CPU.msize = 1; @@ -356,7 +356,7 @@ set_initial_gprs () memsize = CPU.msize / (1024 * 1024); if (issue_messages > 1) - fprintf (stderr, "Simulated memory of %d Mbytes (0x0 .. 0x%08x)\n", + fprintf (stderr, "Simulated memory of %ld Mbytes (0x0 .. 0x%08lx)\n", memsize, CPU.msize - 1); /* Clean out the GPRs */ -- 1.9.2.459.g68773ac ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] sim/microblaze/interp.c: Use long int format instead of int to avoid compiling warnings 2014-07-01 0:15 [PATCH] sim/microblaze/interp.c: Use long int format instead of int to avoid compiling warnings Chen Gang @ 2014-07-01 0:43 ` Michael Eager 2014-07-01 1:08 ` Chen Gang 0 siblings, 1 reply; 4+ messages in thread From: Michael Eager @ 2014-07-01 0:43 UTC (permalink / raw) To: Chen Gang; +Cc: vapier, cgen, binutils, gdb-patches On 06/30/14 17:14, Chen Gang wrote: > The related warnings (cross compile microblaze-linux-elf): > > ../../../binutils-gdb/sim/microblaze/interp.c: In function ‘microblaze_extract_unsigned_integer’: > ../../../binutils-gdb/sim/microblaze/interp.c:53:6: warning: format ‘%d’ expects argument of type ‘int’, but argument 2 has type ‘long unsigned int’ [-Wformat] > ../../../binutils-gdb/sim/microblaze/interp.c: In function ‘sim_size’: > ../../../binutils-gdb/sim/microblaze/interp.c:329:4: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘long unsigned int’ [-Wformat] > ../../../binutils-gdb/sim/microblaze/interp.c: In function ‘set_initial_gprs’: > ../../../binutils-gdb/sim/microblaze/interp.c:360:7: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘long unsigned int’ [-Wformat] > ../../../binutils-gdb/sim/microblaze/interp.c:360:7: warning: format ‘%x’ expects argument of type ‘unsigned int’, but argument 4 has type ‘long unsigned int’ [-Wformat] > > > Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> > --- > sim/ChangeLog | 5 +++++ > sim/microblaze/interp.c | 6 +++--- > 2 files changed, 8 insertions(+), 3 deletions(-) > > diff --git a/sim/ChangeLog b/sim/ChangeLog > index 35d0920..03c244b 100644 > --- a/sim/ChangeLog > +++ b/sim/ChangeLog > @@ -1,3 +1,8 @@ > +2014-07-01 Chen Gang <gang.chen.5i5j@gmail.com> > + > + * sim/microblaze/interp.c: Use long int format instead of int > + format to avoid compiling warnings. > + > 2014-03-12 Nick Clifton <nickc@redhat.com> > > * MAINTAINERS: Add myself as the maintainer for the MSP430. Please put your ChangeLog in the email, not in a patch. > diff --git a/sim/microblaze/interp.c b/sim/microblaze/interp.c > index cc61278..e92a1d0 100644 > --- a/sim/microblaze/interp.c > +++ b/sim/microblaze/interp.c > @@ -50,7 +50,7 @@ microblaze_extract_unsigned_integer (unsigned char *addr, int len) > > if (len > (int) sizeof (unsigned long)) > printf ("That operation is not available on integers of more than " > - "%d bytes.", sizeof (unsigned long)); > + "%ld bytes.", sizeof (unsigned long)); > > /* Start at the most significant end of the integer, and work towards > the least significant. */ > @@ -325,7 +325,7 @@ sim_size (int size) > { > if (issue_messages) > fprintf (stderr, > - "Not enough VM for simulation of %d bytes of RAM\n", > + "Not enough VM for simulation of %ld bytes of RAM\n", > CPU.msize); > > CPU.msize = 1; > @@ -356,7 +356,7 @@ set_initial_gprs () > memsize = CPU.msize / (1024 * 1024); > > if (issue_messages > 1) > - fprintf (stderr, "Simulated memory of %d Mbytes (0x0 .. 0x%08x)\n", > + fprintf (stderr, "Simulated memory of %ld Mbytes (0x0 .. 0x%08lx)\n", > memsize, CPU.msize - 1); > > /* Clean out the GPRs */ Commit ba8e7d1e2. -- Michael Eager eager@eagercon.com 1960 Park Blvd., Palo Alto, CA 94306 650-325-8077 ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] sim/microblaze/interp.c: Use long int format instead of int to avoid compiling warnings 2014-07-01 0:43 ` Michael Eager @ 2014-07-01 1:08 ` Chen Gang 2014-07-01 1:11 ` Michael Eager 0 siblings, 1 reply; 4+ messages in thread From: Chen Gang @ 2014-07-01 1:08 UTC (permalink / raw) To: Michael Eager; +Cc: vapier, cgen, binutils, gdb-patches On 07/01/2014 08:43 AM, Michael Eager wrote: > On 06/30/14 17:14, Chen Gang wrote: >> The related warnings (cross compile microblaze-linux-elf): >> >> ../../../binutils-gdb/sim/microblaze/interp.c: In function >> âmicroblaze_extract_unsigned_integerâ: >> ../../../binutils-gdb/sim/microblaze/interp.c:53:6: warning: format >> â%dâ expects argument of type âintâ, but argument 2 has type âlong >> unsigned intâ [-Wformat] >> ../../../binutils-gdb/sim/microblaze/interp.c: In function âsim_sizeâ: >> ../../../binutils-gdb/sim/microblaze/interp.c:329:4: warning: >> format â%dâ expects argument of type âintâ, but argument 3 has type >> âlong unsigned intâ [-Wformat] >> ../../../binutils-gdb/sim/microblaze/interp.c: In function >> âset_initial_gprsâ: >> ../../../binutils-gdb/sim/microblaze/interp.c:360:7: warning: >> format â%dâ expects argument of type âintâ, but argument 3 has type >> âlong unsigned intâ [-Wformat] >> ../../../binutils-gdb/sim/microblaze/interp.c:360:7: warning: >> format â%xâ expects argument of type âunsigned intâ, but argument 4 >> has type âlong unsigned intâ [-Wformat] >> >> >> Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> >> --- >> sim/ChangeLog | 5 +++++ >> sim/microblaze/interp.c | 6 +++--- >> 2 files changed, 8 insertions(+), 3 deletions(-) >> >> diff --git a/sim/ChangeLog b/sim/ChangeLog >> index 35d0920..03c244b 100644 >> --- a/sim/ChangeLog >> +++ b/sim/ChangeLog >> @@ -1,3 +1,8 @@ >> +2014-07-01 Chen Gang <gang.chen.5i5j@gmail.com> >> + >> + * sim/microblaze/interp.c: Use long int format instead of int >> + format to avoid compiling warnings. >> + >> 2014-03-12 Nick Clifton <nickc@redhat.com> >> >> * MAINTAINERS: Add myself as the maintainer for the MSP430. > > Please put your ChangeLog in the email, not in a patch. OK, thanks. I shall notice about it next time. If necessary to send patch v2 for it, please let me know. > >> diff --git a/sim/microblaze/interp.c b/sim/microblaze/interp.c >> index cc61278..e92a1d0 100644 >> --- a/sim/microblaze/interp.c >> +++ b/sim/microblaze/interp.c >> @@ -50,7 +50,7 @@ microblaze_extract_unsigned_integer (unsigned char >> *addr, int len) >> >> if (len > (int) sizeof (unsigned long)) >> printf ("That operation is not available on integers of more than " >> - "%d bytes.", sizeof (unsigned long)); >> + "%ld bytes.", sizeof (unsigned long)); >> >> /* Start at the most significant end of the integer, and work towards >> the least significant. */ >> @@ -325,7 +325,7 @@ sim_size (int size) >> { >> if (issue_messages) >> fprintf (stderr, >> - "Not enough VM for simulation of %d bytes of RAM\n", >> + "Not enough VM for simulation of %ld bytes of RAM\n", >> CPU.msize); >> >> CPU.msize = 1; >> @@ -356,7 +356,7 @@ set_initial_gprs () >> memsize = CPU.msize / (1024 * 1024); >> >> if (issue_messages > 1) >> - fprintf (stderr, "Simulated memory of %d Mbytes (0x0 .. 0x%08x)\n", >> + fprintf (stderr, "Simulated memory of %ld Mbytes (0x0 .. >> 0x%08lx)\n", >> memsize, CPU.msize - 1); >> >> /* Clean out the GPRs */ > > > Commit ba8e7d1e2. Excuse me, I do not quite undersand the meaning. I guess it means: "have applied this patch, and its' commit number is ba8e7d1e2...", If what I guess is incorrect, please let me know, and I shall send patch v2 for it. Thanks. -- Chen Gang Open, share, and attitude like air, water, and life which God blessed ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] sim/microblaze/interp.c: Use long int format instead of int to avoid compiling warnings 2014-07-01 1:08 ` Chen Gang @ 2014-07-01 1:11 ` Michael Eager 0 siblings, 0 replies; 4+ messages in thread From: Michael Eager @ 2014-07-01 1:11 UTC (permalink / raw) To: Chen Gang; +Cc: vapier, cgen, binutils, gdb-patches On 06/30/14 18:07, Chen Gang wrote: > On 07/01/2014 08:43 AM, Michael Eager wrote: >> On 06/30/14 17:14, Chen Gang wrote: >>> The related warnings (cross compile microblaze-linux-elf): >>> >>> ../../../binutils-gdb/sim/microblaze/interp.c: In function >>> ‘microblaze_extract_unsigned_integer’: >>> ../../../binutils-gdb/sim/microblaze/interp.c:53:6: warning: format >>> ‘%d’ expects argument of type ‘int’, but argument 2 has type ‘long >>> unsigned int’ [-Wformat] >>> ../../../binutils-gdb/sim/microblaze/interp.c: In function ‘sim_size’: >>> ../../../binutils-gdb/sim/microblaze/interp.c:329:4: warning: >>> format ‘%d’ expects argument of type ‘int’, but argument 3 has type >>> ‘long unsigned int’ [-Wformat] >>> ../../../binutils-gdb/sim/microblaze/interp.c: In function >>> ‘set_initial_gprs’: >>> ../../../binutils-gdb/sim/microblaze/interp.c:360:7: warning: >>> format ‘%d’ expects argument of type ‘int’, but argument 3 has type >>> ‘long unsigned int’ [-Wformat] >>> ../../../binutils-gdb/sim/microblaze/interp.c:360:7: warning: >>> format ‘%x’ expects argument of type ‘unsigned int’, but argument 4 >>> has type ‘long unsigned int’ [-Wformat] >>> >>> >>> Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> >>> --- >>> sim/ChangeLog | 5 +++++ >>> sim/microblaze/interp.c | 6 +++--- >>> 2 files changed, 8 insertions(+), 3 deletions(-) >>> >>> diff --git a/sim/ChangeLog b/sim/ChangeLog >>> index 35d0920..03c244b 100644 >>> --- a/sim/ChangeLog >>> +++ b/sim/ChangeLog >>> @@ -1,3 +1,8 @@ >>> +2014-07-01 Chen Gang <gang.chen.5i5j@gmail.com> >>> + >>> + * sim/microblaze/interp.c: Use long int format instead of int >>> + format to avoid compiling warnings. >>> + >>> 2014-03-12 Nick Clifton <nickc@redhat.com> >>> >>> * MAINTAINERS: Add myself as the maintainer for the MSP430. >> >> Please put your ChangeLog in the email, not in a patch. > > OK, thanks. I shall notice about it next time. If necessary to send > patch v2 for it, please let me know. > >> >>> diff --git a/sim/microblaze/interp.c b/sim/microblaze/interp.c >>> index cc61278..e92a1d0 100644 >>> --- a/sim/microblaze/interp.c >>> +++ b/sim/microblaze/interp.c >>> @@ -50,7 +50,7 @@ microblaze_extract_unsigned_integer (unsigned char >>> *addr, int len) >>> >>> if (len > (int) sizeof (unsigned long)) >>> printf ("That operation is not available on integers of more than " >>> - "%d bytes.", sizeof (unsigned long)); >>> + "%ld bytes.", sizeof (unsigned long)); >>> >>> /* Start at the most significant end of the integer, and work towards >>> the least significant. */ >>> @@ -325,7 +325,7 @@ sim_size (int size) >>> { >>> if (issue_messages) >>> fprintf (stderr, >>> - "Not enough VM for simulation of %d bytes of RAM\n", >>> + "Not enough VM for simulation of %ld bytes of RAM\n", >>> CPU.msize); >>> >>> CPU.msize = 1; >>> @@ -356,7 +356,7 @@ set_initial_gprs () >>> memsize = CPU.msize / (1024 * 1024); >>> >>> if (issue_messages > 1) >>> - fprintf (stderr, "Simulated memory of %d Mbytes (0x0 .. 0x%08x)\n", >>> + fprintf (stderr, "Simulated memory of %ld Mbytes (0x0 .. >>> 0x%08lx)\n", >>> memsize, CPU.msize - 1); >>> >>> /* Clean out the GPRs */ >> >> >> Commit ba8e7d1e2. > > Excuse me, I do not quite undersand the meaning. I guess it means: > > "have applied this patch, and its' commit number is ba8e7d1e2...", That's correct. The patch has been committed as an obvious fix. -- Michael Eager eager@eagercon.com 1960 Park Blvd., Palo Alto, CA 94306 650-325-8077 ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-07-01 1:11 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2014-07-01 0:15 [PATCH] sim/microblaze/interp.c: Use long int format instead of int to avoid compiling warnings Chen Gang 2014-07-01 0:43 ` Michael Eager 2014-07-01 1:08 ` Chen Gang 2014-07-01 1:11 ` Michael Eager
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 read-only IMAP folder(s) and NNTP newsgroup(s).