Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
References: <20230105134622.254560-1-arnd@kernel.org>
 <20230105134622.254560-5-arnd@kernel.org>
User-agent: mu4e 1.8.11; emacs 28.1
From:   Robert Jarzmik <robert.jarzmik@free.fr>
To:     Arnd Bergmann <arnd@kernel.org>
Cc:     Robert Jarzmik <robert.jarzmik@free.fr>,
        Daniel Mack <daniel@zonque.org>,
        Haojian Zhuang <haojian.zhuang@gmail.com>,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        Arnd Bergmann <arnd@arndb.de>,
        Michael Turquette <mturquette@baylibre.com>,
        Stephen Boyd <sboyd@kernel.org>,
        "Rafael J. Wysocki" <rafael@kernel.org>,
        Viresh Kumar <viresh.kumar@linaro.org>,
        Ulf Hansson <ulf.hansson@linaro.org>,
        Dominik Brodowski <linux@dominikbrodowski.net>,
        linux-clk@vger.kernel.org, linux-pm@vger.kernel.org,
        linux-mmc@vger.kernel.org
Subject: Re: [PATCH 04/27] ARM: pxa: drop pxa310/pxa320/pxa93x support
Date:   Sun, 08 Jan 2023 09:49:17 +0100
In-reply-to: <20230105134622.254560-5-arnd@kernel.org>
Message-ID: <m2sfglh02h.fsf@free.fr>
MIME-Version: 1.0
Content-Type: text/plain; format=flowed
Precedence: bulk


Arnd Bergmann <arnd@kernel.org> writes:
> There is currently no devicetree support for any of these three
> SoCs, and no board files remain. As it seems unlikely that 
> anyone
> is going to add DT support soon, let's drop the SoC specific 
> code
> now.

Hi Arnd,

Here you're dropping pxa variant support. For the currently 
"partly" working
boards in devicetree, such a zylonite, this will break their 
current support.

For example the zylonite I have which is working on DT has a 
pxa310 variant.
The cm-x300, which also works in DT, has a pxa320 variant.

What these boards need is their IO mappings and cpufreq to still 
work after
your serie in DT. What bothers me are the changes to :
 - drivers/clk/pxa/clk-pxa3xx.c
 - drivers/cpufreq/pxa3xx-cpufreq.c

Here the clock changes will probably remove the clock provided to 
specific
pxa310/pxa320 drivers for example.

I don't know how you want to proceed, yet this change will break 
some pxa3xx
platforms.

Cheers.

--
Robert