3 years agoarch: arm64: boot: dts: T210 camera devnames
Bhanu Murthy V [Mon, 18 Jul 2016 22:49:34 +0000]
arch: arm64: boot: dts: T210 camera devnames

Add devnames instead of devnodes to improve
flexibility in binding userspace and kernel drivers.

Bug 1758789
Bug 200219854

Change-Id: I680a5a968e1c5dc32783470dfcc881b791491934
Signed-off-by: Bhanu Murthy V <bmurthyv@nvidia.com>
Reviewed-on: http://git-master/r/1151351
(cherry picked from commit 457c67e29e7a5892bb6ace057059e803470fd83d)
Reviewed-on: http://git-master/r/1183082
GVS: Gerrit_Virtual_Submit
Reviewed-by: Jihoon Bang <jbang@nvidia.com>

3 years agodriver: media: tegra: Update tegra camera_common
Frank Chen [Sat, 16 Jul 2016 00:33:08 +0000]
driver: media: tegra: Update tegra camera_common

1. Undefine standard V4L2 GAIN/Exposure CIDs and
switch to our own custom CIDs.

2. Add helper function to read mode specific info
from device tree.

Jira: CHWI-246

Bug 200168141

DEPENDS ON: <http://git-master/r/1176440>

Change-Id: I7bfd69a030031f8b450472ee9124cf08a030b1a9
Signed-off-by: Frank Chen <frankc@nvidia.com>
Reviewed-on: http://git-master/r/1183834
Reviewed-by: Esen Chen <esenc@nvidia.com>
Tested-by: Esen Chen <esenc@nvidia.com>
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Jihoon Bang <jbang@nvidia.com>

3 years agogpu: nvgpu: suppress prints in submit path
Deepak Nibade [Tue, 24 May 2016 08:21:26 +0000]
gpu: nvgpu: suppress prints in submit path

When we run out of gpfifo space or private command buffer
space, we have error spew like below :
__gk20a_channel_syncpt_incr: not enough priv cmd buffer space
gk20a_submit_channel_gpfifo: fail

Dumping these prints to UART cause increase in submit
latencies

But on these failures, we return -ENOSPC to UMD and then
UMD retries the submit, hence it might be unnecessary to dump
these prints

Hence, remove the error prints of insufficient space
and use gk20a_dbg_fn() instead of gk20a_err() to print failure
in gk20a_submit_channel_gpfifo()

Bug 200202653

Change-Id: I49efd7c6c554dd4fbfa4e66d196eb352e69f92c6
Signed-off-by: Deepak Nibade <dnibade@nvidia.com>
Reviewed-on: http://git-master/r/1152378
(cherry picked from commit dc7af18bf8056c213165d4cd1c55ea0fba9f1341)
Reviewed-on: http://git-master/r/1190065
Reviewed-by: Bharat Nihalani <bnihalani@nvidia.com>
Tested-by: Bharat Nihalani <bnihalani@nvidia.com>
GVS: Gerrit_Virtual_Submit

3 years agothermal: Avoid scheduling work during poweroff
Rohit Vaswani [Wed, 20 Jul 2016 02:01:17 +0000]
thermal: Avoid scheduling work during poweroff

During a critical thermal trip, orderly_poweroff is called.
However, if the userspace is slow to react to poweroff (systemd),
and the thermal zone is configured with polling mode because of
either polling_delay or passive_delay, thermal_core may end up
in calling orderly_poweroff multiple times.
This eventually slows the userspace even more by
scheduling overwhelming number of threads in the system.

Relevant dump_stack spew during the shutdown process:
 dump_backtrace+0x0/0xf4
 show_stack+0x14/0x1c
 dump_stack+0x20/0x28
 handle_critical_trips+0x80/0xa0
 handle_thermal_trip+0x40/0x64
 thermal_zone_device_update+0xf0/0x110
 thermal_zone_device_check+0x10/0x18
 process_one_work+0x274/0x430
 worker_thread+0x184/0x294
 kthread+0xc0/0xc8

We should not try to spawn-off more threads once we have
decided to shutdown i.e. we should disable polling mode
once it is known that a critical trip is going to result
in a poweroff. This patch does that by setting the polling
and passive delay to zero which cancels the repeated work
and allows userspace to shutdown.

Change-Id: Ia8714c288f9a332dc8ea7a3ffc7156fc6095ed9e
Signed-off-by: Rohit Vaswani <rvaswani@nvidia.com>
Reviewed-on: http://git-master/r/1183768
Reviewed-by: Navneet Kumar <navneetk@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Wen Yi <wyi@nvidia.com>
Reviewed-by: Ninad Malwade <nmalwade@nvidia.com>
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agodrivers: camera: Fix v4l2 driver compliance
Bhanu Murthy V [Tue, 19 Jul 2016 01:00:42 +0000]
drivers: camera: Fix v4l2 driver compliance

Fix s/g/query dv timings compliance, report not
supported if the sub device on CSI does not support
the timing fops.
Fix s/g crop fops for sub devices which does not
support cropping.
Fix subdev used in s/g param fops.

Bug 1675242
Bug 200214096

Change-Id: If262ca8038694ffb05273a99268b86984b1532c1
Signed-off-by: Bhanu Murthy V <bmurthyv@nvidia.com>
Reviewed-on: http://git-master/r/1183150
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>

3 years agodts: plugin-manager: Fix ov5693 csi port index
Frank Chen [Wed, 20 Jul 2016 21:54:22 +0000]
dts: plugin-manager: Fix ov5693 csi port index

Correct csi port index for ov5693 sensor. Incorrect
index will cause dpd and mfi failures.

Bug 200220580

Change-Id: Ie9de7e02db739c45dba0ffba56f0070623b1a024
Signed-off-by: Frank Chen <frankc@nvidia.com>
Reviewed-on: http://git-master/r/1184514
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Wenjia Zhou <wenjiaz@nvidia.com>
Reviewed-by: Jihoon Bang <jbang@nvidia.com>
Reviewed-by: Bhanu Murthy V <bmurthyv@nvidia.com>
GVS: Gerrit_Virtual_Submit

3 years agoarm64: dts: Add new dts file for Darcy with new memory
Daniel Fu [Wed, 8 Jun 2016 13:18:19 +0000]
arm64: dts: Add new dts file for Darcy with new memory

Bug 200203997

Change-Id: I17bfe732dd71c7d59a5ef70665c8dd07f9f86546
Signed-off-by: Daniel Fu <danifu@nvidia.com>
Reviewed-on: http://git-master/r/1161188
(cherry picked from commit 790caf00c2ef2e025ed8651c0aec5c8e4539a545)
Reviewed-on: http://git-master/r/1183414
GVS: Gerrit_Virtual_Submit
Reviewed-by: Vinayak Pane <vpane@nvidia.com>

3 years agodrivers: camera: Ignore resolution errors for CSI
Bhanu Murthy V [Thu, 14 Jul 2016 17:01:06 +0000]
drivers: camera: Ignore resolution errors for CSI

Ignore only frame height or width short/long errors
from CSI. Resolution mismatch can occur the actual lines
can be inclusive of embedded data lines which application
ignores anyway. Report failure only when the pixel parser
or CIL status is wrong.

Bug 1708456

Change-Id: I96fda6b6802e1497de2d5ca074c416d3b6ea5e40
Signed-off-by: Bhanu Murthy V <bmurthyv@nvidia.com>
Reviewed-on: http://git-master/r/1181511
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Frank Chen <frankc@nvidia.com>
Reviewed-by: Jihoon Bang <jbang@nvidia.com>

3 years agodrivers: camera: support maxframerate property
Bhanu Murthy V [Thu, 7 Jul 2016 01:24:09 +0000]
drivers: camera: support maxframerate property

Add check for maxframerate to distinguish different
mode tables needed by sensor for optimal use.

Bug 1769654

Change-Id: If8db48af46df984e440fff3472f0ea93145d5a01
Signed-off-by: Bhanu Murthy V <bmurthyv@nvidia.com>
Reviewed-on: http://git-master/r/1176484
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>

3 years agoarm64: dts: imx274: enable multiple sensor modes
Josh Kuo [Thu, 7 Jul 2016 08:23:26 +0000]
arm64: dts: imx274: enable multiple sensor modes

enable multiple sensor modes for imx274.

support 4k@60fps, 4k@30fps and 1080p@60fps

Bug 1769654

Change-Id: Id25bc8279f273add5df1d76e0b5ff7de19224462
Signed-off-by: Josh Kuo <joshk@nvidia.com>
Reviewed-on: http://git-master/r/1176815
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>

3 years agodrivers: imx274: Add maxfps properties
Bhanu Murthy V [Thu, 7 Jul 2016 01:25:36 +0000]
drivers: imx274: Add maxfps properties

Add max fps properties for imx274 driver

Bug 1769654

Change-Id: I063ef6f0ebb5b98625a047d0c2e4297b6a2bb532
Signed-off-by: Bhanu Murthy V <bmurthyv@nvidia.com>
Reviewed-on: http://git-master/r/1176485
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>

3 years agoinclude: media: Add max fps properties
Bhanu Murthy V [Thu, 7 Jul 2016 01:22:12 +0000]
include: media: Add max fps properties

Add maxfps default and current format fields
to camera common data to identify proper
sensor mode id.

Bug 1769654

Change-Id: I10b09b437a97f80f16d5e625b50a99bc819c4670
Signed-off-by: Bhanu Murthy V <bmurthyv@nvidia.com>
Reviewed-on: http://git-master/r/1176483
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Jihoon Bang <jbang@nvidia.com>

3 years agoinclude: media: Add maxframerate field
Bhanu Murthy V [Thu, 7 Jul 2016 01:19:01 +0000]
include: media: Add maxframerate field

Add maxframerate field as part of pixelformat
to distinguish different framerate ranges support
as two different modes in sensor drivers.
e.g: 4k@(0-30) and 4K@(0-60).
More parameters might be changed in sensor with a
framerate range for optimal usage.

Bug 1769654

Change-Id: I80e6a36911e802d9f7b17b61248b6268f9b0439d
Signed-off-by: Bhanu Murthy V <bmurthyv@nvidia.com>
Reviewed-on: http://git-master/r/1176482
Reviewed-by: Jihoon Bang <jbang@nvidia.com>

3 years agomedia: tc358840: BUG_ON for overrunning array
Ahung Cheng [Wed, 6 Jul 2016 05:16:59 +0000]
media: tc358840: BUG_ON for overrunning array

In print_avi_infoframe, it allocates array buffer of 17 bytes.
It might run into a case to access array buffer at byte position 28.
in hdmi_infoframe_unpack when switch case value is SPD.

bug 200192413

Change-Id: I3624d471353f8c5b0ed317fc30e61beee7290439
Signed-off-by: Ahung Cheng <ahcheng@nvidia.com>
Reviewed-on: http://git-master/r/1176086
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Jihoon Bang <jbang@nvidia.com>

3 years agodrivers: imx274: adding multiple sensor modes
Josh Kuo [Thu, 23 Jun 2016 05:52:39 +0000]
drivers: imx274: adding multiple sensor modes

adding multiple sensor modes for imx274.

support 4k@60fps, 4k@30fps and 1080p@60fps

Bug 1769654

Change-Id: I1753d63e0c8adaafc59951de8c67b6314da24304
Signed-off-by: Josh Kuo <joshk@nvidia.com>
Reviewed-on: http://git-master/r/1170059
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Bhanu Murthy V <bmurthyv@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Shiva Dubey <sdubey@nvidia.com>
Reviewed-by: Jihoon Bang <jbang@nvidia.com>

3 years agoarm64: tegra21: enable CONFIG_SWAP
Shreshtha SAHU [Wed, 20 Jul 2016 04:42:18 +0000]
arm64: tegra21: enable CONFIG_SWAP

boot.img has increased by 38592 bytes.

Bug 1790199

Change-Id: Icb37ba97da8003cc70e2f205c1104ac6a926c54e
Signed-off-by: Shreshtha SAHU <ssahu@nvidia.com>
Reviewed-on: http://git-master/r/1183845
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoarm: arm64: boot: dts: gang_mode
Ahung Cheng [Wed, 8 Jun 2016 03:34:44 +0000]
arm: arm64: boot: dts: gang_mode

Add gang_mode entry for TC358840

bug 1772273

Change-Id: Ib245d1fb1437fde8c0fce7e64dd0eed1329fdd8d
Signed-off-by: Ahung Cheng <ahcheng@nvidia.com>
(cherry picked from commit 5e9c441b7a7857d8e22cdd95e08daaf8c898b46b)
Signed-off-by: Ahung Cheng <ahcheng@nvidia.com>
Reviewed-on: http://git-master/r/1165728
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>

3 years agodrivers: media: camera: Gang Mode from device tree
Ahung Cheng [Fri, 3 Jun 2016 10:22:05 +0000]
drivers: media: camera: Gang Mode from device tree

Add support for reversing CSI mapping by looking for a new entry,
gang-mode. Set gang mode to GANG_MODE_R_L, instead of default
GANG_MODE_L_R, to swap input ports.

bug 1772273

Change-Id: Ia2be129354e4aca3c7236125667f18e950061f63
Reviewed-on: http://git-master/r/1158678
(cherry picked from commit b595683f9b562b55323ed185e3a4405a31b722b5)
Signed-off-by: Ahung Cheng <ahcheng@nvidia.com>
(cherry picked from commit 7cc03562332180e21090c288ffaa5dfd89ec2df3)
Signed-off-by: Ahung Cheng <ahcheng@nvidia.com>
Reviewed-on: http://git-master/r/1165727
GVS: Gerrit_Virtual_Submit
Reviewed-by: Jihoon Bang <jbang@nvidia.com>
Reviewed-by: Bhanu Murthy V <bmurthyv@nvidia.com>
Reviewed-by: Hayden Du <haydend@nvidia.com>

3 years agov4l2-event: v4l2_event_queue: do nothing if vdev == NULL
Hans Verkuil [Tue, 23 Jun 2015 09:20:23 +0000]
v4l2-event: v4l2_event_queue: do nothing if vdev == NULL

If the vdev pointer == NULL, then just return.

This makes it easier for subdev drivers to use this function without having to
check if the sd->devnode pointer is NULL or not.

Bug 1770474

Change-Id: I4d9798b58ea13e987dcf50442a96694a8f803291
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Signed-off-by: Ken Chang <kenc@nvidia.com>
Reviewed-on: http://git-master/r/1154256
(cherry picked from commit 4e09f1b068c7f67926258f6b0a802fe5691ccd4b)
Signed-off-by: Ahung Cheng <ahcheng@nvidia.com>
Reviewed-on: http://git-master/r/1165707
Reviewed-by: Hayden Du <haydend@nvidia.com>

3 years agoARM64: DTS: t210: sdhci pd disable
Bitan Biswas [Mon, 15 Feb 2016 11:41:48 +0000]
ARM64: DTS: t210: sdhci pd disable

sdmmc power domain disabled for T210

bug 200155814

Change-Id: Ie0ae38b517354a8223e5fd8ecf89c60ab65d1a70
Signed-off-by: Bitan Biswas <bbiswas@nvidia.com>
Reviewed-on: http://git-master/r/1011959
(cherry picked from commit 544ed2d95a1fd5564ad072a91ba1a156ff2d41b0)
Reviewed-on: http://git-master/r/1177604
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Pavan Kunapuli <pkunapuli@nvidia.com>
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agommc: sdhci: allow clock enable in suspend sequence
Bitan Biswas [Wed, 17 Feb 2016 12:24:10 +0000]
mmc: sdhci: allow clock enable in suspend sequence

Some host with SDHCI_QUIRK2_NON_STD_RTPM deviation
need to enable clock in the suspend sequence.
This change enables such platforms to
suspend properly.

bug 200155814

Change-Id: I37969748660965c4b4097837e9b9c6bb7a593b4e
Signed-off-by: Bitan Biswas <bbiswas@nvidia.com>
Reviewed-on: http://git-master/r/1012839
(cherry picked from commit e27946db4cc054ab016459e3e16f3e68a51f0872)
Reviewed-on: http://git-master/r/1177603
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>

3 years agommc: sdhci: mmc detect change timeout update
Bitan Biswas [Fri, 8 Jul 2016 11:15:44 +0000]
mmc: sdhci: mmc detect change timeout update

Higher mmc detect change timeout is needed
even if card is removed during suspend

bug 200196862

Change-Id: Ie771af36324d9e5003d1a4167f306c765848b490
Signed-off-by: Bitan Biswas <bbiswas@nvidia.com>
Reviewed-on: http://git-master/r/1177591
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Pavan Kunapuli <pkunapuli@nvidia.com>
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoFix uninitialized variable
Joe Korty [Wed, 25 May 2016 13:09:41 +0000]
Fix uninitialized variable

Quiet gcc warning by giving 'in6' an initial
value of NULL.

Technically this is wrong, but there seems to
be no path through the function where 'in6' is
not set then used.  If that is wrong, we will
get a NULL pointer kernel panic.  Though bad,
that is better then the current situation,
where some random spot in the kernel gets
modified via the following of an undefined
pointer value in 'in6'.

include/net/ipv6.h:417:34: warning:
'in6' may be used uninitialized in this function
[-Wmaybe-uninitialized]
return ((ul1[0] ^ ul2[0]) | (ul1[1] ^ ul2[1])) == 0UL;

net/ipv4/tcp.c:3513:19: note: 'in6' was declared here
struct in6_addr *in6;

Bug 200187768

Change-Id: I4db7b351ae720c0a34c48ea7f39e95b788164900
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163944
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoRemove unused variables and labels
Joe Korty [Wed, 25 May 2016 13:05:34 +0000]
Remove unused variables and labels

drivers/media/i2c/imx185.c:548:6:
warning: unused variable 'i' [-Wunused-variable]
drivers/media/i2c/imx185.c:644:1:
warning:
label 'fail' defined but not used [-Wunused-label]
drivers/media/i2c/imx185.c:999:7:
warning:unused variable 'dt_name' [-Wunused-variable]
drivers/media/i2c/imx185.c:560:9:
warning: 'gain' may be used uninitialized in this
function [-Wmaybe-uninitialized]

Bug 200187768

Change-Id: I6e66765abe97fc8923ff48c778a5a3befddb4fc1
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163943
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>

3 years agoSilence uninitialized variable warning
Joe Korty [Wed, 25 May 2016 12:58:56 +0000]
Silence uninitialized variable warning

When we go through the switch() case
where 'send_direct' is not initialized, the
code later on that uses 'send_direct' will
be skipped over.  So it is safe to set this
'send_direct' to any value we want in order
to quiet this gcc warning.

net/mac80211/cfg.c: In function 'ieee80211_tdls_mgmt':
net/mac80211/cfg.c:3225:5: warning:
'send_direct' may be used uninitialized in this
function [-Wmaybe-uninitialized]
if (send_direct) {

Bug 200187768

Change-Id: I01128d4682898b01e0512881029497821bc93201
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163942
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoAdd default case in switch statement
Joe Korty [Wed, 25 May 2016 12:57:27 +0000]
Add default case in switch statement

Provide a reasonable default value
(ie, the original value) for smps_mode
when a new value is not being assigned.

This forces the value of 'changed' to be correct
when no attempt at changing is being done to
'smps_mode'.

net/mac80211/ht.c: In function
'ieee80211_ht_cap_ie_to_sta_ht_cap':
net/mac80211/ht.c:232:5: warning:
'smps_mode' may be used uninitialized in this
function [-Wmaybe-uninitialized]
if (smps_mode != sta->sta.smps_mode)

Bug 200187768

Change-Id: Ife7dbe09e911d801c4fc1e85c3f8ad40cd681c09
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163941
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoFix 'const' propagation prob
Joe Korty [Wed, 25 May 2016 12:47:49 +0000]
Fix 'const' propagation prob

The strings pointed to are unchanging while in use,
so add a 'const' to the string definition.

drivers/usb/misc/usb_nvshieldled.c:
In function 'nvshieldled_probe':
drivers/usb/misc/usb_nvshieldled.c:394:7: warning:
passing argument 3 of 'of_property_read_string' from
incompatible pointer type [-Wincompatible-pointer-types]
       &edp_name)) {

In file included from drivers/usb/misc/usb_nvshieldled.c:26:0:
include/linux/of.h:268:12: note: expected 'const char **' but
argument is of type 'char **'
extern int of_property_read_string(struct device_node *np,

Bug 200187768

Change-Id: I989ecdf4d0c40ef032c2056889db6b0e399748bb
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163936
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoFix the casting issue
Gaurav Singh [Wed, 25 May 2016 12:45:10 +0000]
Fix the casting issue

Allow a 64 bit pointer to be cast and put into a
32-bit integer, when that pointer points to a
location within a 32 bit userspace.

Normally this is a very bad thing to do, the compiler
is right to warn us about this, but in this very
specific case, it is OK.

In file included from fs/compat_binfmt_elf.c:128:0:
fs/binfmt_elf.c: In function 'create_elf_tables':
/cuba/jak/tegra/r23.1/a/arch/arm64/include/asm/elf.h:143:7:
warning: cast from pointer to
integer of different size [-Wpointer-to-int-cast]
(elf_addr_t)current->mm->context.vdso);

fs/binfmt_elf.c:216:26: note:
in definition of macro 'NEW_AUX_ENT'
elf_info[ei_index++] = val;

fs/binfmt_elf.c:226:2: note:
in expansion of macro 'ARCH_DLINFO'
ARCH_DLINFO;

Bug 200187768

Change-Id: Ia9086a0b43c608e672d04cec209ea7edc1d3a806
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163935
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoFix uninitialized variable
Joe Korty [Wed, 25 May 2016 12:56:24 +0000]
Fix uninitialized variable

sound/soc/codecs/rt5639.c: In function 'rt5639_pll_calc':
sound/soc/codecs/rt5639.c:2833:19: warning:
'm_t' may be used uninitialized in this
function [-Wmaybe-uninitialized]
 pll_code->m_code = m;

Bug 200187768

Change-Id: I86981ef397ff3f8bd0bf80e723d00cb1ca937ef7
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163940
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoFix kernel warnings
Joe Korty [Wed, 25 May 2016 12:54:57 +0000]
Fix kernel warnings

1) rt5659_enable_push_button_irq() is not yet being
used, so comment it out with #ifdef FIXME.
2) Give mt_t an initial value of zero, for those
paths where it is not initialized.

sound/soc/codecs/rt5659.c:1314:13: warning:
'rt5659_enable_push_button_irq' defined but
not used [-Wunused-function]
static void
rt5659_enable_push_button_irq(struct snd_soc_codec *codec,

sound/soc/codecs/rt5659.c: In function 'rt5659_pll_calc':
sound/soc/codecs/rt5659.c:3636:19: warning:
'm_t' may be used uninitialized in this
function [-Wmaybe-uninitialized]
pll_code->m_code = m;

Bug 200187768

Change-Id: Ibc5aef1383dd6eb140968e5cf4925f8675d36832
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163939
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agommc: sdhci: Support polling mode
Arun Kannan [Thu, 14 Jul 2016 00:58:16 +0000]
mmc: sdhci: Support polling mode

Add support for broken-cd property in DT.

Bug 1765751

Change-Id: I5196de66ce9458971d7d8bf5f34f5c1351477f6a
Signed-off-by: Arun Kannan <akannan@nvidia.com>
Reviewed-on: http://git-master/r/1181053
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Pavan Kunapuli <pkunapuli@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Kamal Balagopalan <kbalagopalan@nvidia.com>

3 years agousb: gadget: xudc: enable RX detector control
Mark Kuo [Mon, 23 May 2016 11:02:00 +0000]
usb: gadget: xudc: enable RX detector control

Enable receiver detector control when necessary.

Bug 200162414
Bug 200179626

Change-Id: Ief7aeb0556cd7f6f4bd308f8c715ef68acf86b4b
Signed-off-by: Mark Kuo <mkuo@nvidia.com>
Signed-off-by: Hans Yang <hansy@nvidia.com>
Reviewed-on: http://git-master/r/1169998
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: ChihMin Cheng <ccheng@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Ashutosh Jha <ajha@nvidia.com>

3 years agoxhci-hcd: add success/failure soft retry counters
JC Kuo [Wed, 4 May 2016 11:30:39 +0000]
xhci-hcd: add success/failure soft retry counters

This commit adds counters for successful and failed soft retry
attempts.

bug 200162414

Change-Id: I7406cea4368da85c78c8fe350398031fc8121015
Signed-off-by: JC Kuo <jckuo@nvidia.com>
Reviewed-on: http://git-master/r/1169997
GVS: Gerrit_Virtual_Submit
Reviewed-by: ChihMin Cheng <ccheng@nvidia.com>
Tested-by: Mark Kuo <mkuo@nvidia.com>
Reviewed-by: Ashutosh Jha <ajha@nvidia.com>

3 years agoxhci-tegra: t210: support XHCI "soft retry"
JC Kuo [Tue, 22 Dec 2015 13:33:15 +0000]
xhci-tegra: t210: support XHCI "soft retry"

This commit implements a TEGRA210 XHCI specific programming sequence
which needs to be done along with "soft retry".

Verified E2220-ERS.

new added
- wait for U0 when is going to assert clame_en_early, timeout in 300us

Bug 200162414

Change-Id: I9b9e1eeb39d259416c199819808020c48171ab57
Signed-off-by: JC Kuo <jckuo@nvidia.com>
Reviewed-on: http://git-master/r/1169996
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: ChihMin Cheng <ccheng@nvidia.com>
Tested-by: Mark Kuo <mkuo@nvidia.com>
Reviewed-by: Ashutosh Jha <ajha@nvidia.com>

3 years agoxhci-hcd: support soft retry on SS transfer error
JC Kuo [Sun, 20 Dec 2015 06:37:02 +0000]
xhci-hcd: support soft retry on SS transfer error

This commit implements XHCI "soft retry" for SuperSpeed endpoints which
encounters transfer errors.
When transfer error happens on an SuperSpeed endpoint, XHCI driver will
1. queue a "reset endpoint" command with TSP=1 (Transfer State Preserve)

2. invoke a HCD driver specific callback "->endpoint_soft_retry()" to let
HCD driver has a chance to configure its hardware

3. ring door bell for the endpoint upon seeing the command completion

bug 200162414

Change-Id: I19df13614f794437c7f4d4369dbdaed13e1da85a
Signed-off-by: JC Kuo <jckuo@nvidia.com>
Reviewed-on: http://git-master/r/1169995
GVS: Gerrit_Virtual_Submit
Reviewed-by: ChihMin Cheng <ccheng@nvidia.com>
Tested-by: Mark Kuo <mkuo@nvidia.com>
Reviewed-by: Ashutosh Jha <ajha@nvidia.com>

3 years agoplatform: tegra: padctl: add receiver detector API
Mark Kuo [Mon, 23 May 2016 09:52:37 +0000]
platform: tegra: padctl: add receiver detector API

Support clamp_en_early and receiver detector controls.

A copy of JC's original work at http://git-master/r/#/c/1141282/3 on
tegra_usb_pad_ctrl.c

Bug 200162414
Bug 200179626

Change-Id: I8a29f7d90bfedaaa1b2609f46f511754eaa1e71f
Signed-off-by: Mark Kuo <mkuo@nvidia.com>
Reviewed-on: http://git-master/r/1169994
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: ChihMin Cheng <ccheng@nvidia.com>
Reviewed-by: Ashutosh Jha <ajha@nvidia.com>

3 years agovideo: tegra: dc: sysfs for HDMI AVI content type
Ahung Cheng [Tue, 14 Jun 2016 04:43:47 +0000]
video: tegra: dc: sysfs for HDMI AVI content type

Updated avi infoframe would be picked up by HW for next frame.

Sysfs node:
/sys/devices/platform/host1x/tegradc.1/hdmi_config/it_content_type
Value:
        0x0 = HDMI_AVI_IT_CONTENT_GRAPHICS,
        0x1 = HDMI_AVI_IT_CONTENT_PHOTO,
        0x2 = HDMI_AVI_IT_CONTENT_CINEMA,
        0x3 = HDMI_AVI_IT_CONTENT_GAME,
        0x4 = HDMI_AVI_IT_CONTENT_NONE, (Default)

bug 1772096

Change-Id: I9c9f79dcf0b6a9ff192ad0d457c55f38e21c2cfd
Signed-off-by: Ahung Cheng <ahcheng@nvidia.com>
Reviewed-on: http://git-master/r/1164025
(cherry picked from commit 708f5ad20644a2946f3e936e8c267333294d6408)
Signed-off-by: Ahung Cheng <ahcheng@nvidia.com>
Reviewed-on: http://git-master/r/1170952
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Animesh Kishore <ankishore@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agodrivers: imx2xx: Fail probe if device unavailable
Bhanu Murthy V [Wed, 13 Jul 2016 01:40:38 +0000]
drivers: imx2xx: Fail probe if device unavailable

Return error if otp read or fuse id read fails
Turn off power before returning error and remove
check for device not available during power off as
it's already verified during power on.

Bug 1784114

Change-Id: Id217b87fb8352c426f68468b1520da9520b0dcc7
Signed-off-by: Bhanu Murthy V <bmurthyv@nvidia.com>
Reviewed-on: http://git-master/r/1180011
(cherry picked from commit 7ecab59b36e15224e0ea3901055be20794f61f6e)
Reviewed-on: http://git-master/r/1180977
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Jihoon Bang <jbang@nvidia.com>

3 years agomedia: tegra: graph: clean unlinked channels
Frank Chen [Sat, 9 Jul 2016 00:43:48 +0000]
media: tegra: graph: clean unlinked channels

We need to clean all the unlinked channels when
failed to find any imager device in DT.

Failed to do so will leave a false video node in
/dev directory.

Bug 1768704

Change-Id: Ifdbcd2ebeb0f8c1b1a2d28d3b31e4928b6978842
Signed-off-by: Frank Chen <frankc@nvidia.com>
Reviewed-on: http://git-master/r/1177935
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: David Wang (SW-TEGRA) <davidw@nvidia.com>
Reviewed-by: Bhanu Murthy V <bmurthyv@nvidia.com>
Reviewed-by: Jihoon Bang <jbang@nvidia.com>

3 years agoARM64: DT: E3326 auto detect support
Frank Chen [Thu, 2 Jun 2016 05:09:44 +0000]
ARM64: DT: E3326 auto detect support

Add back E3326 auto detect support since
both cboot and uboot are supporting plugin
manager now.

Bug 1768704

Change-Id: If33edf7aa999d1e87cf603824addeee9eabd23e5
Signed-off-by: Frank Chen <frankc@nvidia.com>
Reviewed-on: http://git-master/r/1175910
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Bhanu Murthy V <bmurthyv@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Jihoon Bang <jbang@nvidia.com>

3 years agoplatform: tegra: move pstore to carveout memory
Bibek Basu [Tue, 12 Jul 2016 07:07:38 +0000]
platform: tegra: move pstore to carveout memory

only carveout memory is gauranteed to be not
touched by bootloader. L4T uses NCT carveout
for Pstore .This change makes it explicit

Bug 200211744

Change-Id: I783f93a40f828784c181731cf21f6fcc9c95d718
Signed-off-by: Bibek Basu <bbasu@nvidia.com>
Reviewed-on: http://git-master/r/1178848
GVS: Gerrit_Virtual_Submit
Reviewed-by: Naveen Kumar S <nkumars@nvidia.com>

3 years agovideo: tegra: dc: HDMI SPD InfoFrame support
Ahung Cheng [Tue, 21 Jun 2016 10:39:53 +0000]
video: tegra: dc: HDMI SPD InfoFrame support

Add support to send SPD over Generic InfoFrame.

Please be noted that once SPD InfoFrame support is
enabled, HDR InfoFrame will be disabled as only one
of them can use Generic InfoFrame at one time.

To enable SPD InfoFrame, please add this in kernel config
 CONFIG_TEGRA_HDMI_SPD_INFOFRAME=y

And add source product description format as below under
hdmi-display in device tree
spd-infoframe {
vendor-name = "XXX";
product-description = "YYY";
source-information = <N>;
};

bug 200211468

Change-Id: Ifa8e1f54e1cb64fd507a483b24f8962a96be180e
Signed-off-by: Ahung Cheng <ahcheng@nvidia.com>
Reviewed-on: http://git-master/r/1168466
(cherry picked from commit 5604cf8cbc53f91b6945cb933526249113fcbbe8)
Signed-off-by: Ahung Cheng <ahcheng@nvidia.com>
Reviewed-on: http://git-master/r/1170967
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoarch: arm64: boot: dts: Remove barometer
Sunny Li [Wed, 29 Jun 2016 04:44:23 +0000]
arch: arm64: boot: dts: Remove barometer

Remove barometer override option in plugin-manager

Bug 200201316

Change-Id: I22bc7151e17ff07abdc18ddc7764ccac475875ca
Signed-off-by: Sunny Li <sunnyl@nvidia.com>
Reviewed-on: http://git-master/r/1172923
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Hayden Du <haydend@nvidia.com>

3 years agoFix uninitialized variable
Joe Korty [Wed, 25 May 2016 12:52:48 +0000]
Fix uninitialized variable

drivers/staging/iio/meter/ina3221.c: In function
'ina3221_get_shuntv_offset':
drivers/staging/iio/meter/ina3221.c:1021:25: warning:
'offset' may be used uninitialized in this function
[-Wmaybe-uninitialized] shuntv_offset->offset = offset;

Bug 200187768

Change-Id: Iba86f1ed92ab10d702172f577ea59848d337e0a1
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163938
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>

3 years agopcie: host: tegra: fix l1ss programming
Vidya Sagar [Tue, 5 Jul 2016 10:19:52 +0000]
pcie: host: tegra: fix l1ss programming

L1SS configuration parameters need to be programmed
only for Tegra root ports and immediate devices (EP/UP port).
This change fixes the unwanted programming being done
to other devices apart from the ones mentioned above

Bug 200215912

Change-Id: Id8d8ec9952f3c7f7249de0f5c2ed244721e2fb57
Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
Reviewed-on: http://git-master/r/1175569
(cherry picked from commit a006e75c1e505eb66386f8be24682d515f726111)
Reviewed-on: http://git-master/r/1176200
(cherry picked from commit 21f3796d6522578736342798947b1c99eec41c94)
Reviewed-on: http://git-master/r/1176996
(cherry picked from commit 0e20e95939447170b88b63bdb4a2365f07127bc6)
Reviewed-on: http://git-master/r/1178428
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bharat Nihalani <bnihalani@nvidia.com>

3 years agopcie: host: tegra: re-org L1SS reg programming
Vidya Sagar [Wed, 15 Jun 2016 16:13:13 +0000]
pcie: host: tegra: re-org L1SS reg programming

performs all L1SS registers programming before L1SS is
enabled by kernel

Bug 200210426

Change-Id: I48fe1a31933b9dfa9733b2550b1555db9be568b2
Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
Reviewed-on: http://git-master/r/1165056
(cherry picked from commit 931881c4ce318b8d6b42db2300347db8546994ae)
Reviewed-on: http://git-master/r/1172629
(cherry picked from commit 67ad51482f14086c84986b2bf7eed9ff1d18a342)
Reviewed-on: http://git-master/r/1178427
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bharat Nihalani <bnihalani@nvidia.com>

3 years agoPCI: save and restore L1-SS registers
Vidya Sagar [Wed, 16 Dec 2015 08:17:56 +0000]
PCI: save and restore L1-SS registers

saves and restores L1 PM-SS registers during suspend and resume
this makes sure consistency of L1SS states across suspend/resume
cycle

Bug 200161255

Change-Id: I68b25e9433d0f08e4fadbe4547db7fb811f59462
Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
Reviewed-on: http://git-master/r/923504
(cherry picked from commit 39530fa01a94d0f57f5fbb0febc56ac33f8e0024)
Reviewed-on: http://git-master/r/1165055
(cherry picked from commit 7db113ba07654ce0494b96fd2eb8bf2c3dd02407)
Reviewed-on: http://git-master/r/1172628
(cherry picked from commit 7348b2855df4c29c964b33db08d1687ed6c0d216)
Reviewed-on: http://git-master/r/1178426
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bharat Nihalani <bnihalani@nvidia.com>

3 years agoPCI: Add support for save/restore of extended capabilities
Alex Williamson [Tue, 17 Dec 2013 23:43:45 +0000]
PCI: Add support for save/restore of extended capabilities

Current save/restore is specific to standard capabilities.

Bug 200210426

Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
(cherry picked from commit fd0f7f73ca96bb0f8723b5e59759ad43bab88954)

Change-Id: I58e5d0ea452947d329e099ba54fa9059370693fc
Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
Reviewed-on: http://git-master/r/1165054
(cherry picked from commit 61353ada0c3cae8f9302766b06158582387151f2)
Reviewed-on: http://git-master/r/1172627
(cherry picked from commit 8a2e41a87a7426c16916688205c31996422efa04)
Reviewed-on: http://git-master/r/1178425
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bharat Nihalani <bnihalani@nvidia.com>

3 years agoPCI: Fix comment typo for pci_add_cap_save_buffer()
Yijing Wang [Thu, 1 Aug 2013 13:05:27 +0000]
PCI: Fix comment typo for pci_add_cap_save_buffer()

Fix trivial comment typo for pci_add_cap_save_buffer().

Bug 200210426

Signed-off-by: Yijing Wang <wangyijing@huawei.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
(cherry picked from commit ce1be10bf6dc8406ae773f0ac6265585a4154d37)

Change-Id: I7c657211f6c230ad2ac216f8539ade8205964626
Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
Reviewed-on: http://git-master/r/1165053
(cherry picked from commit 791fe08f3aff6b141b4f406b394f8cc275665f5f)
Reviewed-on: http://git-master/r/1172626
(cherry picked from commit f47fc199c5f0ab097739361df2981ab71607119f)
Reviewed-on: http://git-master/r/1178424
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bharat Nihalani <bnihalani@nvidia.com>

3 years agopcie: host: tegra: disable aspm-l0s by default
Vidya Sagar [Mon, 2 Nov 2015 09:20:51 +0000]
pcie: host: tegra: disable aspm-l0s by default

disables aspm-l0s state by default except for those
which are whitelisted.
It also disables CLK-PM if the end point isn't pulling
clkreq down

Bug 200151223
Bug 1687368

Change-Id: I594888b63a9e3b45fff61871b40bca623998e9e4
Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
Reviewed-on: http://git-master/r/826164
(cherry picked from commit fed09295fc4e5182c94414ef59d27ba48fdae44e)
Reviewed-on: http://git-master/r/831188
(cherry picked from commit 2bc1e873f28d2dbb469ae916cd34bcf2caa54f13)
Reviewed-on: http://git-master/r/1165052
(cherry picked from commit c321a58efb4193a4f46a67553bb4fe03519517f6)
Reviewed-on: http://git-master/r/1172625
(cherry picked from commit a80880386b5066c1c1f58b55e1ec7af6b06600ca)
Reviewed-on: http://git-master/r/1178423
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bharat Nihalani <bnihalani@nvidia.com>

3 years agoPCI/ASPM: enable aspm-l1ss support
Vidya Sagar [Mon, 19 Oct 2015 16:05:02 +0000]
PCI/ASPM: enable aspm-l1ss support

enables ASPM-L1 sub-states support

Bug 200143430

Change-Id: I236d09df6ea63d4ae72ff99d976026cc9a1ba933
Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
Reviewed-on: http://git-master/r/819699
(cherry picked from commit c29c8155a63cf296905aa04f2ed1c0382ade2a56)
Reviewed-on: http://git-master/r/822729
(cherry picked from commit 53c3407b88d0964ca9f8653bb3901ccf623d43d9)
Reviewed-on: http://git-master/r/1165051
(cherry picked from commit 4aed2786b41ea74bb6a503674b192f15275baeb7)
Reviewed-on: http://git-master/r/1172624
(cherry picked from commit c0256dde68fb20a33554308e7b9e40fc5e1e5a92)
Reviewed-on: http://git-master/r/1178422
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bharat Nihalani <bnihalani@nvidia.com>

3 years agopcie: host: tegra: fix aspm-l1ss enabling
Vidya Sagar [Fri, 23 Oct 2015 08:12:24 +0000]
pcie: host: tegra: fix aspm-l1ss enabling

fixes the way aspm-l1ss gets enabled when multiple
pcie end points are connected

Bug 200143430

Change-Id: I1dc7fadb7092e8251378a84c3e4df538e5423d3e
Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
Reviewed-on: http://git-master/r/821939
(cherry picked from commit 1f8efcf7faafd3641e9e2dfcb2d1956ff5926017)
Reviewed-on: http://git-master/r/822730
(cherry picked from commit 3c23d02d338c2b616966a4567be07e208fcb0541)
Reviewed-on: http://git-master/r/1165050
(cherry picked from commit 61004b3d58110509c749a7574d3b53297f133e96)
Reviewed-on: http://git-master/r/1172623
(cherry picked from commit 11eeeace1b24a51d1be05eb9b052500ef8ea2d71)
Reviewed-on: http://git-master/r/1178421
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bharat Nihalani <bnihalani@nvidia.com>

3 years agoRevert "pcie: host: tegra: cleanup clkreq dir programming"
Vidya Sagar [Tue, 28 Jun 2016 17:32:50 +0000]
Revert "pcie: host: tegra: cleanup clkreq dir programming"

This reverts commit b57a2766df3545d7f0f3d4229830369643bc18de.

This will be fixed properly along with other fixes in
http://git-master/r/#/c/1172629/

Bug 200210426

Change-Id: I1b11160d333f4957c1ac0296c804cc2379cb9923
Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
Reviewed-on: http://git-master/r/1172622
(cherry picked from commit 9b9c524437b958d186728c224393531a1e04f634)
Reviewed-on: http://git-master/r/1178420
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bharat Nihalani <bnihalani@nvidia.com>

3 years agoFix improper 'const' propagation
Joe Korty [Wed, 25 May 2016 12:51:19 +0000]
Fix improper 'const' propagation

It turns out that data 'data' points to
is always constant, so we can declare that
the pointer 'data' points to constant data
without problem.

drivers/net/wireless/iwlwifi/mvm/fw.c:90:64: warning:
initialization discards 'const' qualifier from
pointer target type [-Wdiscarded-array-qualifiers]
{.size = sizeof(_buf), .data = &_buf}
                                                                ^
drivers/net/wireless/iwlwifi/mvm/fw.c:93:8: note:
in expansion of macro 'CALIB_SIZE_N_DATA'
[9] = CALIB_SIZE_N_DATA(wkp_calib_values_tx_iq_skew),

drivers/net/wireless/iwlwifi/mvm/fw.c:90:64: warning:
initialization discards 'const' qualifier from
pointer target type [-Wdiscarded-array-qualifiers]

drivers/net/wireless/iwlwifi/mvm/fw.c:94:9: note:
in expansion of macro 'CALIB_SIZE_N_DATA'
[11] = CALIB_SIZE_N_DATA(wkp_calib_values_rx_iq_skew),

Bug 200187768

Change-Id: Ib1d0dbd3b44238feb39f54d1ebff00c2edd09254
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163937
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agodrivers: imx219: Fix power on sequence
Bhanu Murthy V [Wed, 15 Jun 2016 23:19:46 +0000]
drivers: imx219: Fix power on sequence

Fix the delay and order for the power on
sequence for IMX219 driver.

Bug 200210065

Change-Id: I55e1ac469c43b3f39739612f730bb9f64c20738f
Signed-off-by: Bhanu Murthy V <bmurthyv@nvidia.com>
Reviewed-on: http://git-master/r/1165292
(cherry picked from commit afae6029c0334537d3ca93784685bfaee41cc50d)
Reviewed-on: http://git-master/r/1177913
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Jihoon Bang <jbang@nvidia.com>

3 years agomedia: tegra_camera: add SoC fops for CSI
Bryan Wu [Sat, 18 Jun 2016 00:16:13 +0000]
media: tegra_camera: add SoC fops for CSI

Add an SoC abstraction layer for for CSI which can be extended to
support different Tegra SoC in the future. Now start with T210.

Bug 1779020

Change-Id: I7c081ccb008bb7775591300495154aaf4505aee5
Signed-off-by: Bryan Wu <pengw@nvidia.com>
Reviewed-on: http://git-master/r/1169883
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Jihoon Bang <jbang@nvidia.com>

3 years agomedia: tegra_camera: add SoC fops for VI
Bryan Wu [Fri, 17 Jun 2016 20:29:48 +0000]
media: tegra_camera: add SoC fops for VI

Add an SoC abstraction layer for for VI which can be extended to
support different Tegra SoC in the future. Now start with T210.

Bug 1779020

Change-Id: I29a8388f9343a1ffc895148b8ffb36c18d7f5c19
Signed-off-by: Bryan Wu <pengw@nvidia.com>
Reviewed-on: http://git-master/r/1169882
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Jihoon Bang <jbang@nvidia.com>

3 years agonet: wireless: bcmdhd: reduce packet timeout to 100ms
Leo He [Fri, 4 Mar 2016 02:57:30 +0000]
net: wireless: bcmdhd: reduce packet timeout to 100ms

bug 200153029

This packet timeout prevents system entering suspend.
This 500ms timeout is too long, which will slow
resume/suspend cycle in WiFi wakeup, so reducing it to 100ms.
Also, BRCM confirmed that it's safe to remove this timeout.

Change-Id: I62dd11e292d3c9fbfe959d76fc510c2115958b7e
Signed-off-by: Leo He <leoh@nvidia.com>
Reviewed-on: http://git-master/r/1023810
(cherry picked from commit e1395c20bf28b4eb5412534d46da9949db6cc8a0)
Reviewed-on: http://git-master/r/1176046
GVS: Gerrit_Virtual_Submit
Reviewed-by: Hayden Du <haydend@nvidia.com>

3 years agoarm64: dts: wdr/non-wdr 30fps modes for imx185
esen chen [Mon, 13 Jun 2016 08:52:41 +0000]
arm64: dts: wdr/non-wdr 30fps modes for imx185

for modes:
IMX185_MODE_1920X1080_CROP_30FPS
IMX185_MODE_1920X1080_CROP_HDR_30FPS

Bug 200168141

Change-Id: If4cbd3e40178596d90b7909de34b0464f2438b4e
Signed-off-by: esen chen <esenc@nvidia.com>
Reviewed-on: http://git-master/r/1163236
(cherry picked from commit a332e5ea8e86133436c4ad39a1f1141b4135c70a)
Reviewed-on: http://git-master/r/1174274
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: David Wang (SW-TEGRA) <davidw@nvidia.com>
Reviewed-by: Jihoon Bang <jbang@nvidia.com>

3 years agodriver: media: i2c: update imx185 driver
esen chen [Fri, 1 Jul 2016 02:01:21 +0000]
driver: media: i2c: update imx185 driver

Added proper equation to translate gain
Update imx185 sensor mode table

Bug 1744523
Bug 200168141

Change-Id: I806a833f5e055c5660f3151562fcd28f622573bc
Signed-off-by: esen chen <esenc@nvidia.com>
Reviewed-on: http://git-master/r/1174271
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: David Wang (SW-TEGRA) <davidw@nvidia.com>
Reviewed-by: Jihoon Bang <jbang@nvidia.com>

3 years agovideo: tegra: dc: fix bit depth of yuv422 mode
Michael Frydrych [Fri, 17 Jun 2016 14:39:30 +0000]
video: tegra: dc: fix bit depth of yuv422 mode

Bit depth of yuv422 mode was set to 10-bits, while it should
have been set to 12-bits. Incorrect bit depth resulted in the
mode not being intercepted when setting yuv422 black.

Bug 200102634
Bug 200182153

Change-Id: I6de42816a9aad92bc2c8bdfad70b1727a49fe94a
Signed-off-by: Michael Frydrych <mfrydrych@nvidia.com>
Reviewed-on: http://git-master/r/1167820
(cherry picked from commit 39aabae1c46710a5dd609690ab12746075640f7f)
Reviewed-on: http://git-master/r/1166991
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Santosh Galma <galmar@nvidia.com>
Reviewed-by: Mitch Luban <mluban@nvidia.com>

3 years agoarch: arm64: kernel: disable the regs dump
Ninad Malwade [Tue, 28 Jun 2016 06:59:44 +0000]
arch: arm64: kernel: disable the regs dump

- When user space app tried to call a syscall
  which is not implemented in kernel, then
  it dumps the process name, syscall number
  and the registers along with PC and LR.
- This leads to kernel_warning GVS failures.
- Since, the regs dump is just for the
  information purspose, it's safe to ignore
  and thus, putting it under CONFIG_DEBUG_USER.
- This is surfaced while trying to port 16.04
  rootfs and it's a must change for switching.

Bug 1731796

Change-Id: I776dd9dccf4d75cca6a3541b7fb64afe37fc4fcb
Signed-off-by: Ninad Malwade <nmalwade@nvidia.com>
Reviewed-on: http://git-master/r/1172352
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Rajkumar Kasirajan <rkasirajan@nvidia.com>
Reviewed-by: Shreshtha Sahu <ssahu@nvidia.com>

3 years agoCast a 'void *' pointer to a 'char *'
Joe Korty [Wed, 25 May 2016 12:42:30 +0000]
Cast a 'void *' pointer to a 'char *'

It is clear from the below that we want to
reference the pointer as a character string,
so cast it as such before using it.

drivers/net/wireless/bcmdhd/dhd_common.c:429:30:
warning: dereferencing 'void *' pointer
     value = *(uint*)&ioc->buf[strlen(param)+1];

Bug 200187768

Change-Id: If6a6cc80b39b03709b55e9e2850e174886593fb6
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163934
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoPointers fit inside unsigned long
Joe Korty [Wed, 25 May 2016 12:12:43 +0000]
Pointers fit inside unsigned long

drivers/net/ethernet/smsc/smc91x.c:
In function 'smc_probe':
drivers/net/ethernet/smsc/smc91x.c:1899:7: warning:
cast from pointer to integer of
different size [-Wpointer-to-int-cast]
if (((unsigned int)ioaddr &
(0x3e0 << SMC_IO_SHIFT)) != val) {

Bug 200187768

Change-Id: I67b452a65f442bfe1b6fe6933a134638424c6acc
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163931
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoFix the return type explicitly
Joe Korty [Wed, 25 May 2016 12:08:14 +0000]
Fix the return type explicitly

gcc does not like longs being compared
to pointers; tell the compiler that this
is OK by hand-casting the long integer to
the required pointer type first.

kernel/sched/core.c: In function 'idle_cpu_relaxed':
kernel/sched/core.c:3863:39: warning:
comparison between pointer and integer
if (cpu_relaxed_read_long(&rq->curr) != rq->idle)

Bug 200187768

Change-Id: Ic6ea947cb4f0af9b24edaebbd2b32eb1a6dfedbe
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163929
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoFix the return type explicitly
Joe Korty [Wed, 25 May 2016 12:06:00 +0000]
Fix the return type explicitly

gcc does not like a long put into a pointer
without an explicit cast.

kernel/mutex.c: In function 'mspin_unlock':
kernel/mutex.c:155:17: warning:
assignment makes pointer from integer
without cast [-Wint-conversion]
while (!(next = cpu_relaxed_read_long(&(node->next))))

Bug 200187768

Change-Id: I5fb1da7fffb4f24173c8fd92ee0291ed1ed188e4
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163928
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoFix __init section mismatch in usb gadget driver
Joe Korty [Wed, 25 May 2016 09:08:19 +0000]
Fix __init section mismatch in usb gadget driver

gcc complains when __init variables or __init functions
are referenced from non-init code, as by the time the
non-init code is run, the __init variables and functions
it references might no longer exist.

Fix by removing the __init from the afunc_bind defition.
This allows it to be called from anywhere, anytime without
problem.

WARNING: drivers/usb/gadget/g_audio.o(.text+0x101c): Section
mismatch in reference from the function audio_bind_config()
to the function .init.text:afunc_bind()
The function audio_bind_config() references
the function __init afunc_bind().
This is often because audio_bind_config lacks a __init
annotation or the annotation of afunc_bind is wrong.

Bug 200187768

Change-Id: I3df30f66a100273b6b1d9b1a29a2ded022027690
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163919
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoCorrect variable declaration
Joe Korty [Wed, 25 May 2016 08:06:24 +0000]
Correct variable declaration

Fix variable declaration in the middle of a
block of C code.Add brackets to make the
declaration appear at the top of its own
subblock of code.

drivers/net/wireless/bcmdhd/dhd_linux.c:7143:2:
warning:ISO C90 forbids mixed declarations
and code [-Wdeclaration-after-statement]

Bug 200187768

Change-Id: I431a47b0c91f9ddf48d20a48d87f646b2f72cea6
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163918
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>

3 years agoCorrect the declaration of variable
Joe Korty [Wed, 25 May 2016 07:51:40 +0000]
Correct the declaration of variable

Do not declare a variable in the middle of a
C code block.Moved the declaration to the front
of the block.

sound/soc/codecs/audience/es-d300.c:
In function '_es_stop_route':
sound/soc/codecs/audience/es-d300.c:294:4:
warning: ISO C90 forbids mixed declarations
and code [-Wdeclaration-after-statement]
u32 cmd = escore->dhwpt_cmd & 0xFFFF0000;

Bug 200187768

Change-Id: I45660f1d958ad56db3204da533791d324a5431a6
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163917
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoUse corretct format specifier
Joe Korty [Wed, 25 May 2016 07:38:03 +0000]
Use corretct format specifier

cur_state is of size long, not int,
so use %ld rather than %d to print it.

drivers/thermal/adaptive_skin.c:695:11: warning:
format '%d' expects argument of type 'int',
but argument 3 has type 'long unsigned int' [-Wformat=]

Bug 200187768

Change-Id: Ie3035384ae0c0e3358ae8ac0a7f402dcdbb202b5
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163916
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoFix integer size errors in printf format
Joe Korty [Wed, 25 May 2016 07:36:52 +0000]
Fix integer size errors in printf format

drivers/mmc/core/debugfs.c:544:44: warning:
format '%u' expects argument of type 'unsigned int',
but argument 2 has type 'long long unsigned int' [-Wformat=]
drivers/mmc/core/debugfs.c:553:45: warning:
format '%u' expects argument of type 'unsigned int',
but argument 2 has type 'long long unsigned int' [-Wformat=]
drivers/mmc/core/debugfs.c:562:42: warning:
format '%u' expects argument of type 'unsigned int',
but argument 2 has type 'long long unsigned int' [-Wformat=]
drivers/mmc/core/debugfs.c:572:41: warning:
format '%u' expects argument of type 'unsigned int',
but argument 2 has type 'long long unsigned int' [-Wformat=]

Bug 200187768

Change-Id: Iec7e0de8b01dac30713cb7e545b5ada98b5ffb1c
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163915
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoCorrect format specifier
Joe Korty [Wed, 25 May 2016 07:34:27 +0000]
Correct format specifier

%d -> %zd printk format statement transformation

sound/soc/codecs/audience/escore-vs.c:870:24:
warning: format '%d' expects argument of type 'int',
but argument 5 has type 'size_t
{aka long unsigned int}' [-Wformat=]

Bug 200187768

Change-Id: I809a198fdb14a46dac1e9dbee89dda225b46df66
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163914
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoUse atomic_read to access atomic var
Joe Korty [Wed, 25 May 2016 07:32:13 +0000]
Use atomic_read to access atomic var

Also fix a few format statement mismatches,
%u -> %lu.

sound/soc/codecs/audience/es-d300.c:1317:12: warning:
format '%d' expects argument of type 'int',
but argument 5 has type 'atomic_t
{aka struct <anonymous>}' [-Wformat=]
sound/soc/codecs/audience/es-d300.c:1332:12: warning:
format '%d' expects argument of type 'int',
but argument 5 has type 'atomic_t
{aka struct <anonymous>}' [-Wformat=]
sound/soc/codecs/audience/es-d300.c:1355:12: warning:
format '%d' expects argument of type 'int',
but argument 5 has type 'atomic_t
{aka struct <anonymous>}' [-Wformat=]
sound/soc/codecs/audience/es-d300.c:1386:12: warning:
format '%d' expects argument of type 'int',
but argument 5 has type 'atomic_t
{aka struct <anonymous>}' [-Wformat=]
sound/soc/codecs/audience/es-d300.c:1398:12: warning:
format '%d' expects argument of type 'int',
but argument 5 has type 'atomic_t
{aka struct <anonymous>}' [-Wformat=]
sound/soc/codecs/audience/es-d300.c:1412:12: warning:
format '%d' expects argument of type 'int',
but argument 5 has type 'atomic_t
{aka struct <anonymous>}' [-Wformat=]
sound/soc/codecs/audience/es-d300.c:1433:12: warning:
format '%d' expects argument of type 'int',
but argument 5 has type 'atomic_t
{aka struct <anonymous>}' [-Wformat=]
include/linux/kern_levels.h:4:18: warning:
format '%d' expects argument of type 'int',
but argument 4 has type 'long unsigned int' [-Wformat=]
include/linux/kern_levels.h:4:18: warning:
format '%d' expects argument of type 'int',
but argument 4 has type 'long unsigned int' [-Wformat=]

.... plus others, most %d -> %zd transformations.

Bug 200187768

Change-Id: I891f626c63343ada222ea6375213a799a6262ff2
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163913
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoCorrect format specifier
Joe Korty [Wed, 25 May 2016 07:29:05 +0000]
Correct format specifier

d -> %zd format statement transformation, fixes

sound/soc/codecs/audience/es755.c:
In function 'codec_reg_dump_store':
sound/soc/codecs/audience/es755.c:528:11:
warning: format '%d' expects argument of type 'int',
but argument 5 has type 'size_t
{aka long unsigned int}' [-Wformat=]
pr_debug("%s(): requested - %s, count=%d\n",
__func__, buf, count);

Bug 200187768

Change-Id: I661f46104bc6740e993e84f95b575719d7978733
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163912
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agopr_err requires size_t be printed with %zu
Joe Korty [Wed, 25 May 2016 07:27:18 +0000]
pr_err requires size_t be printed with %zu

drivers/usb/gadget/rndis.c:
In function 'rndis_rm_hdr':
include/linux/kern_levels.h:4:18: warning:
format '%u' expects argument of type 'unsigned int',
but argument 3 has type 'long unsigned int' [-Wformat=]

Bug 200187768

Change-Id: Ie44cf041e9f814e607d6f91949f64b1a1e9e6cd6
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163911
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agolongs are printed with %lx
Joe Korty [Wed, 25 May 2016 07:22:41 +0000]
longs are printed with %lx

sound/pci/hda/hda_intel.c:
In function 'azx_single_wait_for_response':
sound/pci/hda/hda_intel.c:208:13: warning:
format '%x' expects argument of
type 'unsigned int',
but argument 3 has type 'long unsigned int' [-Wformat=]

sound/pci/hda/hda_intel.c:1147:14:
note: in expansion of macro 'SFX'
snd_printd(SFX "%s: get_response timeout: IRS=0x%x\n",

sound/pci/hda/hda_intel.c:
In function 'azx_single_send_cmd':
sound/pci/hda/hda_intel.c:208:13: warning:
format '%x' expects argument of type 'unsigned int',
but argument 3 has type 'long unsigned int' [-Wformat=]

sound/pci/hda/hda_intel.c:1175:14:
note: in expansion of macro 'SFX'
snd_printd(SFX "%s: send_cmd timeout: IRS=0x%x, val=0x%x\n",

Bug 200187768

Change-Id: Ib8985940ea39544dea1075a2a6aa8f0a7006993f
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163910
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoPointer types are printed with %p
Joe Korty [Wed, 25 May 2016 07:19:59 +0000]
Pointer types are printed with %p

drivers/misc/gps/gps.c:828:3: note:
in expansion of macro 'GPS_DBG'
GPS_DBG("copy_to_user error: 0x%X 0x%X, %d\n",
(unsigned int)buf, (unsigned int)dev->dat_buf,
dev->dat_len);

Bug 200187768

Change-Id: Id0c634639e1a94639cc86051ce0b37dcaf1dfbd7
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163909
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agodrivers: IMX274: fix AE blink in manual exposure
Josh Kuo [Wed, 11 May 2016 06:57:04 +0000]
drivers: IMX274: fix AE blink in manual exposure

- remove digital gain setting from mode tables
- set to 4k60fps

Bug 200192121

Change-Id: I6bcfbed25d1eaba842adef446561891bff50ca2b
Signed-off-by: Josh Kuo <joshk@nvidia.com>
Reviewed-on: http://git-master/r/1166527
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Hayden Du <haydend@nvidia.com>

3 years agoCorrect g_ether module init & exit fn
Joe Korty [Wed, 25 May 2016 11:55:45 +0000]
Correct g_ether module init & exit fn

Replace u_ether.c's use of __initcall and __exitcall
with direct invocations of its init and exit function
from ether.c's module_init and module_exit functions.

We have to do this as 1) __initcall cannot be used
from within a module, and 2) module_init cannot be
declared more than once inside any .c file, and
3) ether.c #includes u_ether.c.

Fortunately, ether.c unconditionally includes u_ether.c,
so we can unconditionally add the u_ether.c's init code
directly to ether.c's init function.

include/linux/init.h:298:27:
error: redefinition of '__inittest'
static inline initcall_t __inittest(void)

include/linux/init.h:291:30: note:
in expansion of macro 'module_init'

drivers/usb/gadget/u_ether.c:1216:1: note:
in expansion of macro 'device_initcall'
device_initcall(gether_exit);

include/linux/init.h:298:27: note:
previous definition of '__inittest' was here
static inline initcall_t __inittest(void)

include/linux/init.h:291:30: note:
in expansion of macro 'module_init'

drivers/usb/gadget/u_ether.c:1209:1: note:
in expansion of macro 'device_initcall'
device_initcall(gether_init);

drivers/usb/gadget/u_ether.c: In function '__inittest':
drivers/usb/gadget/u_ether.c:1216:17: warning:
return from
incompatible pointer type [-Wincompatible-pointer-types]
device_initcall(gether_exit);

include/linux/init.h:299:11:
note: in definition of macro 'module_init'
  { return initfn; }

Bug 200187768

Change-Id: I37262041ae79aa2c1672499f735517b53ae5fc45
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163924
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>

3 years agoFix the explicit cast
Joe Korty [Wed, 25 May 2016 12:15:05 +0000]
Fix the explicit cast

gcc requires that when an integer value is being
returned through a pointer, that value be explicity
cast first to a pointer type.

drivers/usb/gadget/u_ether.c:
In function 'gether_setup_name':
drivers/usb/gadget/u_ether.c:963:24: warning:
return makes pointer from integer
without a cast [-Wint-conversion]
return -ENOMEM;

Bug 200187768

Change-Id: I19d6abbd12c812779587fd097a3277ca3f91caf4
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163932
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>

3 years agoFix unwanted #ifdef in swtich case
Joe Korty [Wed, 25 May 2016 12:04:04 +0000]
Fix unwanted #ifdef in swtich case

SCAN_SUSPEND_ABORT is always defined in the
enumeration,so no need to '#ifdef' out its
use in those kernels for which this feature
is not compiled in.

net/mac80211/scan.c: In function 'ieee80211_scan_work':
net/mac80211/scan.c:758:3: warning:
enumeration value 'SCAN_SUSPEND_ABORT'
not handled in switch [-Wswitch]
switch (local->next_scan_state) {

Bug 200187768

Change-Id: Ic30505196c5e676b10be64ed5e4d4e16d77c5b30
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163927
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoInclude required header file
Joe Korty [Wed, 25 May 2016 12:02:34 +0000]
Include required header file

Insert missing include file, felt
when CONFIG_COMPAT=n

drivers/devfreq/governor_wmark_active.c:
In function 'devfreq_watermark_target_freq':
drivers/devfreq/governor_wmark_active.c:146:25:
error: implicit declaration of function
'ktime_get' [-Werror=implicit-function-declaration]
ktime_t current_time = ktime_get();

drivers/devfreq/governor_wmark_active.c:146:25:
error: invalid initializer

Bug 200187768

Change-Id: I0b2deda3cbbc9b2af5c4846e0a20614eadb04353
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163926
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoInclude the required header file
Joe Korty [Wed, 25 May 2016 11:58:40 +0000]
Include the required header file

Insert missing #include file, felt
when CONFIG_COMPAT=n.

drivers/firmware/tegra/bpmp.c:
In function 'bpmp_ping_show':
drivers/firmware/tegra/bpmp.c:465:7: error:
implicit declaration of
function 'ktime_get' [-Werror=implicit-function-declaration]
tm = ktime_get();

drivers/firmware/tegra/bpmp.c:465:5: error:
incompatible types when assigning to type 'ktime_t
{aka union ktime}' from type 'int'
tm = ktime_get();

include/linux/ktime.h:85:30: error: request for member
'tv64' in something not a structure or union
({ (ktime_t){ .tv64 = (lhs).tv64 - (rhs).tv64 }; })
                              ^
drivers/firmware/tegra/bpmp.c:468:7: note: in expansion
of macro 'ktime_sub'
tm = ktime_sub(ktime_get(), tm);

Bug 200187768

Change-Id: I4c355276560971eb98ab1ffa87713e665d857457
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163925
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoFix trace_sched_switch_end definition
Gaurav Singh [Wed, 25 May 2016 11:46:34 +0000]
Fix trace_sched_switch_end definition

We model what we do here on other empty (no arg)
tracepoint function definitions in the kernel.

In file included from include/trace/define_trace.h:86:0,
from include/trace/events/sched.h:452,
from kernel/sched/core.c:94:
include/trace/events/sched.h:
In function 'ftrace_define_fields_sched_switch_end':
include/trace/ftrace.h:327:27:
warning: unused variable 'field' [-Wunused-variable]
struct ftrace_raw_##call field;

include/trace/ftrace.h:32:2:
note: in expansion of macro 'DECLARE_EVENT_CLASS'
DECLARE_EVENT_CLASS(name,

include/trace/events/sched.h:155:1:
note: in expansion of macro 'TRACE_EVENT'
TRACE_EVENT(sched_switch_end,

Bug 200187768

Change-Id: I4d5a2b4717d62ca90018d0e3847c185d7cc590ac
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163923
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoFix return type of xchg()
Joe Korty [Wed, 25 May 2016 09:17:20 +0000]
Fix return type of xchg()

The kernel does not like it when the return
value of xchg() is not being used, so tell
it when that is indeed the proper use.

drivers/media/platform/tegra/camera.c:
In function 'camera_remove':
/cuba/jak/tegra/r23.1/a/arch/arm64/include/asm/cmpxchg.h:75:3:
warning: value computed is not used [-Wunused-value]
((__typeof__(*(ptr)))__xchg((unsigned long)(x),
(ptr),sizeof(*(ptr))))

/cuba/jak/tegra/r23.1/a/arch/arm64/include/asm/atomic.h:198:30:
note: in expansion of macro 'xchg'

drivers/media/platform/tegra/camera.c:1201:2:
note: in expansion of macro 'atomic_xchg'
atomic_xchg(&cam_desc.in_use, 0);

drivers/media/platform/tegra/camera.c:
In function 'camera_shutdown':
/cuba/jak/tegra/r23.1/a/arch/arm64/include/asm/cmpxchg.h:75:3:
warning: value computed is not used [-Wunused-value]
((__typeof__(*(ptr)))__xchg((unsigned long)(x),
(ptr),sizeof(*(ptr))))

Bug 200187768

Change-Id: Iacc15eec6ab5e72dd3d671376a68971bd307fac3
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163922
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoCorrect arguments declaration
Joe Korty [Wed, 25 May 2016 09:13:55 +0000]
Correct arguments declaration

include/linux/spinlock.h:198:3: note:
in expansion of macro 'typecheck'
typecheck(unsigned long, flags);

include/linux/spinlock.h:323:2: note:
in expansion of macro 'raw_spin_lock_irqsave'
raw_spin_lock_irqsave(spinlock_check(lock), flags);

drivers/mmc/core/core.c:407:2: note:
in expansion of macro 'spin_lock_irqsave'
spin_lock_irqsave(&host->que_lock, flags);

include/linux/typecheck.h:11:18: warning:
comparison of distinct pointer types lacks a cast
(void)(&__dummy == &__dummy2);
                  ^
include/linux/spinlock.h:198:3: note:
in expansion of macro 'typecheck'
typecheck(unsigned long, flags);

include/linux/spinlock.h:323:2: note:
in expansion of macro 'raw_spin_lock_irqsave'
raw_spin_lock_irqsave(spinlock_check(lock), flags);

drivers/mmc/core/core.c:480:2: note:
in expansion of macro 'spin_lock_irqsave'
spin_lock_irqsave(&host->que_lock, flags);

Bug 200187768

Change-Id: I1340ceda88676f6f3a7ce95d951dee187fffba96
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163921
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agofix incompatible ptr type
Joe Korty [Wed, 25 May 2016 09:12:26 +0000]
fix incompatible ptr type

The '&length' argument must be a long*, not an int*.

drivers/of/of_reserved_mem.c:215:54: warning:
passing argument 3 of 'of_get_flat_dt_prop' from
incompatible pointer type [-Wincompatible-pointer-types]
drivers/of/of_reserved_mem.c:215:54: warning:
passing argument 3 of 'of_get_flat_dt_prop' from
incompatible pointer type [-Wincompatible-pointer-types]
prop = of_get_flat_dt_prop(node, "linux,phandle", &len);

In file included from drivers/of/of_reserved_mem.c:18:0:
include/linux/of_fdt.h:88:14: note:
expected 'long unsigned int *' but argument is of type 'int *'
extern void *of_get_flat_dt_prop(unsigned long node,
const char *name,

Bug 200187768

Change-Id: Iaa3a60a9fbae620d21fb21d779ccd3e2c1229d6f
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163920
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoPrint size_t variables with %zd
Joe Korty [Wed, 25 May 2016 07:17:45 +0000]
Print size_t variables with %zd

size_t variables to be used with %zd,
not %d and %zu, not %u.

Bug 200187768

Change-Id: I824f88635de1cddeaaa74f7031f88f91788fb3a1
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163908
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoFormat size_t with %zd, not %d
Joe Korty [Wed, 25 May 2016 07:12:50 +0000]
Format size_t with %zd, not %d

drivers/media/platform/tegra/ov9772.c:2223:3:
note: in expansion of macro 'dev_dbg'
dev_dbg(&info->i2c_client->dev, "%s CAPS_RD n=%d\n",

Bug 200187768

Change-Id: I88c9ca52ab9b0387a92ff6046e7518ecca59c354
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163907
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoCorrect format specifier
Joe Korty [Wed, 25 May 2016 06:48:55 +0000]
Correct format specifier

u64 data types are displayed with %llu, not %u

drivers/mmc/core/debugfs.c:498:39: warning:
format '%u' expects argument of type 'unsigned int',
but argument 2 has type 'long long unsigned int'
[-Wformat=] mmc_dbg_card_speed_class_get, NULL, "%u\n");
                                       ^
include/linux/fs.h:2667:29: note:
in definition of macro 'DEFINE_SIMPLE_ATTRIBUTE'
  __simple_attr_check_format(__fmt, 0ull);  \

Bug 200187768

Change-Id: I6bfabdf34eb318320a85ad30c2a1fb3f6afd0193
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163905
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoCorrect format specifier
Joe Korty [Wed, 25 May 2016 06:45:43 +0000]
Correct format specifier

Use %zd, not %d, for display of size_t in printk's

drivers/tty/tty_buffer.c:
In function 'tty_insert_flip_string_fixed_flag_lock':
include/linux/kern_levels.h:4:18: warning:
format '%d' expects argument of type 'int', but
argument 3 has type 'size_t {aka long unsigned int}'
[-Wformat=]

drivers/tty/tty_buffer.c:354:4: note: in expansion of
macro 'pr_err'
pr_err("%s: No space: Requested %d, memory used: %d\n",

Bug 200187768

Change-Id: I5c8937a4786343325231800d7be58ef0205aae6f
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163904
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoRemove unused variable 'p_cmd'
Joe Korty [Wed, 25 May 2016 06:44:00 +0000]
Remove unused variable 'p_cmd'

sound/soc/codecs/audience/escore.c:
In function 'escore_wakeup':
sound/soc/codecs/audience/escore.c:830:6:
warning: unused variable 'p_cmd' [-Wunused-variable]
u32 p_cmd = ES_GET_POWER_STATE << 16;

Bug 200187768

Change-Id: I17b73812270cc1c0610516009c8f64fa7f14eb6d
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163903
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoRemove unused variable
Joe Korty [Wed, 25 May 2016 06:42:14 +0000]
Remove unused variable

net/core/sysctl_net_core.c:24:12:
warning: 'one' defined but not used [-Wunused-variable]

Bug 200187768

Change-Id: I07436c7ac789ad6a7fe9fbbb2019b24df82cabbc
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163902
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoGive ampud_status an initial value of zero.
Joe Korty [Wed, 25 May 2016 06:40:39 +0000]
Give ampud_status an initial value of zero.

It is not actually used anywhere its value
is passed to, so any value would do.

drivers/net/wireless/iwlwifi/mvm/rx.c:
In function 'iwl_mvm_rx_rx_mpdu':
drivers/net/wireless/iwlwifi/mvm/rx.c:362:2:
warning: 'ampdu_status' may be used uninitialized
in this function [-Wmaybe-uninitialized]
iwl_mvm_pass_packet_to_mac80211(mvm, hdr, len, ampdu_status,

Bug 200187768

Change-Id: Ie30c23dd8fbcb8c1e5851cdf69b56666df302ce0
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163901
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoRemove unused label
Joe Korty [Wed, 25 May 2016 06:39:09 +0000]
Remove unused label

net/wireless/nl80211.c:
In function 'cfg80211_ch_switch_notify':
net/wireless/nl80211.c:10425:1: warning:
label 'out' defined but not used [-Wunused-label]

Bug 200187768

Change-Id: Ida1bd16d9afdc976d4aab77875975d613d23b2ee
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163900
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoUse #ifdef for the functions
Joe Korty [Wed, 25 May 2016 06:37:28 +0000]
Use #ifdef for the functions

Put the same '#ifdef' around the definition of
a function that is present around its use, so
that both definition and use are either present
or not present in the resulting binary.

drivers/base/cpu.c:291:12: warning:
'cpu_uevent' defined but not used [-Wunused-function]

Bug 200187768

Change-Id: I44201a9561ab7407777fb40134306b1c2868ea4f
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163899
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoFix minor gcc problems in rtrace.c
Joe Korty [Wed, 25 May 2016 06:35:04 +0000]
Fix minor gcc problems in rtrace.c

1) when storing a pointer into an int,
tell the compiler that this
is OK by casting the pointer first to an int.

2) remove unused variable 'phys'.
This is not being used even under #ifdef.

fs/pstore/rtrace.c:58:13: warning: assignment makes
integer from pointer without a cast [-Wint-conversion]
fs/pstore/rtrace.c:48:16: warning:
unused variable 'phys' [-Wunused-variable]

Bug 200187768

Change-Id: I590fe1f4b1af24e2ac9fd7f023a06c8a49f1b4bc
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163898
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoRemove unused label
Joe Korty [Tue, 24 May 2016 11:19:08 +0000]
Remove unused label

Wunused-label is a stupid gcc option as removing the
warnings it produces often makes the code harder to
read. But such is life ....

fs/pstore/ram.c:633:1: warning: label 'fail_init_fprz'
defined but not used [-Wunused-label]
fs/pstore/ram.c:629:1: warning: label 'fail_init_mprz'
defined but not used [-Wunused-label]

Bug 200187768

Change-Id: Ia9756fc029cf5882ee89fb871aabecae036d439a
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163897
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoDelete unused function.
Joe Korty [Tue, 24 May 2016 11:01:23 +0000]
Delete unused function.

Clearly, the intent at one time was for it to be used,
but either the code that used it got deleted, or
compat_binfmt_elf.c never got the feature that was
expected to use it.

fs/compat_binfmt_elf.c: At top level:
fs/compat_binfmt_elf.c:58:13: warning:
'cputime_to_compat_timeval' defined but not used
[-Wunused-function]
static void cputime_to_compat_timeval(const cputime_t cputime,

Bug 200187768

Change-Id: I23eaf92cc988105e559b67238799d864bd62600f
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163896
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>

3 years agoRemove unused label
Joe Korty [Tue, 24 May 2016 10:53:38 +0000]
Remove unused label

drivers/mmc/core/debugfs.c:
In function 'mmc_get_ext_csd_byte_val':
drivers/mmc/core/debugfs.c:519:1: warning:
label 'out_free' defined but not used
[-Wunused-label] out_free:

Bug 200187768

Change-Id: I7c16c6ab716032dc1c894d81c3f564426210b2fc
Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Gaurav Singh <gaursingh@nvidia.com>
Reviewed-on: http://git-master/r/1163895
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>