Re: [U-Boot] [PATCH 1/5] x86: Remove include directory in arch/x86/dts

2014-12-29 Thread Bin Meng
Hi Simon,

On Tue, Dec 30, 2014 at 3:39 AM, Simon Glass  wrote:
> Hi Bin,
>
> On 28 December 2014 at 23:04, Bin Meng  wrote:
>> Hi Simon,
>>
>> On Sun, Dec 28, 2014 at 9:02 AM, Simon Glass  wrote:
>>> Hi Bin,
>>>
>>> On 23 December 2014 at 22:06, Bin Meng  wrote:
 The include directory in arch/x86/dts is not used, so remove it.

 Signed-off-by: Bin Meng 
 ---
>>>
>>> Actually we do need this - the Chrome OS keyboard .dtsi will use a
>>> binding when I get around to updating it.
>>>
>>
>> OK, so maybe we can just put anything we do need into the include
>> instead of a symbolic link to the genric one?
>
> The idea as (I understand it) is that include/dt-bindings is the
> standard directory for all archs to put their bindings. This is how
> ARM works too.
>

OK, thanks for the clarification.

Regards,
Bin
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 1/5] x86: Remove include directory in arch/x86/dts

2014-12-29 Thread Simon Glass
Hi Bin,

On 28 December 2014 at 23:04, Bin Meng  wrote:
> Hi Simon,
>
> On Sun, Dec 28, 2014 at 9:02 AM, Simon Glass  wrote:
>> Hi Bin,
>>
>> On 23 December 2014 at 22:06, Bin Meng  wrote:
>>> The include directory in arch/x86/dts is not used, so remove it.
>>>
>>> Signed-off-by: Bin Meng 
>>> ---
>>
>> Actually we do need this - the Chrome OS keyboard .dtsi will use a
>> binding when I get around to updating it.
>>
>
> OK, so maybe we can just put anything we do need into the include
> instead of a symbolic link to the genric one?

The idea as (I understand it) is that include/dt-bindings is the
standard directory for all archs to put their bindings. This is how
ARM works too.

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 1/5] x86: Remove include directory in arch/x86/dts

2014-12-28 Thread Bin Meng
Hi Simon,

On Sun, Dec 28, 2014 at 9:02 AM, Simon Glass  wrote:
> Hi Bin,
>
> On 23 December 2014 at 22:06, Bin Meng  wrote:
>> The include directory in arch/x86/dts is not used, so remove it.
>>
>> Signed-off-by: Bin Meng 
>> ---
>
> Actually we do need this - the Chrome OS keyboard .dtsi will use a
> binding when I get around to updating it.
>

OK, so maybe we can just put anything we do need into the include
instead of a symbolic link to the genric one?

Regards,
Bin
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 1/5] x86: Remove include directory in arch/x86/dts

2014-12-27 Thread Simon Glass
Hi Bin,

On 23 December 2014 at 22:06, Bin Meng  wrote:
> The include directory in arch/x86/dts is not used, so remove it.
>
> Signed-off-by: Bin Meng 
> ---

Actually we do need this - the Chrome OS keyboard .dtsi will use a
binding when I get around to updating it.

>
>  arch/x86/dts/include/dt-bindings | 1 -
>  1 file changed, 1 deletion(-)
>  delete mode 12 arch/x86/dts/include/dt-bindings
>
> diff --git a/arch/x86/dts/include/dt-bindings 
> b/arch/x86/dts/include/dt-bindings
> deleted file mode 12
> index 0cecb3d..000
> --- a/arch/x86/dts/include/dt-bindings
> +++ /dev/null
> @@ -1 +0,0 @@
> -../../../../include/dt-bindings
> \ No newline at end of file
> --
> 1.8.2.1
>

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 1/5] x86: Remove include directory in arch/x86/dts

2014-12-23 Thread Bin Meng
The include directory in arch/x86/dts is not used, so remove it.

Signed-off-by: Bin Meng 
---

 arch/x86/dts/include/dt-bindings | 1 -
 1 file changed, 1 deletion(-)
 delete mode 12 arch/x86/dts/include/dt-bindings

diff --git a/arch/x86/dts/include/dt-bindings b/arch/x86/dts/include/dt-bindings
deleted file mode 12
index 0cecb3d..000
--- a/arch/x86/dts/include/dt-bindings
+++ /dev/null
@@ -1 +0,0 @@
-../../../../include/dt-bindings
\ No newline at end of file
-- 
1.8.2.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot