Re: [PATCH v2 2/2] ARM: dts: vfxxx: Add property for minimum sample time

Shawn Guo
2015-07-10T08:53:47Z

On Wed, Jun 24, 2015 at 02:03:41PM +0530, Sanchayan Maity wrote:
> Add a device tree property which allows to specify the minimum sample
> time which can be used to calculate the actual ADC cycles required
> depending on the hardware.
>
> Signed-off-by: Sanchayan Maity <maitysanchayan@gmail.com>
> ---
> arch/arm/boot/dts/vfxxx.dtsi | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm/boot/dts/vfxxx.dtsi b/arch/arm/boot/dts/vfxxx.dtsi
> index 90a03d5..71d9c08 100644
> --- a/arch/arm/boot/dts/vfxxx.dtsi
> +++ b/arch/arm/boot/dts/vfxxx.dtsi
> @@ -229,6 +229,7 @@
> status = "disabled";
> fsl,adck-max-frequency = <30000000>, <40000000>,
> <20000000>;
> + min-sample-time = <1000>;
> };
>
> wdoga5: wdog@4003e000 {
> @@ -447,6 +448,7 @@
> status = "disabled";
> fsl,adck-max-frequency = <30000000>, <40000000>,
> <20000000>;
> + min-sample-time = <1000>;

Can we code 1000 as the default in kernel driver, so that only boards
requiring different value need to have this property? Doing so makes
the property optional rather than required.

Shawn

> };
>
> esdhc1: esdhc@400b2000 {
> --
> 2.4.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>