From mboxrd@z Thu Jan 1 00:00:00 1970 From: Javier Martinez Canillas Subject: Re: [PATCH] PM / Domains: Restrict "samsung,power-domain" checks to ARCH_EXYNOS Date: Fri, 21 Oct 2016 11:18:07 -0300 Message-ID: References: <1477049656-27535-1-git-send-email-geert+renesas@glider.be> <20161021135803.GB3289@kozik-lap> <96efd5b7-5f62-8146-13de-44b5ff7dd496@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <96efd5b7-5f62-8146-13de-44b5ff7dd496@samsung.com> Sender: linux-samsung-soc-owner@vger.kernel.org To: Sylwester Nawrocki , Krzysztof Kozlowski Cc: "Rafael J. Wysocki" , q@kozik-lap.samsung.com, Geert Uytterhoeven , "Rafael J. Wysocki" , Kevin Hilman , Ulf Hansson , Kukjin Kim , Greg Kroah-Hartman , Linux PM , "linux-arm-kernel@lists.infradead.org" , Linux Samsung SoC List-Id: linux-pm@vger.kernel.org Hello Sylwester, On 10/21/2016 11:14 AM, Sylwester Nawrocki wrote: > On 10/21/2016 03:58 PM, Krzysztof Kozlowski wrote: >> The samsung,power-domain was made deprecated (although not explicitly) >> in January 2015 (0da658704136 ("ARM: dts: convert to generic power >> domain bindings for exynos DT")) so how about: >> 1. Printing a dev_warn() about usage of deprecated bindings. >> 2. Complete removal in January 2017? > > I doubt anyone will ever use new mainline kernel with older dts/dtb > so IMHO it makes sense to queue a patch removing support for the > deprecated compatible just now and don't bother with a warning. > FWIW I agree with you. I don't know of any Exynos machine that ships a DT as read-only. Even consumer devices like the Exynos Chromebooks use a FIT image (kernel + FDT bundled), so the DT can always be updated. Removing the support for the deprecated property sound sensible to me. Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America