From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id BE52AC433FE for ; Mon, 23 May 2022 14:53:49 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=ip0QEPf6ohJDb2xNjrtiM7vDx/pmBL8OQ7jB0dv1zj4=; b=Np8RDSHBPb2Pqu KUkgxL4JKA1/lIYzjTm1O8tntwO81IUNdU7fhLSjxMU1CrimFHB8FGnAqzgwvpDaeqHqDu5olNDGv J2K9b0ge46b4/gOyOdTH+4Zl4703OjXNg3VqjbbOrjk8UqqLpfQJC+2sdX5Dfg82oQkZMsp80aqZI au01728Yj00Qa6vsMgJ8VQkgchWY02cm4hMaU5FMtIHAZBbn2JEtvEgcdHRFgoaIP+yhNgm4mo4dI +suELe2xBV+F3cuiK3KyvhlYJZUG4pY6Eb7HZsbk3ukhGIx/v93omvaCH49YM0iWITQtcp8eKa0tD QA4vuiivtWLlCk99gsUg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nt9Q8-004n4h-AP; Mon, 23 May 2022 14:52:32 +0000 Received: from mail-wr1-x431.google.com ([2a00:1450:4864:20::431]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nt9Q4-004n1g-6B for linux-arm-kernel@lists.infradead.org; Mon, 23 May 2022 14:52:29 +0000 Received: by mail-wr1-x431.google.com with SMTP id e28so21136698wra.10 for ; Mon, 23 May 2022 07:52:26 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:content-transfer-encoding:in-reply-to; bh=WM2w2iSrfnCpI7JacIA3BcUph/2ijVWcwtEgBCcbpIc=; b=r7xEqlLCksQ1soquqg6YMFdzig2CSZJJ1ZRaPdQchwXYnn4/u92ZxMh2tX3eTolH+q 1bm4OVGSNSyKP0xWNjtUUNO3BvDCCjxhoDGj3/lo2vkzrUD4ymuu9l9s3M/r+3M8cG25 oof5UFhLCXyem8IV14F126PePSViDheuZ5U4iveFPIRfvwVa38CkpgHlw5371o++fVk6 k6GUYWa3/z0YAXUH/07TG6G7HCR0OxWjQlTerJ4VYF8gc/Ia3lnIZDNVgp2NfyYKY6pI AvkA5sauIbX+omqXhaeSv7+9U52ciaa4xutw4PmMl62qN4OKihtCHnexWHgbnh4UGKmH Ep2A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to; bh=WM2w2iSrfnCpI7JacIA3BcUph/2ijVWcwtEgBCcbpIc=; b=fOIBEAMlzXgYdlTyamxkCy+LchFqs4EB8QsDHmdtwIfmPKmIsNc/yTfXUaXBCKQxxa 7j/IVzsfub5I5nl744RTp1T8trQrPZG9A65+bVYwm89ulDhn4OaNHM7bPQOfHaolzGuF K+J9XxzPCp9ACoEBTkBtfBe2+Ov0MERLUm0suHHSXjrBfxdNJ9+rAeo27pdIyXjO3niz XOyNyZGYEnt6gIR5xeeleEVPHMNA3DQ92u//2EbmPsbSifXc/wjCfzfUv3B0wAJ7o7tw PJsTYcDfCHmBHUNBm0VrZ1syuZQJdsrJSJQa+Z2XGVsH9Jhee5U0AG1TQwZfNEcs1w+k 6DTw== X-Gm-Message-State: AOAM532dFMjjo8MxAUMqMXbdTXw7PX+HOglLEgrPJOjQVkhRjTaU7aiR O3mrkhq8GlL8K+LtwbRgytQYjg== X-Google-Smtp-Source: ABdhPJx8g7hX75odBmTIOpI6lWQKWgpFUQOsgvhtxc8fxWoHcWpwS5f35ULO7SpybNMKCN12jx3Kqw== X-Received: by 2002:a05:6000:1a88:b0:20e:6eb4:124e with SMTP id f8-20020a0560001a8800b0020e6eb4124emr18015038wry.8.1653317545273; Mon, 23 May 2022 07:52:25 -0700 (PDT) Received: from google.com (cpc155339-bagu17-2-0-cust87.1-3.cable.virginm.net. [86.27.177.88]) by smtp.gmail.com with ESMTPSA id c21-20020a05600c149500b0039750c29cf7sm1758385wmh.40.2022.05.23.07.52.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 23 May 2022 07:52:24 -0700 (PDT) Date: Mon, 23 May 2022 15:52:22 +0100 From: Lee Jones To: Kavyasree Kotagiri Subject: Re: [PATCH v2 4/4] mux: lan966: Add support for flexcom mux controller Message-ID: References: <20220509084920.14529-1-kavyasree.kotagiri@microchip.com> <20220509084920.14529-5-kavyasree.kotagiri@microchip.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20220509084920.14529-5-kavyasree.kotagiri@microchip.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220523_075228_285969_4A9F4D00 X-CRM114-Status: GOOD ( 31.32 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, alexandre.belloni@bootlin.com, linux-kernel@vger.kernel.org, claudiu.beznea@microchip.com, krzysztof.kozlowski+dt@linaro.org, Manohar.Puri@microchip.com, linux@armlinux.org.uk, UNGLinuxDriver@microchip.com, peda@axentia.se, linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org T24gTW9uLCAwOSBNYXkgMjAyMiwgS2F2eWFzcmVlIEtvdGFnaXJpIHdyb3RlOgoKPiBMQU45NjYg U29DIGhhdmUgNSBmbGV4Y29tcy4gRWFjaCBmbGV4Y29tIGhhcyAyIGNoaXAtc2VsZWN0cy4KPiBG b3IgZWFjaCBjaGlwIHNlbGVjdCBvZiBlYWNoIGZsZXhjb20gdGhlcmUgaXMgYSBjb25maWd1cmF0 aW9uCj4gcmVnaXN0ZXIgRkxFWENPTV9TSEFSRURbMC00XTpTU19NQVNLWzAtMV0uIFRoZSB3aWR0 aCBvZgo+IGNvbmZpZ3VyYXRpb24gcmVnaXN0ZXIgaXMgMjEgYmVjYXVzZSB0aGVyZSBhcmUgMjEg c2hhcmVkIHBpbnMKPiBvbiBlYWNoIG9mIHdoaWNoIHRoZSBjaGlwIHNlbGVjdCBjYW4gYmUgbWFw cGVkLiBFYWNoIGJpdCBvZiB0aGUKPiByZWdpc3RlciByZXByZXNlbnRzIGEgZGlmZmVyZW50IEZM RVhDT01fU0hBUkVEIHBpbi4KPiAKPiBTaWduZWQtb2ZmLWJ5OiBLYXZ5YXNyZWUgS290YWdpcmkg PGthdnlhc3JlZS5rb3RhZ2lyaUBtaWNyb2NoaXAuY29tPgo+IC0tLQo+ICBhcmNoL2FybS9tYWNo LWF0OTEvS2NvbmZpZyAgfCAgIDIgKwoKPiAgZHJpdmVycy9tZmQvYXRtZWwtZmxleGNvbS5jIHwg IDU1ICsrKysrKysrKysrKysrKy0KCkNhbiB0aGlzIGJlIHNlcGFyYXRlZCBpbnRvIGl0cyBvd24g cGF0Y2g/Cgo+ICBkcml2ZXJzL211eC9LY29uZmlnICAgICAgICAgfCAgMTIgKysrKwo+ICBkcml2 ZXJzL211eC9NYWtlZmlsZSAgICAgICAgfCAgIDIgKwo+ICBkcml2ZXJzL211eC9sYW45NjYtZmx4 LmMgICAgfCAxMjEgKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrCj4gIDUgZmls ZXMgY2hhbmdlZCwgMTkxIGluc2VydGlvbnMoKyksIDEgZGVsZXRpb24oLSkKPiAgY3JlYXRlIG1v ZGUgMTAwNjQ0IGRyaXZlcnMvbXV4L2xhbjk2Ni1mbHguYwo+IAo+IGRpZmYgLS1naXQgYS9hcmNo L2FybS9tYWNoLWF0OTEvS2NvbmZpZyBiL2FyY2gvYXJtL21hY2gtYXQ5MS9LY29uZmlnCj4gaW5k ZXggMjc5ODEwMzgxMjU2Li4yNmZiMGY0ZTFiNzkgMTAwNjQ0Cj4gLS0tIGEvYXJjaC9hcm0vbWFj aC1hdDkxL0tjb25maWcKPiArKysgYi9hcmNoL2FybS9tYWNoLWF0OTEvS2NvbmZpZwo+IEBAIC03 NCw2ICs3NCw4IEBAIGNvbmZpZyBTT0NfTEFOOTY2Cj4gIAlzZWxlY3QgRFdfQVBCX1RJTUVSX09G Cj4gIAlzZWxlY3QgQVJNX0dJQwo+ICAJc2VsZWN0IE1FTU9SWQo+ICsJc2VsZWN0IE1VTFRJUExF WEVSCj4gKwlzZWxlY3QgTVVYX0xBTjk2Ngo+ICAJaGVscAo+ICAJICBUaGlzIGVuYWJsZXMgc3Vw cG9ydCBmb3IgQVJNdjcgYmFzZWQgTWljcm9jaGlwIExBTjk2NiBTb0MgZmFtaWx5Lgo+ICAKPiBk aWZmIC0tZ2l0IGEvZHJpdmVycy9tZmQvYXRtZWwtZmxleGNvbS5jIGIvZHJpdmVycy9tZmQvYXRt ZWwtZmxleGNvbS5jCj4gaW5kZXggNTU5ZWI0ZDM1MmI2Li43Y2ZkMGZjM2Y0ZjAgMTAwNjQ0Cj4g LS0tIGEvZHJpdmVycy9tZmQvYXRtZWwtZmxleGNvbS5jCj4gKysrIGIvZHJpdmVycy9tZmQvYXRt ZWwtZmxleGNvbS5jCj4gQEAgLTE3LDYgKzE3LDcgQEAKPiAgI2luY2x1ZGUgPGxpbnV4L2lvLmg+ Cj4gICNpbmNsdWRlIDxsaW51eC9jbGsuaD4KPiAgI2luY2x1ZGUgPGR0LWJpbmRpbmdzL21mZC9h dG1lbC1mbGV4Y29tLmg+Cj4gKyNpbmNsdWRlIDxsaW51eC9tdXgvY29uc3VtZXIuaD4KPiAgCj4g IC8qIEkvTyByZWdpc3RlciBvZmZzZXRzICovCj4gICNkZWZpbmUgRkxFWF9NUgkJMHgwCS8qIE1v ZGUgUmVnaXN0ZXIgKi8KPiBAQCAtMjgsNiArMjksMTAgQEAKPiAgI2RlZmluZSBGTEVYX01SX09Q TU9ERShvcG1vZGUpCSgoKG9wbW9kZSkgPDwgRkxFWF9NUl9PUE1PREVfT0ZGU0VUKSAmCVwKPiAg CQkJCSBGTEVYX01SX09QTU9ERV9NQVNLKQo+ICAKPiArc3RydWN0IGF0bWVsX2ZsZXhfY2FwcyB7 Cj4gKwlib29sIGhhc19mbHhfbXV4OwoKV2h5IGRvZXMgdGhpcyBuZWVkIGl0J3Mgb3duIHN0cnVj dD8KCj4gK307Cj4gKwo+ICBzdHJ1Y3QgYXRtZWxfZmxleGNvbSB7Cj4gIAl2b2lkIF9faW9tZW0g KmJhc2U7Cj4gIAl1MzIgb3Btb2RlOwo+IEBAIC0zNyw2ICs0Miw3IEBAIHN0cnVjdCBhdG1lbF9m bGV4Y29tIHsKPiAgc3RhdGljIGludCBhdG1lbF9mbGV4Y29tX3Byb2JlKHN0cnVjdCBwbGF0Zm9y bV9kZXZpY2UgKnBkZXYpCj4gIHsKPiAgCXN0cnVjdCBkZXZpY2Vfbm9kZSAqbnAgPSBwZGV2LT5k ZXYub2Zfbm9kZTsKPiArCWNvbnN0IHN0cnVjdCBhdG1lbF9mbGV4X2NhcHMgKmNhcHM7Cj4gIAlz dHJ1Y3QgcmVzb3VyY2UgKnJlczsKPiAgCXN0cnVjdCBhdG1lbF9mbGV4Y29tICpkZGF0YTsKPiAg CWludCBlcnI7Cj4gQEAgLTc2LDEzICs4Miw2MCBAQCBzdGF0aWMgaW50IGF0bWVsX2ZsZXhjb21f cHJvYmUoc3RydWN0IHBsYXRmb3JtX2RldmljZSAqcGRldikKPiAgCSAqLwo+ICAJd3JpdGVsKEZM RVhfTVJfT1BNT0RFKGRkYXRhLT5vcG1vZGUpLCBkZGF0YS0+YmFzZSArIEZMRVhfTVIpOwo+ICAK PiArCWNhcHMgPSBvZl9kZXZpY2VfZ2V0X21hdGNoX2RhdGEoJnBkZXYtPmRldik7Cj4gKwlpZiAo IWNhcHMpIHsKPiArCQlkZXZfZXJyKCZwZGV2LT5kZXYsICJDb3VsZCBub3QgcmV0cmlldmUgZmxl eGNvbSBjYXBzXG4iKTsKCmRldl9lcnIoKSBhbHJlYWR5IHByaW50cyBvdXQgdGhlIGRldmljZSBu YW1lLCBzbyB5b3UgY2FuIGRyb3AgdGhlCiJmbGV4Y29tIiBwYXJ0LiAgQWxzbywgcGxlYXNlIGV4 cGFuZCAnY2FwcycuICBJJ20gYXNzdW1pbmcgdGhhdCdzCmNhcGFiaWxpdGllcz8KCj4gKwkJcmV0 dXJuIC1FSU5WQUw7Cj4gKwl9Cj4gKwo+ICsJLyogRmxleGNvbSBNdXggKi8KPiArCWlmIChjYXBz LT5oYXNfZmx4X211eCAmJiBvZl9wcm9wZXJ0eV9yZWFkX2Jvb2wobnAsICJtdXgtY29udHJvbHMi KSkgewo+ICsJCXN0cnVjdCBtdXhfY29udHJvbCAqZmx4X211eDsKCldoYXQgaXMgJ2ZseCc/Cgo+ ICsJCXN0cnVjdCBvZl9waGFuZGxlX2FyZ3MgYXJnczsKPiArCQlpbnQgaSwgY291bnQ7Cj4gKwo+ ICsJCWZseF9tdXggPSBkZXZtX211eF9jb250cm9sX2dldCgmcGRldi0+ZGV2LCBOVUxMKTsKPiAr CQlpZiAoSVNfRVJSKGZseF9tdXgpKQo+ICsJCQlyZXR1cm4gUFRSX0VSUihmbHhfbXV4KTsKPiAr Cj4gKwkJY291bnQgPSBvZl9wcm9wZXJ0eV9jb3VudF9zdHJpbmdzKG5wLCAibXV4LWNvbnRyb2wt bmFtZXMiKTsKPiArCQlmb3IgKGkgPSAwOyBpIDwgY291bnQ7IGkrKykgewo+ICsJCQllcnIgPSBv Zl9wYXJzZV9waGFuZGxlX3dpdGhfZml4ZWRfYXJncyhucCwgIm11eC1jb250cm9scyIsIDEsIGks ICZhcmdzKTsKPiArCQkJaWYgKGVycikKPiArCQkJCWJyZWFrOwoKTm8gbXV4X2NvbnRyb2xfZGVz ZWxlY3QoKSBmb3IgcHJldmlvdXMgaXRlcmF0aW9ucz8KCj4gKwkJCWVyciA9IG11eF9jb250cm9s X3NlbGVjdChmbHhfbXV4LCBhcmdzLmFyZ3NbMF0pOwo+ICsJCQlpZiAoIWVycikgewo+ICsJCQkJ bXV4X2NvbnRyb2xfZGVzZWxlY3QoZmx4X211eCk7CgpOb3Qgc3VyZSBJIHNlZSB0aGUgcG9pbnQg aW4gc2VsZWN0aW5nIHRoZW4gZGVzZWxlY3RpbmcuCgpJcyBpdCBqdXN0IGEgdGVzdD8KCklmIHNv LCB3aHkgZG9uJ3QgeW91IHdhaXQgdW50aWwgeW91IG5lZWQgdG8gc2VsZWN0IGl0LCB0aGVuIGFw cGx5Cm5vcm1hbCBlcnJvciBoYW5kbGluZyBwcm90b2NvbHMgdGhlcmUgaW5zdGVhZD8KCj4gKwkJ CX0gZWxzZSB7Cj4gKwkJCQlkZXZfZXJyKCZwZGV2LT5kZXYsICJGYWlsZWQgdG8gc2VsZWN0IEZM RVhDT00gbXV4XG4iKTsKPiArCQkJCXJldHVybiBlcnI7Cj4gKwkJCX0KPiArCQl9Cj4gKwl9Cj4g Kwo+ICAJY2xrX2Rpc2FibGVfdW5wcmVwYXJlKGRkYXRhLT5jbGspOwo+ICAKPiAgCXJldHVybiBk ZXZtX29mX3BsYXRmb3JtX3BvcHVsYXRlKCZwZGV2LT5kZXYpOwo+ICB9Cj4gIAo+ICtzdGF0aWMg Y29uc3Qgc3RydWN0IGF0bWVsX2ZsZXhfY2FwcyBhdG1lbF9mbGV4Y29tX2NhcHMgPSB7fTsKCldo eSBub3QganVzdCBsZWF2ZSAuZGF0YSBhcyBOVUxMPwoKPiArc3RhdGljIGNvbnN0IHN0cnVjdCBh dG1lbF9mbGV4X2NhcHMgbGFuOTY2eF9mbGV4Y29tX2NhcHMgPSB7Cj4gKwkuaGFzX2ZseF9tdXgg PSB0cnVlLAo+ICt9Owo+ICsKPiAgc3RhdGljIGNvbnN0IHN0cnVjdCBvZl9kZXZpY2VfaWQgYXRt ZWxfZmxleGNvbV9vZl9tYXRjaFtdID0gewo+IC0JeyAuY29tcGF0aWJsZSA9ICJhdG1lbCxzYW1h NWQyLWZsZXhjb20iIH0sCj4gKwl7Cj4gKwkJLmNvbXBhdGlibGUgPSAiYXRtZWwsc2FtYTVkMi1m bGV4Y29tIiwKPiArCQkuZGF0YSA9ICZhdG1lbF9mbGV4Y29tX2NhcHMsCgpBbmQgdGhpcyBjYW4n dCBiZSBhIERUIHByb3BlcnR5PwoKPiArCX0sCj4gKwo+ICsJewo+ICsJCS5jb21wYXRpYmxlID0g Im1pY3JvY2hpcCxsYW45NjYtZmxleGNvbSIsCj4gKwkJLmRhdGEgPSAmbGFuOTY2eF9mbGV4Y29t X2NhcHMsCj4gKwl9LAoKVGhpcyBhbGwgc2VlbXMgbGlrZSBhIGxvdCBvZiBob29wLWp1bXBpbmcu CgpXaHkgbm90IGp1c3QgZG86CgogIGlmIChvZl9kZXZpY2VfaXNfY29tcGF0aWJsZShucCwgImxh bjk2NnhfZmxleGNvbV9jYXBzIikpCgo+ICsKPiAgCXsgLyogc2VudGluZWwgKi8gfQo+ICB9Owo+ ICBNT0RVTEVfREVWSUNFX1RBQkxFKG9mLCBhdG1lbF9mbGV4Y29tX29mX21hdGNoKTsKCi0tIApM ZWUgSm9uZXMgW+adjueQvOaWr10KUHJpbmNpcGFsIFRlY2huaWNhbCBMZWFkIC0gRGV2ZWxvcGVy IFNlcnZpY2VzCkxpbmFyby5vcmcg4pSCIE9wZW4gc291cmNlIHNvZnR3YXJlIGZvciBBcm0gU29D cwpGb2xsb3cgTGluYXJvOiBGYWNlYm9vayB8IFR3aXR0ZXIgfCBCbG9nCgpfX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpsaW51eC1hcm0ta2VybmVsIG1haWxp bmcgbGlzdApsaW51eC1hcm0ta2VybmVsQGxpc3RzLmluZnJhZGVhZC5vcmcKaHR0cDovL2xpc3Rz LmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9saW51eC1hcm0ta2VybmVsCg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id C6755C433F5 for ; Mon, 23 May 2022 14:52:32 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237327AbiEWOwa (ORCPT ); Mon, 23 May 2022 10:52:30 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54976 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S237330AbiEWOw3 (ORCPT ); Mon, 23 May 2022 10:52:29 -0400 Received: from mail-wr1-x42c.google.com (mail-wr1-x42c.google.com [IPv6:2a00:1450:4864:20::42c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B93F854F98 for ; Mon, 23 May 2022 07:52:26 -0700 (PDT) Received: by mail-wr1-x42c.google.com with SMTP id m20so10611253wrb.13 for ; Mon, 23 May 2022 07:52:26 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:content-transfer-encoding:in-reply-to; bh=WM2w2iSrfnCpI7JacIA3BcUph/2ijVWcwtEgBCcbpIc=; b=r7xEqlLCksQ1soquqg6YMFdzig2CSZJJ1ZRaPdQchwXYnn4/u92ZxMh2tX3eTolH+q 1bm4OVGSNSyKP0xWNjtUUNO3BvDCCjxhoDGj3/lo2vkzrUD4ymuu9l9s3M/r+3M8cG25 oof5UFhLCXyem8IV14F126PePSViDheuZ5U4iveFPIRfvwVa38CkpgHlw5371o++fVk6 k6GUYWa3/z0YAXUH/07TG6G7HCR0OxWjQlTerJ4VYF8gc/Ia3lnIZDNVgp2NfyYKY6pI AvkA5sauIbX+omqXhaeSv7+9U52ciaa4xutw4PmMl62qN4OKihtCHnexWHgbnh4UGKmH Ep2A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to; bh=WM2w2iSrfnCpI7JacIA3BcUph/2ijVWcwtEgBCcbpIc=; b=ov3o4cRAwKOpGWxTj0iSnxEE5ezn+I6gZ6bNkJ5CT55uFQ5DRQcm+Y0VGUohUpRKVm FP5aVjEHtVRIFh4B9ZTSuvY502l1C3uAHK2LfOQ7yqB9lpvm3qg7/XJrdceBUydAEH3A nHvvbJ9yoUxMR9KxLU6MAJKeRkFLdQa1NPLT6BQ7Kxfalhl9Vn0dBadlaleadeNywxiL JgaNiTY0J58bKIVU6+FJKLkgbrxv7zOH1CKB4tegMXxnvU/4+TcUYrKYQDc4mpexZOv+ trOc0zxHH88KNcynD3U0Mj+L869AAtID04WTCAGcyD0CJe65j63TtZ9GtTxmJZUlnb6v 8pjQ== X-Gm-Message-State: AOAM533zb3JpgIOklBYM85I3Z3cTM6DSQ8nss1Cr6mtYpKmH31YCCBm3 fts3tkW0yS61LOb/KKAPnSFcsg== X-Google-Smtp-Source: ABdhPJx8g7hX75odBmTIOpI6lWQKWgpFUQOsgvhtxc8fxWoHcWpwS5f35ULO7SpybNMKCN12jx3Kqw== X-Received: by 2002:a05:6000:1a88:b0:20e:6eb4:124e with SMTP id f8-20020a0560001a8800b0020e6eb4124emr18015038wry.8.1653317545273; Mon, 23 May 2022 07:52:25 -0700 (PDT) Received: from google.com (cpc155339-bagu17-2-0-cust87.1-3.cable.virginm.net. [86.27.177.88]) by smtp.gmail.com with ESMTPSA id c21-20020a05600c149500b0039750c29cf7sm1758385wmh.40.2022.05.23.07.52.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 23 May 2022 07:52:24 -0700 (PDT) Date: Mon, 23 May 2022 15:52:22 +0100 From: Lee Jones To: Kavyasree Kotagiri Cc: krzysztof.kozlowski+dt@linaro.org, nicolas.ferre@microchip.com, alexandre.belloni@bootlin.com, claudiu.beznea@microchip.com, peda@axentia.se, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux@armlinux.org.uk, Manohar.Puri@microchip.com, UNGLinuxDriver@microchip.com Subject: Re: [PATCH v2 4/4] mux: lan966: Add support for flexcom mux controller Message-ID: References: <20220509084920.14529-1-kavyasree.kotagiri@microchip.com> <20220509084920.14529-5-kavyasree.kotagiri@microchip.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20220509084920.14529-5-kavyasree.kotagiri@microchip.com> Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Mon, 09 May 2022, Kavyasree Kotagiri wrote: > LAN966 SoC have 5 flexcoms. Each flexcom has 2 chip-selects. > For each chip select of each flexcom there is a configuration > register FLEXCOM_SHARED[0-4]:SS_MASK[0-1]. The width of > configuration register is 21 because there are 21 shared pins > on each of which the chip select can be mapped. Each bit of the > register represents a different FLEXCOM_SHARED pin. > > Signed-off-by: Kavyasree Kotagiri > --- > arch/arm/mach-at91/Kconfig | 2 + > drivers/mfd/atmel-flexcom.c | 55 +++++++++++++++- Can this be separated into its own patch? > drivers/mux/Kconfig | 12 ++++ > drivers/mux/Makefile | 2 + > drivers/mux/lan966-flx.c | 121 ++++++++++++++++++++++++++++++++++++ > 5 files changed, 191 insertions(+), 1 deletion(-) > create mode 100644 drivers/mux/lan966-flx.c > > diff --git a/arch/arm/mach-at91/Kconfig b/arch/arm/mach-at91/Kconfig > index 279810381256..26fb0f4e1b79 100644 > --- a/arch/arm/mach-at91/Kconfig > +++ b/arch/arm/mach-at91/Kconfig > @@ -74,6 +74,8 @@ config SOC_LAN966 > select DW_APB_TIMER_OF > select ARM_GIC > select MEMORY > + select MULTIPLEXER > + select MUX_LAN966 > help > This enables support for ARMv7 based Microchip LAN966 SoC family. > > diff --git a/drivers/mfd/atmel-flexcom.c b/drivers/mfd/atmel-flexcom.c > index 559eb4d352b6..7cfd0fc3f4f0 100644 > --- a/drivers/mfd/atmel-flexcom.c > +++ b/drivers/mfd/atmel-flexcom.c > @@ -17,6 +17,7 @@ > #include > #include > #include > +#include > > /* I/O register offsets */ > #define FLEX_MR 0x0 /* Mode Register */ > @@ -28,6 +29,10 @@ > #define FLEX_MR_OPMODE(opmode) (((opmode) << FLEX_MR_OPMODE_OFFSET) & \ > FLEX_MR_OPMODE_MASK) > > +struct atmel_flex_caps { > + bool has_flx_mux; Why does this need it's own struct? > +}; > + > struct atmel_flexcom { > void __iomem *base; > u32 opmode; > @@ -37,6 +42,7 @@ struct atmel_flexcom { > static int atmel_flexcom_probe(struct platform_device *pdev) > { > struct device_node *np = pdev->dev.of_node; > + const struct atmel_flex_caps *caps; > struct resource *res; > struct atmel_flexcom *ddata; > int err; > @@ -76,13 +82,60 @@ static int atmel_flexcom_probe(struct platform_device *pdev) > */ > writel(FLEX_MR_OPMODE(ddata->opmode), ddata->base + FLEX_MR); > > + caps = of_device_get_match_data(&pdev->dev); > + if (!caps) { > + dev_err(&pdev->dev, "Could not retrieve flexcom caps\n"); dev_err() already prints out the device name, so you can drop the "flexcom" part. Also, please expand 'caps'. I'm assuming that's capabilities? > + return -EINVAL; > + } > + > + /* Flexcom Mux */ > + if (caps->has_flx_mux && of_property_read_bool(np, "mux-controls")) { > + struct mux_control *flx_mux; What is 'flx'? > + struct of_phandle_args args; > + int i, count; > + > + flx_mux = devm_mux_control_get(&pdev->dev, NULL); > + if (IS_ERR(flx_mux)) > + return PTR_ERR(flx_mux); > + > + count = of_property_count_strings(np, "mux-control-names"); > + for (i = 0; i < count; i++) { > + err = of_parse_phandle_with_fixed_args(np, "mux-controls", 1, i, &args); > + if (err) > + break; No mux_control_deselect() for previous iterations? > + err = mux_control_select(flx_mux, args.args[0]); > + if (!err) { > + mux_control_deselect(flx_mux); Not sure I see the point in selecting then deselecting. Is it just a test? If so, why don't you wait until you need to select it, then apply normal error handling protocols there instead? > + } else { > + dev_err(&pdev->dev, "Failed to select FLEXCOM mux\n"); > + return err; > + } > + } > + } > + > clk_disable_unprepare(ddata->clk); > > return devm_of_platform_populate(&pdev->dev); > } > > +static const struct atmel_flex_caps atmel_flexcom_caps = {}; Why not just leave .data as NULL? > +static const struct atmel_flex_caps lan966x_flexcom_caps = { > + .has_flx_mux = true, > +}; > + > static const struct of_device_id atmel_flexcom_of_match[] = { > - { .compatible = "atmel,sama5d2-flexcom" }, > + { > + .compatible = "atmel,sama5d2-flexcom", > + .data = &atmel_flexcom_caps, And this can't be a DT property? > + }, > + > + { > + .compatible = "microchip,lan966-flexcom", > + .data = &lan966x_flexcom_caps, > + }, This all seems like a lot of hoop-jumping. Why not just do: if (of_device_is_compatible(np, "lan966x_flexcom_caps")) > + > { /* sentinel */ } > }; > MODULE_DEVICE_TABLE(of, atmel_flexcom_of_match); -- Lee Jones [李琼斯] Principal Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog