Stefan,
On Wed, May 11, 2016 at 06:27:52AM -0400, Stefan Berger wrote:
> Jeremiah Mahler wrote on 05/10/2016 09:26:17 PM:
>
[...]
>
> I forgot to cc you on the patch and obviously you weren't cc'ed through
> the Reported-by: line.
>
> Here'
Stefan,
On Tue, May 10, 2016 at 11:05:43AM -0400, Stefan Berger wrote:
> Jeremiah Mahler wrote on 05/10/2016 09:55:23 AM:
>
> >
> > all,
> >
> > My machine is locking up during suspend and I have bisected the
> > problem to this patch (e89f8b1ade9cc1a) in
is called before ops is null'd and the sysfs core synchronizes this
> + * removal so that no callbacks are running or can run again
>*/
> - sysfs_remove_group(&chip->dev.parent->kobj, &tpm_dev_group);
> + WARN_ON(chip->groups_cnt != 0);
> + chip->groups[chip->groups_cnt++] = &tpm_dev_group;
> }
> diff --git a/drivers/char/tpm/tpm.h b/drivers/char/tpm/tpm.h
> index 8bc6fb8..508e8e0 100644
> --- a/drivers/char/tpm/tpm.h
> +++ b/drivers/char/tpm/tpm.h
> @@ -168,9 +168,9 @@ struct tpm_chip {
>
> struct dentry **bios_dir;
>
> -#ifdef CONFIG_ACPI
> - const struct attribute_group *groups[2];
> + const struct attribute_group *groups[3];
> unsigned int groups_cnt;
> +#ifdef CONFIG_ACPI
> acpi_handle acpi_dev_handle;
> char ppi_version[TPM_PPI_VERSION_LEN + 1];
> #endif /* CONFIG_ACPI */
> @@ -471,7 +471,8 @@ extern dev_t tpm_devt;
> extern const struct file_operations tpm_fops;
> extern struct idr dev_nums_idr;
>
> -ssize_t tpm_getcap(struct device *, __be32, cap_t *, const char *);
> +ssize_t tpm_getcap(struct tpm_chip *chip, __be32 subcap_id, cap_t *cap,
> +const char *desc);
> ssize_t tpm_transmit(struct tpm_chip *chip, const char *buf,
>size_t bufsiz);
> ssize_t tpm_transmit_cmd(struct tpm_chip *chip, void *cmd, int len,
> @@ -496,8 +497,7 @@ extern struct tpm_chip *tpmm_chip_alloc(struct device
> *pdev,
> extern int tpm_chip_register(struct tpm_chip *chip);
> extern void tpm_chip_unregister(struct tpm_chip *chip);
>
> -int tpm_sysfs_add_device(struct tpm_chip *chip);
> -void tpm_sysfs_del_device(struct tpm_chip *chip);
> +void tpm_sysfs_add_device(struct tpm_chip *chip);
>
> int tpm_pcr_read_dev(struct tpm_chip *chip, int pcr_idx, u8 *res_buf);
>
> --
> 2.4.3
>
--
- Jeremiah Mahler
Hi David,
On Wed, Mar 02, 2016 at 01:00:21PM -0800, David Ahern wrote:
> On 3/2/16 12:31 PM, Jeremiah Mahler wrote:
> >>On Tue, Mar 01, 2016 at 08:11:54AM +, Dexuan Cui wrote:
> >>>Hi, I got this line every 10 seconds with today's linux-next in a Hyper-V
> &
Hi David,
On Tue, Mar 01, 2016 at 12:07:49PM -0800, Jeremiah Mahler wrote:
> Hi all,
>
> On Tue, Mar 01, 2016 at 08:11:54AM +, Dexuan Cui wrote:
> > Hi, I got this line every 10 seconds with today's linux-next in a Hyper-V
> > guest, even
> > when I didn&#
Hi Darren, Andy,
On Mon, Feb 29, 2016 at 03:13:05PM -0800, Darren Hart wrote:
> On Tue, Feb 23, 2016 at 08:29:40AM -0800, Jeremiah Mahler wrote:
> > Hi Andy,
> >
> > Running the latest linux-next I am getting a Kconfig recursive
> > dependency detected message
s on INPUT
depends on ACPI_VIDEO || ACPI_VIDEO = n
select INPUT_SPARSEKMAP
+ select DMI
---help---
Say Y here if you want to support WMI-based hotkeys on Dell laptops.
--
2.7.0
I have also attached my .config in case that is useful.
--
- Jeremiah Mahler
Konstantin, Andrew,
On Fri, Feb 05, 2016 at 02:19:40PM -0800, Andrew Morton wrote:
> On Fri, 5 Feb 2016 10:05:02 -0800 Jeremiah Mahler wrote:
>
[...]
>
> This should fix it up.
>
> From: Konstantin Khlebnikov
> Subject: radix-tree: fix oops after radix_tree_i
Konstantin,
On Sun, Feb 07, 2016 at 11:27:53AM +0300, Konstantin Khlebnikov wrote:
> On Sat, Feb 6, 2016 at 9:18 PM, Jeremiah Mahler wrote:
[...]
> >> -static __always_inline unsigned
> >> +static __always_inline long
> >> radix_tree_chunk_siz
Andrew,
On Fri, Feb 05, 2016 at 02:19:40PM -0800, Andrew Morton wrote:
> On Fri, 5 Feb 2016 10:05:02 -0800 Jeremiah Mahler wrote:
>
[...]
> > unable to handle kernel NULL pointer dereference
>
> This should fix it up.
>
[...]
>
> include/linux/radix-tree.h |
resses correctly, this indicates that the
fault is triggered when the value in the slot pointer is accessed.
Perhaps slot is being incremented beyond its valid range?
--
- Jeremiah Mahler
gt; 1 file changed, 14 insertions(+), 13 deletions(-)
>
[...]
This patch does fix the problem I was having [1]. Thanks for the fix.
[1]: https://lkml.org/lkml/2016/1/4/791
Tested-by: Jeremiah Mahler
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-ke
fix.
>
> Regards,
>
> Hans
>
[...]
Thanks, that fixed the problem [1] :-)
[1]: https://lkml.org/lkml/2016/1/4/704
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.o
itialized drm 1.1.0 20060810
[2.415803] tpm_tis 00:05: TPM is disabled/deactivated (0x6)
[2.431086] wmi: Mapper loaded
[...]
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
Mor
all,
On Tue, Jan 05, 2016 at 12:46:30AM +0200, Kirill A. Shutemov wrote:
> On Mon, Jan 04, 2016 at 01:22:03PM -0800, Andrew Morton wrote:
> > On Thu, 24 Dec 2015 09:12:53 -0800 Jeremiah Mahler
> > wrote:
> >
> > > all,
> > >
> > > I have starte
.config I used. I can send an objdump, but do you want
a dump of the kernel, where the cgroup code is?
--
- Jeremiah Mahler
#
# Automatically generated file; DO NOT EDIT.
# Linux/x86 4.4.0-rc7 Kernel Configuration
#
CONFIG_64BIT=y
CONFIG_X86_64=y
CONFIG_X86=y
CONFIG_INSTRUCTION_DECODER=y
CONFIG_PERF
Andrew, Michal,
On Tue, Dec 29, 2015 at 10:30:37AM -0800, Andrew Morton wrote:
> On Tue, 29 Dec 2015 17:23:47 +0100 Michal Hocko wrote:
>
> > [CCing Andrew]
> >
> > On Thu 24-12-15 09:12:53, Jeremiah Mahler wrote:
> > > all,
> > >
> > > I hav
ion between vector assigning and cleanup
>
> Thanks,
> Gerry
According to my original bisect [1], it is the second patch.
[1]: https://lkml.org/lkml/2015/12/20/11
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel&quo
all,
I have started seeing a "Bad rss-counter" message in the logs with
the latest linux-next 20151222+.
[ 458.282192] BUG: Bad rss-counter state mm:8800c5a96000 idx:3 val:3894
I can test patches if anyone has any ideas.
--
- Jeremiah Mahler
--
To unsubscribe from this list
0.35 No irq handler for vector
[ 10.286484] do_IRQ: 0.35 No irq handler for vector
...
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.o
somebody,
On Sat, Dec 19, 2015 at 11:33:44PM -0800, Jeremiah Mahler wrote:
[...]
> I performed a bisect and found that the following patch introduced the bug,
> which is still present in the latest linux-next 20151218+.
>
> From 41c7518a5d14543fa4aa1b5b9994ac26b38c0406 Mon Sep 17 0
Jiang Liu,
On Thu, Dec 17, 2015 at 07:40:33PM -0800, Jeremiah Mahler wrote:
> all,
>
> I just started getting these "No irq handler for vector" messages
> after upgrading to linux-next 20151217+.
>
>
> (from the first boot)
> ...
> [2.282652]
62
...
I can test patches if anyone has any ideas :-)
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please r
On Sat, Dec 05, 2015 at 01:20:01PM -0800, Jeremiah Mahler wrote:
> all,
>
> Running the latest linux-next (20151202+) on an Acer C720, it will
> sporadically be very slow to resume from a suspend. Sometimes it
> resumes normally, other times it can take over a minute. During th
. done.
[ 2621.988444] cfg80211: World regulatory domain updated:
[ 2621.988448] cfg80211: DFS Master region: unset
...
Anyone else having this problem? Any suggestions for how to debug?
Thanks,
--
- Jeremiah Mahler
[0.00] Initializing cgroup subsys cpuset
[0.00] Initializing
device *pdev)
> +{
> + int err;
> + struct chromeos_firmware_data *gpios;
> +
> + gpios = devm_kmalloc(&pdev->dev, sizeof(gpios), GFP_KERNEL);
Should this be 'sizeof(*gpios)' so it allocates enough room for the
entire struct instead of just the pointer?
[.
.
>
> This patch sorts out these issues.
>
> Fixes: 9b774d5cf2db
> Reported-by: Jeremiah Mahler
> Signed-off-by: Jarkko Sakkinen
> Tested-by: Jeremiah Mahler
> ---
> drivers/char/tpm/tpm-chip.c | 18 ++
> 1 file changed, 10 insertions(+), 8 deletions(
Jarkko,
On Sun, Nov 08, 2015 at 02:49:06AM +0200, Jarkko Sakkinen wrote:
> On Sat, Nov 07, 2015 at 03:11:47PM -0800, Jeremiah Mahler wrote:
> > Jarkko,
> >
[...]
> >
> > It sounds like you have the problem figured out and have a good fix.
> > If I can do
Jarkko,
On Sun, Nov 08, 2015 at 12:31:09AM +0200, Jarkko Sakkinen wrote:
> On Sat, Nov 07, 2015 at 10:08:56AM -0800, Jeremiah Mahler wrote:
> > Jarkko,
> >
> > On Sat, Nov 07, 2015 at 01:41:37PM +0200, Jarkko Sakkinen wrote:
> > > On Sat, Nov 07, 2015 at 12:55:43P
'git bisect' more difficult because it
will point to the wrong patches. If this were my patch series I would
combine this patch with the later ones that actually start using the
code.
[...]
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kern
Jarkko,
On Sat, Nov 07, 2015 at 01:41:37PM +0200, Jarkko Sakkinen wrote:
> On Sat, Nov 07, 2015 at 12:55:43PM +0200, Jarkko Sakkinen wrote:
> > On Fri, Nov 06, 2015 at 06:55:18PM -0800, Jeremiah Mahler wrote:
> > > Jarkko,
> > >
[...]
> > > > + entr
> + if (!entry) {
> + kernfs_put(target);
> + return -ENOENT;
> + }
> +
On an Acer C720 this call to kernfs_find_and_get fails resulting in
a failed resume after suspend.
Apparently it can't find an object for the name "ppi".
This bug d
Jarkko,
On Wed, Nov 04, 2015 at 10:17:05AM -0800, Jeremiah Mahler wrote:
> Jarkko, all,
>
> On Fri, Oct 16, 2015 at 09:40:23PM +0300, Jarkko Sakkinen wrote:
> > Moved PPI attributes to the character device directory. This aligns with
> > the sysfs guidelines and makes them r
; --
> 2.5.0
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.
Jarkko,
On Thu, Nov 05, 2015 at 07:46:30PM +0200, Jarkko Sakkinen wrote:
> On Thu, Nov 05, 2015 at 08:47:58AM -0800, Jeremiah Mahler wrote:
> > Jarkko,
> >
> > On Thu, Nov 05, 2015 at 01:05:45PM +0200, Jarkko Sakkinen wrote:
> > > On Thu, Nov 05, 2015 at 11:22:55A
Jarkko,
On Thu, Nov 05, 2015 at 01:05:45PM +0200, Jarkko Sakkinen wrote:
> On Thu, Nov 05, 2015 at 11:22:55AM +0200, Jarkko Sakkinen wrote:
> > On Wed, Nov 04, 2015 at 10:17:05AM -0800, Jeremiah Mahler wrote:
> > > Jarkko, all,
> > >
[...]
> > >
> >
i_dev_handle, tpm_ppi_uuid,
> @@ -356,16 +355,5 @@ int tpm_add_ppi(struct tpm_chip *chip)
> ACPI_FREE(obj);
> }
>
> - rc = sysfs_create_group(&chip->pdev->kobj, &ppi_attr_grp);
> -
> - if (!rc)
> - chip->flags
r name but
> didn't update it here.
>
> Fix the I2C adapter lookup by using the "Synopsys DesignWare I2C adapter"
> name.
>
> Reported-by: Jeremiah Mahler
> Fixes: d80d134182ba ("i2c: designware: Move common probe code into
> i2c_dw_probe()")
> S
Jarkko,
On Tue, Nov 03, 2015 at 09:28:55AM +0200, Jarkko Nikula wrote:
> Hi
>
> On 03.11.2015 06:02, Jeremiah Mahler wrote:
> >Jarkko, all,
> >
> >Commit d80d134182ba5 introduced a bug which causes a cyapa based touch
> >pad on an Acer C720 Chromebook to bec
/i2c-designware-core.h| 5 +--
drivers/i2c/busses/i2c-designware-pcidrv.c | 30 ++
drivers/i2c/busses/i2c-designware-platdrv.c | 28 ++---
4 files changed, 49 insertions(+), 63 deletions(-)
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line &qu
Jani,
On Mon, Jun 15, 2015 at 02:40:42PM +0300, Jani Nikula wrote:
> On Mon, 15 Jun 2015, Ville Syrjälä wrote:
> > On Mon, Jun 15, 2015 at 01:25:38AM -0700, Jeremiah Mahler wrote:
> >> Daniel,
> >>
> >> On Mon, Jun 15, 2015 at 08:57:47AM +0200, Daniel Vetter
ing updates:
> 1) Add Gen6 trackpad device support;
> 2) Add report proximity data function support for Gen5 and Gen6 devices;
> 3) Fully support runtime suspend/resume power management;
> 4) Add of_match_table mechanism support.
>
> --
> 1.9.1
>
This patch series works good
f the active list is empty
>
> Thanks, Daniel
>
The bug is still present with that patch applied. And it is still
present up to linux-next 20150611.
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to maj
formation. */
sp: ^^^
> + cyapa_gen6_bl_read_app_info(cyapa);
> + goto out;
> + }
> +
> + cyapa->state = CYAPA_STATE_GEN6_APP;
> +
> + case CYAPA_STATE_GEN6_APP:
> + /*
> + * If trackpad device in deep sleep mode,
> + * the app command will fail.
> + * So always try to reset trackpad device to full active when
> + * the device state is requeried.
> + */
[...]
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
t; length = 0;
> - if (gen5_pip->resp_len)
> - length = *gen5_pip->resp_len;
> + if (pip->resp_len)
> + length = *pip->resp_len;
> cyapa_emp
often to remove the __exit annotation of
crypto_fpu_exit() so it may be used outside an exit section.
Fix the warning by removing the __exit annotation.
Signed-off-by: Jeremiah Mahler
---
arch/x86/crypto/fpu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/crypto/fpu
t
error: patch failed:
Documentation/devicetree/bindings/vendor-prefixes.txt:48
error: Documentation/devicetree/bindings/vendor-prefixes.txt: patch does
not apply
Patch failed at 0001 input: cyapa: add of match device support and
description document
--
- Jeremiah Mahler
--
T
all,
On Sat, Jun 06, 2015 at 08:09:34PM -0700, Jeremiah Mahler wrote:
> all,
>
> On all my machines with Intel graphics I get the following warning
> in the logs when the machine is suspended. Apparently some part of
> the graphics system is busy when it should be idle. This is p
036] xhci_hcd :00:14.0: System wakeup enabled by ACPI
[ 33.594059] PM: noirq suspend of devices complete after 16.226 msecs
[ 33.594498] ACPI: Preparing to enter system sleep state S3
[ 33.595066] ACPI : EC: EC stopped
...
--
- Jeremiah Mahler
--
To unsubscribe from this list: s
d just 11n on the other, would they
both have problems? This would tell as whether it was something inside
the kernel or if it was really wireless interference.
[...]
>
> Regards,
> Jonas Thiem
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linu
John,
On Thu, Jun 04, 2015 at 03:54:35PM -0700, John Stultz wrote:
> On Wed, Jun 3, 2015 at 5:56 PM, Jeremiah Mahler wrote:
[...]
>
>
> So I suspect the problem is the change to clock_was_set_seq in
> timekeeping_update is done prior to mirroring the time state to the
> shado
Thomas,
On Thu, Jun 04, 2015 at 01:22:45PM +0200, Thomas Gleixner wrote:
> On Wed, 3 Jun 2015, Jeremiah Mahler wrote:
[...]
>
> I had to wrap my head around that for quite a while, but I think I
> have decoded the issue. Can you please test the patch below whether it
> solv
tions(+), 13 deletions(-)
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Miklos,
On Mon, Jun 01, 2015 at 02:03:02PM +0200, Miklos Szeredi wrote:
> On Mon, Jun 1, 2015 at 3:31 AM, Jeremiah Mahler wrote:
> > fuse_conn_init(fc) zeros the entire fc structure before it assigns
> > specific values. Since fc->release is set before fuse_conn_init(fc)
ase after calling fuse_conn_init(fc) instead of before.
Signed-off-by: Jeremiah Mahler
Fixes: daccc092d1a2 ("fuse: initialize fc->release before calling it")
---
fs/fuse/inode.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/fuse/inode.c b/fs/fuse/inode.c
index 0
Miklos, all,
On Sat, May 30, 2015 at 01:03:24AM -0700, Jeremiah Mahler wrote:
> all,
>
> When running update-grub on a system with a Windows partition the kernel
> will oops with a "BUG: unable to handle kernel NULL pointer dereference"
> message. It does this dur
a2
...
Any help diagnosing this bug would be appreciated.
--
- Jeremiah Mahler
-- Logs begin at Mon 2015-02-16 08:29:12 PST, end at Fri 2015-05-29 10:59:05
PDT. --
May 29 10:54:55 hudson systemd-journal[187]: Runtime journal is using 8.0M (max
allowed 73.1M, trying to leave 109.7M free of 723.4
.unsol_event = snd_hda_jack_unsol_event,
> + .stream_pm = snd_hda_gen_stream_pm,
> #ifdef CONFIG_PM
> .suspend = via_suspend,
> .check_power_status = via_check_power_status,
That patch fixes the problem. Thanks!
--
- Jeremiah Mahler
--
To unsubscribe from
Takashi,
On Fri, May 29, 2015 at 09:37:44AM +0200, Takashi Iwai wrote:
> At Fri, 29 May 2015 00:27:14 -0700,
> Jeremiah Mahler wrote:
> >
[...]
>
> I still don't figure out why this breaks.
> Could you give alsa-info.sh output of the affected machine?
>
>
>
-
sound/pci/hda/patch_via.c | 1 -
4 files changed, 4 insertions(+), 4 deletions(-)
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org
Johannes,
On Tue, May 19, 2015 at 12:15:28AM -0700, Jeremiah Mahler wrote:
> Johannes,
>
> You mentioned off-list that you might have a fix for this somewhere in
> mac80211-next. Do you have any idea when this will make it in to -next?
> It is still broken as of
Johannes,
You mentioned off-list that you might have a fix for this somewhere in
mac80211-next. Do you have any idea when this will make it in to -next?
It is still broken as of -next 20150518.
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kerne
Dan, Johannes,
On Wed, May 13, 2015 at 06:05:31PM -0500, Dan Williams wrote:
> On Wed, 2015-05-13 at 14:49 -0700, Jeremiah Mahler wrote:
> > all,
> >
> > Running the latest linux-next (20150513) on an Acer C720 causes the
> > machine to lockup as the window manager is
ot in a form that can be applied.
- And there are probably more problems as well...
I would review how to submit patches and then try again.
I recommend watching the video by Greg Kroah-Hartman on how
to submit your first kernel patch [1].
[1]: https://www.youtube.com/watch?v=LLBrBBImJt4
[.
achments, they should be inline.
See Documentation/SubmittingPatches for more info.
[...]
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/m
Ted,
On Sat, Jan 31, 2015 at 09:02:30PM -0500, Theodore Ts'o wrote:
> On Fri, Jan 30, 2015 at 07:40:51PM -0800, Jeremiah Mahler wrote:
> >
[...]
>
> Thanks for the bug report. I was able to reproduce the problem, and
> found the problem. Unfortunately I accidental
oah-Hartman's presentation on how to write and submit your first
Linux kernel patch [1].
[1]: https://www.youtube.com/watch?v=LLBrBBImJt4
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vge
--
13 files changed, 186 insertions(+), 35 deletions(-)
If I can do anything else to help, let me know.
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://v
Daniel, Matt,
On Sat, Jan 24, 2015 at 12:24:55PM +0100, Daniel Vetter wrote:
> On Fri, Jan 23, 2015 at 10:57:32PM -0800, Jeremiah Mahler wrote:
> > all,
> >
> > On Tue, Jan 20, 2015 at 06:48:42AM +0100, Daniel Vetter wrote:
> > > On Mon, Jan 19, 2015 at 08:4
ere that the transitional plane helpers aren't up to
> the task? If so please reference that in your patch.
>
> And we still need a hack for the "changed fb cursor" issue, I'll whip
> something up.
> -Daniel
> --
> Daniel Vetter
> Software Engineer, Intel
Dudley,
On Thu, Jan 22, 2015 at 09:57:42AM +, Dudley Du wrote:
> > -Original Message-
> > From: Jeremiah Mahler [mailto:jmmah...@gmail.com]
> > Sent: 2015?1?22? 17:31
> > To: Dudley Du
[...]
> > > struct gen5_bl_metadata_row_params {
> > >
On Thu, Jan 22, 2015 at 12:54:53PM +0300, Dan Carpenter wrote:
> On Thu, Jan 22, 2015 at 01:31:29AM -0800, Jeremiah Mahler wrote:
> > I wasn't able to reproduce the sparse warning. Was it for a specific
> > architecture? Perhaps include a link to the email sent by the k
* and the row id must be 0x01ff.
> + */
> + app_integrity = &image->records[flash_records_count - 1];
> +
> + if (app_integrity->flash_array_id != 0x00 ||
> + get_unaligned_be16(&app_integrity->row_number) != 0x01ff) {
I could be wrong but it looks
atches with checkpatch.pl before submitting them.
There are a bunch of "space required after ','" errors in this patch.
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kerne
int. 'resid' is inside a while
loop which is only entered if it is positive. Casting it as a
__u32 can be perormed without a loss of data or change in
functionality.
Fix the warning by casting 'resid' as a __u32.
Reported-by: kbuild test robot
Signed-off-by: Jeremiah Mahle
revert ea2c67bb4aff
So I just reverted back to next-20150112.
git reset --hard next-20150112
I hope the owner of the prior patch responds promptly and fixes their
bug. But for the time being the cyapa patch series works when that
other bug is avoided.
If you have any suggestions, let me k
Dudley,
On Sat, Jan 17, 2015 at 02:44:15AM -0800, Jeremiah Mahler wrote:
> Dudley,
>
> Your patch series applies without issue but unfortunately I
> could not test them because I ran in to a bug created by someone
> elses patch (not yours) [1].
>
> [1]: https://lkml.
t; the "case" labels.
>
It is looking better. Now put this in the patch log message and send
version 2.
> Thanks & Regards,
> Shailendra
> On Jan 17, 2015 4:28 PM, "Jeremiah Mahler" wrote:
>
> > Shailendra,
> >
[...]
--
- Jeremiah Mahler
--
To
d.
Was it white space fixes? Was it indentation of cases statements?
> Thanks & Regards,
> Shailendra
> On Jan 17, 2015 3:45 PM, "Jeremiah Mahler" wrote:
>
> > Shailendra,
> >
> > Please describe in detail which specific coding issues you fixed.
> > I
nter bootloader failed for some chipset takes longer time issue.
[...]
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.h
Verma wrote:
>
> Signed-off-by: Shailendra Verma
> ---
> drivers/input/evdev.c| 30 --
> drivers/input/mousedev.c | 18 --
> 2 files changed, 32 insertions(+), 16 deletions(-)
[...]
--
- Jeremiah Mahler
--
To unsubscribe fr
Signed-off-by: Matt Roper
Reviewed-by: Ander Conselvan de Oliveira
Signed-off-by: Daniel Vetter
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info a
it is helpful to add a version number so people
can tell that this is a new revision. For example: [PATCH v2].
git format-patch can add this automatically with the '-v' option.
[...]
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kern
e (struct libcfs_device_userstate *ldu)
> +static void kportal_memhog_free(struct libcfs_device_userstate *ldu)
> {
[...]
I would have included this change in with the previous patch since it is
that patch that has the checkpatch warning.
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the li
't used outside of this file so making them static
looks correct.
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
minimum.
Fixes: 6dda730e55f4 ("respect the VBT minimum backlight brightness")
Signed-off-by: Jeremiah Mahler
---
Notes:
Changes in v2:
- Expand the comparision for the possible case when
the level is less than the minimum.
drivers/gpu/drm/i915/intel_panel.c | 2
Jani,
On Mon, Jan 12, 2015 at 12:31:09PM +0200, Jani Nikula wrote:
> On Sat, 10 Jan 2015, Jeremiah Mahler wrote:
[...]
>
> I think part of the problem is that the userspace sets brightness to
> minimum before suspend, but apparently does not restore it after
> resume. The dmes
line "unsubscribe linux-doc" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
Series looks good.
I'm glad I am not the only one with OCD around here :-)
Reviewed-by: Jeremiah Mahler
--
- Jeremiah Mahler
--
T
current
design.
[...]
Reviewed-by: Jeremiah Mahler
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
errors.
Change the default output for the generic serial driver from error to
debug to silence these non-critical errors.
Signed-off-by: Jeremiah Mahler
---
drivers/usb/serial/generic.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/usb/serial/generic.c b/drivers/usb
usb_serial_generic_submit_read_urb() handles quietly without an error.
Signed-off-by: Jeremiah Mahler
---
drivers/usb/serial/generic.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/usb/serial/generic.c b/drivers/usb/serial/generic.c
index 2d7207b..ccf1df7 100644
what host controller and devices are used.
- fix subject: it is silencing all errors, not just one as in
previous versions.
Changes in v2:
- Instead of handling -EPROTO specially, use dev_dbg instead of
dev_err like other drivers do.
Jeremiah Mahler (2):
usb: serial: silence a
Johan,
On Sun, Jan 11, 2015 at 12:36:18PM +0100, Johan Hovold wrote:
> On Sat, Jan 10, 2015 at 04:44:32PM -0800, Jeremiah Mahler wrote:
> > If a USB serial device is unplugged while there is an active program
> > using the device it will spam the logs with -EPROTO (71) messages as
usb_serial_generic_submit_read_urb() handles quietly without an error.
Signed-off-by: Jeremiah Mahler
---
drivers/usb/serial/generic.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/usb/serial/generic.c b/drivers/usb/serial/generic.c
index 2d7207b..ccf1df7 100644
errors.
Change the default output for the generic serial driver from error to
debug.
Signed-off-by: Jeremiah Mahler
---
drivers/usb/serial/generic.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/usb/serial/generic.c b/drivers/usb/serial/generic.c
index 1bd1922
behavior.
This change is beneficial when using daemons such as slcand, which is
similar to pppd or slip, that cannot determine whether they should exit
until after the USB serial device is unplugged. Producing these error
messages for a normal use case is not helpful.
Jeremiah Mahler (2):
usb
ned-off-by: Jeremiah Mahler
---
drivers/gpu/drm/i915/intel_panel.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/intel_panel.c
b/drivers/gpu/drm/i915/intel_panel.c
index 4d63839..4ef4d66 100644
--- a/drivers/gpu/drm/i915/intel_panel.c
+++ b/drivers/gp
tualize the max value
exposed to the userspace.
Signed-off-by: Jani Nikula
Reviewed-by: Jesse Barnes
[danvet: s/BUG_ON/WARN_ON/]
Signed-off-by: Daniel Vetter
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the bo
.@rad-ic.com
You need your full legal name in your Signed-off-by:
--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
struct spi_device *spi)
> return spi_gpio;
> }
>
> -static inline struct spi_gpio_platform_data * __pure
> +static inline struct spi_gpio_platform_data *__pure
> spi_to_pdata(const struct spi_device *spi)
> {
> return &spi_to_spi_gpio(spi)->pdata;
>
1 - 100 of 268 matches
Mail list logo
