ASoC: Intel: remove GFP_ATOMIC, use GFP_KERNEL
authorPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Thu, 1 Nov 2018 21:34:52 +0000 (16:34 -0500)
committerMark Brown <broonie@kernel.org>
Tue, 6 Nov 2018 17:24:36 +0000 (17:24 +0000)
GFP_ATOMIC is not required on any Intel drivers, use GFP_KERNEL
instead. A first cleanup was merged in April but missed a number
occurrences and new ones were added by copy/paste inertia.

While we are at it, make checkpatch happy with a sizeof(*msg)

Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/intel/atom/sst/sst_pvt.c
sound/soc/intel/boards/cht_bsw_rt5672.c
sound/soc/intel/boards/glk_rt5682_max98357a.c
sound/soc/intel/boards/kbl_da7219_max98927.c
sound/soc/intel/boards/skl_hda_dsp_generic.c

index af93244..00a37a0 100644 (file)
@@ -166,11 +166,11 @@ int sst_create_ipc_msg(struct ipc_post **arg, bool large)
 {
        struct ipc_post *msg;
 
-       msg = kzalloc(sizeof(struct ipc_post), GFP_ATOMIC);
+       msg = kzalloc(sizeof(*msg), GFP_KERNEL);
        if (!msg)
                return -ENOMEM;
        if (large) {
-               msg->mailbox_data = kzalloc(SST_MAILBOX_SIZE, GFP_ATOMIC);
+               msg->mailbox_data = kzalloc(SST_MAILBOX_SIZE, GFP_KERNEL);
                if (!msg->mailbox_data) {
                        kfree(msg);
                        return -ENOMEM;
index 51f0d45..9de64f4 100644 (file)
@@ -403,7 +403,7 @@ static int snd_cht_mc_probe(struct platform_device *pdev)
        const char *i2c_name;
        int i;
 
-       drv = devm_kzalloc(&pdev->dev, sizeof(*drv), GFP_ATOMIC);
+       drv = devm_kzalloc(&pdev->dev, sizeof(*drv), GFP_KERNEL);
        if (!drv)
                return -ENOMEM;
 
index c4b94e2..c74c4f1 100644 (file)
@@ -603,7 +603,7 @@ static int geminilake_audio_probe(struct platform_device *pdev)
 {
        struct glk_card_private *ctx;
 
-       ctx = devm_kzalloc(&pdev->dev, sizeof(*ctx), GFP_ATOMIC);
+       ctx = devm_kzalloc(&pdev->dev, sizeof(*ctx), GFP_KERNEL);
        if (!ctx)
                return -ENOMEM;
 
index 3bc78e3..58eb0fe 100644 (file)
@@ -935,7 +935,7 @@ static int kabylake_audio_probe(struct platform_device *pdev)
 {
        struct kbl_codec_private *ctx;
 
-       ctx = devm_kzalloc(&pdev->dev, sizeof(*ctx), GFP_ATOMIC);
+       ctx = devm_kzalloc(&pdev->dev, sizeof(*ctx), GFP_KERNEL);
        if (!ctx)
                return -ENOMEM;
 
index b6f287f..15c502d 100644 (file)
@@ -140,7 +140,7 @@ static int skl_hda_audio_probe(struct platform_device *pdev)
 
        dev_dbg(&pdev->dev, "%s: entry\n", __func__);
 
-       ctx = devm_kzalloc(&pdev->dev, sizeof(*ctx), GFP_ATOMIC);
+       ctx = devm_kzalloc(&pdev->dev, sizeof(*ctx), GFP_KERNEL);
        if (!ctx)
                return -ENOMEM;