[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <glaiaikudbtuahnwnqoduretcnmrpb7lg2gerlnntcltrjfcko@4vgjb4v66rad>
Date: Mon, 12 Feb 2024 00:52:10 +0100
From: Sebastian Reichel <sebastian.reichel@...labora.com>
To: Andrew Davis <afd@...com>
Cc: Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>,
Cristian Ciocaltea <cristian.ciocaltea@...il.com>, Florian Fainelli <florian.fainelli@...adcom.com>,
Ray Jui <rjui@...adcom.com>, Scott Branden <sbranden@...adcom.com>,
Bjorn Andersson <andersson@...nel.org>, Konrad Dybcio <konrad.dybcio@...aro.org>,
Sean Wang <sean.wang@...iatek.com>, Matthias Brugger <matthias.bgg@...il.com>,
AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>, linux-pm@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-actions@...ts.infradead.org, linux-arm-msm@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-mediatek@...ts.infradead.org
Subject: Re: [PATCH v3 18/19] power: reset: syscon-poweroff: Move device data
into a struct
Hi Andrew,
On Thu, Feb 08, 2024 at 11:04:09AM -0600, Andrew Davis wrote:
> Currently all these device data elements are top level global variables.
> Move these into a struct. This will be used in the next patch when
> the global variable usage is removed. Doing this in two steps makes
> the patches easier to read.
>
> Signed-off-by: Andrew Davis <afd@...com>
> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>
> ---
> drivers/power/reset/syscon-poweroff.c | 36 +++++++++++++++------------
> 1 file changed, 20 insertions(+), 16 deletions(-)
>
> diff --git a/drivers/power/reset/syscon-poweroff.c b/drivers/power/reset/syscon-poweroff.c
> index 1b2ce7734260c..4899a019256e8 100644
> --- a/drivers/power/reset/syscon-poweroff.c
> +++ b/drivers/power/reset/syscon-poweroff.c
> @@ -15,15 +15,19 @@
> #include <linux/pm.h>
> #include <linux/regmap.h>
>
> -static struct regmap *map;
> -static u32 offset;
> -static u32 value;
> -static u32 mask;
> +struct syscon_poweroff_data {
> + struct regmap *map;
> + u32 offset;
> + u32 value;
> + u32 mask;
> +};
> +
> +static struct syscon_poweroff_data *data;
This patch is broken without the follow-up patch, since data is
never allocated. You need to move the memory allocation from the
next patch to this one.
Greetings,
-- Sebastian
Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)
Powered by blists - more mailing lists