From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michal Simek <monstr@monstr.eu>
Date: Thu, 30 Jan 2014 08:38:40 +0100
Subject: [U-Boot] [RFC PATCH 1/2] common: Add new clk command
In-Reply-To: <52E0CA61.5040304@monstr.eu>
References: <a80ed6081b2a53afde032ebe288d3984c501860b.1390388559.git.michal.simek@xilinx.com>
	<20140122194434.GR20094@book.gsilab.sittig.org>
	<52E0CA61.5040304@monstr.eu>
Message-ID: <52EA0180.80409@monstr.eu>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 01/23/2014 08:53 AM, Michal Simek wrote:
> On 01/22/2014 08:44 PM, Gerhard Sittig wrote:
>> On Wed, Jan 22, 2014 at 12:02 +0100, Michal Simek wrote:
>>>
>>> --- /dev/null
>>> +++ b/common/cmd_clk.c
>>> @@ -0,0 +1,51 @@
>>> +/*
>>> + * Copyright (C) 2013 Xilinx, Inc.
>>> + *
>>> + * SPDX-License-Identifier:	GPL-2.0+
>>> + */
>>> +#include <common.h>
>>> +#include <command.h>
>>> +#include <clk.h>
>>> +
>>> +int __weak soc_clk_dump(void)
>>> +{
>>> +	puts("Not implemented\n");
>>> +	return 1;
>>> +}
>>> +
>>> +static int do_clk_dump(cmd_tbl_t *cmdtp, int flag, int argc,
>>> +		       char *const argv[])
>>> +{
>>> +	return soc_clk_dump();
>>> +}
>>
>> Is there a specific reason to not pass on the remaining (not yet
>> consumed) command line arguments?  Future implementations may
>> want to take a clock item's name, or a clock group's name, or
>> options related to the format or verbosity of the dump, et al.
> 
> Only one reason is that I don't need it for my zynq implementation.
> If this is necessary there is no problem to pass them because
> it is internal API. Also I prefer to pass just arguments
> which I need.
> 
> I have looked at i.MXes cases and they do in general what
> my zynq implementation. They can just include clk.h and
> change do_mx6_showclocks to soc_clk_dump.

Stefano: Can you give me your ACK?

Tom: Do you have any problem with this new clk command?

Thanks,
Michal


-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 263 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140130/eaa74ce4/attachment.pgp>