Add a dummy clock provider that registers a simple callback that
in turn always returns the dummy clock for any clk_get call.
In addition, consumers of the dummy clock provider would require
support for dummy resets, which would make a dummy assert/deassert
of the reset signals.
Change-Id: I08fcb174fd0e0c49f8069e106b48597bcdfe847d
Signed-off-by: Taniya Das <tdas@codeaurora.org>
Signed-off-by: Deepak Katragadda <dkatraga@codeaurora.org>
Signed-off-by: David Dai <daidavid1@codeaurora.org>
This is a snapshot of msm_rtb.h file taken as of msm-4.19
commit <cf8acfa8c9b53ad> (Merge "msm: camera: core : Validate
the dev name during the node ioctl handler").
Change-Id: Ia2a2147369328ca6da177bd51c20217df3c4f157
Signed-off-by: Raghavendra Rao Ananta <rananta@codeaurora.org>
Enable the stub-regulator driver so that regulator consumers can
successfully request handles to stub-regulator devices used on
Lahaina boards.
Change-Id: I5d406a068cbd06e29f3fd6d2801779b1ad8e6564
Signed-off-by: David Collins <collinsd@codeaurora.org>
Add a header file for the RPMh regulator voltage level constants
to be used in device tree regulator configurations.
Change-Id: Ia0c47c245bc1d2f005c785a702638579cf545804
Signed-off-by: David Collins <collinsd@codeaurora.org>
Add a stub-regulator driver which can be used to simulate any
particular regulator. This stub regulator driver must support
regulator_set_voltage() which is not possible via the existing
dummy regulator. Stub regulator devices can be used to verify
that regulator consumers make correct regulator framework calls
before it is possible to implement true regulator control.
Change-Id: Ia73d245419e44d74712b813e58de1973112233c4
Signed-off-by: David Collins <collinsd@codeaurora.org>
Vendor device tree directories shouldn't show up in the kernel git.
Add device tree vendor directories to gitignore to accomplish this.
Change-Id: I1697b38a0a018ea61eefd0d3e344dd375604388a
Signed-off-by: Venkata Narendra Kumar Gutta <vnkgutta@codeaurora.org>
Signed-off-by: Isaac J. Manjarres <isaacm@codeaurora.org>
Enable tech packages to have kernel drivers that are physically
located in the kernel source tree but not present in the kernel
repository. This is done via a manifest line item to fetch a
techpackage-module into $KERNEL/techpack/$techpackage-module.
$KERNEL/techpack/Kbuild will automatically detect any
subdirectories and link them to the kernel's kbuild system.
The resulting layout of techpack within kernel source would be as
follows :-
kernel/
└── techpack/
├── Kbuild
├── stub/
│ ├── Makefile
│ ├── include/
│ │ └── uapi/
│ │ └── Kbuild
│ └── stub.c
└── techpackage-module/
├── Makefile
└── include/
└── uapi/
└── Kbuild
$KERNEL/techpack only contains Kbuild (no Makefile) at the
toplevel and this Kbuild takes care of both compiling the
subdirectories and of exporting the needed header files therein.
The reason for having only Kbuild at the top is that Kbuild
and Makefile cannot exist together in same directory and Makefile
doesn't cater to uapi header installation.
stub is an empty techpackage-module which serves both as a sample
layout and satisfies the requirement of kernel build system, by
providing necessary buit-in.o, when no other techpackage-module has
been pulled under techpack/.
$KERNEL/techpack/techpackage-module should have a Makefile at the
top and Kbuild under $KERNEL/techpack/techpackage-module/include/uapi
directory. The uapi headers pertaining to a techpackage-module
should reside under $KERNEL/techpack/techpackage-module/include/uapi
and associated Kbuild should have necessary rules to export it.
Change-Id: I0d0ced38566907d2074831edde0934833f666eff
Signed-off-by: Bryan Huntsman <bryanh@codeaurora.org>
CONFIGs that are unset in base defconfig (# CONFIG_X is not set),
but gets enabled in fragments through menuconfig, the diff is shown
as: -# CONFIG_X is not set. However, for the fragments to work
correctly, it must be set to 'y'. Hence, explicitly set them in the
config fragments.
Change-Id: Ic1c1578fb8109c09f3f39b9d254474d2c341a6d6
Signed-off-by: Raghavendra Rao Ananta <rananta@codeaurora.org>
ABI snapshot comparision is failing against any new gerrit which is
checked-in. This is due to ABI snapshot is not maintaining the full
paths to the source files in the ABI dump. Also several other fixes
are made in the libabigail upstream master branch which are addressing
several fixes. Hence update the ABI snapshot to fix the issues seeing
in the kernel automation.
Snapshot is taken based on kernel/build project.
https://android.googlesource.com/kernel/build/+/46f9bd0a1e800e424612231670e605393e2ba4ac
Change-Id: I2acd58d2093fa597a8999356e36f2b8aa10356b3
Signed-off-by: Murali Nalajala <mnalajal@codeaurora.org>
Add clock ID constants for all Qualcomm Technologies, Inc. Lahaina
clock controllers which consumers can use to reference individual
clocks.
Change-Id: Id60f27989a773b02ff08278eb430a3314132198c
Signed-off-by: David Dai <daidavid1@codeaurora.org>
The code under the techpack directory is maintained
in different repositories, and thus, should be ignored
by the kernel git repository, so add the techpack
directory to the gitignore.
Change-Id: Id8f737e3ddf36ca983b81e62bf7b881d61916353
Signed-off-by: Isaac J. Manjarres <isaacm@codeaurora.org>
Exclude the files generated by GKI scripts, such as:
arch/*/configs/vendor/*-gki_defconfig
arch/*/configs/vendor/*-qgki-debug_defconfig
arch/*/configs/vendor/*-qgki_defconfig
These defconfigs are auto-generated at build time,
and should not be modified by the developers.
Change-Id: Ia965c23e2358b9ab0479b7415e2a60af5b892cd0
Signed-off-by: Raghavendra Rao Ananta <rananta@codeaurora.org>
Previous ABI snapshot comparison is failing when it is
compared against new code changes due to the environment
where the snapshot is built. Update the snapshot based
on new code in a correct build environment.
Change-Id: Ie49d93981c08359b8e96522738e21ad416e4a097
Signed-off-by: Murali Nalajala <mnalajal@codeaurora.org>
Continuation headers baffle checkpatch, as it can only operate
on one line of context at a time. When continuation headers are found,
put them up with the header they're continuing so the whole thing
can be parsed in a single line of context.
Change-Id: I2d22ed056f8203ae6dae473b01ae66bc1963ae75
Signed-off-by: Gregory Bean <gbean@codeaurora.org>
Signed-off-by: Rishabh Bhatnagar <rishabhb@codeaurora.org>
Signed-off-by: Raghavendra Rao Ananta <rananta@codeaurora.org>
Initial ABI definition file which was checked in is not
based on "gki_defconfig". Which is generated with ARCH_QCOM
enabled. This adds extra QCOM functions which we do not want
to care about. Intention of ABI checker is to figure out
whether the base ABI file is modified/removed/impacted. If
there are any additions (functions/structures) that happened
and if it does not impact the base ABI then it should be ok
from the ABI checker perspective.
Change-Id: Ic9eb7fad33ab6f076f637523d21d33451a369068
Signed-off-by: Murali Nalajala <mnalajal@codeaurora.org>
smp_processor_id() complains if called from a preemptible
context. Hence, disable the preemption just before calling
this and enable it back later.
Change-Id: I6721796d11186eb15021b37d5ea8250afa0edda0
Signed-off-by: Raghavendra Rao Ananta <rananta@codeaurora.org>
Enable ARM JTAG DCC driver to get the kernel logs on trace32
term window.
Change-Id: Ic0e9d0d89c82d43985dd123a6c5e13287a2fbf0f
Signed-off-by: Raghavendra Rao Ananta <rananta@codeaurora.org>
Some debuggers, such as Trace32 from Lauterbach GmbH, do not handle
reads/writes from/to DCC on secondary cores. Each core has its
own DCC device registers, so when a core reads or writes from/to DCC,
it only accesses its own DCC device. Since kernel code can run on
any core, every time the kernel wants to write to the console, it
might write to a different DCC.
In SMP mode, Trace32 only uses the DCC on core 0. In AMP mode, it
creates multiple windows, and each window shows the DCC output
only from that core's DCC. The result is that console output is
either lost or scattered across windows.
Selecting this option will enable code that serializes all console
input and output to core 0. The DCC driver will create input and
output FIFOs that all cores will use. Reads and writes from/to DCC
are handled by a workqueue that runs only core 0.
Change-Id: I1e83d254ff2feaf235d212571ed791a9e84e547f
Acked-by: Adam Wallis <awallis@codeaurora.org>
Signed-off-by: Shanker Donthineni <shankerd@codeaurora.org>
Signed-off-by: Kyle Yan <kyan@codeaurora.org>
Signed-off-by: Rishabh Bhatnagar <rishabhb@codeaurora.org>
Add initial ABI definition file based on gki_defconfig.
This denition file is used in the automation while the
new changes are merge into the kernel. This process
detects ABI break if the new changes are introduced
any API/structure changes which could impact the other
exported functions or modules.
This initial snapshot is taken from the commit 553049f.
Initial definition present in the current directory is
also based on gki_config i.e abi_gki_aarch64.out, But
this definition can't be used for ABI comparision because
of compiler being used to generate this file could be
different. Hence generate the one by our own and maintain
the same.
Change-Id: I3a5a744e81a004659065c1b44c98baddfa242fbb
Signed-off-by: Murali Nalajala <mnalajal@codeaurora.org>
If code is merged upstream first it is possible that the same author
may end up attempting to merge the same code into a downstream kernel.
Since code coming from upstream is always a backport and may involve
modifications it is appropriate for the author to add another
Signed-off-by line to the bottom of the commit log but this will
cause checkpatch to complain about duplicate signatures. By making
the duplicate signature warning a different error class
(DUPLICATE_SIGN_OFF instead of BAD_SIGN_OFF) this will let us ignore
this error with --ignore DUPLICATE_SIGN_OFF without losing the benefits
of the other signature checks.
Change-Id: Ic0dedbadcc1b319061db324f88740e5320bfff09
Signed-off-by: Jordan Crouse <jcrouse@codeaurora.org>
Exempt copyright statements from the 80 column rule as line-splitting or
other kinds of work-arounds decrease readability in diffs. Identify the
copyright statement by looking for '+/* Copyright (c)', case-insensitive,
at the beginning of the line.
Change-Id: I57aafd569ff2173ca7aa3f636532060fc93b2be1
Signed-off-by: Bryan Huntsman <bryanh@codeaurora.org>
Upstream has merged a few commit text length checks
that make our own checks redundant. Furthermore,
upstream also has a newer exception list (such as
allowing Fixes: lines to be >75 characters) that
we should take advantage of.
Change-Id: I894482c1a4182ad71146d5d433610d4cb210cb1d
Signed-off-by: Vikram Mulukutla <markivx@codeaurora.org>
Signed-off-by: Rishabh Bhatnagar <rishabhb@codeaurora.org>
Rather than forcing authors of reverts to truncate the summary line
because the "Revert" prefix added pushes it over the character limit,
excuse reverts from this rule.
Change-Id: I395dfff3327e360ef935d4a685c38df6577e3867
Signed-off-by: Matt Wagantall <mattw@codeaurora.org>
Signed-off-by: Stepan Moskovchenko <stepanm@codeaurora.org>
Signed-off-by: Rishabh Bhatnagar <rishabhb@codeaurora.org>
Commit text is almost always necessary to explain why a change is
needed. Exceptions are rare enough that these can be granted through
manual checkpatch overrides.
Change-Id: I926b2276f717940c2fec77a6709fa3088b1bf0c3
Signed-off-by: Steve Muckle <smuckle@codeaurora.org>
Signed-off-by: Stepan Moskovchenko <stepanm@codeaurora.org>
Signed-off-by: Rishabh Bhatnagar <rishabhb@codeaurora.org>
When patches contain extended character sets, patches will contain
MIME headers after the subject line, which should not be confused
for a too-long summary line.
Change-Id: I66f70b465ef804444a3bdb49867654b5613b13a7
Signed-off-by: Gregory Bean <gbean@codeaurora.org>
(cherry picked from commit 8e6b9d3790595198a34320f1c3f4504cd258fed1)
Signed-off-by: Stepan Moskovchenko <stepanm@codeaurora.org>
Signed-off-by: Rishabh Bhatnagar <rishabhb@codeaurora.org>
Checkpatch is throwing a warning for lines that contains file delta
in the commit log.
WARNING: Possible unwrapped commit description
(prefer a maximum 75 chars per line)
kerneldir/msm-abc-xyz123.dtsi
| 68 ++++++++++++++++++++++++++++++++++++++++----
This file delta line of the patch should be ignored for maximum characters
per line check and is taken care of already, but it's not covering if the
file has '-' in its name. Update the conditional check to cover this case.
Change-Id: If37691aa897f8a991c1643a08fbde45cc4afaaf7
Signed-off-by: Venkata Narendra Kumar Gutta <vnkgutta@codeaurora.org>
Warn on summary or commit text lines greater than 75 characters.
The summary and commit text are indented and may wrap on a terminal
if they are longer than 75 characters.
Change-Id: I37b13dc870cb797980ffdd978ebf3e15bb624afa
Signed-off-by: David Keitel <dkeitel@codeaurora.org>
Signed-off-by: Rishabh Bhatnagar <rishabhb@codeaurora.org>
Now that mb() does what we want, dsb() should be discouraged.
Change-Id: Ib8fe8f44f669753c3d91fac3c6e598e117d6d90e
Signed-off-by: Gregory Bean <gbean@codeaurora.org>
(cherry picked from commit 9c0619be7b93ad114d6f33a749d905ddff93df7d)
Signed-off-by: Stepan Moskovchenko <stepanm@codeaurora.org>
Fixed case when no warning generated for long subject line that is
wrapped to more than one line, and all lines are less than line limit.
New warning message added:
"non-blank line after summary line"
Now there are two warnings possible for the subject line, the original
line over limit and the new one. Depending on the error(s) any
combination of the two warnings are possible.
Commit text requirements now:
1) Must be less than 75 characters
2) Must be followed by blank line.
Change-Id: Id28d7a2eee561a1b1f45e51e346be3868c1fbad5
Signed-off-by: David Keitel <dkeitel@codeaurora.org>
Currently checkpatch.pl throws error for any use of "dsb"
keyword. Add exceptions when it is used in a variable or a
function name. Expectations are when 'dsb' keyword is used in a
variable or a function name, it is prefixed with class [-_>*\.]
and/or suffixed with class [-_\.;].
Change-Id: I06cd21e42135334bb41cf31f13088b118fcf3ac9
Signed-off-by: Sarangdhar Joshi <spjoshi@codeaurora.org>
filp_open allows people to get around the ban on sys_open.
Close the loophole.
Change-Id: I6e2be62e848cbc064e07008d0886c0d003c8be4b
Signed-off-by: Gregory Bean <gbean@codeaurora.org>
(cherry picked from commit bb181a18a813a70176f71a0c64aa572fcfbef0f0)
Signed-off-by: Stepan Moskovchenko <stepanm@codeaurora.org>
Signed-off-by: Rishabh Bhatnagar <rishabhb@codeaurora.org>
