On Fri, Jun 07, 2019 at 09:34:18AM -0700, Ajay Gupta wrote:
> Hi Heikki and Wolfram
> The latest set (v5) fix comments from Wolfram on further refactoring
> master_xfer() function in i2c-nvidia-gpuc.c file and removing extra comments
> in patch 2/5.
>
> Patches can go through either usb or i2c tre
> Changes from v3->v4:
> - Further refactor master_xfer based on Wolfram's comment.
Yay, looks even better. One thing to improve, though.
> status = gpu_i2c_stop(i2cd);
send_stop = false;
> - if (status < 0)
> - return status;
> + if (status < 0) {
> +
> + pm_runtime_mark_last_busy(i2cd->dev);
> + pm_runtime_put_autosuspend(i2cd->dev);
Much better to have this only once!
> +/*
> + * We need gpu_i2c_suspend() even if it is stub, for runtime pm to work
> + * correctly. Without it, lspci shows runtime pm status as "D0" for the card.
> + *
On Fri, Jun 07, 2019 at 11:25:10AM +0300, Heikki Krogerus wrote:
> On Mon, Jun 03, 2019 at 10:05:43AM -0700, Ajay Gupta wrote:
> > From: Ajay Gupta
> >
> > The change enables runtime pm support to UCSI CCG driver.
> > Added ucsi_resume() function to enable notification after
> > system reusme. Ex
time resume of i2c client
> which is UCSI Cypress CCGx driver. CCG driver will call the ISR
> for any connector change event only if NVIDIA GPU has old
> CCG firmware with the known issue.
>
> Signed-off-by: Ajay Gupta
Can't say if this is a valid workaround but here is my ack
> @@ -211,6 +212,8 @@ static int gpu_i2c_master_xfer(struct i2c_adapter *adap,
> goto exit;
> }
>
> + pm_runtime_mark_last_busy(i2cd->dev);
> + pm_runtime_put_autosuspend(i2cd->dev);
> return i;
> exit:
> if (send_stop) {
> @@ -218,6 +221,8 @@ static int
not sent before
> exiting
>
> 2) When i2c stop failed after all transfers and we do not need to
> send another stop before exiting.
>
> Signed-off-by: Ajay Gupta
Nice! It was only a brainstorming suggestion to use a dedicated flag.
But I must say I like the outcome; much better re
> > I am not nacking this, yet the use of goto here seems too much for my
> > taste. If
> > you could try refactoring the whole code (dunno, maybe using a flag when to
> > use stop?), I'd appreciate this.
> Ok, I will add a local variable to make it more readable.
I was brainstorming here, I am
> + /* runtime resume ccgx client so that it can see for any
> + * connector change event. Old ccg firmware has known
> + * issue of not triggering interrupt when a device is
> + * connected to runtime resume the controller.
> + */
Check coding style for comments.
BTW how
> diff --git a/drivers/i2c/busses/i2c-nvidia-gpu.c
> b/drivers/i2c/busses/i2c-nvidia-gpu.c
> index 1c8f708f212b..9d347583f8dc 100644
> --- a/drivers/i2c/busses/i2c-nvidia-gpu.c
> +++ b/drivers/i2c/busses/i2c-nvidia-gpu.c
> @@ -175,6 +175,7 @@ static int gpu_i2c_master_xfer(struct i2c_adapter *adap
> > I can't send an incremental patch for this, I need to ask: did you try
> > SMBUS_QUICK with it?
> Tried it today using i2cdetect and it looks good to me.
Yup, looks good. Thanks for checking!
signature.asc
Description: PGP signature
On Fri, Oct 26, 2018 at 09:36:59AM -0700, Ajay Gupta wrote:
> Latest NVIDIA GPU cards have a Cypress CCGx Type-C controller
> over I2C interface.
>
> This UCSI I2C driver uses I2C bus driver interface for communicating
> with Type-C controller.
>
> Signed-off-by: Ajay Gupta
Applied to for-curre
> It will be good if both of them can go together either through I2C or
> USB subsystem. Since USB part is reviewed and has Acked-by Heikki so please
> take that also.
Hmmm, technicallly, there is no need to let them go together. There is
no strict dependency. Yet, since Heikki gave his ack, I a
> Hmm... It would be a bit difficult (requires to download patches again as
> mbox).
> I would rather give you a hint how I resolve such problem when apply a tag to
> a
> pile of patches.
I don't mean now, for future patch series, that is... With this one, all
is fine...
> git filter-branch --
> @@ -140,5 +140,6 @@ obj-$(CONFIG_I2C_SIBYTE) += i2c-sibyte.o
> obj-$(CONFIG_I2C_XGENE_SLIMPRO) += i2c-xgene-slimpro.o
> obj-$(CONFIG_SCx200_ACB) += scx200_acb.o
> obj-$(CONFIG_I2C_FSI)+= i2c-fsi.o
> +obj-$(CONFIG_I2C_NVIDIA_GPU) += i2c-nvidia-gpu.o
Oh, I fixed this up an
On Mon, Oct 29, 2018 at 02:42:37PM +0200, Andy Shevchenko wrote:
> On Fri, Oct 26, 2018 at 09:36:57AM -0700, Ajay Gupta wrote:
> > Hi Heikki and Wolfram,
> >
> > These two changes add support for USB Type-C interface on latest NVIDIA GPU
> > card.
> > The Type-C controller used is Cypress CCGx an
On Fri, Oct 26, 2018 at 09:36:58AM -0700, Ajay Gupta wrote:
> Latest NVIDIA GPU card has USB Type-C interface. There is a
> Type-C controller which can be accessed over I2C.
>
> This driver adds I2C bus driver to communicate with Type-C controller.
> I2C client driver will be part of USB Type-C UC
On Wed, Oct 24, 2018 at 04:51:32PM +0300, Jarkko Nikula wrote:
> Platform drivers don't need dummy runtime PM callbacks that just return
> success and non-NULL pm pointer in their struct device_driver in order
> to have runtime PM happening. This has changed since following commits:
>
> 05aa55dddb
enient generic runtime pm callbacks")
> 543f2503a956 ("PM / platform_bus: Allow runtime PM by default")
> 8b313a38ecff ("PM / Platform: Use generic runtime PM callbacks directly")
>
> Signed-off-by: Jarkko Nikula
Acked-by: Wolfram Sang
>
enient generic runtime pm callbacks")
> 543f2503a956 ("PM / platform_bus: Allow runtime PM by default")
> 8b313a38ecff ("PM / Platform: Use generic runtime PM callbacks directly")
>
> Signed-off-by: Jarkko Nikula
Acked-by: Wolfram Sang
signature.asc
Description: PGP signature
> At least the I2C driver part of this was on my todo list as well (just a
> bit lower :/). I wanted to find out why they have been there in the
> first place. Do you know if such callbacks were needed "back in the
> days"?
I see now that you referenced the relevant commits in the patch
descripti
On Wed, Oct 24, 2018 at 04:51:31PM +0300, Jarkko Nikula wrote:
> I noticed these independent Renesas drivers have needless dummy runtime
> PM callbacks. I don't have the HW so only build tested.
>
> Patches can be applied independently to their own subsystems. I wanted to
> send them together if s
> I don't think SMBus is an option in this case since the intended client
> (Cypress
> something in patch 2/2) does xfers that would need 16-bit commands and I think
> they are always 8-bit in SMBus, no?
Yes. Command is 8 bit, data can be 16. Thanks for the heads up!
signature.asc
Description
Hi,
> > > We got confirmation from HW team about 4 byte read limitation. There
> > > has to be a STOP after every single read cycle. One read cycle
> > > supports maximum of
> > > 4 byte burst. I will update the patches with a comment on this.
> >
> > Could it be that this is more an SMBus contro
On Mon, Oct 01, 2018 at 07:33:02PM +, Ajay Gupta wrote:
> Hi Heikki and Peter,
>
> > > > > Can we get the working set in while the issues is being debugged?
> > > >
> > > > I'm not the one making the decision, and I don't even know if this
> > > > is going through the i2c or the usb tree? If i
On Wed, Sep 05, 2018 at 10:17:26PM +, Ajay Gupta wrote:
> Hi Greg,
>
> > > +static int ccg_read(struct ucsi_ccg *uc, u16 rab, u8 *data, u32 len)
> > > +{
> > > + struct device *dev = uc->dev;
> > > + struct i2c_client *client = uc->client;
> > > + unsigned char buf[2];
> > > + struct i2c_msg m
On Wed, Aug 29, 2018 at 05:33:17PM -0700, ajayk...@gmail.com wrote:
> From: Ajay Gupta
Can you add your full name to the gmail address, too? That would make
patch processing a tad easier.
Thanks,
Wolfram
signature.asc
Description: PGP signature
> grouped together. Wolfgang, can we have your ack for Greg merging both
> through the USB tree?
My name is "Wolfram".
signature.asc
Description: PGP signature
c17 ("typec: tcpm: Represent source supply through...")
> Cc: Adam Thomson
> Signed-off-by: Hans de Goede
Acked-by: Wolfram Sang
signature.asc
Description: PGP signature
On Tue, Apr 24, 2018 at 10:32:34PM +0200, Krzysztof Kozlowski wrote:
> The Exynos5440 is not actively developed, there are no development
> boards available and probably there are no real products with it.
> Remove wide-tree support for Exynos5440.
>
> Signed-off-by: Krzysztof Kozlowski
Applied
ptr =
- platform_get_drvdata(pdev)
+ dev_get_drvdata(d)
<... when != pdev
- &pdev->dev
+ d
...>
Kind regards,
Wolfram
Wolfram Sang (61):
ARM: plat-samsung: simplify getting .drvdata
ata: simplify getting .drvdata
auxdisplay: simplify getting .drvda
We should get drvdata from struct device directly. Going via
platform_device is an unneeded step back and forth.
Signed-off-by: Wolfram Sang
---
Build tested only. buildbot is happy. Please apply individually.
drivers/usb/phy/phy-am335x.c | 6 ++
1 file changed, 2 insertions(+), 4
We should get drvdata from struct device directly. Going via
platform_device is an unneeded step back and forth.
Signed-off-by: Wolfram Sang
---
Build tested only. buildbot is happy. Please apply individually.
drivers/usb/mtu3/mtu3_plat.c | 6 ++
1 file changed, 2 insertions(+), 4
On Tue, Feb 06, 2018 at 09:50:40AM +0100, Wolfram Sang wrote:
> Due to a typo, the mask was destroyed by a comparison instead of a bit
> shift.
>
> Reported-by: Geert Uytterhoeven
> Signed-off-by: Wolfram Sang
> ---
> Only build tested.
Add. note: according to 'git g
Due to a typo, the mask was destroyed by a comparison instead of a bit
shift.
Reported-by: Geert Uytterhoeven
Signed-off-by: Wolfram Sang
---
Only build tested.
drivers/usb/gadget/udc/goku_udc.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/usb/gadget/udc
> Acked-by: Daniel Thompson
Added, thanks!
signature.asc
Description: PGP signature
> > Even if it doesn't, it still requires Acks.
>
> As long as this has been compile tested with omap1_defconfig:
>
> Acked-by: Tony Lindgren
Hah, just in time. I was about to send out the new version, thanks! :D
buildbot did test it and I did compile tests, too.
signature.asc
Description:
> > Given the triviality of the change for non-MFD subsystems, can we apply
> > this for 4.14?
>
> I can't apply anything without Acks for *all* of the subsystems above.
Well, there are cases when you can. Those should be exceptions and well
justified, of course.
> My suggestion would be to col
On Thu, Jul 06, 2017 at 08:03:52AM +0100, Lee Jones wrote:
> On Thu, 06 Jul 2017, Thierry Reding wrote:
>
> > On Mon, May 22, 2017 at 12:02:10AM +0200, Wolfram Sang wrote:
> > > include/linux/i2c is not for client devices. Move the header file to a
> >
On Tue, May 23, 2017 at 08:18:19AM +0100, Lee Jones wrote:
> On Mon, 22 May 2017, Wolfram Sang wrote:
>
> > include/linux/i2c is not for client devices. Move the header file to a
> > more appropriate location.
> >
> > Signed-off-by: Wolfram Sang
> > ---
>
> So, if I2C is a bit different, then it might simply make sense to keep
> the function local for I2C now? This seems like a sensible start to me
> meanwhile.
Then again, the DMA API would be for drivers. If the USB core wants to
check for capable buffers, such a helper might be nice nonetheless.
> > Possibly more useful would be a variant of (IIRC) dma_map_for_device()
> > that will allocate a suitable bounce buffer for non-dma memory.
> > I think it can already do so for memory that is outside the address
> > range that the device can address (eg for a 32bit PCIe master in 64bit
> > syst
> > +/* only works in process context because of stack detection */
> > +static inline bool is_dma_capable_addr(void *addr)
> > +{
> > + return !(is_vmalloc_or_module_addr(addr) ||
> > +object_is_on_stack(addr));
>
> This does not catch kmap'ed buffers which are not directly DMA'abl
y not sure if dma-mapping.h is a good place for such
a function. I just couldn't think of a better one for now.
Second note: I am not even sure the checks are complete (kmapped mem?).
I am not an MM expert. But that just strengthens the argument of having
on centralized place IMO.
Signed-of
> For my own reference:
> Acked-for-MFD-by: Lee Jones
>
> I guess this will be going through the MFD tree?
I'd prefer that, yes. Thanks!
signature.asc
Description: PGP signature
On Mon, May 22, 2017 at 10:52:43AM +0100, Mark Brown wrote:
> On Mon, May 22, 2017 at 12:02:10AM +0200, Wolfram Sang wrote:
> > include/linux/i2c is not for client devices. Move the header file to a
> > more appropriate location.
>
> Acked-by: Mark Brown
Thanks!
> I&#
include/linux/i2c is not for client devices. Move the header file to a
more appropriate location.
Signed-off-by: Wolfram Sang
---
arch/arm/mach-omap1/board-h2-mmc.c | 2 +-
arch/arm/mach-omap1/board-h2.c | 2 +-
arch/arm/mach-omap1/board-h3-mmc.c | 2 +-
arch/arm/mach-omap1
include/linux/i2c is not for client devices. Move the header file to a
more appropriate location.
Signed-off-by: Wolfram Sang
---
arch/arm/mach-omap2/common.h| 2 +-
arch/arm/mach-omap2/omap_twl.c | 2 +-
drivers/gpio/gpio-twl4030.c | 2 +-
drivers/iio/adc
h can be found here:
git://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git i2c/platform_data
Thanks and kind regards,
Wolfram
Wolfram Sang (3):
mfd: dm355evm_msp: move header file out of I2C realm
mfd: tps65010: move header file out of I2C realm
mfd: twl: move header file out o
On Mon, Apr 03, 2017 at 09:51:32AM +0200, Zoltan Boszormenyi wrote:
> Use the common mutex from driver/usb/host/pci-quirks.c in the
> i2c-piix4 driver to synchronize access to the I/O port pair
> 0xcd6 / 0xcd7.
>
> At the same time, remove the request_region() call to reserve
> these I/O ports, so
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/core/hub.c | 9 +++--
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c
index 1d5fc32d06d007..b48dc76385b652 100644
--- a
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/core/hcd.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/usb/core/hcd.c b/drivers/usb/core/hcd.c
index d2e3f655c26f6c..a0c87b617eddea 100644
--- a
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/atm/speedtch.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/usb/atm/speedtch.c b/drivers/usb/atm/speedtch.c
index 0270d1312f834f..5083eb5b0d5ed9 100644
--- a/drivers/usb
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/host/max3421-hcd.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/usb/host/max3421-hcd.c b/drivers/usb/host/max3421-hcd.c
index 2f7690092a7ffb
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/misc/usbsevseg.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/usb/misc/usbsevseg.c b/drivers/usb/misc/usbsevseg.c
index 1fe6b73c22f300
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/atm/ueagle-atm.c | 9 ++---
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/drivers/usb/atm/ueagle-atm.c b/drivers/usb/atm/ueagle-atm.c
index f198291630d7dd
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/misc/legousbtower.c | 16
1 file changed, 4 insertions(+), 12 deletions(-)
diff --git a/drivers/usb/misc/legousbtower.c b/drivers/usb/misc/legousbtower.c
index
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/misc/ftdi-elan.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/usb/misc/ftdi-elan.c b/drivers/usb/misc/ftdi-elan.c
index 45f303c1cc49ef..9beba9748d0f24 100644
--- a/drivers
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/storage/alauda.c | 11 +++
1 file changed, 3 insertions(+), 8 deletions(-)
diff --git a/drivers/usb/storage/alauda.c b/drivers/usb/storage/alauda.c
index 1d8b03c8103053
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/core/message.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/usb/core/message.c b/drivers/usb/core/message.c
index 0406a59f05510c..5ab5c1a81462b6
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/misc/adutux.c | 13 +++--
1 file changed, 3 insertions(+), 10 deletions(-)
diff --git a/drivers/usb/misc/adutux.c b/drivers/usb/misc/adutux.c
index c34a0b6980cdcd
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/misc/usblcd.c | 9 ++---
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/drivers/usb/misc/usblcd.c b/drivers/usb/misc/usblcd.c
index 1184390508e983..9f48419abc46b9
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/musb/da8xx.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/usb/musb/da8xx.c b/drivers/usb/musb/da8xx.c
index b03d3b867fca2f..3c4dd1658f28db 100644
--- a
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/gadget/udc/udc-xilinx.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/usb/gadget/udc/udc-xilinx.c
b/drivers/usb/gadget/udc/udc-xilinx.c
index
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/renesas_usbhs/pipe.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/usb/renesas_usbhs/pipe.c b/drivers/usb/renesas_usbhs/pipe.c
index c238772b9e9e1a
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/gadget/udc/fsl_qe_udc.c | 16
1 file changed, 4 insertions(+), 12 deletions(-)
diff --git a/drivers/usb/gadget/udc/fsl_qe_udc.c
b/drivers/usb/gadget/udc/fsl_qe_udc.c
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/wusbcore/security.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/usb/wusbcore/security.c b/drivers/usb/wusbcore/security.c
index b66faaf3e842e8
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/dwc2/gadget.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/usb/dwc2/gadget.c b/drivers/usb/dwc2/gadget.c
index af46adfae41ca5..2ff03ae08e144d
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/misc/yurex.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/usb/misc/yurex.c b/drivers/usb/misc/yurex.c
index bb606bdc25e558..54e53ac4c08fd7 100644
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/misc/cypress_cy7c63.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/usb/misc/cypress_cy7c63.c
b/drivers/usb/misc/cypress_cy7c63.c
index 402b94dd253159
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/class/usbtmc.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/usb/class/usbtmc.c b/drivers/usb/class/usbtmc.c
index 22c235adacb3ad..da4f2509f56775 100644
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/misc/iowarrior.c | 20 ++--
1 file changed, 6 insertions(+), 14 deletions(-)
diff --git a/drivers/usb/misc/iowarrior.c b/drivers/usb/misc/iowarrior.c
index
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/misc/appledisplay.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/usb/misc/appledisplay.c b/drivers/usb/misc/appledisplay.c
index f7c48703347fb7..0b2617fe1c9b6f 100644
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/misc/ldusb.c | 20 +---
1 file changed, 5 insertions(+), 15 deletions(-)
diff --git a/drivers/usb/misc/ldusb.c b/drivers/usb/misc/ldusb.c
index 84890791c2f830
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/misc/trancevibrator.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/usb/misc/trancevibrator.c
b/drivers/usb/misc/trancevibrator.c
index 4145314a515b95
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/renesas_usbhs/mod_gadget.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/drivers/usb/renesas_usbhs/mod_gadget.c
b/drivers/usb/renesas_usbhs/mod_gadget.c
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/usb-skeleton.c | 9 ++---
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/drivers/usb/usb-skeleton.c b/drivers/usb/usb-skeleton.c
index 89e02a7529b785..5133a0792eb041
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/host/xhci.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/usb/host/xhci.c b/drivers/usb/host/xhci.c
index 01d96c9b3a75b9..1a4ca02729c274 100644
--- a
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/wusbcore/crypto.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/usb/wusbcore/crypto.c b/drivers/usb/wusbcore/crypto.c
index 33acd1599e99e9
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/misc/uss720.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/usb/misc/uss720.c b/drivers/usb/misc/uss720.c
index 256d02da444d0f..356d312add5779 100644
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/core/urb.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/usb/core/urb.c b/drivers/usb/core/urb.c
index c601e25b609fb5..a9039696476e68 100644
--- a
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/host/fhci-hcd.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/usb/host/fhci-hcd.c b/drivers/usb/host/fhci-hcd.c
index 0960f41f945aba..55a0ae6f2d7430
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/renesas_usbhs/mod_host.c | 10 ++
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/drivers/usb/renesas_usbhs/mod_host.c
b/drivers/usb/renesas_usbhs/mod_host.c
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/misc/idmouse.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/usb/misc/idmouse.c b/drivers/usb/misc/idmouse.c
index 5105397e62fc76..2975e80b7a56f9 100644
--- a/drivers/usb
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/misc/cytherm.c | 32
1 file changed, 8 insertions(+), 24 deletions(-)
diff --git a/drivers/usb/misc/cytherm.c b/drivers/usb/misc/cytherm.c
index
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/host/xhci-tegra.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/usb/host/xhci-tegra.c b/drivers/usb/host/xhci-tegra.c
index 0f53ae0f464e6d..d39b37be71f01f 100644
--- a
x27;t save CLEAR_TT_BUFFER state"), I
left it. This series saves ~4.5KB of "out of memory" permutations in .text and
.rodata. For modified lines, (x == NULL) was replaced with (!NULL) as well.
This seems to be the dominant style in this subsystem and checkpatch recommends
it as well (and I prefe
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/host/uhci-hcd.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/usb/host/uhci-hcd.c b/drivers/usb/host/uhci-hcd.c
index a7de8e8bb4582b..5d3d914ab4fb44
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/storage/sddr09.c | 14 --
1 file changed, 4 insertions(+), 10 deletions(-)
diff --git a/drivers/usb/storage/sddr09.c b/drivers/usb/storage/sddr09.c
index c5797fa2125e94
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/misc/lvstest.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/usb/misc/lvstest.c b/drivers/usb/misc/lvstest.c
index 961faae62ea087..af2952b918b843 100644
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/musb/am35x.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/usb/musb/am35x.c b/drivers/usb/musb/am35x.c
index c41fe588d14d97..c14577dbedf71a 100644
--- a
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/wusbcore/wa-nep.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/usb/wusbcore/wa-nep.c b/drivers/usb/wusbcore/wa-nep.c
index 6140100ad50e41
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/gadget/udc/goku_udc.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/usb/gadget/udc/goku_udc.c
b/drivers/usb/gadget/udc/goku_udc.c
index d2205d9e0c8b43
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/atm/cxacru.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/usb/atm/cxacru.c b/drivers/usb/atm/cxacru.c
index 18b281d73a39ee..f9fe86b6f7b5b6 100644
--- a
All kmalloc-based functions print enough information on failures.
Signed-off-by: Wolfram Sang
---
drivers/usb/atm/usbatm.c | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/drivers/usb/atm/usbatm.c b/drivers/usb/atm/usbatm.c
index 5e4f46c5a30046..4dec9df8764b9a 100644
kmalloc will print enough information in case of failure.
Signed-off-by: Wolfram Sang
---
drivers/staging/media/lirc/lirc_imon.c | 9 ++---
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/media/lirc/lirc_imon.c
b/drivers/staging/media/lirc/lirc_imon.c
index
when we talked about it at LCJ in
Tokyo a few weeks ago.
Wolfram Sang (6):
staging: comedi: drivers: usbduxfast: don't print error when
allocating urb fails
staging: media: lirc: lirc_imon: don't print error when allocating urb
fails
staging: media: lirc: lirc_sasem: don'
kmalloc will print enough information in case of failure.
Signed-off-by: Wolfram Sang
---
drivers/staging/comedi/drivers/usbduxfast.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/staging/comedi/drivers/usbduxfast.c
b/drivers/staging/comedi/drivers/usbduxfast.c
kmalloc will print enough information in case of failure.
Signed-off-by: Wolfram Sang
---
drivers/staging/rtl8192u/r8192U_core.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U_core.c
b/drivers/staging/rtl8192u/r8192U_core.c
index
kmalloc will print enough information in case of failure.
Signed-off-by: Wolfram Sang
---
drivers/staging/most/hdm-usb/hdm_usb.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/staging/most/hdm-usb/hdm_usb.c
b/drivers/staging/most/hdm-usb/hdm_usb.c
index
1 - 100 of 245 matches
Mail list logo
