Re: [U-Boot] [PATCH] fpga: Replace char * with const char * for filename
On 20. 02. 19 13:03, Chee, Tien Fong wrote: > On Mon, 2019-02-18 at 13:27 +0100, Michal Simek wrote: >> On 15. 02. 19 8:57, tien.fong.c...@intel.com wrote: >>> >>> From: Tien Fong Chee >>> >>> Ensure the string for filename is always constant, otherwise it can >>> be >>> corrupted by the writing. >> Have you reach any issue with it? > Just to resolve the compiler warning about missing of const. ok. Thanks, Michal ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] fpga: Replace char * with const char * for filename
On Mon, 2019-02-18 at 13:27 +0100, Michal Simek wrote: > On 15. 02. 19 8:57, tien.fong.c...@intel.com wrote: > > > > From: Tien Fong Chee > > > > Ensure the string for filename is always constant, otherwise it can > > be > > corrupted by the writing. > Have you reach any issue with it? Just to resolve the compiler warning about missing of const. > > > > > > > Signed-off-by: Tien Fong Chee > > --- > > drivers/fpga/zynqpl.c |3 ++- > > include/fpga.h|2 +- > > 2 files changed, 3 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/fpga/zynqpl.c b/drivers/fpga/zynqpl.c > > index 499310d..683cf14 100644 > > --- a/drivers/fpga/zynqpl.c > > +++ b/drivers/fpga/zynqpl.c > > @@ -421,7 +421,8 @@ static int zynq_loadfs(xilinx_desc *desc, const > > void *buf, size_t bsize, > > loff_t blocksize, actread; > > loff_t pos = 0; > > int fstype; > > - char *interface, *dev_part, *filename; > > + char *interface, *dev_part; > > + const char *filename; > > > > blocksize = fsinfo->blocksize; > > interface = fsinfo->interface; > > diff --git a/include/fpga.h b/include/fpga.h > > index 195f0bd..51de5c5 100644 > > --- a/include/fpga.h > > +++ b/include/fpga.h > > @@ -41,7 +41,7 @@ typedef struct {/* typedef > > fpga_desc */ > > unsigned int blocksize; > > char *interface; > > char *dev_part; > > - char *filename; > > + const char *filename; > > int fstype; > > } fpga_fs_info; > > > > > Anyway looks good applied. > > Thanks, > Michal ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] fpga: Replace char * with const char * for filename
On 15. 02. 19 8:57, tien.fong.c...@intel.com wrote: > From: Tien Fong Chee > > Ensure the string for filename is always constant, otherwise it can be > corrupted by the writing. Have you reach any issue with it? > > Signed-off-by: Tien Fong Chee > --- > drivers/fpga/zynqpl.c |3 ++- > include/fpga.h|2 +- > 2 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/fpga/zynqpl.c b/drivers/fpga/zynqpl.c > index 499310d..683cf14 100644 > --- a/drivers/fpga/zynqpl.c > +++ b/drivers/fpga/zynqpl.c > @@ -421,7 +421,8 @@ static int zynq_loadfs(xilinx_desc *desc, const void > *buf, size_t bsize, > loff_t blocksize, actread; > loff_t pos = 0; > int fstype; > - char *interface, *dev_part, *filename; > + char *interface, *dev_part; > + const char *filename; > > blocksize = fsinfo->blocksize; > interface = fsinfo->interface; > diff --git a/include/fpga.h b/include/fpga.h > index 195f0bd..51de5c5 100644 > --- a/include/fpga.h > +++ b/include/fpga.h > @@ -41,7 +41,7 @@ typedef struct {/* typedef fpga_desc */ > unsigned int blocksize; > char *interface; > char *dev_part; > - char *filename; > + const char *filename; > int fstype; > } fpga_fs_info; > > Anyway looks good applied. Thanks, Michal ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] [PATCH] fpga: Replace char * with const char * for filename
From: Tien Fong Chee Ensure the string for filename is always constant, otherwise it can be corrupted by the writing. Signed-off-by: Tien Fong Chee --- drivers/fpga/zynqpl.c |3 ++- include/fpga.h|2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/fpga/zynqpl.c b/drivers/fpga/zynqpl.c index 499310d..683cf14 100644 --- a/drivers/fpga/zynqpl.c +++ b/drivers/fpga/zynqpl.c @@ -421,7 +421,8 @@ static int zynq_loadfs(xilinx_desc *desc, const void *buf, size_t bsize, loff_t blocksize, actread; loff_t pos = 0; int fstype; - char *interface, *dev_part, *filename; + char *interface, *dev_part; + const char *filename; blocksize = fsinfo->blocksize; interface = fsinfo->interface; diff --git a/include/fpga.h b/include/fpga.h index 195f0bd..51de5c5 100644 --- a/include/fpga.h +++ b/include/fpga.h @@ -41,7 +41,7 @@ typedef struct {/* typedef fpga_desc */ unsigned int blocksize; char *interface; char *dev_part; - char *filename; + const char *filename; int fstype; } fpga_fs_info; -- 1.7.7.4 ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot