On Tue, Oct 15, 2024 at 9:57 PM Mukesh Ojha wrote:
>
> Multiple call to glink_subdev_stop() for the same remoteproc can happen
> if rproc_stop() fails from Process-A that leaves the rproc state to
> RPROC_CRASHED state later a call to recovery_store from user space in
> Process B triggers rproc_tr
offset will always be positive number and adding
and comparing to the same variable 'size' will
always result in check being false always.
Remove this superfluous check.
Signed-off-by: anish kumar
---
drivers/remoteproc/remoteproc_elf_loader.c | 2 +-
1 file changed, 1 insertion(+),
This patch adds maintainer for maxim audio codecs.
Signed-off-by: Anish Kumar
---
MAINTAINERS | 7 +++
1 file changed, 7 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 8bdd7a7..2128586 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -7387,6 +7387,13 @@ S: Supported
F
cally small values).
As suggested by Wim Van Sebroeck & Guenter Roeck we should
add this functionality of individual watchdog drivers in the core
watchdog core.
Signed-off-by: anish kumar
---
drivers/watchdog/watchdog_dev.c | 34 +-
include/linux/w
Commit-ID: b66a2356d7108a15b8b5c9b8e6213e05ead22cd6
Gitweb: http://git.kernel.org/tip/b66a2356d7108a15b8b5c9b8e6213e05ead22cd6
Author: anish kumar
AuthorDate: Tue, 12 Mar 2013 14:44:08 -0400
Committer: Ingo Molnar
CommitDate: Thu, 14 Mar 2013 08:24:05 +0100
watchdog: Add comments to
will translate to always
if(true) so let's remove those checks as well(patch will be sent
later for that).
Is this return value for some future usecase(?), if yes then my
apologies.
Signed-off-by: anish kumar
---
include/linux/clocksource.h |6 +++---
kernel/time/clocksource.c |
on is being called use
dump_stack().This will give you the call chain leading up to your
function call which you are interested in.
>
>
>
> On Tue, Feb 19, 2013 at 9:25 PM, anish kumar
> wrote:
> > On Tue, 2013-02-19 at 12:16 +0530, chetan cr123 wrote:
> >> HI Al
On Tue, 2013-02-19 at 12:16 +0530, chetan cr123 wrote:
> HI All,
>
> I am working on Sd Card/Block driver
>
> I am registering it as both
>
> 1. register_blkdev()- BLOCK Regsiter
> 2. mmc_register_driver -- MMC regsiter
>
> and filling the mmc_driver structure.
>
> I am not a
On Sun, 2013-02-17 at 14:37 +, Jonathan Andrews wrote:
> > > The dim pixel code is timing critical (but as I said only a tiny
> > > fraction of the total CPU usage). What I need to do is grab the CPU and
> > > prevent any context switch (IRQ or PREEMPT) for this period.
> > why you want to do t
On Sat, 2013-02-16 at 14:37 +, Jonathan Andrews wrote:
> I hope this is the correct place, I expect to get abused.
>
> I'm trying to do a mostly soft real-time task with a very small hard
> real time element.
>
> I've written some code to drive matrix LED signs using a Raspberry Pi.
>
> Sour
From: anish kumar
This watchdog_disabled flag is bit of cryptic.Howerver it's usefullnes is
multifold.
Uses are:
1. Check if smpboot_register_percpu_thread function passed.
2. Makes sure that user enables and disables the watchdog in sequence
i.e. enable watchdog->disable watchdog
On Sat, 2013-02-16 at 09:42 +0100, Ingo Molnar wrote:
> * Don Zickus wrote:
>
> > > >> > + /*
> > > >> > +* We shouldn't enable watchdog threads if it is
> > > >> > +* disabled.This is done by watchdog_disabled
> > > >> > +* variable check in watchdog_*_all_cpus func
On Thu, 2013-02-14 at 16:33 +, Grant Likely wrote:
> On Thu, 14 Feb 2013 08:57:17 +0530, anish singh
> wrote:
> > On Thu, Feb 14, 2013 at 3:06 AM, Grant Likely
> > wrote:
> > > static int deferred_probe_initcall(void)
> > > {
> > > deferred_wq = create_singlethread_workqueue("defe
On Tue, 2013-02-12 at 10:58 +0800, Haojian Zhuang wrote:
> do_initcalls() could call all driver initialization code in kernel_init
> thread. It means that probe() function will be also called from that
> time. After this, kernel could access console & release __init section
> in the same thread.
>
On Sat, 2013-02-09 at 14:57 +0800, Peter Teoh wrote:
>
>
> On Sat, Feb 9, 2013 at 1:47 PM, anish kumar
> .
> Timer interrupts is supposed to cause scheduling and scheduler
> may or
> may not pick up your last process(we always use the ter
On Tue, 2013-02-05 at 16:18 -0500, valdis.kletni...@vt.edu wrote:
> On Wed, 06 Feb 2013 04:43:20 +0800, Jimmy Pan said:
>
> > in fact, i've been always wondering what is the relationship between dmesg
> > and /var/log/message. they diverse a lot...
dmesg is provided by kernel using cat /proc/kmsg.
Commit-ID: c02cf5f8ed6137e2b3b2f10e0fca336e06e09ba4
Gitweb: http://git.kernel.org/tip/c02cf5f8ed6137e2b3b2f10e0fca336e06e09ba4
Author: anish kumar
AuthorDate: Sun, 3 Feb 2013 22:08:23 +0100
Committer: Ingo Molnar
CommitDate: Mon, 4 Feb 2013 11:50:59 +0100
irq_work: Remove return value
From: anish kumar
This watchdog_disabled flag is bit of cryptic.Howerver it's usefullnes is
multifold.
Uses are:
1. Check if smpboot_register_percpu_thread function passed.
2. Makes sure that user enables and disables the watchdog in sequence
i.e. enable watchdog->disable watchdog
On Fri, 2013-02-01 at 09:59 -0500, Don Zickus wrote:
> On Fri, Feb 01, 2013 at 07:19:07PM +0530, anish kumar wrote:
> > From: anish kumar
> >
> > This watchdog_disabled flag is bit of cryptic.Howerver it's usefullnes is
> > multifold.
> > Uses are:
> >
On Sat, 2013-01-26 at 17:24 +0100, Frederic Weisbecker wrote:
> 2012/11/3 anish kumar :
> > From: anish kumar
> >
> > As no one is using the return value of irq_work_queue function
> > it is better to just make it void.
> >
> > Acked-by: Steven
Please ignore this patch.
On Fri, 2013-02-01 at 19:03 +0530, anish kumar wrote:
> From: anish kumar
>
> This watchdog_disabled flag is bit of cryptic.Howerver it's usefullnes is
> multifold.
> Uses are:
> 1. Check if smpboot_register_percpu_thread function passed.
&g
From: anish kumar
This watchdog_disabled flag is bit of cryptic.Howerver it's usefullnes is
multifold.
Uses are:
1. Check if smpboot_register_percpu_thread function passed.
2. Makes sure that user enables and disables the watchdog in sequence
i.e. enable watchdog->disable watchdog
From: anish kumar
This watchdog_disabled flag is bit of cryptic.Howerver it's usefullnes is
multifold.
Uses are:
1. Check if smpboot_register_percpu_thread function passed.
2. Makes sure that user enables and disables the watchdog in sequence
i.e. enable watchdog->disable watchdog
On Wed, 2013-01-30 at 10:51 -0500, Don Zickus wrote:
> On Tue, Jan 29, 2013 at 10:48:27PM +0530, anish kumar wrote:
> > Sorry for digressing from the topic but I think there is something wrong
> > with my understanding or something wrong with the code.So I guess Don
> > can c
On Tue, 2013-01-29 at 10:33 -0500, Don Zickus wrote:
> Hi Mike,
>
> On Tue, Jan 29, 2013 at 05:42:43PM +0400, Mike Lykov wrote:
> >
> > So my questions:
> >
> > 1. Are there a BUG in soft lockup detection mechanizm? Changing
> > watchdog_thresh to 30 have a side effect in production - D-state
>
From: anish kumar
As no one is using the return value of irq_work_queue function
it is better to just make it void.
Acked-by: Steven Rostedt
Signed-off-by: anish kumar
---
include/linux/irq_work.h |2 +-
kernel/irq_work.c|5 ++---
2 files changed, 3 insertions(+), 4 deletions
On Wed, 2013-01-16 at 10:25 +, Anuz Pratap Singh Tomar wrote:
>
>
> On Tue, Jan 15, 2013 at 6:31 AM, horseriver
> wrote:
> hi:
>
>what is the function of do_softirq()?
Softirq is basically same as bottom half except it is run in irq
context.So the question which
>From your comments in this thread https://lkml.org/lkml/2011/3/25/723
>The msm watchdog driver is present in kernel only. It does not use the
>built-in Linux watchdog api. This is because the primary function of
>our watchdog is detecting bus lockups and interrupts being turned off
Doesn't linux
On Sat, 2012-12-22 at 23:11 +0800, Woody Wu wrote:
> On Fri, Dec 21, 2012 at 01:33:03PM -0800, anish kumar wrote:
> > On Fri, 2012-12-21 at 23:34 +0800, Woody Wu wrote:
> > > On Thu, Dec 20, 2012 at 10:05:05AM -0800, anish singh wrote:
> > > > On Dec 20, 2
On Mon, 2012-12-24 at 22:10 +0800, Woody Wu wrote:
> On Fri, Dec 21, 2012 at 01:33:03PM -0800, anish kumar wrote:
> > On Fri, 2012-12-21 at 23:34 +0800, Woody Wu wrote:
> > > On Thu, Dec 20, 2012 at 10:05:05AM -0800, anish singh wrote:
> > > > On Dec 20, 2
On Thu, 2012-12-27 at 11:22 -0500, jonsm...@gmail.com wrote:
> On Thu, Dec 27, 2012 at 3:27 AM, Woody Wu wrote:
> > Hi, list
> >
> > I know this might be a very basic question. But I really don't clear at
> > it.
> >
> > Can a peripheral chip that claims to be ISA or PCI device be used in a
> > A
On Thu, 2012-12-27 at 10:51 +0100, Geert Uytterhoeven wrote:
> On Thu, Dec 27, 2012 at 9:27 AM, Woody Wu wrote:
> > Can a peripheral chip that claims to be ISA or PCI device be used in a
> > ARM based embedded system? For these kind of chips, I only concern
> > about the planar kind of devices, m
On Fri, 2012-12-21 at 23:34 +0800, Woody Wu wrote:
> On Thu, Dec 20, 2012 at 10:05:05AM -0800, anish singh wrote:
> > On Dec 20, 2012 6:30 AM, "Woody Wu" wrote:
> > >
> > > Hi, List
> > >
> > > Where is the Kernel code that handles external interrupts? I want to
> > > have a look at it but haven't
On Fri, 2012-12-14 at 18:05 +1100, NeilBrown wrote:
> On Mon, 10 Sep 2012 10:57:07 -0700 Kevin Hilman
> wrote:
>
>
> > OK thanks, I'll queue this up for v3.6-rc as this should qualify as a
> > regression.
>
> I don't think this did actually get queued. At least I'm still seeing the
> bug in 3.
On Sat, 2012-12-08 at 17:37 +0100, Julia Lawall wrote:
> The function pm860x_charger_probe in the file
> drivers/power/88pm860x_charger.c contains the following code:
>
> count = pdev->num_resources;
> for (i = 0, j = 0; i < count; i++) {
> info->irq[j] = platfo
On Fri, 2012-12-07 at 16:49 +, Alan Cox wrote:
> > I could imagine declaring the activity request buttons to be "input", but
> > for
> > presence detects it is a bit far fetched and would add too much complexity.
>
> Android tries to address this with its switch class driver, but I'm not
> su
On Fri, 2012-12-07 at 15:02 +0800, Han Pingtian wrote:
> Hi there,
>
> I'm wondering this is a typo in Documentation/kernel-parameters.txt
> about "kernelcore=":
>
> In the event, a node is too small to have both
> kernelcore and Movable pages, kernelcore pages will
> take pr
On Sat, 2012-12-01 at 19:05 +0100, Linus Walleij wrote:
> This updates the IRQdomain documentation a bit, by adding a more
> verbose explanation to why we need this, and by adding some
> extended documentation of the irq_domain_simple() usecase.
>
> Signed-off-by: Linus Walleij
Thanks for doing t
On Mon, 2012-12-03 at 01:53 +, Tc, Jenny wrote:
> > We discussed about this patch and then suggest some method to resolve this
> > issue by Myungjoo Ham. Why don't you write additional feature or your
> > opinion based on following patch by Myungjoo Ham?
> > -
> > http://git.kernel.org/?p=linux
On Wed, 2012-11-28 at 20:10 +0530, manty kuma wrote:
> In linux interrupt programming, we do request_irq(...) in this
> function, the first argument is irq number. If i am not wrong, this is
> the interrupt line that we are requesting from kernel. For one
Right.
> particular hardware, is this IRQ
On Fri, 2012-11-09 at 14:05 +, Tc, Jenny wrote:
> > I think that the role of extcon subsystem notify changed
> > state(attached/detached) of cable to notifiee, but if you want to add
> > property feature of cable, you should solve ambiguous issues.
> >
> > First,
> > This patch only support th
On Wed, 2012-11-07 at 17:45 +0100, Nicholas Mc Guire wrote:
> On Wed, 07 Nov 2012, Hendrik Visage wrote:
>
> > On Thu, Nov 8, 2012 at 9:01 AM, Jimmy Pan wrote:
> > > I understand how atomic operation work on unary core processors, I think
> > > it just disables the interrupt and dominate the cpu
Hello,
I have below question and I would really appreciate a _CORRECT_ answer.
What are the sequence of steps that happen in CPU, cache, TLB, VM, HDD
leading to execution of “x = 7” which isn’t present in cache or sysmem
nor translation in TLB. Also specify if any interrupts, exceptions or
faults
Hello Dan,
Is this patch of yours picked up by anyone?
On Sun, 2012-09-30 at 09:38 +0530, anish kumar wrote:
> On Sat, 2012-09-29 at 10:13 +0300, Dan Carpenter wrote:
> > psy->properties is an enum (32 bit type) so adding sizeof() puts us
> > four times further along than we in
From: anish kumar
As no one is using the return value of irq_work_queue function
it is better to just make it void.
Acked-by: Steven Rostedt
Signed-off-by: anish kumar
---
kernel/irq_work.c |5 ++---
1 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/kernel/irq_work.c b
From: anish kumar
As no one is using the return value of irq_work_queue function
it is better to just make it void.
Signed-off-by: anish kumar
---
kernel/irq_work.c |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/kernel/irq_work.c b/kernel/irq_work.c
index 1588e3b
On Wed, 2012-10-31 at 10:15 -0400, Steven Rostedt wrote:
> On Wed, 2012-10-31 at 23:02 +0900, anish kumar wrote:
> > From: anish kumar
> >
> > As no one is using the return value of irq_work_queue function
> > it is better to just make it void.
> >
> > Th
From: anish kumar
As no one is using the return value of irq_work_queue function
it is better to just make it void.
This patch is just a way to understand if there is some future
plan to use it but in any case please let me know the reason.
---
kernel/irq_work.c | 21 ++---
1
On Tue, 2012-10-30 at 14:45 -0400, Steven Rostedt wrote:
> On Wed, 2012-10-31 at 03:33 +0900, anish kumar wrote:
>
> > > CPU 0 CPU 1
> > >
> > > data = something flags = IRQ_WORK_BUSY
> > > s
On Tue, 2012-10-30 at 22:23 -0400, Steven Rostedt wrote:
> On Wed, 2012-10-31 at 01:36 +0100, Frederic Weisbecker wrote:
> > 2012/10/30 anish kumar :
> > > As I understand without the memory barrier proposed by you the situation
> > > wou
> The IRQ_WORK_BUSY flag is set right before we execute the
> work. Once this flag value is set, the work enters a
> claimable state again.
>
> This is necessary because if we want to enqueue a work but we
> fail the claim, we want to ensure that the CPU where that work
> is still pending will see
On Sat, 2012-10-27 at 09:23 +0800, Ming Lei wrote:
> This patch introduces one module parameter of 'path' in firmware_class
> to support customizing firmware image search path, so that people can
> use its own firmware path if the default built-in paths can't meet their
> demand[1], and the typical
ping...
On Sat, 2012-10-13 at 00:32 +0900, anish kumar wrote:?
> Hello tglx,
>
> I just found the below inconsistency while going through the code.
>
>
> kernel/irq/manage.c
>
> if (new->flags & IRQF_ONESHOT) {
> /*
> * The t
On Sat, 2012-10-13 at 14:21 +0900, anish kumar wrote:
> On Fri, 2012-10-12 at 22:52 +0200, Thomas Gleixner wrote:
> > On Fri, 12 Oct 2012, Liu, Chuansheng wrote:
> > > > -Original Message-
> > > > From: anish kumar [mailto:anish198519851...@gmail.com]
>
From: anish kumar
Extcon core doesn't free the memory when we do unregister.
Kfree is added in the remove path as it was missing.
Signed-off-by: anish kumar
---
drivers/extcon/extcon-max77693.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/drivers/extcon/e
Hi Choi,
This is based on http://git.infradead.org/users/kmpark/linux-samsung
this branch.
anish
On Sun, 2012-10-21 at 13:46 +0900, anish kumar wrote:
> From: anish kumar
>
> There was a case where free and list_del can be called twice
> on the same pointer.So fixed it by re-arrang
From: anish kumar
There was a case where free and list_del can be called twice
on the same pointer.So fixed it by re-arranging the code and
removing a function which was not needed.
Signed-off-by: anish kumar
---
drivers/extcon/extcon-class.c | 71
On Sat, 2012-10-20 at 13:08 +0900, anish kumar wrote:
> On Fri, 2012-10-19 at 10:34 +0530, Arun KS wrote:
> > Hi Anish,
> >
> > On Mon, May 28, 2012 at 9:16 AM, anish singh
> > wrote:
> > On Mon, May 28, 2012 at 2:57 AM, richard -rw- weinberger
> &g
On Sat, 2012-10-20 at 11:37 +0900, Chanwoo Choi wrote:
> On 10/20/2012 11:30 AM, anish kumar wrote:
> > On Sat, 2012-10-20 at 10:57 +0900, Chanwoo Choi wrote:
> >> On 10/19/2012 02:12 AM, anish kumar wrote:
> >>> From: anish kumar
> >>>
> >>&g
On Sat, 2012-10-20 at 10:57 +0900, Chanwoo Choi wrote:
> On 10/19/2012 02:12 AM, anish kumar wrote:
> > From: anish kumar
> >
> > We don't need a release call in this file as we are doing
> > everything needed in unregister call and we don't have any
> &g
From: anish kumar
We don't need a release call in this file as we are doing
everything needed in unregister call and we don't have any
more pointer to free up.
Signed-off-by: anish kumar
---
drivers/extcon/extcon-class.c |4 +---
1 files changed, 1 insertions(+), 3 deletion
On Fri, 2012-10-12 at 22:52 +0200, Thomas Gleixner wrote:
> On Fri, 12 Oct 2012, Liu, Chuansheng wrote:
> > > -Original Message-
> > > From: anish kumar [mailto:anish198519851...@gmail.com]
> > > Sent: Friday, October 12, 2012 11:25 PM
> > > To: Li
On Fri, 2012-10-12 at 15:29 +, Liu, Chuansheng wrote:
>
> > -Original Message-
> > From: anish kumar [mailto:anish198519851...@gmail.com]
> > Sent: Friday, October 12, 2012 11:25 PM
> > To: Liu, Chuansheng
> > Cc: Thomas Gleixner; linux-kernel@vger.k
Hello tglx,
I just found the below inconsistency while going through the code.
kernel/irq/manage.c
if (new->flags & IRQF_ONESHOT) {
/*
* The thread_mask for the action is or'ed to
* desc->thread_active to indicate that the
* IRQF_ONESHOT thread handler has
On Fri, 2012-10-12 at 14:57 +, Liu, Chuansheng wrote:
> > On SMP an interrupt which is raised after the ack() again before the
> > handler finishes, can invoke another delivery on a different CPU,
> > which then sees the IRQ_INPROGESS flag, masks it and flags it
> > PENDING. When the primary ha
On Thu, 2012-10-11 at 01:20 +, Tc, Jenny wrote:
> > From: anish kumar [mailto:anish198519851...@gmail.com]
> > Sent: Wednesday, October 10, 2012 8:15 PM
> > To: Tc, Jenny
> > Cc: myungjoo@samsung.com; cw00.c...@samsung.com; linux-
> > ker...@vger.kernel.org
&
On Wed, 2012-10-10 at 15:53 +0530, Jenny TC wrote:
> For some cables a boolean variable will not be enough to represent
> the state and properties of the cable. For example a charger cable can
> have states CONNECT,DISCOONECT,SUSPEND(Host suspend for SDP cable),
> RESUME(Host wakeup), and UPDATE (t
On Fri, 2012-10-05 at 09:27 -0400, Theodore Ts'o wrote:
> On Fri, Oct 05, 2012 at 09:51:55AM +0100, Iain Fraser wrote:
> >
> > I understand the interrupts and softirq's run in interrupt context (
> > as opposed to process context ). But what I
> > don't understand is why you cannot sleep in interr
On Tue, 2012-10-02 at 15:14 +, Pallala, Ramakrishna wrote:
> > On Tue, 2012-10-02 at 12:54 +, Pallala, Ramakrishna wrote:
> > > > > How can I schedule driver's worker and threaded IRQ on the same core?
> > > > >
> > > > > For work queue we have schedule_delayed_work_on () API available.
> >
On Tue, 2012-10-02 at 12:54 +, Pallala, Ramakrishna wrote:
> > > How can I schedule driver's worker and threaded IRQ on the same core?
> > >
> > > For work queue we have schedule_delayed_work_on () API available.
> > > But how to schedule the threaded IRQ on the same core where my worker
> > th
On Tue, 2012-10-02 at 12:19 +, Pallala, Ramakrishna wrote:
> Hi,
>
> How can I schedule driver's worker and threaded IRQ on the same core?
>
> For work queue we have schedule_delayed_work_on () API available.
> But how to schedule the threaded IRQ on the same core where my worker thread
> is
On Sat, 2012-09-29 at 10:13 +0300, Dan Carpenter wrote:
> psy->properties is an enum (32 bit type) so adding sizeof() puts us
> four times further along than we intended. It should be cast to a char
> pointer before doing the math.
You really read this driver to find out this one.
Good one.
>
> S
On Fri, 2012-09-28 at 08:33 +0800, Axel Lin wrote:
> This driver can be built as a module, add MODULE_LICENSE for it.
> For completeness, also adds MODULE_AUTHOR and MODULE_DESCRIPTION.
Aha I forgot that.Good one.
>
> Signed-off-by: Axel Lin
> ---
> drivers/extcon/extcon-adc-jack.c |5 +
On Fri, 2012-09-28 at 07:36 +0800, Axel Lin wrote:
> On failure, request_any_context_irq() returns a negative value.
> On success, it returns either IRQC_IS_HARDIRQ or IRQC_IS_NESTED.
>
> Also ensure adc_jack_probe() return 0 on success.
this patch makes sense.
>
> Signed-off-by: Axel Lin
> ---
On Fri, 2012-09-28 at 22:44 +0800, Ming Lei wrote:
> On Fri, Sep 28, 2012 at 10:31 PM, anish kumar
> wrote:
>
> > Ming,
> > Somehow I feel here that the problem reported by Russell is totally
> > different from your understanding of problem.I humbly urge you to have
&
On Fri, 2012-09-28 at 21:53 +0800, Ming Lei wrote:
> On Fri, Sep 28, 2012 at 11:31 AM, anish singh
> wrote:
> > Hello Ming,
> > Though I am not an expert in this driver core area but
> > I have been following this fix.So have some queries below:
> >
> > On Fri, Sep 28, 2012 at 6:22 AM, Ming Lei w
On Sat, 2012-09-22 at 10:50 +0100, Jonathan Cameron wrote:
> On 09/21/2012 11:52 PM, Anton Vorontsov wrote:
> > On Fri, Sep 21, 2012 at 09:40:14PM +0530, anish kumar wrote:
> >> From: anish kumar
> >>
> >> In last version:
> >> Addressed concerns
From: anish kumar
In last version:
Addressed concerns raised by lars:
a. made the adc_bat per device.
b. get the IIO channel using hardcoded channel names.
c. Minor issues related to gpio_is_valid and some code
refactoring.
In V1:
Addressed concerns raised by Anton:
a. changed the struct
Ignore this.Some problem with my mail client.
On Fri, 2012-09-21 at 20:19 +0530, anish kumar wrote:
> From: anish kumar
>
> In last version:
> Addressed concerns raised by lars:
> a. made the adc_bat per device.
> b. get the IIO channel using hardcoded channel names.
> c. Mi
From: anish kumar
In last version:
Addressed concerns raised by lars:
a. made the adc_bat per device.
b. get the IIO channel using hardcoded channel names.
c. Minor issues related to gpio_is_valid and some code
refactoring.
In V1:
Addressed concerns raised by Anton:
a. changed the struct
On Thu, 2012-09-20 at 18:45 -0700, Anton Vorontsov wrote:
> On Tue, Sep 18, 2012 at 11:33:20PM +0530, anish kumar wrote:
> > From: anish kumar
> >
> > In last version:
> > Addressed concerns raised by lars:
> > a. made the adc_bat per device.
> > b. get
From: anish kumar
In last version:
Addressed concerns raised by lars:
a. made the adc_bat per device.
b. get the IIO channel using hardcoded channel names.
c. Minor issues related to gpio_is_valid and some code
refactoring.
In V1:
Addressed concerns raised by Anton:
a. changed the struct
From: anish kumar
In last version:
Addressed concerns raised by lars:
a. made the adc_bat per device.
b. get the IIO channel using hardcoded channel names.
c. Minor issues related to gpio_is_valid and some code
refactoring.
In this version:
Addressed concerns raised by Anton:
a. changed the
From: anish kumar
In this version:
Addressed concerns raised by lars:
a. made the adc_bat per device.
b. get the IIO channel using hardcoded channel names.
c. Minor issues related to gpio_is_valid and some code
refactoring.
Signed-off-by: anish kumar
---
drivers/power/generic-adc-battery.c
From: anish kumar
This is the cleaned up code after the valuable inputs from
the Jonathan, Lars and Anton.
I have tried to accomodate all the concerns however please
let me know incase something is missed out.
Signed-off-by: anish kumar
---
drivers/power/generic-adc-battery.c | 431
On Sat, 2012-09-08 at 11:09 +0100, Jonathan Cameron wrote:
> On 09/08/2012 08:10 AM, anish kumar wrote:
> > Thanks for your time.
> > On Fri, 2012-09-07 at 08:52 +0100, Jonathan Cameron wrote:
> >> On 02/09/12 16:39, anish kumar wrote:
> >>> From: anish kumar
From: anish kumar
This commit- 314be14bb renamed the _st_ functions to loose the bit
that was meant for staging version but forgot to change
the documentation which still have _st_ sprinkled in some of the
places.
Signed-off-by: anish kumar
---
drivers/staging/iio/Documentation/inkernel.txt
On Fri, 2012-09-07 at 10:49 +0200, Lars-Peter Clausen wrote:
> On 09/02/2012 05:39 PM, anish kumar wrote:
> > From: anish kumar
> >
> > This patch is to use IIO to write a generic batttery driver.
> > There are some inherent assumptions here:
> > 1.User is ha
Thanks for your time.
On Fri, 2012-09-07 at 08:52 +0100, Jonathan Cameron wrote:
> On 02/09/12 16:39, anish kumar wrote:
> > From: anish kumar
> >
> > This patch is to use IIO to write a generic batttery driver.
> battery
> > There are some inherent assumptions h
On Sun, 2012-09-02 at 14:34 -0700, Anton Vorontsov wrote:
> On Sun, Sep 02, 2012 at 09:44:04PM +0530, Sannu K wrote: [...]
> > Just Curious what is the use of this module? Is there any user space
> > program uses this? When ACPI drivers for battery is available how
> > useful will this be?
>
> Tha
On Sun, 2012-09-02 at 07:51 -0700, Guenter Roeck wrote:
> On Sat, Sep 01, 2012 at 11:31:11AM +0200, Lars-Peter Clausen wrote:
> > On 08/31/2012 06:04 PM, Guenter Roeck wrote:
> > > On Thu, Aug 30, 2012 at 09:42:57PM -0400, Vivien Didelot wrote:
> > >> The MAX197 is an A/D converter, made by Maxim.
From: anish kumar
This patch is to use IIO to write a generic batttery driver.
There are some inherent assumptions here:
1.User is having both main battery and backup battery.
2.Both batteries use same channel to get the information.
Signed-off-by: anish kumar
---
drivers/power/Kconfig
From: anish kumar
without this patch we get below warning:
WARNING: externs should be avoided in .c files
132: FILE: extcon/extcon-class.c:
int extcon_set_state(struct extcon_dev *edev, u32 state);
So just removed the declaration.
Signed-off-by: anish kumar
---
drivers/extcon/extcon-class.c
From: anish kumar
Rather than re-inventing the wheel we can use the hamming function
to calculate the number of bits set to check for violation of
exclusivity.
Signed-off-by: anish kumar
---
drivers/extcon/extcon-class.c | 14 +-
1 files changed, 5 insertions(+), 9 deletions
From: anish kumar
With this change now individual drivers can use standard cable
names as below:
static const char *arizona_cable[] = {
extcon_cable_name[EXTCON_USB],
extcon_cable_name[EXTCON_USB_HOST],
"CUSTOM_CABLE"
NULL,
}
Signed-off-by: anish kumar
---
driv
From: anish kumar
Right now we are forced to define our own cable names instead of what
is exported by extcon framework.We should use the standard cable
names.With this change, individual drivers can define cable names
as below:
edev.list_of_cable = EXTCON_MIC_IN_SUPPORT
Hello,
I am trying to write a generic batttery driver using IIO and I have
one some below
questions:
Why do we have the representation of backup battery in this
driver(s3c_adc_battery.c) and when does the
s3c_adc_backup_bat_get_property function gets called?
As I understand, it is as this:It is
On Mon, 2012-08-20 at 06:56 +, 함명주 wrote:
> > From: anish kumar
> >
> > Instead of "const char **supported_cable" it is better to have
> > it as "const char *const *supported_cable".
> >
> > Signed-off-by: anish kumar
>
> Could
From: anish kumar
Instead of "const char **supported_cable" it is better to have
it as "const char *const *supported_cable".
Signed-off-by: anish kumar
---
include/linux/extcon.h |2 +-
include/linux/extcon/extcon-adc-jack.h |2 +-
2 files changed
From: anish kumar
External connector devices that decides connection information based on
ADC values may use adc-jack device driver. The user simply needs to
provide a table of adc range and connection states. Then, extcon
framework will automatically notify others.
Changes in V1:
added Lars
1 - 100 of 114 matches
Mail list logo
