From: "Nayak, Rajendra" <rnayak@codeaurora.org>
To: Paolo Pisati <p.pisati@gmail.com>, Andy Gross <andy.gross@linaro.org>
Cc: linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 0/5] dts patches for qcom tsens support
Date: Wed, 24 Aug 2016 17:48:43 +0530 [thread overview]
Message-ID: <57BD90A3.3000106@codeaurora.org> (raw)
In-Reply-To: <20160824102256.GA14048@harukaze>
On 8/24/2016 3:52 PM, Paolo Pisati wrote:
> On Wed, Aug 17, 2016 at 02:33:40PM -0500, Andy Gross wrote:
>> On Wed, Aug 17, 2016 at 10:48:43AM +0530, Rajendra Nayak wrote:
>>> Hey Andy,
>>>
>>> This is a respin of v2 with some minor fixes pointed out by Rob.
>>> Please pull these in for 4.9
>>>
>>> Thanks,
>>> Rajendra
>>
>> I pulled these in.
>
> Did you try to read the content of the qfprom from userspace?
Hi, no I haven't tried it, I can give it a try tomorrow when I
have access to my board and see whats going wrong, but I did
look at Andys for-next and the defconfig still does not have
either nvmem or qcom-qfprom drivers enabled.
Did you manually enable them? or are you seeing this with the drivers
still disabled?
>
> $ uname -a
> Linux dragon410c 4.8.0-rc1+ #6 SMP PREEMPT Wed Aug 24 11:11:02 CEST 2016 aarch64
> aarch64 aarch64 GNU/Linux
> $ lsmod
> Module Size Used by
> nvmem_qfprom 16384 0
> nvmem_core 24576 1 nvmem_qfprom
> $ ls -la /sys/bus/nvmem/devices/
> total 0
> drwxr-xr-x 2 root root 0 Aug 24 10:17 .
> drwxr-xr-x 4 root root 0 Aug 24 10:15 ..
> lrwxrwxrwx 1 root root 0 Aug 24 10:17 qfprom0 ->
> ../../../devices/platform/soc/5c000.qfprom/qfprom0
> $ cat /sys/bus/nvmem/devices/qfprom0/nvmem
>
> [spontaneous reboot]
>
> This using agross's for-next tree as of today ("54ba896 Merge branch
> 'arm64-defconfig-for-4.9' into all-for-4.8") and defconfig.
>
next prev parent reply other threads:[~2016-08-24 12:19 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-17 5:18 [PATCH v3 0/5] dts patches for qcom tsens support Rajendra Nayak
2016-08-17 5:18 ` [PATCH v3 1/5] arm: dts: msm8974: Add thermal zones, tsens and qfprom nodes Rajendra Nayak
2016-08-17 5:18 ` [PATCH v3 2/5] arm: dts: apq8064: " Rajendra Nayak
2016-08-17 5:18 ` [PATCH v3 3/5] arm: dts: apq8084: " Rajendra Nayak
2016-08-17 5:18 ` [PATCH v3 4/5] arm64: dts: msm8916: " Rajendra Nayak
2016-08-17 5:18 ` [PATCH v3 5/5] arm64: dts: msm8996: " Rajendra Nayak
2016-08-17 19:33 ` [PATCH v3 0/5] dts patches for qcom tsens support Andy Gross
2016-08-24 10:22 ` Paolo Pisati
2016-08-24 12:18 ` Nayak, Rajendra [this message]
2016-08-24 22:03 ` Andy Gross
2016-08-25 3:13 ` Bjorn Andersson
2016-08-25 3:21 ` Andy Gross
2016-08-25 12:21 ` Paolo Pisati
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=57BD90A3.3000106@codeaurora.org \
--to=rnayak@codeaurora.org \
--cc=andy.gross@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=p.pisati@gmail.com \
/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