+Neil
On Mon, 23 Jul 2018 15:06:43 -0700
Brian Norris wrote:
> Hi Boris,
>
> On Mon, Jul 23, 2018 at 1:10 PM, Boris Brezillon
> wrote:
> > On Mon, 23 Jul 2018 11:13:50 -0700
> > Brian Norris wrote:
> >> I noticed this got merged, but I wanted to put my 2 ce
On Tue, 24 Jul 2018 08:46:33 +1000
NeilBrown wrote:
> On Mon, Jul 23 2018, Brian Norris wrote:
>
> > Hi Boris,
> >
> > On Mon, Jul 23, 2018 at 1:10 PM, Boris Brezillon
> > wrote:
> >> On Mon, 23 Jul 2018 11:13:50 -0700
> >> Brian Norris wrote:
x27;} [-Wformat=]
> ../drivers/mtd/maps/solutionengine.c:62:72: note: format string is defined
> here
> printk(KERN_NOTICE "Solution Engine: Flash at 0x%08lx, EPROM at 0x%08lx\n",
> ^
>
On Tue, 24 Jul 2018 12:52:02 -0700
Brian Norris wrote:
> > > Or even better: put this hack behind a DT flag, so that one has to
> > > admit that their board design is broken before it will even do
> > > anything. Proposal: "linux,badly-designed-flash-reset".
> >
> > I think we can remove the "
On Sat, 21 Jul 2018 18:08:13 +0200
Nicholas Mc Guire wrote:
> wait_for_completion_timeout returns an unsigned long not int. declare a
> suitably type timeout and fix up assignment and check.
>
> Signed-off-by: Nicholas Mc Guire
> Reported-by: Vignesh R
> Fixes: 140623410536 ("mtd: spi-nor: Add
On Wed, 25 Jul 2018 18:42:44 +0900
Masahiro Yamada wrote:
>
> You need to remove kfree(denali->buf)
> from denali_remove(), right?
Absolutely.
>
> void denali_remove(struct denali_nand_info *denali)
> {
> struct mtd_info *mtd = nand_to_mtd(&denali->nand);
>
> nand_release(mtd
On Wed, 25 Jul 2018 16:56:28 +0200
Miquel Raynal wrote:
> Mails to wenyou.y...@microchip.com are not deliverable.
> Drop him as Microchip/Atmel NAND controller driver maintainer.
>
> Signed-off-by: Miquel Raynal
Acked-by: Boris Brezillon
> ---
> MAINTAINERS | 1 -
&g
On Wed, 25 Jul 2018 16:56:28 +0200
Miquel Raynal wrote:
> Mails to wenyou.y...@microchip.com are not deliverable.
> Drop him as Microchip/Atmel NAND controller driver maintainer.
>
> Signed-off-by: Miquel Raynal
> ---
> MAINTAINERS | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/MAINT
>
> Signed-off-by: Eric Anholt
Acked-by: Boris Brezillon
A few comments below (no need to address them, those are just minor
things that can be fixed later on, or things I'm not comfortable
with but cannot be addressed easily).
> ---
>
> This patch is just a proposal fo
On Thu, 11 May 2017 11:31:27 -0700
Eric Anholt wrote:
> drm_encoder_cleanup() finishes with memsetting it to 0, already.
>
> Signed-off-by: Eric Anholt
Acked-by: Boris Brezillon
> ---
> drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c | 8 +---
> 1 file changed,
way.
>
> v2: Fix build with CONFIG_DRM=m, drop "dev" argument that should just
> be the panel's dev, move kerneldoc up a level and document
> _remove().
>
> Signed-off-by: Eric Anholt
> Acked-by: Daniel Vetter
Reviewed-by: Boris Brezillon
>
"dev" argument.
>
> Signed-off-by: Eric Anholt
> Acked-by: Daniel Vetter
Reviewed-by: Boris Brezillon
> ---
> drivers/gpu/drm/vc4/Kconfig | 2 +-
> drivers/gpu/drm/vc4/vc4_dsi.c | 154
> ++
> 2 files changed, 21 insertio
On Thu, 11 May 2017 11:31:24 -0700
Eric Anholt wrote:
> Another 100 lines of boilerplate gone, while allowing for bridges to
> be connected in the display chain.
>
> Signed-off-by: Eric Anholt
Reviewed-by: Boris Brezillon
> ---
> drivers/gpu/drm/v
ould be when you do things that require the link to be
> brought up), but there may be issues here.
>
> Signed-off-by: Eric Anholt
Reviewed-by: Boris Brezillon
> ---
>
> Note that I haven't tested this change, and am not committed to this
> patch. It's ju
On Thu, 11 May 2017 11:31:26 -0700
Eric Anholt wrote:
> Another 100 lines of boilerplate gone. Bridges aren't supported yet,
> but will be trivial to add later.
>
> Signed-off-by: Eric Anholt
> ---
>
[...]
> @@ -1082,28 +993,13 @@ int ltdc_load(struct drm_device *ddev)
>
> DRM_INFO(
Hi Masahiro,
Sorry for the late reply.
On Mon, 8 May 2017 12:40:47 +0900
Masahiro Yamada wrote:
> Hi Boris,
>
>
> 2017-04-29 1:32 GMT+09:00 Boris Brezillon
> :
>
> >> + for (setting = caps->ecc_settings; setting->step; setting++) {
> >> +
On Sat, 13 May 2017 08:10:53 -0300
Mauro Carvalho Chehab wrote:
> This patch series convert the following books to ReST:
> - librs
> - mtdnand
> - sh
>
> And it is based on my previous series of conversion patches.
>
> After this series, there will be just one DocBook pending
Hi Christoph,
On Wed, 10 Jan 2018 16:48:17 +0100
Christoph Hellwig wrote:
> On Wed, Jan 10, 2018 at 04:25:22PM +0100, Boris Brezillon wrote:
> > On Wed, 10 Jan 2018 15:15:43 +
> > Robin Murphy wrote:
> >
> > > phys_to_dma() is an internal helper for c
On Fri, 12 Jan 2018 11:57:55 +0100
Arnd Bergmann wrote:
> On Fri, Jan 12, 2018 at 11:49 AM, Olof's autobuilder wrote:
> > Here are the build results from automated periodic testing.
> >
> > The tree being built was stable-rc, found at:
> >
> > URL:
> > git://git.kernel.org/pub/scm/linux/kernel/
On Thu, 11 Jan 2018 17:26:59 +0100
Nicolas Ferre wrote:
> Update Wenyou Yang email address.
> Take advantage of this update to move this entry to the MICROCHIP / ATMEL
> location and add the DT binding documentation link.
>
> Signed-off-by: Nicolas Ferre
> Acked-by: Wenyou Yang
Applied.
Than
On Tue, 30 Jan 2018 08:44:30 +0900
KOBAYASHI Yoshitake wrote:
> On 2017/12/27 15:06, KOBAYASHI Yoshitake wrote:
> > On 2017/12/19 20:56, Boris Brezillon wrote:
> >> On Tue, 19 Dec 2017 20:42:36 +0900
> >> KOBAYASHI Yoshitake wrote:
> >>
> >&g
Alexey Khoroshilov
> ---
> v2: Add nand_cleanup() to undone nand_scan_tail() as Boris Brezillon noted.
> v3: Rename error labels, remove of_node_put() per Boris Brezillon request.
Should be separated in 3 patches IMO:
1/ remove the unnecessary of_node_put()
2/ rename error label in
Hi Stefan,
On Mon, 29 Jan 2018 15:44:40 +0100
Stefan Agner wrote:
> In case fsl,use-minimum-ecc is set, the driver tries to determine
> ECC layout by using the ECC information provided by the MTD stack.
> However, in case the NAND chip does not provide any information,
> the driver currently fai
On Tue, 30 Jan 2018 14:46:47 +0100
Miquel Raynal wrote:
> Hi Geert,
>
> On Tue, 30 Jan 2018 14:23:21 +0100
> Geert Uytterhoeven wrote:
>
> > If NO_DMA=y:
> >
> > ERROR: "bad_dma_ops" [drivers/mtd/nand/marvell_nand.ko] undefined!
> >
> > Add a dependency on HAS_DMA to fix this.
> >
> > F
On Thu, 18 Jan 2018 15:11:32 +0100
Boris Brezillon wrote:
> On Thu, 18 Jan 2018 15:08:01 +0100
> Boris Brezillon wrote:
>
> > On Thu, 18 Jan 2018 14:05:05 +
> > Wei Yongjun wrote:
> >
> > > Fix to return error code -ENOMEM from the kmem_cache_alloc
On Wed, 17 Jan 2018 11:25:33 +
Wei Yongjun wrote:
> There is a error message within devm_ioremap_resource
> already, so remove the dev_err call to avoid redundant
> error message.
Applied.
Thanks,
Boris
>
> Signed-off-by: Wei Yongjun
> ---
> drivers/mtd/onenand/omap2.c | 4 +---
> 1 fi
On Fri, 19 Jan 2018 07:55:31 +
Colin King wrote:
> From: Colin Ian King
>
> Trivial fix to spelling mistakes in dev_err error message text.
Applied.
Thanks,
Boris
>
> Signed-off-by: Colin Ian King
> ---
> drivers/mtd/nand/marvell_nand.c | 2 +-
> 1 file changed, 1 insertion(+), 1 del
On Fri, 19 Jan 2018 07:59:54 +
Colin King wrote:
> From: Colin Ian King
>
> Variable oob_len is assigned and never read, hence it is redundant and
> can be removed.
>
> Cleans up clang warnings:
>
> drivers/mtd/nand/marvell_nand.c:1356:6: warning: Value stored to 'oob_len'
> during its in
Hi Rob,
On Sun, 7 Jan 2018 15:14:25 +0100
Boris Brezillon wrote:
> Hi Rob,
>
> On Tue, 26 Dec 2017 12:29:34 -0600
> Rob Herring wrote:
>
> > >> > > > +Optional properties
> > >> > > > +---
> > >>
undone nand_scan_tail() as Boris Brezillon noted.
>
> drivers/mtd/nand/vf610_nfc.c | 25 +++--
> 1 file changed, 15 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/mtd/nand/vf610_nfc.c b/drivers/mtd/nand/vf610_nfc.c
> index 8037d4b48a05..2dac25a8ccbf 1006
undone nand_scan_tail() as Boris Brezillon noted.
>
> drivers/mtd/nand/vf610_nfc.c | 25 +++--
> 1 file changed, 15 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/mtd/nand/vf610_nfc.c b/drivers/mtd/nand/vf610_nfc.c
> index 8037d4b48a05..2dac25a8ccbf 1006
On Fri, 22 Dec 2017 11:09:15 +0100
Andrea Adami wrote:
> On Wed, Dec 20, 2017 at 6:45 AM, Wei Yongjun wrote:
> > Fixes the following sparse warnings:
> >
> > drivers/mtd/parsers/sharpslpart.c:222:6: warning:
> > symbol 'sharpsl_nand_cleanup_ftl' was not declared. Should it be static?
> >
> > Si
Hi Rob,
On Tue, 26 Dec 2017 12:29:34 -0600
Rob Herring wrote:
> >> > > > +Optional properties
> >> > > > +---
> >> > > > +- reg: static address. Only valid is the device has a static
> >> > > > address.
> >> > > > +- i3c-dynamic-address: dynamic address to be assigned to this
>
On Tue, 26 Dec 2017 14:59:00 +0100
Cyrille Pitchen wrote:
> Hi Vignesh
>
> Le 26/12/2017 à 14:42, Vignesh R a écrit :
> > Hi Cyrille,
> >
> > Thanks for doing this series! One comment below.
> >
> > On 24-Dec-17 10:06 AM, Cyrille Pitchen wrote:
> > [...]
> >> diff --git a/drivers/mtd/spi-nor
Hi Cyrille,
On Sun, 24 Dec 2017 05:36:04 +0100
Cyrille Pitchen wrote:
> This patch has two purposes:
>
> 1 - To fix the compatible issue between the MTD and SPI sub-systems
>
> The MTD sub-system has no particular requirement about the memory areas it
> uses. Especially, ubifs is well known fo
On Mon, 8 Jan 2018 08:23:02 +0100
Daniel Lezcano wrote:
> On 07/01/2018 19:44, Alexandre Belloni wrote:
> > On 07/01/2018 at 19:07:13 +0100, Daniel Lezcano wrote:
> >> On 05/01/2018 15:30, Alexandre Belloni wrote:
> >>> With the new TCB clocksource driver, atmel platforms are now able to boot
Hi Arnd,
On Mon, 8 Jan 2018 14:16:25 +0100
Arnd Bergmann wrote:
> On Mon, Jan 8, 2018 at 12:25 PM, kernelci.org bot wrote:
> >
> > stable-rc/linux-4.4.y build: 178 builds: 4 failed, 174 passed, 8 errors
> > (v4.4.110-18-g5da3d9af3a4b)
> > Full Build Summary:
> > https://kernelci.org/build/sta
On Mon, 8 Jan 2018 15:01:38 +0100
Arnd Bergmann wrote:
> On Mon, Jan 8, 2018 at 2:44 PM, gregkh wrote:
> > On Mon, Jan 08, 2018 at 02:16:25PM +0100, Arnd Bergmann wrote:
> >> On Mon, Jan 8, 2018 at 12:25 PM, kernelci.org bot
> >> wrote:
> >> >
> >> > stable-rc/linux-4.4.y build: 178 builds
cron memories
Boris Brezillon (15):
mtd: nand: hynix: Don't wait after applying new read-retry params
mtd: nand: provide several helpers to do common NAND operations
mtd: nand: force drivers to explicitly send READ/PROG commands
mtd: nand: denali: Avoid using ecc->co
g each RCL job to make sure that the sampling of the
> previous RCL's output is correct.
>
> Fixes flickering in the top left of 3DMMES Taiji.
>
> Signed-off-by: Eric Anholt
> Fixes: ca26d28bbaa3 ("drm/vc4: improve throughput by pipelining binning and
>
On Thu, 18 Jan 2018 14:05:05 +
Wei Yongjun wrote:
> Fix to return error code -ENOMEM from the kmem_cache_alloc() error
> handling case instead of 0, as done elsewhere in this function.
I guess you've used a static analysis code to detect this problem, can
you name it in the commit message, a
On Thu, 18 Jan 2018 15:08:01 +0100
Boris Brezillon wrote:
> On Thu, 18 Jan 2018 14:05:05 +
> Wei Yongjun wrote:
>
> > Fix to return error code -ENOMEM from the kmem_cache_alloc() error
> > handling case instead of 0, as done elsewhere in this function.
>
> I
On Tue, 9 Jan 2018 11:56:20 +0100
Maxime Ripard wrote:
> There's a bunch of drivers that duplicate the same function to know if a
> particular format embeds an alpha component or not.
>
> Let's create a helper to avoid duplicating that logic.
>
> Cc: Boris Br
On Tue, 9 Jan 2018 11:56:21 +0100
Maxime Ripard wrote:
> Now that the core has a drm format helper to tell if a format embeds an
> alpha component in it, let's use it.
>
> Cc: Boris Brezillon
Acked-by: Boris Brezillon
> Signed-off-by: Maxime Ripard
> ---
> d
On Tue, 9 Jan 2018 11:56:25 +0100
Maxime Ripard wrote:
> Some drivers duplicate the logic to create a property to store a per-plane
> alpha.
>
> Let's create a helper in order to move that to the core.
>
> Cc: Boris Brezillon
Reviewed-by: Boris Brezillon
> Cc:
On Tue, 9 Jan 2018 11:56:26 +0100
Maxime Ripard wrote:
> Now that we have support for per-plane alpha in the core, let's use it.
>
> Cc: Boris Brezillon
Acked-by: Boris Brezillon
> Signed-off-by: Maxime Ripard
> ---
> drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.h
didn't exist when the offending code was first merged, but it does now.
>
> Signed-off-by: Robin Murphy
Acked-by: Boris Brezillon
> ---
>
> Found by inspection and compile-tested only
>
> drivers/crypto/marvell/cesa.c | 19 ---
> 1 file changed, 1
On Sat, 13 Jan 2018 17:56:16 +0100
Christophe JAILLET wrote:
> 'page_buf' and 'oob_buf' are allocated with 'devm_kzalloc()' and should not
> have __iommem decoration.
^ __iomem
No need to send a new version, I'll fix it when applying.
>
> Remove these decorations and some useless cast
On Sat, 13 Jan 2018 17:56:16 +0100
Christophe JAILLET wrote:
> 'page_buf' and 'oob_buf' are allocated with 'devm_kzalloc()' and should not
> have __iommem decoration.
>
> Remove these decorations and some useless casting.
>
Applied.
Thanks,
Boris
> Signed-off-by: Christophe JAILLET
> ---
>
On Tue, 16 Jan 2018 08:43:40 +0100
Arnd Bergmann wrote:
> The omap2 onenand driver is now available for compile-testing, which
> uncovers a warning in configurations that have a 64-bit resource_size_t:
>
> drivers/mtd/onenand/omap2.c: In function 'omap2_onenand_probe':
> drivers/mtd/onenand/omap
> Author: Rafał Miłecki
> Date: Thu Jan 4 08:05:34 2018 +0100
>
> mtd: ofpart: add of_match_table with "fixed-partitions"
>
> This allows using this parser with any flash driver that takes care of
> setting of_node (using mtd_set_of_node helper) correct
On Tue, 16 Jan 2018 14:56:52 +0100
Peter Rosin wrote:
> On 2018-01-16 14:36, Boris Brezillon wrote:
> > Hi Peter,
> >
> > On Tue, 16 Jan 2018 14:28:38 +0100
> > Peter Rosin wrote:
> >
> >> Hi!
> >>
> >> When trying next-20180116 I
On Tue, 16 Jan 2018 16:02:35 +0100
Peter Rosin wrote:
> On 2018-01-16 15:21, Boris Brezillon wrote:
> > On Tue, 16 Jan 2018 14:56:52 +0100
> > Peter Rosin wrote:
> >
> >> Hmmm, I guess the question is if the command line should override the
> >> device
ion);
> + ret = mtd_ooblayout_ecc(mtd, i, &oobregion);
Ditto.
> if (ret < 0) {
> if (ret != -ERANGE)
> return ret;
Thanks,
Boris
--
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
(mtd, section, &oobregion);
> + ret = mtd_ooblayout_ecc(mtd, section++, &oobregion);
> if (ret < 0) {
> if (ret != -ERANGE)
> return ret;
--
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
{ \
> [...]
>
> The right-hand side of this comparison is now using val2 instead of
> val3. (This bug seems to be unfixed upstream.)
Indeed. This being said, it's not buggy since all users of
map_word_andequal() pass the same value to val2 and val3.
Maybe we should just patch the macro and all call-sites to remove val3.
>
> Ben.
>
--
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
doesn't appear to do any harm.
I wonder why the fix was backported to stable releases in the first
place. AFAICS, there's no Cc-stable or Fixes tag in the original
commit. It's probably something in the backport-to-stable process I'm
not aware of.
Anyway, not an issues since the changes seems to be harmless.
--
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
ut any further
modification, and leaving info->archdata to NULL has the exact same
effect (see [1]).
Regards,
Boris
[1]https://elixir.bootlin.com/linux/v4.16-rc6/source/drivers/i2c/i2c-core-base.c#L711
--
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
closer look
and it seems the only user of info->archdata is i2c-core-of.c, so we
can even remove the info->archdata field.
--
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
Hi Geert,
On Mon, 26 Mar 2018 12:22:24 +0200
Geert Uytterhoeven wrote:
> Hi Boris,
>
> On Fri, Mar 23, 2018 at 12:00 PM, Boris Brezillon
> wrote:
> > From: Boris Brezillon
> >
> > A new I3C subsystem has been added and a generic description has been
> >
Hi Geert,
On Mon, 26 Mar 2018 12:17:26 +0200
Geert Uytterhoeven wrote:
> Hi Boris,
>
> On Fri, Mar 23, 2018 at 12:00 PM, Boris Brezillon
> wrote:
> > Document the Cadence I3C gpio expander bindings.
> >
> > Signed-off-by: Boris Brezillon
>
> Thanks
On Mon, 26 Mar 2018 12:12:54 +0200
Geert Uytterhoeven wrote:
> Hi Boris,
>
> On Fri, Mar 23, 2018 at 12:00 PM, Boris Brezillon
> wrote:
> > Document the Cadence I3C gpio expander bindings.
> >
> > Signed-off-by: Boris Brezillon
>
> Thanks for your p
mtd: ftl: Use DIV_ROUND_UP()
Boris Brezillon (23):
mtd: Make sure the device supports erase operations in mtd_erase()
mtd: nand: Get rid of comments giving the file path inside the file itself
mtd: nand: Stop using full path when referring to files placed in the
same dir
mtd: nand
Hi Ladislav,
On Wed, 11 Apr 2018 08:26:07 +0200
Ladislav Michl wrote:
> Hi Andreas,
>
> On Wed, Apr 11, 2018 at 06:59:03AM +0200, Andreas Kemnade wrote:
> > Hi Ladis,
> >
> > On Tue, 10 Apr 2018 22:56:43 +0200
> > Ladislav Michl wrote:
> >
> > > Hi Nikolaus,
> > >
> > > On Tue, Apr 10, 20
On Wed, 11 Apr 2018 09:36:56 +0200
Ladislav Michl wrote:
> Hi Boris,
>
> On Wed, Apr 11, 2018 at 09:15:28AM +0200, Boris Brezillon wrote:
> > Hi Ladislav,
> >
> > On Wed, 11 Apr 2018 08:26:07 +0200
> > Ladislav Michl wrote:
> >
> > > Hi And
On Wed, 11 Apr 2018 10:27:46 +0200
Ladislav Michl wrote:
> On Wed, Apr 11, 2018 at 10:08:06AM +0200, Boris Brezillon wrote:
> > On Wed, 11 Apr 2018 09:36:56 +0200
> > Ladislav Michl wrote:
> >
> > > Hi Boris,
> > >
> > > On Wed, Apr 11,
On Wed, 11 Apr 2018 16:44:10 +0200
Peter Rosin wrote:
> Hi Nicolas,
>
> Boris asked for your input on this (the datasheet difference appears to
> have no bearing on the issue) elsewhere in the tree of messages. It's
> now been a week or so and I'm starting to wonder if you missed this
> altogeth
On Wed, 11 Apr 2018 17:10:43 +0200
Peter Rosin wrote:
> On 2018-04-11 16:59, Boris Brezillon wrote:
> > On Wed, 11 Apr 2018 16:44:10 +0200
> > Peter Rosin wrote:
> >
> >> Hi Nicolas,
> >>
> >> Boris asked for your input on this (the datasheet d
On Mon, 9 Apr 2018 13:58:13 -0700
Eric Anholt wrote:
> Signed-off-by: Eric Anholt
> Fixes: 65101d8c9108 ("drm/vc4: Expose performance counters to userspace")
Reviewed-by: Boris Brezillon
> ---
> drivers/gpu/drm/vc4/vc4_drv.c | 1 +
> 1 file changed, 1 insertion(+)
On Wed, 11 Apr 2018 11:12:24 +0200
Ladislav Michl wrote:
> On Wed, Apr 11, 2018 at 10:52:01AM +0200, Boris Brezillon wrote:
> > On Wed, 11 Apr 2018 10:27:46 +0200
> > Ladislav Michl wrote:
> >
> > > On Wed, Apr 11, 2018 at 10:08:06AM +0200, Boris Brezillon w
gt; @@ -2157,8 +2162,7 @@ static int cfi_intelext_lock(struct mtd_info *mtd,
> loff_t ofs, uint64_t len)
> ofs, len, DO_XXLOCK_ONEBLOCK_LOCK);
>
> #ifdef DEBUG_LOCK_BITS
> - printk(KERN_DEBUG "%s: lock status after, ret=%d\n",
> -
, this way maybe I'll have reviews
on the doc itself :-).
Thanks,
Boris
Boris Brezillon (11):
i2c: Export of_i2c_get_board_info()
i3c: Add core I3C infrastructure
docs: driver-api: Add I3C documentation
i3c: Add sysfs ABI spec
dt-bindings: i3c: Document core bindings
dt-bi
From: Boris Brezillon
Add the I3C documentation describing the protocol, the master driver API
and the device driver API.
Signed-off-by: Boris Brezillon
---
Changes in v2:
- Moved out of patch "i3c: Add core I3C infrastructure"
- Add link to the I3C spec
- Move rst files in Doc
Document the Cadence I3C gpio expander bindings.
Signed-off-by: Boris Brezillon
---
.../devicetree/bindings/gpio/gpio-cdns-i3c.txt | 38 ++
1 file changed, 38 insertions(+)
create mode 100644 Documentation/devicetree/bindings/gpio/gpio-cdns-i3c.txt
diff --git a
Add a driver for Cadence I3C GPIO expander.
Signed-off-by: Boris Brezillon
---
drivers/gpio/Kconfig | 11 ++
drivers/gpio/Makefile| 1 +
drivers/gpio/gpio-cdns-i3c.c | 380 +++
3 files changed, 392 insertions(+)
create mode 100644
From: Boris Brezillon
I3C busses have to know about all I2C devices connected on the I3C bus
to properly initialize the I3C master, and I2C frames can't be sent on
the bus until this initialization is done.
We can't let the I2C core parse the DT and instantiate I2C devices as
p
From: Boris Brezillon
Add a driver for Cadence I3C master IP.
Signed-off-by: Boris Brezillon
---
Changes in v3:
- Adjust to match I3C framework changes
- Implement support the CMD RESPONSE QUEUE and IBI QUEUE added in the
latest revision of Cadence master IP
- Remove support for HDR modes
From: Boris Brezillon
Document Cadence I3C master DT bindings.
Signed-off-by: Boris Brezillon
---
.../devicetree/bindings/i3c/cdns,i3c-master.txt| 45 ++
1 file changed, 45 insertions(+)
create mode 100644 Documentation/devicetree/bindings/i3c/cdns,i3c-master.txt
The reg property of devices connected to an I3C bus have 3 cells, and
filling them manually is not trivial. Provides macros to help doing
that.
Signed-off-by: Boris Brezillon
---
include/dt-bindings/i3c/i3c.h | 28
1 file changed, 28 insertions(+)
create mode
Create an entry for the I3C subsystem and mark it as maintained by me.
There's no official git repository, patchwork instance, mailing list or
website yet, but this will be added after the subsystem has been
accepted.
Signed-off-by: Boris Brezillon
---
MAINTAINERS | 9 +
1 file ch
From: Boris Brezillon
A new I3C subsystem has been added and a generic description has been
created to represent the I3C bus and the devices connected on it.
Document this generic representation.
Signed-off-by: Boris Brezillon
---
Changes in v3:
- Rename {i2c,i3c}-scl-frequency DT prop into
On Fri, 23 Mar 2018 12:00:09 +0100
Boris Brezillon wrote:
> This patch series is a proposal for a new I3C [1] subsystem.
>
> This infrastructure is not complete yet and will be extended over
> time.
>
> There are a few design choices that are worth mentioning because they
&g
From: Boris Brezillon
Add core infrastructure to support I3C in Linux and document it.
This infrastructure is not complete yet and will be extended over
time.
There are a few design choices that are worth mentioning because they
impact the way I3C device drivers can interact with their devices
From: Boris Brezillon
Document sysfs files/directories/symlinks exposed by the I3C subsystem.
Signed-off-by: Boris Brezillon
---
Changes in v2:
- new patch
---
Documentation/ABI/testing/sysfs-bus-i3c | 95 +
1 file changed, 95 insertions(+)
create mode 100644
ot;;
> > + i2c-scl-frequency = <10>;
>
> Another s/frequency/hz/ instance, similar to those reported by Thomas.
Will fix it in v4.
Thanks,
Boris
--
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
On Fri, 23 Mar 2018 12:10:35 +0100
Thomas Petazzoni wrote:
> Hello,
>
> On Fri, 23 Mar 2018 12:00:18 +0100, Boris Brezillon wrote:
>
> > +Optional properties defined by the generic binding (see
> > +Documentation/devicetree/bindings/i3c/i3c.txt for more details):
> &
16 insertions(+), 26 deletions(-)
--
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
On Tue, 10 Apr 2018 09:55:58 +0200
Miquel Raynal wrote:
> > Hi Abhishek,
> >
> > On Tue, 10 Apr 2018 11:39:35 +0530, Abhishek Sahu
> > wrote:
> >
> > > On 2018-04-06 18:01, Miquel Raynal wrote:
> > > > Hi Abhishek,
> > > >
> > > > On Wed, 4 Apr 2018 18:12:17 +0530, Abhishek Sahu
> > >
Hi Marcin
On Mon, 9 Apr 2018 20:15:51 -0400
Marcin Ziemianowicz wrote:
> This is a series of patches which resolves set_rate() for the PLL not
> having any effect and therefore the USB Host port not working. Also, a
> few messages were added which may be helpful in the future when others
> are w
Hi Marcin,
On Mon, 9 Apr 2018 20:16:21 -0400
Marcin Ziemianowicz wrote:
> I noticed that when debugging some USB clocking issue that there weren't
> many ways to tell what the state of the USB clocking system was. This
> adds a few logging statements to see what the relevant code is trying to
>
Hi Marcin,
On Mon, 9 Apr 2018 20:16:49 -0400
Marcin Ziemianowicz wrote:
> When a USB device is connected to the USB host port on the SAM9N12 then
> you get "-62" error which seems to indicate USB replies from the device
> are timing out. Looking around, I saw the USB bus was running at half
> sp
support for hardware that "only works by mistake" may be good
> idea, but maybe it is slightly too surprising for a -stable.
I wouldn't say "work by mistake" but "seems to work at first but in the
end breaks", so definitely a candidate for -stable IMO.
Regards,
Boris
[1]https://github.com/bbrezillon/linux/tree/nand/mlc
--
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
uch presumptuous
statements (you can have a look at these slides if you want some details
about why this is not so simple [1]).
I'm definitely not saying supporting MLC NANDs in Linux is impossible,
and if you're interested in working on this topic I'd be happy to help.
But please don&#
ill looks
wrong. It's probably better to move the of_node_put() and the below
idr_remove() call in the ->release() hook if you want to use
put_device().
> idr_remove(&mtd_idr, i);
> fail_locked:
--
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
On Wed, 5 Sep 2018 16:36:38 +0200
Ricardo Ribalda Delgado wrote:
> @@ -234,9 +234,11 @@ static int gpio_flash_probe(struct platform_device *pdev)
> state->map.copy_to= gf_copy_to;
> state->map.bankwidth = pdata->width;
> state->map.size = state->win_size * (1 << state
On Wed, 5 Sep 2018 16:36:42 +0200
Ricardo Ribalda Delgado wrote:
> +static int gpio_flash_probe_gpios(struct platform_device *pdev,
> + struct async_state *state)
> +{
> + struct physmap_flash_data *pdata;
> + struct device_node *dn;
> + struct resource
On Wed, 5 Sep 2018 16:36:40 +0200
Ricardo Ribalda Delgado wrote:
> By replacing the array with an integer we can avoid completely
> the bit comparison loop if the value has not changed (by far
> the most common case).
>
> Signed-off-by: Ricardo Ribalda Delgado
> ---
> drivers/mtd/maps/gpio-ad
> Changelog v2:
>
> From Boris Brezillon:
> -Add Fixes and cc:stable
>
> From kbuild:
> - Fix warnings
>
> - Rebase
>
> Ricardo Ribalda Delgado (8):
> mtd: maps: gpio-addr-flash: Replace custom printk
> mtd: maps: gpio-addr-flash: Fix ioremapped size
Hi Chuanhua,
On Fri, 21 Sep 2018 15:06:26 +0800
Chuanhua Han wrote:
> Before we add this spi_transfer to the spi_message chain table, we need
> bits_per_word_mask based on spi_control to set the bits_per_word of
> this spi_transfer.
It's not clear to me what you're trying to fix/improve. Can yo
Hi Chuanhua,
On Fri, 21 Sep 2018 15:06:27 +0800
Chuanhua Han wrote:
> This patch fixes the problem that the XSPI mode of the dspi controller
> cannot transfer data properly.
> In XSPI mode, cmd_fifo is written before tx_fifo, which transforms the
> byte order of sending and receiving data.
Agai
701 - 800 of 5738 matches
Mail list logo
