[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Tue, 3 Mar 2020 14:29:57 -0800
From: Jolly Shah <jolly.shah@...inx.com>
To: Sudeep Holla <sudeep.holla@....com>,
Jolly Shah <jolly.shah@...inx.com>
Cc: ard.biesheuvel@...aro.org, mingo@...nel.org,
gregkh@...uxfoundation.org, matt@...eblueprint.co.uk,
hkallweit1@...il.com, keescook@...omium.org,
dmitry.torokhov@...il.com, michal.simek@...inx.com,
rajanv@...inx.com, linux-arm-kernel@...ts.infradead.org,
linux-kernel@...r.kernel.org, Tejas Patel <tejas.patel@...inx.com>
Subject: Re: [PATCH] arch: arm64: xilinx: Make zynqmp_firmware driver optional
Hi Sudeep,
> ------Original Message------
> From: Sudeep.holla@....com <sudeep.holla@....com>
> Sent: Wednesday, February 26, 2020 3:46AM
> To: Jolly Shah <jolly.shah@...inx.com>
> Cc: Ard.biesheuvel@...aro.org <ard.biesheuvel@...aro.org>,
Mingo@...nel.org <mingo@...nel.org>, 'Greg Kh'
<gregkh@...uxfoundation.org>, Matt@...eblueprint.co.uk
<matt@...eblueprint.co.uk>, Hkallweit1@...il.com <hkallweit1@...il.com>,
Keescook@...omium.org <keescook@...omium.org>, Dmitry.torokhov@...il.com
<dmitry.torokhov@...il.com>, Michal Simek <michal.simek@...inx.com>,
Rajan Vaja <rajanv@...inx.com>, Linux-arm-kernel@...ts.infradead.org
<linux-arm-kernel@...ts.infradead.org>, Linux-kernel@...r.kernel.org
<linux-kernel@...r.kernel.org>, Tejas Patel <tejas.patel@...inx.com>,
Sudeep.holla@....com <sudeep.holla@....com>
> Subject: Re: [PATCH] arch: arm64: xilinx: Make zynqmp_firmware driver
optional
>
> On Tue, Feb 25, 2020 at 04:04:20PM -0800, Jolly Shah wrote:
>> From: Tejas Patel <tejas.patel@...inx.com>
>>
>> Make zynqmp_firmware driver as optional to disable it, if user don't
>> want to use default zynqmp firmware interface.
>>
>
> This patch on it own is simple and looks fine. However I expect the
> single binary to work with and without this option on the same platform.
> If zynqmp_firmware is not critical, the system should continue to work
> fine either way. The zynqmp_firmware driver should gracefully exit with
> error(if any).
>
Sure. Will address it in next version.
Thanks,
Jolly Shah
> --
> Regards,
> Sudeep
>
Powered by blists - more mailing lists