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;
Subject: Re: [PATCH v2] ARM: tegra: enforce PM requirement
To:     Russell King - ARM Linux admin <linux@armlinux.org.uk>,
        Sameer Pujar <spujar@nvidia.com>
Cc:     jhogan@kernel.org, thierry.reding@gmail.com, jonathanh@nvidia.com,
        linux-kernel@vger.kernel.org
References: <1551169537-17548-1-git-send-email-spujar@nvidia.com>
 <20190226091308.ycruijz2lcziv5gt@shell.armlinux.org.uk>
From:   Dmitry Osipenko <digetx@gmail.com>
Message-ID: <562c4307-74ad-fd9d-74e7-36ac19315d0b@gmail.com>
Date:   Tue, 26 Feb 2019 19:52:03 +0300
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101
 Thunderbird/60.5.1
MIME-Version: 1.0
In-Reply-To: <20190226091308.ycruijz2lcziv5gt@shell.armlinux.org.uk>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

26.02.2019 12:13, Russell King - ARM Linux admin пишет:
> On Tue, Feb 26, 2019 at 01:55:37PM +0530, Sameer Pujar wrote:
>> The requirement for this came while adding runtime PM support for HDA
>> driver. There were concerns about driver explicitly handling !PM case.
>> In general, drivers need to handle !PM case with work arounds for
>> managing clocks and power explicitly, which is not really necessary
>> when PM support on tegra is in good shape. In fact ARM 64-bit Tegra
>> platforms enforce PM support and there is no reason why this cannot be
>> done for 32-bit.
>>
>> More details with regards to above can be found in following patch,
>> http://patchwork.ozlabs.org/patch/1036645/
>>
>> This patch selects PM unconditionally and drivers can rely on runtime
>> PM framework for clock and power management.
> 
> What if the drivers are re-used on another SoC IP?  Doesn't this lead
> to unexpected failures?
> 
> If you want to do this, maybe also make those drivers depend on PM as
> well?

The commit message is inaccurate, it is intended for the Tegra HDA driver and not for some generic driver. The overall final intent is to remove dependency on the PM availability for all of Tegra drivers to "make Tegra maintainers life easier".