From: Jarkko Nikula <jarkko.nikula-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
To: Luis Oliveira
<Luis.Oliveira-HKixBCOQz3hWk0Htik3J/w@public.gmane.org>,
wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
mark.rutland-5wv7dgnIgG8@public.gmane.org,
andriy.shevchenko-VuQAYsv1563Yd54FQh9/CA@public.gmane.org,
mika.westerberg-VuQAYsv1563Yd54FQh9/CA@public.gmane.org,
linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: Ramiro.Oliveira-HKixBCOQz3hWk0Htik3J/w@public.gmane.org,
Joao.Pinto-HKixBCOQz3hWk0Htik3J/w@public.gmane.org,
CARLOS.PALMINHA-HKixBCOQz3hWk0Htik3J/w@public.gmane.org
Subject: Re: [PATCH v7 3/6] i2c: designware: MASTER mode as separated driver
Date: Wed, 22 Mar 2017 16:38:42 +0200 [thread overview]
Message-ID: <92bee7a2-4320-cff7-ef18-9770d6fe12fe@linux.intel.com> (raw)
In-Reply-To: <c224fa537fc1daf9f3256ce7c8da12f73f7035a7.1490007620.git.lolivei-HKixBCOQz3hWk0Htik3J/w@public.gmane.org>
On 03/20/17 13:10, Luis Oliveira wrote:
> - The functions related to I2C master mode of operation were transformed
> in a single driver.
> - Common definitions were moved to i2c-designware-core.h
> - The i2c-designware-core is now only a library file, the functions
> associated are in a source file called i2c-designware-common and
> are used by both i2c-designware-master and i2c-designware-slave.
> - To decrease noise in namespace common i2c_dw_*() functions are
> now using ops to keep them private.
> - Designware PCI driver had to be changed to match the previous ops
> functions implementation.
>
> Almost all of the "core" source is now part of the "master" source. The
> difference is the functions used by both modes and they are in the
> "common" source file.
>
> Signed-off-by: Luis Oliveira <lolivei-HKixBCOQz3hWk0Htik3J/w@public.gmane.org>
> Acked-by: Andy Shevchenko <andriy.shevchenko-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
> ---
> v6-v7
> - Moved kernel doc comments from patch 4/6 to this patch. @Jarkko
> - Fix Designware I2C PCI driver error when looking for previous extern
> functions.
>
> drivers/i2c/busses/Makefile | 1 +
> drivers/i2c/busses/i2c-designware-common.c | 274 +++++++++++++++
> drivers/i2c/busses/i2c-designware-core.h | 146 +++++++-
> ...c-designware-core.c => i2c-designware-master.c} | 367 +--------------------
> drivers/i2c/busses/i2c-designware-pcidrv.c | 8 +-
> drivers/i2c/busses/i2c-designware-platdrv.c | 6 +-
> 6 files changed, 433 insertions(+), 369 deletions(-)
> create mode 100644 drivers/i2c/busses/i2c-designware-common.c
> rename drivers/i2c/busses/{i2c-designware-core.c => i2c-designware-master.c} (64%)
>
Acked-by: Jarkko Nikula <jarkko.nikula-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: Jarkko Nikula <jarkko.nikula@linux.intel.com>
To: Luis Oliveira <Luis.Oliveira@synopsys.com>,
wsa@the-dreams.de, robh+dt@kernel.org, mark.rutland@arm.com,
andriy.shevchenko@linux.intel.com,
mika.westerberg@linux.intel.com, linux-i2c@vger.kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Cc: Ramiro.Oliveira@synopsys.com, Joao.Pinto@synopsys.com,
CARLOS.PALMINHA@synopsys.com
Subject: Re: [PATCH v7 3/6] i2c: designware: MASTER mode as separated driver
Date: Wed, 22 Mar 2017 16:38:42 +0200 [thread overview]
Message-ID: <92bee7a2-4320-cff7-ef18-9770d6fe12fe@linux.intel.com> (raw)
In-Reply-To: <c224fa537fc1daf9f3256ce7c8da12f73f7035a7.1490007620.git.lolivei@synopsys.com>
On 03/20/17 13:10, Luis Oliveira wrote:
> - The functions related to I2C master mode of operation were transformed
> in a single driver.
> - Common definitions were moved to i2c-designware-core.h
> - The i2c-designware-core is now only a library file, the functions
> associated are in a source file called i2c-designware-common and
> are used by both i2c-designware-master and i2c-designware-slave.
> - To decrease noise in namespace common i2c_dw_*() functions are
> now using ops to keep them private.
> - Designware PCI driver had to be changed to match the previous ops
> functions implementation.
>
> Almost all of the "core" source is now part of the "master" source. The
> difference is the functions used by both modes and they are in the
> "common" source file.
>
> Signed-off-by: Luis Oliveira <lolivei@synopsys.com>
> Acked-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> ---
> v6-v7
> - Moved kernel doc comments from patch 4/6 to this patch. @Jarkko
> - Fix Designware I2C PCI driver error when looking for previous extern
> functions.
>
> drivers/i2c/busses/Makefile | 1 +
> drivers/i2c/busses/i2c-designware-common.c | 274 +++++++++++++++
> drivers/i2c/busses/i2c-designware-core.h | 146 +++++++-
> ...c-designware-core.c => i2c-designware-master.c} | 367 +--------------------
> drivers/i2c/busses/i2c-designware-pcidrv.c | 8 +-
> drivers/i2c/busses/i2c-designware-platdrv.c | 6 +-
> 6 files changed, 433 insertions(+), 369 deletions(-)
> create mode 100644 drivers/i2c/busses/i2c-designware-common.c
> rename drivers/i2c/busses/{i2c-designware-core.c => i2c-designware-master.c} (64%)
>
Acked-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
next prev parent reply other threads:[~2017-03-22 14:38 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-20 11:10 [PATCH v7 0/6] i2c: designware: add I2C SLAVE support Luis Oliveira
2017-03-20 11:10 ` [PATCH v7 1/6] i2c: designware: Cleaning and comment style fixes Luis Oliveira
2017-03-20 11:10 ` [PATCH v7 2/6] i2c: designware: refactoring of the i2c-designware Luis Oliveira
2017-03-20 11:10 ` [PATCH v7 3/6] i2c: designware: MASTER mode as separated driver Luis Oliveira
[not found] ` <c224fa537fc1daf9f3256ce7c8da12f73f7035a7.1490007620.git.lolivei-HKixBCOQz3hWk0Htik3J/w@public.gmane.org>
2017-03-22 14:38 ` Jarkko Nikula [this message]
2017-03-22 14:38 ` Jarkko Nikula
2017-03-22 14:47 ` Jarkko Nikula
2017-03-22 14:56 ` Luis Oliveira
2017-03-22 14:56 ` Luis Oliveira
2017-03-20 11:10 ` [PATCH v7 4/6] i2c: designware: introducing I2C_SLAVE definitions Luis Oliveira
2017-03-22 14:39 ` Jarkko Nikula
[not found] ` <55d2ccf235080a77347ad9e3a34263ca9cc8f80d.1490007620.git.lolivei-HKixBCOQz3hWk0Htik3J/w@public.gmane.org>
2017-03-22 14:53 ` Jarkko Nikula
2017-03-22 14:53 ` Jarkko Nikula
2017-03-20 11:10 ` [PATCH v7 5/6] i2c: designware: add SLAVE mode functions Luis Oliveira
2017-03-22 15:02 ` Jarkko Nikula
2017-03-20 11:10 ` [PATCH v7 6/6] i2c: designware: enable SLAVE in platform module Luis Oliveira
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=92bee7a2-4320-cff7-ef18-9770d6fe12fe@linux.intel.com \
--to=jarkko.nikula-vuqaysv1563yd54fqh9/ca@public.gmane.org \
--cc=CARLOS.PALMINHA-HKixBCOQz3hWk0Htik3J/w@public.gmane.org \
--cc=Joao.Pinto-HKixBCOQz3hWk0Htik3J/w@public.gmane.org \
--cc=Luis.Oliveira-HKixBCOQz3hWk0Htik3J/w@public.gmane.org \
--cc=Ramiro.Oliveira-HKixBCOQz3hWk0Htik3J/w@public.gmane.org \
--cc=andriy.shevchenko-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \
--cc=mika.westerberg-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \
--cc=robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.