public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: vikasm <vikas.manocha@st.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/2] stv0991: fdt: add stv0991 device tree
Date: Fri, 1 May 2015 16:43:00 -0700	[thread overview]
Message-ID: <55440F84.90004@st.com> (raw)
In-Reply-To: <CAPnjgZ23v+8ZZ7ZEZ0aubb-ibD+ge5MExhGdgRNrPC235HPEwA@mail.gmail.com>

Hi Simon,

On 05/01/2015 04:45 PM, Simon Glass wrote:
> Hi Vikas,
>
> On 1 May 2015 at 17:18, vikasm <vikas.manocha@st.com> wrote:
>> Thanks Simon,
>>
>> On 05/01/2015 04:01 PM, Simon Glass wrote:
>>> Hi Vikas,
>>>
>>> On 1 May 2015 at 16:43, Vikas Manocha <vikas.manocha@st.com> wrote:
>>>
>>> <commit message here>
>> There is not much to add in the commit message apart from info in the title,
>> that is why i didn't add any message in the body.  Please let me know if i am
>> missing something ?
> How about:
>
> "Add a device tree for the St Micro STV0991 board and enable device
> tree control. We will progressively add device tree support to drivers
> used by this board"

Thanks, i will send V2.

Rgds,
Vikas

>
>>>> Signed-off-by: Vikas Manocha <vikas.manocha@st.com>
>>>> ---
>>>>  arch/arm/dts/Makefile     |    1 +
>>>>  arch/arm/dts/stv0991.dts  |   23 +++++++++++++++++++++++
>>>>  configs/stv0991_defconfig |    1 +
>>>>  include/configs/stv0991.h |    3 +++
>>>>  4 files changed, 28 insertions(+)
>>>>  create mode 100644 arch/arm/dts/stv0991.dts
>>>>
>>>> diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
>>>> index 46a6171..86faf58 100644
>>>> --- a/arch/arm/dts/Makefile
>>>> +++ b/arch/arm/dts/Makefile
>>>> @@ -54,6 +54,7 @@ dtb-$(CONFIG_SOCFPGA) +=                              \
>>>>         socfpga_arria5_socdk.dtb                        \
>>>>         socfpga_cyclone5_socdk.dtb                      \
>>>>         socfpga_cyclone5_socrates.dtb
>>>> +dtb-$(CONFIG_TARGET_STV0991) += stv0991.dtb
>>> Is this the only chip in this soc family? If so fine. If not, could
>>> your CONFIG be a little broader so we can (later) list all the DTBs
>>> here?
>> Yes, it is the only chip :-(, hope there will be more.
> OK.
>
>> Rgds,
>> Vikas
>>
>>>>  dtb-$(CONFIG_LS102XA) += ls1021a-qds.dtb \
>>>>         ls1021a-twr.dtb
>>>> diff --git a/arch/arm/dts/stv0991.dts b/arch/arm/dts/stv0991.dts
>>>> new file mode 100644
>>>> index 0000000..b25c48b
>>>> --- /dev/null
>>>> +++ b/arch/arm/dts/stv0991.dts
>>>> @@ -0,0 +1,23 @@
>>>> +/dts-v1/;
>>>> +
>>>> +/ {
>>>> +       model = "ST STV0991 application board";
>>>> +       compatible = "st,stv0991";
>>>> +       #address-cells = <1>;
>>>> +       #size-cells = <1>;
>>>> +
>>>> +       chosen {
>>>> +               stdout-path = &uart0;
>>>> +       };
>>>> +
>>>> +       memory {
>>>> +               device_type="memory";
>>>> +               reg = <0x0 0x198000>;
>>>> +       };
>>>> +
>>>> +       uart0: serial at 0x80406000 {
>>>> +               compatible = "arm,pl011", "arm,primecell";
>>>> +               reg = <0x80406000 0x1000>;
>>>> +               clock = <2700000>;
>>>> +       };
>>>> +};
>>>> diff --git a/configs/stv0991_defconfig b/configs/stv0991_defconfig
>>>> index 76ba41b..d9edc06 100644
>>>> --- a/configs/stv0991_defconfig
>>>> +++ b/configs/stv0991_defconfig
>>>> @@ -5,3 +5,4 @@ CONFIG_SYS_MALLOC_F_LEN=0x2000
>>>>  CONFIG_ETH_DESIGNWARE=y
>>>>  CONFIG_NETDEVICES=y
>>>>  CONFIG_NET=y
>>>> +CONFIG_DEFAULT_DEVICE_TREE="stv0991"
>>>> diff --git a/include/configs/stv0991.h b/include/configs/stv0991.h
>>>> index 2f65eda..750eebd 100644
>>>> --- a/include/configs/stv0991.h
>>>> +++ b/include/configs/stv0991.h
>>>> @@ -80,4 +80,7 @@
>>>>  #define CONFIG_AUTOBOOT_PROMPT                 \
>>>>         "Hit SPACE in %d seconds to stop autoboot.\n", bootdelay
>>>>  #undef CONFIG_HAS_VBAR
>>>> +#define CONFIG_OF_EMBED
>>>> +#define CONFIG_OF_CONTROL
>>>> +#define CONFIG_OF_LIBFDT
>>>>  #endif /* __CONFIG_H */
>>>> --
>>>> 1.7.9.5
>>>>
>>> Regards,
>>> Simon
> Regards,
> Simon

  reply	other threads:[~2015-05-01 23:43 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-01 22:43 [U-Boot] [PATCH 0/2] stv0991: Add flat device tree support Vikas Manocha
2015-05-01 22:43 ` [U-Boot] [PATCH 1/2] stv0991: fdt: add stv0991 device tree Vikas Manocha
2015-05-01 23:01   ` Simon Glass
2015-05-01 23:18     ` vikasm
2015-05-01 23:45       ` Simon Glass
2015-05-01 23:43         ` vikasm [this message]
2015-05-01 22:43 ` [U-Boot] [PATCH 2/2] stv0991: use fdt for serial port platform data Vikas Manocha
2015-05-01 23:02   ` Simon Glass

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=55440F84.90004@st.com \
    --to=vikas.manocha@st.com \
    --cc=u-boot@lists.denx.de \
    /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: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox