March 2016 - Alsa-devel - mailman.alsa-project.org

[alsa-devel] [PATCH 1/2] sound/usb/media: use core routine to initialize media_device
by Mauro Carvalho Chehab 16 Mar '16

16 Mar '16

The media core has now a routine to initialize media_device for USB devices. Use it, instead of doing its own logic, as it warrants that all USB drivers will behave the same. It also warrants that the device will get the same data, no matter if it was initialized initially via snd-usb-audio or via some other driver, like au0828. Signed-off-by: Mauro Carvalho Chehab <mchehab(a)osg.samsung.com> --- sound/usb/media.c | 18 +++++------------- 1 file changed, 5 insertions(+), 13 deletions(-) diff --git a/sound/usb/media.c b/sound/usb/media.c index 93a50d01490c..44a5de91f748 100644 --- a/sound/usb/media.c +++ b/sound/usb/media.c @@ -263,19 +263,11 @@ int media_snd_device_create(struct snd_usb_audio *chip, mdev = media_device_get_devres(&usbdev->dev); if (!mdev) return -ENOMEM; - if (!mdev->dev) { - /* register media device */ - mdev->dev = &usbdev->dev; - if (usbdev->product) - strlcpy(mdev->model, usbdev->product, - sizeof(mdev->model)); - if (usbdev->serial) - strlcpy(mdev->serial, usbdev->serial, - sizeof(mdev->serial)); - strcpy(mdev->bus_info, usbdev->devpath); - mdev->hw_revision = le16_to_cpu(usbdev->descriptor.bcdDevice); - media_device_init(mdev); - } + + /* Initialize media device */ + if (!mdev->dev) + media_device_usb_init(mdev, usbdev, NULL); + if (!media_devnode_is_registered(&mdev->devnode)) { ret = media_device_register(mdev); if (ret) { -- 2.5.0

1 0

[alsa-devel] [PATCH] media: add GFP flag to media_*() that could get called in atomic context
by Shuah Khan 16 Mar '16

16 Mar '16

Add GFP flags to media_create_pad_link(), media_create_intf_link(), media_devnode_create(), and media_add_link() that could get called in atomic context to allow callers to pass in the right flags for memory allocation. tree-wide driver changes for media_*() GFP flags change: Change drivers to add gfpflags to interffaces, media_create_pad_link(), media_create_intf_link() and media_devnode_create(). Signed-off-by: Shuah Khan <shuahkh(a)osg.samsung.com> Suggested-by: Mauro Carvalho Chehab <mchehab(a)osg.samsung.com> --- Ran through kbuild-all compile testing. Tested the changes in Win-TV HVR-950Q device drivers/media/dvb-core/dvbdev.c | 26 +++++++----- drivers/media/i2c/s5c73m3/s5c73m3-core.c | 6 ++- drivers/media/i2c/s5k5baf.c | 3 +- drivers/media/i2c/smiapp/smiapp-core.c | 3 +- drivers/media/i2c/tvp5150.c | 3 +- drivers/media/media-entity.c | 30 ++++++++------ drivers/media/platform/exynos4-is/media-dev.c | 19 +++++---- drivers/media/platform/omap3isp/isp.c | 47 ++++++++++++++-------- drivers/media/platform/s3c-camif/camif-core.c | 4 +- drivers/media/platform/vsp1/vsp1_drm.c | 6 +-- drivers/media/platform/vsp1/vsp1_drv.c | 9 +++-- drivers/media/platform/xilinx/xilinx-vipp.c | 4 +- drivers/media/usb/au0828/au0828-core.c | 3 +- drivers/media/usb/uvc/uvc_entity.c | 2 +- drivers/media/v4l2-core/v4l2-dev.c | 5 ++- drivers/media/v4l2-core/v4l2-device.c | 3 +- drivers/media/v4l2-core/v4l2-mc.c | 25 +++++++----- drivers/staging/media/davinci_vpfe/dm365_ipipeif.c | 3 +- drivers/staging/media/davinci_vpfe/dm365_isif.c | 2 +- drivers/staging/media/davinci_vpfe/dm365_resizer.c | 10 +++-- .../staging/media/davinci_vpfe/vpfe_mc_capture.c | 10 ++--- drivers/staging/media/omap4iss/iss.c | 17 +++++--- drivers/staging/media/omap4iss/iss_csi2.c | 6 ++- drivers/staging/media/omap4iss/iss_ipipeif.c | 3 +- drivers/staging/media/omap4iss/iss_resizer.c | 3 +- include/media/media-entity.h | 9 +++-- sound/usb/media.c | 15 ++++--- 27 files changed, 170 insertions(+), 106 deletions(-) diff --git a/drivers/media/dvb-core/dvbdev.c b/drivers/media/dvb-core/dvbdev.c index e1684c5..57f3e1e 100644 --- a/drivers/media/dvb-core/dvbdev.c +++ b/drivers/media/dvb-core/dvbdev.c @@ -399,7 +399,8 @@ static int dvb_register_media_device(struct dvb_device *dvbdev, dvbdev->intf_devnode = media_devnode_create(dvbdev->adapter->mdev, intf_type, 0, - DVB_MAJOR, minor); + DVB_MAJOR, minor, + GFP_KERNEL); if (!dvbdev->intf_devnode) return -ENOMEM; @@ -416,7 +417,7 @@ static int dvb_register_media_device(struct dvb_device *dvbdev, return 0; link = media_create_intf_link(dvbdev->entity, &dvbdev->intf_devnode->intf, - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, GFP_KERNEL); if (!link) return -ENOMEM; #endif @@ -558,7 +559,8 @@ static int dvb_create_io_intf_links(struct dvb_adapter *adap, if (strncmp(entity->name, name, strlen(name))) continue; link = media_create_intf_link(entity, intf, - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, + GFP_KERNEL); if (!link) return -ENOMEM; } @@ -680,7 +682,8 @@ int dvb_create_media_graph(struct dvb_adapter *adap, } if (demux && ca) { ret = media_create_pad_link(demux, 1, ca, - 0, MEDIA_LNK_FL_ENABLED); + 0, MEDIA_LNK_FL_ENABLED, + GFP_KERNEL); if (ret) return -ENOMEM; } @@ -693,7 +696,8 @@ int dvb_create_media_graph(struct dvb_adapter *adap, strlen(DVR_TSOUT))) { ret = media_create_pad_link(demux, ++dvr_pad, - entity, 0, 0); + entity, 0, 0, + GFP_KERNEL); if (ret) return ret; } @@ -701,7 +705,8 @@ int dvb_create_media_graph(struct dvb_adapter *adap, strlen(DEMUX_TSOUT))) { ret = media_create_pad_link(demux, ++demux_pad, - entity, 0, 0); + entity, 0, 0, + GFP_KERNEL); if (ret) return ret; } @@ -713,14 +718,16 @@ int dvb_create_media_graph(struct dvb_adapter *adap, media_device_for_each_intf(intf, mdev) { if (intf->type == MEDIA_INTF_T_DVB_CA && ca) { link = media_create_intf_link(ca, intf, - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, + GFP_KERNEL); if (!link) return -ENOMEM; } if (intf->type == MEDIA_INTF_T_DVB_FE && tuner) { link = media_create_intf_link(tuner, intf, - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, + GFP_KERNEL); if (!link) return -ENOMEM; } @@ -732,7 +739,8 @@ int dvb_create_media_graph(struct dvb_adapter *adap, */ if (intf->type == MEDIA_INTF_T_DVB_DVR && demux) { link = media_create_intf_link(demux, intf, - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, + GFP_KERNEL); if (!link) return -ENOMEM; } diff --git a/drivers/media/i2c/s5c73m3/s5c73m3-core.c b/drivers/media/i2c/s5c73m3/s5c73m3-core.c index 08af58f..dd994f1 100644 --- a/drivers/media/i2c/s5c73m3/s5c73m3-core.c +++ b/drivers/media/i2c/s5c73m3/s5c73m3-core.c @@ -1484,11 +1484,13 @@ static int s5c73m3_oif_registered(struct v4l2_subdev *sd) ret = media_create_pad_link(&state->sensor_sd.entity, S5C73M3_ISP_PAD, &state->oif_sd.entity, OIF_ISP_PAD, - MEDIA_LNK_FL_IMMUTABLE | MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_IMMUTABLE | MEDIA_LNK_FL_ENABLED, + GFP_KERNEL); ret = media_create_pad_link(&state->sensor_sd.entity, S5C73M3_JPEG_PAD, &state->oif_sd.entity, OIF_JPEG_PAD, - MEDIA_LNK_FL_IMMUTABLE | MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_IMMUTABLE | MEDIA_LNK_FL_ENABLED, + GFP_KERNEL); return ret; } diff --git a/drivers/media/i2c/s5k5baf.c b/drivers/media/i2c/s5k5baf.c index db82ed0..3692bec 100644 --- a/drivers/media/i2c/s5k5baf.c +++ b/drivers/media/i2c/s5k5baf.c @@ -1759,7 +1759,8 @@ static int s5k5baf_registered(struct v4l2_subdev *sd) ret = media_create_pad_link(&state->cis_sd.entity, PAD_CIS, &state->sd.entity, PAD_CIS, MEDIA_LNK_FL_IMMUTABLE | - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, + GFP_KERNEL); return ret; } diff --git a/drivers/media/i2c/smiapp/smiapp-core.c b/drivers/media/i2c/smiapp/smiapp-core.c index a215efe..8c2e9ad 100644 --- a/drivers/media/i2c/smiapp/smiapp-core.c +++ b/drivers/media/i2c/smiapp/smiapp-core.c @@ -2508,7 +2508,8 @@ static int smiapp_register_subdevs(struct smiapp_sensor *sensor) &last->sd.entity, last->sink_pad, MEDIA_LNK_FL_ENABLED | - MEDIA_LNK_FL_IMMUTABLE); + MEDIA_LNK_FL_IMMUTABLE, + GFP_KERNEL); if (rval) { dev_err(&client->dev, "media_create_pad_link failed\n"); diff --git a/drivers/media/i2c/tvp5150.c b/drivers/media/i2c/tvp5150.c index ff18444..fd99683 100644 --- a/drivers/media/i2c/tvp5150.c +++ b/drivers/media/i2c/tvp5150.c @@ -1197,7 +1197,8 @@ static int tvp5150_registered_async(struct v4l2_subdev *sd) return ret; ret = media_create_pad_link(input, 0, &sd->entity, - DEMOD_PAD_IF_INPUT, 0); + DEMOD_PAD_IF_INPUT, 0, + GFP_KERNEL); if (ret < 0) { media_device_unregister_entity(input); return ret; diff --git a/drivers/media/media-entity.c b/drivers/media/media-entity.c index 66a5392..115435b 100644 --- a/drivers/media/media-entity.c +++ b/drivers/media/media-entity.c @@ -566,14 +566,14 @@ EXPORT_SYMBOL_GPL(media_entity_put); * Links management */ -static struct media_link *media_add_link(struct list_head *head) +static struct media_link *media_add_link(struct list_head *head, gfp_t gfpflags) { struct media_link *link; if (in_atomic()) dump_stack(); - link = kzalloc(sizeof(*link), GFP_KERNEL); + link = kzalloc(sizeof(*link), gfpflags); if (link == NULL) return NULL; @@ -615,7 +615,8 @@ static void __media_entity_remove_link(struct media_entity *entity, int media_create_pad_link(struct media_entity *source, u16 source_pad, - struct media_entity *sink, u16 sink_pad, u32 flags) + struct media_entity *sink, u16 sink_pad, u32 flags, + gfp_t gfpflags) { struct media_link *link; struct media_link *backlink; @@ -624,7 +625,7 @@ media_create_pad_link(struct media_entity *source, u16 source_pad, BUG_ON(source_pad >= source->num_pads); BUG_ON(sink_pad >= sink->num_pads); - link = media_add_link(&source->links); + link = media_add_link(&source->links, gfpflags); if (link == NULL) return -ENOMEM; @@ -639,7 +640,7 @@ media_create_pad_link(struct media_entity *source, u16 source_pad, /* Create the backlink. Backlinks are used to help graph traversal and * are not reported to userspace. */ - backlink = media_add_link(&sink->links); + backlink = media_add_link(&sink->links, gfpflags); if (backlink == NULL) { __media_entity_remove_link(source, link); return -ENOMEM; @@ -682,7 +683,7 @@ int media_create_pad_links(const struct media_device *mdev, /* Trivial case: 1:1 relation */ if (source && sink) return media_create_pad_link(source, source_pad, - sink, sink_pad, flags); + sink, sink_pad, flags, GFP_KERNEL); /* Worse case scenario: n:n relation */ if (!source && !sink) { @@ -696,7 +697,7 @@ int media_create_pad_links(const struct media_device *mdev, continue; ret = media_create_pad_link(source, source_pad, sink, sink_pad, - flags); + flags, GFP_KERNEL); if (ret) return ret; flags &= ~(MEDIA_LNK_FL_ENABLED | @@ -718,10 +719,12 @@ int media_create_pad_links(const struct media_device *mdev, if (source) ret = media_create_pad_link(source, source_pad, - entity, sink_pad, flags); + entity, sink_pad, flags, + GFP_KERNEL); else ret = media_create_pad_link(entity, source_pad, - sink, sink_pad, flags); + sink, sink_pad, flags, + GFP_KERNEL); if (ret) return ret; flags &= ~(MEDIA_LNK_FL_ENABLED | MEDIA_LNK_FL_IMMUTABLE); @@ -890,14 +893,15 @@ static void media_interface_init(struct media_device *mdev, struct media_intf_devnode *media_devnode_create(struct media_device *mdev, u32 type, u32 flags, - u32 major, u32 minor) + u32 major, u32 minor, + gfp_t gfpflags) { struct media_intf_devnode *devnode; if (in_atomic()) dump_stack(); - devnode = kzalloc(sizeof(*devnode), GFP_KERNEL); + devnode = kzalloc(sizeof(*devnode), gfpflags); if (!devnode) return NULL; @@ -921,11 +925,11 @@ EXPORT_SYMBOL_GPL(media_devnode_remove); struct media_link *media_create_intf_link(struct media_entity *entity, struct media_interface *intf, - u32 flags) + u32 flags, gfp_t gfpflags) { struct media_link *link; - link = media_add_link(&intf->links); + link = media_add_link(&intf->links, gfpflags); if (link == NULL) return NULL; diff --git a/drivers/media/platform/exynos4-is/media-dev.c b/drivers/media/platform/exynos4-is/media-dev.c index feb521f..145d90e 100644 --- a/drivers/media/platform/exynos4-is/media-dev.c +++ b/drivers/media/platform/exynos4-is/media-dev.c @@ -798,7 +798,8 @@ static int __fimc_md_create_fimc_sink_links(struct fimc_md *fmd, sink = &fmd->fimc[i]->vid_cap.subdev.entity; ret = media_create_pad_link(source, pad, sink, - FIMC_SD_PAD_SINK_CAM, flags); + FIMC_SD_PAD_SINK_CAM, flags, + GFP_KERNEL); if (ret) return ret; @@ -818,7 +819,8 @@ static int __fimc_md_create_fimc_sink_links(struct fimc_md *fmd, sink = &fmd->fimc_lite[i]->subdev.entity; ret = media_create_pad_link(source, pad, sink, - FLITE_SD_PAD_SINK, 0); + FLITE_SD_PAD_SINK, 0, + GFP_KERNEL); if (ret) return ret; @@ -850,13 +852,13 @@ static int __fimc_md_create_flite_source_links(struct fimc_md *fmd) sink = &fimc->ve.vdev.entity; /* FIMC-LITE's subdev and video node */ ret = media_create_pad_link(source, FLITE_SD_PAD_SOURCE_DMA, - sink, 0, 0); + sink, 0, 0, GFP_KERNEL); if (ret) break; /* Link from FIMC-LITE to IS-ISP subdev */ sink = &fmd->fimc_is->isp.subdev.entity; ret = media_create_pad_link(source, FLITE_SD_PAD_SOURCE_ISP, - sink, 0, 0); + sink, 0, 0, GFP_KERNEL); if (ret) break; } @@ -880,7 +882,8 @@ static int __fimc_md_create_fimc_is_links(struct fimc_md *fmd) /* Link from FIMC-IS-ISP subdev to FIMC */ sink = &fmd->fimc[i]->vid_cap.subdev.entity; ret = media_create_pad_link(source, FIMC_ISP_SD_PAD_SRC_FIFO, - sink, FIMC_SD_PAD_SINK_FIFO, 0); + sink, FIMC_SD_PAD_SINK_FIFO, 0, + GFP_KERNEL); if (ret) return ret; } @@ -893,7 +896,7 @@ static int __fimc_md_create_fimc_is_links(struct fimc_md *fmd) return 0; return media_create_pad_link(source, FIMC_ISP_SD_PAD_SRC_DMA, - sink, 0, 0); + sink, 0, 0, GFP_KERNEL); } /** @@ -944,7 +947,7 @@ static int fimc_md_create_links(struct fimc_md *fmd) ret = media_create_pad_link(&sensor->entity, pad, &csis->entity, CSIS_PAD_SINK, MEDIA_LNK_FL_IMMUTABLE | - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, GFP_KERNEL); if (ret) return ret; @@ -996,7 +999,7 @@ static int fimc_md_create_links(struct fimc_md *fmd) sink = &fmd->fimc[i]->vid_cap.ve.vdev.entity; ret = media_create_pad_link(source, FIMC_SD_PAD_SOURCE, - sink, 0, flags); + sink, 0, flags, GFP_KERNEL); if (ret) break; } diff --git a/drivers/media/platform/omap3isp/isp.c b/drivers/media/platform/omap3isp/isp.c index 5d54e2c..f8d9c84 100644 --- a/drivers/media/platform/omap3isp/isp.c +++ b/drivers/media/platform/omap3isp/isp.c @@ -1669,7 +1669,7 @@ static int isp_link_entity( return -EINVAL; } - return media_create_pad_link(entity, i, input, pad, flags); + return media_create_pad_link(entity, i, input, pad, flags, GFP_KERNEL); } static int isp_register_entities(struct isp_device *isp) @@ -1748,43 +1748,50 @@ static int isp_create_links(struct isp_device *isp) /* Create links between entities and video nodes. */ ret = media_create_pad_link( &isp->isp_csi2a.subdev.entity, CSI2_PAD_SOURCE, - &isp->isp_csi2a.video_out.video.entity, 0, 0); + &isp->isp_csi2a.video_out.video.entity, 0, 0, + GFP_KERNEL); if (ret < 0) return ret; ret = media_create_pad_link( &isp->isp_ccp2.video_in.video.entity, 0, - &isp->isp_ccp2.subdev.entity, CCP2_PAD_SINK, 0); + &isp->isp_ccp2.subdev.entity, CCP2_PAD_SINK, 0, + GFP_KERNEL); if (ret < 0) return ret; ret = media_create_pad_link( &isp->isp_ccdc.subdev.entity, CCDC_PAD_SOURCE_OF, - &isp->isp_ccdc.video_out.video.entity, 0, 0); + &isp->isp_ccdc.video_out.video.entity, 0, 0, + GFP_KERNEL); if (ret < 0) return ret; ret = media_create_pad_link( &isp->isp_prev.video_in.video.entity, 0, - &isp->isp_prev.subdev.entity, PREV_PAD_SINK, 0); + &isp->isp_prev.subdev.entity, PREV_PAD_SINK, 0, + GFP_KERNEL); if (ret < 0) return ret; ret = media_create_pad_link( &isp->isp_prev.subdev.entity, PREV_PAD_SOURCE, - &isp->isp_prev.video_out.video.entity, 0, 0); + &isp->isp_prev.video_out.video.entity, 0, 0, + GFP_KERNEL); if (ret < 0) return ret; ret = media_create_pad_link( &isp->isp_res.video_in.video.entity, 0, - &isp->isp_res.subdev.entity, RESZ_PAD_SINK, 0); + &isp->isp_res.subdev.entity, RESZ_PAD_SINK, 0, + GFP_KERNEL); if (ret < 0) return ret; ret = media_create_pad_link( &isp->isp_res.subdev.entity, RESZ_PAD_SOURCE, - &isp->isp_res.video_out.video.entity, 0, 0); + &isp->isp_res.video_out.video.entity, 0, 0, + GFP_KERNEL); if (ret < 0) return ret; @@ -1792,52 +1799,60 @@ static int isp_create_links(struct isp_device *isp) /* Create links between entities. */ ret = media_create_pad_link( &isp->isp_csi2a.subdev.entity, CSI2_PAD_SOURCE, - &isp->isp_ccdc.subdev.entity, CCDC_PAD_SINK, 0); + &isp->isp_ccdc.subdev.entity, CCDC_PAD_SINK, 0, + GFP_KERNEL); if (ret < 0) return ret; ret = media_create_pad_link( &isp->isp_ccp2.subdev.entity, CCP2_PAD_SOURCE, - &isp->isp_ccdc.subdev.entity, CCDC_PAD_SINK, 0); + &isp->isp_ccdc.subdev.entity, CCDC_PAD_SINK, 0, + GFP_KERNEL); if (ret < 0) return ret; ret = media_create_pad_link( &isp->isp_ccdc.subdev.entity, CCDC_PAD_SOURCE_VP, - &isp->isp_prev.subdev.entity, PREV_PAD_SINK, 0); + &isp->isp_prev.subdev.entity, PREV_PAD_SINK, 0, + GFP_KERNEL); if (ret < 0) return ret; ret = media_create_pad_link( &isp->isp_ccdc.subdev.entity, CCDC_PAD_SOURCE_OF, - &isp->isp_res.subdev.entity, RESZ_PAD_SINK, 0); + &isp->isp_res.subdev.entity, RESZ_PAD_SINK, 0, + GFP_KERNEL); if (ret < 0) return ret; ret = media_create_pad_link( &isp->isp_prev.subdev.entity, PREV_PAD_SOURCE, - &isp->isp_res.subdev.entity, RESZ_PAD_SINK, 0); + &isp->isp_res.subdev.entity, RESZ_PAD_SINK, 0, + GFP_KERNEL); if (ret < 0) return ret; ret = media_create_pad_link( &isp->isp_ccdc.subdev.entity, CCDC_PAD_SOURCE_VP, &isp->isp_aewb.subdev.entity, 0, - MEDIA_LNK_FL_ENABLED | MEDIA_LNK_FL_IMMUTABLE); + MEDIA_LNK_FL_ENABLED | MEDIA_LNK_FL_IMMUTABLE, + GFP_KERNEL); if (ret < 0) return ret; ret = media_create_pad_link( &isp->isp_ccdc.subdev.entity, CCDC_PAD_SOURCE_VP, &isp->isp_af.subdev.entity, 0, - MEDIA_LNK_FL_ENABLED | MEDIA_LNK_FL_IMMUTABLE); + MEDIA_LNK_FL_ENABLED | MEDIA_LNK_FL_IMMUTABLE, + GFP_KERNEL); if (ret < 0) return ret; ret = media_create_pad_link( &isp->isp_ccdc.subdev.entity, CCDC_PAD_SOURCE_VP, &isp->isp_hist.subdev.entity, 0, - MEDIA_LNK_FL_ENABLED | MEDIA_LNK_FL_IMMUTABLE); + MEDIA_LNK_FL_ENABLED | MEDIA_LNK_FL_IMMUTABLE, + GFP_KERNEL); if (ret < 0) return ret; diff --git a/drivers/media/platform/s3c-camif/camif-core.c b/drivers/media/platform/s3c-camif/camif-core.c index 0b44b9a..a49b4c3 100644 --- a/drivers/media/platform/s3c-camif/camif-core.c +++ b/drivers/media/platform/s3c-camif/camif-core.c @@ -266,7 +266,7 @@ static int camif_create_media_links(struct camif_dev *camif) ret = media_create_pad_link(&camif->sensor.sd->entity, 0, &camif->subdev.entity, CAMIF_SD_PAD_SINK, MEDIA_LNK_FL_IMMUTABLE | - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, GFP_KERNEL); if (ret) return ret; @@ -274,7 +274,7 @@ static int camif_create_media_links(struct camif_dev *camif) ret = media_create_pad_link(&camif->subdev.entity, i, &camif->vp[i - 1].vdev.entity, 0, MEDIA_LNK_FL_IMMUTABLE | - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, GFP_KERNEL); } return ret; diff --git a/drivers/media/platform/vsp1/vsp1_drm.c b/drivers/media/platform/vsp1/vsp1_drm.c index 021fe57..f868dec 100644 --- a/drivers/media/platform/vsp1/vsp1_drm.c +++ b/drivers/media/platform/vsp1/vsp1_drm.c @@ -525,7 +525,7 @@ int vsp1_drm_create_links(struct vsp1_device *vsp1) ret = media_create_pad_link(&rpf->entity.subdev.entity, RWPF_PAD_SOURCE, &vsp1->bru->entity.subdev.entity, - i, flags); + i, flags, GFP_KERNEL); if (ret < 0) return ret; @@ -536,7 +536,7 @@ int vsp1_drm_create_links(struct vsp1_device *vsp1) ret = media_create_pad_link(&vsp1->bru->entity.subdev.entity, vsp1->bru->entity.source_pad, &vsp1->wpf[0]->entity.subdev.entity, - RWPF_PAD_SINK, flags); + RWPF_PAD_SINK, flags, GFP_KERNEL); if (ret < 0) return ret; @@ -546,7 +546,7 @@ int vsp1_drm_create_links(struct vsp1_device *vsp1) ret = media_create_pad_link(&vsp1->wpf[0]->entity.subdev.entity, RWPF_PAD_SOURCE, &vsp1->lif->entity.subdev.entity, - LIF_PAD_SINK, flags); + LIF_PAD_SINK, flags, GFP_KERNEL); if (ret < 0) return ret; diff --git a/drivers/media/platform/vsp1/vsp1_drv.c b/drivers/media/platform/vsp1/vsp1_drv.c index 25750a0..7295587 100644 --- a/drivers/media/platform/vsp1/vsp1_drv.c +++ b/drivers/media/platform/vsp1/vsp1_drv.c @@ -125,7 +125,8 @@ static int vsp1_create_sink_links(struct vsp1_device *vsp1, ret = media_create_pad_link(&source->subdev.entity, source->source_pad, - entity, pad, flags); + entity, pad, flags, + GFP_KERNEL); if (ret < 0) return ret; @@ -157,7 +158,7 @@ static int vsp1_uapi_create_links(struct vsp1_device *vsp1) ret = media_create_pad_link(&vsp1->wpf[0]->entity.subdev.entity, RWPF_PAD_SOURCE, &vsp1->lif->entity.subdev.entity, - LIF_PAD_SINK, 0); + LIF_PAD_SINK, 0, GFP_KERNEL); if (ret < 0) return ret; } @@ -169,7 +170,7 @@ static int vsp1_uapi_create_links(struct vsp1_device *vsp1) &rpf->entity.subdev.entity, RWPF_PAD_SINK, MEDIA_LNK_FL_ENABLED | - MEDIA_LNK_FL_IMMUTABLE); + MEDIA_LNK_FL_IMMUTABLE, GFP_KERNEL); if (ret < 0) return ret; } @@ -188,7 +189,7 @@ static int vsp1_uapi_create_links(struct vsp1_device *vsp1) ret = media_create_pad_link(&wpf->entity.subdev.entity, RWPF_PAD_SOURCE, &wpf->video->video.entity, 0, - flags); + flags, GFP_KERNEL); if (ret < 0) return ret; } diff --git a/drivers/media/platform/xilinx/xilinx-vipp.c b/drivers/media/platform/xilinx/xilinx-vipp.c index e795a45..b444db4 100644 --- a/drivers/media/platform/xilinx/xilinx-vipp.c +++ b/drivers/media/platform/xilinx/xilinx-vipp.c @@ -158,7 +158,7 @@ static int xvip_graph_build_one(struct xvip_composite_device *xdev, ret = media_create_pad_link(local, local_pad->index, remote, remote_pad->index, - link_flags); + link_flags, GFP_KERNEL); if (ret < 0) { dev_err(xdev->dev, "failed to create %s:%u -> %s:%u link\n", @@ -272,7 +272,7 @@ static int xvip_graph_build_dma(struct xvip_composite_device *xdev) ret = media_create_pad_link(source, source_pad->index, sink, sink_pad->index, - link_flags); + link_flags, GFP_KERNEL); if (ret < 0) { dev_err(xdev->dev, "failed to create %s:%u -> %s:%u link\n", diff --git a/drivers/media/usb/au0828/au0828-core.c b/drivers/media/usb/au0828/au0828-core.c index 552ac58..de9ab11 100644 --- a/drivers/media/usb/au0828/au0828-core.c +++ b/drivers/media/usb/au0828/au0828-core.c @@ -251,7 +251,8 @@ create_link: ret = media_create_pad_link(decoder, DEMOD_PAD_AUDIO_OUT, mixer, 0, - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, + GFP_KERNEL); if (ret) dev_err(&dev->usbdev->dev, "Mixer Pad Link Create Error: %d\n", ret); diff --git a/drivers/media/usb/uvc/uvc_entity.c b/drivers/media/usb/uvc/uvc_entity.c index ac386bb..d9555dc 100644 --- a/drivers/media/usb/uvc/uvc_entity.c +++ b/drivers/media/usb/uvc/uvc_entity.c @@ -53,7 +53,7 @@ static int uvc_mc_create_links(struct uvc_video_chain *chain, remote_pad = remote->num_pads - 1; ret = media_create_pad_link(source, remote_pad, - sink, i, flags); + sink, i, flags, GFP_KERNEL); if (ret < 0) return ret; } diff --git a/drivers/media/v4l2-core/v4l2-dev.c b/drivers/media/v4l2-core/v4l2-dev.c index d8e5994..b0f53a2 100644 --- a/drivers/media/v4l2-core/v4l2-dev.c +++ b/drivers/media/v4l2-core/v4l2-dev.c @@ -786,7 +786,7 @@ static int video_register_media_controller(struct video_device *vdev, int type) vdev->intf_devnode = media_devnode_create(vdev->v4l2_dev->mdev, intf_type, 0, VIDEO_MAJOR, - vdev->minor); + vdev->minor, GFP_KERNEL); if (!vdev->intf_devnode) { media_device_unregister_entity(&vdev->entity); return -ENOMEM; @@ -797,7 +797,8 @@ static int video_register_media_controller(struct video_device *vdev, int type) link = media_create_intf_link(&vdev->entity, &vdev->intf_devnode->intf, - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, + GFP_KERNEL); if (!link) { media_devnode_remove(vdev->intf_devnode); media_device_unregister_entity(&vdev->entity); diff --git a/drivers/media/v4l2-core/v4l2-device.c b/drivers/media/v4l2-core/v4l2-device.c index 06fa5f1..c7514e7 100644 --- a/drivers/media/v4l2-core/v4l2-device.c +++ b/drivers/media/v4l2-core/v4l2-device.c @@ -266,7 +266,8 @@ int v4l2_device_register_subdev_nodes(struct v4l2_device *v4l2_dev) link = media_create_intf_link(&sd->entity, &vdev->intf_devnode->intf, - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, + GFP_KERNEL); if (!link) { err = -ENOMEM; goto clean_up; diff --git a/drivers/media/v4l2-core/v4l2-mc.c b/drivers/media/v4l2-core/v4l2-mc.c index 40ee864..b1652f9 100644 --- a/drivers/media/v4l2-core/v4l2-mc.c +++ b/drivers/media/v4l2-core/v4l2-mc.c @@ -92,7 +92,8 @@ int v4l2_mc_create_media_graph(struct media_device *mdev) continue; ret = media_create_pad_link(entity, 0, io_v4l, 0, - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, + GFP_KERNEL); if (ret) return ret; } @@ -110,18 +111,21 @@ int v4l2_mc_create_media_graph(struct media_device *mdev) ret = media_create_pad_link(tuner, TUNER_PAD_OUTPUT, if_vid, IF_VID_DEC_PAD_IF_INPUT, - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, + GFP_KERNEL); if (ret) return ret; ret = media_create_pad_link(if_vid, IF_VID_DEC_PAD_OUT, decoder, DEMOD_PAD_IF_INPUT, - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, + GFP_KERNEL); if (ret) return ret; } else { ret = media_create_pad_link(tuner, TUNER_PAD_OUTPUT, decoder, DEMOD_PAD_IF_INPUT, - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, + GFP_KERNEL); if (ret) return ret; } @@ -130,7 +134,8 @@ int v4l2_mc_create_media_graph(struct media_device *mdev) ret = media_create_pad_link(tuner, TUNER_PAD_AUD_OUT, if_aud, IF_AUD_DEC_PAD_IF_INPUT, - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, + GFP_KERNEL); if (ret) return ret; } else { @@ -143,7 +148,7 @@ int v4l2_mc_create_media_graph(struct media_device *mdev) if (io_v4l) { ret = media_create_pad_link(decoder, DEMOD_PAD_VID_OUT, io_v4l, 0, - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, GFP_KERNEL); if (ret) return ret; } @@ -151,7 +156,7 @@ int v4l2_mc_create_media_graph(struct media_device *mdev) if (io_swradio) { ret = media_create_pad_link(decoder, DEMOD_PAD_VID_OUT, io_swradio, 0, - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, GFP_KERNEL); if (ret) return ret; } @@ -159,7 +164,7 @@ int v4l2_mc_create_media_graph(struct media_device *mdev) if (io_vbi) { ret = media_create_pad_link(decoder, DEMOD_PAD_VBI_OUT, io_vbi, 0, - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, GFP_KERNEL); if (ret) return ret; } @@ -174,13 +179,13 @@ int v4l2_mc_create_media_graph(struct media_device *mdev) ret = media_create_pad_link(entity, 0, tuner, TUNER_PAD_RF_INPUT, - flags); + flags, GFP_KERNEL); break; case MEDIA_ENT_F_CONN_SVIDEO: case MEDIA_ENT_F_CONN_COMPOSITE: ret = media_create_pad_link(entity, 0, decoder, DEMOD_PAD_IF_INPUT, - flags); + flags, GFP_KERNEL); break; default: continue; diff --git a/drivers/staging/media/davinci_vpfe/dm365_ipipeif.c b/drivers/staging/media/davinci_vpfe/dm365_ipipeif.c index 633d645..d1442b9 100644 --- a/drivers/staging/media/davinci_vpfe/dm365_ipipeif.c +++ b/drivers/staging/media/davinci_vpfe/dm365_ipipeif.c @@ -977,7 +977,8 @@ vpfe_ipipeif_register_entities(struct vpfe_ipipeif_device *ipipeif, flags = 0; ret = media_create_pad_link(&ipipeif->video_in.video_dev.entity, 0, - &ipipeif->subdev.entity, 0, flags); + &ipipeif->subdev.entity, 0, flags, + GFP_KERNEL); if (ret < 0) goto fail; diff --git a/drivers/staging/media/davinci_vpfe/dm365_isif.c b/drivers/staging/media/davinci_vpfe/dm365_isif.c index 9905789..2470655 100644 --- a/drivers/staging/media/davinci_vpfe/dm365_isif.c +++ b/drivers/staging/media/davinci_vpfe/dm365_isif.c @@ -1824,7 +1824,7 @@ int vpfe_isif_register_entities(struct vpfe_isif_device *isif, /* connect isif to video node */ ret = media_create_pad_link(&isif->subdev.entity, 1, &isif->video_out.video_dev.entity, - 0, flags); + 0, flags, GFP_KERNEL); if (ret < 0) goto out_create_link; return 0; diff --git a/drivers/staging/media/davinci_vpfe/dm365_resizer.c b/drivers/staging/media/davinci_vpfe/dm365_resizer.c index a91395c..067f699 100644 --- a/drivers/staging/media/davinci_vpfe/dm365_resizer.c +++ b/drivers/staging/media/davinci_vpfe/dm365_resizer.c @@ -1833,26 +1833,28 @@ int vpfe_resizer_register_entities(struct vpfe_resizer_device *resizer, /* create link between Resizer Crop----> Resizer A*/ ret = media_create_pad_link(&resizer->crop_resizer.subdev.entity, 1, &resizer->resizer_a.subdev.entity, - 0, flags); + 0, flags, GFP_KERNEL); if (ret < 0) goto out_create_link; /* create link between Resizer Crop----> Resizer B*/ ret = media_create_pad_link(&resizer->crop_resizer.subdev.entity, 2, &resizer->resizer_b.subdev.entity, - 0, flags); + 0, flags, GFP_KERNEL); if (ret < 0) goto out_create_link; /* create link between Resizer A ----> video out */ ret = media_create_pad_link(&resizer->resizer_a.subdev.entity, 1, - &resizer->resizer_a.video_out.video_dev.entity, 0, flags); + &resizer->resizer_a.video_out.video_dev.entity, 0, flags, + GFP_KERNEL); if (ret < 0) goto out_create_link; /* create link between Resizer B ----> video out */ ret = media_create_pad_link(&resizer->resizer_b.subdev.entity, 1, - &resizer->resizer_b.video_out.video_dev.entity, 0, flags); + &resizer->resizer_b.video_out.video_dev.entity, 0, flags, + GFP_KERNEL); if (ret < 0) goto out_create_link; diff --git a/drivers/staging/media/davinci_vpfe/vpfe_mc_capture.c b/drivers/staging/media/davinci_vpfe/vpfe_mc_capture.c index ec46f36..bee9f5c 100644 --- a/drivers/staging/media/davinci_vpfe/vpfe_mc_capture.c +++ b/drivers/staging/media/davinci_vpfe/vpfe_mc_capture.c @@ -447,32 +447,32 @@ static int vpfe_register_entities(struct vpfe_device *vpfe_dev) if (vpfe_dev->sd[i]->entity.num_pads) { ret = media_create_pad_link(&vpfe_dev->sd[i]->entity, 0, &vpfe_dev->vpfe_isif.subdev.entity, - 0, flags); + 0, flags, GFP_KERNEL); if (ret < 0) goto out_resizer_register; } ret = media_create_pad_link(&vpfe_dev->vpfe_isif.subdev.entity, 1, &vpfe_dev->vpfe_ipipeif.subdev.entity, - 0, flags); + 0, flags, GFP_KERNEL); if (ret < 0) goto out_resizer_register; ret = media_create_pad_link(&vpfe_dev->vpfe_ipipeif.subdev.entity, 1, &vpfe_dev->vpfe_ipipe.subdev.entity, - 0, flags); + 0, flags, GFP_KERNEL); if (ret < 0) goto out_resizer_register; ret = media_create_pad_link(&vpfe_dev->vpfe_ipipe.subdev.entity, 1, &vpfe_dev->vpfe_resizer.crop_resizer.subdev.entity, - 0, flags); + 0, flags, GFP_KERNEL); if (ret < 0) goto out_resizer_register; ret = media_create_pad_link(&vpfe_dev->vpfe_ipipeif.subdev.entity, 1, &vpfe_dev->vpfe_resizer.crop_resizer.subdev.entity, - 0, flags); + 0, flags, GFP_KERNEL); if (ret < 0) goto out_resizer_register; diff --git a/drivers/staging/media/omap4iss/iss.c b/drivers/staging/media/omap4iss/iss.c index fb80d2b..933c8a7 100644 --- a/drivers/staging/media/omap4iss/iss.c +++ b/drivers/staging/media/omap4iss/iss.c @@ -1065,7 +1065,7 @@ static int iss_register_entities(struct iss_device *iss) } ret = media_create_pad_link(&sensor->entity, 0, input, pad, - flags); + flags, GFP_KERNEL); if (ret < 0) goto done; } @@ -1110,31 +1110,36 @@ static int iss_create_links(struct iss_device *iss) /* Connect the submodules. */ ret = media_create_pad_link( &iss->csi2a.subdev.entity, CSI2_PAD_SOURCE, - &iss->ipipeif.subdev.entity, IPIPEIF_PAD_SINK, 0); + &iss->ipipeif.subdev.entity, IPIPEIF_PAD_SINK, 0, + GFP_KERNEL); if (ret < 0) return ret; ret = media_create_pad_link( &iss->csi2b.subdev.entity, CSI2_PAD_SOURCE, - &iss->ipipeif.subdev.entity, IPIPEIF_PAD_SINK, 0); + &iss->ipipeif.subdev.entity, IPIPEIF_PAD_SINK, 0, + GFP_KERNEL); if (ret < 0) return ret; ret = media_create_pad_link( &iss->ipipeif.subdev.entity, IPIPEIF_PAD_SOURCE_VP, - &iss->resizer.subdev.entity, RESIZER_PAD_SINK, 0); + &iss->resizer.subdev.entity, RESIZER_PAD_SINK, 0, + GFP_KERNEL); if (ret < 0) return ret; ret = media_create_pad_link( &iss->ipipeif.subdev.entity, IPIPEIF_PAD_SOURCE_VP, - &iss->ipipe.subdev.entity, IPIPE_PAD_SINK, 0); + &iss->ipipe.subdev.entity, IPIPE_PAD_SINK, 0, + GFP_KERNEL); if (ret < 0) return ret; ret = media_create_pad_link( &iss->ipipe.subdev.entity, IPIPE_PAD_SOURCE_VP, - &iss->resizer.subdev.entity, RESIZER_PAD_SINK, 0); + &iss->resizer.subdev.entity, RESIZER_PAD_SINK, 0, + GFP_KERNEL); if (ret < 0) return ret; diff --git a/drivers/staging/media/omap4iss/iss_csi2.c b/drivers/staging/media/omap4iss/iss_csi2.c index aaca39d..4b40132 100644 --- a/drivers/staging/media/omap4iss/iss_csi2.c +++ b/drivers/staging/media/omap4iss/iss_csi2.c @@ -1352,13 +1352,15 @@ int omap4iss_csi2_create_links(struct iss_device *iss) /* Connect the CSI2a subdev to the video node. */ ret = media_create_pad_link(&csi2a->subdev.entity, CSI2_PAD_SOURCE, - &csi2a->video_out.video.entity, 0, 0); + &csi2a->video_out.video.entity, 0, 0, + GFP_KERNEL); if (ret < 0) return ret; /* Connect the CSI2b subdev to the video node. */ ret = media_create_pad_link(&csi2b->subdev.entity, CSI2_PAD_SOURCE, - &csi2b->video_out.video.entity, 0, 0); + &csi2b->video_out.video.entity, 0, 0, + GFP_KERNEL); if (ret < 0) return ret; diff --git a/drivers/staging/media/omap4iss/iss_ipipeif.c b/drivers/staging/media/omap4iss/iss_ipipeif.c index 23de833..2f124b0 100644 --- a/drivers/staging/media/omap4iss/iss_ipipeif.c +++ b/drivers/staging/media/omap4iss/iss_ipipeif.c @@ -827,7 +827,8 @@ int omap4iss_ipipeif_create_links(struct iss_device *iss) /* Connect the IPIPEIF subdev to the video node. */ return media_create_pad_link(&ipipeif->subdev.entity, IPIPEIF_PAD_SOURCE_ISIF_SF, - &ipipeif->video_out.video.entity, 0, 0); + &ipipeif->video_out.video.entity, 0, 0, + GFP_KERNEL); } /* diff --git a/drivers/staging/media/omap4iss/iss_resizer.c b/drivers/staging/media/omap4iss/iss_resizer.c index f1d352c..8d5766a 100644 --- a/drivers/staging/media/omap4iss/iss_resizer.c +++ b/drivers/staging/media/omap4iss/iss_resizer.c @@ -869,7 +869,8 @@ int omap4iss_resizer_create_links(struct iss_device *iss) /* Connect the RESIZER subdev to the video node. */ return media_create_pad_link(&resizer->subdev.entity, RESIZER_PAD_SOURCE_MEM, - &resizer->video_out.video.entity, 0, 0); + &resizer->video_out.video.entity, 0, 0, + GFP_KERNEL); } /* diff --git a/include/media/media-entity.h b/include/media/media-entity.h index 6dc9e4e..0665c2d 100644 --- a/include/media/media-entity.h +++ b/include/media/media-entity.h @@ -595,6 +595,7 @@ static inline void media_entity_cleanup(struct media_entity *entity) {}; * @sink: pointer to &media_entity of the sink pad. * @sink_pad: number of the sink pad in the pads array. * @flags: Link flags, as defined in include/uapi/linux/media.h. + * @gfpflags: Get Free Page (GFP) flags, as defined in include/linux/gfp.h * * Valid values for flags: * A %MEDIA_LNK_FL_ENABLED flag indicates that the link is enabled and can be @@ -613,7 +614,7 @@ static inline void media_entity_cleanup(struct media_entity *entity) {}; */ __must_check int media_create_pad_link(struct media_entity *source, u16 source_pad, struct media_entity *sink, - u16 sink_pad, u32 flags); + u16 sink_pad, u32 flags, gfp_t gfpflags); /** * media_create_pad_links() - creates a link between two entities. @@ -883,11 +884,12 @@ void __media_entity_pipeline_stop(struct media_entity *entity); struct media_intf_devnode * __must_check media_devnode_create(struct media_device *mdev, u32 type, u32 flags, - u32 major, u32 minor); + u32 major, u32 minor, gfp_t gfpflags); /** * media_devnode_remove() - removes a device node interface * * @devnode: pointer to &media_intf_devnode to be freed. + * @gfpflags: Get Free Page (GFP) flags, as defined in include/linux/gfp.h * * When a device node interface is removed, all links to it are automatically * removed. @@ -901,6 +903,7 @@ struct media_link * * @entity: pointer to %media_entity * @intf: pointer to %media_interface * @flags: Link flags, as defined in include/uapi/linux/media.h. + * @gfpflags: Get Free Page (GFP) flags, as defined in include/linux/gfp.h * * * Valid values for flags: @@ -921,7 +924,7 @@ struct media_link * */ __must_check media_create_intf_link(struct media_entity *entity, struct media_interface *intf, - u32 flags); + u32 flags, gfp_t gfpflags); /** * __media_remove_intf_link() - remove a single interface link * diff --git a/sound/usb/media.c b/sound/usb/media.c index 93a50d01..f5ac038 100644 --- a/sound/usb/media.c +++ b/sound/usb/media.c @@ -88,14 +88,16 @@ int media_snd_stream_init(struct snd_usb_substream *subs, struct snd_pcm *pcm, mctl->intf_devnode = media_devnode_create(mdev, intf_type, 0, MAJOR(pcm_dev->devt), - MINOR(pcm_dev->devt)); + MINOR(pcm_dev->devt), + GFP_KERNEL); if (!mctl->intf_devnode) { ret = -ENOMEM; goto unregister_entity; } mctl->intf_link = media_create_intf_link(&mctl->media_entity, &mctl->intf_devnode->intf, - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, + GFP_KERNEL); if (!mctl->intf_link) { ret = -ENOMEM; goto devnode_remove; @@ -107,7 +109,8 @@ int media_snd_stream_init(struct snd_usb_substream *subs, struct snd_pcm *pcm, case MEDIA_ENT_F_AUDIO_MIXER: ret = media_create_pad_link(entity, mixer_pad, &mctl->media_entity, 0, - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, + GFP_KERNEL); if (ret) goto remove_intf_link; break; @@ -180,7 +183,8 @@ int media_snd_mixer_init(struct snd_usb_audio *chip) if (!ctl_intf) { ctl_intf = media_devnode_create(mdev, intf_type, 0, MAJOR(ctl_dev->devt), - MINOR(ctl_dev->devt)); + MINOR(ctl_dev->devt), + GFP_KERNEL); if (!ctl_intf) return -ENOMEM; chip->ctl_intf_media_devnode = ctl_intf; @@ -213,7 +217,8 @@ int media_snd_mixer_init(struct snd_usb_audio *chip) mctl->intf_link = media_create_intf_link(&mctl->media_entity, &ctl_intf->intf, - MEDIA_LNK_FL_ENABLED); + MEDIA_LNK_FL_ENABLED, + GFP_KERNEL); if (!mctl->intf_link) { media_device_unregister_entity(&mctl->media_entity); media_entity_cleanup(&mctl->media_entity); -- 2.5.0

5 11

[alsa-devel] [PATCH 0/3] ALSA: usb-audio: Fix NULL dereferences via invalid EP accesses
by Takashi Iwai 16 Mar '16

16 Mar '16

Hi, this is a small patchset to address the possible Oops by accessing the invalid EP that was borked by a malformed firmware. One patch is a clean up while two are fixes. Takashi === Takashi Iwai (3): ALSA: usb-audio: Fix NULL dereference in create_fixed_stream_quirk() ALSA: usb-audio: Minor code cleanup in create_fixed_stream_quirk() ALSA: usb-audio: Add sanity checks for endpoint accesses sound/usb/clock.c | 2 ++ sound/usb/endpoint.c | 3 +++ sound/usb/mixer_quirks.c | 4 ++++ sound/usb/pcm.c | 2 ++ sound/usb/quirks.c | 22 ++++++++++++++-------- 5 files changed, 25 insertions(+), 8 deletions(-) -- 2.7.3

1 3

[alsa-devel] pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: snd_pcm_avail() returned a value that is exceptionally large
by Klement Sekera 16 Mar '16

16 Mar '16

Hi devs, I'm experiencing random hick-ups on my Carbon X1 2nd version with a Xonar U3 USB sound card plugged into the onelink dock. When this happens, the whole desktop freezes, whatever was last playing (second or two) loops and after a while (~30seconds) everything recovers.. I see this in my logs: Mar 16 11:37:07 ksekera-arch rtkit-daemon[1207]: The canary thread is apparently starving. Taking action. Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: snd_pcm_avail() returned a value that is exceptionally large: 4194088 bytes (23776 ms). Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: Most likely this is a bug in the ALSA driver 'snd_usb_audio'. Please report this issue to the ALSA develop Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: snd_pcm_dump(): Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: Hardware PCM card 2 'USB Advanced Audio Device' device 0 subdevice 0 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: Its setup is: Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: stream : PLAYBACK Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: access : MMAP_INTERLEAVED Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: format : S16_LE Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: subformat : STD Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: channels : 2 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: rate : 44100 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: exact rate : 44100 (44100/1) Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: msbits : 16 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: buffer_size : 88200 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: period_size : 44100 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: period_time : 1000000 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: tstamp_mode : ENABLE Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: tstamp_type : MONOTONIC Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: period_step : 1 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: avail_min : 87319 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: period_event : 0 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: start_threshold : -1 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: stop_threshold : 6206523236469964800 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: silence_threshold: 0 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: silence_size : 0 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: boundary : 6206523236469964800 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: appl_ptr : 20607492 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: hw_ptr : 21567814 Mar 16 11:37:07 ksekera-arch rtkit-daemon[1207]: Demoting known real-time threads. Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: snd_pcm_delay() returned a value that is exceptionally large: -3674960 bytes (-20833 ms). Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: Most likely this is a bug in the ALSA driver 'snd_usb_audio'. Please report this issue to the ALSA develop Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: snd_pcm_dump(): Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: Hardware PCM card 2 'USB Advanced Audio Device' device 0 subdevice 0 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: Its setup is: Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: stream : PLAYBACK Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: access : MMAP_INTERLEAVED Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: format : S16_LE Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: subformat : STD Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: channels : 2 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: rate : 44100 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: exact rate : 44100 (44100/1) Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: msbits : 16 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: buffer_size : 88200 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: period_size : 44100 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: period_time : 1000000 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: tstamp_mode : ENABLE Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: tstamp_type : MONOTONIC Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: period_step : 1 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: avail_min : 87319 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: period_event : 0 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: start_threshold : -1 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: stop_threshold : 6206523236469964800 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: silence_threshold: 0 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: silence_size : 0 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: boundary : 6206523236469964800 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: appl_ptr : 20653792 Mar 16 11:37:12 ksekera-arch pulseaudio[1203]: E: [alsa-sink-USB Audio] alsa-util.c: hw_ptr : 21572797 Mar 16 11:37:07 ksekera-arch rtkit-daemon[1207]: Successfully demoted thread 1203 of process 1203 (/usr/bin/pulseaudio). Mar 16 11:37:07 ksekera-arch rtkit-daemon[1207]: Demoted 1 threads. I'm on arch linux: ksekera@ksekera-arch ~> cat /proc/asound/version Advanced Linux Sound Architecture Driver Version k4.3.6-1-ck. ksekera@ksekera-arch ~> uname -a Linux ksekera-arch 4.3.6-1-ck #1 SMP PREEMPT Fri Feb 19 20:24:27 EST 2016 x86_64 GNU/Linux Let me know if you need any more info, Regards, Klement

1 0

[alsa-devel] [PATCH] ASoC: dapm: Do not traverse widget hooks to snd-soc-dummy
by Harry Pan 16 Mar '16

16 Mar '16

All components are initially given an empty card when registering platform, and since the commit 6e78108bda78 ("ASoC: core: Don't probe the component which is dummy")', snd-soc-dummy will not be probed so that it remains an empty card assigned. This patch ignores to iterate widget hooks to the 'snd-soc-dummy' component, else it will trigger memory fault because of invalid dereference address of card->widgets. Test by grep -rsI "hello" /sys/devices/platform/skl_nau88l25_ssm4567_i2s/ Conflicts: sound/soc/soc-dapm.c Signed-off-by: Harry Pan <harry.pan(a)intel.com> --- sound/soc/soc-dapm.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/sound/soc/soc-dapm.c b/sound/soc/soc-dapm.c index 581175a..0bc15c9 100644 --- a/sound/soc/soc-dapm.c +++ b/sound/soc/soc-dapm.c @@ -2188,6 +2188,9 @@ static ssize_t dapm_widget_show_component(struct snd_soc_component *cmpnt, int count = 0; char *state = "not set"; + if (!strcmp(cmpnt->name, "snd-soc-dummy")) + return 0; + list_for_each_entry(w, &cmpnt->card->widgets, list) { if (w->dapm != dapm) continue; -- 2.1.2

1 0

[alsa-devel] [PATCH 0/6] ASoC: Intel: Skylake: Fix module removal
by Vinod Koul 16 Mar '16

16 Mar '16

Skyalke driver were crashing on module removal when we run "alsa reload". The first issues was caused by component framwork regression which Russell fixed and in in 4.5. In driver there were still some missing order which needs to be fixed. This series first update the list parsing in core, then ensure proper freeup of code objects, and then esnureing driver does proper cleanup and i915 dependency Vinod Koul (6): ALSA: hda: use list macro for parsing on cleanup ASoC: Intel: Skylake: free codec objects on removal ASoC: Intel: Skylake: Freeup properly on skl_dsp_free ASoC: Intel: Skylake: Unmap the address last ASoC: Intel: Skylake: Call i915 exit last ASoC: Intel: Skylake: remove call to pci_dev_put sound/hda/ext/hdac_ext_stream.c | 5 ++--- sound/soc/intel/skylake/skl-sst-dsp.c | 5 +++++ sound/soc/intel/skylake/skl.c | 22 +++++++++++++++------- 3 files changed, 22 insertions(+), 10 deletions(-) Thanks -- 1.9.1

3 14

[alsa-devel] [PATCH] ASoC: rt5616: add I2C dependency
by Arnd Bergmann 16 Mar '16

16 Mar '16

The rt5616 codec registers itself as an i2c driver, but can be enabled even when i2c is turned off, which leads to a build error: codecs/rt5616.c:1419:1: error: data definition has no type or storage class [-Werror] module_i2c_driver(rt5616_i2c_driver); This adds an explicit Kconfig dependency, like the other codec drivers. Signed-off-by: Arnd Bergmann <arnd(a)arndb.de> Fixes: 288bc356a881 ("ASoC: rt5616: allow to build with CONFIG_SND_SOC_RT5616") --- sound/soc/codecs/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/soc/codecs/Kconfig b/sound/soc/codecs/Kconfig index 649e92a252ae..7ef3a0c16478 100644 --- a/sound/soc/codecs/Kconfig +++ b/sound/soc/codecs/Kconfig @@ -629,6 +629,7 @@ config SND_SOC_RT5514 config SND_SOC_RT5616 tristate "Realtek RT5616 CODEC" + depends on I2C config SND_SOC_RT5631 tristate "Realtek ALC5631/RT5631 CODEC" -- 2.7.0

2 1

[alsa-devel] [PATCH v6] ASoC: imx-wm8958: add imx-wm8958 machine driver
by Zidan Wang 16 Mar '16

16 Mar '16

This is an initial imx-wm8958 device-tree-only machine driver. The sound card driver support three dai link, AIF1-DAI, AIF2-DAI and AIF3-DAI. We can configure the cpu dai from device tree, if present it will create corresponding sound card subdevice. At least one cpu dai phandle should be set from device tree. Signed-off-by: Zidan Wang <zidan.wang(a)freescale.com> Acked-by: Nicolin Chen <nicoleotsuka(a)gmail.com> --- v5->v6: some header files should be moved to the next patch. So remove it. .../devicetree/bindings/sound/imx-audio-wm8958.txt | 69 +++++ sound/soc/fsl/Kconfig | 14 + sound/soc/fsl/Makefile | 2 + sound/soc/fsl/imx-wm8958.c | 325 +++++++++++++++++++++ 4 files changed, 410 insertions(+) create mode 100644 Documentation/devicetree/bindings/sound/imx-audio-wm8958.txt create mode 100644 sound/soc/fsl/imx-wm8958.c diff --git a/Documentation/devicetree/bindings/sound/imx-audio-wm8958.txt b/Documentation/devicetree/bindings/sound/imx-audio-wm8958.txt new file mode 100644 index 0000000..7978c62 --- /dev/null +++ b/Documentation/devicetree/bindings/sound/imx-audio-wm8958.txt @@ -0,0 +1,69 @@ +Freescale i.MX audio complex with WM8958 codec + +Required properties: + + - compatible : "fsl,imx-audio-wm8958" + + - model : The user-visible name of this sound complex + + - audio-codec : The phandle of the WM8958 audio codec + + - audio-routing : A list of the connections between audio components. + Each entry is a pair of strings, the first being + the connection's sink, the second being the + connection's source. Valid names could be power + supplies, WM8958 pins, and the jacks on the board: + + Power supplies: + * MICBIAS1 + * MICBIAS2 + + Board connectors: + * Headphone Jack + * Ext Spk + +Optional properties: + + - cpu-dai1 : The phandle of CPU DAI1 controller, which is for + aif1 of wm8958. At least one cpu dai is required. + + - cpu-dai2 : The phandle of CPU DAI2 controller, which is for + aif2 of wm8958. At least one cpu dai is required. + + - cpu-dai3 : The phandle of CPU DAI3 controller, which is for + aif3 of wm8958. At least one cpu dai is required. + + - dai-link1-name : If present, set the dai link1 name and stream name + to the specific strings. Otherwise, set the dai + link1 name and stream name to "AIF1-DAI". + + - dai-link2-name : If present, set the dai link2 name and stream name + to the specific strings. Otherwise, set the dai + link2 name and stream name to "AIF2-DAI". + + - dai-link3-name : If present, set the dai link3 name and stream name + to the specific strings. Otherwise, set the dai + link3 name and stream name to "AIF3-DAI". + + - fsl,cpu-dai1-master : If present, cpu dai1 works as master, and will + provide bit clock and frame clock. Otherwise, + cpu dai1 works as slave. + +Example: + +sound { + compatible = "fsl,imx6ul-ddr3-arm2-wm8958", + "fsl,imx-audio-wm8958"; + model = "wm8958-audio"; + cpu-dai1 = <&sai1>; + dai-link1-name = "Hifi"; + audio-codec = <&codec>; + audio-routing = + "Headphone Jack", "HPOUT1L", + "Headphone Jack", "HPOUT1R", + "Ext Spk", "SPKOUTLP", + "Ext Spk", "SPKOUTLN", + "Ext Spk", "SPKOUTRP", + "Ext Spk", "SPKOUTRN", + "IN1LN", "MICBIAS2"; +}; diff --git a/sound/soc/fsl/Kconfig b/sound/soc/fsl/Kconfig index 35aabf9..f6e5d96 100644 --- a/sound/soc/fsl/Kconfig +++ b/sound/soc/fsl/Kconfig @@ -229,6 +229,20 @@ config SND_SOC_EUKREA_TLV320 Enable I2S based access to the TLV320AIC23B codec attached to the SSI interface +config SND_SOC_IMX_WM8958 + tristate "SoC Audio support for i.MX boards with wm8958" + depends on OF && I2C + select MFD_WM8994 + select SND_SOC_WM8994 + select SND_SOC_IMX_PCM_DMA + select SND_SOC_FSL_SAI + select SND_SOC_FSL_UTILS + select SND_KCTL_JACK + help + SoC Audio support for i.MX boards with WM8958 + Say Y if you want to add support for SoC audio on an i.MX board with + a wm8958 codec. + config SND_SOC_IMX_WM8962 tristate "SoC Audio support for i.MX boards with wm8962" depends on OF && I2C && INPUT diff --git a/sound/soc/fsl/Makefile b/sound/soc/fsl/Makefile index d28dc25..2a781c0 100644 --- a/sound/soc/fsl/Makefile +++ b/sound/soc/fsl/Makefile @@ -54,6 +54,7 @@ snd-soc-mx27vis-aic32x4-objs := mx27vis-aic32x4.o snd-soc-wm1133-ev1-objs := wm1133-ev1.o snd-soc-imx-es8328-objs := imx-es8328.o snd-soc-imx-sgtl5000-objs := imx-sgtl5000.o +snd-soc-imx-wm8958-objs := imx-wm8958.o snd-soc-imx-wm8962-objs := imx-wm8962.o snd-soc-imx-spdif-objs := imx-spdif.o snd-soc-imx-mc13783-objs := imx-mc13783.o @@ -64,6 +65,7 @@ obj-$(CONFIG_SND_SOC_MX27VIS_AIC32X4) += snd-soc-mx27vis-aic32x4.o obj-$(CONFIG_SND_MXC_SOC_WM1133_EV1) += snd-soc-wm1133-ev1.o obj-$(CONFIG_SND_SOC_IMX_ES8328) += snd-soc-imx-es8328.o obj-$(CONFIG_SND_SOC_IMX_SGTL5000) += snd-soc-imx-sgtl5000.o +obj-$(CONFIG_SND_SOC_IMX_WM8958) += snd-soc-imx-wm8958.o obj-$(CONFIG_SND_SOC_IMX_WM8962) += snd-soc-imx-wm8962.o obj-$(CONFIG_SND_SOC_IMX_SPDIF) += snd-soc-imx-spdif.o obj-$(CONFIG_SND_SOC_IMX_MC13783) += snd-soc-imx-mc13783.o diff --git a/sound/soc/fsl/imx-wm8958.c b/sound/soc/fsl/imx-wm8958.c new file mode 100644 index 0000000..5ac200e --- /dev/null +++ b/sound/soc/fsl/imx-wm8958.c @@ -0,0 +1,325 @@ +/* + * Copyright (C) 2015-2016 Freescale Semiconductor, Inc. + * + * The code contained herein is licensed under the GNU General Public + * License. You may obtain a copy of the GNU General Public License + * Version 2 or later at the following locations: + * + * http://www.opensource.org/licenses/gpl-license.html + * http://www.gnu.org/copyleft/gpl.html + */ + +#include <linux/module.h> +#include <linux/of_platform.h> +#include <linux/i2c.h> +#include <linux/clk.h> +#include <sound/soc.h> +#include <sound/control.h> +#include <sound/pcm_params.h> +#include <sound/soc-dapm.h> +#include <linux/mfd/wm8994/registers.h> +#include "../codecs/wm8994.h" + +#define DAI_NAME_SIZE 32 + +#define DAI_LINK_NUM (3) +#define AIF1_DAI (0) +#define AIF2_DAI (1) +#define AIF3_DAI (2) + +#define WM8958_MCLK_MAX (2) + +#define WM8994_FLL(id) (id == AIF1_DAI ? WM8994_FLL1 : WM8994_FLL2) +#define WM8994_SYSCLK_FLL(id) (id == AIF1_DAI ? WM8994_SYSCLK_FLL1 : WM8994_SYSCLK_FLL2) +#define WM8994_FLL_SRC_MCLK(id) (id == AIF1_DAI ? WM8994_FLL_SRC_MCLK1 : WM8994_FLL_SRC_MCLK2) + +struct imx_wm8958_data { + struct snd_soc_dai_link dai_link[DAI_LINK_NUM]; + struct snd_soc_card card; + unsigned long mclk_freq[WM8958_MCLK_MAX]; + bool is_cpu_dai1_master; + bool is_stream_in_use[DAI_LINK_NUM][2]; +}; + +static const struct snd_soc_dapm_widget imx_wm8958_dapm_widgets[] = { + SND_SOC_DAPM_HP("Headphone Jack", NULL), + SND_SOC_DAPM_SPK("Ext Spk", NULL), +}; + +static int imx_wm8958_hw_params(struct snd_pcm_substream *substream, + struct snd_pcm_hw_params *params) +{ + struct snd_soc_pcm_runtime *rtd = substream->private_data; + struct snd_soc_dai *codec_dai = rtd->codec_dai; + struct snd_soc_dai *cpu_dai = rtd->cpu_dai; + struct snd_soc_card *card = rtd->card; + struct device *dev = card->dev; + struct imx_wm8958_data *data = snd_soc_card_get_drvdata(card); + bool tx = substream->stream == SNDRV_PCM_STREAM_PLAYBACK; + bool hifi_dai_sysclk_dir = SND_SOC_CLOCK_OUT; + u32 mclk_id, id = codec_dai->id - 1; + u32 pll_out; + int ret; + + data->is_stream_in_use[id][tx] = true; + + if (data->mclk_freq[id]) + mclk_id = WM8994_FLL_SRC_MCLK(id); + else if (id == AIF1_DAI) + mclk_id = WM8994_FLL_SRC_MCLK2; + else + mclk_id = WM8994_FLL_SRC_MCLK1; + + if (id == AIF1_DAI) { + if (data->is_cpu_dai1_master) + hifi_dai_sysclk_dir = SND_SOC_CLOCK_IN; + + ret = snd_soc_dai_set_sysclk(cpu_dai, + 0, 0, !hifi_dai_sysclk_dir); + + if (ret) { + dev_err(dev, "failed to set cpu sysclk: %d\n", ret); + return ret; + } + + if (data->is_cpu_dai1_master) { + ret = snd_soc_dai_set_sysclk(codec_dai, mclk_id, + data->mclk_freq[mclk_id - 1], + hifi_dai_sysclk_dir); + if (ret) { + dev_err(dev, + "failed to set codec sysclk: %d\n", + ret); + return ret; + } + + return 0; + } + } + + if (params_width(params) == 24) + pll_out = params_rate(params) * 384; + else + pll_out = params_rate(params) * 256; + + ret = snd_soc_dai_set_pll(codec_dai, + WM8994_FLL(id), + mclk_id, + data->mclk_freq[mclk_id - 1], + pll_out); + if (ret) { + dev_err(dev, "failed to set codec pll: %d\n", ret); + return ret; + } + + ret = snd_soc_dai_set_sysclk(codec_dai, + WM8994_SYSCLK_FLL(id), + pll_out, + SND_SOC_CLOCK_OUT); + if (ret) { + dev_err(dev, "failed to set codec sysclk: %d\n", ret); + return ret; + } + + return 0; +} + +static int imx_wm8958_hw_free(struct snd_pcm_substream *substream) +{ + struct snd_soc_pcm_runtime *rtd = substream->private_data; + struct snd_soc_dai *codec_dai = rtd->codec_dai; + struct snd_soc_card *card = rtd->card; + struct imx_wm8958_data *data = snd_soc_card_get_drvdata(card); + bool tx = substream->stream == SNDRV_PCM_STREAM_PLAYBACK; + int id = codec_dai->id - 1; + + data->is_stream_in_use[id][tx] = false; + + if (id == AIF1_DAI && data->is_cpu_dai1_master) + return 0; + + if (!data->is_stream_in_use[id][!tx]) { + /* + * We should connect AIFxCLK source to FLL after enable FLL, + * and disconnect AIF1CLK source to FLL before disable FLL, + * otherwise FLL worked abnormal. + */ + snd_soc_dai_set_sysclk(codec_dai, WM8994_FLL_SRC_MCLK(id), + data->mclk_freq[id], SND_SOC_CLOCK_OUT); + + /* Disable FLL1 after all stream finished. */ + snd_soc_dai_set_pll(codec_dai, WM8994_FLL(id), 0, 0, 0); + } + + return 0; +} + +static struct snd_soc_ops imx_wm8958_ops = { + .hw_params = imx_wm8958_hw_params, + .hw_free = imx_wm8958_hw_free, +}; + +static struct snd_soc_dai_link imx_wm8958_dai_link[] = { + [AIF1_DAI] = { + .name = "AIF1-DAI", + .stream_name = "AIF1-DAI", + .codec_name = "wm8994-codec", + .codec_dai_name = "wm8994-aif1", + .ops = &imx_wm8958_ops, + .dai_fmt = SND_SOC_DAIFMT_I2S | + SND_SOC_DAIFMT_NB_NF, + }, + [AIF2_DAI] = { + .name = "AIF2-DAI", + .stream_name = "AIF2-DAI", + .codec_name = "wm8994-codec", + .codec_dai_name = "wm8994-aif2", + .ops = &imx_wm8958_ops, + .dai_fmt = SND_SOC_DAIFMT_I2S | + SND_SOC_DAIFMT_NB_NF | + SND_SOC_DAIFMT_CBM_CFM, + }, + [AIF3_DAI] = { + .name = "AIF3-DAI", + .stream_name = "AIF3-DAI", + .codec_name = "wm8994-codec", + .codec_dai_name = "wm8994-aif3", + }, +}; + +static int imx_wm8958_probe(struct platform_device *pdev) +{ + struct device_node *cpu_np[DAI_LINK_NUM], *codec_np; + struct device_node *np = pdev->dev.of_node; + struct platform_device *cpu_pdev[DAI_LINK_NUM]; + struct i2c_client *codec_dev; + struct imx_wm8958_data *data; + struct clk *mclk; + char tmp[8], *name; + int num_links = 0; + int ret, i; + + data = devm_kzalloc(&pdev->dev, sizeof(*data), GFP_KERNEL); + if (!data) + return -ENOMEM; + + for (i = 0; i < DAI_LINK_NUM; i++) { + sprintf(tmp, "cpu-dai%d", i + 1); + cpu_np[i] = of_parse_phandle(np, tmp, 0); + if (!cpu_np[i]) + continue; + + cpu_pdev[i] = of_find_device_by_node(cpu_np[i]); + if (!cpu_pdev[i]) { + dev_err(&pdev->dev, + "failed to get cpu dai%d platform device\n", i); + ret = -EINVAL; + goto fail; + } + data->dai_link[num_links] = imx_wm8958_dai_link[i]; + data->dai_link[num_links].cpu_dai_name = dev_name(&cpu_pdev[i]->dev); + data->dai_link[num_links].platform_of_node = cpu_np[i]; + + sprintf(tmp, "dai-link%d-name", i + 1); + if(!of_property_read_string(np, tmp, (const char **)&name)) { + data->dai_link[num_links].name = name; + data->dai_link[num_links].stream_name = name; + } + num_links++; + } + + if (!num_links) { + dev_err(&pdev->dev, "cpu dai phandle missing or invalid\n"); + ret = -EINVAL; + goto fail; + } + + codec_np = of_parse_phandle(np, "audio-codec", 0); + if (!codec_np) { + dev_err(&pdev->dev, "phandle missing or invalid\n"); + ret = -EINVAL; + goto fail; + } + + codec_dev = of_find_i2c_device_by_node(codec_np); + if (!codec_dev || !codec_dev->dev.driver) { + dev_err(&pdev->dev, "failed to find codec platform device\n"); + ret = -EINVAL; + goto fail; + } + + /* cpu dai1 link support cpu dai master and slave mode */ + if (of_property_read_bool(np, "fsl,cpu-dai1-master")) { + data->is_cpu_dai1_master = true; + data->dai_link[AIF1_DAI].dai_fmt |= SND_SOC_DAIFMT_CBS_CFS; + } else { + data->is_cpu_dai1_master = false; + data->dai_link[AIF1_DAI].dai_fmt |= SND_SOC_DAIFMT_CBM_CFM; + } + + for (i = 0; i < WM8958_MCLK_MAX; i++) { + sprintf(tmp, "MCLK%d", i + 1); + mclk = devm_clk_get(&codec_dev->dev, tmp); + if (!IS_ERR(mclk)) + data->mclk_freq[i] = clk_get_rate(mclk); + } + + if (!data->mclk_freq[0] && !data->mclk_freq[1]) { + dev_err(&pdev->dev, "failed to get mclk clock\n"); + ret = -EINVAL; + goto fail; + } + + data->card.dev = &pdev->dev; + ret = snd_soc_of_parse_card_name(&data->card, "model"); + if (ret) + goto fail; + + data->card.num_links = num_links; + data->card.dai_link = data->dai_link; + data->card.dapm_widgets = imx_wm8958_dapm_widgets; + data->card.num_dapm_widgets = ARRAY_SIZE(imx_wm8958_dapm_widgets); + data->card.owner = THIS_MODULE; + + ret = snd_soc_of_parse_audio_routing(&data->card, "audio-routing"); + if (ret) + goto fail; + + platform_set_drvdata(pdev, &data->card); + snd_soc_card_set_drvdata(&data->card, data); + + ret = devm_snd_soc_register_card(&pdev->dev, &data->card); + if (ret) { + dev_err(&pdev->dev, "snd_soc_register_card failed (%d)\n", ret); + goto fail; + } + +fail: + of_node_put(cpu_np[AIF1_DAI]); + of_node_put(cpu_np[AIF2_DAI]); + of_node_put(cpu_np[AIF3_DAI]); + of_node_put(codec_np); + + return ret; +} + +static const struct of_device_id imx_wm8958_dt_ids[] = { + { .compatible = "fsl,imx-audio-wm8958", }, + { /* sentinel */ } +}; +MODULE_DEVICE_TABLE(of, imx_wm8958_dt_ids); + +static struct platform_driver imx_wm8958_driver = { + .driver = { + .name = "imx-wm8958", + .pm = &snd_soc_pm_ops, + .of_match_table = imx_wm8958_dt_ids, + }, + .probe = imx_wm8958_probe, +}; +module_platform_driver(imx_wm8958_driver); + +MODULE_AUTHOR("Freescale Semiconductor, Inc."); +MODULE_DESCRIPTION("Freescale i.MX WM8958 ASoC machine driver"); +MODULE_LICENSE("GPL v2"); +MODULE_ALIAS("platform:imx-wm8958"); -- 1.9.1

1 0

[alsa-devel] [PATCH] ASoC: cs35l32: prevent unexpected parse result of device tree
by Takashi Sakamoto 16 Mar '16

16 Mar '16

Hi, When compiling recent kernel, I got a warning caused by a codec in ASoC. sound/soc/codecs/cs35l32.c: In function ‘cs35l32_i2c_probe’: sound/soc/codecs/cs35l32.c:278:2: warning: ‘val’ may be used uninitialized in this function [-Wmaybe-uninitialized] switch (val) { ^ sound/soc/codecs/cs35l32.c:272:15: note: ‘val’ was declared here unsigned int val; ^ I'm not an ASoC developer, furthermore, have no knowledge about this Cirrus codec. Although, as a quick glance it may include a logical bug to assign wrong information to structure member as a reult of parsing device tree. This patch is my proposal to fix it. I'm happy to receive your comments. Regards Takashi Sakamoto (1): ASoC: cs35l32: fix unexpected parse result of device tree blob sound/soc/codecs/cs35l32.c | 4 ++++ 1 file changed, 4 insertions(+) -- 2.7.0

2 3

[alsa-devel] [PATCH] ASoC: nau8825: fix interrupt fails and unstable after resume
by John Hsu 16 Mar '16

16 Mar '16

This patch is to fix interrupt fails when resume back. In solution, we add IRQ re-initiation, and reconstruct suspend and resume with set_bias_level function. Signed-off-by: John Hsu <KCHSU0(a)nuvoton.com> --- sound/soc/codecs/nau8825.c | 42 ++++++++++++++++++++++++++++++++++++------ 1 file changed, 36 insertions(+), 6 deletions(-) diff --git a/sound/soc/codecs/nau8825.c b/sound/soc/codecs/nau8825.c index 504c969..2e2e11a 100644 --- a/sound/soc/codecs/nau8825.c +++ b/sound/soc/codecs/nau8825.c @@ -1092,6 +1092,36 @@ static int nau8825_set_sysclk(struct snd_soc_codec *codec, int clk_id, return nau8825_configure_sysclk(nau8825, clk_id, freq); } +static int nau8825_resume_setup(struct nau8825 *nau8825) +{ + struct regmap *regmap = nau8825->regmap; + + /* IRQ Output Enable */ + regmap_update_bits(regmap, NAU8825_REG_INTERRUPT_MASK, + NAU8825_IRQ_OUTPUT_EN, NAU8825_IRQ_OUTPUT_EN); + + /* Enable internal VCO needed for interruptions */ + nau8825_configure_sysclk(nau8825, NAU8825_CLK_INTERNAL, 0); + + /* Enable DDACR needed for interrupts */ + regmap_update_bits(regmap, NAU8825_REG_ENA_CTRL, + NAU8825_ENABLE_DACR, NAU8825_ENABLE_DACR); + + /* Chip needs one FSCLK cycle in order to generate interrupts, + * as we cannot guarantee one will be provided by the system. Turning + * master mode on then off enables us to generate that FSCLK cycle + * with a minimum of contention on the clock bus. + */ + regmap_update_bits(regmap, NAU8825_REG_I2S_PCM_CTRL2, + NAU8825_I2S_MS_MASK, NAU8825_I2S_MS_MASTER); + regmap_update_bits(regmap, NAU8825_REG_I2S_PCM_CTRL2, + NAU8825_I2S_MS_MASK, NAU8825_I2S_MS_SLAVE); + + nau8825_restart_jack_detection(regmap); + + return 0; +} + static int nau8825_set_bias_level(struct snd_soc_codec *codec, enum snd_soc_bias_level level) { @@ -1121,6 +1151,8 @@ static int nau8825_set_bias_level(struct snd_soc_codec *codec, "Failed to sync cache: %d\n", ret); return ret; } + if (nau8825->irq) + nau8825_resume_setup(nau8825); } break; @@ -1330,24 +1362,22 @@ static int nau8825_i2c_remove(struct i2c_client *client) #ifdef CONFIG_PM_SLEEP static int nau8825_suspend(struct device *dev) { - struct i2c_client *client = to_i2c_client(dev); struct nau8825 *nau8825 = dev_get_drvdata(dev); + struct snd_soc_codec *codec = snd_soc_dapm_to_codec(nau8825->dapm); - disable_irq(client->irq); + disable_irq(nau8825->irq); + snd_soc_codec_force_bias_level(codec, SND_SOC_BIAS_OFF); regcache_cache_only(nau8825->regmap, true); - regcache_mark_dirty(nau8825->regmap); return 0; } static int nau8825_resume(struct device *dev) { - struct i2c_client *client = to_i2c_client(dev); struct nau8825 *nau8825 = dev_get_drvdata(dev); regcache_cache_only(nau8825->regmap, false); - regcache_sync(nau8825->regmap); - enable_irq(client->irq); + enable_irq(nau8825->irq); return 0; } -- 2.6.4

2 8