Re: arndale ethernet not working on u-boot

2013-11-08 Thread Tushar Behera
On 8 November 2013 13:44, Daniel Lezcano daniel.lezc...@linaro.org wrote:

 Hi all,

 I have an arndale and I am trying to have the ethernet working in u-boot but
 without success.

 (Re)start USB...
 USB0: USB EHCI 1.00
 scanning bus 0 for devices... 2 USB Device(s) found
 scanning usb for storage devices... 0 Storage Device(s) found
 scanning usb for ethernet devices... 0 Ethernet Device(s) found

 I found different bugs in launchpad but without this solved.

 I tried different u-boot but without success.

 Did someone faced this issue and solved it ?

 Thanks

 -- Daniel


We have observed the issue with upstream u-boot, but haven't got any
solution to this problem yet.

-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: arndale board power measurement

2013-07-23 Thread Tushar Behera
On 07/19/2013 08:39 PM, Rajiv Nishtala wrote:
 Hi,
 
 My name is Rajiv Nishtala and I am PhD student at UPC, Barcelona.
 
 I am trying to run some benchmarks on the arndale board and measure the
 power reading from it.
 
 is there a way to measure the power from the board directly, I tried using
 the wattsup .net meter and was not successful.
 
 At the moment, I am trying to use 32 system configuration registers,
 sys_cfs to measure the power, but the method seems to be very difficult for
 me. Would you suggest using any other technique?
 
 From the /sys/ folder, i found the following:
 
 /sys/devices/voltage-regulator.5/regulator/regulator.1
 
 more uevent
 OF_NAME=voltage-regulator
 OF_FULLNAME=/voltage-regulator
 OF_COMPATIBLE_0=regulator-fixed
 OF_COMPATIBLE_N=1
 
 But i am not sure what voltage it is referring to. Could you please shed
 some light here.
 
 Thanks.
 Rajiv Nishtala
 
 

I haven't worked much on power measurement on Arndale, but there are
others in Linaro who are actively using Arndale board for various power
management activities. Ccing the linaro developers' mailing list where
you can get some interesting information.

-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [ANNOUNCE] linux-linaro kernel schedule / llct age

2013-06-03 Thread Tushar Behera
 Timely updates on llct for each mainline -rc are very interesting
 though, in fact we depend on them.


+1.

-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Mali-ization

2013-04-17 Thread Tushar Behera
On 04/16/2013 09:46 AM, Andy Green wrote:
 On 16/04/13 10:08, the mail apparently from Selina Kuo included:
 Hi, John,

 Your assumption is right. The ump code is part of the out-of-tree mali
 driver.


 - Selina Kuo, one of Andy's colleagues ^^
 
 As Selina says it's a code drop we got via Fujitsu from ARM for the OOT
 Mali driver.
 
 However that code is all GPL2, as it should be.
 
 I think any of the options are good,
 
  - make our own repo and keep it in sync with llct
 
  - privately encourage ARM to keep it in sync with Linus' HEAD, publicly
 
  - upstream it so it's always in sync
 
 This obviously helps of all LT who want to / can harmonize their tree on
 llct basis.
 
 Tushar, do you have any opinion?
 

It would always help to have the Mali driver synced with latest upstream
and getting merged into 'llct'. The LT's who need to use this driver can
have their modifications (if any) on top of it.

 Anyone who dealt with this code or at ARM (Tixy?)
 
 FWIW our TSC rep is aware of this and I believe would support whatever
 the best-looking of these solutions is.
 
 -Andy
 
 On 16 April 2013 07:50, John Stultz john.stu...@linaro.org wrote:
 On 04/13/2013 03:22 AM, Andy Green wrote:

 On 13/04/13 09:07, the mail apparently from Andy Green included:

 I'm hoping someone else will write the patches ^^  but if not I'll try
 to sort something out.


 The attached series gets it building cleanly against current llct with
 ION.


 Cool. Although the patches look like they are all against the ump
 driver(which I'm not familiar with), as opposed to changes to the ION
 infrastructure itself. So they won't apply to the AOSP trees, and I
 can't
 send them upstream.

 I assume the ump code is part of the out-of-tree mali driver? Looking at
 llct, I don't see a drivers/base/ump directory.

 https://git.linaro.org/gitweb?p=kernel/linux-linaro-tracking.git;a=tree;f=drivers/base;h=cbad2d664fa248ec366430dbd1724b2959ae43ee;hb=refs/heads/linux-linaro-core-tracking


 But I'm guessing this is the point of your original mail in this
 thread: we
 need a mali tree upon which fixes like these can be applied and then
 pulled
 into llct.

 Or even better, can we get the mali devs to publish a repo of their
 latest
 work (to which fixes like Andy's can be submitted to) which can also be
 pulled into llct?

 thanks
 -john



 ___
 linaro-dev mailing list
 linaro-dev@lists.linaro.org
 http://lists.linaro.org/mailman/listinfo/linaro-dev
 
 


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Samsung topic for 13.03

2013-04-04 Thread Tushar Behera
On 04/04/2013 12:40 PM, Alexander Spyridakis wrote:
 On 21 March 2013 06:33, Tushar Behera tushar.beh...@linaro.org wrote:
 
 I have been observing this system hang when we enable LPAE on 3.9 based
 llct, though the issue is not present with vanilla 3.9-rc kernel. Would
 update later after I debug further.

 
 Hello,
 
 I am not sure if this is still relevant but I get the same behaviour both
 on Arndale and Versatile Express with the latest linaro release.
 
 Check my suggested fix in
 linaro-kernelhttp://lists.linaro.org/pipermail/linaro-kernel/2013-April/003585.html
 .
 
 Regards.
 

We have got an alternate fix here. This would be part of next release.
It would be nice if you can test that at your end too.

https://git.linaro.org/gitweb?p=landing-teams/working/samsung/kernel.git;a=commit;h=2265cef13cc7f3a131d68f3d175299dd23eac5c2

-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Looking for Linaro specific kernel

2013-04-04 Thread Tushar Behera
On 04/04/2013 09:31 PM, Jon Medhurst (Tixy) wrote:
 On Wed, 2013-04-03 at 16:08 +, Young, Chad wrote:
 I am looking for the kernel source 3.5.0-rc7 for the ARM versatile express 
 system, does anyone know where I can find it?
 
 I've found another trick, if you can actually booting a Linaro kernel
 that you want the source for then early during boot you should see on
 the serial console something like:
 
 [0.00] Linux version 3.9.0-rc3-00212-g0f66281 ...
 

You won't get the commit ID always. It basically prints the commit ID if
current HEAD is not signed-tagged (or something like that). That is why,
the hwpack kernels print like this.

Linux version 3.9.0-1-linaro-arndale

 or if you have a console running on the device entering
 
uname -a
 
 will give the same result. The part of the name after the '-g' part is
 the SHA1 of the git commit from which the kernel was built, e.g. in the
 above example it is 0f66281.
 
 You can then use this commit hash at the end of the URL for the gitweb
 interface of the to the git repo like this:
 
 https://git.linaro.org/gitweb?p=kernel/linux-linaro-tracking.git;a=commit;h=0f66281
 
 and if the kernel was indeed built from that git repo then you should
 see the tip commit of the kernel source used to build the kernel.
 
 Now, I don't know how long our kernel names have included the SHA1 of
 the commit, so for old kernels this may not work. Also, some kernel
 releases didn't come out of the main Linaro kernel git. So this method
 isn't foolproof.
 


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Test Result Summary of Linaro 13.03 Release for Linux Linaro ubuntu Quantal.

2013-03-31 Thread Tushar Behera
On 03/28/2013 03:16 PM, Botao Sun wrote:
 Linaro 13.03 Release (Calendar Week 13): Here is test result summary for
 Linux Linaro ubuntu Quantal image on following boards:
 
 https://docs.google.com/a/linaro.org/spreadsheet/ccc?key=0AgB-fT5LL31CdEtLTUtheGpkUU93V19EQnlYQjR4dXc#gid=2
 

[ ... ]

 2. Samsung Arndale + Linux Linaro Quantal (Column I):
 
 https://docs.google.com/a/linaro.org/spreadsheet/ccc?key=0AgB-fT5LL31CdGZJSFdTUWFFYVdhZl8wMFpxLXd2TXc#gid=0
 
 It keeps same status as last week: system hangs during the Power Management
 test, HDMI display is unavailable. All other features work well.
 

I tested with the pre-built image [1]. HDMI is working at my setup.

Can you please paste the bootlog?

Alternately, assuming it was because of EDID failure, can you add
following environment variable to bootargs and test?

drm_kms_helper.edid_firmware=edid-1920x1080.fw

[1]
https://releases.linaro.org/13.03/ubuntu/arndale/arndale-quantal_server_20130324-276.img.gz
-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Samsung topic for 13.03

2013-03-20 Thread Tushar Behera
 Looks like the ll-20130320.0 kernel failed to boot on Arndale:
 http://validation.linaro.org/lava-server/scheduler/job/49985/log_file
 
 The build log is here:
 https://ci.linaro.org/jenkins/job/linux-linaro-tracking_arndale/62/consoleFull
 
 
 The kernel config has been made by the following command:
 'ARCH=arm scripts/kconfig/merge_config.sh
 linaro/configs/linaro-base.conf linaro/configs/ubuntu-minimal.conf
 linaro/configs/arndale.conf linaro/configs/kvm-host.conf'
 
 Tushar,
 Have you tried this configuration?
 
 Thanks,
 Andrey

I have been observing this system hang when we enable LPAE on 3.9 based
llct, though the issue is not present with vanilla 3.9-rc kernel. Would
update later after I debug further.

Riku,
How much essential is it to enable LPAE support for KVM?

-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [ANNOUNCE] linux-linaro kernel schedule for 13.01

2013-01-15 Thread Tushar Behera
On 01/15/2013 02:59 PM, Andrey Konovalov wrote:
 Greetings,
 
 The current 13.01 schedule has been published on wiki:
 
 https://wiki.linaro.org/Platform/DevPlatform/LinuxLinaroKernelSchedule
 
 Initial versions of llct and ll trees has been pushed to g.l.o,
 kernel/linux-linaro-tracking.git yesterday.
 
 The 13.01 release would be v3.8-rc4 based (tentative).
 
 Some changes vs 12.12:
  * new topic in ll for better Snowball support.
Currently this is device tree, serial console, emmc, ethernet.
  * the perf-android patch(es) would be a separate ll topic.
 
 
 Thanks,
 Andrey
 
 ___
 linaro-dev mailing list
 linaro-dev@lists.linaro.org
 http://lists.linaro.org/mailman/listinfo/linaro-dev

Samsung LT tree supporting Origen board on v3.8-rc3 has been pushed to glo.

-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH] clk: remove unreachable code

2013-01-08 Thread Tushar Behera
On 01/08/2013 06:33 PM, Rajagopal Venkat wrote:
 while reparenting a clock, NULL check is done for clock in
 consideration and its new parent. So re-check is not required.
 If done, else part becomes unreachable.
 
 Signed-off-by: Rajagopal Venkat rajagopal.ven...@linaro.org
 ---
  drivers/clk/clk.c |5 +
  1 file changed, 1 insertion(+), 4 deletions(-)
 
 diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c
 index 251e45d..f896584 100644
 --- a/drivers/clk/clk.c
 +++ b/drivers/clk/clk.c
 @@ -1048,10 +1048,7 @@ void __clk_reparent(struct clk *clk, struct clk 
 *new_parent)
  
   hlist_del(clk-child_node);
  
 - if (new_parent)
 - hlist_add_head(clk-child_node, new_parent-children);
 - else
 - hlist_add_head(clk-child_node, clk_orphan_list);
 + hlist_add_head(clk-child_node, new_parent-children);
  
  #ifdef CONFIG_COMMON_CLK_DEBUG
   if (!inited)
 

The same logic holds good for following piece of code too.

1060 |---if (new_parent)
1061 |---|---new_parent_d = new_parent-dentry;
1062 |---else
1063 |---|---new_parent_d = orphandir;


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH] clk: remove unreachable code

2013-01-08 Thread Tushar Behera
On 01/09/2013 11:59 AM, Rajagopal Venkat wrote:
 while reparenting a clock, NULL check is done for clock in
 consideration and its new parent. So re-check is not required.
 If done, else part becomes unreachable.
 
 Signed-off-by: Rajagopal Venkat rajagopal.ven...@linaro.org
 ---

It is good to have revision history of the patches (version number and
changelog).

  drivers/clk/clk.c |   13 ++---
  1 file changed, 2 insertions(+), 11 deletions(-)
 
 diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c
 index 251e45d..1c4097c 100644
 --- a/drivers/clk/clk.c
 +++ b/drivers/clk/clk.c
 @@ -1040,7 +1040,6 @@ void __clk_reparent(struct clk *clk, struct clk 
 *new_parent)
  {
  #ifdef CONFIG_COMMON_CLK_DEBUG
   struct dentry *d;
 - struct dentry *new_parent_d;
  #endif
  
   if (!clk || !new_parent)
 @@ -1048,22 +1047,14 @@ void __clk_reparent(struct clk *clk, struct clk 
 *new_parent)
  
   hlist_del(clk-child_node);
  
 - if (new_parent)
 - hlist_add_head(clk-child_node, new_parent-children);
 - else
 - hlist_add_head(clk-child_node, clk_orphan_list);
 + hlist_add_head(clk-child_node, new_parent-children);
  
  #ifdef CONFIG_COMMON_CLK_DEBUG
   if (!inited)
   goto out;
  
 - if (new_parent)
 - new_parent_d = new_parent-dentry;
 - else
 - new_parent_d = orphandir;
 -
   d = debugfs_rename(clk-dentry-d_parent, clk-dentry,
 - new_parent_d, clk-name);
 + new_parent-dentry, clk-name);
   if (d)
   clk-dentry = d;
   else
 


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: When do we plan to move to Linux 3.8?

2013-01-06 Thread Tushar Behera
On 01/04/2013 08:03 PM, Andrey Konovalov wrote:
 On 01/04/2013 02:28 PM, Jon Medhurst (Tixy) wrote:
 On Fri, 2013-01-04 at 15:14 +0530, Viresh Kumar wrote:
 On 4 January 2013 15:09, Jon Medhurst (Tixy) t...@linaro.org wrote:
 I assume we'll be moving to Linux 3.8 for the January release cycle?
 
 For the 13.01 my plan was to stay at 3.7 plus the stable 3.7.y updates
 (as by the time of code freeze the mainline would be at v3.8-rc4 or so -
 not sure if -rc4 would be stable enough).
 
 But if the topic owners fill comfortable with moving to 3.8, we can go
 that way. Just would like to get confirmations from the owners of the
 heaviest topics.
 
 For my part I have prepared a 3.8 branch for vexpress [1] which doesn't
 yet contain Android patches or bit.LITTLE MP as their respective
 branches aren't on 3.8 yet.

 big.LITTLE MP is 3.8 based now :)
 
 I take it as the vexpress topic (for the ll tree) and the big.LITTLE
 topic (for the llct tree) are ready to switch to 3.8, correct?
 
 This leaves the Android topic for llct (John?) and Samsung LT topic for
 ll (Tushar?).
 

I am ok with 3.8 migration. Samsung LT tree would be available around
-rc3 time.

 
 
 Thanks,
 Andrey


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [ANNOUNCE] linux-linaro kernel schedule for 12.12

2012-12-13 Thread Tushar Behera
On 12/13/2012 02:14 AM, Andrey Konovalov wrote:
 Greetings,
 
 The ll tree has been updated to use llct-20121211.0 as the base.
 It still has just the ARM LT topic; waiting for the v3.7 based topic
 from Samsung LT.

v3.7 based Samsung LT kernel has been pushed to glo. [1]

[1]
http://git.linaro.org/gitweb?p=landing-teams/working/samsung/kernel.git;a=shortlog;h=refs/heads/tracking

Issues:
* Ethernet still broken on Ubuntu. (Same status as 3.7-rc6, interesting
Android kernel built from the same source has working ethernet connection.)
* Android kernel doesn't compile because of some change in gpu/ion. Even
after adding fixes for compilation errors there is crash during booting,
so didn't push those fixes.

I hope we will have some more time for any critical fixes.

 Also the hack by Bero to make perf to build on android has been removed:
 will see if it break the perf build on ubuntu (see the
 LinuxLinaroKernelSchedule for more details).
 
 * December 13 is the last date to update ll for 12.12 *
 
 The current status and the schedule is:
 https://wiki.linaro.org/Platform/DevPlatform/LinuxLinaroKernelSchedule
 
 Thanks,
 Andrey
 
 On 12/11/2012 01:54 PM, Andrey Konovalov wrote:
 The llct based on v3.7 release is there, and is tagged llct-20121211.0
 
 
 ___
 linaro-dev mailing list
 linaro-dev@lists.linaro.org
 http://lists.linaro.org/mailman/listinfo/linaro-dev


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH 3/3] netfilter: xt_quota2: Remove extra parameter from netlink_kernel_create

2012-11-15 Thread Tushar Behera
Required as per commit 9f00d9776bc5 (netlink: hide struct module
parameter in netlink_kernel_create).

Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
 net/netfilter/xt_quota2.c |5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/net/netfilter/xt_quota2.c b/net/netfilter/xt_quota2.c
index 3a9c1f9..8163f37 100644
--- a/net/netfilter/xt_quota2.c
+++ b/net/netfilter/xt_quota2.c
@@ -350,14 +350,15 @@ static struct xt_match quota_mt2_reg[] __read_mostly = {
 static int __init quota_mt2_init(void)
 {
int ret;
+#ifdef CONFIG_NETFILTER_XT_MATCH_QUOTA2_LOG
struct netlink_kernel_cfg cfg = {
.groups = 1,
};
+#endif
pr_debug(xt_quota2: init());
 
 #ifdef CONFIG_NETFILTER_XT_MATCH_QUOTA2_LOG
-   nflognl = netlink_kernel_create(init_net, NETLINK_NFLOG,
-   THIS_MODULE, cfg);
+   nflognl = netlink_kernel_create(init_net, NETLINK_NFLOG, cfg);
if (!nflognl)
return -ENOMEM;
 #endif
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH 0/3] Fixing build error with android-3.7

2012-11-15 Thread Tushar Behera
Following patches are required on android-3.7 branch for the androidized
kernel for Origen to build.

Needs testing on USB gadget functionalities.

Tushar Behera (3):
  usb: gadget: android: Fix build error because of removal of
usb_gadget_controller_number
  usb: gadget: android: Fix build error because of change in composite
driver framework
  netfilter: xt_quota2: Remove extra parameter from
netlink_kernel_create

 drivers/usb/gadget/android.c   |   18 +-
 drivers/usb/gadget/composite.c |5 -
 net/netfilter/xt_quota2.c  |5 +++--
 3 files changed, 12 insertions(+), 16 deletions(-)

-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH 2/3] usb: gadget: android: Fix build error because of change in composite driver framework

2012-11-15 Thread Tushar Behera
Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
 drivers/usb/gadget/android.c   |7 ---
 drivers/usb/gadget/composite.c |5 -
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/drivers/usb/gadget/android.c b/drivers/usb/gadget/android.c
index c26d7be..2b11055 100644
--- a/drivers/usb/gadget/android.c
+++ b/drivers/usb/gadget/android.c
@@ -1175,6 +1175,7 @@ static struct usb_composite_driver android_usb_driver = {
.name   = android_usb,
.dev= device_desc,
.strings= dev_strings,
+   .bind   = android_bind,
.unbind = android_usb_unbind,
.max_speed  = USB_SPEED_HIGH,
 };
@@ -1291,10 +1292,10 @@ static int __init init(void)
_android_dev = dev;
 
/* Override composite driver functions */
-   composite_driver.setup = android_setup;
-   composite_driver.disconnect = android_disconnect;
+   composite_driver_template.setup = android_setup;
+   composite_driver_template.disconnect = android_disconnect;
 
-   return usb_composite_probe(android_usb_driver, android_bind);
+   return usb_composite_probe(android_usb_driver);
 }
 module_init(init);
 
diff --git a/drivers/usb/gadget/composite.c b/drivers/usb/gadget/composite.c
index 957f973..c4460a5 100644
--- a/drivers/usb/gadget/composite.c
+++ b/drivers/usb/gadget/composite.c
@@ -1528,8 +1528,11 @@ composite_resume(struct usb_gadget *gadget)
 }
 
 /*-*/
-
+#if IS_ENABLED(CONFIG_USB_G_ANDROID)
+static struct usb_gadget_driver composite_driver_template = {
+#else
 static const struct usb_gadget_driver composite_driver_template = {
+#endif
.bind   = composite_bind,
.unbind = composite_unbind,
 
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH 1/3] usb: gadget: android: Fix build error because of removal of usb_gadget_controller_number

2012-11-15 Thread Tushar Behera
Required as per commit ed9cbda (usb: gadget: remove
usb_gadget_controller_number()).

Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
 drivers/usb/gadget/android.c |   11 +--
 1 files changed, 1 insertions(+), 10 deletions(-)

diff --git a/drivers/usb/gadget/android.c b/drivers/usb/gadget/android.c
index d2c3393..c26d7be 100644
--- a/drivers/usb/gadget/android.c
+++ b/drivers/usb/gadget/android.c
@@ -1118,7 +1118,7 @@ static int android_bind(struct usb_composite_dev *cdev)
 {
struct android_dev *dev = _android_dev;
struct usb_gadget   *gadget = cdev-gadget;
-   int gcnum, id, ret;
+   int id, ret;
 
/*
 * Start disconnected. Userspace will connect the gadget once
@@ -1156,15 +1156,6 @@ static int android_bind(struct usb_composite_dev *cdev)
strings_dev[STRING_SERIAL_IDX].id = id;
device_desc.iSerialNumber = id;
 
-   gcnum = usb_gadget_controller_number(gadget);
-   if (gcnum = 0)
-   device_desc.bcdDevice = cpu_to_le16(0x0200 + gcnum);
-   else {
-   pr_warning(%s: controller '%s' not recognized\n,
-   longname, gadget-name);
-   device_desc.bcdDevice = __constant_cpu_to_le16(0x);
-   }
-
usb_gadget_set_selfpowered(gadget);
dev-cdev = cdev;
 
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH] gpu: ion: Use RB_CLEAR_NODE instead of rb_init_node

2012-11-15 Thread Tushar Behera
rb_init_node() has been removed from the kernel, use alternate macro.

Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---

This patch also needs to go into android-3.7 tree.

 drivers/gpu/ion/ion.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/gpu/ion/ion.c b/drivers/gpu/ion/ion.c
index 1002ec0..baab410 100644
--- a/drivers/gpu/ion/ion.c
+++ b/drivers/gpu/ion/ion.c
@@ -191,7 +191,7 @@ static struct ion_handle *ion_handle_create(struct 
ion_client *client,
if (!handle)
return ERR_PTR(-ENOMEM);
kref_init(handle-ref);
-   rb_init_node(handle-node);
+   RB_CLEAR_NODE(handle-node);
handle-client = client;
ion_buffer_get(buffer);
handle-buffer = buffer;
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Add board config fragments to llct? (Re: Would like to add couple conf files to the config-boards-tracking branch)

2012-11-15 Thread Tushar Behera
On 11/15/2012 06:05 PM, Jon Medhurst (Tixy) wrote:
 On Thu, 2012-11-15 at 11:37 +, Jon Medhurst (Tixy) wrote:
 On Thu, 2012-11-15 at 09:20 +0530, Tushar Behera wrote:
 On 11/14/2012 07:41 PM, Ryan Harkin wrote:
 On 14 November 2012 11:38, Jon Medhurst (Tixy) t...@linaro.org wrote:
 [...]
 Now that the LT branches included in linux-linaro (ll) are based on
 llct, then they could modify the board configs in llct if required for
 the work in their topics. So at the moment, I can't think of a good
 reason not to pub all the board configs into llct. Can anyone else?

 I don't know if we need the board configs to be sourced from a single
 repo, or allow board configs to be included in llct from LT trees. One
 central repo means that people know where to go

 This seems like the easiest option to me.  Let's do it this way unless
 someone gives a valid objection.

 (Unless we had
 an official maintainer to manage all commits to the tree.)

 I assume this would have to be Andrey.  Andrey, are you OK with that?
 Or does someone else need to do it?

 Each LT that is using LLCT would have to send a patch to get their
 config updated.  So long as this happens in a timely manner, LTs
 should be able to live with that process.


 I suppose we are talking about the basic board config here. That should
 be ok. The new board enablement config fragments should always go in
 with the respective topic branches.

 I agree.
 
 Of course, this can't be done until Linaro's CI jobs and Ubuntu kernel
 packaging jobs support multiple config fragments, so for now we're stuck
 with one config per board. Shall we agree that all of these will live in
 the central location of the config-boards-tracking branch of
 git://git.linaro.org/kernel/configs.git ?
 

Fine with me as long as the basic board config gets pulled into llct. I
will post the Origen patch today.

-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Add board config fragments to llct? (Re: Would like to add couple conf files to the config-boards-tracking branch)

2012-11-15 Thread Tushar Behera
On 11/15/2012 05:31 PM, Ryan Harkin wrote:
 On 15 November 2012 11:37, Jon Medhurst (Tixy) t...@linaro.org wrote:
 On Thu, 2012-11-15 at 09:20 +0530, Tushar Behera wrote:
 On 11/14/2012 07:41 PM, Ryan Harkin wrote:
 On 14 November 2012 11:38, Jon Medhurst (Tixy) t...@linaro.org wrote:
 [...]
 Now that the LT branches included in linux-linaro (ll) are based on
 llct, then they could modify the board configs in llct if required for
 the work in their topics. So at the moment, I can't think of a good
 reason not to pub all the board configs into llct. Can anyone else?

 I don't know if we need the board configs to be sourced from a single
 repo, or allow board configs to be included in llct from LT trees. One
 central repo means that people know where to go

 This seems like the easiest option to me.  Let's do it this way unless
 someone gives a valid objection.

 (Unless we had
 an official maintainer to manage all commits to the tree.)

 I assume this would have to be Andrey.  Andrey, are you OK with that?
 Or does someone else need to do it?

 Each LT that is using LLCT would have to send a patch to get their
 config updated.  So long as this happens in a timely manner, LTs
 should be able to live with that process.


 I suppose we are talking about the basic board config here. That should
 be ok. The new board enablement config fragments should always go in
 with the respective topic branches.

 I agree.
 
 OK, can we do it then?  We're freezing for release today, but it's not
 a big change, so we can get it in, right?
 
 At some point in the past, Andrey asked:
 My question is if this topic branch would be kernel/configs.git,
 
 Sounds perfect
 
 
  config-boards-tracking branch or something else?
 
 I don't have an opinion.  We'll probably have to live with it for a
 long time, but as long as it looks sensible, I'm sure people will be
 ok...
 

config-boards-tracking branch already has got config fragment for Origen
board, but this config file has enabled all the config options used in
all the topic branches too. If we plan to keep it that way, I suggest to
add the basic board config file to config-core-tracking. (It makes sense
as we would be enabling only mainline components that is essential to
boot the board). Any further modification to these can always be done
through LT topic branches.

Andrey, let me know which way you would prefer the patch?

-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Add board config fragments to llct? (Re: Would like to add couple conf files to the config-boards-tracking branch)

2012-11-14 Thread Tushar Behera
On 11/14/2012 07:41 PM, Ryan Harkin wrote:
 On 14 November 2012 11:38, Jon Medhurst (Tixy) t...@linaro.org wrote:
 Adding linaro-dev list and replying with some comments...
 On Tue, 2012-11-13 at 20:20 +0400, Andrey Konovalov wrote:
 The llct tree itself has no suitable .conf or defconfig for vexpress at
 all. That's the problem (wrt the ci jobs).

 The easier way seemed to be a single kernel/configs.git,
 config-boards-tracking branch to provide the config fragments for all
 the llct jobs.
 But it creates several instances of the same board.conf files and adds
 confusion. Agreed.
 Should we do in the jenkins jobs something like 'git checkout
 arm_lt/integration-linaro-vexpress.conf -- linaro/configs/vexpress.conf'
 for vexpress and similar (but different and unique) command for the
 other boards?

 Yes, I see the problem. But getting CI jobs to pull configs direct from
 an LT tree seems like the wrong solution. I guess what people really
 need is configs in linux-linaro-core-tracking (llct) (I'm sure people
 have told me that before and I possibly didn't listen enough).

 Now that the LT branches included in linux-linaro (ll) are based on
 llct, then they could modify the board configs in llct if required for
 the work in their topics. So at the moment, I can't think of a good
 reason not to pub all the board configs into llct. Can anyone else?

 I don't know if we need the board configs to be sourced from a single
 repo, or allow board configs to be included in llct from LT trees. One
 central repo means that people know where to go
 
 This seems like the easiest option to me.  Let's do it this way unless
 someone gives a valid objection.
 
 (Unless we had
 an official maintainer to manage all commits to the tree.)
 
 I assume this would have to be Andrey.  Andrey, are you OK with that?
 Or does someone else need to do it?
 
 Each LT that is using LLCT would have to send a patch to get their
 config updated.  So long as this happens in a timely manner, LTs
 should be able to live with that process.
 

I suppose we are talking about the basic board config here. That should
be ok. The new board enablement config fragments should always go in
with the respective topic branches.

 
 Or course, once Linaro's build and test infrastructure supports config
 fragments fully, then we could have have separate config fragments for
 - basic board config
 - new board enablement
 - special features (e.g. big.LITTLE MP)
 - testing or benchmarking config
 and the configs could live in the tree relevant to the code they apply
 to rather than having a single central board config we have to manage.
 
 That sounds scarey - there would be no one place to get a config, but
 I guess, if you need a config for feature X, you'd also need the
 branch for feature X that contained the source and config, so it would
 work out fine.
 
 Cheers,
 Ryan.
 
 ___
 linaro-dev mailing list
 linaro-dev@lists.linaro.org
 http://lists.linaro.org/mailman/listinfo/linaro-dev
 


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: linux-linaro kernel trees move to 3.7 and 12.11 schedule

2012-11-07 Thread Tushar Behera
On 11/07/2012 01:40 AM, Andrey Konovalov wrote:
 Greetings,
 
 This cycle some more attention is given to the shape of the ci builds
 done from the llct tree (the ones with llct in their name at
 https://ci.linaro.org/jenkins/view/Linux%20Linaro%20Tracking/). See below.
 
 On 10/27/2012 12:39 PM, Andrey Konovalov wrote:
 The 12.11 linux-linaro kernel release will be v3.7-rc5 or v3.7-rc6 based.

 Please make sure to update your topics to 3.7.

 Here is the plan:
 * October 30: initial v3.7-rc3 based llct build, 3.6 based topics will
 not be included if there are conflicts
 
 Done. The tag is llct-20121101.1. v3.7-rc3 based.
 
 Origen failed to boot in LAVA (silence after Uncompressing Linux...
 done, booting the kernel.)

I haven't tested it yet (llct), but it might be because of incorrect
serial port number. I have tested v3.7-rc4 kernel with said change in
config file, it works. Anyways, I will get back tomorrow after testing llct.

 mmc doesn't work on Panda when the kernel is built using the (old)
 config fragments
 (http://validation.linaro.org/lava-server/scheduler/job/37832/log_file).
 Panda boots ok with omap2plus_defconfig.
 vexpress boots ok.
 
 * November 6: updated big-LITTLE-MP topic arrives, llct rebuild
 (llct-20121106)
 
 Done. The tag is llct-20121106.0. v3.7-rc4 based.
 
 Android and big-LITTLE-MP topics are back. New Gator version (v5.12).
 
 *ubuntu-sauce topic will be dropped* Anyone objects?
 
 The same boot failure for Origen. Disabling device tree in the kernel
 config makes the kernel to start booting
 (http://validation.linaro.org/lava-server/scheduler/job/37885/log_file).
 Could be arch/arm/boot/dts/exynos4210-origen.dts issue.
 
 Thanks,
 Andrey
 
 * November 8: linux-linaro (ll) rebuild based on llct-20121106
 * November 13: llct rebuild (llct-20121113)
 * November 15: ll rebuild based on llct-20121113
 * November 20: llct rebuild (llct-20121120)
 * November 22: ll rebuild based on llct-20121120, ll code freeze (no
 massive ll topics updates after that; bugfixes only)

 (The dates above are milestones, there will be additional llct/ll
 rebuilds on other dates as well)

 The config fragment branches to be used in 12.11:
 git://git.linaro.org/kernel/configs.git , config-core-tracking and
 config-boards-tracking branches.

 Thanks,
 Andrey

 
 
 ___
 linaro-dev mailing list
 linaro-dev@lists.linaro.org
 http://lists.linaro.org/mailman/listinfo/linaro-dev


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Planning topcs for inclusion in linux-linaro

2012-10-22 Thread Tushar Behera
On 17 October 2012 21:26, Andrey Konovalov andrey.konova...@linaro.org wrote:
 On 10/17/2012 10:41 AM, Tushar Behera wrote:

 On 10/06/2012 02:37 AM, Andrey Konovalov wrote:

 Greetings,

 Minor change to the plan:
   * The current llct is *llct-20121006.0*
   * October  9: ll rebuild based on llct-20121006.0
   * October 16: ll rebuild based on llct-20121012.N, ll code freeze (no
 massive ll topics updates after that; bugfixes only)


 Samsung LT kernel tree based on llct-20121012.0 has been pushed to g.l.o.

 git://git.linaro.org/landing-teams/working/samsung/kernel.git
 branch: tracking
 tag: samsung-lt-v3.6-1

 Let me know if there are issues after building linux-linaro tree. I
 would be out of office for rest of the week and would be checking the
 mails on coming Monday.


 I've created ll-20121017.0 using the updated Samsung LT's tracking topic.
 The difference from the previous topic version is dropping the two commits:
UBUNTU: dm-raid4-5: Fix compilation warning
and
CONFIG: ORIGEN: ANDROID: Disable HDMI/FIMC temporarily
 I've restored the UBUNTU: dm-raid4-5: Fix compilation warning one, as it
 seems to be the correct fix.

Thanks. This patch needs to picked up in ubuntu-sauce branch.

 So the resulting difference between ll-20121016.0 and ll-20121017.0 is just
 the android_origen_defconfig change:


That is right.

 diff --git a/arch/arm/configs/android_origen_defconfig
 b/arch/arm/configs/android_origen_defconfig
 index d04cf03..869183e 100644
 --- a/arch/arm/configs/android_origen_defconfig
 +++ b/arch/arm/configs/android_origen_defconfig
 @@ -125,6 +125,11 @@ CONFIG_VIDEO_S5K4ECGX_SLSI_4EC=y
  CONFIG_V4L_PLATFORM_DRIVERS=y
  CONFIG_V4L_USB_DRIVERS=y
  CONFIG_VIDEO_SAMSUNG_S5P_FIMC=y
 +CONFIG_VIDEO_S5P_FIMC=y
 +CONFIG_VIDEO_SAMSUNG_S5P_TV=y
 +CONFIG_VIDEO_SAMSUNG_S5P_HDMI=y
 +CONFIG_VIDEO_SAMSUNG_S5P_SDO=y
 +CONFIG_VIDEO_SAMSUNG_S5P_MIXER=y
  CONFIG_V4L_MEM2MEM_DRIVERS=y
  CONFIG_VIDEO_SAMSUNG_S5P_G2D=y
  CONFIG_VIDEO_SAMSUNG_S5P_JPEG=y


 Thanks,
 Andrey

 On 10/05/2012 05:25 PM, Jon Medhurst (Tixy) wrote:

 On Fri, 2012-10-05 at 17:03 +0400, Andrey Konovalov wrote:

 On 10/05/2012 12:12 PM, Jon Medhurst (Tixy) wrote:

 On Thu, 2012-10-04 at 23:02 +0400, Andrey Konovalov wrote:

 don't we need some kind of notification system where you
 announce a plan to rebuild linux-linaro and give us a version of
 llct to
 base our LT branches off?


 Yes, this makes sense.
 Would the following plan be OK:
 (N should normally be 0, but may be a small positive number)
 * till the end of this week please use llct-20121004.0 as the base
 for
 the linux-linaro (ll) topics
 * October  9: ll rebuild based on llct-20121005.N
 * October 16: ll rebuild based on llct-20121012.N
 * October 23: ll rebuild based on llct-20121019.N, ll code freeze
 (no
 massive ll topics updates after that; bugfixes only)


 Is this month following the normal release cycle? If so, isn't the
 release day Oct 24th, and release candidates are built from the
 previous
 Thurday's code, i.e. Oct 18th. I.e. the freeze for the release
 should be
 Oct 16th not 23rd?


 Oops.. Your are right. Why don't we have the Thursday, October 32
 this year?

 So the correct plan is the following:
 * till the end of this week please use llct-20121004.0 as the base
 for
 the linux-linaro (ll) topics
 * October  9: ll rebuild based on llct-20121005.N
 * October 16: ll rebuild based on llct-20121012.N, ll code
 freeze (no
   massive ll topics updates after that; bugfixes only)

 Also, I assume we're not going to try to move to Linux 3.7 this month?


 That's correct. We didn't have the kernel.org release based (vs -rc
 based) ll release for quite a while. And the plan is to stick to v3.6
 for 12.10 release unless someone needs to move to v3.7-rc* by all means
 this cycle.
 After October 19 the llct tree will move to v3.7-rc*, but the ll tree
 will be frozen till the 12.10 is out.


 That all sounds reasonable to me.








-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Planning topcs for inclusion in linux-linaro

2012-10-17 Thread Tushar Behera
On 10/06/2012 02:37 AM, Andrey Konovalov wrote:
 Greetings,
 
 Minor change to the plan:
  * The current llct is *llct-20121006.0*
  * October  9: ll rebuild based on llct-20121006.0
  * October 16: ll rebuild based on llct-20121012.N, ll code freeze (no
massive ll topics updates after that; bugfixes only)
 

Samsung LT kernel tree based on llct-20121012.0 has been pushed to g.l.o.

git://git.linaro.org/landing-teams/working/samsung/kernel.git
branch: tracking
tag: samsung-lt-v3.6-1

Let me know if there are issues after building linux-linaro tree. I
would be out of office for rest of the week and would be checking the
mails on coming Monday.

 Thanks,
 Andrey
 
 On 10/05/2012 05:25 PM, Jon Medhurst (Tixy) wrote:
 On Fri, 2012-10-05 at 17:03 +0400, Andrey Konovalov wrote:
 On 10/05/2012 12:12 PM, Jon Medhurst (Tixy) wrote:
 On Thu, 2012-10-04 at 23:02 +0400, Andrey Konovalov wrote:
 don't we need some kind of notification system where you
 announce a plan to rebuild linux-linaro and give us a version of
 llct to
 base our LT branches off?

 Yes, this makes sense.
 Would the following plan be OK:
 (N should normally be 0, but may be a small positive number)
 * till the end of this week please use llct-20121004.0 as the base for
 the linux-linaro (ll) topics
 * October  9: ll rebuild based on llct-20121005.N
 * October 16: ll rebuild based on llct-20121012.N
 * October 23: ll rebuild based on llct-20121019.N, ll code freeze
 (no
 massive ll topics updates after that; bugfixes only)

 Is this month following the normal release cycle? If so, isn't the
 release day Oct 24th, and release candidates are built from the
 previous
 Thurday's code, i.e. Oct 18th. I.e. the freeze for the release
 should be
 Oct 16th not 23rd?

 Oops.. Your are right. Why don't we have the Thursday, October 32
 this year?

 So the correct plan is the following:
* till the end of this week please use llct-20121004.0 as the base
 for
the linux-linaro (ll) topics
* October  9: ll rebuild based on llct-20121005.N
* October 16: ll rebuild based on llct-20121012.N, ll code
 freeze (no
  massive ll topics updates after that; bugfixes only)

 Also, I assume we're not going to try to move to Linux 3.7 this month?

 That's correct. We didn't have the kernel.org release based (vs -rc
 based) ll release for quite a while. And the plan is to stick to v3.6
 for 12.10 release unless someone needs to move to v3.7-rc* by all means
 this cycle.
 After October 19 the llct tree will move to v3.7-rc*, but the ll tree
 will be frozen till the 12.10 is out.

 That all sounds reasonable to me.

 


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Planning topcs for inclusion in linux-linaro

2012-10-07 Thread Tushar Behera
On 10/06/2012 02:37 AM, Andrey Konovalov wrote:
 Greetings,
 
 Minor change to the plan:
  * The current llct is *llct-20121006.0*
  * October  9: ll rebuild based on llct-20121006.0
  * October 16: ll rebuild based on llct-20121012.N, ll code freeze (no
massive ll topics updates after that; bugfixes only)
 

I am not in office during this week, so won't be able to update the
kernel. Current Samsung LT kernel is based on llct-20121004.0. I will be
updating this to llct-20121012.N next.

 Thanks,
 Andrey
 
 On 10/05/2012 05:25 PM, Jon Medhurst (Tixy) wrote:
 On Fri, 2012-10-05 at 17:03 +0400, Andrey Konovalov wrote:
 On 10/05/2012 12:12 PM, Jon Medhurst (Tixy) wrote:
 On Thu, 2012-10-04 at 23:02 +0400, Andrey Konovalov wrote:
 don't we need some kind of notification system where you
 announce a plan to rebuild linux-linaro and give us a version of
 llct to
 base our LT branches off?

 Yes, this makes sense.
 Would the following plan be OK:
 (N should normally be 0, but may be a small positive number)
 * till the end of this week please use llct-20121004.0 as the base for
 the linux-linaro (ll) topics
 * October  9: ll rebuild based on llct-20121005.N
 * October 16: ll rebuild based on llct-20121012.N
 * October 23: ll rebuild based on llct-20121019.N, ll code freeze
 (no
 massive ll topics updates after that; bugfixes only)

 Is this month following the normal release cycle? If so, isn't the
 release day Oct 24th, and release candidates are built from the
 previous
 Thurday's code, i.e. Oct 18th. I.e. the freeze for the release
 should be
 Oct 16th not 23rd?

 Oops.. Your are right. Why don't we have the Thursday, October 32
 this year?

 So the correct plan is the following:
* till the end of this week please use llct-20121004.0 as the base
 for
the linux-linaro (ll) topics
* October  9: ll rebuild based on llct-20121005.N
* October 16: ll rebuild based on llct-20121012.N, ll code
 freeze (no
  massive ll topics updates after that; bugfixes only)

 Also, I assume we're not going to try to move to Linux 3.7 this month?

 That's correct. We didn't have the kernel.org release based (vs -rc
 based) ll release for quite a while. And the plan is to stick to v3.6
 for 12.10 release unless someone needs to move to v3.7-rc* by all means
 this cycle.
 After October 19 the llct tree will move to v3.7-rc*, but the ll tree
 will be frozen till the 12.10 is out.

 That all sounds reasonable to me.

 


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Planning topcs for inclusion in linux-linaro

2012-10-05 Thread Tushar Behera
On 10/05/2012 01:42 PM, Jon Medhurst (Tixy) wrote:
 On Thu, 2012-10-04 at 23:02 +0400, Andrey Konovalov wrote:
 don't we need some kind of notification system where you
 announce a plan to rebuild linux-linaro and give us a version of llct to
 base our LT branches off?

 Yes, this makes sense.
 Would the following plan be OK:
 (N should normally be 0, but may be a small positive number)
 * till the end of this week please use llct-20121004.0 as the base for 
 the linux-linaro (ll) topics
 * October  9: ll rebuild based on llct-20121005.N
 * October 16: ll rebuild based on llct-20121012.N
 * October 23: ll rebuild based on llct-20121019.N, ll code freeze (no 
 massive ll topics updates after that; bugfixes only)
 
 Is this month following the normal release cycle? If so, isn't the
 release day Oct 24th, and release candidates are built from the previous
 Thurday's code, i.e. Oct 18th. I.e. the freeze for the release should be
 Oct 16th not 23rd?
 
 Also, I assume we're not going to try to move to Linux 3.7 this month?
 
Samsung LT release would be based on Linux-v3.6 final release, no plans
to move to 3.7 during 12.10.

-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH 0/3] kbuild modifications to print git commit ID

2012-09-18 Thread Tushar Behera
The bootlog in Ubuntu/Android images provide minimal information about
the current HEAD from which the kernel was built. This patchset enables
us to get detailed version info of the kernel.

Tested both on Ubuntu and Android images.

Patch 1 is already submitted to kernel-build ML and has been rejected.
Hence review of this patchset is required w.r.t. Android/Ubuntu LEB and
linux-linaro in mind.

Tushar Behera (3):
  kbuild: setlocalversion: ignore private tags while reporting local
version
  kbuild: Add support to extract information about current git commit
HEAD
  init: Add additional print for detailed kernel version

 Makefile|7 +--
 include/linux/printk.h  |1 +
 init/main.c |3 +++
 init/version.c  |3 +++
 scripts/setlocalversion |5 +++--
 5 files changed, 15 insertions(+), 4 deletions(-)

-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH 2/3] kbuild: Add support to extract information about current git commit HEAD

2012-09-18 Thread Tushar Behera
Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
 Makefile |7 +--
 1 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/Makefile b/Makefile
index 0f66f14..7e83768 100644
--- a/Makefile
+++ b/Makefile
@@ -374,6 +374,7 @@ KBUILD_LDFLAGS_MODULE := -T 
$(srctree)/scripts/module-common.lds
 # Read KERNELRELEASE from include/config/kernel.release (if it exists)
 KERNELRELEASE = $(shell cat include/config/kernel.release 2 /dev/null)
 KERNELVERSION = $(VERSION)$(if $(PATCHLEVEL),.$(PATCHLEVEL)$(if 
$(SUBLEVEL),.$(SUBLEVEL)))$(EXTRAVERSION)
+KERNELVERSIONLOCAL= $(shell cat .scmversion 2 /dev/null)
 
 export VERSION PATCHLEVEL SUBLEVEL KERNELRELEASE KERNELVERSION
 export ARCH SRCARCH CONFIG_SHELL HOSTCC HOSTCFLAGS CROSS_COMPILE AS LD CC
@@ -778,7 +779,8 @@ $(vmlinux-dirs): prepare scripts
 include/config/kernel.release: include/config/auto.conf FORCE
$(Q)rm -f $@
$(Q)echo $(KERNELVERSION)$$($(CONFIG_SHELL) 
$(srctree)/scripts/setlocalversion $(srctree))  $@
-
+   $(Q)rm -f .scmversion
+   $(Q)($(CONFIG_SHELL) $(srctree)/scripts/setlocalversion 
--save-scmversion $(srctree))
 
 # Things we need to do before we recursively start building the kernel
 # or the modules are listed in prepare.
@@ -829,7 +831,8 @@ define filechk_utsrelease.h
  echo '$(KERNELRELEASE) exceeds $(uts_len) characters' 2;\
  exit 1; \
fi;   \
-   (echo \#define UTS_RELEASE \$(KERNELRELEASE)\;)
+   (echo \#define UTS_RELEASE \$(KERNELRELEASE)\; \
+   echo \#define KERNEL_VERSION_LOCAL \$(KERNELVERSIONLOCAL)\;)
 endef
 
 define filechk_version.h
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH 3/3] init: Add additional print for detailed kernel version

2012-09-18 Thread Tushar Behera
When CONFIG_LOCALVERSION_AUTO is not defined, kernel boot log prints
only short version. This doesn't have any information regarding the
commit at which the kernel was compiled.

Adding an additional print statement to explicitly tell the current
HEAD.

Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
 include/linux/printk.h |1 +
 init/main.c|3 +++
 init/version.c |3 +++
 3 files changed, 7 insertions(+), 0 deletions(-)

diff --git a/include/linux/printk.h b/include/linux/printk.h
index 9afc01e..a2560f6 100644
--- a/include/linux/printk.h
+++ b/include/linux/printk.h
@@ -6,6 +6,7 @@
 
 extern const char linux_banner[];
 extern const char linux_proc_banner[];
+extern const char linux_scm_version_banner[];
 
 static inline int printk_get_level(const char *buffer)
 {
diff --git a/init/main.c b/init/main.c
index b286730..bad6b9b 100644
--- a/init/main.c
+++ b/init/main.c
@@ -494,6 +494,9 @@ asmlinkage void __init start_kernel(void)
boot_cpu_init();
page_address_init();
printk(KERN_NOTICE %s, linux_banner);
+#if !IS_ENABLED(CONFIG_LOCALVERSION_AUTO)
+   printk(KERN_NOTICE %s, linux_scm_version_banner);
+#endif
setup_arch(command_line);
mm_init_owner(init_mm, init_task);
mm_init_cpumask(init_mm);
diff --git a/init/version.c b/init/version.c
index 86fe0cc..e86cce9 100644
--- a/init/version.c
+++ b/init/version.c
@@ -46,3 +46,6 @@ const char linux_proc_banner[] =
%s version %s
 ( LINUX_COMPILE_BY @ LINUX_COMPILE_HOST )
 ( LINUX_COMPILER ) %s\n;
+
+const char linux_scm_version_banner [] =
+   Detailed version Linux UTS_RELEASE  KERNEL_VERSION_LOCAL \n;
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH] linaro/configs: ubuntu: Disable support for generic OHCI platform driver

2012-09-13 Thread Tushar Behera
OHCI-HCD driver does not support multiple SoC drivers during the compile
time. Hence the generic driver should be disabled in ubuntu.conf and related
OHCI Soc drivers should be enabled in respective board config files.

Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
 linaro/configs/ubuntu.conf |1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/linaro/configs/ubuntu.conf b/linaro/configs/ubuntu.conf
index 5d0a372..88e58df 100644
--- a/linaro/configs/ubuntu.conf
+++ b/linaro/configs/ubuntu.conf
@@ -1556,7 +1556,6 @@ CONFIG_USB_OXU210HP_HCD=m
 CONFIG_USB_ISP116X_HCD=m
 CONFIG_USB_ISP1760_HCD=m
 CONFIG_USB_OHCI_HCD=y
-CONFIG_USB_OHCI_HCD_PLATFORM=y
 CONFIG_USB_OHCI_LITTLE_ENDIAN=y
 CONFIG_USB_U132_HCD=m
 CONFIG_USB_SL811_HCD=m
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH 2/2] UBUNTU: dm-raid4-5: Fix compilation warning

2012-09-13 Thread Tushar Behera
Fixes following compilation warning.
ubuntu/dm-raid4-5/dm-raid4-5.c:4505:2: warning: initialization from 
incompatible pointer type [enabled by default]
ubuntu/dm-raid4-5/dm-raid4-5.c:4505:2: warning: (near initialization for 
‘raid_target.status’) [enabled by default]

Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
 ubuntu/dm-raid4-5/dm-raid4-5.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/ubuntu/dm-raid4-5/dm-raid4-5.c b/ubuntu/dm-raid4-5/dm-raid4-5.c
index e69ab44..9f55f58 100644
--- a/ubuntu/dm-raid4-5/dm-raid4-5.c
+++ b/ubuntu/dm-raid4-5/dm-raid4-5.c
@@ -4246,7 +4246,7 @@ static void raid_devel_stats(struct dm_target *ti, char 
*result,
 }
 
 static int raid_status(struct dm_target *ti, status_type_t type,
-  char *result, unsigned maxlen)
+  unsigned status_flags, char *result, unsigned maxlen)
 {
unsigned p, sz = 0;
char buf[BDEVNAME_SIZE];
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH 1/2] UBUNTU: dm-raid4-5: rename split_io to max_io_len

2012-09-13 Thread Tushar Behera
Commit 542f90381422 (dm: support non power of two target max_io_len)
renames struct dm_target:split_io variable to max_io_len.

Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
 ubuntu/dm-raid4-5/dm-raid4-5.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/ubuntu/dm-raid4-5/dm-raid4-5.c b/ubuntu/dm-raid4-5/dm-raid4-5.c
index e05b0e1..e69ab44 100644
--- a/ubuntu/dm-raid4-5/dm-raid4-5.c
+++ b/ubuntu/dm-raid4-5/dm-raid4-5.c
@@ -4073,7 +4073,7 @@ static int raid_ctr(struct dm_target *ti, unsigned argc, 
char **argv)
 * Make sure that dm core only hands maximum io size
 * length down and pays attention to io boundaries.
 */
-   ti-split_io = rs-set.io_size;
+   ti-max_io_len = rs-set.io_size;
ti-private = rs;
 
/* Initialize work queue to handle this RAID set's io. */
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH 0/2] Android fixes w.r.t. 3.6 kernel

2012-09-10 Thread Tushar Behera
Ping !!!

On 08/31/2012 09:57 AM, Tushar Behera wrote:
 Required for android-3.6 tree.
 
 Tushar Behera (2):
   netfilter: xt_quota2: Move away from NLMSG_PUT()
   netfilter: xt_quota2: Update parameter list in netlink_kernel_create
 
  net/netfilter/xt_quota2.c |   25 -
  1 files changed, 16 insertions(+), 9 deletions(-)
 
Without these patches, build of LT kernel based on llct(v3.6-rc5) fails
with following error message.

net/netfilter/xt_quota2.c: In function ‘quota2_log’:
net/netfilter/xt_quota2.c:85:2: error: implicit declaration of function
‘NLMSG_PUT’ [-Werror=implicit-function-declaration]
net/netfilter/xt_quota2.c:85:6: warning: assignment makes pointer from
integer without a cast [enabled by default]
net/netfilter/xt_quota2.c:110:1: warning: label ‘nlmsg_failure’ defined
but not used [-Wunused-label]
net/netfilter/xt_quota2.c: In function ‘quota_mt2_init’:
net/netfilter/xt_quota2.c:353:12: warning: passing argument 3 of
‘netlink_kernel_create’ makes pointer from integer without a cast
[enabled by default]
In file included from include/linux/if_link.h:5:0,
 from include/linux/netdevice.h:31,
 from include/linux/netfilter/x_tables.h:188,
 from net/netfilter/xt_quota2.c:21:
include/linux/netlink.h:185:21: note: expected ‘struct module *’ but
argument is of type ‘int’
net/netfilter/xt_quota2.c:353:12: error: too many arguments to function
‘netlink_kernel_create’
In file included from include/linux/if_link.h:5:0,
 from include/linux/netdevice.h:31,
 from include/linux/netfilter/x_tables.h:188,
 from net/netfilter/xt_quota2.c:21:
include/linux/netlink.h:185:21: note: declared here



-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH 1/2] netfilter: xt_quota2: Move away from NLMSG_PUT()

2012-08-30 Thread Tushar Behera
Commit c3deafc5261a (netlink: Delete NLMSG_PUT and NLMSG_NEW.) removes
NLMSG_PUT() and recommends use of nlmsg_put() instead.

Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
 net/netfilter/xt_quota2.c |   17 +++--
 1 files changed, 11 insertions(+), 6 deletions(-)

diff --git a/net/netfilter/xt_quota2.c b/net/netfilter/xt_quota2.c
index fb2ef46..865f909 100644
--- a/net/netfilter/xt_quota2.c
+++ b/net/netfilter/xt_quota2.c
@@ -24,6 +24,8 @@
 #include linux/netfilter_ipv4/ipt_ULOG.h
 #endif
 
+#include net/netlink.h
+
 /**
  * @lock:  lock to protect quota writers from each other
  */
@@ -82,9 +84,15 @@ static void quota2_log(unsigned int hooknum,
}
 
/* NLMSG_PUT() uses goto nlmsg_failure */
-   nlh = NLMSG_PUT(log_skb, /*pid*/0, /*seq*/0, qlog_nl_event,
-   sizeof(*pm));
-   pm = NLMSG_DATA(nlh);
+   nlh = nlmsg_put(log_skb, /*pid*/0, /*seq*/0, qlog_nl_event,
+   sizeof(*pm), 0);
+
+   if (!nlh) {
+   pr_debug(xt_quota2: error during NLMSG_PUT\n);
+   return;
+   }
+
+   pm = nlmsg_data(nlh);
if (skb-tstamp.tv64 == 0)
__net_timestamp((struct sk_buff *)skb);
pm-data_len = 0;
@@ -106,9 +114,6 @@ static void quota2_log(unsigned int hooknum,
NETLINK_CB(log_skb).dst_group = 1;
pr_debug(throwing 1 packets to netlink group 1\n);
netlink_broadcast(nflognl, log_skb, 0, 1, GFP_ATOMIC);
-
-nlmsg_failure:  /* Used within NLMSG_PUT() */
-   pr_debug(xt_quota2: error during NLMSG_PUT\n);
 }
 #else
 static void quota2_log(unsigned int hooknum,
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH 0/2] Android fixes w.r.t. 3.6 kernel

2012-08-30 Thread Tushar Behera
Required for android-3.6 tree.

Tushar Behera (2):
  netfilter: xt_quota2: Move away from NLMSG_PUT()
  netfilter: xt_quota2: Update parameter list in netlink_kernel_create

 net/netfilter/xt_quota2.c |   25 -
 1 files changed, 16 insertions(+), 9 deletions(-)

-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH 2/2] netfilter: xt_quota2: Update parameter list in netlink_kernel_create

2012-08-30 Thread Tushar Behera
Commit a31f2d17b331 (netlink: add netlink_kernel_cfg parameter to
netlink_kernel_create) modifies parameter list of netlink_kernel_create.

Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
 net/netfilter/xt_quota2.c |8 +---
 1 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/net/netfilter/xt_quota2.c b/net/netfilter/xt_quota2.c
index 865f909..3a9c1f9 100644
--- a/net/netfilter/xt_quota2.c
+++ b/net/netfilter/xt_quota2.c
@@ -350,12 +350,14 @@ static struct xt_match quota_mt2_reg[] __read_mostly = {
 static int __init quota_mt2_init(void)
 {
int ret;
+   struct netlink_kernel_cfg cfg = {
+   .groups = 1,
+   };
pr_debug(xt_quota2: init());
 
 #ifdef CONFIG_NETFILTER_XT_MATCH_QUOTA2_LOG
-   nflognl = netlink_kernel_create(init_net,
-   NETLINK_NFLOG, 1, NULL,
-   NULL, THIS_MODULE);
+   nflognl = netlink_kernel_create(init_net, NETLINK_NFLOG,
+   THIS_MODULE, cfg);
if (!nflognl)
return -ENOMEM;
 #endif
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: New fast models device-tree topic for linux-linaro

2012-08-16 Thread Tushar Behera
On 08/16/2012 08:24 AM, Tushar Behera wrote:
 On 08/16/2012 12:55 AM, Andrey Konovalov wrote:
 On 08/15/2012 08:47 PM, Jon Medhurst (Tixy) wrote:
 Hi Andrey

 Can you include a new topic to linux-linaro which contains device-tree
 files for ARM's fast models? This branch is tracking-armlt-rtsm [1] in
 the ARM LT working tree.

 Done, thanks! The git tag is ll-20120815.0.

 *Tushar*,

 I've commented out the Samsung LT's topics this time, as they seem to be
 v3.5-rc2 based. Do you have a plan to move them to v3.6-rc*?

 
 v3.6-rc1 based kernel is almost ready, pending some tests. I hope to
 push it to glo by this Friday.
 

Pushed to [1] with tag *samsung-lt-v3.6-rc1*

[1] git://git.linaro.org/landing-teams/working/samsung/kernel.git

 
 Thanks,
 Andrey

 Thanks

 For those interested...

 These device-trees enable the the current Linaro vexpress kernel to boot
 on fast-models when used with the boot-wrapper at [2], this includes
 having a working display. However, on the big.LITLE models the A7 cores
 don't start, giving Failed to boot -38.


 
 


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Reboot call fails in cpu_v7_proc_fin()

2012-08-15 Thread Tushar Behera
On 08/15/2012 11:05 AM, Koteswararao Nelakurthi wrote:
 Hi,
 
 While executing reboot call on my ARMv7 board, the system hangs during
 cpu_proc_fin() call.
 
 Commenting out this line of code, the reboot works properly.
 
 Index: mvlinux/arch/arm/mm/proc-v7.S
 ===
 --- mvlinux.orig/arch/arm/mm/proc-v7.S  2012-08-15 14:12:50.396190110 +0900
 +++ mvlinux/arch/arm/mm/proc-v7.S   2012-08-15 14:21:28.730760416 +0900
 @@ -51,7 +51,7 @@
 mrc p15, 0, r0, c1, c0, 0   @ ctrl register
 bic r0, r0, #0x1000 @ ...i
 bic r0, r0, #0x0006 @ .ca.
 -   mcr p15, 0, r0, c1, c0, 0   @ disable caches
 +   #mcrp15, 0, r0, c1, c0, 0   @ disable caches
 ldmfd   sp!, {pc}
  ENDPROC(cpu_v7_proc_fin)
 
 is it a CPU_V7 specific issue?
 
 Please provide your valuable input to fix the issue.
 
 Any hint also..will help me in debugging further in this issue.
 
 I have seen above fix in one of the linaro mailing list.
 
 Regards,
 koteswararao.
 

I don't think there is any issue with cpu_v7_proc_fin. In case of the
issue that I was facing, it was fixed by adding a proper reset hook to
my platform.

Please go through following link. I hope that would help.

http://www.mail-archive.com/linux-samsung-soc@vger.kernel.org/msg06170.html

-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: New fast models device-tree topic for linux-linaro

2012-08-15 Thread Tushar Behera
On 08/16/2012 12:55 AM, Andrey Konovalov wrote:
 On 08/15/2012 08:47 PM, Jon Medhurst (Tixy) wrote:
 Hi Andrey

 Can you include a new topic to linux-linaro which contains device-tree
 files for ARM's fast models? This branch is tracking-armlt-rtsm [1] in
 the ARM LT working tree.
 
 Done, thanks! The git tag is ll-20120815.0.
 
 *Tushar*,
 
 I've commented out the Samsung LT's topics this time, as they seem to be
 v3.5-rc2 based. Do you have a plan to move them to v3.6-rc*?
 

v3.6-rc1 based kernel is almost ready, pending some tests. I hope to
push it to glo by this Friday.


 Thanks,
 Andrey
 
 Thanks

 For those interested...

 These device-trees enable the the current Linaro vexpress kernel to boot
 on fast-models when used with the boot-wrapper at [2], this includes
 having a working display. However, on the big.LITLE models the A7 cores
 don't start, giving Failed to boot -38.

 


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Call for topics for the 12.06 linux-linaro* trees

2012-06-10 Thread Tushar Behera
On 06/08/2012 07:30 PM, Andy Green wrote:
 On 06/08/2012 08:25 PM, the mail apparently from Andrey Konovalov included:
 Greetings,

 There will be three linux-linaro trees this month. Also some LTs have
 moved to 3.5, while the others will stay on older kernels.

 In addition to linux-linaro-core-tracking and linux-linaro ones
 (described under
 https://wiki.linaro.org/Platform/DevPlatform/LinuxLinaroKernelTreeProcess),

 linux-linaro-tracking tree will be created as a merge of the tracking
 trees of TI (v3.4 based) and Samsung (currently v3.4-rc7 based) LTs.
 

I would be publishing 3.4 based Samsung Kernel today. So, you can use
that in linux-linaro-tracking.


 For reference the TI + Samsung trees I did during Connect can be found
 here:
 
 http://git.linaro.org/gitweb?p=landing-teams/working/ti/kernel.git;a=shortlog;h=refs/heads/unify-samsung-ti
 
 
 This builds and works for OMAP at least.
 
 Probably more LT tracking trees would be added, but I am not sure due to
 the different bases the others use. Each LT's tracking tree act as a
 topic here, so the call for topics doesn't apply to  this tree.

 linux-linaro-core-tracking tree is moving to 3.5 (but not published at
 git.linaro.org yet). For those using the 3.4, llct_3.4 branch has been
 created. This is a copy of the most recent v3.4 based
 linux-linaro-core-tracking tree.
 *New generic (not board specific) topics and updates to the existing
 topics are welcomed for the 3.5 based linux-linaro-core-tracking tree*
 The requirements for a topic can be found here:
 https://wiki.linaro.org/Platform/DevPlatform/LinuxLinaroKernelTreeProcess#Adding_a_topic_to_linux-linaro_kernel_and_maintaining_it


 The current topics are:
 * arm_soc/testing/multiplatform
 * svenkatr/ufs-for-linux-linaro
 * svenkatr/emmc-for-linux-linaro
 * amitdanielk/thermal_exynos4_imx6_work
 
 Not sure if this will now already have it, but this patch
 
 http://www.mail-archive.com/linux-omap@vger.kernel.org/msg63852.html
 
 is important for OMAP... thanks to Tushar for digging up the context of
 it for us.  We're carrying a replica of it in our tracking atm but since
 what was
 
 tracking-thermal_exynos4_imx6-3.4-2012.05
 
 had just one of the pair of patches in older llct, it should be improved
 to have both, if it doesn't already with this update.
 
 -Andy
 


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: 12.05 linux-linaro kernel tree

2012-05-21 Thread Tushar Behera
On 05/18/2012 10:57 PM, Andrey Konovalov wrote:
 Samsung LT's topics:
 topic/base topic/core topic/bl topic/dt topic/fb topic/pd topic/s2ram
 topic/asv_cpufreq topic/led topic/dummy_reg topic/gadget topic/touch
 topic/wlan topic/audio topic/hdmi topic/mfc topic/mali
 topic/cma_origen topic/android_config topic/ubuntu_config
 
Attached patch fixes kernel panic while booting Android on Origen board
using linux-linaro kernel. Since this is touching the core file, I would
like to know if there are any objections to this.

Andrey,
If it is ok, you may either apply the patch or merge [1].

[1] git://git.linaro.org/landing-teams/working/samsung/kernel.git
(llt/umm_fixes)



 drivers/media/video/videobuf2-dma-contig.c |3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/drivers/media/video/videobuf2-dma-contig.c
b/drivers/media/video/videobuf2-dma-contig.c
index 266ae7d..57e643b 100644
--- a/drivers/media/video/videobuf2-dma-contig.c
+++ b/drivers/media/video/videobuf2-dma-contig.c
@@ -273,6 +273,9 @@ static struct vm_area_struct *vb2_dc_get_user_vma(
 static int vb2_dc_get_user_pages(unsigned long start, struct page **pages,
int n_pages, struct vm_area_struct *vma, int write)
 {
+   if (vma-vm_mm == NULL)
+   vma-vm_mm = current-mm;
+
if (vma_is_io(vma)) {
unsigned int i;



-- 
Tushar Behera
From c579b4d6b17a6fb1b15cc681268db54e5b73afaf Mon Sep 17 00:00:00 2001
From: Sachin Kamat sachin.ka...@linaro.org
Date: Mon, 21 May 2012 13:51:19 +0530
Subject: [PATCH] VB2-DC: Fix Null pointer related kernel boot crash

vma-vm_mm should point to the proper mm structure. But the pointer is
NULL here. Explicitly setting it to current-mm to avoid kernel crash
during bootup.

Signed-off-by: Ritesh Kumar Solanki r.sola...@samsung.com
Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
 drivers/media/video/videobuf2-dma-contig.c |3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/drivers/media/video/videobuf2-dma-contig.c b/drivers/media/video/videobuf2-dma-contig.c
index 266ae7d..57e643b 100644
--- a/drivers/media/video/videobuf2-dma-contig.c
+++ b/drivers/media/video/videobuf2-dma-contig.c
@@ -273,6 +273,9 @@ static struct vm_area_struct *vb2_dc_get_user_vma(
 static int vb2_dc_get_user_pages(unsigned long start, struct page **pages,
 	int n_pages, struct vm_area_struct *vma, int write)
 {
+	if (vma-vm_mm == NULL)
+		vma-vm_mm = current-mm;
+
 	if (vma_is_io(vma)) {
 		unsigned int i;
 
-- 
1.7.4.1

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: 12.05 linux-linaro kernel tree

2012-05-17 Thread Tushar Behera
On 05/11/2012 01:04 AM, Andrey Konovalov wrote:
 Samsung:
topic/base topic/core topic/bl topic/dt topic/fb topic/pd
topic/s2ram topic/asv_cpufreq topic/led topic/dummy_reg
topic/gadget topic/touch topic/wlan topic/audio topic/hdmi
topic/mali topic/cma_v24 topic/android_config
 

Following topic branches are available to be pulled to linux-linaro tree.

topic/base topic/core topic/bl topic/dt topic/fb topic/pd topic/s2ram
topic/asv_cpufreq topic/led topic/dummy_reg topic/gadget topic/touch
topic/wlan topic/audio topic/hdmi *topic/mfc* topic/mali
*topic/cma_origen* topic/android_config *topic/ubuntu_config*

* topic/base is based on v3.4-rc7 and has config patches, both internal
and from John's linaro-config-3.4 branch. All other topic branches are
based on topic/base.
* topic/cma_v24 has been dropped. Instead we have topic/cma_origen that
has only Origen specific CMA patches.

With this set of patches, I can boot-test Ubuntu and it fails for
Android. I will update you if the Android related kernel panic is fixed
before 2012.05 release.

Please let me know if you run across any issues while merging these
branches.


 Thanks,
 Andrey
 
 ___
 linaro-dev mailing list
 linaro-dev@lists.linaro.org
 http://lists.linaro.org/mailman/listinfo/linaro-dev


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: New Gator version ready for Linaro kernels / Mali

2012-05-16 Thread Tushar Behera
On 05/17/2012 05:12 AM, Andy Green wrote:
 On 16/05/12 23:58, Somebody in the thread at some point said:
 
 Hi -
 
 ARM have released a new version of Developer Studio 5 (DS-5) and we now
 have a new version of the Gator component [1] to go with this which
 needs updating in all Linaro kernel trees that will be part of the 12.05
 release.

 For those people maintaining kernel trees here is what this means...

 - If your kernels are including the linux-linaro-core-tracking [2]
 branch then you will get the new Gator version from this when it is
 updated over the next couple of days. You don't need to do anything
 except to make sure you are up to date with this branch before the 12.05
 release.
 
 I suspect we'll be issuing tilt-3.3 again, so even though we're on llct
 for tracking we'll have to revert and update.
 
 For those people who have applied Mali driver patches to support
 profiling by Gator: you don't need to modify those Mali patches, just
 take the new version of Gator, this will still work OK.
 
 Just curious... how many LTs have Mali stuff?  If it's more than one, we
 should perhaps be talking about moving it to linux-linaro-core-tracking.
  Presumably that then needs coordination about matching userlands in
 multiple LTs which will need some time.  Even if Mali is in good sync
 today between multiple LTs the architecture of each LT having their own
 copy of what's meant to be permanently in sync invites problems.
 
True, Mali driver code should move to core part. As of now, Samsung LT
is maintaining a version of it, which was part of linux-linaro-2012.04
release.


 -Andy
 


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: 12.05 linux-linaro kernel tree

2012-05-14 Thread Tushar Behera
On 05/12/2012 11:09 PM, Andrey Konovalov wrote:
 Tushar,
 
 On 05/11/2012 09:04 AM, Tushar Behera wrote:
 On 05/11/2012 01:04 AM, Andrey Konovalov wrote:
 Greetings,

 So far I wasn't updating the linux-linaro tree since the 12.04 release.
 (The generic topic updates were being done to the
 linux-linaro-core-tracking tree)

 Now it is time to move the focus to the linux-linaro tree. For one week
 it will use the mainline tip as the base. Then, on next Thursday the
 most recent -rc will be selected as the base, and won't be changed until
 12.05 is released. Most probably it will be v3.4-rc7.

 The 12.05 linux-linaro tree will get the ARM and Samsung LTs topics plus
 the 7 generic topics currently included into the
 linux-linaro-core-tracking tree:
 ufs (ufs-for-linux-linaro)
 emmc (emmc-for-linux-linaro)
 thermal_exynos4_imx6 (thermal_exynos4_imx6_work)
 linaro-android-3.4
 armlt-gator (tracking-armlt-gator)
 umm-wip (umm-3.4rc4-wip)
 linaro-configs-3.4
 If you don't see your generic topic in this list, but you think it
 should be there, please let me know ASAP. If you have a new topic to
 add, please send me the request before the next Thursday, May 17; the
 sooner, the better. The requirements for a topic can be found here:
 https://wiki.linaro.org/Platform/DevPlatform/LinuxLinaroKernelTreeProcess#Adding_a_topic_to_linux-linaro_kernel_and_maintaining_it



 The landing teams - please update your topic branches if needed:
 ARM:
 tracking-armlt-hdlcd tracking-armlt-mmc
 tracking-armlt-arm-arch-fixes tracking-armlt-misc-fixes
 tracking-armlt-ubuntu-config tracking-armlt-android-config
 Samsung:
 topic/base topic/core topic/bl topic/dt topic/fb topic/pd
 topic/s2ram topic/asv_cpufreq topic/led topic/dummy_reg
 topic/gadget topic/touch topic/wlan topic/audio topic/hdmi
 topic/mali topic/cma_v24 topic/android_config

 Is any other LT using CMA patchset? If so, this topic branch can be
 moved into linux-linaro-core-tracking.
 
 That's a good idea, thanks!
 
 The only problem is that your topic/cma_v24 is based on the topic/base,
 and thus includes CONFIG: ORIGEN: commits and an older version of
 linaro-configs-3.4 topic. In particular, the latter recreates
 configs/panda.conf file which has been deleted from the current
 linaro-configs-3.4. Could you please make topic/cma_v24 mainline based
 (drop these CONFIG: ORIGEN: and configs/* changes)? And is the
 CONFIG_CMA_SIZE_MBYTES=32 option Origen specific or generic? If it is
 generic, it should go into a separate file, e.g. configs/cma.conf.
 
I didn't mean to include topic/cma_v24 with the patches from topic/base,
rather the clean patchset from Marek. But now that we have the umm topic
branch in linux-linaro-core-tracking, we don't need topic/cma_v24
anymore. If any fixes with respect to Origen are required, I will queue
them up in another topic branch.

I will shortly send you the list of topic branches for 2012.05 release,
now that 3.4-rc7 is already released.

 Thanks,
 Andrey


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: 12.05 linux-linaro kernel tree

2012-05-10 Thread Tushar Behera
On 05/11/2012 01:04 AM, Andrey Konovalov wrote:
 Greetings,
 
 So far I wasn't updating the linux-linaro tree since the 12.04 release.
 (The generic topic updates were being done to the
 linux-linaro-core-tracking tree)
 
 Now it is time to move the focus to the linux-linaro tree. For one week
 it will use the mainline tip as the base. Then, on next Thursday the
 most recent -rc will be selected as the base, and won't be changed until
 12.05 is released. Most probably it will be v3.4-rc7.
 
 The 12.05 linux-linaro tree will get the ARM and Samsung LTs topics plus
 the 7 generic topics currently included into the
 linux-linaro-core-tracking tree:
ufs (ufs-for-linux-linaro)
emmc (emmc-for-linux-linaro)
thermal_exynos4_imx6 (thermal_exynos4_imx6_work)
linaro-android-3.4
armlt-gator (tracking-armlt-gator)
umm-wip (umm-3.4rc4-wip)
linaro-configs-3.4
 If you don't see your generic topic in this list, but you think it
 should be there, please let me know ASAP. If you have a new topic to
 add, please send me the request before the next Thursday, May 17; the
 sooner, the better. The requirements for a topic can be found here:
 https://wiki.linaro.org/Platform/DevPlatform/LinuxLinaroKernelTreeProcess#Adding_a_topic_to_linux-linaro_kernel_and_maintaining_it
 
 
 The landing teams - please update your topic branches if needed:
 ARM:
tracking-armlt-hdlcd tracking-armlt-mmc
tracking-armlt-arm-arch-fixes tracking-armlt-misc-fixes
tracking-armlt-ubuntu-config tracking-armlt-android-config
 Samsung:
topic/base topic/core topic/bl topic/dt topic/fb topic/pd
topic/s2ram topic/asv_cpufreq topic/led topic/dummy_reg
topic/gadget topic/touch topic/wlan topic/audio topic/hdmi
topic/mali topic/cma_v24 topic/android_config
 
Is any other LT using CMA patchset? If so, this topic branch can be
moved into linux-linaro-core-tracking.

 Thanks,
 Andrey
 
 ___
 linaro-dev mailing list
 linaro-dev@lists.linaro.org
 http://lists.linaro.org/mailman/listinfo/linaro-dev


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Preliminary 12.04 linux-linaro kernel tree

2012-04-18 Thread Tushar Behera
Hi Tixy,

On 04/18/2012 01:24 PM, Jon Medhurst (Tixy) wrote:
 On Wed, 2012-04-18 at 02:16 +0400, Andrey Konovalov wrote:
 Greetings,

 I've pushed the current linux-linaro tree
 to git://git.linaro.org/kernel/linux-linaro-tracking.git , linux-linaro 
 branch.
 
 This fails to build for vexpress because the Samsung Mali topic defaults
 to enabling Mali and UMP, they should probably default to being disabled
 and the Origen defconfig should enable them.
 
 After editing drivers/gpu/arm/mali/Kconfig to make
 config MALI400MP 'default n', and similar with config UMP in
 drivers/gpu/arm/ump/Kconfig then things build OK.
 
 The resulting kernel successfully booted an Oneiric desktop image I had
 from a couple of weeks ago.
 
Ok ... I will update the same in Samsung Mali topic.

-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Preliminary 12.04 linux-linaro kernel tree

2012-04-18 Thread Tushar Behera
Hi Tixy,

On 04/18/2012 01:28 PM, Tushar Behera wrote:
 Hi Tixy,
 
 On 04/18/2012 01:24 PM, Jon Medhurst (Tixy) wrote:
 On Wed, 2012-04-18 at 02:16 +0400, Andrey Konovalov wrote:
 Greetings,

 I've pushed the current linux-linaro tree
 to git://git.linaro.org/kernel/linux-linaro-tracking.git , linux-linaro 
 branch.

 This fails to build for vexpress because the Samsung Mali topic defaults
 to enabling Mali and UMP, they should probably default to being disabled
 and the Origen defconfig should enable them.

 After editing drivers/gpu/arm/mali/Kconfig to make
 config MALI400MP 'default n', and similar with config UMP in
 drivers/gpu/arm/ump/Kconfig then things build OK.

 The resulting kernel successfully booted an Oneiric desktop image I had
 from a couple of weeks ago.

 Ok ... I will update the same in Samsung Mali topic.
 
I have update topic/mali here. [1]

Can you please confirm if you wanted it that way?

-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Thermal release for this month.

2012-04-15 Thread Tushar Behera
On 04/13/2012 07:22 PM, Amit Kucheria wrote:
 Adding Guodong and Tushar to make sure that the LT trees also have
 thermal management turned on for 12.04.
 

In Samsung LT kernel, we have included thermal_v2 patchset from Amit.

 And cc'ing linaro-dev and the release manager. :)
 
 /Amit
 
 On Fri, Apr 13, 2012 at 4:47 PM, Amit Kachhap amit.kach...@linaro.org wrote:
 Hi Andrey,

 Please pull the thermal management work for exynos4 and imx6 platforms
 for this month release.

As for merging various topic branches to linux-linaro-tracking, I would
expect that the individual branches also enable relevant defconfig
options. I don't see any better way to enable a specific feature in
linux-linaro-tracking.

 git://git.linaro.org/people/amitdanielk/linux.git thermal_exynos4_imx6_work

 since commit id 258f742635360175564e9470eb060ff4d4b984e7

 The necessary configs to be enabled are,
 https://wiki.linaro.org/WorkingGroups/PowerManagement/Doc/Kconfig#Thermal

 Thanks,
 Amit Daniel




-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Linaro 12.04 3.4-rc1 based androidization branch is available

2012-04-13 Thread Tushar Behera
On 04/13/2012 02:48 PM, Sangwook Lee wrote:
 Hi John
 
 On 11/04/12 20:19, John Stultz wrote:
 On 04/09/2012 03:18 PM, John Stultz wrote:
 I went ahead and forward ported the AOSP-3.3 tree to 3.4-rc1.

 You can grab it here:
 git://git.linaro.org/people/jstultz/android.git
 linaro-android-3.4-jstultz-rebase

 Sigh. Looks like the Google devs are following close behind and released
 their own 3.4-rc2 based tree.

 Don't get me wrong: Its great to see them keeping up with mainline, but
 I've clearly duplicated their work this cycle.

 I've gone ahead and mirrored the AOSP 3.4 tree here:
 git://git.linaro.org/people/jstultz/android.git linaro-android-3.4
 
 Are you going to add merge_config.sh into this branch?
 
merge_config.sh is in linaro-configs-3.4 on John's tree. The related
patches have also been part of topic/base.


 Andrey: Probably you're call on this. If you've already merged in my
 rebase tree it should be fine for 12.04, but if not you might want to
 jump over to the AOSP 3.4 tree.

 thanks
 -john


 ___
 linaro-dev mailing list
 linaro-dev@lists.linaro.org
 http://lists.linaro.org/mailman/listinfo/linaro-dev
 
 ___
 linaro-dev mailing list
 linaro-dev@lists.linaro.org
 http://lists.linaro.org/mailman/listinfo/linaro-dev


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Linaro 12.04 3.4-rc1 based androidization branch is available

2012-04-13 Thread Tushar Behera
On 04/13/2012 04:19 PM, Tushar Behera wrote:
 On 04/13/2012 02:48 PM, Sangwook Lee wrote:
 Hi John

 On 11/04/12 20:19, John Stultz wrote:
 On 04/09/2012 03:18 PM, John Stultz wrote:
 I went ahead and forward ported the AOSP-3.3 tree to 3.4-rc1.

 You can grab it here:
 git://git.linaro.org/people/jstultz/android.git
 linaro-android-3.4-jstultz-rebase

 Sigh. Looks like the Google devs are following close behind and released
 their own 3.4-rc2 based tree.

 Don't get me wrong: Its great to see them keeping up with mainline, but
 I've clearly duplicated their work this cycle.

 I've gone ahead and mirrored the AOSP 3.4 tree here:
 git://git.linaro.org/people/jstultz/android.git linaro-android-3.4

 Are you going to add merge_config.sh into this branch?

 merge_config.sh is in linaro-configs-3.4 on John's tree. The related
 patches have also been part of topic/base.
 

I mean topic/base in Samsung's Landing team kernel.


 Andrey: Probably you're call on this. If you've already merged in my
 rebase tree it should be fine for 12.04, but if not you might want to
 jump over to the AOSP 3.4 tree.

 thanks
 -john


 ___
 linaro-dev mailing list
 linaro-dev@lists.linaro.org
 http://lists.linaro.org/mailman/listinfo/linaro-dev

 ___
 linaro-dev mailing list
 linaro-dev@lists.linaro.org
 http://lists.linaro.org/mailman/listinfo/linaro-dev
 
 


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Config fragment for Versatile Express

2012-04-04 Thread Tushar Behera
On 04/04/2012 01:02 PM, Jon Medhurst (Tixy) wrote:
 On Wed, 2012-04-04 at 10:05 +0530, Tushar Behera wrote:
 On 04/03/2012 03:59 PM, Jon Medhurst (Tixy) wrote:
 I think it makes sense if this 'upstream' doesn't include board files
 though, they should come from LT trees.

 As for the board specific fragments, I feel it would be better if the
 config-upstream tree has the initial fragment (which I suppose is
 minimal enough to compile the common kernel). That way anyone taking
 linux-linaro-tracking can have at least a working setup.
 
 I thought linux-linaro-tracking was going to operate a bit like
 linux-next, i.e. just a merge of all the topics from LTs and working
 groups, and from which Linaro produces it's releases and does some CI
 build+test. If that's true, then anyone using that tree will always have
 the LT code.

In some cases, the mainline kernel cannot boot a specific board using
the default config file. If we have a config fragment that adds the
necessary changes, people can use that specific board even if LT has no
other enablement patches merged to linux-linaro-tracking.

An example would be the thermal patches, which got merged to
linux-linaro-tracking through PM WG. But for someone to validate those
changes, they won't have a definite config to build
linux-linaro-tracking and run on Origen board. Hence I feel that having
a minimal board fragment, that boots the kernel till console, in
linux-linaro-samsung would be helpful.

As long as we have same set of commits in our tree, we won't have merge
conflicts too.

 
 Though I guess if LT code breaks other boards and has to get temporarily
 dropped, and if that LT code was pulled in as one monolithic topic like
 TI and Samsung trees, then it would probably mean dropping the board
 config too if we had that coming from the LT tree. But in that case,
 does it matter that the broken board can't be built from
 linux-linaro-tracking?
 

That is one concern from my side too, as to how do we build
linux-linaro-tracking kernel and what all things goes into that.

-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Config fragment for Versatile Express

2012-04-03 Thread Tushar Behera
On 04/03/2012 01:43 PM, Jon Medhurst (Tixy) wrote:
 On Mon, 2012-04-02 at 12:30 -0700, John Stultz wrote:
 The difficulty is that as Tixy earlier pointed out, are that the LT 
 kernel trees are mainline based, and thus aren't based off of something 
 that would contain the base/distro/board config fragments.

 One approach we might be able to use is if the board defconfigs really 
 are minimal, and restricted in scope to only the board options, we could 
 switch the merge order (board/distro/base). This way the LT's additive 
 defconfig can be used (from arch/arm/configs/ ) and we can still also 
 get consistent generic options as well.
 
 The mainline defconfigs aren't minimal in the sense we want, they
 include things like file-systems and networking options so somebody can
 build a usable kernel, and I think it's sensible to keep it that way.
 
 For Linaro's purposes we would need a new board config. We could make
 that minimal, but we get back to the idea that topic branches which
 change configs would need to sit on top of the topic with this config.
 Perhaps that is something we can live with if a
 directory-of-config-fragments approach is deemed undesirable.
 
 It's a pity that the title of the thread possibly means no one from
 other LTs are reading. (Probably a bit late to change it now and get
 noticed.)
 
I hope not.

For Samsung LT kernel, we have followed an approach where in the commits
in John's linaro_config_3.3 branch are taken to be stable commits and we
have put those commits as the very first set of commits on LT kernel.
Our LT kernel being a _serialized_ kernel where topic branches sit one
over the other, the related config fragments are made part of the topic
branches.

A sample view of the same is posted at [1].

[1] git://git.linaro.org/landing-teams/working/samsung/kernel.git (lt/next)

-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Config fragment for Versatile Express

2012-04-03 Thread Tushar Behera
On 04/03/2012 03:59 PM, Jon Medhurst (Tixy) wrote:
 On Tue, 2012-04-03 at 14:51 +0530, Tushar Behera wrote:
 For Samsung LT kernel, we have followed an approach where in the commits
 in John's linaro_config_3.3 branch are taken to be stable commits and we
 have put those commits as the very first set of commits on LT kernel.
 Our LT kernel being a _serialized_ kernel where topic branches sit one
 over the other, the related config fragments are made part of the topic
 branches.

 A sample view of the same is posted at [1].

 [1] git://git.linaro.org/landing-teams/working/samsung/kernel.git (lt/next)
 
 The Samsung tree has the non samsung config files, like
 linaro-base.conf. I was wondering if that would cause merge problems if
 every LT had them and at possibly different versions. I guess it doesn't

This method works well if the common config fragments, once committed to
some upstream tree, are stable. Then only it makes sense to base other
topics on top of this.

 matter so long as all LTs got them from the same definitive 'upstream'
 source, and that they didn't edit them.
 
 I think it makes sense if this 'upstream' doesn't include board files
 though, they should come from LT trees.
 
As for the board specific fragments, I feel it would be better if the
config-upstream tree has the initial fragment (which I suppose is
minimal enough to compile the common kernel). That way anyone taking
linux-linaro-tracking can have at least a working setup.

As and when some topics get merged to mainline, we can move those
config-fragments to upstream tree.

 We also need a better upstream than John's personal Android tree :-)
 
I suppose, it would soon get a place in linux-linaro-tracking.

-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [RFC] First pass config fragment breakout for linaro kernel

2012-03-30 Thread Tushar Behera
On 03/29/2012 11:25 PM, John Stultz wrote:
 On 03/28/2012 09:37 PM, Tushar Behera wrote:
 On 03/28/2012 10:17 PM, John Stultz wrote:
 On 03/28/2012 05:24 AM, Tushar Behera wrote:
 On 03/27/2012 12:50 AM, John Stultz wrote:
   So after talking about it at the last Linaro Connect, I've
 finally
 gotten around to making a first pass at providing config fragments for
 the linaro kernel.  I'd like to propose merging this for 12.04, and
 doing so early so we can make sure that all the desired config options
 are present in the fragments and to allow the vairous linaro build
 systems to begin migrating their config generation over.

 The current tree is here:


 http://git.linaro.org/gitweb?p=people/jstultz/android.git;a=shortlog;h=refs/heads/linaro-configs-3.3





 The most relevant commit being:


 http://git.linaro.org/gitweb?p=people/jstultz/android.git;a=commitdiff;h=da8f6d20e1a768cb486005c5ec62582b6f92990d





 This includes config fragments for a linaro-base, an android-ization
 fragment, and  board configs for panda, origen and imx53.

 I suspect we'll need an ubuntu specific fragment as well as all the
 other board fragments.

 There is likely to be quite a bit of churn as we decide what sort of
 configs are really common and which are board specific. But that's ok.

 Configs are generated from the config fragments, as follows:

 ./scripts/kconfig/merge_config.sh ./configs/linaro-base.conf
 ./configs/android.conf ./configs/panda.conf


 You may see warnings, which are not fatal, but should be reported so
 they can be properly cleaned up.

 I'm asking for Build folks to take a look at the above and consider
 how
 they might merge in fragment assembly into their system replacing
 their
 current config generation.

 I'd ask Landing teams to take a look at this, and report any missing
 config options or fragment chunks they'd like to see.

 I have updated origen.conf and linaro-base.conf for testing Samsung LT
 kernel. The results are updated in [1].
 I'll take a look at the changes and try to merge them into my tree.

 Ok. I've merged most of what you added, but made some tweaks of my own
 to quiet some of the warnings at merge time.
 
 Let me know if you see anything badly missing. Some of the items I
 dropped look like they're from feature branches?
 
Right, some of the config options should better move to the feature
branches. I have cleaned up origen.conf so that we can now boot
linux-linaro-tracking kernel till the console. Code dropped at [1].

With the help of a couple of patches[2], I was able to get Ubuntu
booting up till home screen.

I haven't tested Android though.

Following is the list of added patches.
0b066ba ARM: EXYNOS: Increase DMA pool allocator size for framebuffer
This patch is not required if CMA patches are added to the kernel.

fb8fa05 ARM: EXYNOS: Add clkdev lookup entry for lcd clock
A rebased version of this patch is queued for 3.4-rc1.

[1] git://git.linaro.org/landing-teams/working/samsung/kernel.git
(topic/linaro_config_3.3)
[2] git://git.linaro.org/landing-teams/working/samsung/kernel.git
(test/linux-linaro-tracking)


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Ubuntu LEB 12.03 RC images

2012-03-29 Thread Tushar Behera
On 03/29/2012 12:58 PM, Ricardo Salveti wrote:
 On Tue, Mar 27, 2012 at 10:37 AM, Ricardo Salveti
 ricardo.salv...@linaro.org wrote:
 Hi,

 A little bit later than usual, but here it goes the announcement of
 the 12.03 RC images. This time we got a bit delayed because we're
 finally generating and maintaining all the kernel packages ourself,
 and getting them in sync with Launchpad is something that still needs
 some improvement.

 This release should be the last one based on Ubuntu Oneiric (and
 ARMEL), including the latest XBMC release and the usual kernel and
 package updates. From 12.04 on we will be supporting only builds based
 on Ubuntu Precise (12.04, ARMHF), so we should see quite many nice
 improvements there (for early access, check the builds at
 https://ci.linaro.org/jenkins/view/Ubuntu%20Build%20Service/).

 You can find our current test cases at
 https://wiki.linaro.org/Platform/QA/TestCases/Ubuntu, and the 12.03 RC
 builds (for all boards and image flavors) at
 http://snapshots.linaro.org/oneiric/, with build id 20120327-1 for
 hwpacks and 20120327-0 for the rootfs (nano, developer, alip,
 ubuntu-desktop and linaro-tv-xbmc).

 For our main boards we also have a testing spreadsheet, were we
 publish the official release testing, done by the dev platform
 engineers. You can find the links at
 https://wiki.linaro.org/Platform/DevPlatform/Testing (note that you
 can find the bug reports from the test cases by looking at the QA page
 tag links).

 Fathi will be coordinating all respin requests in the next following
 days at linaro-release m-l, and the final image will be published this
 thursday, at releases.linaro.org.
 
 Ok, after a few rounds of testing, issues with ci.linaro.org and
 Launchpad, I'm finally able to publish the current status of the RC
 images, and the respin requests for tomorrow.
 
 RC Images that worked as expected, without needing extra fixes:
 - Ubuntu Desktop:
 http://snapshots.linaro.org/oneiric/linaro-o-ubuntu-desktop/20120327/0/images/tar/
 - Nano: 
 http://snapshots.linaro.org/oneiric/linaro-o-nano/20120327/1/images/tar/
 (1 as 0 didn't build due archive instabilities)
 - Alip: 
 http://snapshots.linaro.org/oneiric/linaro-o-alip/20120327/0/images/tar/
 - Developer: 
 http://snapshots.linaro.org/oneiric/linaro-o-developer/20120327/1/images/tar/
 (1 as 0 didn't build due archive instabilities)
 
 RC Hwpacks that also worked as expected, without needing extra fixes:
 - 3.3 LT Snowball:
 http://snapshots.linaro.org/oneiric/lt-snowball-oneiric/20120327/1/images/hwpack
 - 3.3 LT Vexpress A9:
 http://snapshots.linaro.org/oneiric/lt-vexpress-a9-oneiric/20120327/1/images/hwpack
 - 3.3 Linux Linaro Pandaboard:
 http://snapshots.linaro.org/oneiric/lt-panda-oneiric/20120327/1/images/hwpack
 - 3.3 Linux Linaro Pandaboard X11 Base:
 http://snapshots.linaro.org/oneiric/lt-panda-x11-base-oneiric/20120327/1/images/hwpack
 
 RC Hwpacks that weren't tested yet:
 - 3.1 Linux Linaro EfikaMX:
 http://snapshots.linaro.org/oneiric/efikamx-oneiric/20120327/1/images/hwpack/
 - 3.1 Linux Linaro iMX51:
 http://snapshots.linaro.org/oneiric/imx51-oneiric/20120327/1/images/hwpack/
 - 3.1 LT MX5: 
 http://snapshots.linaro.org/oneiric/lt-mx5-oneiric/20120327/1/images/hwpack/
 - 3.3 Linux Linaro Igep:
 http://snapshots.linaro.org/oneiric/igep-oneiric/20120327/1/images/hwpack/
 - 3.3 Linux Linaro Overo:
 http://snapshots.linaro.org/oneiric/overo-oneiric/20120327/1/images/hwpack/
 
 RC Hwpacks currently broken without a solution yet:
 - 3.3 Linux Linaro Vexpress A9:
 http://snapshots.linaro.org/oneiric/vexpress-a9-oneiric/20120328/1/images/hwpack/
 - 3.3 Linux Linaro Beagleboard (John Rigby is still debugging):
 http://snapshots.linaro.org/oneiric/omap3-oneiric/20120327/1/images/hwpack/
 
 Respin request for images:
 - LinaroTV-XBMC:
 http://snapshots.linaro.org/oneiric/linaro-o-linarotv-xbmc/20120328/3/images/tar/
   - Reason: Fixes at the XBMC package, update to the final 11 Eden release.
 
 Respin requests for hwpacks:
 - 3.2 LT MX6: 
 http://snapshots.linaro.org/oneiric/lt-mx6-oneiric/20120329/1/images/hwpack/
   - Reason: Fixes to improve the booting experience, but still not yet
 fixing it completely :-(
 - 3.1 LT Pandaboard:
   - 
 http://snapshots.linaro.org/oneiric/lt-panda-oneiric/20120329/0/images/hwpack/
   - 
 http://snapshots.linaro.org/oneiric/lt-panda-x11-base-oneiric/20120329/0/images/hwpack/
   - Reason: Kernel updates, config updates and working hw video decoding 
 again.
 
 Thanks,

Status of Origen is missing in the list :(

-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [RFC] First pass config fragment breakout for linaro kernel

2012-03-29 Thread Tushar Behera
On 03/29/2012 10:50 PM, John Stultz wrote:
 On 03/28/2012 09:37 PM, Tushar Behera wrote:
 On 03/28/2012 10:17 PM, John Stultz wrote:
 On 03/28/2012 05:24 AM, Tushar Behera wrote:

 I have not validated the changes in android.conf, but by merging
 linaro-base.conf and origen.conf, I was able to boot my kernel the
 way I
 would have expected when using exynos4_defconfig.

 One thing I notice is that I find far too many debug messages with this
 new config.
 You mean the debug output from the merge_config.sh script is a bit
 noisy?  Yea. Likely we'll have some extra noise as we settle out what
 options needs to be generic vs board specific.  But it should decrease
 over time.

 I was talking about the console log messages upon booting on a target
 board.
 Ok. Can you send me a Before config where you didn't see all the log
 messages?
 

The new log messages are because of this config entry.
CONFIG_PROVE_LOCKING=y

But the information is useful, hence better it stays that way.

 thanks
 -john
 
 
 ___
 linaro-dev mailing list
 linaro-dev@lists.linaro.org
 http://lists.linaro.org/mailman/listinfo/linaro-dev


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [RFC] First pass config fragment breakout for linaro kernel

2012-03-28 Thread Tushar Behera
On 03/27/2012 12:50 AM, John Stultz wrote:
 So after talking about it at the last Linaro Connect, I've finally
 gotten around to making a first pass at providing config fragments for
 the linaro kernel.  I'd like to propose merging this for 12.04, and
 doing so early so we can make sure that all the desired config options
 are present in the fragments and to allow the vairous linaro build
 systems to begin migrating their config generation over.
 
 The current tree is here:
 

 http://git.linaro.org/gitweb?p=people/jstultz/android.git;a=shortlog;h=refs/heads/linaro-configs-3.3
 
 
 
 The most relevant commit being:
 

 http://git.linaro.org/gitweb?p=people/jstultz/android.git;a=commitdiff;h=da8f6d20e1a768cb486005c5ec62582b6f92990d
 
 
 
 This includes config fragments for a linaro-base, an android-ization
 fragment, and  board configs for panda, origen and imx53.
 
 I suspect we'll need an ubuntu specific fragment as well as all the
 other board fragments.
 
 There is likely to be quite a bit of churn as we decide what sort of
 configs are really common and which are board specific. But that's ok.
 
 Configs are generated from the config fragments, as follows:
 
 ./scripts/kconfig/merge_config.sh ./configs/linaro-base.conf
 ./configs/android.conf ./configs/panda.conf
 
 
 You may see warnings, which are not fatal, but should be reported so
 they can be properly cleaned up.
 
 I'm asking for Build folks to take a look at the above and consider how
 they might merge in fragment assembly into their system replacing their
 current config generation.
 
 I'd ask Landing teams to take a look at this, and report any missing
 config options or fragment chunks they'd like to see.
 
I have updated origen.conf and linaro-base.conf for testing Samsung LT
kernel. The results are updated in [1].

I have not validated the changes in android.conf, but by merging
linaro-base.conf and origen.conf, I was able to boot my kernel the way I
would have expected when using exynos4_defconfig.

One thing I notice is that I find far too many debug messages with this
new config.

Going forward, would it be better to have linaro-base.conf, android.conf
and ubuntu.conf managed centrally and each LT managing their board
specific configuration file. That way, we can include the changes in our
board specific configurations in respective topic branches.

[1] git://git.linaro.org/landing-teams/working/samsung/kernel.git
(lt/linaro_config_3.3-g2d)



 thanks
 -john
 
 
 
 ___
 linaro-dev mailing list
 linaro-dev@lists.linaro.org
 http://lists.linaro.org/mailman/listinfo/linaro-dev


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [RFC] First pass config fragment breakout for linaro kernel

2012-03-28 Thread Tushar Behera
On 03/28/2012 10:17 PM, John Stultz wrote:
 On 03/28/2012 05:24 AM, Tushar Behera wrote:
 On 03/27/2012 12:50 AM, John Stultz wrote:
  So after talking about it at the last Linaro Connect, I've finally
 gotten around to making a first pass at providing config fragments for
 the linaro kernel.  I'd like to propose merging this for 12.04, and
 doing so early so we can make sure that all the desired config options
 are present in the fragments and to allow the vairous linaro build
 systems to begin migrating their config generation over.

 The current tree is here:


 http://git.linaro.org/gitweb?p=people/jstultz/android.git;a=shortlog;h=refs/heads/linaro-configs-3.3




 The most relevant commit being:


 http://git.linaro.org/gitweb?p=people/jstultz/android.git;a=commitdiff;h=da8f6d20e1a768cb486005c5ec62582b6f92990d




 This includes config fragments for a linaro-base, an android-ization
 fragment, and  board configs for panda, origen and imx53.

 I suspect we'll need an ubuntu specific fragment as well as all the
 other board fragments.

 There is likely to be quite a bit of churn as we decide what sort of
 configs are really common and which are board specific. But that's ok.

 Configs are generated from the config fragments, as follows:

 ./scripts/kconfig/merge_config.sh ./configs/linaro-base.conf
 ./configs/android.conf ./configs/panda.conf


 You may see warnings, which are not fatal, but should be reported so
 they can be properly cleaned up.

 I'm asking for Build folks to take a look at the above and consider how
 they might merge in fragment assembly into their system replacing their
 current config generation.

 I'd ask Landing teams to take a look at this, and report any missing
 config options or fragment chunks they'd like to see.

 I have updated origen.conf and linaro-base.conf for testing Samsung LT
 kernel. The results are updated in [1].
 I'll take a look at the changes and try to merge them into my tree.
 
 I have not validated the changes in android.conf, but by merging
 linaro-base.conf and origen.conf, I was able to boot my kernel the way I
 would have expected when using exynos4_defconfig.

 One thing I notice is that I find far too many debug messages with this
 new config.
 You mean the debug output from the merge_config.sh script is a bit
 noisy?  Yea. Likely we'll have some extra noise as we settle out what
 options needs to be generic vs board specific.  But it should decrease
 over time.
 
I was talking about the console log messages upon booting on a target board.

 Going forward, would it be better to have linaro-base.conf, android.conf
 and ubuntu.conf managed centrally and each LT managing their board
 specific configuration file. That way, we can include the changes in our
 board specific configurations in respective topic branches.
 So yea, I'd like to delegate/give away as much management of the configs
 as possible.  :)
 
 That said, I do think that we'll need someone looking at the entire
 cross-board fragment picture (since if everyone needs an option, it
 really isn't board specific).   So it might be a good idea to have basic
 board config fragments that work with upstream. Then any board-specific
 feature branches can add their config needs in as a patch on top.
 
 Does that sound reasonable?
Sounds good. Any config that enables a feature on a topic branch
specific to a board should go in a patch in the topic branch itself.


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Slow/broken USB and Ethernet on Snowballs/Origen boards?

2012-03-19 Thread Tushar Behera
On 03/16/2012 05:20 PM, Mans Rullgard wrote:
 On 16 March 2012 04:14, Sachin Kamat sachin.ka...@linaro.org wrote:
 Hi,

 On 15/03/2012, Mans Rullgard mans.rullg...@linaro.org wrote:
 On 14 March 2012 20:04, Jannis Pohlmann jannis.pohlm...@codethink.co.uk
 wrote:
 Hi,

 I am currently playing with a couple of the development boards for which
 there are Linaro hwpacks and LEBs. Since what I am trying to do requires
 a lot of disk and network I/O, I've been paying special attention to the
 data transfer rates I can get out of these boards.

 Below is a brief summary of my findings.

 3) Origen

  * the internal USB hub runs at Full Speed (12 MBit/s), resulting in a
maximum USB disk I/O of 1.5 MByte/s

  * since the board does not feature Ethernet itself, I tried to attach
a USB Ethernet controller to it, but of course the transfer rate
through that is by the I/O upper limit of the USB hub

  * I did not test wireless but it is not feasible for what I am trying
to do anyway

 Which kernel version are you using on the Origen?  I noticed the same
 problem
 a while back, but it appears to have been fixed in the Samsung landing team
 tree.  There is still another bug present in the Origen kernel preventing
 USB-ethernet working with EHCI.  Some patches have been posted, but they
 have not made it into the trees yet.

 These patches have been added to Samsung LT tree [1].

 [1] git://git.linaro.org/landing-teams/working/samsung/kernel.git
 (branch: tracking)
 
 That branch still needs one more patch (attached).
 
Thanks Mans for the patch. I have applied it to our tree. [1]

I would be nice if you can test this kernel once.

[1] git://git.linaro.org/landing-teams/working/samsung/kernel.git (tracking)
 
 
 
 ___
 linaro-dev mailing list
 linaro-dev@lists.linaro.org
 http://lists.linaro.org/mailman/listinfo/linaro-dev


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [Samsung] [U-Boot] [PATCH] EXYNOS: Rename exynos5_tzpc structure to s5p_tzpc

2012-03-14 Thread Tushar Behera
On 03/15/2012 06:53 AM, Minkyu Kang wrote:
 Dear Chander Kashyap,
 
 On 14 March 2012 22:38, Chander Kashyap chander.kash...@linaro.org wrote:
 Hi Kyungmin Park,

 On 14 March 2012 19:02, Kyungmin Park kmp...@infradead.org wrote:
 Hi Chander,

 On Wed, Mar 14, 2012 at 10:14 PM, Chander Kashyap
 chander.kash...@linaro.org wrote:
 TZPC IP is common across S5P and Exynos based SoC'c. Renaming exynos5_tzpc
 in arch/arm/include/asm/arch-exynos/tzpc.h to s5p_tzpc will allow generic
 usase of tzpc.

 Also modify board/samsung/smdk5250/tzpc_init.c to use s5p_tzpc.

 Signed-off-by: Chander Kashyap chander.kash...@linaro.org
 ---
  arch/arm/include/asm/arch-exynos/tzpc.h |2 +-
  board/samsung/smdk5250/tzpc_init.c  |2 +-
  2 files changed, 2 insertions(+), 2 deletions(-)

 diff --git a/arch/arm/include/asm/arch-exynos/tzpc.h 
 b/arch/arm/include/asm/arch-exynos/tzpc.h
 index 2c9a07b..63736ae 100644
 --- a/arch/arm/include/asm/arch-exynos/tzpc.h
 +++ b/arch/arm/include/asm/arch-exynos/tzpc.h
 @@ -22,7 +22,7 @@
  #define __TZPC_H_

  #ifndef __ASSEMBLY__
 -struct exynos5_tzpc {
 +struct s5p_tzpc {
 I think 'exynos' is preferable. Even though each SOC has different
 I tried to carry forward old conventions as in case of watchdog. I
 will change it to exynos.
 
 I agreed with Kyungmin.
 From now, let's called exynos for common name including s5pc1xx and
 s5pc2xx and exynos4 and exynos5.. etc.
 
From the above list, only s5pc1xx series was not named EXYNOS.


 number of tzpc. It can be covered one exynos_tzpc. or  we can define
 it for each SoC.
 One structure is enough as fields are same.

 
 Thanks
 Minkyu Kang.


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH 2/3] ARM: EXYNOS: Add clkdev lookup entry for lcd clock

2012-03-12 Thread Tushar Behera
On 03/10/2012 07:52 PM, Chenglie He wrote:
 I am doing the suspend and resume of s3cfb on exynos. the clk_on and
 clk_off just failed. I think this is a related issue.
 
Without this patch, the probe for s3cfb driver itself fails - hence what
you are seeing must be different.

 On 29 February 2012 13:45, Tushar Behera tushar.beh...@linaro.org wrote:
 
 Hi Kukjin,

 On 12/01/2011 11:20 AM, Tushar Behera wrote:
 The framebuffer driver needs the clock named 'lcd' as its bus
 clock but the equivalent clock on Exynos4 is named as 'fimd'.
 Hence, create a clkdev lookup entry with the name 'lcd' that
 references the 'fimd' clock.

 Signed-off-by: Tushar Behera tushar.beh...@linaro.org
 ---
  arch/arm/mach-exynos/clock.c |   14 +-
  1 files changed, 9 insertions(+), 5 deletions(-)

 diff --git a/arch/arm/mach-exynos/clock.c b/arch/arm/mach-exynos/clock.c
 index 5d8d483..607ec28 100644
 --- a/arch/arm/mach-exynos/clock.c
 +++ b/arch/arm/mach-exynos/clock.c
 @@ -489,11 +489,6 @@ static struct clk init_clocks_off[] = {
   .enable = exynos4_clk_ip_cam_ctrl,
   .ctrlbit= (1  3),
   }, {
 - .name   = fimd,
 - .devname= exynos4-fb.0,
 - .enable = exynos4_clk_ip_lcd0_ctrl,
 - .ctrlbit= (1  0),
 - }, {
   .name   = hsmmc,
   .devname= s3c-sdhci.0,
   .parent = clk_aclk_133.clk,
 @@ -782,6 +777,13 @@ static struct clk clk_pdma1 = {
   .ctrlbit= (1  1),
  };

 +static struct clk clk_fimd0 = {
 + .name   = fimd,
 + .devname= exynos4-fb.0,
 + .enable = exynos4_clk_ip_lcd0_ctrl,
 + .ctrlbit= (1  0),
 +};
 +
  struct clk *clkset_group_list[] = {
   [0] = clk_ext_xtal_mux,
   [1] = clk_xusbxti,
 @@ -1294,6 +1296,7 @@ static struct clksrc_clk *sysclks[] = {
  static struct clk *clk_cdev[] = {
   clk_pdma0,
   clk_pdma1,
 + clk_fimd0,
  };

  static struct clksrc_clk *clksrc_cdev[] = {
 @@ -1318,6 +1321,7 @@ static struct clk_lookup exynos4_clk_lookup[] = {
   CLKDEV_INIT(s3c-sdhci.3, mmc_busclk.2, clk_sclk_mmc3.clk),
   CLKDEV_INIT(dma-pl330.0, apb_pclk, clk_pdma0),
   CLKDEV_INIT(dma-pl330.1, apb_pclk, clk_pdma1),
 + CLKDEV_INIT(exynos4-fb.0, lcd, clk_fimd0),
  };

  static int xtal_rate;

 Would you please review this patch and let me know your opinion? Without
 this patch, frame-buffer support on EXYNOS4 is broken.

 --
 Tushar Behera

 ___
 linaro-dev mailing list
 linaro-dev@lists.linaro.org
 http://lists.linaro.org/mailman/listinfo/linaro-dev

 


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH 1/4] thermal: exynos: Add thermal interface support for linux thermal layer

2012-03-12 Thread Tushar Behera
;
 + goto err_unregister;
 + }
 +
 + th_zone-sensor_conf = sensor_conf;
 +
 + tab_ptr = (struct freq_pctg_table *)sensor_conf-cooling_data.freq_data;
 + tab_size = sensor_conf-cooling_data.freq_pctg_count;
 +
 + /*Register the cpufreq cooling device*/

Space after '/*' and before '*/'.

 + th_zone-cool_dev_size = 1;
 + count = 0;
 + th_zone-cool_dev[count] = cpufreq_cooling_register(
 + (struct freq_pctg_table *)(tab_ptr[count]),
 + tab_size, cpumask_of(0));
 +
 + if (IS_ERR(th_zone-cool_dev[count])) {
 + pr_err(Failed to register cpufreq cooling device\n);
 + ret = -EINVAL;
 + th_zone-cool_dev_size = 0;
 + goto err_unregister;
 + }
 +
 + th_zone-therm_dev = thermal_zone_device_register(sensor_conf-name,
 + 3, NULL, exynos4_dev_ops, 0, 0, 0, 1000);
 + if (IS_ERR(th_zone-therm_dev)) {
 + pr_err(Failed to register thermal zone device\n);
 + ret = -EINVAL;
 + goto err_unregister;
 + }
 +
 + th_zone-active_interval = 1;
 + th_zone-idle_interval = 10;
 +
 + exynos4_set_mode(th_zone-therm_dev, THERMAL_DEVICE_DISABLED);
 +
 + pr_info(Exynos: Kernel Thermal management registered\n);
 +
 + return 0;
 +
 +err_unregister:
 + exynos4_unregister_thermal();
 + return ret;
 +}
 +EXPORT_SYMBOL(exynos4_register_thermal);
 +
 +void exynos4_unregister_thermal(void)
 +{
 + unsigned int i;
 +
 + for (i = 0; i  th_zone-cool_dev_size; i++) {
 + if (th_zone  th_zone-cool_dev[i])
 + cpufreq_cooling_unregister(th_zone-cool_dev[i]);
 + }
 +
 + if (th_zone  th_zone-therm_dev)
 + thermal_zone_device_unregister(th_zone-therm_dev);
 +
 + kfree(th_zone);
 +
 + pr_info(Exynos: Kernel Thermal management unregistered\n);
 +}
 +EXPORT_SYMBOL(exynos4_unregister_thermal);
 diff --git a/include/linux/exynos_thermal.h b/include/linux/exynos_thermal.h
 new file mode 100644
 index 000..186e409
 --- /dev/null
 +++ b/include/linux/exynos_thermal.h
 @@ -0,0 +1,72 @@
 +/* linux/include/linux/exynos_thermal.h
 + *
 + * Copyright (c) 2010-2011 Samsung Electronics Co., Ltd.
 + *   http://www.samsung.com
 + *
 + * This program is free software; you can redistribute it and/or modify
 + * it under the terms of the GNU General Public License version 2 as
 + * published by the Free Software Foundation.
 +*/
 +
 +#ifndef THERMAL_INTERFACE_H
 +#define THERMAL_INTERFACE_H
 +/* CPU Zone information */
 +
 +#define SENSOR_NAME_LEN  16
 +#define MAX_TRIP_COUNT   8
 +
 +#define PANIC_ZONE  4
 +#define WARN_ZONE   3
 +#define MONITOR_ZONE2
 +#define SAFE_ZONE   1
 +#define NO_ACTION   0
 +

TABs for intending in above defines.


 +
 +struct   thermal_trip_point_conf {

A single space after struct should suffice, no TABs there. It is helpful
when we grep for the definition of 'struct thermal_trip_point_conf {'.

 + int trip_val[MAX_TRIP_COUNT];
 + int trip_count;
 +};
 +
 +struct   thermal_cooling_conf {

Ditto.

 + struct freq_pctg_table freq_data[MAX_TRIP_COUNT];
 + int freq_pctg_count;
 +};
 +
 +/**
 + * struct exynos4_tmu_platform_data
 + * @name: name of the temperature sensor
 + * @read_temperature: A function pointer to read temperature info
 + * @private_data: Temperature sensor private data
 + * @sensor_data: Sensor specific information like trigger temperature, level
 + */
 +struct thermal_sensor_conf {
 + charname[SENSOR_NAME_LEN];
 + int (*read_temperature)(void *data);
 + struct  thermal_trip_point_conf trip_data;
 + struct  thermal_cooling_conf cooling_data;
 + void*private_data;
 +};
 +
 +/**
 + * exynos4_register_thermal: Register to the exynos thermal interface.
 + * @sensor_conf:   Structure containing temperature sensor information
 + *
 + * returns zero on success, else negative errno.
 + */
 +int exynos4_register_thermal(struct thermal_sensor_conf *sensor_conf);
 +
 +/**
 + * exynos4_unregister_thermal: Un-register from the exynos thermal interface.
 + *
 + * return not applicable.
 + */
 +void exynos4_unregister_thermal(void);
 +
 +/**
 + * exynos4_report_trigger: Report any trigger level crossed in the
 + *   temperature sensor. This may be useful to take any cooling action.
 + *
 + * return not applicable.
 + */
 +extern void exynos4_report_trigger(void);
 +#endif


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH 1/2] ARM: EXYNOS: Add EHCI AHB burst function

2012-02-29 Thread Tushar Behera
On 02/29/2012 06:31 PM, Thomas Abraham wrote:
 Hi Sangwook,
 
 On 29 February 2012 18:11, Sangwook Lee sangwook@linaro.org wrote:
 Enable burst transfer from AHB for EHCI.
 This fixes data transfer of USB Ethernet with EHCI.
 Without this patch, scp hardly works.

 Signed-off-by: Sangwook Lee sangwook@linaro.org
 ---
  arch/arm/mach-exynos/setup-usb-phy.c  |6 ++
  arch/arm/plat-samsung/devs.c  |2 ++
  arch/arm/plat-samsung/include/plat/ehci.h |   19 +++
  3 files changed, 27 insertions(+), 0 deletions(-)

 diff --git a/arch/arm/mach-exynos/setup-usb-phy.c 
 b/arch/arm/mach-exynos/setup-usb-phy.c
 index 41743d2..5a20460 100644
 --- a/arch/arm/mach-exynos/setup-usb-phy.c
 +++ b/arch/arm/mach-exynos/setup-usb-phy.c
 @@ -18,6 +18,7 @@
  #include mach/regs-usb-phy.h
  #include plat/cpu.h
  #include plat/usb-phy.h
 +#include plat/ehci.h

  static atomic_t host_usage;

 @@ -149,3 +150,8 @@ int s5p_usb_phy_exit(struct platform_device *pdev, int 
 type)

return -EINVAL;
  }
 +
 +void s5p_ehci_burst_enable(struct platform_device *pdev, void __iomem *base)
 +{
 +   writel(EHCI_INSNREG00_ENABLE_BURST, base + EHCI_INSNREG00);
 +}
 
 This functionality can be added in ehci-s5p itself and avoid adding a
 new platform callback in platform data. If this is specific to exynos,
 driver data could be added in ehci-s5p to indicate platforms that need
 this to be enabled.
 
Am I right in assuming that ehci-s5p driver can also be used for
mach-s5pv210?

The related bit-fields are reserved in S5PV210. So, won't it cause any
side-effects?


 Thanks,
 Thomas.
 
 [...]
 --
 To unsubscribe from this list: send the line unsubscribe linux-samsung-soc 
 in
 the body of a message to majord...@vger.kernel.org
 More majordomo info at  http://vger.kernel.org/majordomo-info.html


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH 1/2] ARM: EXYNOS: Add USB HOST register definitions

2012-02-29 Thread Tushar Behera
On 02/29/2012 08:09 PM, Jingoo Han wrote:
 This patch adds USB HOST register definitions. The definition for
 EHCI INSNREG00 regiser and corresponding bit field definitions are
 added.
 
 Signed-off-by: Jingoo Han jg1@samsung.com
 ---
  arch/arm/mach-exynos/include/mach/regs-usb-host.h |   23 
 +
  1 files changed, 23 insertions(+), 0 deletions(-)
  create mode 100644 arch/arm/mach-exynos/include/mach/regs-usb-host.h
 
 diff --git a/arch/arm/mach-exynos/include/mach/regs-usb-host.h 
 b/arch/arm/mach-exynos/include/mach/regs-usb-host.h
 new file mode 100644
 index 000..1a60f27
 --- /dev/null
 +++ b/arch/arm/mach-exynos/include/mach/regs-usb-host.h
 @@ -0,0 +1,23 @@
 +/*
 + * Copyright (C) 2012 Samsung Electronics Co.Ltd
 + *   http://www.samsung.com
 + *
 + * EXYNOS - USB HOST register definitions
 + *
 + * This program is free software; you can redistribute it and/or modify
 + * it under the terms of the GNU General Public License version 2 as
 + * published by the Free Software Foundation.
 + */
 +
 +#ifndef __REGS_USB_HOST_H
 +#define __REGS_USB_HOST_H __FILE__
 +
 +#define EHCI_INSNREG00(base) (base + 0x90)
 +#define EHCI_ENA_INCR16  (0x1  25)
 +#define EHCI_ENA_INCR8   (0x1  24)
 +#define EHCI_ENA_INCR4   (0x1  23)
 +#define EHCI_ENA_INCRX_ALIGN (0x1  22)
 +#define EHCI_ENABLE_DMA_INCR (EHCI_ENA_INCR16 | EHCI_ENA_INCR8| \
 +  EHCI_ENA_INCR4 | EHCI_ENA_INCRX_ALIGN)
 +
As per the definition of bit-fields in other registers, it would be good
to prepend the complete register name before the bit-field defines.

EHCI_INSNREG00_ENA_INCR16 ...

Considering the similarity of this patchset with the earlier patchset
from Sangwook, IMO, it would be appropriate to give him due credit in
both these patches.

 +#endif /* __REGS_USB_HOST_H */


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH 2/3] ARM: EXYNOS: Add clkdev lookup entry for lcd clock

2012-02-29 Thread Tushar Behera
Hi Kukjin,

On 03/01/2012 09:36 AM, Kukjin Kim wrote:
 Jingoo Han wrote:

 Hi Tushar,

 
 (please don't top-post)
 
 -Original Message-
 From: linux-samsung-soc-ow...@vger.kernel.org [mailto:linux-samsung-soc-
 ow...@vger.kernel.org] On Behalf
 Of Tushar Behera
 Sent: Thursday, December 01, 2011 2:50 PM
 To: linux-samsung-...@vger.kernel.org
 Cc: kgene@samsung.com; linaro-dev@lists.linaro.org;
 patc...@linaro.org
 Subject: [PATCH 2/3] ARM: EXYNOS: Add clkdev lookup entry for lcd clock

 The framebuffer driver needs the clock named 'lcd' as its bus
 clock but the equivalent clock on Exynos4 is named as 'fimd'.
 Hence, create a clkdev lookup entry with the name 'lcd' that
 references the 'fimd' clock.

 Signed-off-by: Tushar Behera tushar.beh...@linaro.org

 Acked-by: Jingoo Han jg1@samsung.com

 
 OK, I will apply this with Sylwester's 'reviewed-by' I looked at  before.
 
Thanks. Do you want me rebase this patch on your latest for-next and resend?

 BTW, Tushar, what's the [1/3] and [3/3] in this series? If they are still
 needed now, could you please re-send? Maybe I missed.
 
[PATCH 1/3] ARM: EXYNOS: Increase DMA pool allocator size for framebuffer
- It should be dropped.

[PATCH 3/3] ARM: EXYNOS: Invert VCLK polarity for framebuffer on Origen
board
- It has already been applied.


 Thanks.
 
 Best regards,
 Kgene.
 --
 Kukjin Kim kgene@samsung.com, Senior Engineer,
 SW Solution Development Team, Samsung Electronics Co., Ltd.
 


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH 2/3] ARM: EXYNOS: Add clkdev lookup entry for lcd clock

2012-02-28 Thread Tushar Behera
Hi Kukjin,

On 12/01/2011 11:20 AM, Tushar Behera wrote:
 The framebuffer driver needs the clock named 'lcd' as its bus
 clock but the equivalent clock on Exynos4 is named as 'fimd'.
 Hence, create a clkdev lookup entry with the name 'lcd' that
 references the 'fimd' clock.
 
 Signed-off-by: Tushar Behera tushar.beh...@linaro.org
 ---
  arch/arm/mach-exynos/clock.c |   14 +-
  1 files changed, 9 insertions(+), 5 deletions(-)
 
 diff --git a/arch/arm/mach-exynos/clock.c b/arch/arm/mach-exynos/clock.c
 index 5d8d483..607ec28 100644
 --- a/arch/arm/mach-exynos/clock.c
 +++ b/arch/arm/mach-exynos/clock.c
 @@ -489,11 +489,6 @@ static struct clk init_clocks_off[] = {
   .enable = exynos4_clk_ip_cam_ctrl,
   .ctrlbit= (1  3),
   }, {
 - .name   = fimd,
 - .devname= exynos4-fb.0,
 - .enable = exynos4_clk_ip_lcd0_ctrl,
 - .ctrlbit= (1  0),
 - }, {
   .name   = hsmmc,
   .devname= s3c-sdhci.0,
   .parent = clk_aclk_133.clk,
 @@ -782,6 +777,13 @@ static struct clk clk_pdma1 = {
   .ctrlbit= (1  1),
  };
  
 +static struct clk clk_fimd0 = {
 + .name   = fimd,
 + .devname= exynos4-fb.0,
 + .enable = exynos4_clk_ip_lcd0_ctrl,
 + .ctrlbit= (1  0),
 +};
 +
  struct clk *clkset_group_list[] = {
   [0] = clk_ext_xtal_mux,
   [1] = clk_xusbxti,
 @@ -1294,6 +1296,7 @@ static struct clksrc_clk *sysclks[] = {
  static struct clk *clk_cdev[] = {
   clk_pdma0,
   clk_pdma1,
 + clk_fimd0,
  };
  
  static struct clksrc_clk *clksrc_cdev[] = {
 @@ -1318,6 +1321,7 @@ static struct clk_lookup exynos4_clk_lookup[] = {
   CLKDEV_INIT(s3c-sdhci.3, mmc_busclk.2, clk_sclk_mmc3.clk),
   CLKDEV_INIT(dma-pl330.0, apb_pclk, clk_pdma0),
   CLKDEV_INIT(dma-pl330.1, apb_pclk, clk_pdma1),
 + CLKDEV_INIT(exynos4-fb.0, lcd, clk_fimd0),
  };
  
  static int xtal_rate;

Would you please review this patch and let me know your opinion? Without
this patch, frame-buffer support on EXYNOS4 is broken.

-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Selecting default framebuffer through kernel command-line ?

2012-02-14 Thread Tushar Behera
Hi,

This issue is currently specific to the Ubuntu LEB for Origen and is
also being discussed under the LP bug #906762 [1]. Writing to a wider
audience to get views of others who might have come across similar
situation before.

Currently the LCD FB is registered as /dev/fb0 and the HDMI FB is
registered as /dev/fb1. While booting up, by default /dev/fb0 is used as
the primary display device and the output is routed accordingly. To use
the HDMI display, we need to update /etc/X11/xorg.conf (and this needs
to be updated every time we update the file-system.)

If we don't enable support for LCD FB, the display comes on HDMI during
boot-time, but again that is not desirable considering we want to use
both the LCD and HDMI with the same kernel binary.

Is there any way where we can force the default frame-buffer (whether to
choose /dev/fb0 or /dev/fb1) during boot-time, might be through some
kernel command-line options?

[1] https://bugs.launchpad.net/linaro-landing-team-samsung/+bug/906762
-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: 2012-01 Samsung kernel trees

2012-01-29 Thread Tushar Behera
Hi Vic,

CC: sams...@lists.linaro.org

On 01/30/2012 08:54 AM, Vic Berdin wrote:
 Hi everyone,
 
 
 Any chance the linux-linaro-*-lt-samsung kernel trees will get updated

Samsung LT maintains following tree [1], you may have a look at it.

Is that what you are looking for?

 (soon)? By the way, is there a daily/nightly build source for these trees? I
 am eager to try them out.
 
The daily hwpacks can be found at [2].


[1] git://git.linaro.org/landing-teams/working/samsung/kernel.git (tracking)
[2] http://snapshots.linaro.org/oneiric/lt-origen-oneiric/latest/

 
 TIA - Vic
 
 
 
 ___
 linaro-dev mailing list
 linaro-dev@lists.linaro.org
 http://lists.linaro.org/mailman/listinfo/linaro-dev


-- 
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH] ARM: EXYNOS: Add USB OHCI support to ORIGEN board

2011-12-21 Thread Tushar Behera
Signed-off-by: Tushar Behera tushar.beh...@linaro.org
Signed-off-by: Angus Ainslie angus.ains...@linaro.org
---

This patch are based Kukjin's for-next branch and OHCI related
patches from Jingoo Han.
[PATCH v2 0/3] Support Samsung Exynos OHCI device and driver

 arch/arm/mach-exynos/Kconfig   |1 +
 arch/arm/mach-exynos/mach-origen.c |   13 +
 2 files changed, 14 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-exynos/Kconfig b/arch/arm/mach-exynos/Kconfig
index bd1bb9f1..0da2ced 100644
--- a/arch/arm/mach-exynos/Kconfig
+++ b/arch/arm/mach-exynos/Kconfig
@@ -304,6 +304,7 @@ config MACH_ORIGEN
select SAMSUNG_DEV_PWM
select EXYNOS4_DEV_DMA
select EXYNOS4_DEV_PD
+   select EXYNOS4_DEV_USB_OHCI
select EXYNOS4_SETUP_FIMD0
select EXYNOS4_SETUP_SDHCI
select EXYNOS4_SETUP_USB_PHY
diff --git a/arch/arm/mach-exynos/mach-origen.c 
b/arch/arm/mach-exynos/mach-origen.c
index f56d027..a0116036 100644
--- a/arch/arm/mach-exynos/mach-origen.c
+++ b/arch/arm/mach-exynos/mach-origen.c
@@ -42,6 +42,7 @@
 #include plat/fb.h
 #include plat/mfc.h
 
+#include mach/ohci.h
 #include mach/map.h
 
 /* Following are default values for UCON, ULCON and UFCON UART registers */
@@ -487,6 +488,16 @@ static void __init origen_ehci_init(void)
s5p_ehci_set_platdata(pdata);
 }
 
+/* USB OHCI */
+static struct exynos4_ohci_platdata origen_ohci_pdata;
+
+static void __init origen_ohci_init(void)
+{
+   struct exynos4_ohci_platdata *pdata = origen_ohci_pdata;
+
+   exynos4_ohci_set_platdata(pdata);
+}
+
 static struct gpio_keys_button origen_gpio_keys_table[] = {
{
.code   = KEY_MENU,
@@ -627,6 +638,7 @@ static struct platform_device *origen_devices[] __initdata 
= {
s5p_device_mfc_l,
s5p_device_mfc_r,
s5p_device_mixer,
+   exynos4_device_ohci,
exynos4_device_pd[PD_LCD0],
exynos4_device_pd[PD_TV],
exynos4_device_pd[PD_G3D],
@@ -702,6 +714,7 @@ static void __init origen_machine_init(void)
s3c_sdhci0_set_platdata(origen_hsmmc0_pdata);
 
origen_ehci_init();
+   origen_ohci_init();
clk_xusbxti.rate = 2400;
 
s5p_tv_setup();
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH] i2c-s3c2410: Fix return code of s3c24xx_i2c_parse_dt_gpio

2011-12-18 Thread Tushar Behera

Ping

On 12/09/2011 03:33 PM, Tushar Behera wrote:

s3c24xx_i2c_parse_dt_gpio is called when cfg_gpio is not defined
in the platform data of the i2c device. When DT is not enabled,
the above function always returns -EINVAL. Since there can be
some i2c devices which don't need to configure any gpio lines,
the probe of such devices would fail here. Changing the default
return value to success would fix this issue.

Signed-off-by: Tushar Beheratushar.beh...@linaro.org
---
This patch is rebased on Kukjin's for-next branch.
d3d936c Merge branch 'samsung-fixes-2' into for-next

  drivers/i2c/busses/i2c-s3c2410.c |2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/i2c/busses/i2c-s3c2410.c b/drivers/i2c/busses/i2c-s3c2410.c
index 2754cef..b5caa42 100644
--- a/drivers/i2c/busses/i2c-s3c2410.c
+++ b/drivers/i2c/busses/i2c-s3c2410.c
@@ -786,7 +786,7 @@ static void s3c24xx_i2c_dt_gpio_free(struct s3c24xx_i2c 
*i2c)
  #else
  static int s3c24xx_i2c_parse_dt_gpio(struct s3c24xx_i2c *i2c)
  {
-   return -EINVAL;
+   return 0;
  }

  static void s3c24xx_i2c_dt_gpio_free(struct s3c24xx_i2c *i2c)



--
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH] i2c-s3c2410: Fix return code of s3c24xx_i2c_parse_dt_gpio

2011-12-09 Thread Tushar Behera
s3c24xx_i2c_parse_dt_gpio is called when cfg_gpio is not defined
in the platform data of the i2c device. When DT is not enabled,
the above function always returns -EINVAL. Since there can be
some i2c devices which don't need to configure any gpio lines,
the probe of such devices would fail here. Changing the default
return value to success would fix this issue.

Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
This patch is rebased on Kukjin's for-next branch.
d3d936c Merge branch 'samsung-fixes-2' into for-next

 drivers/i2c/busses/i2c-s3c2410.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/i2c/busses/i2c-s3c2410.c b/drivers/i2c/busses/i2c-s3c2410.c
index 2754cef..b5caa42 100644
--- a/drivers/i2c/busses/i2c-s3c2410.c
+++ b/drivers/i2c/busses/i2c-s3c2410.c
@@ -786,7 +786,7 @@ static void s3c24xx_i2c_dt_gpio_free(struct s3c24xx_i2c 
*i2c)
 #else
 static int s3c24xx_i2c_parse_dt_gpio(struct s3c24xx_i2c *i2c)
 {
-   return -EINVAL;
+   return 0;
 }
 
 static void s3c24xx_i2c_dt_gpio_free(struct s3c24xx_i2c *i2c)
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH 0/2] Add PM_RUNTIME related fixes for PL330

2011-12-06 Thread Tushar Behera
Hi,

When PM_RUNTIME is enabled, PL330 probe fails because of some
mismatch in pm_runtime calls. This patchset fixes those issues.

This patch is based on Kukjin's for-next branch and tested on
EXYNOS4 based Origen board.

d3d936c Merge branch 'samsung-fixes-2' into for-next

Tushar Behera (2):
To Vinod Koul vinod.k...@intel.com:
  DMA: PL330: Remove pm_runtime_xxx calls from pl330 probe/remove

To Kukjin Kim kgene@samsung.com:
  ARM: EXYNOS: Add apb_pclk clkdev entry for mdma1

 arch/arm/mach-exynos/clock.c |1 +
 drivers/dma/pl330.c  |   17 ++---
 2 files changed, 3 insertions(+), 15 deletions(-)

-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH 1/2] DMA: PL330: Remove pm_runtime_xxx calls from pl330 probe/remove

2011-12-06 Thread Tushar Behera
amba_probe() now calls pm_runtime_get_noresume() and pm_runtime_enable()
for the devices before the device probe is called. Hence we don't need
to call pm_runtime_get_xxx and pm_runtime_enable() in device probe again.
In the same way, since amba_remove() calls the respective pm_runtime
functions, those functions need not be called from device remove.

This patch fixes following run time error with pl330 driver.

dma-pl330 dma-pl330.0: Unbalanced pm_runtime_enable!
dma-pl330 dma-pl330.0: failed to get runtime pm

Signed-off-by: Giridhar Maruthy giridhar.maru...@linaro.org
Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
 drivers/dma/pl330.c |   17 ++---
 1 files changed, 2 insertions(+), 15 deletions(-)

diff --git a/drivers/dma/pl330.c b/drivers/dma/pl330.c
index a626e15..cd07121 100644
--- a/drivers/dma/pl330.c
+++ b/drivers/dma/pl330.c
@@ -834,17 +834,7 @@ pl330_probe(struct amba_device *adev, const struct amba_id 
*id)
 
amba_set_drvdata(adev, pdmac);
 
-#ifdef CONFIG_PM_RUNTIME
-   /* to use the runtime PM helper functions */
-   pm_runtime_enable(adev-dev);
-
-   /* enable the power domain */
-   if (pm_runtime_get_sync(adev-dev)) {
-   dev_err(adev-dev, failed to get runtime pm\n);
-   ret = -ENODEV;
-   goto probe_err1;
-   }
-#else
+#ifndef CONFIG_PM_RUNTIME
/* enable dma clk */
clk_enable(pdmac-clk);
 #endif
@@ -977,10 +967,7 @@ static int __devexit pl330_remove(struct amba_device *adev)
res = adev-res;
release_mem_region(res-start, resource_size(res));
 
-#ifdef CONFIG_PM_RUNTIME
-   pm_runtime_put(adev-dev);
-   pm_runtime_disable(adev-dev);
-#else
+#ifndef CONFIG_PM_RUNTIME
clk_disable(pdmac-clk);
 #endif
 
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH 2/2] ARM: EXYNOS: Add apb_pclk clkdev entry for mdma1

2011-12-06 Thread Tushar Behera
Amba core assumes the pclk to be named as apb_pclk. During device probe,
it tries to get that clock and enable that. When PM_RUNTIME is enabled,
dma clock is not explicitly enabled in pl330_probe, which causes device
probe to fail. Adding a clkdev entry for apb_pclk for mdma1 fixes the
problem.

This patch fixes following runtime error.

dma-pl330 dma-pl330.2: PERIPH_ID 0x0, PCELL_ID 0x0 !
dma-pl330: probe of dma-pl330.2 failed with error -22

Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
 arch/arm/mach-exynos/clock.c |1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-exynos/clock.c b/arch/arm/mach-exynos/clock.c
index 28e2842..eb33a7a 100644
--- a/arch/arm/mach-exynos/clock.c
+++ b/arch/arm/mach-exynos/clock.c
@@ -1326,6 +1326,7 @@ static struct clk_lookup exynos4_clk_lookup[] = {
CLKDEV_INIT(s3c-sdhci.3, mmc_busclk.2, clk_sclk_mmc3.clk),
CLKDEV_INIT(dma-pl330.0, apb_pclk, clk_pdma0),
CLKDEV_INIT(dma-pl330.1, apb_pclk, clk_pdma1),
+   CLKDEV_INIT(dma-pl330.2, apb_pclk, clk_mdma1),
 };
 
 static int xtal_rate;
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: error while running l-m-c on one build machine

2011-10-14 Thread Tushar Behera

Hi Fathi,

On Friday 14 October 2011 11:45 AM, Fathi Boudra wrote:

Hi Tushar,

On 14 October 2011 06:40, Tushar Beheratushar.beh...@linaro.org  wrote:

Hi,

In one of the build machines, I am getting following error [1] while
building a qemu image with l-m-c using leb build for origen. The build fails
only on this one machine.


Do yo use same the linaro-image-tools and qemu versions on both machines?



Both the machines are using same version of linaro-image-tools.

As for qemu versions, the working setup has qemu 0.14.1+ whereas the 
faulty build machine has qemu 0.14.0+.


If this is one concern area, I will upgrade qemu on the faulty build 
machine and recheck.




Does anybody have a clue about what might be wrong with that build machine?

The build machine is a x86 based system running Ubuntu 11.04. If any
specific information required, please let me know.

File ./linaro-image-tools/linaro-media-create, line 156, inmodule
verified_files, *hwpacks)
File
~/linaro-image-tools/linaro_image_tools/media_create/chroot_utils.py,
line 64, in install_hwpacks
install_hwpack(chroot_dir, hwpack_file, hwpack_force_yes or hwpack_verified)
File
~/linaro-image-tools/linaro_image_tools/media_create/chroot_utils.py,
line 85, in install_hwpack
cmd_runner.run(args, as_root=True, chroot=chroot_dir).wait()
File
~/linaro-image-tools/linaro_image_tools/cmd_runner.py,
line 100, in wait
raise SubcommandNonZeroReturnValue(self._my_args, returncode)
linaro_image_tools.cmd_runner.SubcommandNonZeroReturnValue: Sub process
['chroot', '/tmp/tmpvZJf7s/binary', 'linaro-hwpack-install',
'/hwpack_linaro-lt-origen_20110929-0_armel_supported.tar.gz'] returned a
non-zero value: 100

[1] http://pastebin.com/AAcCTTi2
--
Regards,
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev







Thanks,
--
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: error while running l-m-c on one build machine

2011-10-14 Thread Tushar Behera

Hi Fathi,

On Friday 14 October 2011 04:22 PM, Tushar Behera wrote:

Hi Fathi,

On Friday 14 October 2011 11:45 AM, Fathi Boudra wrote:

Hi Tushar,

On 14 October 2011 06:40, Tushar Beheratushar.beh...@linaro.org wrote:

Hi,

In one of the build machines, I am getting following error [1] while
building a qemu image with l-m-c using leb build for origen. The
build fails
only on this one machine.


Do yo use same the linaro-image-tools and qemu versions on both machines?



Both the machines are using same version of linaro-image-tools.

As for qemu versions, the working setup has qemu 0.14.1+ whereas the
faulty build machine has qemu 0.14.0+.

If this is one concern area, I will upgrade qemu on the faulty build
machine and recheck.




I upgraded the faulty machine to 11.10 oneiric. With that, I don't find 
any error in l-m-c build.


Thanks for your time.



Does anybody have a clue about what might be wrong with that build
machine?

The build machine is a x86 based system running Ubuntu 11.04. If any
specific information required, please let me know.

File ./linaro-image-tools/linaro-media-create, line 156, inmodule
verified_files, *hwpacks)
File
~/linaro-image-tools/linaro_image_tools/media_create/chroot_utils.py,
line 64, in install_hwpacks
install_hwpack(chroot_dir, hwpack_file, hwpack_force_yes or
hwpack_verified)
File
~/linaro-image-tools/linaro_image_tools/media_create/chroot_utils.py,
line 85, in install_hwpack
cmd_runner.run(args, as_root=True, chroot=chroot_dir).wait()
File
~/linaro-image-tools/linaro_image_tools/cmd_runner.py,
line 100, in wait
raise SubcommandNonZeroReturnValue(self._my_args, returncode)
linaro_image_tools.cmd_runner.SubcommandNonZeroReturnValue: Sub process
['chroot', '/tmp/tmpvZJf7s/binary', 'linaro-hwpack-install',
'/hwpack_linaro-lt-origen_20110929-0_armel_supported.tar.gz']
returned a
non-zero value: 100

[1] http://pastebin.com/AAcCTTi2
--
Regards,
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev







Thanks,



--
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


error while running l-m-c on one build machine

2011-10-13 Thread Tushar Behera

Hi,

In one of the build machines, I am getting following error [1] while 
building a qemu image with l-m-c using leb build for origen. The build 
fails only on this one machine.


Does anybody have a clue about what might be wrong with that build machine?

The build machine is a x86 based system running Ubuntu 11.04. If any 
specific information required, please let me know.


File ./linaro-image-tools/linaro-media-create, line 156, in module
verified_files, *hwpacks)
File
~/linaro-image-tools/linaro_image_tools/media_create/chroot_utils.py,
line 64, in install_hwpacks
install_hwpack(chroot_dir, hwpack_file, hwpack_force_yes or hwpack_verified)
File
~/linaro-image-tools/linaro_image_tools/media_create/chroot_utils.py,
line 85, in install_hwpack
cmd_runner.run(args, as_root=True, chroot=chroot_dir).wait()
File
~/linaro-image-tools/linaro_image_tools/cmd_runner.py,
line 100, in wait
raise SubcommandNonZeroReturnValue(self._my_args, returncode)
linaro_image_tools.cmd_runner.SubcommandNonZeroReturnValue: Sub process
['chroot', '/tmp/tmpvZJf7s/binary', 'linaro-hwpack-install',
'/hwpack_linaro-lt-origen_20110929-0_armel_supported.tar.gz'] returned 
a non-zero value: 100


[1] http://pastebin.com/AAcCTTi2
--
Regards,
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH] ARM: EXYNOS4: Setup consistent dma size at boot time

2011-10-12 Thread Tushar Behera

Hi Kukjin,

On Monday 12 September 2011 11:25 AM, Tushar Behera wrote:

Some of the boards under mach-exynos4 initialize frame-buffers
for which the memory requirement is more than 2MB (Nuri board requires
around 4MB, Origen requires around 2.6MB), hence the default dma pool
allocation size of 2MB is not sufficient. The consistent dma size is
hence increased to successfully allocate memory for those boards.

Depends on ARM: Add init_consistent_dma_size()
by Jon Medhurst (99d1717dd7fecf2b10195b0d864323b952b4eba0).

CC: Jon Medhurstt...@yxit.co.uk
Signed-off-by: Tushar Beheratushar.beh...@linaro.org
---
  arch/arm/mach-exynos4/cpu.c |2 ++
  1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-exynos4/cpu.c b/arch/arm/mach-exynos4/cpu.c
index 2d8a40c..45d8bfa 100644
--- a/arch/arm/mach-exynos4/cpu.c
+++ b/arch/arm/mach-exynos4/cpu.c
@@ -10,6 +10,7 @@

  #includelinux/sched.h
  #includelinux/sysdev.h
+#includelinux/dma-mapping.h

  #includeasm/mach/map.h
  #includeasm/mach/irq.h
@@ -136,6 +137,7 @@ static void exynos4_idle(void)
  void __init exynos4_map_io(void)
  {
iotable_init(exynos4_iodesc, ARRAY_SIZE(exynos4_iodesc));
+   init_consistent_dma_size(SZ_8M);

/* initialize device information early */
exynos4_default_sdhci0();


Would you please consider this patch for 3.2?

--
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH V2 1/2] ARM: S3C2410: Add __init attribute to usb_simtec_init()

2011-10-09 Thread Tushar Behera

Hi Sergei,

On Saturday 08 October 2011 07:48 PM, Sergei Shtylyov wrote:

Hello.

On 07-10-2011 15:55, Tushar Behera wrote:


usb_simtec_init() references s3c_ohci_set_platdata() which is defined
with __init attribute. Hence to remove section mismatch warning, __init
attribute is added to usb_simtec_init().



It removes following two warnigs.



WARNING: vmlinux.o(.text+0x1460c): Section mismatch in reference from
the function usb_simtec_init() to the function
.init.text:s3c_ohci_set_platdata()
The function usb_simtec_init() references the function
__init s3c_ohci_set_platdata().



WARNING: vmlinux.o(.text+0x14650): Section mismatch in reference from
the function usb_simtec_init() to the (unknown reference)
.init.data:(unknown)
The function usb_simtec_init() references the (unknown reference)
__initdata (unknown).



Signed-off-by: Tushar Beheratushar.beh...@linaro.org

[...]


diff --git a/arch/arm/mach-s3c2410/usb-simtec.h
b/arch/arm/mach-s3c2410/usb-simtec.h
index 03842ed..43cc88f 100644
--- a/arch/arm/mach-s3c2410/usb-simtec.h
+++ b/arch/arm/mach-s3c2410/usb-simtec.h
@@ -12,5 +12,5 @@
* published by the Free Software Foundation.
*/

-extern int usb_simtec_init(void);
+extern int __init usb_simtec_init(void);


Function prototypes don't need to be annotated with __init.



I agree that function prototypes don't require to be annotated.

But, will it not be better to have same annotation for both function 
prototypes and the function definitions?



WBR, Sergei



Thanks for your review.
--
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH V2 2/2] ARM: S3C2410: irq: Remove __init attributes to fix compilation warning

2011-10-07 Thread Tushar Behera
The structure *_irq_driver is passed as a parameter in
sysdev_driver_register(). This in turn would add this structure to a
list which may be traversed later.

Hence, even though the functions referenced through this structure have
__init attribute, we cannot possibly add __initdata attribute to it.

To remove compilation warnings, the functions referenced thorugh this
structure are also defined without __init attribute.

It removes following two warnings.

WARNING: vmlinux.o(.data+0x4f58): Section mismatch in reference from the
variable s3c2416_irq_driver to the function .init.text:s3c2416_irq_add()
The variable s3c2416_irq_driver references the function __init s3c2416_irq_add()

WARNING: vmlinux.o(.data+0x7c50): Section mismatch in reference from the
variable s3c2443_irq_driver to the function .init.text:s3c2443_irq_add()
The variable s3c2443_irq_driver references the function __init s3c2443_irq_add()

Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
 arch/arm/mach-s3c2416/irq.c |4 ++--
 arch/arm/mach-s3c2443/irq.c |4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm/mach-s3c2416/irq.c b/arch/arm/mach-s3c2416/irq.c
index 28ad20d..53e8e57 100644
--- a/arch/arm/mach-s3c2416/irq.c
+++ b/arch/arm/mach-s3c2416/irq.c
@@ -194,7 +194,7 @@ static struct irq_chip s3c2416_irq_uart3 = {
 
 /* IRQ initialisation code */
 
-static int __init s3c2416_add_sub(unsigned int base,
+static int s3c2416_add_sub(unsigned int base,
   void (*demux)(unsigned int,
 struct irq_desc *),
   struct irq_chip *chip,
@@ -213,7 +213,7 @@ static int __init s3c2416_add_sub(unsigned int base,
return 0;
 }
 
-static int __init s3c2416_irq_add(struct sys_device *sysdev)
+static int s3c2416_irq_add(struct sys_device *sysdev)
 {
printk(KERN_INFO S3C2416: IRQ Support\n);
 
diff --git a/arch/arm/mach-s3c2443/irq.c b/arch/arm/mach-s3c2443/irq.c
index 83ecb11..18585dd 100644
--- a/arch/arm/mach-s3c2443/irq.c
+++ b/arch/arm/mach-s3c2443/irq.c
@@ -222,7 +222,7 @@ static struct irq_chip s3c2443_irq_cam = {
 
 /* IRQ initialisation code */
 
-static int __init s3c2443_add_sub(unsigned int base,
+static int s3c2443_add_sub(unsigned int base,
   void (*demux)(unsigned int,
 struct irq_desc *),
   struct irq_chip *chip,
@@ -241,7 +241,7 @@ static int __init s3c2443_add_sub(unsigned int base,
return 0;
 }
 
-static int __init s3c2443_irq_add(struct sys_device *sysdev)
+static int s3c2443_irq_add(struct sys_device *sysdev)
 {
printk(S3C2443: IRQ Support\n);
 
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH V2 0/2] ARM: S3C2410: Remove section mismatch warning

2011-10-07 Thread Tushar Behera
While compiling with s3c2410_defconfig, we receive 4 section mismatch
warnings.

Patch 1 addresses a couple of warnings related to usb_simtec_init().
Patch 2 addresses mismatch warnings in a couple of mach-*/irq.c files.

Changes for V2:
* The original patch is splitted to address to different logical
solutions.
* __initdata attribute is removed from struct sysdev_driver
structures and __init attribute is removed from all referenced
function definitions.

The patches are rebased on v3.1-rc8. They are only build tested,
they are not boot tested on any hardware.


Tushar Behera (2):
  ARM: S3C2410: Add __init attribute to usb_simtec_init()
  ARM: S3C2410: irq: Remove __init attributes to fix compilation
warning

 arch/arm/mach-s3c2410/usb-simtec.c |2 +-
 arch/arm/mach-s3c2410/usb-simtec.h |2 +-
 arch/arm/mach-s3c2416/irq.c|4 ++--
 arch/arm/mach-s3c2443/irq.c|4 ++--
 4 files changed, 6 insertions(+), 6 deletions(-)

-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH] ARM: S3C2410: Remove section mismatch warning

2011-10-03 Thread Tushar Behera
Some of the functions and structures did not have _init or __initdata
attributes, even though they were referenced from functions / structures
with those attribute, resulting in section mismatches.

Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---

The patch is rebased on v3.1-rc8.

The patch has only been build tested, they are not boot tested on any
hardware.

 arch/arm/mach-s3c2410/usb-simtec.c |2 +-
 arch/arm/mach-s3c2410/usb-simtec.h |2 +-
 arch/arm/mach-s3c2416/irq.c|2 +-
 arch/arm/mach-s3c2440/clock.c  |4 ++--
 arch/arm/mach-s3c2443/irq.c|2 +-
 5 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/arch/arm/mach-s3c2410/usb-simtec.c 
b/arch/arm/mach-s3c2410/usb-simtec.c
index 29bd3d9..3a1028c 100644
--- a/arch/arm/mach-s3c2410/usb-simtec.c
+++ b/arch/arm/mach-s3c2410/usb-simtec.c
@@ -104,7 +104,7 @@ static struct s3c2410_hcd_info usb_simtec_info __initdata = 
{
 };
 
 
-int usb_simtec_init(void)
+int __init usb_simtec_init(void)
 {
int ret;
 
diff --git a/arch/arm/mach-s3c2410/usb-simtec.h 
b/arch/arm/mach-s3c2410/usb-simtec.h
index 03842ed..43cc88f 100644
--- a/arch/arm/mach-s3c2410/usb-simtec.h
+++ b/arch/arm/mach-s3c2410/usb-simtec.h
@@ -12,5 +12,5 @@
  * published by the Free Software Foundation.
 */
 
-extern int usb_simtec_init(void);
+extern int __init usb_simtec_init(void);
 
diff --git a/arch/arm/mach-s3c2416/irq.c b/arch/arm/mach-s3c2416/irq.c
index 28ad20d..153cb2f 100644
--- a/arch/arm/mach-s3c2416/irq.c
+++ b/arch/arm/mach-s3c2416/irq.c
@@ -234,7 +234,7 @@ static int __init s3c2416_irq_add(struct sys_device *sysdev)
return 0;
 }
 
-static struct sysdev_driver s3c2416_irq_driver = {
+static struct sysdev_driver s3c2416_irq_driver __initdata = {
.add= s3c2416_irq_add,
 };
 
diff --git a/arch/arm/mach-s3c2440/clock.c b/arch/arm/mach-s3c2440/clock.c
index f9e6bda..3b3bec5 100644
--- a/arch/arm/mach-s3c2440/clock.c
+++ b/arch/arm/mach-s3c2440/clock.c
@@ -108,7 +108,7 @@ static struct clk s3c2440_clk_ac97 = {
.ctrlbit= S3C2440_CLKCON_CAMERA,
 };
 
-static int s3c2440_clk_add(struct sys_device *sysdev)
+static int __init s3c2440_clk_add(struct sys_device *sysdev)
 {
struct clk *clock_upll;
struct clk *clock_h;
@@ -137,7 +137,7 @@ static int s3c2440_clk_add(struct sys_device *sysdev)
return 0;
 }
 
-static struct sysdev_driver s3c2440_clk_driver = {
+static struct sysdev_driver s3c2440_clk_driver __initdata = {
.add= s3c2440_clk_add,
 };
 
diff --git a/arch/arm/mach-s3c2443/irq.c b/arch/arm/mach-s3c2443/irq.c
index 83ecb11..1d483d9 100644
--- a/arch/arm/mach-s3c2443/irq.c
+++ b/arch/arm/mach-s3c2443/irq.c
@@ -265,7 +265,7 @@ static int __init s3c2443_irq_add(struct sys_device *sysdev)
return 0;
 }
 
-static struct sysdev_driver s3c2443_irq_driver = {
+static struct sysdev_driver s3c2443_irq_driver __initdata = {
.add= s3c2443_irq_add,
 };
 
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH] ARM: S3C2410: Remove section mismatch warning

2011-10-03 Thread Tushar Behera

Hi Russell,

On Monday 03 October 2011 03:29 PM, Russell King - ARM Linux wrote:

On Mon, Oct 03, 2011 at 03:10:41PM +0530, Tushar Behera wrote:

Some of the functions and structures did not have _init or __initdata
attributes, even though they were referenced from functions / structures
with those attribute, resulting in section mismatches.


Firstly - it's a good idea to include the warnings which you're fixing
in the commit log text, so that people know exactly what is being fixed.


Thanks for your review.

Sure, I will add it in next revision.

[ snip ]


diff --git a/arch/arm/mach-s3c2416/irq.c b/arch/arm/mach-s3c2416/irq.c
index 28ad20d..153cb2f 100644
--- a/arch/arm/mach-s3c2416/irq.c
+++ b/arch/arm/mach-s3c2416/irq.c
@@ -234,7 +234,7 @@ static int __init s3c2416_irq_add(struct sys_device *sysdev)
return 0;
  }

-static struct sysdev_driver s3c2416_irq_driver = {
+static struct sysdev_driver s3c2416_irq_driver __initdata = {
.add= s3c2416_irq_add,
  };



I remain entirely unconvinced that this is correct.  As a result of
the sysdev_driver_register(s3c2416_sysclass,s3c2416_irq_driver);
call, this structure is placed on a list.

If this structure is marked __initdata, then the memory behind the
structure will be freed and overwritten - however, it's still on a
list which might be walked.  Such a walk would cause a kernel oops
or might even be an exploitable security hole if that page ends up
in userspace - especially as said structure contains function calls
which would be called in privileged mode.



The function s3c2416_irq_add() is defined with __init attribute. Also a 
cascade of functions called from s3c2416_irq_add() are also defined with 
__init attribute.


Would it be a good idea to remove __init attribute of all these 
functions (there are 2 of them) called from s3c2416_irq_add() instead?



The same comment applies to the other sysdev driver structures you're
marking __initdata too.


--
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH] gpio/samsung: Move SoC specific codes within macro

2011-10-02 Thread Tushar Behera
In drivers/gpio/gpio-samsung.c, there are certain structures
and functions which are not getting used if the particular
CPU is not selected. These code segments are moved under CPU
specific macros to remove compilation warnings.

Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
It has been build tested with s3c2410_defconfig, s3c6400_defconfig,
s5p64x0_defconfig, s5pc100_defconfig, s5pv210_defconfig and
exynos4_defconfig.

The patch has been rebased onto
Commit: gpio/samsung: correct pin configuration for S5PC100/S5PC110/EXYNOS4

on git://github.com/kgene/linux-samsung.git (next/topic-gpio-samsung)

 drivers/gpio/gpio-samsung.c |   10 ++
 1 files changed, 10 insertions(+), 0 deletions(-)

diff --git a/drivers/gpio/gpio-samsung.c b/drivers/gpio/gpio-samsung.c
index b6be77a..33d62d1 100644
--- a/drivers/gpio/gpio-samsung.c
+++ b/drivers/gpio/gpio-samsung.c
@@ -318,6 +318,7 @@ static unsigned samsung_gpio_getcfg_4bit(struct 
samsung_gpio_chip *chip,
return S3C_GPIO_SPECIAL(con);
 }
 
+#ifdef CONFIG_PLAT_S3C24XX
 /*
  * s3c24xx_gpio_setcfg_abank - S3C24XX style GPIO configuration (Bank A)
  * @chip: The gpio chip that is being configured.
@@ -379,7 +380,9 @@ static unsigned s3c24xx_gpio_getcfg_abank(struct 
samsung_gpio_chip *chip,
 
return S3C_GPIO_SFN(con);
 }
+#endif
 
+#if defined(CONFIG_CPU_S5P6440) || defined(CONFIG_CPU_S5P6450)
 static int s5p64x0_gpio_setcfg_rbank(struct samsung_gpio_chip *chip,
 unsigned int off, unsigned int cfg)
 {
@@ -417,6 +420,7 @@ static int s5p64x0_gpio_setcfg_rbank(struct 
samsung_gpio_chip *chip,
 
return 0;
 }
+#endif
 
 static void __init samsung_gpiolib_set_cfg(struct samsung_gpio_cfg *chipcfg,
   int nr_chips)
@@ -438,10 +442,12 @@ struct samsung_gpio_cfg s3c24xx_gpiocfg_default = {
.get_config = samsung_gpio_getcfg_2bit,
 };
 
+#ifdef CONFIG_PLAT_S3C24XX
 static struct samsung_gpio_cfg s3c24xx_gpiocfg_banka = {
.set_config = s3c24xx_gpio_setcfg_abank,
.get_config = s3c24xx_gpio_getcfg_abank,
 };
+#endif
 
 static struct samsung_gpio_cfg exynos4_gpio_cfg = {
.set_pull   = exynos4_gpio_setpull,
@@ -450,6 +456,7 @@ static struct samsung_gpio_cfg exynos4_gpio_cfg = {
.get_config = samsung_gpio_getcfg_4bit,
 };
 
+#if defined(CONFIG_CPU_S5P6440) || defined(CONFIG_CPU_S5P6450)
 static struct samsung_gpio_cfg s5p64x0_gpio_cfg_rbank = {
.cfg_eint   = 0x3,
.set_config = s5p64x0_gpio_setcfg_rbank,
@@ -457,6 +464,7 @@ static struct samsung_gpio_cfg s5p64x0_gpio_cfg_rbank = {
.set_pull   = samsung_gpio_setpull_updown,
.get_pull   = samsung_gpio_getpull_updown,
 };
+#endif
 
 static struct samsung_gpio_cfg samsung_gpio_cfgs[] = {
{
@@ -682,6 +690,7 @@ static int samsung_gpiolib_4bit2_output(struct gpio_chip 
*chip,
return 0;
 }
 
+#ifdef CONFIG_PLAT_S3C24XX
 /* The next set of routines are for the case of s3c24xx bank a */
 
 static int s3c24xx_gpiolib_banka_input(struct gpio_chip *chip, unsigned offset)
@@ -717,6 +726,7 @@ static int s3c24xx_gpiolib_banka_output(struct gpio_chip 
*chip,
local_irq_restore(flags);
return 0;
 }
+#endif
 
 /* The next set of routines are for the case of s5p64x0 bank r */
 
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH V4] ARM: EXYNOS4: Add machine support for 7 LCD on ORIGEN

2011-09-15 Thread Tushar Behera
ORIGEN board is fitted with 7 LCD panel HV070WSA. The pixel
resolution of the LCD panel is 1024x600.

Also power domain device for LCD0 is registered.

Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
Changes for V4:
* Added gpio_free() call
* Removed .refresh value as it was same as default value.
Changes for V3:
* Added error check for gpio request in LCD power function
Changes for V2:
* Added power domain device registration for LCD0

 arch/arm/mach-exynos4/Kconfig   |3 ++
 arch/arm/mach-exynos4/mach-origen.c |   60 +++
 2 files changed, 63 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-exynos4/Kconfig b/arch/arm/mach-exynos4/Kconfig
index 48f18f7..4f28871 100644
--- a/arch/arm/mach-exynos4/Kconfig
+++ b/arch/arm/mach-exynos4/Kconfig
@@ -222,6 +222,9 @@ config MACH_ORIGEN
select S3C_DEV_RTC
select S3C_DEV_WDT
select S3C_DEV_HSMMC2
+   select S5P_DEV_FIMD0
+   select EXYNOS4_DEV_PD
+   select EXYNOS4_SETUP_FIMD0
select EXYNOS4_SETUP_SDHCI
help
  Machine support for ORIGEN based on Samsung EXYNOS4210
diff --git a/arch/arm/mach-exynos4/mach-origen.c 
b/arch/arm/mach-exynos4/mach-origen.c
index ed59f86..0833fcb 100644
--- a/arch/arm/mach-exynos4/mach-origen.c
+++ b/arch/arm/mach-exynos4/mach-origen.c
@@ -14,16 +14,22 @@
 #include linux/platform_device.h
 #include linux/io.h
 #include linux/input.h
+#include linux/lcd.h
+
+#include video/platform_lcd.h
 
 #include asm/mach/arch.h
 #include asm/mach-types.h
 
 #include plat/regs-serial.h
+#include plat/regs-fb-v4.h
 #include plat/exynos4.h
 #include plat/cpu.h
 #include plat/devs.h
 #include plat/sdhci.h
 #include plat/iic.h
+#include plat/pd.h
+#include plat/fb.h
 
 #include mach/map.h
 
@@ -79,10 +85,62 @@ static struct s3c_sdhci_platdata origen_hsmmc2_pdata 
__initdata = {
.clk_type   = S3C_SDHCI_CLK_DIV_EXTERNAL,
 };
 
+static void lcd_hv070wsa_set_power(struct plat_lcd_data *pd, unsigned int 
power)
+{
+   int ret;
+
+   if (power)
+   ret = gpio_request_one(EXYNOS4_GPE3(4),
+   GPIOF_OUT_INIT_HIGH, GPE3_4);
+   else
+   ret = gpio_request_one(EXYNOS4_GPE3(4),
+   GPIOF_OUT_INIT_LOW, GPE3_4);
+
+   gpio_free(EXYNOS4_GPE3(4));
+
+   if (ret)
+   pr_err(failed to request gpio for LCD power: %d\n, ret);
+}
+
+static struct plat_lcd_data origen_lcd_hv070wsa_data = {
+   .set_power = lcd_hv070wsa_set_power,
+};
+
+static struct platform_device origen_lcd_hv070wsa = {
+   .name   = platform-lcd,
+   .dev.parent = s5p_device_fimd0.dev,
+   .dev.platform_data  = origen_lcd_hv070wsa_data,
+};
+
+static struct s3c_fb_pd_win origen_fb_win0 = {
+   .win_mode = {
+   .left_margin= 64,
+   .right_margin   = 16,
+   .upper_margin   = 64,
+   .lower_margin   = 16,
+   .hsync_len  = 48,
+   .vsync_len  = 3,
+   .xres   = 1024,
+   .yres   = 600,
+   },
+   .max_bpp= 32,
+   .default_bpp= 24,
+};
+
+static struct s3c_fb_platdata origen_lcd_pdata __initdata = {
+   .win[0] = origen_fb_win0,
+   .vidcon0= VIDCON0_VIDOUT_RGB | VIDCON0_PNRMODE_RGB,
+   .vidcon1= VIDCON1_INV_HSYNC | VIDCON1_INV_VSYNC,
+   .setup_gpio = exynos4_fimd0_gpio_setup_24bpp,
+};
+
 static struct platform_device *origen_devices[] __initdata = {
+   exynos4_device_pd[PD_LCD0],
s3c_device_hsmmc2,
s3c_device_rtc,
s3c_device_wdt,
+   s5p_device_fimd0,
+   origen_lcd_hv070wsa,
 };
 
 static void __init origen_map_io(void)
@@ -95,7 +153,9 @@ static void __init origen_map_io(void)
 static void __init origen_machine_init(void)
 {
s3c_sdhci2_set_platdata(origen_hsmmc2_pdata);
+   s5p_fimd0_set_platdata(origen_lcd_pdata);
platform_add_devices(origen_devices, ARRAY_SIZE(origen_devices));
+   s5p_device_fimd0.dev.parent = exynos4_device_pd[PD_LCD0].dev;
 }
 
 MACHINE_START(ORIGEN, ORIGEN)
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH V4] ARM: EXYNOS4: Add machine support for 7 LCD on ORIGEN

2011-09-15 Thread Tushar Behera

Dear Wolfgang Denk,

On Thursday 15 September 2011 02:44 PM, Wolfgang Denk wrote:

Dear Tushar Behera,

In message1316076867-2138-1-git-send-email-tushar.beh...@linaro.org  you 
wrote:

ORIGEN board is fitted with 7 LCD panel HV070WSA. The pixel
resolution of the LCD panel is 1024x600.

...

+static struct s3c_fb_pd_win origen_fb_win0 = {
+   .win_mode = {
+   .left_margin= 64,
+   .right_margin   = 16,
+   .upper_margin   = 64,
+   .lower_margin   = 16,
+   .hsync_len  = 48,
+   .vsync_len  = 3,
+   .xres   = 1024,
+   .yres   = 600,
+   },
+   .max_bpp= 32,
+   .default_bpp= 24,
+};


Does it still make sense to hard-code such parameters?

In PowerPC-land we pass display mode information in the device tree
using a verbatim EDID block.

Would it be not better (and way more flexible) to do the same here,
now that ARM has device tree support?


Thanks for your suggestions.

Currently work for enabling device tree support for EXYNOS4 based 
machine is going on. Once it is done, we should be able to pass this 
information through device tree blob.


For non-DT machines, IMHO, we have to follow the current approach.

Best regards,

Wolfgang Denk




--
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH V2] ARM: EXYNOS4: Add machine support for 7 LCD on ORIGEN

2011-09-14 Thread Tushar Behera
ORIGEN board is fitted with 7 LCD panel HV070WSA. The pixel
resolution of the LCD panel is 1024x600.

Also power domain device for LCD0 is registered.

Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
Changes for V2:
* Added power domain device registration for LCD0

The patch is rebased on [1]. For proper working of LCD on ORIGEN,
following patches are needed. These patches are already submitted to
the mailing list.

a. ARM: EXYNOS4: Add PWM backlight support on Origen
Author: Giridhar Maruthy
b. ARM: EXYNOS4: Configure MAX8997 PMIC for Origen
Author: Inderpal Singh

[1] git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git 
(for-next)

 arch/arm/mach-exynos4/Kconfig   |3 ++
 arch/arm/mach-exynos4/mach-origen.c |   53 +++
 2 files changed, 56 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-exynos4/Kconfig b/arch/arm/mach-exynos4/Kconfig
index 48f18f7..4f28871 100644
--- a/arch/arm/mach-exynos4/Kconfig
+++ b/arch/arm/mach-exynos4/Kconfig
@@ -222,6 +222,9 @@ config MACH_ORIGEN
select S3C_DEV_RTC
select S3C_DEV_WDT
select S3C_DEV_HSMMC2
+   select S5P_DEV_FIMD0
+   select EXYNOS4_DEV_PD
+   select EXYNOS4_SETUP_FIMD0
select EXYNOS4_SETUP_SDHCI
help
  Machine support for ORIGEN based on Samsung EXYNOS4210
diff --git a/arch/arm/mach-exynos4/mach-origen.c 
b/arch/arm/mach-exynos4/mach-origen.c
index ed59f86..ed04f63 100644
--- a/arch/arm/mach-exynos4/mach-origen.c
+++ b/arch/arm/mach-exynos4/mach-origen.c
@@ -14,16 +14,22 @@
 #include linux/platform_device.h
 #include linux/io.h
 #include linux/input.h
+#include linux/lcd.h
+
+#include video/platform_lcd.h
 
 #include asm/mach/arch.h
 #include asm/mach-types.h
 
 #include plat/regs-serial.h
+#include plat/regs-fb-v4.h
 #include plat/exynos4.h
 #include plat/cpu.h
 #include plat/devs.h
 #include plat/sdhci.h
 #include plat/iic.h
+#include plat/pd.h
+#include plat/fb.h
 
 #include mach/map.h
 
@@ -79,10 +85,55 @@ static struct s3c_sdhci_platdata origen_hsmmc2_pdata 
__initdata = {
.clk_type   = S3C_SDHCI_CLK_DIV_EXTERNAL,
 };
 
+static void lcd_hv070wsa_set_power(struct plat_lcd_data *pd, unsigned int 
power)
+{
+   int gpio = EXYNOS4_GPE3(4);
+
+   gpio_request(gpio, GPE3_4);
+   gpio_direction_output(gpio, power);
+   gpio_free(gpio);
+}
+
+static struct plat_lcd_data origen_lcd_hv070wsa_data = {
+   .set_power = lcd_hv070wsa_set_power,
+};
+
+static struct platform_device origen_lcd_hv070wsa = {
+   .name   = platform-lcd,
+   .dev.parent = s5p_device_fimd0.dev,
+   .dev.platform_data  = origen_lcd_hv070wsa_data,
+};
+
+static struct s3c_fb_pd_win origen_fb_win0 = {
+   .win_mode = {
+   .left_margin= 64,
+   .right_margin   = 16,
+   .upper_margin   = 64,
+   .lower_margin   = 16,
+   .hsync_len  = 48,
+   .vsync_len  = 3,
+   .xres   = 1024,
+   .yres   = 600,
+   .refresh= 60,
+   },
+   .max_bpp= 32,
+   .default_bpp= 24,
+};
+
+static struct s3c_fb_platdata origen_lcd_pdata __initdata = {
+   .win[0] = origen_fb_win0,
+   .vidcon0= VIDCON0_VIDOUT_RGB | VIDCON0_PNRMODE_RGB,
+   .vidcon1= VIDCON1_INV_HSYNC | VIDCON1_INV_VSYNC,
+   .setup_gpio = exynos4_fimd0_gpio_setup_24bpp,
+};
+
 static struct platform_device *origen_devices[] __initdata = {
+   exynos4_device_pd[PD_LCD0],
s3c_device_hsmmc2,
s3c_device_rtc,
s3c_device_wdt,
+   s5p_device_fimd0,
+   origen_lcd_hv070wsa,
 };
 
 static void __init origen_map_io(void)
@@ -95,7 +146,9 @@ static void __init origen_map_io(void)
 static void __init origen_machine_init(void)
 {
s3c_sdhci2_set_platdata(origen_hsmmc2_pdata);
+   s5p_fimd0_set_platdata(origen_lcd_pdata);
platform_add_devices(origen_devices, ARRAY_SIZE(origen_devices));
+   s5p_device_fimd0.dev.parent = exynos4_device_pd[PD_LCD0].dev;
 }
 
 MACHINE_START(ORIGEN, ORIGEN)
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH V2] ARM: EXYNOS4: Add machine support for 7 LCD on ORIGEN

2011-09-14 Thread Tushar Behera

Hi Fabio,

On Wednesday 14 September 2011 05:06 PM, Fabio Estevam wrote:

On Wed, Sep 14, 2011 at 8:01 AM, Tushar Beheratushar.beh...@linaro.org  wrote:
...

+static void lcd_hv070wsa_set_power(struct plat_lcd_data *pd, unsigned int 
power)
+{
+   int gpio = EXYNOS4_GPE3(4);
+
+   gpio_request(gpio, GPE3_4);
+   gpio_direction_output(gpio, power);


You should check for returned errors for these functions.


Ok.

Will this be better?

static void lcd_hv070wsa_set_power(struct plat_lcd_data *pd, \
unsigned int power)
{
int ret;
unsigned long flag = power ? GPIOF_OUT_INIT_HIGH : \
GPIOF_OUT_INIT_LOW;

ret = gpio_request_one(EXYNOS4_GPE3(4), flag, GPE3_4);

if (ret)
printk(KERN_ERR Could not request gpio for LCD power);
}


Regards,

Fabio Estevam



--
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH V2] ARM: EXYNOS4: Add machine support for 7 LCD on ORIGEN

2011-09-14 Thread Tushar Behera

Hi Fabio Estevam,

On Wednesday 14 September 2011 06:09 PM, Fabio Estevam wrote:

On Wed, Sep 14, 2011 at 9:07 AM, Tushar Beheratushar.beh...@linaro.org  wrote:
...


Will this be better?

static void lcd_hv070wsa_set_power(struct plat_lcd_data *pd, \
unsigned int power)
{
int ret;
unsigned long flag = power ? GPIOF_OUT_INIT_HIGH : \
GPIOF_OUT_INIT_LOW;

ret = gpio_request_one(EXYNOS4_GPE3(4), flag, GPE3_4);

if (ret)
printk(KERN_ERR Could not request gpio for LCD power);
}


Looks better. You can use pr_err instead of printk(KERN_ERR .

Thanks for your review comments. I will send the next version with these 
changes.

Regards,

Fabio Estevam



--
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH V3] ARM: EXYNOS4: Add machine support for 7 LCD on ORIGEN

2011-09-14 Thread Tushar Behera
ORIGEN board is fitted with 7 LCD panel HV070WSA. The pixel
resolution of the LCD panel is 1024x600.

Also power domain device for LCD0 is registered.

Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
Changes for V3:
* Added error check for gpio request in LCD power function
Changes for V2:
* Added power domain device registration for LCD0

The patch is rebased on [1]. For proper working of LCD on ORIGEN,
following patches are needed. These patches are already submitted to
the mailing list.

a. ARM: EXYNOS4: Add PWM backlight support on Origen
Author: Giridhar Maruthy
b. ARM: EXYNOS4: Configure MAX8997 PMIC for Origen
Author: Inderpal Singh

[1] git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git 
(for-next)

 arch/arm/mach-exynos4/Kconfig   |3 ++
 arch/arm/mach-exynos4/mach-origen.c |   54 +++
 2 files changed, 57 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-exynos4/Kconfig b/arch/arm/mach-exynos4/Kconfig
index 48f18f7..4f28871 100644
--- a/arch/arm/mach-exynos4/Kconfig
+++ b/arch/arm/mach-exynos4/Kconfig
@@ -222,6 +222,9 @@ config MACH_ORIGEN
select S3C_DEV_RTC
select S3C_DEV_WDT
select S3C_DEV_HSMMC2
+   select S5P_DEV_FIMD0
+   select EXYNOS4_DEV_PD
+   select EXYNOS4_SETUP_FIMD0
select EXYNOS4_SETUP_SDHCI
help
  Machine support for ORIGEN based on Samsung EXYNOS4210
diff --git a/arch/arm/mach-exynos4/mach-origen.c 
b/arch/arm/mach-exynos4/mach-origen.c
index ed59f86..0dcd527 100644
--- a/arch/arm/mach-exynos4/mach-origen.c
+++ b/arch/arm/mach-exynos4/mach-origen.c
@@ -14,16 +14,22 @@
 #include linux/platform_device.h
 #include linux/io.h
 #include linux/input.h
+#include linux/lcd.h
+
+#include video/platform_lcd.h
 
 #include asm/mach/arch.h
 #include asm/mach-types.h
 
 #include plat/regs-serial.h
+#include plat/regs-fb-v4.h
 #include plat/exynos4.h
 #include plat/cpu.h
 #include plat/devs.h
 #include plat/sdhci.h
 #include plat/iic.h
+#include plat/pd.h
+#include plat/fb.h
 
 #include mach/map.h
 
@@ -79,10 +85,56 @@ static struct s3c_sdhci_platdata origen_hsmmc2_pdata 
__initdata = {
.clk_type   = S3C_SDHCI_CLK_DIV_EXTERNAL,
 };
 
+static void lcd_hv070wsa_set_power(struct plat_lcd_data *pd, unsigned int 
power)
+{
+   int ret;
+   unsigned long flag = power ? GPIOF_OUT_INIT_HIGH : GPIOF_OUT_INIT_LOW;
+
+   ret = gpio_request_one(EXYNOS4_GPE3(4), flag, GPE3_4);
+   if (ret)
+   pr_err(Could not request gpio line for LCD power.\n);
+}
+
+static struct plat_lcd_data origen_lcd_hv070wsa_data = {
+   .set_power = lcd_hv070wsa_set_power,
+};
+
+static struct platform_device origen_lcd_hv070wsa = {
+   .name   = platform-lcd,
+   .dev.parent = s5p_device_fimd0.dev,
+   .dev.platform_data  = origen_lcd_hv070wsa_data,
+};
+
+static struct s3c_fb_pd_win origen_fb_win0 = {
+   .win_mode = {
+   .left_margin= 64,
+   .right_margin   = 16,
+   .upper_margin   = 64,
+   .lower_margin   = 16,
+   .hsync_len  = 48,
+   .vsync_len  = 3,
+   .xres   = 1024,
+   .yres   = 600,
+   .refresh= 60,
+   },
+   .max_bpp= 32,
+   .default_bpp= 24,
+};
+
+static struct s3c_fb_platdata origen_lcd_pdata __initdata = {
+   .win[0] = origen_fb_win0,
+   .vidcon0= VIDCON0_VIDOUT_RGB | VIDCON0_PNRMODE_RGB,
+   .vidcon1= VIDCON1_INV_HSYNC | VIDCON1_INV_VSYNC,
+   .setup_gpio = exynos4_fimd0_gpio_setup_24bpp,
+};
+
 static struct platform_device *origen_devices[] __initdata = {
+   exynos4_device_pd[PD_LCD0],
s3c_device_hsmmc2,
s3c_device_rtc,
s3c_device_wdt,
+   s5p_device_fimd0,
+   origen_lcd_hv070wsa,
 };
 
 static void __init origen_map_io(void)
@@ -95,7 +147,9 @@ static void __init origen_map_io(void)
 static void __init origen_machine_init(void)
 {
s3c_sdhci2_set_platdata(origen_hsmmc2_pdata);
+   s5p_fimd0_set_platdata(origen_lcd_pdata);
platform_add_devices(origen_devices, ARRAY_SIZE(origen_devices));
+   s5p_device_fimd0.dev.parent = exynos4_device_pd[PD_LCD0].dev;
 }
 
 MACHINE_START(ORIGEN, ORIGEN)
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH V2] ARM: EXYNOS4: Add machine support for 7 LCD on ORIGEN

2011-09-14 Thread Tushar Behera

Hi Kukjin,

On Thursday 15 September 2011 10:24 AM, Kukjin Kim wrote:

Tushar Behera wrote:


Hi Fabio,

On Wednesday 14 September 2011 05:06 PM, Fabio Estevam wrote:

On Wed, Sep 14, 2011 at 8:01 AM, Tushar Beheratushar.beh...@linaro.org

wrote:

...

+static void lcd_hv070wsa_set_power(struct plat_lcd_data *pd, unsigned int

power)

+{
+   int gpio = EXYNOS4_GPE3(4);
+
+   gpio_request(gpio, GPE3_4);
+   gpio_direction_output(gpio, power);


You should check for returned errors for these functions.


Ok.

Will this be better?

static void lcd_hv070wsa_set_power(struct plat_lcd_data *pd, \


No need '\'


unsigned int power)
{
int ret;
unsigned long flag = power ? GPIOF_OUT_INIT_HIGH : \


Same as above.


GPIOF_OUT_INIT_LOW;

ret = gpio_request_one(EXYNOS4_GPE3(4), flag, GPE3_4);

if (ret)
printk(KERN_ERR Could not request gpio for LCD power);
}


How about following?

if (power)
ret = gpio_request_one(EXYNOS4_GPE3(4), GPIOF_OUT_INIT_HIGH, 
GPE3_4);
else
ret = gpio_request_one(EXYNOS4_GPE3(4), GPIOF_OUT_INIT_LOW, 
GPE3_4);

if (ret)
pr_err(failed to request gpio for LCD power: %d\n, ret);

Thanks, I will update accordingly.

Also I will remove the .refresh value as suggested in the other thread.


Thanks.

Best regards,
Kgene.
--
Kukjin Kimkgene@samsung.com, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.




--
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH] ARM: EXYNOS4: Setup consistent dma size at boot time

2011-09-12 Thread Tushar Behera
Some of the boards under mach-exynos4 initialize frame-buffers
for which the memory requirement is more than 2MB (Nuri board requires
around 4MB, Origen requires around 2.6MB), hence the default dma pool
allocation size of 2MB is not sufficient. The consistent dma size is
hence increased to successfully allocate memory for those boards.

Depends on ARM: Add init_consistent_dma_size()
by Jon Medhurst (99d1717dd7fecf2b10195b0d864323b952b4eba0).

CC: Jon Medhurst t...@yxit.co.uk
Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
 arch/arm/mach-exynos4/cpu.c |2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-exynos4/cpu.c b/arch/arm/mach-exynos4/cpu.c
index 2d8a40c..45d8bfa 100644
--- a/arch/arm/mach-exynos4/cpu.c
+++ b/arch/arm/mach-exynos4/cpu.c
@@ -10,6 +10,7 @@
 
 #include linux/sched.h
 #include linux/sysdev.h
+#include linux/dma-mapping.h
 
 #include asm/mach/map.h
 #include asm/mach/irq.h
@@ -136,6 +137,7 @@ static void exynos4_idle(void)
 void __init exynos4_map_io(void)
 {
iotable_init(exynos4_iodesc, ARRAY_SIZE(exynos4_iodesc));
+   init_consistent_dma_size(SZ_8M);
 
/* initialize device information early */
exynos4_default_sdhci0();
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH] ARM: EXYNOS4: convert boot_params to atag_offset

2011-09-11 Thread Tushar Behera
Based on ARM: introduce atag_offset to replace boot_params
by Nicolas Pitre (2bb9839e312ed55a6d5824ffa6077ce3d7d63b1e).

Since boot_params variable is deleted from machine_desc, the variable
is modified in the newer board files.

CC: Nicolas Pitre nicolas.pi...@linaro.org
Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
This is required when Kukjin's for-next branch is merged with
Russell's devel-stable branch.

Nicolas's patchset already has changes for other EXYNSO4 based boards.

 arch/arm/mach-exynos4/mach-origen.c   |2 +-
 arch/arm/mach-exynos4/mach-smdk4212.c |2 +-
 arch/arm/mach-exynos4/mach-smdkv310.c |2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm/mach-exynos4/mach-origen.c 
b/arch/arm/mach-exynos4/mach-origen.c
index ed59f86..b5f6f38 100644
--- a/arch/arm/mach-exynos4/mach-origen.c
+++ b/arch/arm/mach-exynos4/mach-origen.c
@@ -100,7 +100,7 @@ static void __init origen_machine_init(void)
 
 MACHINE_START(ORIGEN, ORIGEN)
/* Maintainer: JeongHyeon Kim jh...@insignal.co.kr */
-   .boot_params= S5P_PA_SDRAM + 0x100,
+   .atag_offset= 0x100,
.init_irq   = exynos4_init_irq,
.map_io = origen_map_io,
.init_machine   = origen_machine_init,
diff --git a/arch/arm/mach-exynos4/mach-smdk4212.c 
b/arch/arm/mach-exynos4/mach-smdk4212.c
index 3479a93..8c41ae1 100644
--- a/arch/arm/mach-exynos4/mach-smdk4212.c
+++ b/arch/arm/mach-exynos4/mach-smdk4212.c
@@ -284,7 +284,7 @@ static void __init smdk4212_machine_init(void)
 
 MACHINE_START(SMDK4212, SMDK4212)
/* Maintainer: Kukjin Kim kgene@samsung.com */
-   .boot_params= S5P_PA_SDRAM + 0x100,
+   .atag_offset= 0x100,
.init_irq   = exynos4_init_irq,
.map_io = smdk4212_map_io,
.init_machine   = smdk4212_machine_init,
diff --git a/arch/arm/mach-exynos4/mach-smdkv310.c 
b/arch/arm/mach-exynos4/mach-smdkv310.c
index 57cf632..7ce4d8b 100644
--- a/arch/arm/mach-exynos4/mach-smdkv310.c
+++ b/arch/arm/mach-exynos4/mach-smdkv310.c
@@ -344,7 +344,7 @@ MACHINE_END
 
 MACHINE_START(SMDKC210, SMDKC210)
/* Maintainer: Kukjin Kim kgene@samsung.com */
-   .boot_params= S5P_PA_SDRAM + 0x100,
+   .atag_offset= 0x100,
.init_irq   = exynos4_init_irq,
.map_io = smdkv310_map_io,
.init_machine   = smdkv310_machine_init,
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH] ARM: EXYNOS4: Add machine support for 7 LCD on ORIGEN

2011-09-07 Thread Tushar Behera
ORIGEN board is fitted with 7 LCD panel HV070WSA. The pixel
resolution of the LCD panel is 1024x600.

Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
The patch is rebased on [1]. But for proper working of LCD on ORIGEN,
following patches are needed.

a. ARM: EXYNOS4: Add PWM backlight support on Origen
http://www.spinics.net/lists/linux-samsung-soc/msg06309.html

b. ARM: EXYNOS4: Configure MAX8997 PMIC for Origen
http://www.spinics.net/lists/linux-samsung-soc/msg06282.html

[1] git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git

 arch/arm/mach-exynos4/Kconfig   |2 +
 arch/arm/mach-exynos4/mach-origen.c |   50 +++
 2 files changed, 52 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-exynos4/Kconfig b/arch/arm/mach-exynos4/Kconfig
index 48f18f7..8656f49 100644
--- a/arch/arm/mach-exynos4/Kconfig
+++ b/arch/arm/mach-exynos4/Kconfig
@@ -222,6 +222,8 @@ config MACH_ORIGEN
select S3C_DEV_RTC
select S3C_DEV_WDT
select S3C_DEV_HSMMC2
+   select S5P_DEV_FIMD0
+   select EXYNOS4_SETUP_FIMD0
select EXYNOS4_SETUP_SDHCI
help
  Machine support for ORIGEN based on Samsung EXYNOS4210
diff --git a/arch/arm/mach-exynos4/mach-origen.c 
b/arch/arm/mach-exynos4/mach-origen.c
index ed59f86..f2cea78 100644
--- a/arch/arm/mach-exynos4/mach-origen.c
+++ b/arch/arm/mach-exynos4/mach-origen.c
@@ -14,16 +14,21 @@
 #include linux/platform_device.h
 #include linux/io.h
 #include linux/input.h
+#include linux/lcd.h
+
+#include video/platform_lcd.h
 
 #include asm/mach/arch.h
 #include asm/mach-types.h
 
 #include plat/regs-serial.h
+#include plat/regs-fb-v4.h
 #include plat/exynos4.h
 #include plat/cpu.h
 #include plat/devs.h
 #include plat/sdhci.h
 #include plat/iic.h
+#include plat/fb.h
 
 #include mach/map.h
 
@@ -79,10 +84,54 @@ static struct s3c_sdhci_platdata origen_hsmmc2_pdata 
__initdata = {
.clk_type   = S3C_SDHCI_CLK_DIV_EXTERNAL,
 };
 
+static void lcd_hv070wsa_set_power(struct plat_lcd_data *pd, unsigned int 
power)
+{
+   int gpio = EXYNOS4_GPE3(4);
+
+   gpio_request(gpio, GPE3_4);
+   gpio_direction_output(gpio, power);
+   gpio_free(gpio);
+}
+
+static struct plat_lcd_data origen_lcd_hv070wsa_data = {
+   .set_power = lcd_hv070wsa_set_power,
+};
+
+static struct platform_device origen_lcd_hv070wsa = {
+   .name   = platform-lcd,
+   .dev.parent = s5p_device_fimd0.dev,
+   .dev.platform_data  = origen_lcd_hv070wsa_data,
+};
+
+static struct s3c_fb_pd_win origen_fb_win0 = {
+   .win_mode = {
+   .left_margin= 64,
+   .right_margin   = 16,
+   .upper_margin   = 64,
+   .lower_margin   = 16,
+   .hsync_len  = 48,
+   .vsync_len  = 3,
+   .xres   = 1024,
+   .yres   = 600,
+   .refresh= 60,
+   },
+   .max_bpp= 32,
+   .default_bpp= 24,
+};
+
+static struct s3c_fb_platdata origen_lcd_pdata __initdata = {
+   .win[0] = origen_fb_win0,
+   .vidcon0= VIDCON0_VIDOUT_RGB | VIDCON0_PNRMODE_RGB,
+   .vidcon1= VIDCON1_INV_HSYNC | VIDCON1_INV_VSYNC,
+   .setup_gpio = exynos4_fimd0_gpio_setup_24bpp,
+};
+
 static struct platform_device *origen_devices[] __initdata = {
s3c_device_hsmmc2,
s3c_device_rtc,
s3c_device_wdt,
+   s5p_device_fimd0,
+   origen_lcd_hv070wsa,
 };
 
 static void __init origen_map_io(void)
@@ -95,6 +144,7 @@ static void __init origen_map_io(void)
 static void __init origen_machine_init(void)
 {
s3c_sdhci2_set_platdata(origen_hsmmc2_pdata);
+   s5p_fimd0_set_platdata(origen_lcd_pdata);
platform_add_devices(origen_devices, ARRAY_SIZE(origen_devices));
 }
 
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[RESEND PATCH] ARM: EXYNOS4: Add machine support for 7 LCD on ORIGEN

2011-09-07 Thread Tushar Behera
ORIGEN board is fitted with 7 LCD panel HV070WSA. The pixel
resolution of the LCD panel is 1024x600.

Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
The original patch-mail bounced back from linux-samsung-...@vger.kernel.org,
hence resending the patch again.

The patch is rebased on [1]. For proper working of LCD on ORIGEN,
following patches are needed. These patches are already submitted to
the mailing list.

a. ARM: EXYNOS4: Add PWM backlight support on Origen
Author: Giridhar Maruthy
b. ARM: EXYNOS4: Configure MAX8997 PMIC for Origen
Author: Inderpal Singh

[1] git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git

 arch/arm/mach-exynos4/Kconfig   |2 +
 arch/arm/mach-exynos4/mach-origen.c |   50 +++
 2 files changed, 52 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-exynos4/Kconfig b/arch/arm/mach-exynos4/Kconfig
index 48f18f7..8656f49 100644
--- a/arch/arm/mach-exynos4/Kconfig
+++ b/arch/arm/mach-exynos4/Kconfig
@@ -222,6 +222,8 @@ config MACH_ORIGEN
select S3C_DEV_RTC
select S3C_DEV_WDT
select S3C_DEV_HSMMC2
+   select S5P_DEV_FIMD0
+   select EXYNOS4_SETUP_FIMD0
select EXYNOS4_SETUP_SDHCI
help
  Machine support for ORIGEN based on Samsung EXYNOS4210
diff --git a/arch/arm/mach-exynos4/mach-origen.c 
b/arch/arm/mach-exynos4/mach-origen.c
index ed59f86..f2cea78 100644
--- a/arch/arm/mach-exynos4/mach-origen.c
+++ b/arch/arm/mach-exynos4/mach-origen.c
@@ -14,16 +14,21 @@
 #include linux/platform_device.h
 #include linux/io.h
 #include linux/input.h
+#include linux/lcd.h
+
+#include video/platform_lcd.h
 
 #include asm/mach/arch.h
 #include asm/mach-types.h
 
 #include plat/regs-serial.h
+#include plat/regs-fb-v4.h
 #include plat/exynos4.h
 #include plat/cpu.h
 #include plat/devs.h
 #include plat/sdhci.h
 #include plat/iic.h
+#include plat/fb.h
 
 #include mach/map.h
 
@@ -79,10 +84,54 @@ static struct s3c_sdhci_platdata origen_hsmmc2_pdata 
__initdata = {
.clk_type   = S3C_SDHCI_CLK_DIV_EXTERNAL,
 };
 
+static void lcd_hv070wsa_set_power(struct plat_lcd_data *pd, unsigned int 
power)
+{
+   int gpio = EXYNOS4_GPE3(4);
+
+   gpio_request(gpio, GPE3_4);
+   gpio_direction_output(gpio, power);
+   gpio_free(gpio);
+}
+
+static struct plat_lcd_data origen_lcd_hv070wsa_data = {
+   .set_power = lcd_hv070wsa_set_power,
+};
+
+static struct platform_device origen_lcd_hv070wsa = {
+   .name   = platform-lcd,
+   .dev.parent = s5p_device_fimd0.dev,
+   .dev.platform_data  = origen_lcd_hv070wsa_data,
+};
+
+static struct s3c_fb_pd_win origen_fb_win0 = {
+   .win_mode = {
+   .left_margin= 64,
+   .right_margin   = 16,
+   .upper_margin   = 64,
+   .lower_margin   = 16,
+   .hsync_len  = 48,
+   .vsync_len  = 3,
+   .xres   = 1024,
+   .yres   = 600,
+   .refresh= 60,
+   },
+   .max_bpp= 32,
+   .default_bpp= 24,
+};
+
+static struct s3c_fb_platdata origen_lcd_pdata __initdata = {
+   .win[0] = origen_fb_win0,
+   .vidcon0= VIDCON0_VIDOUT_RGB | VIDCON0_PNRMODE_RGB,
+   .vidcon1= VIDCON1_INV_HSYNC | VIDCON1_INV_VSYNC,
+   .setup_gpio = exynos4_fimd0_gpio_setup_24bpp,
+};
+
 static struct platform_device *origen_devices[] __initdata = {
s3c_device_hsmmc2,
s3c_device_rtc,
s3c_device_wdt,
+   s5p_device_fimd0,
+   origen_lcd_hv070wsa,
 };
 
 static void __init origen_map_io(void)
@@ -95,6 +144,7 @@ static void __init origen_map_io(void)
 static void __init origen_machine_init(void)
 {
s3c_sdhci2_set_platdata(origen_hsmmc2_pdata);
+   s5p_fimd0_set_platdata(origen_lcd_pdata);
platform_add_devices(origen_devices, ARRAY_SIZE(origen_devices));
 }
 
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH] ARM: EXYNOS4: Add HDMI support for Origen

2011-09-07 Thread Tushar Behera

Hi Deepak,

On Wednesday 07 September 2011 10:17 PM, Deepak Saxena wrote:

Will this patch and the patch Tushar just posted apply
cleanly with each other? You are both touching mach-origen.c,
changing origen_devices, and adding support for video output
so it might be good in a future situations such as this to
post as one patch series with both of your sign-offs.


Thanks for your suggestions.

In the current situation, though the patches touch the same files, 
logically they enable unrelated modules. So, would it be good to post 
them as part of a single patch series?


And also, we are not sure which patch would go in first. So we thought 
it best to rebase both of them to the maintainer tree-top.


For these kind of scenarios, I would better host the patches on my tree 
on glo till acceptance from the maintainer.



Thanks,
~Deepak

On 7 September 2011 03:54, Sachin Kamatsachin.ka...@linaro.org  wrote:

This patch adds HDMI (TVout) support for Origen board.

This patch is based on for-next branch of Kukjin Kim's tree:
http://git.kernel.org/?p=linux/kernel/git/kgene/linux-samsung.git;a=summary

This patch is tested using Hatim's patch:
http://www.spinics.net/lists/linux-samsung-soc/msg06367.html

Signed-off-by: Sachin Kamatsachin.ka...@linaro.org
---
  arch/arm/mach-exynos4/Kconfig   |2 ++
  arch/arm/mach-exynos4/mach-origen.c |4 
  2 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-exynos4/Kconfig b/arch/arm/mach-exynos4/Kconfig
index 181fb04..4f908f7 100644
--- a/arch/arm/mach-exynos4/Kconfig
+++ b/arch/arm/mach-exynos4/Kconfig
@@ -225,6 +225,8 @@ config MACH_ORIGEN
select S3C_DEV_WDT
select S3C_DEV_HSMMC2
select EXYNOS4_SETUP_SDHCI
+   select S5P_DEV_I2C_HDMIPHY
+   select S5P_DEV_TV
help
  Machine support for ORIGEN based on Samsung EXYNOS4210

diff --git a/arch/arm/mach-exynos4/mach-origen.c 
b/arch/arm/mach-exynos4/mach-origen.c
index ed59f86..6ec68ee 100644
--- a/arch/arm/mach-exynos4/mach-origen.c
+++ b/arch/arm/mach-exynos4/mach-origen.c
@@ -81,8 +81,11 @@ static struct s3c_sdhci_platdata origen_hsmmc2_pdata 
__initdata = {

  static struct platform_device *origen_devices[] __initdata = {
s3c_device_hsmmc2,
+s5p_device_i2c_hdmiphy,
s3c_device_rtc,
s3c_device_wdt,
+s5p_device_hdmi,
+s5p_device_mixer,
  };

  static void __init origen_map_io(void)
@@ -95,6 +98,7 @@ static void __init origen_map_io(void)
  static void __init origen_machine_init(void)
  {
s3c_sdhci2_set_platdata(origen_hsmmc2_pdata);
+   s5p_i2c_hdmiphy_set_platdata(NULL);
platform_add_devices(origen_devices, ARRAY_SIZE(origen_devices));
  }

--
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev




--
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH 3/3] ARM: EXYNOS4: Add support for 8-bit bus width in SDHCI for ORIGEN

2011-09-02 Thread Tushar Behera

On Wednesday 31 August 2011 06:31 AM, Kukjin Kim wrote:

Tushar Behera wrote:


Platform data for SDHCI controller on ORIGEN board is missing the
support for 8-bit bus width. The platform data is extended in sync
with other EXYNOS4 machines.

Signed-off-by: Tushar Beheratushar.beh...@linaro.org
---
  arch/arm/mach-exynos4/mach-origen.c |8 
  1 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-exynos4/mach-origen.c

b/arch/arm/mach-exynos4/mach-

origen.c
index ae18812..6b6cd77 100644
--- a/arch/arm/mach-exynos4/mach-origen.c
+++ b/arch/arm/mach-exynos4/mach-origen.c
@@ -75,11 +75,19 @@ static struct s3c2410_uartcfg origen_uartcfgs[]

__initdata =

{
  static struct s3c_sdhci_platdata origen_hsmmc0_pdata __initdata = {
.cd_type= S3C_SDHCI_CD_INTERNAL,
.clk_type   = S3C_SDHCI_CLK_DIV_EXTERNAL,
+#ifdef CONFIG_EXYNOS4_SDHCI_CH0_8BIT
+   .max_width  = 8,
+   .host_caps  = MMC_CAP_8_BIT_DATA,
+#endif
  };

  static struct s3c_sdhci_platdata origen_hsmmc2_pdata __initdata = {
.cd_type= S3C_SDHCI_CD_INTERNAL,
.clk_type   = S3C_SDHCI_CLK_DIV_EXTERNAL,
+#ifdef CONFIG_EXYNOS4_SDHCI_CH2_8BIT
+   .max_width  = 8,
+   .host_caps  = MMC_CAP_8_BIT_DATA,
+#endif
  };

  static struct platform_device *origen_devices[] __initdata = {
--
1.7.4.1


Hi Tushar,

I wonder the bus width of SDHCI controller can be changed manually on ORIGEN
like SMDK board.


Thanks for your review.

On ORIGEN board, we have wire connections for HSMMC-0/2/3 between the 
MMC port and the SoC. Hence ideally we can work with HSMMC2 in both 
4-bit and 8-bit mode. However HSMMC0 can only work in 4-bit mode.


Also IIRC WLAN would be using HSMMC-3 controller for its operations. So 
we would have conflict when HSMMC2 is working in 8-bit mode and WLAN is 
also enabled. Hence it appears better to drop this patch now.



Thanks.

Best regards,
Kgene.
--
Kukjin Kimkgene@samsung.com, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.


--
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH 2/3] ARM: EXYNOS4: Add support for secondary MMC port on ORIGEN

2011-09-01 Thread Tushar Behera

Hi Kukjin,

On Wednesday 31 August 2011 06:23 AM, Kukjin Kim wrote:

Tushar Behera wrote:

-Original Message-
From: Tushar Behera [mailto:tushar.beh...@linaro.org]
Sent: Friday, August 26, 2011 6:39 PM
To: linux-samsung-...@vger.kernel.org
Cc: linaro-dev@lists.linaro.org; kgene@samsung.com; patc...@linaro.org
Subject: [PATCH 2/3] ARM: EXYNOS4: Add support for secondary MMC port on
ORIGEN

Secondary MMC port on ORIGEN is connected to sdhci instance 0. Support
for secondary MMC port is extended by registering sdhci instance 0.

Since sdhci instance 2 can contain a bootable media, sdhci instance 0
is registered after instance 2.


Would be helpful if above comments could be included in codes :)


Signed-off-by: Tushar Beheratushar.beh...@linaro.org
---
  arch/arm/mach-exynos4/Kconfig   |1 +
  arch/arm/mach-exynos4/mach-origen.c |7 +++
  2 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-exynos4/Kconfig b/arch/arm/mach-exynos4/Kconfig
index e6925de..4c14d5e 100644
--- a/arch/arm/mach-exynos4/Kconfig
+++ b/arch/arm/mach-exynos4/Kconfig
@@ -229,6 +229,7 @@ config MACH_ORIGEN
select CPU_EXYNOS4210
select S3C_DEV_RTC
select S3C_DEV_WDT
+   select S3C_DEV_HSMMC
select S3C_DEV_HSMMC2
select EXYNOS4_SETUP_SDHCI
help
diff --git a/arch/arm/mach-exynos4/mach-origen.c

b/arch/arm/mach-exynos4/mach-

origen.c
index e280270..ae18812 100644
--- a/arch/arm/mach-exynos4/mach-origen.c
+++ b/arch/arm/mach-exynos4/mach-origen.c
@@ -72,6 +72,11 @@ static struct s3c2410_uartcfg origen_uartcfgs[]

__initdata = {

},
  };

+static struct s3c_sdhci_platdata origen_hsmmc0_pdata __initdata = {
+   .cd_type= S3C_SDHCI_CD_INTERNAL,
+   .clk_type   = S3C_SDHCI_CLK_DIV_EXTERNAL,
+};
+
  static struct s3c_sdhci_platdata origen_hsmmc2_pdata __initdata = {
.cd_type= S3C_SDHCI_CD_INTERNAL,
.clk_type   = S3C_SDHCI_CLK_DIV_EXTERNAL,
@@ -79,6 +84,7 @@ static struct s3c_sdhci_platdata origen_hsmmc2_pdata
__initdata = {

  static struct platform_device *origen_devices[] __initdata = {
s3c_device_hsmmc2,
+   s3c_device_hsmmc0,
s3c_device_rtc,
s3c_device_wdt,
  };
@@ -93,6 +99,7 @@ static void __init origen_map_io(void)
  static void __init origen_machine_init(void)
  {
s3c_sdhci2_set_platdata(origen_hsmmc2_pdata);
+   s3c_sdhci0_set_platdata(origen_hsmmc0_pdata);
platform_add_devices(origen_devices, ARRAY_SIZE(origen_devices));
  }

--
1.7.4.1


OK, will apply.
If you don't mind, I will add comments the reason of the ordering when I
apply this.


Thanks. That would be great.


Thanks.

Best regards,
Kgene.
--
Kukjin Kimkgene@samsung.com, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.




--
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [PATCH 3/3] ARM: EXYNOS4: Add support for 8-bit bus width in SDHCI for ORIGEN

2011-09-01 Thread Tushar Behera

Hi Kukjin,

On Wednesday 31 August 2011 06:31 AM, Kukjin Kim wrote:

Tushar Behera wrote:


Platform data for SDHCI controller on ORIGEN board is missing the
support for 8-bit bus width. The platform data is extended in sync
with other EXYNOS4 machines.

Signed-off-by: Tushar Beheratushar.beh...@linaro.org
---
  arch/arm/mach-exynos4/mach-origen.c |8 
  1 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-exynos4/mach-origen.c

b/arch/arm/mach-exynos4/mach-

origen.c
index ae18812..6b6cd77 100644
--- a/arch/arm/mach-exynos4/mach-origen.c
+++ b/arch/arm/mach-exynos4/mach-origen.c
@@ -75,11 +75,19 @@ static struct s3c2410_uartcfg origen_uartcfgs[]

__initdata =

{
  static struct s3c_sdhci_platdata origen_hsmmc0_pdata __initdata = {
.cd_type= S3C_SDHCI_CD_INTERNAL,
.clk_type   = S3C_SDHCI_CLK_DIV_EXTERNAL,
+#ifdef CONFIG_EXYNOS4_SDHCI_CH0_8BIT
+   .max_width  = 8,
+   .host_caps  = MMC_CAP_8_BIT_DATA,
+#endif
  };

  static struct s3c_sdhci_platdata origen_hsmmc2_pdata __initdata = {
.cd_type= S3C_SDHCI_CD_INTERNAL,
.clk_type   = S3C_SDHCI_CLK_DIV_EXTERNAL,
+#ifdef CONFIG_EXYNOS4_SDHCI_CH2_8BIT
+   .max_width  = 8,
+   .host_caps  = MMC_CAP_8_BIT_DATA,
+#endif
  };

  static struct platform_device *origen_devices[] __initdata = {
--
1.7.4.1


Hi Tushar,

I wonder the bus width of SDHCI controller can be changed manually on ORIGEN
like SMDK board.


I will do further test and update you about the results.


Thanks.

Best regards,
Kgene.
--
Kukjin Kimkgene@samsung.com, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.




--
Tushar Behera

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


[PATCH 1/3] ARM: EXYNOS4: Fix sdhci card detection for ORIGEN

2011-08-26 Thread Tushar Behera
Fix incorrect value of cd_type field in platform data for sdhci
device.

Based on ARM: EXYNOS4: Fix card detection for sdhci 0 and 2.
commit a0d8efedb203b5b908dd46cea38201761e2380f9

Signed-off-by: Tushar Behera tushar.beh...@linaro.org
---
 arch/arm/mach-exynos4/mach-origen.c |4 +---
 1 files changed, 1 insertions(+), 3 deletions(-)

diff --git a/arch/arm/mach-exynos4/mach-origen.c 
b/arch/arm/mach-exynos4/mach-origen.c
index ed59f86..e280270 100644
--- a/arch/arm/mach-exynos4/mach-origen.c
+++ b/arch/arm/mach-exynos4/mach-origen.c
@@ -73,9 +73,7 @@ static struct s3c2410_uartcfg origen_uartcfgs[] __initdata = {
 };
 
 static struct s3c_sdhci_platdata origen_hsmmc2_pdata __initdata = {
-   .cd_type= S3C_SDHCI_CD_GPIO,
-   .ext_cd_gpio= EXYNOS4_GPK2(2),
-   .ext_cd_gpio_invert = 1,
+   .cd_type= S3C_SDHCI_CD_INTERNAL,
.clk_type   = S3C_SDHCI_CLK_DIV_EXTERNAL,
 };
 
-- 
1.7.4.1


___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


  1   2   >