From: "Edwin Chiu 邱垂峰" <edwin.chiu@sunplus.com>
To: Rob Herring <robh@kernel.org>
Cc: Edwin chiu <edwinchiu0505tw@gmail.com>,
"daniel.lezcano@linaro.org" <daniel.lezcano@linaro.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"rafael@kernel.org" <rafael@kernel.org>,
"linux-pm@vger.kernel.org" <linux-pm@vger.kernel.org>
Subject: RE: [PATCH 1/2] dt-bingings:arm:sunplus:add sp7021 compatible string to sunplus,idle-state.yaml
Date: Mon, 15 Nov 2021 09:07:36 +0000 [thread overview]
Message-ID: <b119d112eff2425a81e823787ea3866d@sphcmbx02.sunplus.com.tw> (raw)
In-Reply-To: <YY7xp3qcO68rJJUC@robh.at.kernel.org>
Hi Rob:
Thanks your response.
Reply your question as below:
> > 1) The checkpatch warning is WARNING: From:/Signed-off-by: email address mismatch: 'From: Edwin
> chiu <edwinchiu0505tw@gmail.com>' != 'Signed-off-by: Edwin chiu <edwin.chiu@sunplus.com>'
> > ==> I want explain to you that <edwin.chiu@sunplus.com> is my email in the company, but it disagree
> operate by "git send-email".
> > So I used <edwinchiu0505tw@gmail.com> in my linux system to operate with "git send-email".
> > Actually I checkpatch in my side was 0 error 0 warning.
>
> The email used to send and the author emails are independent in git.
> There should be a 'From:' line in the resulting patch when the sender and author don't match.
>
> Maybe you need to set sendemail.from config option to your corp email.
>
==> Please understanding it is our company secure policy.
It disagree operate by "git send-email".
But I am pretty sure I can receive email on both edwin.chiu@sunplus.com and edwinchiu0505tw@gmail.com .
> >
> > 2) Regarding to dtbs-check issue, your list warnings related to others company's yaml file, it not my
> yaml file case.
> > What is the purpose you want? Or what should I do?
>
> You aren't adding anything custom, so why are you even adding the schema? arm/idle-states.yaml
> should be enough.
>
===> Yes, I want to apply arm/idle-state structure to my idle function.
I thought every function submit, all need edit yaml file together.
Do you mean I can submit cpuidle-sunplus.c without my sunplus,idle-state.yaml ?
On the other hand, I confirm my yaml file can pass dt_binding_check and dtbs_check as below:
edwinchiu@scdiu3:~/code/code1/linux/kernel$ make dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/arm/sunplus/sunplus,idle-state.yaml
DTEX Documentation/devicetree/bindings/arm/sunplus/sunplus,idle-state.example.dts
DTC Documentation/devicetree/bindings/arm/sunplus/sunplus,idle-state.example.dt.yaml
CHECK Documentation/devicetree/bindings/arm/sunplus/sunplus,idle-state.example.dt.yaml
edwinchiu@scdiu3:~/code/code1/linux/kernel$
edwinchiu@scdiu3:~/code/code1/linux/kernel$ export ARCH=arm
edwinchiu@scdiu3:~/code/code1/linux/kernel$ echo $ARCH
arm
edwinchiu@scdiu3:~/code/code1/linux/kernel$ make dtbs_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/arm/sunplus/sunplus,idle-state.yaml
SYNC include/config/auto.conf.cmd
SCHEMA Documentation/devicetree/bindings/processed-schema.json
UPD include/config/kernel.release
DTC arch/arm/boot/dts/sp7021-bpi-f2p.dtb
DTC arch/arm/boot/dts/sp7021-bpi-f2s.dtb
DTC arch/arm/boot/dts/sp7021-demov2.dtb
DTC arch/arm/boot/dts/sp7021-demov3.dtb
DTC arch/arm/boot/dts/sp7021-ev.dtb
DTC arch/arm/boot/dts/sp7021-ev-ttl.dtb
DTC arch/arm/boot/dts/sp7021-ltpp3g2revD.dtb
DTC arch/arm/boot/dts/sp7021-ltpp3g2-sunplus.dtb
DTC arch/arm/boot/dts/sp7021-bpi-f2p.dt.yaml
CHECK arch/arm/boot/dts/sp7021-bpi-f2p.dt.yaml
DTC arch/arm/boot/dts/sp7021-bpi-f2s.dt.yaml
CHECK arch/arm/boot/dts/sp7021-bpi-f2s.dt.yaml
DTC arch/arm/boot/dts/sp7021-demov2.dt.yaml
CHECK arch/arm/boot/dts/sp7021-demov2.dt.yaml
DTC arch/arm/boot/dts/sp7021-demov3.dt.yaml
CHECK arch/arm/boot/dts/sp7021-demov3.dt.yaml
DTC arch/arm/boot/dts/sp7021-ev.dt.yaml
CHECK arch/arm/boot/dts/sp7021-ev.dt.yaml
DTC arch/arm/boot/dts/sp7021-ev-ttl.dt.yaml
CHECK arch/arm/boot/dts/sp7021-ev-ttl.dt.yaml
DTC arch/arm/boot/dts/sp7021-ltpp3g2revD.dt.yaml
CHECK arch/arm/boot/dts/sp7021-ltpp3g2revD.dt.yaml
DTC arch/arm/boot/dts/sp7021-ltpp3g2-sunplus.dt.yaml
CHECK arch/arm/boot/dts/sp7021-ltpp3g2-sunplus.dt.yaml
edwinchiu@scdiu3:~/code/code1/linux/kernel$
邱垂峰 EdwinChiu
智能運算專案
T: +886-3-5786005 ext.2590
edwin.chiu@sunplus.com
300 新竹科學園區創新一路19號
> -----Original Message-----
> From: Rob Herring <robh@kernel.org>
> Sent: Saturday, November 13, 2021 6:59 AM
> To: Edwin Chiu 邱垂峰 <edwin.chiu@sunplus.com>
> Cc: Edwin chiu <edwinchiu0505tw@gmail.com>; daniel.lezcano@linaro.org; devicetree@vger.kernel.org;
> linux-kernel@vger.kernel.org; rafael@kernel.org; linux-pm@vger.kernel.org
> Subject: Re: [PATCH 1/2] dt-bingings:arm:sunplus:add sp7021 compatible string to
> sunplus,idle-state.yaml
>
> On Tue, Nov 09, 2021 at 06:58:24AM +0000, Edwin Chiu 邱垂峰 wrote:
> > Hi Rob:
> >
> > Look into the full log, it show you check 3 programs
> > ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > Context Check Description
> > robh/checkpatch warning total: 0 errors, 1 warnings, 67 lines checked
> > robh/dt-meta-schema success
> > robh/dtbs-check fail build log
> > ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> >
> > 1) The checkpatch warning is WARNING: From:/Signed-off-by: email address mismatch: 'From: Edwin
> chiu <edwinchiu0505tw@gmail.com>' != 'Signed-off-by: Edwin chiu <edwin.chiu@sunplus.com>'
> > ==> I want explain to you that <edwin.chiu@sunplus.com> is my email in the company, but it disagree
> operate by "git send-email".
> > So I used <edwinchiu0505tw@gmail.com> in my linux system to operate with "git send-email".
> > Actually I checkpatch in my side was 0 error 0 warning.
>
> The email used to send and the author emails are independent in git.
> There should be a 'From:' line in the resulting patch when the sender and author don't match.
>
> Maybe you need to set sendemail.from config option to your corp email.
>
> >
> > 2) Regarding to dtbs-check issue, your list warnings related to others company's yaml file, it not my
> yaml file case.
> > What is the purpose you want? Or what should I do?
>
> You aren't adding anything custom, so why are you even adding the schema? arm/idle-states.yaml
> should be enough.
>
> >
> > 3) Execute=>make dtbs_check
> > DT_SCHEMA_FILES=Documentation/devicetree/bindings/arm/sunplus/sunplus,
> > idle-state.yaml There are many config select items…..I don't know how
> > to select them.
> > (I execute "make config " and "make all" were pass) Can you teach me
> > how to do "make dtbs_check"?
>
> Did you read Documentation/devicetree/bindings/writing-schema.rst?
>
> To check all dts files (for an arch) and run the validation, do:
>
> make allmodconfig
> make dtbs_check
>
> Rob
>
> >
> > 邱垂峰 EdwinChiu
> > 智能運算專案
> > T: +886-3-5786005 ext.2590
> > edwin.chiu@sunplus.com
> > 300 新竹科學園區創新一路19號
> >
> > -----Original Message-----
> > From: Rob Herring <robh@kernel.org>
> > Sent: Monday, November 8, 2021 9:03 PM
> > To: Edwin chiu <edwinchiu0505tw@gmail.com>
> > Cc: robh+dt@kernel.org; daniel.lezcano@linaro.org;
> > devicetree@vger.kernel.org; linux-kernel@vger.kernel.org;
> > rafael@kernel.org; linux-pm@vger.kernel.org; Edwin Chiu 邱垂峰
> > <edwin.chiu@sunplus.com>
> > Subject: Re: [PATCH 1/2] dt-bingings:arm:sunplus:add sp7021 compatible
> > string to sunplus,idle-state.yaml
> >
> > On Mon, 08 Nov 2021 15:55:09 +0800, Edwin chiu wrote:
> > > Add the compatible string for cpuidle state on sp7021
> > >
> > > Signed-off-by: Edwin chiu <edwin.chiu@sunplus.com>
> > > ---
> > > .../bindings/arm/sunplus/sunplus,idle-state.yaml | 55 ++++++++++++++++++++++
> > > MAINTAINERS | 6 +++
> > > 2 files changed, 61 insertions(+)
> > > create mode 100644
> > > Documentation/devicetree/bindings/arm/sunplus/sunplus,idle-state.yam
> > > l
> > >
> >
> > Running 'make dtbs_check' with the schema in this patch gives the following warnings. Consider if
> they are expected or the schema is incorrect. These may not be new warnings.
> >
> > Note that it is not yet a requirement to have 0 warnings for dtbs_check.
> > This will change in the future.
> >
> > Full log is available here: https://patchwork.ozlabs.org/patch/1552228
> >
> >
> > spc: compatible:0: 'arm,idle-state' was expected
> > arch/arm/boot/dts/qcom-apq8064-asus-nexus7-flo.dt.yaml
> > arch/arm/boot/dts/qcom-apq8064-cm-qs600.dt.yaml
> > arch/arm/boot/dts/qcom-apq8064-ifc6410.dt.yaml
> > arch/arm/boot/dts/qcom-apq8064-sony-xperia-yuga.dt.yaml
> > arch/arm/boot/dts/qcom-apq8074-dragonboard.dt.yaml
> > arch/arm/boot/dts/qcom-apq8084-ifc6540.dt.yaml
> > arch/arm/boot/dts/qcom-apq8084-mtp.dt.yaml
> > arch/arm/boot/dts/qcom-msm8974-fairphone-fp2.dt.yaml
> > arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dt.yaml
> > arch/arm/boot/dts/qcom-msm8974-samsung-klte.dt.yaml
> > arch/arm/boot/dts/qcom-msm8974-sony-xperia-amami.dt.yaml
> > arch/arm/boot/dts/qcom-msm8974-sony-xperia-castor.dt.yaml
> > arch/arm/boot/dts/qcom-msm8974-sony-xperia-honami.dt.yaml
> >
> > spc: compatible: Additional items are not allowed ('arm,idle-state' was unexpected)
> > arch/arm/boot/dts/qcom-apq8064-asus-nexus7-flo.dt.yaml
> > arch/arm/boot/dts/qcom-apq8064-cm-qs600.dt.yaml
> > arch/arm/boot/dts/qcom-apq8064-ifc6410.dt.yaml
> > arch/arm/boot/dts/qcom-apq8064-sony-xperia-yuga.dt.yaml
> > arch/arm/boot/dts/qcom-apq8074-dragonboard.dt.yaml
> > arch/arm/boot/dts/qcom-apq8084-ifc6540.dt.yaml
> > arch/arm/boot/dts/qcom-apq8084-mtp.dt.yaml
> > arch/arm/boot/dts/qcom-msm8974-fairphone-fp2.dt.yaml
> > arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dt.yaml
> > arch/arm/boot/dts/qcom-msm8974-samsung-klte.dt.yaml
> > arch/arm/boot/dts/qcom-msm8974-sony-xperia-amami.dt.yaml
> > arch/arm/boot/dts/qcom-msm8974-sony-xperia-castor.dt.yaml
> > arch/arm/boot/dts/qcom-msm8974-sony-xperia-honami.dt.yaml
> >
> > spc: compatible: ['qcom,idle-state-spc', 'arm,idle-state'] is too long
> > arch/arm/boot/dts/qcom-apq8064-asus-nexus7-flo.dt.yaml
> > arch/arm/boot/dts/qcom-apq8064-cm-qs600.dt.yaml
> > arch/arm/boot/dts/qcom-apq8064-ifc6410.dt.yaml
> > arch/arm/boot/dts/qcom-apq8064-sony-xperia-yuga.dt.yaml
> > arch/arm/boot/dts/qcom-apq8074-dragonboard.dt.yaml
> > arch/arm/boot/dts/qcom-apq8084-ifc6540.dt.yaml
> > arch/arm/boot/dts/qcom-apq8084-mtp.dt.yaml
> > arch/arm/boot/dts/qcom-msm8974-fairphone-fp2.dt.yaml
> > arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dt.yaml
> > arch/arm/boot/dts/qcom-msm8974-samsung-klte.dt.yaml
> > arch/arm/boot/dts/qcom-msm8974-sony-xperia-amami.dt.yaml
> > arch/arm/boot/dts/qcom-msm8974-sony-xperia-castor.dt.yaml
> > arch/arm/boot/dts/qcom-msm8974-sony-xperia-honami.dt.yaml
> >
next prev parent reply other threads:[~2021-11-15 9:14 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-11-08 7:55 [PATCH 0/2] Add cpuidle driver for Sunplus SP7021 Edwin chiu
2021-11-08 7:55 ` [PATCH 1/2] dt-bingings:arm:sunplus:add sp7021 compatible string to sunplus,idle-state.yaml Edwin chiu
2021-11-08 13:03 ` Rob Herring
2021-11-09 6:58 ` Edwin Chiu 邱垂峰
2021-11-12 22:58 ` Rob Herring
2021-11-15 9:07 ` Edwin Chiu 邱垂峰 [this message]
2021-11-26 5:59 ` Edwin Chiu 邱垂峰
2021-12-03 7:04 ` Edwin Chiu 邱垂峰
2021-11-08 7:55 ` [PATCH 2/2] cpuidle:sunplus:create cpuidle driver for sunplus sp7021 Edwin chiu
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=b119d112eff2425a81e823787ea3866d@sphcmbx02.sunplus.com.tw \
--to=edwin.chiu@sunplus.com \
--cc=daniel.lezcano@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=edwinchiu0505tw@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=rafael@kernel.org \
--cc=robh@kernel.org \
/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
Powered by JetHome