().
Reviewed-by: Andy Shevchenko
Signed-off-by: Hans de Goede
---
Changes in v3:
- This is a new patch in v3 of this patch-set
---
drivers/extcon/extcon-arizona.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/extcon/extcon-arizona.c b/drivers/extcon/extcon-arizona.c
ndy Shevchenko
Signed-off-by: Hans de Goede
---
sound/soc/codecs/arizona-jack.c | 42 ++---
sound/soc/codecs/arizona.h | 1 -
2 files changed, 18 insertions(+), 25 deletions(-)
diff --git a/sound/soc/codecs/arizona-jack.c b/sound/soc/codecs/arizona-jack.c
index 5b40316d0
-drivers for codecs which are converted
to use the new helper functions from arizona-jack.c are modified to
create a snd_soc_jack through snd_soc_card_jack_new() and register
this jack with the codec through snd_soc_component_set_jack().
Reviewed-by: Andy Shevchenko
Signed-off-by: Hans de Goede
---
cs/arizona-jack.c .
This is a preparation patch for converting the arizona extcon-driver into
a helper library for letting the arizona codec-drivers directly report jack
state through the standard sound/soc/soc-jack.c functions.
Signed-off-by: Hans de Goede
---
Changes in v4:
- Add sound/so
.
Reviewed-by: Andy Shevchenko
Acked-by: Charles Keepax
Signed-off-by: Hans de Goede
---
sound/soc/codecs/arizona-jack.c | 97 ++---
sound/soc/codecs/arizona.h | 36
2 files changed, 65 insertions(+), 68 deletions(-)
diff --git a/sound/soc/codecs
d-by: Andy Shevchenko
Signed-off-by: Hans de Goede
---
sound/soc/codecs/arizona-jack.c | 149 +---
sound/soc/codecs/arizona.h | 7 +-
2 files changed, 47 insertions(+), 109 deletions(-)
diff --git a/sound/soc/codecs/arizona-jack.c b/sound/soc/codecs/arizona-ja
ned-off-by: Hans de Goede
---
sound/soc/codecs/wm5102.c | 12 +++-
sound/soc/codecs/wm5110.c | 12 +++-
sound/soc/codecs/wm8997.c | 14 --
sound/soc/codecs/wm8998.c | 9 +
4 files changed, 43 insertions(+), 4 deletions(-)
diff --git a/sound/soc/codecs/wm5102.c b
Add jack detect support by creating a jack and calling
snd_soc_component_set_jack to register the created jack
with the codec.
Reviewed-by: Andy Shevchenko
Signed-off-by: Hans de Goede
---
sound/soc/intel/boards/bytcr_wm5102.c | 28 ++-
1 file changed, 27 insertions
Cleanup the use of dev_foo functions used for logging:
1. Many of these are unnecessarily split over multiple lines
2. Use dev_err_probe() in cases where we might get a -EPROBE_DEFER
return value
Suggested-by: Andy Shevchenko
Signed-off-by: Hans de Goede
---
Changes in v4:
- While at it
Hi,
On 1/24/21 8:53 PM, Andy Shevchenko wrote:
> On Sat, Jan 23, 2021 at 2:17 PM Hans de Goede wrote:
>>
>> Cleanup the use of dev_foo functions used for logging:
>>
>> 1. Many of these are unnecessarily split over multiple lines
>> 2. Use dev_err_prob
Hi,
On 1/27/21 11:01 PM, Pavel Machek wrote:
> Hi!
>
> Booting a 5.11-rc2 kernel with lockdep enabled inside a virtualbox vm
> (which still
> emulates good old piix ATA controllers) I get the below lockdep splat
> early on during boot:
>
> This seems to be led-class rela
Hi,
On 1/6/21 10:01 AM, Joe Perches wrote:
> On Wed, 2021-01-06 at 14:36 +0800, YANG LI wrote:
>> The snprintf() function returns the number of characters which would
>> have been printed if there were enough space, but the scnprintf()
>> returns the number of characters which were actually printe
Hi,
On 1/5/21 6:18 PM, Rafael J. Wysocki wrote:
> On Mon, Jan 4, 2021 at 9:58 PM Hans de Goede wrote:
>> Patch 1/2 does use a slightly different approach then I suggest above,
>> thinking more about this it would be cleaner IMHO to just pass the
>> cur_profile pointer to
Hi Filipe,
On 1/6/21 8:07 PM, Filipe Laíns wrote:
> Hey,
>
> Some of the new Logitech receivers do not have the DJ interface, this creates
> an
> issue userspace applications like libratbag, as seen in [1], because we can't
> identify the device based on the hidraw PID.
>
> There are two soluti
Hi,
On 12/21/20 7:39 PM, Maximilian Luz wrote:
> Hello,
>
> Here is version three of the Surface System Aggregator Module (SAM/SSAM)
> driver series, adding initial support for the embedded controller on 5th
> and later generation Microsoft Surface devices. Initial support includes
> the ACPI int
Hi Perry,
On 12/28/20 2:28 PM, Perry Yuan wrote:
> From: Perry Yuan
>
> add support for dell privacy driver for the dell units equipped
> hardware privacy design, which protect users privacy
> of audio and camera from hardware level. once the audio or camera
> privacy mode enabled, any applicati
ake more than 1 second
> but less than 2.5 seconds and emit a warning on resume for those
> scenarios.
>
> Suggested-by: Aaron Ma
> Suggested-by: Sasha Netfin
> Suggested-by: Hans de Goede
> CC: Mark Pearson
> Fixes: f15bb6dde738cc8fa0 ("e1000e: Add support for S0ix&
nses) that it may
> warrant caching their allocations to reduce possible memory
> fragmentation.
>
> Signed-off-by: Maximilian Luz
Thanks, patch looks good to me:
Reviewed-by: Hans de Goede
Regards,
Hans
> ---
> drivers/platform/surface/aggregator/core.c| 27 ++-
ost other event types. In case of a larger event payload, event item
> allocation will fall back to kzalloc().
>
> Signed-off-by: Maximilian Luz
Thanks, patch looks good to me:
Reviewed-by: Hans de Goede
Regards,
Hans
> ---
> .../platform/surface/aggregator/controller.c |
th the error injection capabilities introduced in the
> subsequent commit.
>
> Signed-off-by: Maximilian Luz
Thanks, patch looks good to me:
Reviewed-by: Hans de Goede
Regards,
Hans
> ---
>
> Changes in v1 (from RFC):
> - add copyright line
> - change SPDX id
ebugging of
> the Surface Aggregator communication layer.
>
> Signed-off-by: Maximilian Luz
Thanks, patch looks good to me:
Reviewed-by: Hans de Goede
Regards,
Hans
> ---
>
> Changes in v1 (from RFC):
> - remove unnecessary default in Kconfig entry
>
> Cha
to manually establish device links for that purpose
> in the client driver (as has to be done with the platform devices).
>
> Signed-off-by: Maximilian Luz
Thanks, patch looks good to me:
Reviewed-by: Hans de Goede
Regards,
Hans
> ---
>
> Changes in v1 (from RFC):
&g
to manually establish device links for that purpose
> in the client driver (as has to be done with the platform devices).
>
> Signed-off-by: Maximilian Luz
Thanks, patch looks good to me:
Reviewed-by: Hans de Goede
Regards,
Hans
> ---
>
> Changes in v1 (from RFC):
&g
>
> Signed-off-by: Maximilian Luz
Thanks, patch looks good to me:
Reviewed-by: Hans de Goede
Regards,
Hans
> ---
>
> Changes in v1 (from RFC):
> - change SPDX identifier to GPL-2.0+ (was GPL-2.0-or-later)
> - fix typos
>
> Changes in v2:
> - add archit
Hi,
On 12/3/20 10:26 PM, Maximilian Luz wrote:
> Add a misc-device providing user-space access to the Surface Aggregator
> EC, mainly intended for debugging, testing, and reverse-engineering.
> This interface gives user-space applications the ability to send
> requests to the EC and receive the co
re required to provide said
> functionality on those devices.
>
> Signed-off-by: Maximilian Luz
Thanks, patch looks good to me:
Reviewed-by: Hans de Goede
Regards,
Hans
> ---
>
> Changes in v1 (from RFC):
> - add copyright lines
> - change SPDX identifier to GPL-2.0
Hi,
On 12/16/20 1:17 AM, Linus Torvalds wrote:
> On Mon, Dec 14, 2020 at 4:43 AM Hans de Goede wrote:
>>
>> - New Intel PMT telemetry and crashlog drivers
>
> These have _very_ annoying Kconfig setups.
>
> First it asks about INTEL_PMT support.
>
>
soft Surface drivers, even though this question is
> irrelevant.
>
> Fix this by moving the dependency on ACPI from the individual driver
> symbols to SURFACE_PLATFORMS.
>
> Signed-off-by: Geert Uytterhoeven
Thanks, patch looks good to me:
Reviewed-by: Hans de Goede
Maximilian, c
around for BIOS bug
Elia Devito (2):
platform/x86: intel-hid: Add support for SW_TABLET_MODE
platform/x86: intel-hid: Add alternative method to enable switches
Hans de Goede (14):
platform/x86: touchscreen_dmi: Add info for the Predia Basic tablet
platform/x86: acer-wmi: Drop no-
Hi All,
Sasha (and the other intel-wired-lan folks), thank you for investigating this
further and for coming up with a better solution.
Mario, thank you for implementing the new scheme.
I've tested this patch set on a Lenovo X1C8 with vPRO and AMT enabled in the
BIOS
(the previous issues were s
Hi,
On 12/14/20 4:34 PM, Mario Limonciello wrote:
> This flag can be used by an end user to disable S0ix flows on a
> buggy system or by an OEM for development purposes.
>
> If you need this flag to be persisted across reboots, it's suggested
> to use a udev rule to call adjust it until the kerne
ontrol methods used for device enumeration.
Thank you for you work on this!
I started with reviewing the series so that I would know what to
expect during testing. All 3 patches look good to me:
Reviewed-by: Hans de Goede
As is the series does not fix the Bluetooth HID misreporting on
the s
Hi,
On 12/14/20 8:36 PM, Limonciello, Mario wrote:
>> Hi All,
>>
>> Sasha (and the other intel-wired-lan folks), thank you for investigating this
>> further and for coming up with a better solution.
>>
>> Mario, thank you for implementing the new scheme.
>>
>
> Sure.
>
>> I've tested this patch
r.
> - Increase ULP timeout to 2.5 seconds, but show a warning after 1 second.
Thank you. I've given v5 a test on a Lenovo X1 Carbon 8th gen (AMT capable)
and things work fine there with v5:
Tested-by: Hans de Goede
Regards,
Hans
> Changes from v3 to v4:
> - Drop patch 1 for p
adjust it until the kernel could have your
> configuration in a disallow list.
>
> Signed-off-by: Mario Limonciello
Thanks, patch looks good to me:
Reviewed-by: Hans de Goede
Regards,
Hans
> ---
> drivers/net/ethernet/intel/e1000e/e1000.h | 1 +
> drivers/ne
systems")
> Reviewed-by: Alexander Duyck
> Signed-off-by: Mario Limonciello
Thanks, patch looks good to me:
Reviewed-by: Hans de Goede
Regards,
Hans
> ---
> drivers/net/ethernet/intel/e1000e/netdev.c | 45 +-
> 1 file changed, 2 insertions(+), 43 del
:
- Add info for the Irbis TW118 tablet
- Add info for the Predia Basic tablet
Benjamin Berg (1):
platform/x86: thinkpad_acpi: Send tablet mode switch at wakeup time
Hans de Goede (4):
platform/x86: thinkpad_acpi: Do not
Hi,
On 11/24/20 11:27 AM, Christoph Hellwig wrote:
> On Mon, Nov 23, 2020 at 03:49:09PM +0100, Hans de Goede wrote:
>> Hi,
>>
>> +Cc Christoph Hellwig
>>
>> Christoph, this is still an issue, so I've been looking around a bit and
>> think this
>&g
Hi,
On 11/27/20 12:41 PM, Hans de Goede wrote:
> Hi,
>
> On 11/24/20 11:27 AM, Christoph Hellwig wrote:
>> On Mon, Nov 23, 2020 at 03:49:09PM +0100, Hans de Goede wrote:
>>> Hi,
>>>
>>> +Cc Christoph Hellwig
>>>
>>> Christoph, t
Hi,
On 11/27/20 3:58 PM, Mike Travis wrote:
>
>
> On 11/26/2020 2:45 AM, Hans de Goede wrote:
>> Hi,
>>
>> On 11/25/20 6:29 PM, Mike Travis wrote:
>>> Add "deprecated" message to any access to old /proc/sgi_uv/* leaves.
>>>
>
Hi,
On 11/27/20 5:19 PM, Christoph Hellwig wrote:
> On Fri, Nov 27, 2020 at 01:32:16PM +0100, Hans de Goede wrote:
>> I ran some more tests, I can confirm that reverting:
>>
>> 5df7ef7d32fe "uas: bump hw_max_sectors to 2048 blocks for SS or faster
>> drives"
i_add_host())?
I would expect that to work / avoid the regression, so yes that is
a good option.
If you can provide me with a patch doing that, then I can test it to
make sure it does indeed fix the regression.
Regards,
Hans
>
> On Sat, 28 Nov 2020 at 02:12, Hans de Goede wrote:
previous set).
Here is my ack for the drivers/platform/x86 bits being merged through the
x86/tip tree:
Acked-by: Hans de Goede
Regards,
Hans
>
> v2: Updated to apply to v3 of dependency patch set listed above.
>
> Mike Travis (5):
> x86/platform/uv: Add kernel interfa
Hi All,
On 11/29/20 4:23 AM, Jarkko Sakkinen wrote:
> On Tue, Nov 24, 2020 at 10:45:01PM +0100, Hans de Goede wrote:
>> Hi,
>>
>> On 11/24/20 6:52 PM, Jerry Snitselaar wrote:
>>>
>>> Jarkko Sakkinen @ 2020-11-23 20:26 MST:
>>>
>>>> On W
Hi Stephen,
On 11/29/20 6:43 PM, Stephen Rothwell wrote:
> Hi all,
>
> In commit
>
> 912b341585e3 ("platform/x86: mlx-platform: Remove PSU EEPROM from MSN274x
> platform configuration")
>
> Fixes tag
>
> Fixes: ef08e14a3 ("platform/x86: mlx-platform: Add support for new msn274x
> system
Hi,
On 11/20/20 12:11 PM, Matthias Brugger wrote:
>
>
> On 20/11/2020 11:05, Dmitry Osipenko wrote:
>> 20.11.2020 12:52, matthias@kernel.org пишет:
>>> From: Matthias Brugger
>>>
>>> Apart from a firmware binary the chip needs a config file used by the
>>> FW. Add the config files to modinf
Hi,
On 11/23/20 7:53 AM, Dmitry Torokhov wrote:
> On Fri, Oct 30, 2020 at 10:56:20AM +0100, Hans de Goede wrote:
>> Hi,
>>
>> On 10/29/20 9:36 PM, Dmitry Torokhov wrote:
>>> Hi Andrej,
>>>
>>> On Thu, Oct 29, 2020 at 06:03:11PM +0100, Andrej V
irq_set_irq_type(irq, IRQ_TYPE_LEVEL_LOW);
> ^
> 2 errors generated.
>
> Fixes: 78a5b53e9fb4 ("Input: soc_button_array - work around DSDTs which
> modify the irqflags")
> Reported-by: kernel test robot
> Signed-off-by: Dmitry Torokh
Hi,
On 02/22/2014 09:31 AM, Maxime Ripard wrote:
This should be before the registration. Otherwise, you're racy.
Nope, we only need this to get the data on sunxi_mmc_remove,
everywhere else the data is found through the mmc-host struct.
Still, if anyone makes a following patch using the p
and not on the irqchip.
Cc: Carlo Caione
Signed-off-by: Hans de Goede
---
include/linux/interrupt.h | 2 ++
kernel/irq/chip.c | 4
kernel/irq/internals.h| 2 ++
kernel/irq/manage.c | 5 -
4 files changed, 12 insertions(+), 1 deletion(-)
diff --git a/include/linux
Hi All,
So after sleeping a night on this I'm not so sure anymore this patch
is such such a good idea. So self-nack.
Still I welcome feedback on this issue...
On 03/12/2014 12:13 AM, Hans de Goede wrote:
> In some cases we want to do an ack right before the unmask of an irq.
>
The comment was claiming that we were masking all irqs, while the code actually
*un*masks all of them.
Signed-off-by: Hans de Goede
Acked-by: Maxime Ripard
---
drivers/irqchip/irq-sun4i.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/irqchip/irq-sun4i.c b/drivers
The ENMI needs to have the ack done *after* clearing the interrupt source,
otherwise we will get a spurious interrupt for each real interrupt. Switch
to the new handle_fasteoi_late_irq handler which gives us the desired behavior.
Signed-off-by: Hans de Goede
---
drivers/irqchip/irq-sun4i.c | 11
Hi,
On 03/12/2014 11:38 AM, Thomas Gleixner wrote:
> On Wed, 12 Mar 2014, Hans de Goede wrote:
> And how is that different from the non threaded case?
>
> mask()
> ack() <-- irq line is still active
> handle() <-- irq line goes i
after having
read the vector-reg once.
Signed-off-by: Hans de Goede
---
drivers/irqchip/irq-sun4i.c | 18 --
1 file changed, 16 insertions(+), 2 deletions(-)
diff --git a/drivers/irqchip/irq-sun4i.c b/drivers/irqchip/irq-sun4i.c
index a5438d8..5c25048 100644
--- a/drivers
be issued right before the unmask.
Signed-off-by: Hans de Goede
> In some cases we want to do an ack right before the unmask of an irq.
>
> A typical example of such a case is having an i2c device which uses a level
> interrupt. Such devices usually have an interrupt status register
Hi All,
Here is v2 of my sun4i IRQ 0 / ENMI fixes fixes, changelog below:
v2:
PATCH 1/4 irq: Add handle_fasteoi_late_irq irq handler
New patch by Thomas Gleixner, adding a new irq handler with the "call eoi
after the irq-handler has run its course" behavior we need to avoid double
irqs on the EN
Hi,
On 03/12/2014 06:17 PM, Hans de Goede wrote:
> From: Thomas Gleixner
>
> Add an irq handler for transparent controllers
>
> Only a single callback will be issued to the chip: an ->eoi() call when
> the interrupt has been serviced. Same as handle_fasteoi_irq, but
> w
Hi,
On 03/13/2014 12:12 PM, Thomas Gleixner wrote:
> On Thu, 13 Mar 2014, Maxime Ripard wrote:
>
>> On Wed, Mar 12, 2014 at 06:17:07PM +0100, Hans de Goede wrote:
>>> The ENMI needs to have the ack done *after* clearing the interrupt source,
>>> otherwise we wil
Hi,
On 03/13/2014 03:46 PM, Thomas Gleixner wrote:
> On Wed, 12 Mar 2014, Hans de Goede wrote:
>
>> The ENMI needs to have the ack done *after* clearing the interrupt source,
>> otherwise we will get a spurious interrupt for each real interrupt. Switch
>> to the new
k conditions were true for the
ONESHOT case the code would do: "unmask; eoi", now it does
"eoi; unmask" I believe the new behavior is more correct, but, since
it is a behavior change I thought I should point this out.
I've also given this a test-run on sun4i and it works as
The comment was claiming that we were masking all irqs, while the code actually
*un*masks all of them.
Signed-off-by: Hans de Goede
Acked-by: Maxime Ripard
---
drivers/irqchip/irq-sun4i.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/irqchip/irq-sun4i.c b/drivers
Hi All,
Here is v3 of my sun4i IRQ 0 / ENMI fixes patch-set, changelog below:
v2:
PATCH 1/4 irq: Add handle_fasteoi_late_irq irq handler
New patch by Thomas Gleixner, adding a new irq handler with the "call eoi
after the irq-handler has run its course" behavior we need to avoid double
irqs on th
handle_fasteoi_irq. This uses a separate irq_chip struct for IRQ 0,
since we only want this behavior for IRQ 0.
Signed-off-by: Hans de Goede
---
drivers/irqchip/irq-sun4i.c | 19 ---
1 file changed, 16 insertions(+), 3 deletions(-)
diff --git a/drivers/irqchip/irq-sun4i.c b/drivers
From: Thomas Gleixner
This flag must be used in combination with handle_fasteoi_irq, when set
handle_fasteoi_irq will delay the calling of chip->irq_eoi until the threaded
handler has run.
Reviewed-by: Hans de Goede
Tested-by: Hans de Goede
Signed-off-by: Hans de Goede
---
include/li
after having
read the vector-reg once.
Signed-off-by: Hans de Goede
Acked-by: Maxime Ripard
---
drivers/irqchip/irq-sun4i.c | 18 --
1 file changed, 16 insertions(+), 2 deletions(-)
diff --git a/drivers/irqchip/irq-sun4i.c b/drivers/irqchip/irq-sun4i.c
index a5438d8..5c25048
Hi,
On 03/13/2014 07:03 PM, Hans de Goede wrote:
> From: Thomas Gleixner
>
> This flag must be used in combination with handle_fasteoi_irq, when set
> handle_fasteoi_irq will delay the calling of chip->irq_eoi until the threaded
> handler has run.
>
> Reviewed-by: Ha
Since sun4i and sun5i are single core SOCs there is no need to mask non
oneshot IRQs, to achieve this we use handle_fasteoi_irq with a dummy eoi.
Signed-off-by: Hans de Goede
---
drivers/irqchip/irq-sun4i.c | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/drivers
Now that we're using handle_fasteio_irq for all interrupts, we can get rid
of having 2 irq_chip structs by making sun4i_irq_ack a nop for all irqs
except irq 0.
Signed-off-by: Hans de Goede
---
drivers/irqchip/irq-sun4i.c | 27 ---
1 file changed, 4 insertions(+
Hi All,
So here is a second patchset for irq-sun4i.c I'm not 100% sure about these
changes. They all seem the right thing to do, but they definitely need a
good review first.
I've run various tests with this set and everything works as advertised.
Regards,
Hans
--
To unsubscribe from this list:
register like this, if the other bits are writeable (and the data sheet says
they are not) they should be write 1 to clear, since otherwise a read /
modify / write can race with a device raising an interrupt and then clear
the pending bit unintentionally
Signed-off-by: Hans de Goede
Hi,
On 03/14/2014 12:35 PM, Thomas Gleixner wrote:
> On Thu, 13 Mar 2014, Hans de Goede wrote:
>
>> Since sun4i and sun5i are single core SOCs there is no need to mask non
>> oneshot IRQs, to achieve this we use handle_fasteoi_irq with a dummy eoi.
>
> This is slightly
Hi All,
Here is v2 of my patchset for sun4i-irq.c to use handle_fasteoi_irq for all
irqs + follow up clean-up patch.
Changes since v2:
-adjust commit msg based on Thomas' comments, and merge patch 1 and 2 as
they make more sense as 1 patch
Regards,
Hans
--
To unsubscribe from this list: send t
register like this, if the other bits are writeable (and the data sheet says
they are not) they should be write 1 to clear, since otherwise a read /
modify / write can race with a device raising an interrupt and then clear
the pending bit unintentionally
Signed-off-by: Hans de Goede
EADED | IRQCHIP_EOI_IF_HANDLED will now influence
all interrupts rather then just irq 0, but that does not matter as the eoi
is now a nop anyways for all interrupts but irq 0.
Signed-off-by: Hans de Goede
---
drivers/irqchip/irq-sun4i.c | 18 --
1 file changed, 4 insertions(+), 14 deletions(-)
diff --
Hi,
On 03/16/2014 03:05 PM, Tim Gardner wrote:
> Gents - while reviewing some patches for arm64 in Ubuntu I noticed that
> there is at least one patch sequence in linux-next that breaks the build.
>
> 156c5887948cd191417f18026aab9ce26e5a95da ahci-platform: Add support for
> devices with more t
Hi,
On 03/17/2014 02:06 PM, Bartlomiej Zolnierkiewicz wrote:
> Signed-off-by: Bartlomiej Zolnierkiewicz
Thanks for doing these clean-ups.
Acked-by: Hans de Goede
Regards,
Hans
> ---
> drivers/ata/ahci_sunxi.c |2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
&
Hi,
On 03/17/2014 02:08 PM, Bartlomiej Zolnierkiewicz wrote:
> Signed-off-by: Bartlomiej Zolnierkiewicz
Thanks for doing these clean-ups.
Acked-by: Hans de Goede
Regards,
Hans
> ---
> drivers/ata/ahci_sunxi.c | 44 ++--
> 1 file
Hi,
On 02/28/2014 05:37 PM, Maxime Ripard wrote:
> In order for the DMA controller to work for SDRAM to devices transfers, the
> AHB
> clock should be reparented on the PLL6.
>
> Force that parenting in the clock driver.
>
> Signed-off-by: Maxime Ripard
> ---
> drivers/clk/sunxi/clk-sunxi.c |
working on this.
Patches 1/2 seem sensible to me, and 3/4 are obviously a good idea. Also
all patches look good to me, so this series is:
Acked-by: Hans de Goede
Regards,
Hans
>
> Best regards,
> --
> Bartlomiej Zolnierkiewicz
> Samsung R&D Institute Poland
> Samsung Electr
Hi,
On 02/11/2014 03:17 PM, Nishanth Menon wrote:
> On 02/11/2014 07:53 AM, Roger Quadros wrote:
>> On 02/10/2014 08:10 PM, Roger Quadros wrote:
>>> The OMAP EHCI controller is not compatible with the EHCI
>>> platform HCD driver so don't claim that we are.
>>>
>>> This fixes boot on OMAP platform
Hi,
On 02/19/2014 12:36 AM, Tejun Heo wrote:
> On Mon, Feb 17, 2014 at 12:56:58PM +, Lee Jones wrote:
>> Cc: Tejun Heo
>> Cc: linux-...@vger.kernel.org
>> Signed-off-by: Lee Jones
>
> Please consider basing it on top of Hans' refactored ahci_platform.
>
> http://www.spinics.net/lists/linu
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Hi,
On 02/19/2014 10:46 AM, Maxime Ripard wrote:
> Hi Hans,
>
> On Tue, Feb 18, 2014 at 09:49:21PM +0100, Hans de Goede wrote:
>> Hi,
>>
>> On 02/18/2014 04:37 PM, Maxime Ripard wrote:
>>
>>
>>
&g
Hi,
On 02/20/2014 08:30 PM, Chris Rankin wrote:
>
>
> Hi,
>
> I have an old Logitech webcam, with USB IDs 046d:08b3. When I try to use this
> camera now, I see this error in the dmesg log:
>
> [ 2883.852464] pwc: isoc_init() submit_urb 0 failed with error -28
>
>
> This error is apparently
Hi,
On 01/22/2014 09:11 AM, Roger Quadros wrote:
Hi,
On 01/21/2014 03:59 PM, Hans de Goede wrote:
Hi,
On 01/21/2014 12:59 PM, Roger Quadros wrote:
On 01/21/2014 10:34 AM, Roger Quadros wrote:
On 01/20/2014 06:48 PM, Hans de Goede wrote:
Hi,
On 01/20/2014 03:41 PM, Roger Quadros wrote
Hi,
On 01/08/2014 12:28 PM, Roger Quadros wrote:
On 01/08/2014 03:35 PM, Arnd Bergmann wrote:
On Wednesday 08 January 2014 15:29:18 Kishon Vijay Abraham I wrote:
+ hpriv->phy = devm_phy_get(dev, "sata-phy");
+ if (IS_ERR(hpriv->phy)) {
+ dev_dbg(dev, "can't get sata-phy\n")
Hi,
On 01/08/2014 02:14 PM, Hans de Goede wrote:
Hi,
On 01/08/2014 12:28 PM, Roger Quadros wrote:
On 01/08/2014 03:35 PM, Arnd Bergmann wrote:
On Wednesday 08 January 2014 15:29:18 Kishon Vijay Abraham I wrote:
+ hpriv->phy = devm_phy_get(dev, "sata-phy");
+ if (IS_E
all that ugly :)
I've also run various tests and it seems to work as advertised (I've not
managed to trigger the error path though AFAIK).
Acked-by: Hans de Goede
Sarah Sharp
drivers/usb/host/xhci-mem.c | 106 +---
1 file changed, 80 insert
e the code cleaner, and those functions are moved to
make them static.]
Signed-off-by: Gerd Hoffmann
Signed-off-by: Hans de Goede
Signed-off-by: Sarah Sharp
---
drivers/usb/host/xhci-mem.c | 132 +---
drivers/usb/host/xhci.h | 1 +
2 files changed, 9
Hi Tejun,
I think it would be a good idea to merge ahci upstream using the
ahci_imx.c method for now. You already indicated that you were not against
doing that for now.
Oliver is working on getting a cleaner solution for this, but doing this
properly takes tinme, and we would like to move forwa
Hi,
On 07/18/2014 09:11 AM, Mikko Perttunen wrote:
> This patch adds device tree binding documentation for the SATA
> controller found on NVIDIA Tegra SoCs.
>
> Signed-off-by: Mikko Perttunen
> ---
> v5: remove ordering requirement again
Acked-by: Hans de Goed
andle it ourselves.
> this allows use of ahci_platform while having a special sequence
> for the clock.
Thanks, I like this one, much better then what we had before.
Acked-by: Hans de Goede
Regards,
Hans
>
> drivers/ata/Kconfig | 9 ++
> drivers/ata/Makefile | 1 +
Hi,
On 07/18/2014 09:16 AM, Mikko Perttunen wrote:
> So here's v5: this time, as suggested, I handle the sata clock myself and let
> ahci_platform handle it too, leading it to be prepared+enabled twice. This
> works fine, and allows us to remove the DT ordering requirement.
>
> I also have in t
Hi,
On 07/18/2014 02:30 PM, Antoine Ténart wrote:
> The ahci_platform driver is a generic driver using the libahci_platform
> functions. Add a generic compatible to avoid having an endless list of
> compatibles with no differences for the same driver.
>
> Signed-off-by: Antoine Ténart
> ---
> d
Hi,
On 07/18/2014 03:57 PM, Tejun Heo wrote:
(cc'ing Hans who's now maintaining libahci-platform.)
Note I was already following this thread as I'm subscribed to
linux-ide now.
On Fri, Jul 18, 2014 at 02:29:59PM +0200, Antoine Ténart wrote:
Tejun, Kishon, Sebastian,
I looked into the AHCI
Hi,
On 07/19/2014 12:18 PM, Hans de Goede wrote:
The problem is that:
1) We need to enable resources before we can do ahci_save_initial_config()
2) We must do ahci_save_initial_config() before we can do ata_host_alloc_pinfo()
3) Therefor we don't have port_info at enable_resources
Hi,
On 04/23/2014 01:36 PM, Kefeng Wang wrote:
> Add a dynamic host flags argument to make ahci_platform_init_host more
> flexible,
> then remove the AHCI_HFLAGS(...) argument from some driver's ata_port_info,
> and pass that in as the new arguments.
>
> Cc: Hans de Geode
> Signed-off-by: Kefen
Hi,
On 04/23/2014 02:36 PM, Kefeng Wang wrote:
> channge in v5:
>
> 1) drop AHCI_HFLAGS(...) from ahci_xgene.c and do use host_flags
>in ahci_platform_init_host, it's my carelessness.
>
Thanks, looks good now, this series is:
Reviewed-by: Hans de Goede
Regards,
Ha
len(r->s);
^
In file included from scripts/kconfig/zconf.tab.c:2537:0:
scripts/kconfig/menu.c:551:19: note: ‘jump’ was declared here
struct jump_key *jump;
Signed-off-by: Hans de Goede
---
scripts/kconfig/menu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff -
Hi,
On 07/31/2014 10:10 PM, Jeroen Hofstee wrote:
> Hello Hans,
>
> On 31-07-14 16:21, Hans de Goede wrote:
>> This fixes the following compiler warning:
>>
>> In file included from scripts/kconfig/zconf.tab.c:2537:0:
>> scripts/kconfig/menu.c: In function ‘ge
1001 - 1100 of 2701 matches
Mail list logo
