Received: by 10.223.176.46 with SMTP id f43csp3934797wra; Tue, 23 Jan 2018 01:21:17 -0800 (PST) X-Google-Smtp-Source: AH8x226hcZGIY9dYfgIpAlCKTm0C8MkO9UA3qLbbpxHNdMFgGt2QXeHeTB26JU3nhaPVSIutB01y X-Received: by 10.101.81.68 with SMTP id g4mr7619929pgq.276.1516699276962; Tue, 23 Jan 2018 01:21:16 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1516699276; cv=none; d=google.com; s=arc-20160816; b=uJiMnrxdSme0nmuZYLGCGtmG+wkEPGm690MfBo8Oo9BYIr00RnQoRl4mmRSLWHIiHa qNNyV+nxssCkaYDq5r9nQlf58w4XcwLn89PHC6ntprxf78aoLFXtn9tWlTFfz0xeh5k5 7PIxLf4w+s0obEUwPR+/hTfPXLcEi073kpzVzT1BOGabmSACtqttt4vuHwiIZPizkrzL OIVum6QhyBgQxxe5ualAXamEOAhmMmZA5G3yEWCDcXfWAMx2UakJUSoyCqQrox7x287F G+/gXFKtemukwHVGWdN2G1gLjN9M8kCfGLqcOqA3xqfWvOuedlbP1G4Jx1Ix21av2At3 BMkQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:dkim-signature :arc-authentication-results; bh=BkJzVow+kDZ56MxWCs3c01RPwIWJr/sKzbE7JY6oqm0=; b=AYF+7gP/38XyNFTtOqag2kZ7Ay5i+rKx9K1coMIrePWSiwLQQjYFLzIfayrwAhUvI8 7N5surYTM0RDPanTs3Pk1rUzj7ht52Ponh8DnqFO4g1+xZja4bMwwbhATsiyhPi4riIP nRrjH6UfleKhwHiXJg2BZLyXxheysSsO5Gd5CyHWmqQGKnet9Yo/rXqvTKzOW8YciRAq t0YdATaWWKJ7Ras0lC8THvtYwIR/VuhxL/9475Hu9TTclxe9czyYbXTTze7kmqDABArt 9qAZwePAbMUt3EG5vKgzCXU40OQOLK9CCRc6gvqM/HDhsf7Rmqy0QvIuoiczBcAILdST L4Mw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=iDtHUutU; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id 5-v6si4431623plx.385.2018.01.23.01.21.01; Tue, 23 Jan 2018 01:21:16 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=iDtHUutU; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751224AbeAWJUg (ORCPT + 99 others); Tue, 23 Jan 2018 04:20:36 -0500 Received: from mail-wr0-f194.google.com ([209.85.128.194]:45765 "EHLO mail-wr0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751156AbeAWJUd (ORCPT ); Tue, 23 Jan 2018 04:20:33 -0500 Received: by mail-wr0-f194.google.com with SMTP id 16so11676341wry.12 for ; Tue, 23 Jan 2018 01:20:33 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=BkJzVow+kDZ56MxWCs3c01RPwIWJr/sKzbE7JY6oqm0=; b=iDtHUutUUnryAEy+XHu1Fqn1eTE1BFS0AQ78C+FB14DZBsd/bN/ugUICYydo32duTJ XzZFOnTANhGH1EpAYyVq9+j8ArsJpWKAAa6R9gPdiUkxZiuqeM2JwqTi8bCpBRzHvrVG otQ4ytpcDqQyUPW975zm6zbyiO8qmLGfULC3I= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=BkJzVow+kDZ56MxWCs3c01RPwIWJr/sKzbE7JY6oqm0=; b=cCE5T1MltzqHmgsNFgofOuDnm/JYd4v40oFK9vkIf+dOgPgpa1lswAejcdIQpRXTB3 Cj5nxmJLNtHIzBrFY6cgJki3F4B0nzAXfPwVQ21IUNOMOesI9d/28m8xBm+MCD3R3xhv efQrhio0MOUYXci45O7s5PjWrGQ2hsLK6f3gT/nAzj4OgmbEf6Joqcbxhbk3kf4mtpjr UaFQtaaYZMI5LdgW8+DYU08kfZ+sfqKLieLBQgHXxY2FqE/Zli+ov3TLaCYgjGD1Io+0 X+/PAN+reDRqQHc9DlgO6Knk56RhdF4VdPzoCCl+EgNgQTaX+nBQ+KjgFCQ0jEpo0c8b sVOg== X-Gm-Message-State: AKwxyteNqVKRXLpitUprmZanbiPnTjmWWLL6ObXHrxn/cLFPi/r5d34R 37VrnMJEj4TeIL/jPsDC1X1GiQ== X-Received: by 10.223.128.110 with SMTP id 101mr1556117wrk.177.1516699232384; Tue, 23 Jan 2018 01:20:32 -0800 (PST) Received: from [192.168.0.20] (cpc90716-aztw32-2-0-cust92.18-1.cable.virginm.net. [86.26.100.93]) by smtp.googlemail.com with ESMTPSA id c54sm26861663wrg.68.2018.01.23.01.20.31 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 23 Jan 2018 01:20:31 -0800 (PST) Subject: Re: [PATCH 1/4] dmaengine: qcom: bam_dma: make bam clk optional To: Vinod Koul Cc: Andy Gross , dmaengine@vger.kernel.org, Rob Herring , Mark Rutland , David Brown , Dan Williams , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, yanhe@quicinc.com, ramkri@qti.qualcomm.com, sdharia@quicinc.com References: <20180116190236.14558-1-srinivas.kandagatla@linaro.org> <20180116190236.14558-2-srinivas.kandagatla@linaro.org> <20180119055221.GV18649@localhost> <8dfa8ba1-6e98-a8e4-614c-592861cef571@linaro.org> <20180123091905.GY18649@localhost> From: Srinivas Kandagatla Message-ID: <86774341-19e6-5653-49ae-cb7614490d1b@linaro.org> Date: Tue, 23 Jan 2018 09:20:30 +0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: <20180123091905.GY18649@localhost> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 23/01/18 09:19, Vinod Koul wrote: > On Mon, Jan 22, 2018 at 09:55:01AM +0000, Srinivas Kandagatla wrote: > >>>> @@ -1180,13 +1180,14 @@ static int bam_dma_probe(struct platform_device *pdev) >>>> "qcom,controlled-remotely"); >>>> bdev->bamclk = devm_clk_get(bdev->dev, "bam_clk"); >>> >>> but you still do clk_get unconditionally? >> >> Only reason to do this way is to not break existing users in the mainline. >> >> remotely controlled BAM is already supported in upstream driver, there are >> users of this who pass clk from device tree, If I make this conditional then >> subsequent reads to the BAM registers for those instances might crash the >> system. > > But these instances are remote controlled, so if we stop representing them > in Linux, why would we read them? Plan is that we would transition those users once we get these bindings/changes in. Currently I don't have access to any of those devices so I made the changes safe, such that it does not break devices on mainline. --srini >