From patchwork Mon Feb 18 15:25:00 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Viorel Suman X-Patchwork-Id: 10818391 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id AD6711399 for ; Mon, 18 Feb 2019 15:25:23 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 95F9D2A09F for ; Mon, 18 Feb 2019 15:25:23 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 89CD92ABF6; Mon, 18 Feb 2019 15:25:23 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.7 required=2.0 tests=BAD_ENC_HEADER,BAYES_00, DKIM_INVALID,DKIM_SIGNED,MAILING_LIST_MULTI autolearn=ham version=3.3.1 Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id E96B52A09F for ; Mon, 18 Feb 2019 15:25:08 +0000 (UTC) Received: from alsa0.perex.cz (localhost [127.0.0.1]) by alsa0.perex.cz (Postfix) with ESMTP id 31893265525; Mon, 18 Feb 2019 16:25:07 +0100 (CET) X-Original-To: alsa-devel@alsa-project.org Delivered-To: alsa-devel@alsa-project.org Received: by alsa0.perex.cz (Postfix, from userid 1000) id E8677265527; Mon, 18 Feb 2019 16:25:04 +0100 (CET) Received: from EUR03-AM5-obe.outbound.protection.outlook.com (mail-eopbgr30074.outbound.protection.outlook.com [40.107.3.74]) by alsa0.perex.cz (Postfix) with ESMTP id B423D265523 for ; Mon, 18 Feb 2019 16:25:01 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=qWKPaf2jF/kciS0KBtUJjSvvYpofoLLLsHtskMy7AOY=; b=RC4/vMbcRM0b7BUF+Cbfq/s0S5xBpm5WfH6KVOiVBV5OdpkI4b6kEl3KO1kaQqwXwbWuo52weKhiaLGMEbR6UfOfPFo0RPduUE0t9lUBpg0LfRGC92xit3ktU8Wkw+6psL9rY7m9b0RmJfKiSDW7K02BE9IM+7BBqYmKbfM7X6U= Received: from AM6PR04MB4693.eurprd04.prod.outlook.com (20.177.33.222) by AM6PR04MB4533.eurprd04.prod.outlook.com (20.177.38.21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1622.19; Mon, 18 Feb 2019 15:25:00 +0000 Received: from AM6PR04MB4693.eurprd04.prod.outlook.com ([fe80::78b6:cae3:86df:a936]) by AM6PR04MB4693.eurprd04.prod.outlook.com ([fe80::78b6:cae3:86df:a936%4]) with mapi id 15.20.1622.018; Mon, 18 Feb 2019 15:25:00 +0000 From: Viorel Suman To: Timur Tabi , Nicolin Chen , Xiubo Li , Fabio Estevam Thread-Topic: [PATCH] ASoC: fsl_spdif: fix sysclk_df type Thread-Index: AQHUx54cV0e+cjZqWUucEsNTrxgp8Q== Date: Mon, 18 Feb 2019 15:25:00 +0000 Message-ID: <1550503474-18865-1-git-send-email-viorel.suman@nxp.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-clientproxiedby: AM5P189CA0013.EURP189.PROD.OUTLOOK.COM (2603:10a6:206:15::26) To AM6PR04MB4693.eurprd04.prod.outlook.com (2603:10a6:20b:7::30) authentication-results: spf=none (sender IP is ) smtp.mailfrom=viorel.suman@nxp.com; x-ms-exchange-messagesentrepresentingtype: 1 x-mailer: git-send-email 2.7.4 x-originating-ip: [95.76.156.53] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: cfca4032-1908-4cac-ef41-08d695b53f1a x-ms-office365-filtering-ht: Tenant x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600110)(711020)(4605104)(4618075)(2017052603328)(7153060)(7193020); SRVR:AM6PR04MB4533; x-ms-traffictypediagnostic: AM6PR04MB4533: x-microsoft-exchange-diagnostics: =?iso-8859-1?Q?1; AM6PR04MB4533; 23:0tKOhhWx3KQ6/7xkBy7/0F4KkCr4NS10Ca646U1?= =?iso-8859-1?q?5yq53MEi5VmPQuD?= =?iso-8859-1?q?YFrx4jAu1TyNAse6he3rAsnU/1JtUL53mfbz6h/jtMK7IbWmLS48taqS2cJO?= =?iso-8859-1?q?loqFmGxkUeKOV2Zs4TroQsqPPQHIpagBD2nTa9zm+peDiuz1o2KhKYksq3F3?= =?iso-8859-1?q?CshbOOMxCnbICwcf8zwn0IcDwUTKp0IWRtCOzNoEMZMR+vBfLwVMaBJ+npvt?= =?iso-8859-1?q?YI/Ovepgy2UAw5ePnkFs7s7byBqtGollY0Jl3BYmvXtfWFTldtNvO7b58Ri6?= =?iso-8859-1?q?j/MmiTuqFAW/XQPZmZomXkN/0TVDb6zx9CWKoeMmUUQuZUOaE6tY+2EKYxQj?= =?iso-8859-1?q?bQiOH0fVnhzfKWxf7zelS/bEzLaqIRMsyM+FQCyiHw0XX5ArA77tGJf4jBH6?= =?iso-8859-1?q?Awi4o0ZXKCBn7CEjlywogDemE9exjNxwnn5WIUOn7K/oZwqs/ugqUXho7BnC?= =?iso-8859-1?q?RtK0JPIE9Y4yMwkfBrXfMSUNwWRRcalUzs1kx74/SvrpGMjO0R2CCL51NKd2?= =?iso-8859-1?q?cuvzzp/m1TUTbNv2uMU2r8w8gbyaqBS3/C+pyXyvGAq4vuuV5eaQl7B5r6zi?= =?iso-8859-1?q?/3FHbuQ/9XCNdz3sPnmgABoz6xRZfImu4Fe5TCwyqIE0GbkIWS78rohQ/HVR?= =?iso-8859-1?q?zjNAWTjgmjmGz/TlG2RNzGQ/tX7WFxsIIfxU6rOYtOUsB9euaj11ycN2vM8T?= =?iso-8859-1?q?D+FO6dbULqxGg+xnhmxgw+hWUdZcNPsmMM0Z4scrUOq/cN+5WTr5mqg4JCLC?= =?iso-8859-1?q?NOLp/XBYqF2Qyyjy9BqKD/tUVtfnNTnwVc6NFl1XKpw5cO+R8qP8bdtPYc04?= =?iso-8859-1?q?lpDQy8Shq7ZBxpBJlbPD/5z1YYc7nwx5e120duV3FVcAiFEdmrb70aAB8b7e?= =?iso-8859-1?q?oBrAdyfozP5zbyDNvWQG6p/103/DRSrpjEIqTghnyBAxd2iVQSwRpY+rtuYh?= =?iso-8859-1?q?nLv+QHHTikbVcuotneF6nQm+4zfwjCiNH7MlLULjMbC0yWnZ2bOCLLwtxU4T?= =?iso-8859-1?q?wgMgnq2DOSDGqlY2MrhpoFBb5VIMzze7t3Bp6ZWKyy+nGbpHqCqMSPBT1UVk?= =?iso-8859-1?q?QKcp05DcwxTbxoG7vrvqnPz9s/i2M6Frj8PBFilmY1Y8HZm61IxGay6zMQXX?= =?iso-8859-1?q?KR7nur0KiEAdHMF+Bc93Oss6ytkOGN77MGJFLI+61hMhIQgDpEqdIMn6oEy0?= =?iso-8859-1?q?I=3D?= x-microsoft-antispam-prvs: x-forefront-prvs: 09525C61DB x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(39860400002)(136003)(346002)(376002)(366004)(396003)(189003)(199004)(6512007)(2616005)(5660300002)(8676002)(36756003)(26005)(81156014)(7416002)(8936002)(81166006)(86362001)(25786009)(7736002)(4326008)(2906002)(305945005)(476003)(486006)(97736004)(53936002)(14444005)(110136005)(50226002)(256004)(6436002)(66066001)(99286004)(68736007)(6486002)(52116002)(186003)(6506007)(71200400001)(54906003)(386003)(478600001)(14454004)(44832011)(316002)(106356001)(105586002)(6116002)(3846002)(102836004)(71190400001); DIR:OUT; SFP:1101; SCL:1; SRVR:AM6PR04MB4533; H:AM6PR04MB4693.eurprd04.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1; received-spf: None (protection.outlook.com: nxp.com does not designate permitted sender hosts) x-ms-exchange-senderadcheck: 1 x-microsoft-antispam-message-info: pSxKWc02S85XakGHFhnFONMpx+PbALMoLXwW4+rworsWz8OiQcIURnjH5L9l8c5/NEJDQxsbPm/cELca2Xr13Bcp8FnOmq8zNIui9skybTpQb7Hc9YB2sNv9JeeHdjNqCq8TJHY9Kbu9Lan7UrM48yBaGYyF8KCC+zi3K4hsyyjz3EshssQF4Qmj8WIvKsKrW3KdVXItAAEXy45BCzKxJ+Hix8w+cPFTs2SYnRIa4CYRErGmn6zE4WGS4i1eaYW3KAz/yq2bjWv0ttCg4YGmy5bGMS+9NtBsszX/vBGwdsBn2USPm++Jbn7xh9Br4Qz/DCD6C07lwCUmygNHzascBTgd8MVWrgidZ6OXiDTpykfPf4UyZVCdNEZB5TOpao32Urnqct0BChTK980+yh1Y+IGIPk1xI30/ZkCzBczQ6uU= MIME-Version: 1.0 X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: cfca4032-1908-4cac-ef41-08d695b53f1a X-MS-Exchange-CrossTenant-originalarrivaltime: 18 Feb 2019 15:24:59.2557 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM6PR04MB4533 Cc: "alsa-devel@alsa-project.org" , "linux-kernel@vger.kernel.org" , Takashi Iwai , Liam Girdwood , Viorel Suman , Mark Brown , dl-linux-imx , "linuxppc-dev@lists.ozlabs.org" Subject: [alsa-devel] [PATCH] ASoC: fsl_spdif: fix sysclk_df type X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org X-Virus-Scanned: ClamAV using ClamSMTP According to RM SPDIF STC SYSCLK_DF field is 9-bit wide, values being in 0..511 range. Use a proper type to handle sysclk_df. Signed-off-by: Viorel Suman Acked-by: Nicolin Chen --- sound/soc/fsl/fsl_spdif.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/sound/soc/fsl/fsl_spdif.c b/sound/soc/fsl/fsl_spdif.c index a26686e..4842e6d 100644 --- a/sound/soc/fsl/fsl_spdif.c +++ b/sound/soc/fsl/fsl_spdif.c @@ -96,7 +96,7 @@ struct fsl_spdif_priv { bool dpll_locked; u32 txrate[SPDIF_TXRATE_MAX]; u8 txclk_df[SPDIF_TXRATE_MAX]; - u8 sysclk_df[SPDIF_TXRATE_MAX]; + u16 sysclk_df[SPDIF_TXRATE_MAX]; u8 txclk_src[SPDIF_TXRATE_MAX]; u8 rxclk_src; struct clk *txclk[SPDIF_TXRATE_MAX]; @@ -376,7 +376,8 @@ static int spdif_set_sample_rate(struct snd_pcm_substream *substream, struct platform_device *pdev = spdif_priv->pdev; unsigned long csfs = 0; u32 stc, mask, rate; - u8 clk, txclk_df, sysclk_df; + u16 sysclk_df; + u8 clk, txclk_df; int ret; switch (sample_rate) { @@ -1109,8 +1110,9 @@ static u32 fsl_spdif_txclk_caldiv(struct fsl_spdif_priv *spdif_priv, static const u32 rate[] = { 32000, 44100, 48000, 96000, 192000 }; bool is_sysclk = clk_is_match(clk, spdif_priv->sysclk); u64 rate_ideal, rate_actual, sub; - u32 sysclk_dfmin, sysclk_dfmax; - u32 txclk_df, sysclk_df, arate; + u32 arate; + u16 sysclk_dfmin, sysclk_dfmax, sysclk_df; + u8 txclk_df; /* The sysclk has an extra divisor [2, 512] */ sysclk_dfmin = is_sysclk ? 2 : 1;