Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1521722 > unrolled thread
| Started by | Stanimir Varbanov <stanimir.varbanov@linaro.org> |
|---|---|
| First post | 2016-11-14 16:00 +0100 |
| Last post | 2016-11-14 16:10 +0100 |
| Articles | 2 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH v3 0/9] Qualcomm video decoder/encoder driver Stanimir Varbanov <stanimir.varbanov@linaro.org> - 2016-11-14 16:00 +0100
Re: [PATCH v3 0/9] Qualcomm video decoder/encoder driver Hans Verkuil <hverkuil@xs4all.nl> - 2016-11-14 16:10 +0100
| From | Stanimir Varbanov <stanimir.varbanov@linaro.org> |
|---|---|
| Date | 2016-11-14 16:00 +0100 |
| Subject | Re: [PATCH v3 0/9] Qualcomm video decoder/encoder driver |
| Message-ID | <sDqTL-8jU-7@gated-at.bofh.it> |
Hi, On 11/11/2016 02:11 PM, Javier Martinez Canillas wrote: > Hello Hans, > > On Fri, Nov 11, 2016 at 8:49 AM, Hans Verkuil <hverkuil@xs4all.nl> wrote: >> Hi Stanimir, >> >> Overall it looks good. As you saw, I do have some comments but nothing major. >> >> One question: you use qcom as the directory name. How about using qualcomm? >> >> It's really not that much longer and a bit more obvious. >> >> Up to you, though. >> > > It seems qcom is more consistent to the name used in most subsystems > for Qualcomm: > > $ find -name *qcom > ./arch/arm/mach-qcom > ./arch/arm64/boot/dts/qcom > ./Documentation/devicetree/bindings/soc/qcom > ./sound/soc/qcom > ./drivers/pinctrl/qcom > ./drivers/soc/qcom > ./drivers/clk/qcom > > $ find -name *qualcomm > ./drivers/net/ethernet/qualcomm Also qcom is the vendor prefix used in [1] [1] Documentation/devicetree/bindings/vendor-prefixes.txt -- regards, Stan
[toc] | [next] | [standalone]
| From | Hans Verkuil <hverkuil@xs4all.nl> |
|---|---|
| Date | 2016-11-14 16:10 +0100 |
| Message-ID | <sDr3s-aQ-5@gated-at.bofh.it> |
| In reply to | #1521722 |
On 11/14/2016 03:59 PM, Stanimir Varbanov wrote: > Hi, > > On 11/11/2016 02:11 PM, Javier Martinez Canillas wrote: >> Hello Hans, >> >> On Fri, Nov 11, 2016 at 8:49 AM, Hans Verkuil <hverkuil@xs4all.nl> wrote: >>> Hi Stanimir, >>> >>> Overall it looks good. As you saw, I do have some comments but nothing major. >>> >>> One question: you use qcom as the directory name. How about using qualcomm? >>> >>> It's really not that much longer and a bit more obvious. >>> >>> Up to you, though. >>> >> >> It seems qcom is more consistent to the name used in most subsystems >> for Qualcomm: >> >> $ find -name *qcom >> ./arch/arm/mach-qcom >> ./arch/arm64/boot/dts/qcom >> ./Documentation/devicetree/bindings/soc/qcom >> ./sound/soc/qcom >> ./drivers/pinctrl/qcom >> ./drivers/soc/qcom >> ./drivers/clk/qcom >> >> $ find -name *qualcomm >> ./drivers/net/ethernet/qualcomm > > Also qcom is the vendor prefix used in [1] > > [1] Documentation/devicetree/bindings/vendor-prefixes.txt > qcom it is, then :-) Hans
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web