From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukasz Majewski Date: Sat, 18 May 2019 22:52:17 +0200 Subject: [U-Boot] [PATCH v4 07/13] dm: clk: Define clk_get_parent() for clk operations In-Reply-To: References: <20190516221042.3583-1-lukma@denx.de> <20190516221042.3583-8-lukma@denx.de> Message-ID: <20190518225217.14f2a3fc@jawa> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Sat, 18 May 2019 10:08:36 -0600 Simon Glass wrote: > On Thu, 16 May 2019 at 16:11, Lukasz Majewski wrote: > > > > This commit adds the clk_get_parent() function, which is responsible > > for getting the parent's struct clock pointer. > > > > U-boot's DM support for getting parent is different (the parent > > relationship is in udevice) than the one in common clock framework > > (CCF) in Linux. To obtain the pointer to struct clk of parent the > > pdev->driver_data field is read. > > > > Signed-off-by: Lukasz Majewski > > > > --- > > > > Changes in v4: None > > Changes in v3: > > - New patch > > > > drivers/clk/clk-uclass.c | 15 +++++++++++++++ > > include/clk.h | 9 +++++++++ > > 2 files changed, 24 insertions(+) > > Please can you add a test for this? It is implicitly covered here: http://patchwork.ozlabs.org/patch/1100767/ but maybe shall I add a separate test case? However, considering the other reply to the driver_data field usage, those tests would probably need redesign anyway. > > Regards, > Simon Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 488 bytes Desc: OpenPGP digital signature URL: