Instead of producing a too wide table, let's split it per
subsystem, and use a better notation in order to make easier
for the reader to identify how a feature is supported on
multiple architectures.
Signed-off-by: Mauro Carvalho Chehab
---
scripts/get_feat.pl | 113
, a good thing to do.
Regards,
Mauro
Mauro Carvalho Chehab (6):
scripts: get_feat.pl: add a script to handle Documentation/features
scripts: get_feat.pl: improve matrix output
scripts: get_feat.pl: use its implementation for list-arch.sh
sphinx: kernel_feat.py: add a script to parse feature
Add a feature list matrix at the admin-guide.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/admin-guide/features.rst | 3 +++
Documentation/admin-guide/index.rst| 1 +
Documentation/conf.py | 2 +-
3 files changed, 5 insertions(+), 1 deletion(-)
create mode 100644
features a given architecture supports;
3) Output a matrix with features x architectures.
Signed-off-by: Mauro Carvalho Chehab
---
scripts/get_feat.pl | 474
1 file changed, 474 insertions(+)
create mode 100755 scripts/get_feat.pl
diff --git a/scripts
changes the features list order to be
case-insensitive, in order to better match the output of
the existing script.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/features/list-arch.sh | 17 +--
scripts/get_feat.pl | 77 -
2 files changed, 66
Em Fri, 27 Nov 2020 09:20:53 -0500
Willem de Bruijn escreveu:
> On Fri, Nov 27, 2020 at 1:46 AM Mauro Carvalho Chehab
> wrote:
> >
> > As reported on:
> >
> > https://lore.kernel.org/linux-media/20190627222020.45909-1-willemdebruijn.ker...@gmail.co
)
media: vidtv: psi: Implement an Event Information Table (EIT)
media: vidtv: psi: extract descriptor chaining code into a helper
media: vidtv: Move s302m specific fields into encoder context
media: vidtv: psi: fix missing assignments in while loops
Mauro Carvalho Chehab (30
Em Sat, 30 May 2020 10:41:17 +0800
Jia-Ju Bai escreveu:
> The value hdev->sfr.kva is stored in DMA memory, and it is assigned to
> sfr, so sfr->buf_size can be modified at anytime by malicious hardware.
> In this case, a buffer overflow may happen when the code
> "sfr->data[sfr->buf_size - 1]"
;
- if firmware has loaded;
- if the LNBf was powered on.
Using status = 0 seems to ensure that everything will be
properly powered up.
So, instead of the proposed solution, let's just set
status = 0.
Reported-by: syzbot
Reported-by: Willem de Bruijn
Signed-off-by: Mauro Carvalho Chehab
Add documentation for enum rc_proto and struct lirc_scancode
at the generated docs.
Signed-off-by: Mauro Carvalho Chehab
---
.../userspace-api/media/rc/lirc-dev-intro.rst | 11 +--
include/uapi/linux/lirc.h | 2 +-
2 files changed, 10 insertions
Make the license for those documents also compatible with GPLv2.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/userspace-api/media/rc/keytable.c.rst | 2 +-
Documentation/userspace-api/media/rc/lirc-dev-intro.rst | 2 +-
Documentation/userspace-api/media/rc/lirc
Hi Sean,
Those two patches improve LIRC uAPI docs by adding the structs documented at
lirc.h to the output of
Documentation/output/userspace-api/media/rc/lirc-dev-intro.html
Please review.
Mauro Carvalho Chehab (2):
media: docs: uAPI: rc: dual-license the docs with GPL v2
media: rc
Hi Linus,
Please pull from:
git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media
tags/media/v5.10-2
For:
- a rand Kconfig fixup for mtk-vcodec;
- a fix at h264 handling at cedrus codec driver;
- some warning fixes when config PM is not enabled at marvell-ccic;
- two
Instead of first zeroing all fields at the mux structs and
then filling, do some initialization for the const data
when they're created.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_mux.c | 46 +++-
1 file changed, 26 insertions(+), 20
, let's change it, just to make easier
to check for the new code while reading it with DVB tools
like dvbinspector.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_bridge.c | 3 ++-
drivers/media/test-drivers/vidtv/vidtv_common.h | 1 -
2 files changed, 2 insertions
}
$sharp = 0;
$note = "";
$octave = "";
};
};
print "\n";
Signed-off-by: Mauro Carvalho Chehab
---
.../media/test-drivers/vidtv/vidtv_s302m.c| 38 ++-
1 file changed, 20 insertions(+), 18 deletions(
Initialize the fields of the arguments directly when
declaring it, and pass the args as a pointer, instead of
copying them.
Signed-off-by: Mauro Carvalho Chehab
---
.../media/test-drivers/vidtv/vidtv_bridge.c | 28 ++-
drivers/media/test-drivers/vidtv/vidtv_mux.c | 28
The Linux stack is too short. So, using recursive functions
is a very bad idea. Convert those into non-recursive ones.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_psi.c | 75 +++-
1 file changed, 41 insertions(+), 34 deletions(-)
diff --git
The tone generator logic were repeating the song after the
first silent. There's also a wrong logic at the note
offset calculus, which may create some noise.
Fix it.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_s302m.c | 14 --
drivers/media/test
Place some text at EIT data, and use ISO 8859-15 encoding for
the German letter "ü" (u mit umlat) letter.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_channel.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/te
The EIT header ID field should not contain the network ID, but,
instead, the service_id of the program described at EIT.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_channel.c | 3 ++-
drivers/media/test-drivers/vidtv/vidtv_psi.c | 5 +++--
drivers/media
On normal TS streams, the NIT table has its own entry at PAT,
but not at PMT.
While here, properly handle alloc problems when creating
PMT entries.
Signed-off-by: Mauro Carvalho Chehab
---
.../media/test-drivers/vidtv/vidtv_channel.c | 8 ++--
drivers/media/test-drivers/vidtv/vidtv_mux.c
As the service currently broadcasts just audio, change the
service type to reflect that.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_channel.c | 2 +-
drivers/media/test-drivers/vidtv/vidtv_psi.h | 1 +
2 files changed, 2 insertions(+), 1 deletion
Cleanup the table_section_crc32_write_into() function
by initializing struct psi_write_args only once and by
passing the args as a pointer.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_psi.c | 39 +---
1 file changed, 18 insertions(+), 21
While the original plan was to use the first movement of
the 5th Symphony, it was opted to use the Für Elise song,
instead.
Fix it.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_channel.c | 2 +-
drivers/media/test-drivers/vidtv/vidtv_s302m.c | 10
The service_id there should be equal to the one used
on other tables, otherwise, EIT entries won't be valid.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_channel.c | 3 ++-
drivers/media/test-drivers/vidtv/vidtv_psi.c | 5 +++--
drivers/media/test-drivers
The current event is using an undefined date. Instead, it
should be the timestamp when the EIT table was generated.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_psi.c | 44 ++--
1 file changed, 41 insertions(+), 3 deletions(-)
diff --git
- Pass struct header_write_args as a pointer, instead of
passing as a var;
- Initialize the psi_args struct only once.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_psi.c | 42 +---
1 file changed, 19 insertions(+), 23 deletions(-)
diff
This function initializes the psi_args twice, and receives
a struct, instead of a pointer to a struct.
Clean it up.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_psi.c | 104 +--
1 file changed, 51 insertions(+), 53 deletions(-)
diff --git
There are a few warnings produced by checkpatch.pl on
strict mode.
They're all trivial. Address them.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_bridge.c | 1 -
drivers/media/test-drivers/vidtv/vidtv_channel.c | 2 --
drivers/media/test-drivers/vidtv
Avoid initializing the structs multiple times and pass the
PAT struct as a pointer, instead of a var.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_mux.c | 2 +-
drivers/media/test-drivers/vidtv/vidtv_psi.c | 57 ++--
drivers/media/test-drivers
- Pass struct vidtv_psi_pmt_write_args as a pointer;
- Avoid initializing structs multiple times.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_mux.c | 2 +-
drivers/media/test-drivers/vidtv/vidtv_psi.c | 91 ++--
drivers/media/test-drivers
-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_mux.c | 30 ++--
drivers/media/test-drivers/vidtv/vidtv_pes.c | 178 +--
drivers/media/test-drivers/vidtv/vidtv_pes.h | 2 +-
3 files changed, 102 insertions(+), 108 deletions(-)
diff --git a/drivers/media
- pass struct vidtv_psi_sdt_write_args as a pointer;
- avoid initializing struct fields multiple times.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_mux.c | 2 +-
drivers/media/test-drivers/vidtv/vidtv_psi.c | 85 ++--
drivers/media/test
, start_streaming
will return an error that userspace can detect:
ERRORDMX_SET_PES_FILTER failed (PID = 0x2000): 12 Cannot allocate
memory
and the driver doesn't crash.
Signed-off-by: Mauro Carvalho Chehab
---
.../media/test-drivers/vidtv/vidtv_bridge.c | 2 +
.../media/test-drivers/vidtv
Update the vidtv documentation with the relevant changes
after the last patches.
Signed-off-by: Mauro Carvalho Chehab
---
.../driver-api/media/drivers/vidtv.rst| 85 ---
1 file changed, 72 insertions(+), 13 deletions(-)
diff --git a/Documentation/driver-api/media
- pass struct vidtv_psi_eit_write_args as a pointer;
- avoid initializing struct fields multiple times.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_mux.c | 2 +-
drivers/media/test-drivers/vidtv/vidtv_psi.c | 86 ++--
drivers/media/test
-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_psi.c | 128 +--
1 file changed, 62 insertions(+), 66 deletions(-)
diff --git a/drivers/media/test-drivers/vidtv/vidtv_psi.c
b/drivers/media/test-drivers/vidtv/vidtv_psi.c
index 92fdf1149f0d..36f1c56d9e24 100644
Do some cleanups at the coding style of the driver:
- remove "inline" declarations;
- use reverse xmas-tree for local var declarations;
- Adjust some indent to avoid breaking 80-cols;
- Cleanup some comments.
No functional changes.
Signed-off-by: Mauro Carvalho Chehab
---
.../
Fix existing issues at the kernel-doc markups and add them to
the vidtv.rst file.
Signed-off-by: Mauro Carvalho Chehab
---
.../driver-api/media/drivers/vidtv.rst| 27 +
.../media/test-drivers/vidtv/vidtv_bridge.h | 2 +-
.../media/test-drivers/vidtv/vidtv_channel.h | 1
- Place the includes on alphabetical order;
- get rid of asm/byteorder.h;
- add bug.h at vidtv_s302m.c, as it is needed by
inux/fixp-arith.h
Signed-off-by: Mauro Carvalho Chehab
---
.../media/test-drivers/vidtv/vidtv_bridge.c | 8 +++
.../media/test-drivers/vidtv/vidtv_bridge.h | 2
- pass struct vidtv_psi_nit_write_args as a pointer;
- avoid initializing struct fields multiple times.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_mux.c | 2 +-
drivers/media/test-drivers/vidtv/vidtv_psi.c | 91 ++--
drivers/media/test
Initialize the destination buffer/size and the initial
offset when creating the local var.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_mux.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/drivers/media/test-drivers/vidtv
Right now, there's no need to access the length of some
tables. So, drop the unused functions.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/test-drivers/vidtv/vidtv_psi.c | 35
1 file changed, 35 deletions(-)
diff --git a/drivers/media/test-drivers/vidtv
appear on Kernel 5.10,
my plan is to merge those, together with Daniel's patches for Kernel 5.10.
Regards,
Mauro
Mauro Carvalho Chehab (31):
media: vidtv: reorganize includes
media: vidtv: add error checks
media: vidtv: don't use recursive functions
media: vidtv: fix the name
com/KSPP/linux/issues/115
> Signed-off-by: Gustavo A. R. Silva
Reviewed-by: Mauro Carvalho Chehab
> ---
> drivers/media/pci/saa7134/saa7134-tvaudio.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/media/pci/saa7134/saa7134-tvaudio.c
> b/drivers/media/pci/
com/KSPP/linux/issues/115
> Signed-off-by: Gustavo A. R. Silva
Reviewed-by: Mauro Carvalho Chehab
> ---
> drivers/media/platform/rcar_jpu.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/media/platform/rcar_jpu.c
> b/drivers/media/platform/rcar_j
e.
>
> Link: https://github.com/KSPP/linux/issues/115
> Signed-off-by: Gustavo A. R. Silva
Reviewed-by: Mauro Carvalho Chehab
> ---
> drivers/media/usb/dvb-usb-v2/af9015.c | 1 +
> drivers/media/usb/dvb-usb-v2/lmedm04.c | 1 +
> 2 files changed, 2 insertions(+)
>
> diff --gi
com/KSPP/linux/issues/115
> Signed-off-by: Gustavo A. R. Silva
Reviewed-by: Mauro Carvalho Chehab
> ---
> drivers/media/dvb-core/dvb_frontend.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/media/dvb-core/dvb_frontend.c
> b/drivers/media/dvb-core/dvb
com/KSPP/linux/issues/115
> Signed-off-by: Gustavo A. R. Silva
Reviewed-by: Mauro Carvalho Chehab
> ---
> drivers/staging/media/atomisp/pci/runtime/isys/src/rx.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/staging/media/atomisp/pci/runtime/isys/src/
ext case.
Reviewed-by: Mauro Carvalho Chehab
>
> Link: https://github.com/KSPP/linux/issues/115
> Signed-off-by: Gustavo A. R. Silva
> ---
> drivers/media/dvb-frontends/cx24120.c | 1 +
> drivers/media/dvb-frontends/dib0090.c | 2 ++
> drivers/media/dvb-frontends/drxk_har
Em Mon, 16 Nov 2020 18:38:33 +
Mark Brown escreveu:
> On Mon, Nov 16, 2020 at 01:59:30PM +0100, Mauro Carvalho Chehab wrote:
>
> > This driver is ready for mainstream. Move it out of staging.
>
> There's quite a few issues here, to be honest I'm disappointed some of
>
Hi Mark,
Em Mon, 16 Nov 2020 18:38:33 +
Mark Brown escreveu:
> > This driver is ready for mainstream. Move it out of staging.
>
> There's quite a few issues here, to be honest I'm disappointed some of
> them weren't caught during staging review, this needs fairly substantial
> work and
Em Mon, 16 Nov 2020 09:31:06 -0600
Rob Herring escreveu:
> On Mon, Nov 16, 2020 at 01:59:27PM +0100, Mauro Carvalho Chehab wrote:
> > The phy USB3 driver for Hisilicon 970 (hi3670) is ready
> > for mainstream. Mode it from staging into the main driver's
>
> s/Mode/Move/
Em Tue, 17 Nov 2020 02:12:01 +
Nícolas F. R. A. Prado escreveu:
> Sphinx 3.1 introduced namespaces for C cross-references. With this,
> each C domain type/function declaration is put inside the namespace that
> was active at the time of its declaration.
>
> Add support for automatic
Em Mon, 16 Nov 2020 18:24:04 +0100
Daniel Vetter escreveu:
> On Mon, Nov 16, 2020 at 05:38:04PM +0100, Mauro Carvalho Chehab wrote:
> > Em Mon, 16 Nov 2020 16:36:06 +0100
> > Daniel Vetter escreveu:
> >
> > > On Mon, Nov 16, 2020 at 11:18:06AM +010
Em Mon, 16 Nov 2020 16:36:06 +0100
Daniel Vetter escreveu:
> On Mon, Nov 16, 2020 at 11:18:06AM +0100, Mauro Carvalho Chehab wrote:
> > Some identifiers have different names between their prototypes
> > and the kernel-doc markup.
> >
> > Signed-off-by: Mauro Ca
The following commit has been merged into the irq/core branch of tip:
Commit-ID: 8c67d247dcad67fbdd07c8bab9818d0b8d9240bf
Gitweb:
https://git.kernel.org/tip/8c67d247dcad67fbdd07c8bab9818d0b8d9240bf
Author:Mauro Carvalho Chehab
AuthorDate:Mon, 16 Nov 2020 11:18:15 +01:00
The following commit has been merged into the timers/core branch of tip:
Commit-ID: 66981c37b3199d293c58f84cf2366e86a06e1a3d
Gitweb:
https://git.kernel.org/tip/66981c37b3199d293c58f84cf2366e86a06e1a3d
Author:Mauro Carvalho Chehab
AuthorDate:Mon, 16 Nov 2020 11:18:14 +01
This driver is ready for mainstream. Move it out of staging.
Signed-off-by: Mauro Carvalho Chehab
---
MAINTAINERS | 7 +-
drivers/regulator/Kconfig | 9 +
drivers/regulator/Makefile| 1 +
drivers/regulator
The I2C buses are not declared at the device tree. As this will
be needed by further patches, add them, keeping all in
disabled state. Per-board settings can override it.
Signed-off-by: Mauro Carvalho Chehab
---
arch/arm64/boot/dts/hisilicon/hi3670.dtsi | 71 +++
1 file
Add a device tree for the HiSilicon 6421v600 SPMI PMIC, used
on HiKey970 board.
As we now have support for it, change the fixed regulators
used by the SD I/O to use the proper LDO supplies.
Signed-off-by: Mauro Carvalho Chehab
---
.../boot/dts/hisilicon/hi3670-hikey970.dts| 22
This patch series finish addressing support for Hikey 970
USB.
It moves the power management and USB3 drivers out of
staging, adding the device tree changes required for USB3
to work on Hikey 970.
Mauro Carvalho Chehab (8):
phy: phy-hi3670-usb3: move driver from staging into phy
spmi: hi6421
The Hisilicon 6421v600 SPMI driver is ready for mainstream.
So, move it from staging.
Signed-off-by: Mauro Carvalho Chehab
---
.../spmi/hisilicon,hisi-spmi-controller.yaml | 62 +++
MAINTAINERS | 7 +
drivers/spmi/Kconfig | 9
Add the USB3 bindings for Kirin 970 phy and Hikey 970 board.
Signed-off-by: Mauro Carvalho Chehab
---
.../boot/dts/hisilicon/hi3670-hikey970.dts| 102 ++
arch/arm64/boot/dts/hisilicon/hi3670.dtsi | 64 +++
2 files changed, 166 insertions(+)
diff --git a/arch
There are several pinctrl settings that are missing at this
DT file.
Also, the entries are out of order.
Add the missing bits, as they'll be required by the DRM driver - and
probably by other drivers not upstreamed yet.
Reorder the entres, adding the missing bits.
Signed-off-by: Mauro Carvalho
This driver is ready for mainstream. So, move it out of staging.
Signed-off-by: Mauro Carvalho Chehab
---
.../mfd/hisilicon,hi6421-spmi-pmic.yaml | 159
MAINTAINERS | 7 +
drivers/mfd/Kconfig | 15 +
drivers/mfd
The phy USB3 driver for Hisilicon 970 (hi3670) is ready
for mainstream. Mode it from staging into the main driver's
phy/ directory.
Signed-off-by: Mauro Carvalho Chehab
---
.../bindings/phy/phy-hi3670-usb3.yaml | 72 ++
MAINTAINERS | 9 +-
drivers
Kernel-doc only supports having the comment just before
the identifier.
The markup for init_completion is actually for
__init_completion.
Signed-off-by: Mauro Carvalho Chehab
---
Thats said, IMHO, it would make sense to simply
rename __init_completion() to init_completion() and drop
this file cause the
parser to completely miss the identifier name of each
function.
This prevents improving the script to do some needed validation
tests.
Address this part. Yet, furter changes are needed in order
for it to fully follow the specs.
Signed-off-by: Mauro Carvalho Chehab
---
tools/testing
minor English syntax issues, as suggested
by Matthew:
will used -> will be used
similar with -> similar to
Suggested-by: Mattew Wilcox # English fixes
Signed-off-by: Mauro Carvalho Chehab
---
mm/gup.c| 24 +---
mm/page_alloc.
Two markups are at the wrong place. Kernel-doc only
support having the comment just before the identifier.
Also, some identifiers have different names between their
prototypes and the kernel-doc markup.
Signed-off-by: Mauro Carvalho Chehab
---
fs/dcache.c | 72
Probaly this was due to a cut and paste issue.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/rapidio/rio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/rapidio/rio.c b/drivers/rapidio/rio.c
index 606986c5ba2c..16b59b472b3d 100644
--- a/drivers/rapidio/rio.c
Some identifiers have different names between their prototypes
and the kernel-doc markup.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/video/fbdev/core/fbcmap.c | 2 +-
drivers/video/hdmi.c | 3 ++-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/video
A function has a different name between their prototype
and its kernel-doc markup.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/hsi/hsi_core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/hsi/hsi_core.c b/drivers/hsi/hsi_core.c
index 47f0208aa7c3..c3fb5beb846e
Some functions have different names between their prototypes
and the kernel-doc markup.
Others need to be fixed, as kernel-doc markups should use this format:
identifier - description
Signed-off-by: Mauro Carvalho Chehab
---
drivers/infiniband/core/cm.c | 5
There are some common comments marked, instead, with kernel-doc
notation, which won't work.
While here, rename an identifier, in order to match the
function prototype below kernel-doc markup.
Acked-by: Richard Gong
Signed-off-by: Mauro Carvalho Chehab
---
include/linux/firmware/intel
the function name.
The hrtimer_is_queued prototype has a typo: it is using
'=' character instead of '-' to split:
identifier - description
as required by kernel-doc markup.
Signed-off-by: Mauro Carvalho Chehab
---
include/linux/hrtimer.h | 6 +-
kernel/time/hrtimer.c | 2 +-
2 files
Some identifiers have different names between their prototypes
and the kernel-doc markup.
Signed-off-by: Mauro Carvalho Chehab
---
net/core/datagram.c | 2 +-
net/core/dev.c| 4 ++--
net/core/skbuff.c | 2 +-
net/ethernet/eth.c| 6 +++---
net/sunrpc/rpc_pipe.c | 3 ++-
5 files
Kernel-doc markups should use this format:
identifier - description
While here, fix a kernel-doc tag that was using, instead,
a normal comment block.
Signed-off-by: Mauro Carvalho Chehab
---
kernel/resource.c | 24 ++--
1 file changed, 14 insertions(+), 10 deletions
A function has a different name between their prototype
and its kernel-doc markup.
Signed-off-by: Mauro Carvalho Chehab
---
include/linux/connector.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/connector.h b/include/linux/connector.h
index cb732643471b
A function has a different name between their prototype
and its kernel-doc markup.
Signed-off-by: Mauro Carvalho Chehab
---
include/linux/w1.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/w1.h b/include/linux/w1.h
index 949d3b10e531..9a2a0ef39018 100644
hlist_add_behing -> hlist_add_behind
Signed-off-by: Mauro Carvalho Chehab
---
include/linux/list.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/list.h b/include/linux/list.h
index 89bdc92e75c3..f2af4b4aa4e9 100644
--- a/include/linux/list.h
+++ b/incl
Some identifiers have different names between their prototypes
and the kernel-doc markup.
Signed-off-by: Mauro Carvalho Chehab
---
kernel/irq/chip.c | 2 +-
kernel/irq/generic-chip.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/kernel/irq/chip.c b/kernel/irq
.
@maintainers: feel free to pick the patches and
apply them directly on your trees, as all patches on
this series are independent from the other ones.
--
v4:
- Patches got rebased and got some acks.
Mauro Carvalho Chehab (27):
net: phy: fix kernel-doc markups
net: datagram: fix some kernel
-by: Mauro Carvalho Chehab
---
kernel/sched/core.c | 16
kernel/sched/fair.c | 2 +-
2 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/kernel/sched/core.c b/kernel/sched/core.c
index 02076e6d3792..9d41378ae8f3 100644
--- a/kernel/sched/core.c
+++ b/kernel/sched
The kernel-doc markup inside share.c is actually for
__parport_register_driver. The actual goal seems to be
to document parport_register_driver().
So, fix the existing markup and add a new one.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/parport/share.c | 2 +-
include/linux/parport.h
Some identifiers have different names between their prototypes
and the kernel-doc markup.
In the specific case of netif_subqueue_stopped(), keep the
current markup for __netif_subqueue_stopped(), adding a
new one for netif_subqueue_stopped().
Signed-off-by: Mauro Carvalho Chehab
---
include
The documented struct is psz_head and not psz_buffer.
Reviewed-by: Kees Cook
Signed-off-by: Mauro Carvalho Chehab
---
fs/pstore/zone.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/pstore/zone.c b/fs/pstore/zone.c
index 3ce89216670c..46a1610c06fa 100644
--- a/fs/pstore
Some functions have different names between their prototypes
and the kernel-doc markup.
Reviewed-by: Andrew Lunn
Signed-off-by: Mauro Carvalho Chehab
---
drivers/net/phy/mdio_bus.c | 2 +-
drivers/net/phy/phy-c45.c| 2 +-
drivers/net/phy/phy.c| 2 +-
drivers/net/phy/phy_device.c
fix one typo:
ccw driver -> ccw_driver
and one function rename.
Signed-off-by: Mauro Carvalho Chehab
---
arch/s390/include/asm/ccwdev.h | 2 +-
arch/s390/include/asm/cio.h| 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/s390/include/asm/ccwdev.h b/a
Some identifiers have different names between their prototypes
and the kernel-doc markup.
Others need to be fixed, as kernel-doc markups should use this format:
identifier - description
Signed-off-by: Mauro Carvalho Chehab
---
drivers/gpu/drm/drm_atomic_state_helper.c | 2
A function has a different name between their prototype
and its kernel-doc markup.
Signed-off-by: Mauro Carvalho Chehab
---
lib/crc7.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/crc7.c b/lib/crc7.c
index 6a848d73e804..3848e313b722 100644
--- a/lib/crc7.c
+++ b/lib
-off-by: Mauro Carvalho Chehab
---
fs/jbd2/journal.c | 34 ++
fs/jbd2/transaction.c | 31 ---
include/linux/jbd2.h | 2 +-
3 files changed, 35 insertions(+), 32 deletions(-)
diff --git a/fs/jbd2/journal.c b/fs/jbd2/journal.c
index
Some identifiers have different names between their prototypes
and the kernel-doc markup.
Acked-by: Mike Rapoport
Signed-off-by: Mauro Carvalho Chehab
---
include/linux/memblock.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/linux/memblock.h b/include/linux
documented.
Signed-off-by: Mauro Carvalho Chehab
---
scripts/kernel-doc | 62 ++
1 file changed, 46 insertions(+), 16 deletions(-)
diff --git a/scripts/kernel-doc b/scripts/kernel-doc
index 9b6ddeb097e9..919acae23fad 100755
--- a/scripts/kernel-doc
+++ b/scri
Hi Jonathan,
Em Sun, 8 Nov 2020 16:56:21 +
Jonathan Cameron escreveu:
> > PS.: the IIO subsystem is the one that currently has more duplicated
> > ABI entries:
> > $ ./scripts/get_abi.pl validate 2>&1|grep iio
> > Warning: /sys/bus/iio/devices/iio:deviceX/in_accel_x_calibbias is defined 2
Em Thu, 5 Nov 2020 15:00:17 +
Matthew Wilcox escreveu:
> On Fri, Oct 23, 2020 at 06:33:43PM +0200, Mauro Carvalho Chehab wrote:
> > Kernel-doc currently expects that the kernel-doc markup to come
> > just before the function/enum/struct/union/typedef prototype.
> >
Em Mon, 2 Nov 2020 12:41:53 -0800
Randy Dunlap escreveu:
> On 11/2/20 12:35 PM, Mauro Carvalho Chehab wrote:
> > Em Mon, 2 Nov 2020 11:23:25 -0800
> > Randy Dunlap escreveu:
> >
> >> On 11/2/20 8:59 AM, Mauro Carvalho Chehab wrote:
> >>> Em Mon, 2
Em Mon, 2 Nov 2020 11:23:25 -0800
Randy Dunlap escreveu:
> On 11/2/20 8:59 AM, Mauro Carvalho Chehab wrote:
> > Em Mon, 2 Nov 2020 08:02:34 -0800
> > Randy Dunlap escreveu:
> >
> >> On 11/2/20 12:33 AM, Mauro Carvalho Chehab wrote:
> >>> Em
501 - 600 of 7435 matches
Mail list logo