From: hl <hl@rock-chips.com> To: "Heiko Stübner" <heiko@sntech.de>, "Sudeep Holla" <sudeep.holla@arm.com>, "Lorenzo Pieralisi" <lorenzo.pieralisi@arm.com> Cc: tixy@linaro.org, typ@rock-chips.com, linux-rockchip@lists.infradead.org, airlied@linux.ie, mturquette@baylibre.com, dbasehore@chromium.org, sboyd@codeaurora.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, dianders@chromium.org, xsf@rock-chips.com, cw00.choi@samsung.com, kyungmin.park@samsung.com, myungjoo.ham@samsung.com, linux-arm-kernel@lists.infradead.org, mark.yao@rock-chips.com Subject: Re: [PATCH v3 1/7] firmware: rockchip: sip: Add rockchip SIP runtime service Date: Sun, 24 Jul 2016 16:00:35 +0800 [thread overview] Message-ID: <579475A3.2040506@rock-chips.com> (raw) In-Reply-To: <1497407.KruMAzy5tb@diego> Hi Heiko, On 2016年07月23日 04:50, Heiko Stübner wrote: > Hi again, > > one bigger thing I noticed only now. > > Am Freitag, 22. Juli 2016, 17:07:14 schrieben Sie: >> diff --git a/drivers/firmware/rockchip_sip.c >> b/drivers/firmware/rockchip_sip.c new file mode 100644 >> index 0000000..7756af9 >> --- /dev/null >> +++ b/drivers/firmware/rockchip_sip.c >> @@ -0,0 +1,64 @@ >> +/* >> + * This program is free software; you can redistribute it and/or modify >> + * it under the terms of the GNU General Public License version 2 as >> + * published by the Free Software Foundation. >> + * >> + * This program is distributed in the hope that it will be useful, >> + * but WITHOUT ANY WARRANTY; without even the implied warranty of >> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >> + * GNU General Public License for more details. >> + * >> + * Copyright (C) 2016 ARM Limited >> + */ >> +#include <linux/errno.h> >> +#include <linux/linkage.h> >> +#include <linux/of.h> >> +#include <linux/pm.h> >> +#include <linux/printk.h> >> +#include "rockchip_sip.h" >> + >> +typedef unsigned long (psci_fn)(unsigned long, unsigned long, >> + unsigned long, unsigned long); >> +asmlinkage psci_fn __invoke_psci_fn_smc; >> + >> +#define CONFIG_DRAM_INIT 0x00 >> +#define CONFIG_DRAM_SET_RATE 0x01 >> +#define CONFIG_DRAM_ROUND_RATE 0x02 >> +#define CONFIG_DRAM_SET_AT_SR 0x03 >> +#define CONFIG_DRAM_GET_BW 0x04 >> +#define CONFIG_DRAM_GET_RATE 0x05 >> +#define CONFIG_DRAM_CLR_IRQ 0x06 >> +#define CONFIG_DRAM_SET_PARAM 0x07 >> + >> +uint64_t sip_smc_ddr_init(void) >> +{ >> + return __invoke_psci_fn_smc(SIP_DDR_FREQ, 0, >> + 0, CONFIG_DRAM_INIT); > I don't think that is legal to use. For one this function itself is declared > static in the psci code - most likely for a specific reason. > > And also if anything invoke_psci_fn would hold the correct pointer depending > on the calling method. > > But as said above, accessing psci static stuff is most likely wrong. Maybe the > two psci people I've included can tell us how this is to be accessed. > Thanks Heiko. Hope Sudeep Holla and Lorenzo Pieralisi can give me advice. > Heiko > >> +} >> + >> +uint64_t sip_smc_set_ddr_param(uint64_t param) >> +{ >> + return __invoke_psci_fn_smc(SIP_DDR_FREQ, param, >> + 0, CONFIG_DRAM_SET_PARAM); >> +} >> + >> +uint64_t sip_smc_set_ddr_rate(uint64_t rate) >> +{ >> + return __invoke_psci_fn_smc(SIP_DDR_FREQ, rate, 0, >> + CONFIG_DRAM_SET_RATE); >> +} >> + >> +uint64_t sip_smc_get_ddr_rate(void) >> +{ >> + return __invoke_psci_fn_smc(SIP_DDR_FREQ, 0, 0, CONFIG_DRAM_GET_RATE); >> +} >> + >> +uint64_t sip_smc_clr_ddr_irq(void) >> +{ >> + return __invoke_psci_fn_smc(SIP_DDR_FREQ, 0, 0, CONFIG_DRAM_CLR_IRQ); >> +} >> + >> +uint64_t sip_smc_get_call_count(void) >> +{ >> + return __invoke_psci_fn_smc(SIP_SVC_CALL_COUNT, 0, 0, 0); >> +} > > _______________________________________________ > Linux-rockchip mailing list > Linux-rockchip@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-rockchip > > > -- Lin Huang
WARNING: multiple messages have this Message-ID (diff)
From: hl@rock-chips.com (hl) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v3 1/7] firmware: rockchip: sip: Add rockchip SIP runtime service Date: Sun, 24 Jul 2016 16:00:35 +0800 [thread overview] Message-ID: <579475A3.2040506@rock-chips.com> (raw) In-Reply-To: <1497407.KruMAzy5tb@diego> Hi Heiko, On 2016?07?23? 04:50, Heiko St?bner wrote: > Hi again, > > one bigger thing I noticed only now. > > Am Freitag, 22. Juli 2016, 17:07:14 schrieben Sie: >> diff --git a/drivers/firmware/rockchip_sip.c >> b/drivers/firmware/rockchip_sip.c new file mode 100644 >> index 0000000..7756af9 >> --- /dev/null >> +++ b/drivers/firmware/rockchip_sip.c >> @@ -0,0 +1,64 @@ >> +/* >> + * This program is free software; you can redistribute it and/or modify >> + * it under the terms of the GNU General Public License version 2 as >> + * published by the Free Software Foundation. >> + * >> + * This program is distributed in the hope that it will be useful, >> + * but WITHOUT ANY WARRANTY; without even the implied warranty of >> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >> + * GNU General Public License for more details. >> + * >> + * Copyright (C) 2016 ARM Limited >> + */ >> +#include <linux/errno.h> >> +#include <linux/linkage.h> >> +#include <linux/of.h> >> +#include <linux/pm.h> >> +#include <linux/printk.h> >> +#include "rockchip_sip.h" >> + >> +typedef unsigned long (psci_fn)(unsigned long, unsigned long, >> + unsigned long, unsigned long); >> +asmlinkage psci_fn __invoke_psci_fn_smc; >> + >> +#define CONFIG_DRAM_INIT 0x00 >> +#define CONFIG_DRAM_SET_RATE 0x01 >> +#define CONFIG_DRAM_ROUND_RATE 0x02 >> +#define CONFIG_DRAM_SET_AT_SR 0x03 >> +#define CONFIG_DRAM_GET_BW 0x04 >> +#define CONFIG_DRAM_GET_RATE 0x05 >> +#define CONFIG_DRAM_CLR_IRQ 0x06 >> +#define CONFIG_DRAM_SET_PARAM 0x07 >> + >> +uint64_t sip_smc_ddr_init(void) >> +{ >> + return __invoke_psci_fn_smc(SIP_DDR_FREQ, 0, >> + 0, CONFIG_DRAM_INIT); > I don't think that is legal to use. For one this function itself is declared > static in the psci code - most likely for a specific reason. > > And also if anything invoke_psci_fn would hold the correct pointer depending > on the calling method. > > But as said above, accessing psci static stuff is most likely wrong. Maybe the > two psci people I've included can tell us how this is to be accessed. > Thanks Heiko. Hope Sudeep Holla and Lorenzo Pieralisi can give me advice. > Heiko > >> +} >> + >> +uint64_t sip_smc_set_ddr_param(uint64_t param) >> +{ >> + return __invoke_psci_fn_smc(SIP_DDR_FREQ, param, >> + 0, CONFIG_DRAM_SET_PARAM); >> +} >> + >> +uint64_t sip_smc_set_ddr_rate(uint64_t rate) >> +{ >> + return __invoke_psci_fn_smc(SIP_DDR_FREQ, rate, 0, >> + CONFIG_DRAM_SET_RATE); >> +} >> + >> +uint64_t sip_smc_get_ddr_rate(void) >> +{ >> + return __invoke_psci_fn_smc(SIP_DDR_FREQ, 0, 0, CONFIG_DRAM_GET_RATE); >> +} >> + >> +uint64_t sip_smc_clr_ddr_irq(void) >> +{ >> + return __invoke_psci_fn_smc(SIP_DDR_FREQ, 0, 0, CONFIG_DRAM_CLR_IRQ); >> +} >> + >> +uint64_t sip_smc_get_call_count(void) >> +{ >> + return __invoke_psci_fn_smc(SIP_SVC_CALL_COUNT, 0, 0, 0); >> +} > > _______________________________________________ > Linux-rockchip mailing list > Linux-rockchip at lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-rockchip > > > -- Lin Huang
next prev parent reply other threads:[~2016-07-24 8:00 UTC|newest] Thread overview: 54+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-07-22 9:07 [PATCH v3 0/7] rk3399 support ddr frequency scaling Lin Huang 2016-07-22 9:07 ` Lin Huang 2016-07-22 9:07 ` [PATCH v3 1/7] firmware: rockchip: sip: Add rockchip SIP runtime service Lin Huang 2016-07-22 9:07 ` Lin Huang 2016-07-22 9:07 ` Lin Huang 2016-07-22 10:00 ` Heiko Stübner 2016-07-22 10:00 ` Heiko Stübner 2016-07-22 20:50 ` Heiko Stübner 2016-07-22 20:50 ` Heiko Stübner 2016-07-24 8:00 ` hl [this message] 2016-07-24 8:00 ` hl 2016-07-25 17:36 ` Sudeep Holla 2016-07-25 17:36 ` Sudeep Holla 2016-07-26 1:13 ` hl 2016-07-26 1:13 ` hl 2016-07-22 21:32 ` kbuild test robot 2016-07-22 21:32 ` kbuild test robot 2016-07-22 21:32 ` kbuild test robot 2016-07-26 18:29 ` Mark Rutland 2016-07-26 18:29 ` Mark Rutland 2016-07-22 9:07 ` [PATCH v3 2/7] clk: rockchip: add new clock-type for the ddrclk Lin Huang 2016-07-22 9:07 ` Lin Huang 2016-07-24 9:09 ` Heiko Stübner 2016-07-24 9:09 ` Heiko Stübner 2016-07-24 9:09 ` Heiko Stübner 2016-07-22 9:07 ` [PATCH v3 3/7] clk: rockchip: rk3399: add SCLK_DDRCLK ID for ddrc Lin Huang 2016-07-22 9:07 ` Lin Huang 2016-07-22 10:08 ` Heiko Stübner 2016-07-22 10:08 ` Heiko Stübner 2016-07-22 9:07 ` [PATCH v3 4/7] clk: rockchip: rk3399: add ddrc clock support Lin Huang 2016-07-22 9:07 ` Lin Huang 2016-07-22 9:07 ` [PATCH v3 5/7] PM / devfreq: event: support rockchip dfi controller Lin Huang 2016-07-22 9:07 ` Lin Huang 2016-07-22 20:28 ` Paul Gortmaker 2016-07-22 20:28 ` Paul Gortmaker 2016-07-22 20:28 ` Paul Gortmaker 2016-07-22 9:07 ` [PATCH v3 6/7] PM / devfreq: rockchip: add devfreq driver for rk3399 dmc Lin Huang 2016-07-22 9:07 ` Lin Huang 2016-07-22 20:24 ` Paul Gortmaker 2016-07-22 20:24 ` Paul Gortmaker 2016-07-22 20:24 ` Paul Gortmaker 2016-07-24 7:54 ` hl 2016-07-24 7:54 ` hl 2016-07-24 7:54 ` hl 2016-07-25 6:01 ` Chanwoo Choi 2016-07-25 6:01 ` Chanwoo Choi 2016-07-25 8:47 ` hl 2016-07-25 8:47 ` hl 2016-07-25 9:45 ` Chanwoo Choi 2016-07-25 9:45 ` Chanwoo Choi 2016-07-26 1:15 ` hl 2016-07-26 1:15 ` hl 2016-07-22 9:07 ` [PATCH v3 7/7] drm/rockchip: Add dmc notifier in vop driver Lin Huang 2016-07-22 9:07 ` Lin Huang
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=579475A3.2040506@rock-chips.com \ --to=hl@rock-chips.com \ --cc=airlied@linux.ie \ --cc=cw00.choi@samsung.com \ --cc=dbasehore@chromium.org \ --cc=dianders@chromium.org \ --cc=dri-devel@lists.freedesktop.org \ --cc=heiko@sntech.de \ --cc=kyungmin.park@samsung.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-rockchip@lists.infradead.org \ --cc=lorenzo.pieralisi@arm.com \ --cc=mark.yao@rock-chips.com \ --cc=mturquette@baylibre.com \ --cc=myungjoo.ham@samsung.com \ --cc=sboyd@codeaurora.org \ --cc=sudeep.holla@arm.com \ --cc=tixy@linaro.org \ --cc=typ@rock-chips.com \ --cc=xsf@rock-chips.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.