Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <cover.1587485099.git.daniele.alessandrelli@intel.com>
 <13ca92165fab2827b6d439661e75f5b91ef083c2.1587485099.git.daniele.alessandrelli@intel.com>
 <20200501081002.GA1055721@kroah.com> <f60aece195cd0700728fc38b0398949a82b72fc3.camel@linux.intel.com>
 <20200524212851.GG1192@bug>
In-Reply-To: <20200524212851.GG1192@bug>
From:   Arnd Bergmann <arnd@arndb.de>
Date:   Mon, 25 May 2020 10:18:57 +0200
Message-ID: <CAK8P3a225pqBfzQ19e6Gt0s_tYBp29xLb8EG==hhz=1wc7aVCA@mail.gmail.com>
Subject: Re: [PATCH 1/1] soc: keembay: Add Keem Bay IMR driver
To:     Pavel Machek <pavel@ucw.cz>
Cc:     Daniele Alessandrelli <daniele.alessandrelli@linux.intel.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Rob Herring <robh@kernel.org>,
        Andy Shevchenko <andriy.shevchenko@intel.com>,
        Paul J Murphy <paul.j.murphy@intel.com>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Sun, May 24, 2020 at 11:28 PM Pavel Machek <pavel@ucw.cz> wrote:
>
> Hi!
>
> > > Like I said above, you just broke multi-system kernels by always
> > > trying
> > > to do this.  Trigger off of a hardware device that only your platform
> > > has in order to properly load and run.  As-is, you don't want to do
> > > this.
> >
> > My bad, I didn't consider the issue of multi-platform ARM kernels.
> >
> > The problem is that I need this code to be run early at boot, so I
> > don't think I can make this a module.
>
> How early is early enough?
>
> What bootloader are you using?
>
> I believe you should simply fix your bootloader not to pass locked memory to the kernel.
>
> Alternatively, take that memory off the "memory available" maps, and only re-add it once
> it is usable.
>
> Anything else is dangerous.

Agreed, this sounds like an incompatible extension of the boot protocol
that we should otherwise not merge.

However, there is also a lot of missing information here, and it is always
possible they are trying to something for a good reason. As long as the
problem that the bootloader is trying to solve is explained well enough
in the changelog, we can discuss it to see how it should be done properly.

       Arnd