* [U-Boot] [PATCH 1/2] qe: transform parameter to compatible type @ 2015-05-05 7:53 Zhao Qiang 2015-05-05 7:53 ` [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy Zhao Qiang ` (2 more replies) 0 siblings, 3 replies; 11+ messages in thread From: Zhao Qiang @ 2015-05-05 7:53 UTC (permalink / raw) To: u-boot when using printf, the parameter type need to be compatible type, so transform them to compatible type Signed-off-by: Zhao Qiang <B45475@freescale.com> --- drivers/qe/qe.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/qe/qe.c b/drivers/qe/qe.c index 84e1433..f1df0a4 100644 --- a/drivers/qe/qe.c +++ b/drivers/qe/qe.c @@ -313,9 +313,10 @@ static void qe_upload_microcode(const void *base, if (ucode->major || ucode->minor || ucode->revision) printf("QE: uploading microcode '%s' version %u.%u.%u\n", - ucode->id, ucode->major, ucode->minor, ucode->revision); + (char *)ucode->id, (u16)ucode->major, (u16)ucode->minor, + (u16)ucode->revision); else - printf("QE: uploading microcode '%s'\n", ucode->id); + printf("QE: uploading microcode '%s'\n", (char *)ucode->id); /* Use auto-increment */ out_be32(&qe_immr->iram.iadd, be32_to_cpu(ucode->iram_offset) | -- 2.1.0.27.g96db324 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy 2015-05-05 7:53 [U-Boot] [PATCH 1/2] qe: transform parameter to compatible type Zhao Qiang @ 2015-05-05 7:53 ` Zhao Qiang 2015-05-05 8:32 ` Michael Trimarchi 2015-08-04 15:45 ` York Sun 2015-06-29 18:27 ` [U-Boot] [PATCH 1/2] qe: transform parameter to compatible type York Sun 2015-08-04 15:44 ` York Sun 2 siblings, 2 replies; 11+ messages in thread From: Zhao Qiang @ 2015-05-05 7:53 UTC (permalink / raw) To: u-boot strncpy is safer than strcpy, use it to instead of strcpy. Signed-off-by: Zhao Qiang <B45475@freescale.com> --- drivers/qe/qe.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/qe/qe.c b/drivers/qe/qe.c index f1df0a4..08620b2 100644 --- a/drivers/qe/qe.c +++ b/drivers/qe/qe.c @@ -435,7 +435,7 @@ int qe_upload_firmware(const struct qe_firmware *firmware) * saved microcode information and put in the new. */ memset(&qe_firmware_info, 0, sizeof(qe_firmware_info)); - strcpy(qe_firmware_info.id, (char *)firmware->id); + strncpy(qe_firmware_info.id, (char *)firmware->id, 62); qe_firmware_info.extended_modes = firmware->extended_modes; memcpy(qe_firmware_info.vtraps, firmware->vtraps, sizeof(firmware->vtraps)); -- 2.1.0.27.g96db324 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy 2015-05-05 7:53 ` [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy Zhao Qiang @ 2015-05-05 8:32 ` Michael Trimarchi 2015-05-05 9:15 ` qiang.zhao at freescale.com 2015-08-04 15:45 ` York Sun 1 sibling, 1 reply; 11+ messages in thread From: Michael Trimarchi @ 2015-05-05 8:32 UTC (permalink / raw) To: u-boot Hi On Tue, May 5, 2015 at 3:53 PM, Zhao Qiang <B45475@freescale.com> wrote: > strncpy is safer than strcpy, use it to instead of strcpy. > > Signed-off-by: Zhao Qiang <B45475@freescale.com> > --- > drivers/qe/qe.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/qe/qe.c b/drivers/qe/qe.c > index f1df0a4..08620b2 100644 > --- a/drivers/qe/qe.c > +++ b/drivers/qe/qe.c > @@ -435,7 +435,7 @@ int qe_upload_firmware(const struct qe_firmware *firmware) > * saved microcode information and put in the new. > */ > memset(&qe_firmware_info, 0, sizeof(qe_firmware_info)); > - strcpy(qe_firmware_info.id, (char *)firmware->id); > + strncpy(qe_firmware_info.id, (char *)firmware->id, 62); is 62 somenthing that can be calculated? > qe_firmware_info.extended_modes = firmware->extended_modes; > memcpy(qe_firmware_info.vtraps, firmware->vtraps, > sizeof(firmware->vtraps)); > -- > 2.1.0.27.g96db324 > > _______________________________________________ > U-Boot mailing list > U-Boot at lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot ^ permalink raw reply [flat|nested] 11+ messages in thread
* [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy 2015-05-05 8:32 ` Michael Trimarchi @ 2015-05-05 9:15 ` qiang.zhao at freescale.com 2015-05-05 9:40 ` Michael Trimarchi 0 siblings, 1 reply; 11+ messages in thread From: qiang.zhao at freescale.com @ 2015-05-05 9:15 UTC (permalink / raw) To: u-boot On Tue, May 5, 2015 at 4:33 PM, Michael Trimarchi wrote: > -----Original Message----- > From: Michael Trimarchi [mailto:michael at amarulasolutions.com] > Sent: Tuesday, May 05, 2015 4:33 PM > To: Zhao Qiang-B45475 > Cc: u-boot at lists.denx.de; Sun York-R58495 > Subject: Re: [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy > > Hi > > On Tue, May 5, 2015 at 3:53 PM, Zhao Qiang <B45475@freescale.com> wrote: > > strncpy is safer than strcpy, use it to instead of strcpy. > > > > Signed-off-by: Zhao Qiang <B45475@freescale.com> > > --- > > drivers/qe/qe.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/qe/qe.c b/drivers/qe/qe.c index f1df0a4..08620b2 > > 100644 > > --- a/drivers/qe/qe.c > > +++ b/drivers/qe/qe.c > > @@ -435,7 +435,7 @@ int qe_upload_firmware(const struct qe_firmware > *firmware) > > * saved microcode information and put in the new. > > */ > > memset(&qe_firmware_info, 0, sizeof(qe_firmware_info)); > > - strcpy(qe_firmware_info.id, (char *)firmware->id); > > + strncpy(qe_firmware_info.id, (char *)firmware->id, 62); > > is 62 somenthing that can be calculated? The fireware->id is defined 62 bytes in the struct. > > > qe_firmware_info.extended_modes = firmware->extended_modes; > > memcpy(qe_firmware_info.vtraps, firmware->vtraps, > > sizeof(firmware->vtraps)); > > -- > > 2.1.0.27.g96db324 > > > > _______________________________________________ > > U-Boot mailing list > > U-Boot at lists.denx.de > > http://lists.denx.de/mailman/listinfo/u-boot Best Regards Zhao Qiang ^ permalink raw reply [flat|nested] 11+ messages in thread
* [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy 2015-05-05 9:15 ` qiang.zhao at freescale.com @ 2015-05-05 9:40 ` Michael Trimarchi 2015-05-06 1:40 ` qiang.zhao at freescale.com 0 siblings, 1 reply; 11+ messages in thread From: Michael Trimarchi @ 2015-05-05 9:40 UTC (permalink / raw) To: u-boot Hi On Tue, May 5, 2015 at 5:15 PM, qiang.zhao at freescale.com <qiang.zhao@freescale.com> wrote: > On Tue, May 5, 2015 at 4:33 PM, Michael Trimarchi wrote: >> -----Original Message----- >> From: Michael Trimarchi [mailto:michael at amarulasolutions.com] >> Sent: Tuesday, May 05, 2015 4:33 PM >> To: Zhao Qiang-B45475 >> Cc: u-boot at lists.denx.de; Sun York-R58495 >> Subject: Re: [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy >> >> Hi >> >> On Tue, May 5, 2015 at 3:53 PM, Zhao Qiang <B45475@freescale.com> wrote: >> > strncpy is safer than strcpy, use it to instead of strcpy. >> > >> > Signed-off-by: Zhao Qiang <B45475@freescale.com> >> > --- >> > drivers/qe/qe.c | 2 +- >> > 1 file changed, 1 insertion(+), 1 deletion(-) >> > >> > diff --git a/drivers/qe/qe.c b/drivers/qe/qe.c index f1df0a4..08620b2 >> > 100644 >> > --- a/drivers/qe/qe.c >> > +++ b/drivers/qe/qe.c >> > @@ -435,7 +435,7 @@ int qe_upload_firmware(const struct qe_firmware >> *firmware) >> > * saved microcode information and put in the new. >> > */ >> > memset(&qe_firmware_info, 0, sizeof(qe_firmware_info)); >> > - strcpy(qe_firmware_info.id, (char *)firmware->id); >> > + strncpy(qe_firmware_info.id, (char *)firmware->id, 62); >> >> is 62 somenthing that can be calculated? > > The fireware->id is defined 62 bytes in the struct. > and you don't like sizeof Michael > >> >> > qe_firmware_info.extended_modes = firmware->extended_modes; >> > memcpy(qe_firmware_info.vtraps, firmware->vtraps, >> > sizeof(firmware->vtraps)); >> > -- >> > 2.1.0.27.g96db324 >> > >> > _______________________________________________ >> > U-Boot mailing list >> > U-Boot at lists.denx.de >> > http://lists.denx.de/mailman/listinfo/u-boot > Best Regards > Zhao Qiang -- | Michael Nazzareno Trimarchi Amarula Solutions BV | | COO - Founder Cruquiuskade 47 | | +31(0)851119172 Amsterdam 1018 AM NL | | [`as] http://www.amarulasolutions.com | ^ permalink raw reply [flat|nested] 11+ messages in thread
* [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy 2015-05-05 9:40 ` Michael Trimarchi @ 2015-05-06 1:40 ` qiang.zhao at freescale.com 2015-05-06 5:21 ` Michael Trimarchi 2015-06-29 18:30 ` York Sun 0 siblings, 2 replies; 11+ messages in thread From: qiang.zhao at freescale.com @ 2015-05-06 1:40 UTC (permalink / raw) To: u-boot On Tue, May 5, 2015 at 5:41 PM, Michael Trimarchi wrote: > -----Original Message----- > From: Michael Trimarchi [mailto:michael at amarulasolutions.com] > Sent: Tuesday, May 05, 2015 5:41 PM > To: Zhao Qiang-B45475 > Cc: u-boot at lists.denx.de; Sun York-R58495 > Subject: Re: [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy > > Hi > > On Tue, May 5, 2015 at 5:15 PM, qiang.zhao at freescale.com > <qiang.zhao@freescale.com> wrote: > > On Tue, May 5, 2015 at 4:33 PM, Michael Trimarchi wrote: > >> -----Original Message----- > >> From: Michael Trimarchi [mailto:michael at amarulasolutions.com] > >> Sent: Tuesday, May 05, 2015 4:33 PM > >> To: Zhao Qiang-B45475 > >> Cc: u-boot at lists.denx.de; Sun York-R58495 > >> Subject: Re: [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy > >> > >> Hi > >> > >> On Tue, May 5, 2015 at 3:53 PM, Zhao Qiang <B45475@freescale.com> > wrote: > >> > strncpy is safer than strcpy, use it to instead of strcpy. > >> > > >> > Signed-off-by: Zhao Qiang <B45475@freescale.com> > >> > --- > >> > drivers/qe/qe.c | 2 +- > >> > 1 file changed, 1 insertion(+), 1 deletion(-) > >> > > >> > diff --git a/drivers/qe/qe.c b/drivers/qe/qe.c index > >> > f1df0a4..08620b2 > >> > 100644 > >> > --- a/drivers/qe/qe.c > >> > +++ b/drivers/qe/qe.c > >> > @@ -435,7 +435,7 @@ int qe_upload_firmware(const struct qe_firmware > >> *firmware) > >> > * saved microcode information and put in the new. > >> > */ > >> > memset(&qe_firmware_info, 0, sizeof(qe_firmware_info)); > >> > - strcpy(qe_firmware_info.id, (char *)firmware->id); > >> > + strncpy(qe_firmware_info.id, (char *)firmware->id, 62); > >> > >> is 62 somenthing that can be calculated? > > > > The fireware->id is defined 62 bytes in the struct. > > > > and you don't like sizeof Thank you for your suggestion. If there are something else need to be modify, I will use sizeof instead. > > Michael > > > > > > >> > >> > qe_firmware_info.extended_modes = firmware->extended_modes; > >> > memcpy(qe_firmware_info.vtraps, firmware->vtraps, > >> > sizeof(firmware->vtraps)); > >> > -- > >> > 2.1.0.27.g96db324 > >> > > >> > _______________________________________________ > >> > U-Boot mailing list > >> > U-Boot at lists.denx.de > >> > http://lists.denx.de/mailman/listinfo/u-boot > > Best Regards > > Zhao Qiang > > > > -- > | Michael Nazzareno Trimarchi Amarula Solutions BV | > | COO - Founder Cruquiuskade 47 | > | +31(0)851119172 Amsterdam 1018 AM NL | > | [`as] http://www.amarulasolutions.com | ^ permalink raw reply [flat|nested] 11+ messages in thread
* [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy 2015-05-06 1:40 ` qiang.zhao at freescale.com @ 2015-05-06 5:21 ` Michael Trimarchi 2015-06-29 18:30 ` York Sun 1 sibling, 0 replies; 11+ messages in thread From: Michael Trimarchi @ 2015-05-06 5:21 UTC (permalink / raw) To: u-boot Hi On Wed, May 6, 2015 at 9:40 AM, qiang.zhao at freescale.com <qiang.zhao@freescale.com> wrote: > On Tue, May 5, 2015 at 5:41 PM, Michael Trimarchi wrote: > >> -----Original Message----- >> From: Michael Trimarchi [mailto:michael at amarulasolutions.com] >> Sent: Tuesday, May 05, 2015 5:41 PM >> To: Zhao Qiang-B45475 >> Cc: u-boot at lists.denx.de; Sun York-R58495 >> Subject: Re: [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy >> >> Hi >> >> On Tue, May 5, 2015 at 5:15 PM, qiang.zhao at freescale.com >> <qiang.zhao@freescale.com> wrote: >> > On Tue, May 5, 2015 at 4:33 PM, Michael Trimarchi wrote: >> >> -----Original Message----- >> >> From: Michael Trimarchi [mailto:michael at amarulasolutions.com] >> >> Sent: Tuesday, May 05, 2015 4:33 PM >> >> To: Zhao Qiang-B45475 >> >> Cc: u-boot at lists.denx.de; Sun York-R58495 >> >> Subject: Re: [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy >> >> >> >> Hi >> >> >> >> On Tue, May 5, 2015 at 3:53 PM, Zhao Qiang <B45475@freescale.com> >> wrote: >> >> > strncpy is safer than strcpy, use it to instead of strcpy. >> >> > >> >> > Signed-off-by: Zhao Qiang <B45475@freescale.com> >> >> > --- >> >> > drivers/qe/qe.c | 2 +- >> >> > 1 file changed, 1 insertion(+), 1 deletion(-) >> >> > >> >> > diff --git a/drivers/qe/qe.c b/drivers/qe/qe.c index >> >> > f1df0a4..08620b2 >> >> > 100644 >> >> > --- a/drivers/qe/qe.c >> >> > +++ b/drivers/qe/qe.c >> >> > @@ -435,7 +435,7 @@ int qe_upload_firmware(const struct qe_firmware >> >> *firmware) >> >> > * saved microcode information and put in the new. >> >> > */ >> >> > memset(&qe_firmware_info, 0, sizeof(qe_firmware_info)); >> >> > - strcpy(qe_firmware_info.id, (char *)firmware->id); >> >> > + strncpy(qe_firmware_info.id, (char *)firmware->id, 62); >> >> >> >> is 62 somenthing that can be calculated? >> > >> > The fireware->id is defined 62 bytes in the struct. >> > >> >> and you don't like sizeof > > Thank you for your suggestion. > If there are something else need to be modify, > I will use sizeof instead. > Sorry for boring you ;) Michael >> >> Michael >> >> >> >> > >> >> >> >> > qe_firmware_info.extended_modes = firmware->extended_modes; >> >> > memcpy(qe_firmware_info.vtraps, firmware->vtraps, >> >> > sizeof(firmware->vtraps)); >> >> > -- >> >> > 2.1.0.27.g96db324 >> >> > >> >> > _______________________________________________ >> >> > U-Boot mailing list >> >> > U-Boot at lists.denx.de >> >> > http://lists.denx.de/mailman/listinfo/u-boot >> > Best Regards >> > Zhao Qiang >> >> >> >> -- >> | Michael Nazzareno Trimarchi Amarula Solutions BV | >> | COO - Founder Cruquiuskade 47 | >> | +31(0)851119172 Amsterdam 1018 AM NL | >> | [`as] http://www.amarulasolutions.com | -- | Michael Nazzareno Trimarchi Amarula Solutions BV | | COO - Founder Cruquiuskade 47 | | +31(0)851119172 Amsterdam 1018 AM NL | | [`as] http://www.amarulasolutions.com | ^ permalink raw reply [flat|nested] 11+ messages in thread
* [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy 2015-05-06 1:40 ` qiang.zhao at freescale.com 2015-05-06 5:21 ` Michael Trimarchi @ 2015-06-29 18:30 ` York Sun 1 sibling, 0 replies; 11+ messages in thread From: York Sun @ 2015-06-29 18:30 UTC (permalink / raw) To: u-boot On 05/05/2015 06:40 PM, Zhao Qiang-B45475 wrote: > On Tue, May 5, 2015 at 5:41 PM, Michael Trimarchi wrote: > >> -----Original Message----- >> From: Michael Trimarchi [mailto:michael at amarulasolutions.com] >> Sent: Tuesday, May 05, 2015 5:41 PM >> To: Zhao Qiang-B45475 >> Cc: u-boot at lists.denx.de; Sun York-R58495 >> Subject: Re: [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy >> >> Hi >> >> On Tue, May 5, 2015 at 5:15 PM, qiang.zhao at freescale.com >> <qiang.zhao@freescale.com> wrote: >>> On Tue, May 5, 2015 at 4:33 PM, Michael Trimarchi wrote: >>>> -----Original Message----- >>>> From: Michael Trimarchi [mailto:michael at amarulasolutions.com] >>>> Sent: Tuesday, May 05, 2015 4:33 PM >>>> To: Zhao Qiang-B45475 >>>> Cc: u-boot at lists.denx.de; Sun York-R58495 >>>> Subject: Re: [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy >>>> >>>> Hi >>>> >>>> On Tue, May 5, 2015 at 3:53 PM, Zhao Qiang <B45475@freescale.com> >> wrote: >>>>> strncpy is safer than strcpy, use it to instead of strcpy. >>>>> >>>>> Signed-off-by: Zhao Qiang <B45475@freescale.com> >>>>> --- >>>>> drivers/qe/qe.c | 2 +- >>>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>>> >>>>> diff --git a/drivers/qe/qe.c b/drivers/qe/qe.c index >>>>> f1df0a4..08620b2 >>>>> 100644 >>>>> --- a/drivers/qe/qe.c >>>>> +++ b/drivers/qe/qe.c >>>>> @@ -435,7 +435,7 @@ int qe_upload_firmware(const struct qe_firmware >>>> *firmware) >>>>> * saved microcode information and put in the new. >>>>> */ >>>>> memset(&qe_firmware_info, 0, sizeof(qe_firmware_info)); >>>>> - strcpy(qe_firmware_info.id, (char *)firmware->id); >>>>> + strncpy(qe_firmware_info.id, (char *)firmware->id, 62); >>>> >>>> is 62 somenthing that can be calculated? >>> >>> The fireware->id is defined 62 bytes in the struct. >>> >> >> and you don't like sizeof > > Thank you for your suggestion. > If there are something else need to be modify, > I will use sizeof instead. Looks like the id is a null-terminated string. Do you face the same issue as patch 1/2 in this set? Would it be better to change the type to char? York ^ permalink raw reply [flat|nested] 11+ messages in thread
* [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy 2015-05-05 7:53 ` [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy Zhao Qiang 2015-05-05 8:32 ` Michael Trimarchi @ 2015-08-04 15:45 ` York Sun 1 sibling, 0 replies; 11+ messages in thread From: York Sun @ 2015-08-04 15:45 UTC (permalink / raw) To: u-boot On 05/05/2015 12:53 AM, Zhao Qiang wrote: > strncpy is safer than strcpy, use it to instead of strcpy. > > Signed-off-by: Zhao Qiang <B45475@freescale.com> > --- Applied to u-boot-fsl-qoriq master. Thanks. York ^ permalink raw reply [flat|nested] 11+ messages in thread
* [U-Boot] [PATCH 1/2] qe: transform parameter to compatible type 2015-05-05 7:53 [U-Boot] [PATCH 1/2] qe: transform parameter to compatible type Zhao Qiang 2015-05-05 7:53 ` [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy Zhao Qiang @ 2015-06-29 18:27 ` York Sun 2015-08-04 15:44 ` York Sun 2 siblings, 0 replies; 11+ messages in thread From: York Sun @ 2015-06-29 18:27 UTC (permalink / raw) To: u-boot On 05/05/2015 12:53 AM, Zhao Qiang wrote: > when using printf, the parameter type need to be compatible > type, so transform them to compatible type > > Signed-off-by: Zhao Qiang <B45475@freescale.com> > --- > drivers/qe/qe.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/qe/qe.c b/drivers/qe/qe.c > index 84e1433..f1df0a4 100644 > --- a/drivers/qe/qe.c > +++ b/drivers/qe/qe.c > @@ -313,9 +313,10 @@ static void qe_upload_microcode(const void *base, > > if (ucode->major || ucode->minor || ucode->revision) > printf("QE: uploading microcode '%s' version %u.%u.%u\n", > - ucode->id, ucode->major, ucode->minor, ucode->revision); > + (char *)ucode->id, (u16)ucode->major, (u16)ucode->minor, > + (u16)ucode->revision); > else > - printf("QE: uploading microcode '%s'\n", ucode->id); > + printf("QE: uploading microcode '%s'\n", (char *)ucode->id); > Would it be more appropriate to fix the structure in qe.h? Is the size fixed for major and minor? They are declared as u8. I can't find doc/README.qe_firmware as mentioned in the comment. York ^ permalink raw reply [flat|nested] 11+ messages in thread
* [U-Boot] [PATCH 1/2] qe: transform parameter to compatible type 2015-05-05 7:53 [U-Boot] [PATCH 1/2] qe: transform parameter to compatible type Zhao Qiang 2015-05-05 7:53 ` [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy Zhao Qiang 2015-06-29 18:27 ` [U-Boot] [PATCH 1/2] qe: transform parameter to compatible type York Sun @ 2015-08-04 15:44 ` York Sun 2 siblings, 0 replies; 11+ messages in thread From: York Sun @ 2015-08-04 15:44 UTC (permalink / raw) To: u-boot On 05/05/2015 12:53 AM, Zhao Qiang wrote: > when using printf, the parameter type need to be compatible > type, so transform them to compatible type > > Signed-off-by: Zhao Qiang <B45475@freescale.com> > --- Applied to u-boot-fsl-qoriq master. Thanks. York ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2015-08-04 15:45 UTC | newest] Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2015-05-05 7:53 [U-Boot] [PATCH 1/2] qe: transform parameter to compatible type Zhao Qiang 2015-05-05 7:53 ` [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy Zhao Qiang 2015-05-05 8:32 ` Michael Trimarchi 2015-05-05 9:15 ` qiang.zhao at freescale.com 2015-05-05 9:40 ` Michael Trimarchi 2015-05-06 1:40 ` qiang.zhao at freescale.com 2015-05-06 5:21 ` Michael Trimarchi 2015-06-29 18:30 ` York Sun 2015-08-04 15:45 ` York Sun 2015-06-29 18:27 ` [U-Boot] [PATCH 1/2] qe: transform parameter to compatible type York Sun 2015-08-04 15:44 ` York Sun
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.