Hi Antoine,
On 2020-06-19 14:22, Antoine Tenart wrote:
[...]
@@ -999,9 +1553,35 @@ int vsc8584_ptp_probe(struct phy_device *phydev)
if (!vsc8531->ptp)
return -ENOMEM;
+ mutex_init(>phc_lock);
mutex_init(>ts_lock);
+ /* Retrieve the shared load/save
Hi Antoine,
Feels weird to review my own patches a year later having written them,
almost nostalgic :)
The review is mostly nitpicks.
On 2020-06-19 14:22, Antoine Tenart wrote:
[...]
@@ -373,6 +374,21 @@ struct vsc8531_private {
unsigned long ingr_flows;
unsigned long
Hi Antoine,
On 2020-06-19 14:22, Antoine Tenart wrote:
This patch takes in account the use of the 1588 block in the MACsec
initialization, as a conditional configuration has to be done (when the
1588 block is used).
Signed-off-by: Antoine Tenart
---
drivers/net/phy/mscc/mscc_macsec.c | 4
Hi Josh,
On Fri, Dec 14, 2018 at 07:57:37AM -0500, Josh Boyer wrote:
> On Thu, Dec 6, 2018 at 5:34 AM Quentin Schulz
> wrote:
> >
> > Hi all,
> >
> > Both VSC8574 and VSC8584 Ethernet PHYs require a firmware patch to be
> > applied before making use of
Hi Lee,
On Mon, Dec 10, 2018 at 06:27:18AM +, Lee Jones wrote:
> On Sat, 08 Dec 2018, Quentin Schulz wrote:
>
> > Hi Lee,
> >
> > On Fri, Dec 07, 2018 at 07:22:37PM +, Lee Jones wrote:
> > > On Fri, 07 Dec 2018, Vasily Khoruzhick wrote:
> > >
Hi Lee,
On Fri, Dec 07, 2018 at 07:22:37PM +, Lee Jones wrote:
> On Fri, 07 Dec 2018, Vasily Khoruzhick wrote:
>
> > On Fri, Dec 7, 2018 at 8:40 AM Lee Jones wrote:
> >
> > > My OCD-dar is going crazy.
> > >
> > > Why haven't you used the same alignment as is already there?
> > >
> > > If
Hi Lee,
On Fri, Dec 07, 2018 at 07:22:37PM +, Lee Jones wrote:
> On Fri, 07 Dec 2018, Vasily Khoruzhick wrote:
>
> > On Fri, Dec 7, 2018 at 8:40 AM Lee Jones wrote:
> >
> > > My OCD-dar is going crazy.
> > >
> > > Why haven't you used the same alignment as is already there?
> > >
> > > If
(2018-12-06
11:02:51 +0100)
Quentin Schulz (1):
microchip: add firmware for VSC8574 and VSC8584 Ethernet PHYs
LICENCE.microchip| 40
WHENCE
(2018-12-06
11:02:51 +0100)
Quentin Schulz (1):
microchip: add firmware for VSC8574 and VSC8584 Ethernet PHYs
LICENCE.microchip| 40
WHENCE
Hi Baolin,
On Thu, Nov 01, 2018 at 03:22:18PM +0800, Baolin Wang wrote:
> Hi Quentin,
>
> On 29 October 2018 at 22:48, Quentin Schulz
> wrote:
[...]
> >
> >> + return len;
> >> + } else if (len > POWER_SUPPLY_OCV_TEMP_MAX) {
>
Hi Baolin,
On Thu, Nov 01, 2018 at 03:22:18PM +0800, Baolin Wang wrote:
> Hi Quentin,
>
> On 29 October 2018 at 22:48, Quentin Schulz
> wrote:
[...]
> >
> >> + return len;
> >> + } else if (len > POWER_SUPPLY_OCV_TEMP_MAX) {
>
Hi Jonathan,
On Sun, Oct 28, 2018 at 03:40:11PM +, Jonathan Cameron wrote:
> On Wed, 24 Oct 2018 08:56:33 -0500
> Rob Herring wrote:
>
> > On Tue, 23 Oct 2018 21:53:23 +0300, Oskari Lemmela wrote:
> > > The AXP803 ADC is compatible with AXP813 ADC, but add
> > > specific compatible for it.
Hi Jonathan,
On Sun, Oct 28, 2018 at 03:40:11PM +, Jonathan Cameron wrote:
> On Wed, 24 Oct 2018 08:56:33 -0500
> Rob Herring wrote:
>
> > On Tue, 23 Oct 2018 21:53:23 +0300, Oskari Lemmela wrote:
> > > The AXP803 ADC is compatible with AXP813 ADC, but add
> > > specific compatible for it.
Hi Oskari,
On Mon, Oct 08, 2018 at 09:19:07PM +0300, Oskari Lemmela wrote:
> As axp20x-ac-power-supply now supports AXP813, add a cell for it.
>
Reviewed-by: Quentin Schulz
Thanks,
Quentin
signature.asc
Description: PGP signature
Hi Oskari,
On Mon, Oct 08, 2018 at 09:19:07PM +0300, Oskari Lemmela wrote:
> As axp20x-ac-power-supply now supports AXP813, add a cell for it.
>
Reviewed-by: Quentin Schulz
Thanks,
Quentin
signature.asc
Description: PGP signature
Hi Oskari,
On Mon, Oct 08, 2018 at 09:19:05PM +0300, Oskari Lemmela wrote:
> Sopine baseboard have ACIN and battery connectors.
>
Reviewed-by: Quentin Schulz
Thanks,
Quentin
signature.asc
Description: PGP signature
Hi Oskari,
On Mon, Oct 08, 2018 at 09:19:05PM +0300, Oskari Lemmela wrote:
> Sopine baseboard have ACIN and battery connectors.
>
Reviewed-by: Quentin Schulz
Thanks,
Quentin
signature.asc
Description: PGP signature
Hi Oskari,
On Mon, Oct 08, 2018 at 09:19:04PM +0300, Oskari Lemmela wrote:
> AXP803 is compatible with AXP813. Add DT nodes ADC, GPIO,
> AC and battery power supplies.
>
Reviewed-by: Quentin Schulz
Thanks,
Quentin
signature.asc
Description: PGP signature
Hi Oskari,
On Mon, Oct 08, 2018 at 09:19:04PM +0300, Oskari Lemmela wrote:
> AXP803 is compatible with AXP813. Add DT nodes ADC, GPIO,
> AC and battery power supplies.
>
Reviewed-by: Quentin Schulz
Thanks,
Quentin
signature.asc
Description: PGP signature
Hi Oskari,
On Mon, Oct 08, 2018 at 09:19:02PM +0300, Oskari Lemmela wrote:
> The AXP803/AXP813 AC power supply can limit input current and voltage.
>
Reviewed-by: Quentin Schulz
Thanks,
Quentin
signature.asc
Description: PGP signature
Hi Oskari,
On Mon, Oct 08, 2018 at 09:19:02PM +0300, Oskari Lemmela wrote:
> The AXP803/AXP813 AC power supply can limit input current and voltage.
>
Reviewed-by: Quentin Schulz
Thanks,
Quentin
signature.asc
Description: PGP signature
Hi Oskari,
On Mon, Oct 08, 2018 at 09:19:03PM +0300, Oskari Lemmela wrote:
> Add AC power supply subnode for AXP81X PMIC.
>
Reviewed-by: Quentin Schulz
Thanks,
Quentin
signature.asc
Description: PGP signature
Hi Oskari,
On Mon, Oct 08, 2018 at 09:19:03PM +0300, Oskari Lemmela wrote:
> Add AC power supply subnode for AXP81X PMIC.
>
Reviewed-by: Quentin Schulz
Thanks,
Quentin
signature.asc
Description: PGP signature
c: add support for VSC8574 PHY")
Reviewed-by: Quentin Schulz
Thanks!
Quentin
signature.asc
Description: PGP signature
c: add support for VSC8574 PHY")
Reviewed-by: Quentin Schulz
Thanks!
Quentin
signature.asc
Description: PGP signature
lect/restore_page
> functions")
Reviewed-by: Quentin Schulz
Thanks!
Quentin
signature.asc
Description: PGP signature
lect/restore_page
> functions")
Reviewed-by: Quentin Schulz
Thanks!
Quentin
signature.asc
Description: PGP signature
Hi Gustavo,
On Tue, Oct 09, 2018 at 09:39:53PM +0200, Gustavo A. R. Silva wrote:
> In case memory resources for *fw* were successfully allocated,
> release them before return.
>
> Addresses-Coverity-ID: 1473968 ("Resource leak")
> Fixes: 00d70d8e0e78 ("net: phy: mscc: add support for VSC8574
Hi Gustavo,
On Tue, Oct 09, 2018 at 09:39:53PM +0200, Gustavo A. R. Silva wrote:
> In case memory resources for *fw* were successfully allocated,
> release them before return.
>
> Addresses-Coverity-ID: 1473968 ("Resource leak")
> Fixes: 00d70d8e0e78 ("net: phy: mscc: add support for VSC8574
Hi Frieder,
On Mon, Oct 08, 2018 at 11:53:21AM +0200, Frieder Schrempf wrote:
> Hi,
>
> On 27.09.2018 10:14, Maxime Ripard wrote:
> > On Wed, Sep 26, 2018 at 10:19:22PM +0200, Hans de Goede wrote:
> > > On 26-09-18 16:44, Frieder Schrempf wrote:
> > > > Hi,
> > > >
> > > > On Fri, Feb 09, 2018
Hi Frieder,
On Mon, Oct 08, 2018 at 11:53:21AM +0200, Frieder Schrempf wrote:
> Hi,
>
> On 27.09.2018 10:14, Maxime Ripard wrote:
> > On Wed, Sep 26, 2018 at 10:19:22PM +0200, Hans de Goede wrote:
> > > On 26-09-18 16:44, Frieder Schrempf wrote:
> > > > Hi,
> > > >
> > > > On Fri, Feb 09, 2018
size of SERDES_MAX
> in the for loop.
>
> Quentin Schulz pointed out that SERDES_MAX is a valid value to
> index ctrl->phys. So, I updated SERDES_MAX to be SERDES6G_MAX + 1
> in include/dt-bindings/phy/phy-ocelot-serdes.h.
>
> Then I changed the condition in the for loo
size of SERDES_MAX
> in the for loop.
>
> Quentin Schulz pointed out that SERDES_MAX is a valid value to
> index ctrl->phys. So, I updated SERDES_MAX to be SERDES6G_MAX + 1
> in include/dt-bindings/phy/phy-ocelot-serdes.h.
>
> Then I changed the condition in the for loo
for loop from
> i <= SERDES_MAX to i < SERDES_MAX.
>
Reviewed-by: Quentin Schulz
Thanks,
Quentin
signature.asc
Description: PGP signature
for loop from
> i <= SERDES_MAX to i < SERDES_MAX.
>
Reviewed-by: Quentin Schulz
Thanks,
Quentin
signature.asc
Description: PGP signature
, because the size of
> array ctrl->phys is SERDES_MAX.
>
> Partially fix this by updating SERDES_MAX to be SERDES6G_MAX + 1.
>
> Notice that this is the first part of the solution to
> the out-of-bounds bug mentioned above. Although this
> change is not dependent on any
, because the size of
> array ctrl->phys is SERDES_MAX.
>
> Partially fix this by updating SERDES_MAX to be SERDES6G_MAX + 1.
>
> Notice that this is the first part of the solution to
> the out-of-bounds bug mentioned above. Although this
> change is not dependent on any
Hi Gustavo,
On Mon, Oct 08, 2018 at 08:06:49PM +0200, Gustavo A. R. Silva wrote:
> Currently, there is an out-of-bounds read on array ctrl->phys,
> once variable i reaches the maximum array size of SERDES_MAX
> in the for loop.
>
> Fix this by changing the condition in the for loop from
> i <=
Hi Gustavo,
On Mon, Oct 08, 2018 at 08:06:49PM +0200, Gustavo A. R. Silva wrote:
> Currently, there is an out-of-bounds read on array ctrl->phys,
> once variable i reaches the maximum array size of SERDES_MAX
> in the for loop.
>
> Fix this by changing the condition in the for loop from
> i <=
to EEE in order to improve performance.
Reviewed-by: Florian Fainelli
Signed-off-by: Raju Lakkaraju
Signed-off-by: Quentin Schulz
---
drivers/net/phy/mscc.c | 65 ++
1 file changed, 65 insertions(+)
diff --git a/drivers/net/phy/mscc.c b/drivers/net/phy
-by: Andrew Lunn
Signed-off-by: Raju Lakkaraju
Signed-off-by: Quentin Schulz
---
drivers/net/phy/mscc.c | 119 +
1 file changed, 119 insertions(+)
diff --git a/drivers/net/phy/mscc.c b/drivers/net/phy/mscc.c
index 52198be46c68..47fbab489287 100644
to EEE in order to improve performance.
Reviewed-by: Florian Fainelli
Signed-off-by: Raju Lakkaraju
Signed-off-by: Quentin Schulz
---
drivers/net/phy/mscc.c | 65 ++
1 file changed, 65 insertions(+)
diff --git a/drivers/net/phy/mscc.c b/drivers/net/phy
-by: Andrew Lunn
Signed-off-by: Raju Lakkaraju
Signed-off-by: Quentin Schulz
---
drivers/net/phy/mscc.c | 119 +
1 file changed, 119 insertions(+)
diff --git a/drivers/net/phy/mscc.c b/drivers/net/phy/mscc.c
index 52198be46c68..47fbab489287 100644
r_supply_property psp,
> + const union power_supply_propval *val)
> +{
Argh, missed this one in the first version. Since you're introducing it
with the AXP813 and it isn't used with the AXP20X, I'd name it
axp813_ac_power_set_property. I'll let Maxime or Chen-Yu confirm though.
With the modification in the header from the previous patch in this
patch,
Reviewed-by: Quentin Schulz
Thanks!
Quentin
signature.asc
Description: PGP signature
r_supply_property psp,
> + const union power_supply_propval *val)
> +{
Argh, missed this one in the first version. Since you're introducing it
with the AXP813 and it isn't used with the AXP20X, I'd name it
axp813_ac_power_set_property. I'll let Maxime or Chen-Yu confirm though.
With the modification in the header from the previous patch in this
patch,
Reviewed-by: Quentin Schulz
Thanks!
Quentin
signature.asc
Description: PGP signature
Hi Oskari,
On Sun, Oct 07, 2018 at 12:18:35AM +0300, Oskari Lemmela wrote:
> AXP803 is compatible with AXP813.
> Adding needed cells for AC and battery power supplies.
>
> AXP813 AC power supply cell added.
>
> Signed-off-by: Oskari Lemmela
> ---
> drivers/mfd/axp20x.c | 22
Hi Oskari,
On Sun, Oct 07, 2018 at 12:18:35AM +0300, Oskari Lemmela wrote:
> AXP803 is compatible with AXP813.
> Adding needed cells for AC and battery power supplies.
>
> AXP813 AC power supply cell added.
>
> Signed-off-by: Oskari Lemmela
> ---
> drivers/mfd/axp20x.c | 22
Hi Oskari,
On Sun, Oct 07, 2018 at 12:18:33AM +0300, Oskari Lemmela wrote:
> AXP803 is compatible with AXP813. Add DT nodes ADC, GPIO,
> AC and battery power supplies.
>
> Signed-off-by: Oskari Lemmela
> ---
> arch/arm64/boot/dts/allwinner/axp803.dtsi | 31 +++
> 1 file
Hi Oskari,
On Sun, Oct 07, 2018 at 12:18:33AM +0300, Oskari Lemmela wrote:
> AXP803 is compatible with AXP813. Add DT nodes ADC, GPIO,
> AC and battery power supplies.
>
> Signed-off-by: Oskari Lemmela
> ---
> arch/arm64/boot/dts/allwinner/axp803.dtsi | 31 +++
> 1 file
Hi Oskari,
On Sun, Oct 07, 2018 at 12:18:32AM +0300, Oskari Lemmela wrote:
> Add AC power supply subnode for AXP81X PMIC.
>
> Signed-off-by: Oskari Lemmela
> ---
> arch/arm/boot/dts/axp81x.dtsi | 5 +
> 1 file changed, 5 insertions(+)
>
> diff --git a/arch/arm/boot/dts/axp81x.dtsi
Hi Oskari,
On Sun, Oct 07, 2018 at 12:18:32AM +0300, Oskari Lemmela wrote:
> Add AC power supply subnode for AXP81X PMIC.
>
> Signed-off-by: Oskari Lemmela
> ---
> arch/arm/boot/dts/axp81x.dtsi | 5 +
> 1 file changed, 5 insertions(+)
>
> diff --git a/arch/arm/boot/dts/axp81x.dtsi
Hi Oskari,
On Sun, Oct 07, 2018 at 12:18:30AM +0300, Oskari Lemmela wrote:
> AXP813 AC power supply support with input current and
> voltage limiting support.
>
> AXP803 AC and battery power supply support.
>
Great to see a new developer taking interest in those PMICs :)
I received the v2
Hi Oskari,
On Sun, Oct 07, 2018 at 12:18:30AM +0300, Oskari Lemmela wrote:
> AXP813 AC power supply support with input current and
> voltage limiting support.
>
> AXP803 AC and battery power supply support.
>
Great to see a new developer taking interest in those PMICs :)
I received the v2
Hi Oskari,
On Thu, Oct 04, 2018 at 10:34:10PM +0300, Oskari Lemmela wrote:
> AXP803 PMIC is register compatible with AXP813.
>
> Added support for AXP803/AXP813 AC power supply.
> AXP8x3 is capable to limit input current and minimum input voltage.
> Both of these register values are writeable.
>
Hi Oskari,
On Thu, Oct 04, 2018 at 10:34:10PM +0300, Oskari Lemmela wrote:
> AXP803 PMIC is register compatible with AXP813.
>
> Added support for AXP803/AXP813 AC power supply.
> AXP8x3 is capable to limit input current and minimum input voltage.
> Both of these register values are writeable.
>
Hi Andrew,
On Fri, Sep 14, 2018 at 07:27:54PM +0200, Andrew Lunn wrote:
>
> > struct vsc8531_private {
> > int rate_magic;
> > u16 supp_led_modes;
> > @@ -181,6 +354,7 @@ struct vsc8531_private {
> > struct vsc85xx_hw_stat *hw_stats;
> > u64 *stats;
> > int nstats;
> > +
Hi Andrew,
On Fri, Sep 14, 2018 at 07:27:54PM +0200, Andrew Lunn wrote:
>
> > struct vsc8531_private {
> > int rate_magic;
> > u16 supp_led_modes;
> > @@ -181,6 +354,7 @@ struct vsc8531_private {
> > struct vsc85xx_hw_stat *hw_stats;
> > u64 *stats;
> > int nstats;
> > +
Hi Andrew,
On Fri, Sep 14, 2018 at 07:02:21PM +0200, Andrew Lunn wrote:
> On Fri, Sep 14, 2018 at 06:26:38PM +0200, Quentin Schulz wrote:
> > Hi Alexandre,
> >
> > On Fri, Sep 14, 2018 at 04:54:46PM +0200, Alexandre Belloni wrote:
> > > Hi,
> > >
>
Hi Andrew,
On Fri, Sep 14, 2018 at 07:02:21PM +0200, Andrew Lunn wrote:
> On Fri, Sep 14, 2018 at 06:26:38PM +0200, Quentin Schulz wrote:
> > Hi Alexandre,
> >
> > On Fri, Sep 14, 2018 at 04:54:46PM +0200, Alexandre Belloni wrote:
> > > Hi,
> > >
>
`if (x != 0)` is basically a more verbose version of `if (x)` so let's
use the latter so it's consistent throughout the whole driver.
Signed-off-by: Quentin Schulz
---
drivers/net/phy/mscc.c | 30 +++---
1 file changed, 15 insertions(+), 15 deletions(-)
diff --git
`if (x != 0)` is basically a more verbose version of `if (x)` so let's
use the latter so it's consistent throughout the whole driver.
Signed-off-by: Quentin Schulz
---
drivers/net/phy/mscc.c | 30 +++---
1 file changed, 15 insertions(+), 15 deletions(-)
diff --git
Here, the rc variable is either used only for the condition right after
the assignment or right before being used as the return value of the
function it's being used in.
So let's remove this unneeded temporary variable whenever possible.
Signed-off-by: Quentin Schulz
---
drivers/net/phy/mscc.c
Here, the rc variable is either used only for the condition right after
the assignment or right before being used as the return value of the
function it's being used in.
So let's remove this unneeded temporary variable whenever possible.
Signed-off-by: Quentin Schulz
---
drivers/net/phy/mscc.c
Since HSIO address space can be used by different drivers (PLL, SerDes
muxing, temperature sensor), let's move it somewhere it can be included
by all drivers.
Acked-by: Alexandre Belloni
Signed-off-by: Quentin Schulz
---
drivers/net/ethernet/mscc/ocelot.h | 1 +-
drivers/net/ethernet
Since HSIO address space can be used by different drivers (PLL, SerDes
muxing, temperature sensor), let's move it somewhere it can be included
by all drivers.
Acked-by: Alexandre Belloni
Signed-off-by: Quentin Schulz
---
drivers/net/ethernet/mscc/ocelot.h | 1 +-
drivers/net/ethernet
Hi David,
On Tue, Sep 04, 2018 at 10:48:27AM -0700, David Miller wrote:
> From: Quentin Schulz
> Date: Mon, 3 Sep 2018 10:48:47 +0200
>
> > LEDs modes are set the same way, except they are offset by 4 times the
> > index of the LED.
> >
> > Let's factorize
Hi David,
On Tue, Sep 04, 2018 at 10:48:27AM -0700, David Miller wrote:
> From: Quentin Schulz
> Date: Mon, 3 Sep 2018 10:48:47 +0200
>
> > LEDs modes are set the same way, except they are offset by 4 times the
> > index of the LED.
> >
> > Let's factorize
The Microsemi Ocelot can mux SerDes lanes (aka macros) to different
switch ports or even make it act as a PCIe interface.
This adds support for the muxing of the SerDes.
Signed-off-by: Quentin Schulz
---
drivers/phy/Kconfig | 1 +
drivers/phy/Makefile | 1
The Microsemi Ocelot can mux SerDes lanes (aka macros) to different
switch ports or even make it act as a PCIe interface.
This adds support for the muxing of the SerDes.
Signed-off-by: Quentin Schulz
---
drivers/phy/Kconfig | 1 +
drivers/phy/Makefile | 1
for the deferring of the probe in the SerDes driver,
- use constants in the Device Tree to select the SerDes macro in use with
a port,
- adapt the SerDes driver to use those constants,
- add a header file in include/dt-bindings for the constants,
- fix space/tab issue,
Quentin Schulz (11
-by: Quentin Schulz
---
drivers/net/ethernet/mscc/ocelot.h | 73
drivers/net/ethernet/mscc/ocelot_regs.c | 92 +++--
include/soc/mscc/ocelot_hsio.h | 74
3 files changed, 83 insertions(+), 156 deletions(-)
diff --git a/drivers/net
for the deferring of the probe in the SerDes driver,
- use constants in the Device Tree to select the SerDes macro in use with
a port,
- adapt the SerDes driver to use those constants,
- add a header file in include/dt-bindings for the constants,
- fix space/tab issue,
Quentin Schulz (11
-by: Quentin Schulz
---
drivers/net/ethernet/mscc/ocelot.h | 73
drivers/net/ethernet/mscc/ocelot_regs.c | 92 +++--
include/soc/mscc/ocelot_hsio.h | 74
3 files changed, 83 insertions(+), 156 deletions(-)
diff --git a/drivers/net
Hi Linus,
On Mon, Aug 06, 2018 at 01:06:23PM +0200, Linus Walleij wrote:
> Hi Quentin, sorry for delays!
>
No worries :)
> On Wed, Jul 25, 2018 at 2:27 PM Quentin Schulz
> wrote:
>
> > This GPIO controller can serve as an interrupt controller as well on the
Hi Linus,
On Mon, Aug 06, 2018 at 01:06:23PM +0200, Linus Walleij wrote:
> Hi Quentin, sorry for delays!
>
No worries :)
> On Wed, Jul 25, 2018 at 2:27 PM Quentin Schulz
> wrote:
>
> > This GPIO controller can serve as an interrupt controller as well on the
The GPIO controller also serves as an interrupt controller for events
on the GPIO it handles.
An interrupt occurs whenever a GPIO line has changed.
Signed-off-by: Quentin Schulz
---
arch/mips/boot/dts/mscc/ocelot.dtsi | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch/mips/boot/dts
only after the line gets hold in its default
state, this is what's done with the fasteoi functions.
Only IRQ_TYPE_EDGE_BOTH and IRQ_TYPE_LEVEL_HIGH are supported for now.
Signed-off-by: Quentin Schulz
---
drivers/pinctrl/Kconfig | 1 +
drivers/pinctrl/pinctrl-ocelot.c | 102
The GPIO controller also serves as an interrupt controller for events
on the GPIO it handles.
An interrupt occurs whenever a GPIO line has changed.
Signed-off-by: Quentin Schulz
---
arch/mips/boot/dts/mscc/ocelot.dtsi | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch/mips/boot/dts
only after the line gets hold in its default
state, this is what's done with the fasteoi functions.
Only IRQ_TYPE_EDGE_BOTH and IRQ_TYPE_LEVEL_HIGH are supported for now.
Signed-off-by: Quentin Schulz
---
drivers/pinctrl/Kconfig | 1 +
drivers/pinctrl/pinctrl-ocelot.c | 102
-by: Quentin Schulz
---
arch/mips/boot/dts/mscc/ocelot.dtsi | 16
1 file changed, 16 insertions(+)
diff --git a/arch/mips/boot/dts/mscc/ocelot.dtsi
b/arch/mips/boot/dts/mscc/ocelot.dtsi
index 7096915f26e0..d7f0e3551500 100644
--- a/arch/mips/boot/dts/mscc/ocelot.dtsi
+++ b/arch/mips
-by: Quentin Schulz
---
arch/mips/boot/dts/mscc/ocelot.dtsi | 16
1 file changed, 16 insertions(+)
diff --git a/arch/mips/boot/dts/mscc/ocelot.dtsi
b/arch/mips/boot/dts/mscc/ocelot.dtsi
index 7096915f26e0..d7f0e3551500 100644
--- a/arch/mips/boot/dts/mscc/ocelot.dtsi
+++ b/arch/mips
The length of memory address space for MIIM0 is from 0x7107009c to
0x710700bf included which is 36 bytes long in decimal, or 0x24 bytes in
hexadecimal and not 0x36.
Fixes: 49b031690abe ("MIPS: mscc: Add switch to ocelot")
Signed-off-by: Quentin Schulz
---
arch/mips/boot/dts/mscc/o
The length of memory address space for MIIM0 is from 0x7107009c to
0x710700bf included which is 36 bytes long in decimal, or 0x24 bytes in
hexadecimal and not 0x36.
Fixes: 49b031690abe ("MIPS: mscc: Add switch to ocelot")
Signed-off-by: Quentin Schulz
---
arch/mips/boot/dts/mscc/o
Brezillon
Signed-off-by: Quentin Schulz
Reviewed-by: Boris Brezillon
---
drivers/mtd/ubi/cdev.c | 11 +++
drivers/mtd/ubi/vmt.c | 3 +++
include/uapi/mtd/ubi-user.h | 18 --
3 files changed, 30 insertions(+), 2 deletions(-)
diff --git a/drivers/mtd/ubi
Brezillon
Signed-off-by: Quentin Schulz
Reviewed-by: Boris Brezillon
---
drivers/mtd/ubi/cdev.c | 11 +++
drivers/mtd/ubi/vmt.c | 3 +++
include/uapi/mtd/ubi-user.h | 18 --
3 files changed, 30 insertions(+), 2 deletions(-)
diff --git a/drivers/mtd/ubi
,
Quentin Schulz (2):
ubi: provide a way to skip CRC checks
ubi: expose the volume CRC check skip flag
drivers/mtd/ubi/cdev.c | 11 +++
drivers/mtd/ubi/kapi.c | 2 +-
drivers/mtd/ubi/ubi-media.h | 6 ++
drivers/mtd/ubi/ubi.h | 4
drivers/mtd/ubi/vmt.c | 12
,
Quentin Schulz (2):
ubi: provide a way to skip CRC checks
ubi: expose the volume CRC check skip flag
drivers/mtd/ubi/cdev.c | 11 +++
drivers/mtd/ubi/kapi.c | 2 +-
drivers/mtd/ubi/ubi-media.h | 6 ++
drivers/mtd/ubi/ubi.h | 4
drivers/mtd/ubi/vmt.c | 12
at the block granularity instead of verifying
the whole volume.
Skipping this test drastically improves the boot-time.
Suggested-by: Boris Brezillon
Signed-off-by: Quentin Schulz
Reviewed-by: Boris Brezillon
Reviewed-by: Richard Weinberger
---
drivers/mtd/ubi/kapi.c | 2 +-
drivers/mtd
at the block granularity instead of verifying
the whole volume.
Skipping this test drastically improves the boot-time.
Suggested-by: Boris Brezillon
Signed-off-by: Quentin Schulz
Reviewed-by: Boris Brezillon
Reviewed-by: Richard Weinberger
---
drivers/mtd/ubi/kapi.c | 2 +-
drivers/mtd
:
- fix a few typos,
v2:
- use volume flags instead of arguments on the kernel command line as
suggested by Richard,
Quentin Schulz (2):
ubi: provide a way to skip CRC checks
ubi: expose the volume CRC check skip flag
drivers/mtd/ubi/cdev.c | 7 +++
drivers/mtd/ubi/kapi.c
volume for which we are trying to skip the CRC check
is static.
Suggested-by: Boris Brezillon
Signed-off-by: Quentin Schulz
Reviewed-by: Boris Brezillon
---
drivers/mtd/ubi/cdev.c | 7 +++
drivers/mtd/ubi/vmt.c | 3 +++
include/uapi/mtd/ubi-user.h | 18 --
3
:
- fix a few typos,
v2:
- use volume flags instead of arguments on the kernel command line as
suggested by Richard,
Quentin Schulz (2):
ubi: provide a way to skip CRC checks
ubi: expose the volume CRC check skip flag
drivers/mtd/ubi/cdev.c | 7 +++
drivers/mtd/ubi/kapi.c
volume for which we are trying to skip the CRC check
is static.
Suggested-by: Boris Brezillon
Signed-off-by: Quentin Schulz
Reviewed-by: Boris Brezillon
---
drivers/mtd/ubi/cdev.c | 7 +++
drivers/mtd/ubi/vmt.c | 3 +++
include/uapi/mtd/ubi-user.h | 18 --
3
at the block granularity instead of verifying
the whole volume.
Skipping this test drastically improves the boot-time.
Suggested-by: Boris Brezillon
Signed-off-by: Quentin Schulz
Reviewed-by: Boris Brezillon
Reviewed-by: Richard Weinberger
---
drivers/mtd/ubi/kapi.c | 2 +-
drivers/mtd
at the block granularity instead of verifying
the whole volume.
Skipping this test drastically improves the boot-time.
Suggested-by: Boris Brezillon
Signed-off-by: Quentin Schulz
Reviewed-by: Boris Brezillon
Reviewed-by: Richard Weinberger
---
drivers/mtd/ubi/kapi.c | 2 +-
drivers/mtd
> > Am Donnerstag, 28. Juni 2018, 09:40:53 CEST schrieb Quentin Schulz:
> > > > Now that we have the logic for skipping CRC check for static UBI volumes
> > > > in the core, let's expose it to users.
> > > >
> > > > This makes use of a padd
> > Am Donnerstag, 28. Juni 2018, 09:40:53 CEST schrieb Quentin Schulz:
> > > > Now that we have the logic for skipping CRC check for static UBI volumes
> > > > in the core, let's expose it to users.
> > > >
> > > > This makes use of a padd
at the block granularity instead of verifying
the whole volume.
Skipping this test drastically improves the boot-time.
Suggested-by: Boris Brezillon
Signed-off-by: Quentin Schulz
Reviewed-by: Boris Brezillon
---
drivers/mtd/ubi/kapi.c | 2 +-
drivers/mtd/ubi/ubi-media.h | 6
volume for which we are trying to skip the CRC check
is static.
Suggested-by: Boris Brezillon
Signed-off-by: Quentin Schulz
Reviewed-by: Boris Brezillon
---
drivers/mtd/ubi/cdev.c | 4
drivers/mtd/ubi/vmt.c | 3 +++
include/uapi/mtd/ubi-user.h | 16 ++--
3 files
volume for which we are trying to skip the CRC check
is static.
Suggested-by: Boris Brezillon
Signed-off-by: Quentin Schulz
Reviewed-by: Boris Brezillon
---
drivers/mtd/ubi/cdev.c | 4
drivers/mtd/ubi/vmt.c | 3 +++
include/uapi/mtd/ubi-user.h | 16 ++--
3 files
1 - 100 of 1305 matches
Mail list logo