diff options
author | Jia-Ju Bai <baijiaju1990@gmail.com> | 2018-07-26 23:44:24 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab+samsung@kernel.org> | 2018-07-27 08:15:55 -0400 |
commit | 5bfffa0c86915e5afbfe56d9790457b2a9887f2d (patch) | |
tree | 7e8436afda5102623083b55321659183fe2644ed /drivers/media/pci/ivtv | |
parent | 9d08ba6de0c52d87b1f9619df3f6faacde2134b7 (diff) |
media: pci: ivtv: Replace GFP_ATOMIC with GFP_KERNEL
ivtv_probe() and ivtvfb_init_card() are never called in atomic context.
They call kzalloc() with GFP_ATOMIC, which is not necessary.
GFP_ATOMIC can be replaced with GFP_KERNEL.
This is found by a static analysis tool named DCNS written by myself.
Signed-off-by: Jia-Ju Bai <baijiaju1990@gmail.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
Diffstat (limited to 'drivers/media/pci/ivtv')
-rw-r--r-- | drivers/media/pci/ivtv/ivtv-driver.c | 2 | ||||
-rw-r--r-- | drivers/media/pci/ivtv/ivtvfb.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/drivers/media/pci/ivtv/ivtv-driver.c b/drivers/media/pci/ivtv/ivtv-driver.c index 6b2ffdc96961..dd727098daf4 100644 --- a/drivers/media/pci/ivtv/ivtv-driver.c +++ b/drivers/media/pci/ivtv/ivtv-driver.c | |||
@@ -999,7 +999,7 @@ static int ivtv_probe(struct pci_dev *pdev, const struct pci_device_id *pci_id) | |||
999 | int vbi_buf_size; | 999 | int vbi_buf_size; |
1000 | struct ivtv *itv; | 1000 | struct ivtv *itv; |
1001 | 1001 | ||
1002 | itv = kzalloc(sizeof(struct ivtv), GFP_ATOMIC); | 1002 | itv = kzalloc(sizeof(struct ivtv), GFP_KERNEL); |
1003 | if (itv == NULL) | 1003 | if (itv == NULL) |
1004 | return -ENOMEM; | 1004 | return -ENOMEM; |
1005 | itv->pdev = pdev; | 1005 | itv->pdev = pdev; |
diff --git a/drivers/media/pci/ivtv/ivtvfb.c b/drivers/media/pci/ivtv/ivtvfb.c index b19058e36853..5ddaa8ed11a5 100644 --- a/drivers/media/pci/ivtv/ivtvfb.c +++ b/drivers/media/pci/ivtv/ivtvfb.c | |||
@@ -1178,7 +1178,7 @@ static int ivtvfb_init_card(struct ivtv *itv) | |||
1178 | } | 1178 | } |
1179 | 1179 | ||
1180 | itv->osd_info = kzalloc(sizeof(struct osd_info), | 1180 | itv->osd_info = kzalloc(sizeof(struct osd_info), |
1181 | GFP_ATOMIC|__GFP_NOWARN); | 1181 | GFP_KERNEL|__GFP_NOWARN); |
1182 | if (itv->osd_info == NULL) { | 1182 | if (itv->osd_info == NULL) { |
1183 | IVTVFB_ERR("Failed to allocate memory for osd_info\n"); | 1183 | IVTVFB_ERR("Failed to allocate memory for osd_info\n"); |
1184 | return -ENOMEM; | 1184 | return -ENOMEM; |