diff options
| author | Myles Watson <mylesgw@gmail.com> | 2010-05-21 17:15:55 +0000 |
|---|---|---|
| committer | Myles Watson <mylesgw@gmail.com> | 2010-05-21 17:15:55 +0000 |
| commit | 8d96ed433641c44404387f555458fab46b3832ee (patch) | |
| tree | d793404c9bdbcecfc6b7875838af32c9c315277b /src | |
| parent | c25cc11ae32e10a7d0c9f04c29bfb7eca4d9c210 (diff) | |
| download | coreboot-8d96ed433641c44404387f555458fab46b3832ee.tar.xz | |
Get rid of this warning:
src/cpu/amd/model_10xxx/fidvid.c:758:
warning: 'fid_max' may be used uninitialized in this function
Quoting Marc:
It [fid_max] should be initialized to 0. The !nb_cof_vid_update would mean that
the fidmax shouldn't change so the value isn't important, but 0 would be the
safest if there is another hole in the logic and CPUs are not matched.
Signed-off-by: Myles Watson <mylesgw@gmail.com>
Acked-by: Myles Watson <mylesgw@gmail.com>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@5577 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
Diffstat (limited to 'src')
| -rw-r--r-- | src/cpu/amd/model_10xxx/fidvid.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/cpu/amd/model_10xxx/fidvid.c b/src/cpu/amd/model_10xxx/fidvid.c index c155cac8f7..7007d4cf8f 100644 --- a/src/cpu/amd/model_10xxx/fidvid.c +++ b/src/cpu/amd/model_10xxx/fidvid.c @@ -755,7 +755,7 @@ static int init_fidvid_bsp(u32 bsp_apicid, u32 nodes) struct fidvid_st fv; device_t dev; u32 vid_max; - u32 fid_max; + u32 fid_max=0; u8 nb_cof_vid_update; u32 reg1fc; u8 pvimode; |