of those cases where I would suggest collect ACKs
from affected subsystem maintainers and send a pull request
to the SoC tree for this hairy bundle.
Yours,
Linus Walleij
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverpr
.
>
> Signed-off-by: Sergio Paracuellos
Acked-by: Linus Walleij
Yours,
Linus Walleij
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
merge window.
Applied patches 1-7 to the pinctrl tree, patch 8 needs to be sent
to Greg.
Yours,
Linus Walleij
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
merge window.
Reviewed-by: Linus Walleij
If Greg wants he can queue them last minute. Else I'll apply these
after the merge window, no big deal.
Yours,
Linus Walleij
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject
I will eventually apply directly anyway).
Yours,
Linus Walleij
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Hi Serigio,
I dug around some to try to understand the patch I think I get
it now :)
Squash this with the third patch so it becomes a
"move" of this file, preserving history. With that:
Acked-by: Linus Walleij
I have ideas, but it is better to move the driver out
of staging a
ion...
enum: [gpio, i2c, spi, uart1, uart2, uart3, rgmii1, rgmii2,
mdio, nand1, nand2, sdhci]
Note: the function names are fine but the group names are a bit
confusion since often a group can be used for more than one
function, and e.g.
function = "i2c";
group = "uart1
o my subsystem before the merge window
if there is time. I'll provide ACK.
Yours,
Linus Walleij
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
On Mon, Dec 7, 2020 at 2:57 PM Sergio Paracuellos
wrote:
> On Mon, Dec 7, 2020 at 2:05 PM Linus Walleij wrote:
> > After this I think the driver looks good and can graduate from staging.
> > Can you send a patch to move this to drivers/pinctrl next
> >
> > I think dri
een
> gpio node and pinctrl node and we can stop using the deprecated
> pinctrl_add_gpio_range() function.
>
> Signed-off-by: Sergio Paracuellos
Reviewed-by: Linus Walleij
After this I think the driver looks good and can graduate from staging.
Can you send a patch to move this to dri
-by: Linus Walleij
---
ChangeLog v1->v2:
- Rebased on v5.9-rc1
---
.../media/atomisp/i2c/atomisp-lm3554.c| 68 ---
.../media/atomisp/include/media/lm3554.h | 7 +-
2 files changed, 32 insertions(+), 43 deletions(-)
diff --git a/drivers/staging/media/atomisp/i2c/atom
This makes the driver use the irqchip template to assign
properties to the gpio_irq_chip instead of using the
explicit calls to gpiochip_irqchip_add(). The irqchip is
instead added while adding the gpiochip.
Cc: Nishad Kamdar
Cc: Johan Hovold
Signed-off-by: Linus Walleij
---
drivers/staging
e Pouiller
Signed-off-by: Linus Walleij
---
ChangeLog v4->v5:
- Make the wakeup GPIO optional.
ChangeLog v3->v4:
- Finally figured out how to compile this by selecting SPI
host and deselecting MMC host.
- Initialize the reset GPIO as OUT_LOW
- Initialize the wakeup GPIO as OUT_LOW
- Drop one
e Pouiller
Signed-off-by: Linus Walleij
---
ChangeLog v3->v4:
- Finally figured out how to compile this by selecting SPI
host and deselecting MMC host.
- Initialize the reset GPIO as OUT_LOW
- Initialize the wakeup GPIO as OUT_LOW
- Drop one more desc_to_gpio()
- Update the warning if GPIO is no
On Sun, Jun 28, 2020 at 12:43 PM Greg KH wrote:
> On Sun, Jun 28, 2020 at 10:52:36AM +0200, Linus Walleij wrote:
> > ChangeLog v2->v3:
> > - ERR_CAST not PTR_CAST
> > ChangeLog v1->v2:
> > - Fixed a cast and a variable name.
> > - I still don't know
e Pouiller
Signed-off-by: Linus Walleij
---
ChangeLog v2->v3:
- ERR_CAST not PTR_CAST
ChangeLog v1->v2:
- Fixed a cast and a variable name.
- I still don't know how to compile this but hey the zeroday
robot does.
---
drivers/staging/wfx/bus_spi.c | 11 +
driver
e Pouiller
Signed-off-by: Linus Walleij
---
ChangeLog v1->v2:
- Fixed a cast and a variable name.
- I still don't know how to compile this but hey the zeroday
robot does.
---
drivers/staging/wfx/bus_spi.c | 11 +
drivers/staging/wfx/main.c| 42 ---
d
-by: Linus Walleij
---
.../media/atomisp/i2c/atomisp-lm3554.c| 68 ---
.../media/atomisp/include/media/lm3554.h | 7 +-
2 files changed, 32 insertions(+), 43 deletions(-)
diff --git a/drivers/staging/media/atomisp/i2c/atomisp-lm3554.c
b/drivers/staging/media/atomisp
e Pouiller
Signed-off-by: Linus Walleij
---
drivers/staging/wfx/bus_spi.c | 11 +
drivers/staging/wfx/main.c| 42 ---
drivers/staging/wfx/main.h| 2 --
3 files changed, 9 insertions(+), 46 deletions(-)
diff --git a/drivers/staging/wfx/bus_spi.c
On Tue, Jan 7, 2020 at 10:31 AM Bartosz Golaszewski
wrote:
> wt., 7 sty 2020 o 10:29 Linus Walleij napisał(a):
> > > Ugh, I now noticed I responded to Thierry only after applying this to my
> > > tree.
> > >
> > > Anyway, it shouldn't be a problem. I'
On Tue, Jan 7, 2020 at 9:25 AM Bartosz Golaszewski
wrote:
> pon., 6 sty 2020 o 23:59 Linus Walleij napisał(a):
> > On Sun, Dec 15, 2019 at 7:31 PM Dmitry Osipenko wrote:
> >
> > > I was investigating why CPU hangs during of GPIO driver suspend and in
>
_relaxed accessors
> gpio: tegra: Properly handle irq_set_irq_wake() error
> gpio: tegra: Use NOIRQ phase for suspend/resume
All three patches applied with Thierry's review/test tag.
Yours,
Linus Walleij
___
devel mailing list
clarifying why ^= SPI_CS_HIGH is the right
choice here.
Reported-by: Mark Brown
Signed-off-by: Linus Walleij
---
drivers/staging/fbtft/fb_uc1611.c| 12 +---
drivers/staging/fbtft/fb_watterott.c | 13 ++---
2 files changed, 19 insertions(+), 6 deletions(-)
diff --git a/drivers
zysztofik
Patch applied with Marek's Tested-by.
Thanks to both of you for digging in and fixing this up!
Now we are in good shape for the v4.20 cycle :)
Yours,
Linus Walleij
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
ed up processing of applications
> using whole GPIO banks as I/O ports, while not breaking simultaneous
> manipulation of consecutive pins of the same chip which don't follow
> the equal numbering rule.
>
> Cc: Jonathan Corbet
> Signed-off-by: Janusz Krzysztofik
Patch applied!
already processed via fast bitmap path, possibly resulting in an
> infinite loop. Fix it.
>
> Signed-off-by: Janusz Krzysztofik
Patch applied!
Thanks for working on getting this into shape!
Yours,
Linus Walleij
___
devel mailing list
de
r testing on this platform!
> Booting hangs after detecting MMC cards. Reverting this patch fixes the
> boot. I will try later to add some debugs and investigate it further what
> really happens when booting hangs.
How typical. I hope we can fix it, because this should mean speedups
for your platf
On Thu, Sep 13, 2018 at 2:22 AM Linus Walleij wrote:
> On Wed, Sep 5, 2018 at 11:49 PM Janusz Krzysztofik
> wrote:
>
> > The goal is to boost performance of get/set array functions while
> > processing GPIO arrays which represent pins of a signle chip in
> > ha
ally excited to merge this!
Yours,
Linus Walleij
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
should move forward
with my plan to send a sed script to Torvalds just renaming all
of those to something sane in the next merge window.
Like __assign_bit() -> assign_bit_nonatomic()
Yours,
Linus Walleij
___
devel mailing list
de...@linuxdriverproject
lon
> who suggested using GPIO API modified to accept bitmaps, and by Linus
> Walleij who suggested still more great ideas for further immprovement
> of the proposed API changes - thanks!
>
> The goal is to boost performans of get/set array functions while
> processing GPIO arrays
odule remove
> as the reference was fetched in probe function.
> Updated the TODO file
>
> Signed-off-by: Ajay Singh
Reviewed-by: Linus Walleij
Thanks Ajay!
Yours,
Linus Walleij
___
devel mailing list
de...@linuxdriverproject.org
http://driverde
n "wide".
I changed it while applying the patch.
Yours,
Linus Walleij
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
On Thu, Jul 5, 2018 at 3:43 PM Sergio Paracuellos
wrote:
> Add a devicetree binding documentation for the mt7621 gpio.
>
> Signed-off-by: Sergio Paracuellos
> Reviewed-by: NeilBrown
Patch applied with Rob's review tag.
Yours,
L
On Thu, Jun 14, 2018 at 4:33 PM, Rob Herring wrote:
> On Thu, Jun 14, 2018 at 8:14 AM, Linus Walleij
> wrote:
>> On Wed, Jun 13, 2018 at 9:28 PM, Rob Herring wrote:
>>
>>>> "Some system-on-chips (SoCs) use the concept of GPIO banks. ...
>>>> Usual
a, it is mediatek,mt7321-gpio, so it has 3x32 GPIOs
indexed from 0..95".
No need of overspecifying stuff.
Yours,
Linus Walleij
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
that case to concatenate the offsets and instead of
having an extra cell 0, 1 and offsets 0-31, 0-31
have two cells and offsets 0-63.
My reasoning is that since it is represented by a single device
we are indexing into that one device from 0-n.
Yours,
Linus Walleij
___
A second thought:
On Fri, Jun 8, 2018 at 1:59 PM, Linus Walleij wrote:
>> + select GPIOLIB_IRQCHIP
>
> You are not using this so I guess remove that line.
(...)
>> +static int
>> +mediatek_gpio_to_irq(struct gpio_chip *chip, unsigned int pin)
>> +{
>>
= mediatek_gpio_irq_unmask,
> + .irq_mask = mediatek_gpio_irq_mask,
> + .irq_mask_ack = mediatek_gpio_irq_mask,
> + .irq_set_type = mediatek_gpio_irq_type,
> +};
When implementing custom irqchips it is important to also
implement .irq_request_resources() and
.irq_release_resources() and make sure these call
gpiochip_[un]lock_as_irq().
See for example
drivers/gpio/gpio-dwapb.c
for an example.
> +static const struct of_device_id mediatek_gpio_match[] = {
> + { .compatible = "mediatek,mt7621-gpio" },
> + {},
> +};
> +MODULE_DEVICE_TABLE(of, mediatek_gpio_match);
> +
> +static struct platform_driver mediatek_gpio_driver = {
> + .probe = mediatek_gpio_probe,
> + .driver = {
> + .name = "mt7621_gpio",
> + .of_match_table = mediatek_gpio_match,
> + },
> +};
> +
> +module_platform_driver(mediatek_gpio_driver);
If you're not implementing .remove() I don't think this will
really work fine as a module. Also the Kconfig is a bool.
I guess you want to use
builtin_platform_driver()?
Yours,
Linus Walleij
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
On Thu, Apr 19, 2018 at 1:01 PM, Marc Dietrich <marvi...@gmx.de> wrote:
> Use GPIO descriptors instead of relying on the old method.
>
> Cc: Linus Walleij <linus.wall...@linaro.org>
> Signed-off-by: Marc Dietrich <marvi...@gmx.de>
> ---
> This obsolets the ToD
To make sure that these drivers do not leave staging before they
are properly converted to use the new GPIO descriptor API, and the
GPIOLIB_IRQCHIP helper library, create the TODO file with these work
items.
Cc: Johan Hovold <jo...@kernel.org>
Signed-off-by: Linus Walleij <l
GPIO drivers should include only, the
header is deprecated.
Cc: John Crispin <j...@phrozen.org>
Cc: Zhiyong Tao <zhiyong@mediatek.com>
Cc: Sean Wang <sean.w...@mediatek.com>
Signed-off-by: Linus Walleij <linus.wall...@linaro.org>
---
I was philosophizing whether
To make sure that this driver does not leave staging before it
is properly converted to use the new GPIO descriptor API,
augment the TODO file with this work item.
Cc: Andres Salomon <dilin...@queued.net>
Cc: Jens Frederich <jfreder...@gmail.com>
Signed-off-by: Linus Walleij
To make sure that this driver does not leave staging before it
is properly converted to use the new GPIO descriptor API,
augment the TODO file with this work item.
Cc: Johnny Kim <johnny@microchip.com>
Cc: Nicolas Ferre <nicolas.fe...@microchip.com>
Signed-off-by: Linus Walleij
To make sure that this driver does not leave staging before it
is properly converted to use the new GPIO descriptor API,
augment the TODO file with this work item.
Cc: Marc Dietrich <marvi...@gmx.de>
Signed-off-by: Linus Walleij <linus.wall...@linaro.org>
---
drivers/staging/n
To make sure that these drivers do not leave staging before they
are properly converted to use the new GPIO descriptor API, create
the TODO file with this work item.
Cc: Thomas Petazzoni <thomas.petazz...@free-electrons.com>
Cc: Noralf Tronnes <no...@tronnes.org>
Signed-off-by: L
To make sure this driver does not leave staging without a proper
conversion to the GPIO descriptor API, leave a note in the TODO.
Cc: Magnus Damm <damm+rene...@opensource.se>
Cc: Simon Horman <horms+rene...@verge.net.au>
Cc: Geert Uytterhoeven <geert+rene...@glider.be>
Si
Trying to be helpful for people who want to clean up the
staging drivers: point out the drivers that need to be
converted to use GPIO descriptors and a little bit on
how to do it.
Linus Walleij (9):
staging: emxx_udc: Add GPIO descriptor work to TODO
staging: fbtft: Add TODO file with GPIO
To make sure that these drivers do not leave staging before they
are properly converted to use the new GPIO descriptor API,
augment the TODO file with this work item.
Cc: Alan Cox <a...@linux.intel.com>
Cc: Andy Shevchenko <andriy.shevche...@linux.intel.com>
Signed-off-by: L
To make sure that these drivers do not leave staging before they
are properly converted to use the new GPIO descriptor API,
augment the TODO file with this work item.
Cc: Jonathan Cameron <jonathan.came...@huawei.com>
Signed-off-by: Linus Walleij <linus.wall...@linaro.org>
---
dri
t take the old switchdev-based
SMI-driver and convert it to DSA.
I bet I can do that :D
Well, I will try. Because it's blocking me to work on the Gemini
ethernet driver.
Yours,
Linus Walleij
___
devel mailing list
de...@lin
Top posting and resending since net...@vger.kernel.org
is the right mail address for this. Mea culpa.
Linus Walleij
On Sat, Oct 14, 2017 at 11:35 AM, Linus Walleij
<linus.wall...@linaro.org> wrote:
> On Thu, Oct 5, 2017 at 11:16 AM, Razvan Stefanescu
> <razvan.stefane...
ethernet switch subsystem so we can house these
drivers in a proper way?
What we need AFAICT:
- Consensus on userspace ABI
- Consensus on ethtool extenstions
- Consensus on where in drivers/net this goes
You can kick me for not knowing what I'm talking about and how complex the
p
On Thu, Dec 29, 2016 at 11:27 PM, Steve Longerbeam
<slongerb...@gmail.com> wrote:
> Add optional reset-gpios pin control. If present, de-assert the
> specified reset gpio pin to bring the chip out of reset.
>
> Signed-off-by: Steve Longerbeam <steve_longerb...@mentor.com
ONNECTOR is selected.
> +
> + This driver can also be built as a module. If so, the module
> + will be called gpio-cbc-presence.
At least tell us *what* it is detecting the presence of.
Apart from this it has some of the same issues pointed out in the
other drivers, correc
tion (0 = out, 1 = in)
> + bit[1]: init high
> + bit[2]: active low
Can't you just refer to the generic bindings?
Apart from that it looks fine.
Yours,
Linus Walleij
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
ng IRQ will be pretty quick and
easy to fix up I guess, the IRQ drivers will require some
effort. An approach is to split these in two: one for the basic
GPIO and a separate add-on patch for the IRQ functionality.
Yours,
Linus Walleij
_
Cc: Sandeep Patil <patil_sand...@projectara.com>
Cc: Rui Miguel Silva <rui.si...@linaro.org>
Cc: Greg Kroah-Hartman <gre...@linuxfoundation.org>
Signed-off-by: Linus Walleij <linus.wall...@linaro.org>
---
ChangeLog v1->v2:
- Add the hunk adding select GPIOLIB_IRQCHIP to
Cc: Sandeep Patil <patil_sand...@projectara.com>
Cc: Rui Miguel Silva <rui.si...@linaro.org>
Cc: Greg Kroah-Hartman <gre...@linuxfoundation.org>
Signed-off-by: Linus Walleij <linus.wall...@linaro.org>
---
Greybus folks: please look at this. I expect something like this
to b
On Fri, Dec 11, 2015 at 7:58 AM, Martyn Welch <mar...@welchs.me.uk> wrote:
> On 09/12/15 13:50, Linus Walleij wrote:
>>
>> This makes the driver use the data pointer added to the gpio_chip
>> to store a pointer to the state container instead of relying on
>> con
ail.com>
Cc: de...@driverdev.osuosl.org
Signed-off-by: Linus Walleij <linus.wall...@linaro.org>
---
Greg, please ACK this so I can take this through the GPIO tree.
---
drivers/staging/vme/devices/vme_pio2_gpio.c | 17 ++---
1 file changed, 6 insertions(+), 11 deletions(-)
diff --g
#ifdef and instead mark
> the functions as __maybe_unused, which is a nicer anyway, as it
> provides build testing for all the code in all configurations
> and is harder to get wrong.
>
> Signed-off-by: Arnd Bergmann <a...@arndb.de>
Acked-by: Li
that once the touchscreen portions are in place.
I'm just not quite familiar with the structure of the new RMI4
framework so I will need some time on it.
Yours,
Linus Walleij
___
devel mailing list
de...@linuxdriverproject.org
http
63 matches
Mail list logo