Re: [U-Boot] [PATCH 01/10] ARM:dts: Added device tree for socfpga arria10 development kit sdmmc

2016-12-18 Thread Chee, Tien Fong
On Rab, 2016-12-07 at 14:52 +0100, Marek Vasut wrote:
> On 12/07/2016 11:30 AM, Chee, Tien Fong wrote:
> > 
> > On Sel, 2016-12-06 at 13:44 +0100, Marek Vasut wrote:
> > > 
> > > On 12/06/2016 08:50 AM, Chee Tien Fong wrote:
> > > > 
> > > > 
> > > > From: Tien Fong Chee 
> > > > 
> > > > This is initial version of device tree for the Intel socfpga
> > > > arria10
> > > > development kit with sdmmc.
> > > > 
> > > > Signed-off-by: Tien Fong Chee 
> > > > Cc: Marek Vasut 
> > > > Cc: Dinh Nguyen 
> > > > Cc: Chin Liang See 
> > > > Cc: Tien Fong 
> > > > ---
> > > >  arch/arm/dts/Makefile   |3 +-
> > > >  arch/arm/dts/socfpga_arria10.dtsi   |  859
> > > > +++
> > > >  arch/arm/dts/socfpga_arria10_handoff_sdmmc.dtsi |  473
> > > > +
> > > >  arch/arm/dts/socfpga_arria10_socdk_sdmmc.dts|   33 +
> > > >  4 files changed, 1367 insertions(+), 1 deletions(-)
> > > >  create mode 100644 arch/arm/dts/socfpga_arria10.dtsi
> > > >  create mode 100644
> > > > arch/arm/dts/socfpga_arria10_handoff_sdmmc.dtsi
> > > Isn't the handoff file board-specific ?
> > > 
> > board and flash specific. How bout this name
> > "socfpga_arria10_socdk_sdmmc_handoff"?
> Certainly better
> 
> > 
> > > 
> > > > 
> > > >  create mode 100644
> > > > arch/arm/dts/socfpga_arria10_socdk_sdmmc.dts
> > > Does this patchset somehow supersede the previous one [1] or what
> > > is
> > > the
> > > plan here ?
> > > 
> > Yeah, the series patches here based on 01-arria10.
> Just rebase and repost the whole thing on top of u-boot/master
> 
Okay,noted.
> > 
> > > 
> > > [1]
> > > http://git.denx.de/?p=u-boot/u-boot-socfpga.git;a=shortlog;h=refs
> > > /hea
> > > ds/01-arria10
> 
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 01/10] ARM:dts: Added device tree for socfpga arria10 development kit sdmmc

2016-12-07 Thread Marek Vasut
On 12/07/2016 11:30 AM, Chee, Tien Fong wrote:
> On Sel, 2016-12-06 at 13:44 +0100, Marek Vasut wrote:
>> On 12/06/2016 08:50 AM, Chee Tien Fong wrote:
>>>
>>> From: Tien Fong Chee 
>>>
>>> This is initial version of device tree for the Intel socfpga
>>> arria10
>>> development kit with sdmmc.
>>>
>>> Signed-off-by: Tien Fong Chee 
>>> Cc: Marek Vasut 
>>> Cc: Dinh Nguyen 
>>> Cc: Chin Liang See 
>>> Cc: Tien Fong 
>>> ---
>>>  arch/arm/dts/Makefile   |3 +-
>>>  arch/arm/dts/socfpga_arria10.dtsi   |  859
>>> +++
>>>  arch/arm/dts/socfpga_arria10_handoff_sdmmc.dtsi |  473
>>> +
>>>  arch/arm/dts/socfpga_arria10_socdk_sdmmc.dts|   33 +
>>>  4 files changed, 1367 insertions(+), 1 deletions(-)
>>>  create mode 100644 arch/arm/dts/socfpga_arria10.dtsi
>>>  create mode 100644 arch/arm/dts/socfpga_arria10_handoff_sdmmc.dtsi
>> Isn't the handoff file board-specific ?
>>
> board and flash specific. How bout this name
> "socfpga_arria10_socdk_sdmmc_handoff"?

Certainly better

>>>  create mode 100644 arch/arm/dts/socfpga_arria10_socdk_sdmmc.dts
>> Does this patchset somehow supersede the previous one [1] or what is
>> the
>> plan here ?
>>
> Yeah, the series patches here based on 01-arria10.

Just rebase and repost the whole thing on top of u-boot/master

>> [1]
>> http://git.denx.de/?p=u-boot/u-boot-socfpga.git;a=shortlog;h=refs/hea
>> ds/01-arria10


-- 
Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 01/10] ARM:dts: Added device tree for socfpga arria10 development kit sdmmc

2016-12-07 Thread Chee, Tien Fong
On Sel, 2016-12-06 at 13:44 +0100, Marek Vasut wrote:
> On 12/06/2016 08:50 AM, Chee Tien Fong wrote:
> > 
> > From: Tien Fong Chee 
> > 
> > This is initial version of device tree for the Intel socfpga
> > arria10
> > development kit with sdmmc.
> > 
> > Signed-off-by: Tien Fong Chee 
> > Cc: Marek Vasut 
> > Cc: Dinh Nguyen 
> > Cc: Chin Liang See 
> > Cc: Tien Fong 
> > ---
> >  arch/arm/dts/Makefile   |3 +-
> >  arch/arm/dts/socfpga_arria10.dtsi   |  859
> > +++
> >  arch/arm/dts/socfpga_arria10_handoff_sdmmc.dtsi |  473
> > +
> >  arch/arm/dts/socfpga_arria10_socdk_sdmmc.dts|   33 +
> >  4 files changed, 1367 insertions(+), 1 deletions(-)
> >  create mode 100644 arch/arm/dts/socfpga_arria10.dtsi
> >  create mode 100644 arch/arm/dts/socfpga_arria10_handoff_sdmmc.dtsi
> Isn't the handoff file board-specific ?
> 
board and flash specific. How bout this name
"socfpga_arria10_socdk_sdmmc_handoff"?
> > 
> >  create mode 100644 arch/arm/dts/socfpga_arria10_socdk_sdmmc.dts
> Does this patchset somehow supersede the previous one [1] or what is
> the
> plan here ?
> 
Yeah, the series patches here based on 01-arria10.
> [1]
> http://git.denx.de/?p=u-boot/u-boot-socfpga.git;a=shortlog;h=refs/hea
> ds/01-arria10
> 
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 01/10] ARM:dts: Added device tree for socfpga arria10 development kit sdmmc

2016-12-06 Thread Marek Vasut
On 12/06/2016 08:50 AM, Chee Tien Fong wrote:
> From: Tien Fong Chee 
> 
> This is initial version of device tree for the Intel socfpga arria10
> development kit with sdmmc.
> 
> Signed-off-by: Tien Fong Chee 
> Cc: Marek Vasut 
> Cc: Dinh Nguyen 
> Cc: Chin Liang See 
> Cc: Tien Fong 
> ---
>  arch/arm/dts/Makefile   |3 +-
>  arch/arm/dts/socfpga_arria10.dtsi   |  859 
> +++
>  arch/arm/dts/socfpga_arria10_handoff_sdmmc.dtsi |  473 +
>  arch/arm/dts/socfpga_arria10_socdk_sdmmc.dts|   33 +
>  4 files changed, 1367 insertions(+), 1 deletions(-)
>  create mode 100644 arch/arm/dts/socfpga_arria10.dtsi
>  create mode 100644 arch/arm/dts/socfpga_arria10_handoff_sdmmc.dtsi

Isn't the handoff file board-specific ?

>  create mode 100644 arch/arm/dts/socfpga_arria10_socdk_sdmmc.dts

Does this patchset somehow supersede the previous one [1] or what is the
plan here ?

[1]
http://git.denx.de/?p=u-boot/u-boot-socfpga.git;a=shortlog;h=refs/heads/01-arria10

-- 
Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot