From: "Daniel W. S. Almeida" <dwlsalmeida@gmail.com>
To: Shuah Khan <skhan@linuxfoundation.org>
Cc: "kstewart@linuxfoundation.org" <kstewart@linuxfoundation.org>,
"sean@mess.org" <sean@mess.org>,
"mchehab+huawei@kernel.org" <mchehab+huawei@kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"tglx@linutronix.de" <tglx@linutronix.de>,
"linux-kernel-mentees@lists.linuxfoundation.org"
<linux-kernel-mentees@lists.linuxfoundation.org>,
"allison@lohutok.net" <allison@lohutok.net>,
"linux-media@vger.kernel.org" <linux-media@vger.kernel.org>
Subject: Re: [Linux-kernel-mentees] [RFC, WIP, v6 02/10] media: vidtv: implement a tuner driver
Date: Wed, 3 Jun 2020 14:19:12 -0300 [thread overview]
Message-ID: <3273ABE6-71E6-4FCE-9901-E02A87B3F48F@getmailspring.com> (raw)
In-Reply-To: <deb6b49a-5f50-1d40-6577-84fe787150ec@linuxfoundation.org>
Hi Shuah, thanks for reviewing this!
>> From: "Daniel W. S. Almeida" <dwlsalmeida@gmail.com>
>
> Are you using git send-email to send patches. Why do have this
> From line in here? Same comment on all other patches in this series.
I am using 'git format-patch --to=[...] --cc=[...]' and then 'git send-email'.
Git format-patch usually adds these two lines for every patch it generates:
1 From f966b0ea574a36b96ddfb8a8a716b174bc150eba Mon Sep 17 00:00:00 2001
2 From: "Daniel W. S. Almeida" <dwlsalmeida@gmail.com>
I have no idea whether these are necessary, so I just leave them there :)
The first line is specially weird.
As for this:
>> + switch (c->delivery_system) {
>
> Add indetation for the below:
>
>> + case SYS_DVBT:
>> + case SYS_DVBT2:
>> + valid_freqs = config.vidtv_valid_dvb_t_freqs;
>> + array_sz = ARRAY_SIZE(config.vidtv_valid_dvb_t_freqs);
>> + break;
I am not sure I understand. Indenting the case statements leads to a
checkpatch complaint:
>SWITCH_CASE_INDENT_LEVEL: switch and case should be at the
>same indent
And the actual statements under 'case' are already indented.
Thanks
- Daniel
_______________________________________________
Linux-kernel-mentees mailing list
Linux-kernel-mentees@lists.linuxfoundation.org
https://lists.linuxfoundation.org/mailman/listinfo/linux-kernel-mentees
next prev parent reply other threads:[~2020-06-03 17:19 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-20 7:03 [Linux-kernel-mentees] [RFC, WIP, v6 00/10] media: vidtv: implement a virtual DVB driver Daniel W. S. Almeida
2020-05-20 7:03 ` [Linux-kernel-mentees] [RFC, WIP, v6 01/10] media: vidtv: add Kconfig entry Daniel W. S. Almeida
2020-05-28 22:50 ` Shuah Khan
2020-05-20 7:03 ` [Linux-kernel-mentees] [RFC, WIP, v6 02/10] media: vidtv: implement a tuner driver Daniel W. S. Almeida
2020-05-28 23:26 ` Shuah Khan
2020-06-03 17:19 ` Daniel W. S. Almeida [this message]
2020-05-20 7:03 ` [Linux-kernel-mentees] [RFC, WIP, v6 03/10] media: vidtv: implement a demodulator driver Daniel W. S. Almeida
2020-05-28 23:39 ` Shuah Khan
2020-05-20 7:03 ` [Linux-kernel-mentees] [RFC, WIP, v6 04/10] media: vidtv: add a bridge driver Daniel W. S. Almeida
2020-06-02 23:13 ` Shuah Khan
2020-05-20 7:03 ` [Linux-kernel-mentees] [RFC, WIP, v6 05/10] media: vidtv: add wrappers for memcpy and memset Daniel W. S. Almeida
2020-06-01 16:51 ` Shuah Khan
2020-05-20 7:03 ` [Linux-kernel-mentees] [RFC, WIP, v6 06/10] media: vidtv: add MPEG TS common code Daniel W. S. Almeida
2020-05-20 7:03 ` [Linux-kernel-mentees] [RFC, WIP, v6 07/10] media: vidtv: implement a PSI generator Daniel W. S. Almeida
2020-05-20 7:03 ` [Linux-kernel-mentees] [RFC, WIP, v6 08/10] media: vidtv: implement a PES packetizer Daniel W. S. Almeida
2020-05-20 7:03 ` [Linux-kernel-mentees] [RFC, WIP, v6 09/10] media: vidtv: Implement a SMPTE 302M encoder Daniel W. S. Almeida
2020-05-20 7:03 ` [Linux-kernel-mentees] [RFC, WIP, v6 10/10] media: vidtv: Add a MPEG Transport Stream Multiplexer Daniel W. S. Almeida
2020-05-28 21:39 ` [Linux-kernel-mentees] [RFC, WIP, v6 00/10] media: vidtv: implement a virtual DVB driver Shuah Khan
2020-05-28 23:57 ` Shuah Khan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=3273ABE6-71E6-4FCE-9901-E02A87B3F48F@getmailspring.com \
--to=dwlsalmeida@gmail.com \
--cc=allison@lohutok.net \
--cc=kstewart@linuxfoundation.org \
--cc=linux-kernel-mentees@lists.linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab+huawei@kernel.org \
--cc=sean@mess.org \
--cc=skhan@linuxfoundation.org \
--cc=tglx@linutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).