From: Vinod Koul <vkoul@kernel.org>
To: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Cc: alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org,
tiwai@suse.de, broonie@kernel.org, gregkh@linuxfoundation.org,
liam.r.girdwood@linux.intel.com, jank@cadence.com,
joe@perches.com, srinivas.kandagatla@linaro.org,
Sanyog Kale <sanyog.r.kale@intel.com>
Subject: Re: [PATCH v4 10/22] soundwire: mipi_disco: fix alignment issues
Date: Thu, 2 May 2019 11:04:51 +0530 [thread overview]
Message-ID: <20190502053451.GD3845@vkoul-mobl.Dlink> (raw)
In-Reply-To: <20190501155745.21806-11-pierre-louis.bossart@linux.intel.com>
On 01-05-19, 10:57, Pierre-Louis Bossart wrote:
> Use Linux style. In some cases parenthesis alignment is modified to
> keep the code readable.
lgtm, It would have been okay to have this and previous as a single
patch. It is the same module, but more split is welcome, makes it easier to
review in smaller chunks.
Thanks
>
> Reviewed-by: Takashi Iwai <tiwai@suse.de>
> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
> ---
> drivers/soundwire/mipi_disco.c | 112 ++++++++++++++++++---------------
> 1 file changed, 62 insertions(+), 50 deletions(-)
>
> diff --git a/drivers/soundwire/mipi_disco.c b/drivers/soundwire/mipi_disco.c
> index fdeba0c3b589..95efa82dc73b 100644
> --- a/drivers/soundwire/mipi_disco.c
> +++ b/drivers/soundwire/mipi_disco.c
> @@ -35,11 +35,12 @@ int sdw_master_read_prop(struct sdw_bus *bus)
> int nval, i;
>
> device_property_read_u32(bus->dev,
> - "mipi-sdw-sw-interface-revision", &prop->revision);
> + "mipi-sdw-sw-interface-revision",
> + &prop->revision);
>
> /* Find master handle */
> snprintf(name, sizeof(name),
> - "mipi-sdw-master-%d-subproperties", bus->link_id);
> + "mipi-sdw-master-%d-subproperties", bus->link_id);
>
> link = device_get_named_child_node(bus->dev, name);
> if (!link) {
> @@ -56,7 +57,8 @@ int sdw_master_read_prop(struct sdw_bus *bus)
> prop->clk_stop_mode |= SDW_CLK_STOP_MODE1;
>
> fwnode_property_read_u32(link,
> - "mipi-sdw-max-clock-frequency", &prop->max_freq);
> + "mipi-sdw-max-clock-frequency",
> + &prop->max_freq);
>
> nval = fwnode_property_read_u32_array(link,
> "mipi-sdw-clock-frequencies-supported", NULL, 0);
> @@ -64,7 +66,7 @@ int sdw_master_read_prop(struct sdw_bus *bus)
>
> prop->num_freq = nval;
> prop->freq = devm_kcalloc(bus->dev, prop->num_freq,
> - sizeof(*prop->freq), GFP_KERNEL);
> + sizeof(*prop->freq), GFP_KERNEL);
> if (!prop->freq)
> return -ENOMEM;
>
> @@ -91,44 +93,47 @@ int sdw_master_read_prop(struct sdw_bus *bus)
>
> prop->num_clk_gears = nval;
> prop->clk_gears = devm_kcalloc(bus->dev, prop->num_clk_gears,
> - sizeof(*prop->clk_gears), GFP_KERNEL);
> + sizeof(*prop->clk_gears),
> + GFP_KERNEL);
> if (!prop->clk_gears)
> return -ENOMEM;
>
> fwnode_property_read_u32_array(link,
> - "mipi-sdw-supported-clock-gears",
> - prop->clk_gears, prop->num_clk_gears);
> + "mipi-sdw-supported-clock-gears",
> + prop->clk_gears,
> + prop->num_clk_gears);
> }
>
> fwnode_property_read_u32(link, "mipi-sdw-default-frame-rate",
> - &prop->default_frame_rate);
> + &prop->default_frame_rate);
>
> fwnode_property_read_u32(link, "mipi-sdw-default-frame-row-size",
> - &prop->default_row);
> + &prop->default_row);
>
> fwnode_property_read_u32(link, "mipi-sdw-default-frame-col-size",
> - &prop->default_col);
> + &prop->default_col);
>
> prop->dynamic_frame = fwnode_property_read_bool(link,
> "mipi-sdw-dynamic-frame-shape");
>
> fwnode_property_read_u32(link, "mipi-sdw-command-error-threshold",
> - &prop->err_threshold);
> + &prop->err_threshold);
>
> return 0;
> }
> EXPORT_SYMBOL(sdw_master_read_prop);
>
> static int sdw_slave_read_dp0(struct sdw_slave *slave,
> - struct fwnode_handle *port, struct sdw_dp0_prop *dp0)
> + struct fwnode_handle *port,
> + struct sdw_dp0_prop *dp0)
> {
> int nval;
>
> fwnode_property_read_u32(port, "mipi-sdw-port-max-wordlength",
> - &dp0->max_word);
> + &dp0->max_word);
>
> fwnode_property_read_u32(port, "mipi-sdw-port-min-wordlength",
> - &dp0->min_word);
> + &dp0->min_word);
>
> nval = fwnode_property_read_u32_array(port,
> "mipi-sdw-port-wordlength-configs", NULL, 0);
> @@ -136,8 +141,8 @@ static int sdw_slave_read_dp0(struct sdw_slave *slave,
>
> dp0->num_words = nval;
> dp0->words = devm_kcalloc(&slave->dev,
> - dp0->num_words, sizeof(*dp0->words),
> - GFP_KERNEL);
> + dp0->num_words, sizeof(*dp0->words),
> + GFP_KERNEL);
> if (!dp0->words)
> return -ENOMEM;
>
> @@ -146,20 +151,21 @@ static int sdw_slave_read_dp0(struct sdw_slave *slave,
> dp0->words, dp0->num_words);
> }
>
> - dp0->flow_controlled = fwnode_property_read_bool(
> - port, "mipi-sdw-bra-flow-controlled");
> + dp0->flow_controlled = fwnode_property_read_bool(port,
> + "mipi-sdw-bra-flow-controlled");
>
> - dp0->simple_ch_prep_sm = fwnode_property_read_bool(
> - port, "mipi-sdw-simplified-channel-prepare-sm");
> + dp0->simple_ch_prep_sm = fwnode_property_read_bool(port,
> + "mipi-sdw-simplified-channel-prepare-sm");
>
> - dp0->device_interrupts = fwnode_property_read_bool(
> - port, "mipi-sdw-imp-def-dp0-interrupts-supported");
> + dp0->device_interrupts = fwnode_property_read_bool(port,
> + "mipi-sdw-imp-def-dp0-interrupts-supported");
>
> return 0;
> }
>
> static int sdw_slave_read_dpn(struct sdw_slave *slave,
> - struct sdw_dpn_prop *dpn, int count, int ports, char *type)
> + struct sdw_dpn_prop *dpn, int count, int ports,
> + char *type)
> {
> struct fwnode_handle *node;
> u32 bit, i = 0;
> @@ -173,7 +179,7 @@ static int sdw_slave_read_dpn(struct sdw_slave *slave,
>
> for_each_set_bit(bit, &addr, 32) {
> snprintf(name, sizeof(name),
> - "mipi-sdw-dp-%d-%s-subproperties", bit, type);
> + "mipi-sdw-dp-%d-%s-subproperties", bit, type);
>
> dpn[i].num = bit;
>
> @@ -184,9 +190,9 @@ static int sdw_slave_read_dpn(struct sdw_slave *slave,
> }
>
> fwnode_property_read_u32(node, "mipi-sdw-port-max-wordlength",
> - &dpn[i].max_word);
> + &dpn[i].max_word);
> fwnode_property_read_u32(node, "mipi-sdw-port-min-wordlength",
> - &dpn[i].min_word);
> + &dpn[i].min_word);
>
> nval = fwnode_property_read_u32_array(node,
> "mipi-sdw-port-wordlength-configs", NULL, 0);
> @@ -194,8 +200,9 @@ static int sdw_slave_read_dpn(struct sdw_slave *slave,
>
> dpn[i].num_words = nval;
> dpn[i].words = devm_kcalloc(&slave->dev,
> - dpn[i].num_words,
> - sizeof(*dpn[i].words), GFP_KERNEL);
> + dpn[i].num_words,
> + sizeof(*dpn[i].words),
> + GFP_KERNEL);
> if (!dpn[i].words)
> return -ENOMEM;
>
> @@ -205,28 +212,28 @@ static int sdw_slave_read_dpn(struct sdw_slave *slave,
> }
>
> fwnode_property_read_u32(node, "mipi-sdw-data-port-type",
> - &dpn[i].type);
> + &dpn[i].type);
>
> fwnode_property_read_u32(node,
> - "mipi-sdw-max-grouping-supported",
> - &dpn[i].max_grouping);
> + "mipi-sdw-max-grouping-supported",
> + &dpn[i].max_grouping);
>
> dpn[i].simple_ch_prep_sm = fwnode_property_read_bool(node,
> "mipi-sdw-simplified-channelprepare-sm");
>
> fwnode_property_read_u32(node,
> - "mipi-sdw-port-channelprepare-timeout",
> - &dpn[i].ch_prep_timeout);
> + "mipi-sdw-port-channelprepare-timeout",
> + &dpn[i].ch_prep_timeout);
>
> fwnode_property_read_u32(node,
> "mipi-sdw-imp-def-dpn-interrupts-supported",
> &dpn[i].device_interrupts);
>
> fwnode_property_read_u32(node, "mipi-sdw-min-channel-number",
> - &dpn[i].min_ch);
> + &dpn[i].min_ch);
>
> fwnode_property_read_u32(node, "mipi-sdw-max-channel-number",
> - &dpn[i].max_ch);
> + &dpn[i].max_ch);
>
> nval = fwnode_property_read_u32_array(node,
> "mipi-sdw-channel-number-list", NULL, 0);
> @@ -234,7 +241,8 @@ static int sdw_slave_read_dpn(struct sdw_slave *slave,
>
> dpn[i].num_ch = nval;
> dpn[i].ch = devm_kcalloc(&slave->dev, dpn[i].num_ch,
> - sizeof(*dpn[i].ch), GFP_KERNEL);
> + sizeof(*dpn[i].ch),
> + GFP_KERNEL);
> if (!dpn[i].ch)
> return -ENOMEM;
>
> @@ -265,13 +273,13 @@ static int sdw_slave_read_dpn(struct sdw_slave *slave,
> "mipi-sdw-modes-supported", &dpn[i].modes);
>
> fwnode_property_read_u32(node, "mipi-sdw-max-async-buffer",
> - &dpn[i].max_async_buffer);
> + &dpn[i].max_async_buffer);
>
> dpn[i].block_pack_mode = fwnode_property_read_bool(node,
> "mipi-sdw-block-packing-mode");
>
> fwnode_property_read_u32(node, "mipi-sdw-port-encoding-type",
> - &dpn[i].port_encoding);
> + &dpn[i].port_encoding);
>
> /* TODO: Read audio mode */
>
> @@ -293,7 +301,7 @@ int sdw_slave_read_prop(struct sdw_slave *slave)
> int num_of_ports, nval, i, dp0 = 0;
>
> device_property_read_u32(dev, "mipi-sdw-sw-interface-revision",
> - &prop->mipi_revision);
> + &prop->mipi_revision);
>
> prop->wake_capable = device_property_read_bool(dev,
> "mipi-sdw-wake-up-unavailable");
> @@ -311,10 +319,10 @@ int sdw_slave_read_prop(struct sdw_slave *slave)
> "mipi-sdw-simplified-clockstopprepare-sm-supported");
>
> device_property_read_u32(dev, "mipi-sdw-clockstopprepare-timeout",
> - &prop->clk_stop_timeout);
> + &prop->clk_stop_timeout);
>
> device_property_read_u32(dev, "mipi-sdw-slave-channelprepare-timeout",
> - &prop->ch_prep_timeout);
> + &prop->ch_prep_timeout);
>
> device_property_read_u32(dev,
> "mipi-sdw-clockstopprepare-hard-reset-behavior",
> @@ -333,13 +341,13 @@ int sdw_slave_read_prop(struct sdw_slave *slave)
> "mipi-sdw-port15-read-behavior", &prop->p15_behave);
>
> device_property_read_u32(dev, "mipi-sdw-master-count",
> - &prop->master_count);
> + &prop->master_count);
>
> device_property_read_u32(dev, "mipi-sdw-source-port-list",
> - &prop->source_ports);
> + &prop->source_ports);
>
> device_property_read_u32(dev, "mipi-sdw-sink-port-list",
> - &prop->sink_ports);
> + &prop->sink_ports);
>
> /* Read dp0 properties */
> port = device_get_named_child_node(dev, "mipi-sdw-dp-0-subproperties");
> @@ -348,7 +356,8 @@ int sdw_slave_read_prop(struct sdw_slave *slave)
> } else {
>
> prop->dp0_prop = devm_kzalloc(&slave->dev,
> - sizeof(*prop->dp0_prop), GFP_KERNEL);
> + sizeof(*prop->dp0_prop),
> + GFP_KERNEL);
> if (!prop->dp0_prop)
> return -ENOMEM;
>
> @@ -364,23 +373,25 @@ int sdw_slave_read_prop(struct sdw_slave *slave)
> /* Allocate memory for set bits in port lists */
> nval = hweight32(prop->source_ports);
> prop->src_dpn_prop = devm_kcalloc(&slave->dev, nval,
> - sizeof(*prop->src_dpn_prop), GFP_KERNEL);
> + sizeof(*prop->src_dpn_prop),
> + GFP_KERNEL);
> if (!prop->src_dpn_prop)
> return -ENOMEM;
>
> /* Read dpn properties for source port(s) */
> sdw_slave_read_dpn(slave, prop->src_dpn_prop, nval,
> - prop->source_ports, "source");
> + prop->source_ports, "source");
>
> nval = hweight32(prop->sink_ports);
> prop->sink_dpn_prop = devm_kcalloc(&slave->dev, nval,
> - sizeof(*prop->sink_dpn_prop), GFP_KERNEL);
> + sizeof(*prop->sink_dpn_prop),
> + GFP_KERNEL);
> if (!prop->sink_dpn_prop)
> return -ENOMEM;
>
> /* Read dpn properties for sink port(s) */
> sdw_slave_read_dpn(slave, prop->sink_dpn_prop, nval,
> - prop->sink_ports, "sink");
> + prop->sink_ports, "sink");
>
> /* some ports are bidirectional so check total ports by ORing */
> nval = prop->source_ports | prop->sink_ports;
> @@ -388,7 +399,8 @@ int sdw_slave_read_prop(struct sdw_slave *slave)
>
> /* Allocate port_ready based on num_of_ports */
> slave->port_ready = devm_kcalloc(&slave->dev, num_of_ports,
> - sizeof(*slave->port_ready), GFP_KERNEL);
> + sizeof(*slave->port_ready),
> + GFP_KERNEL);
> if (!slave->port_ready)
> return -ENOMEM;
>
> --
> 2.17.1
--
~Vinod
next prev parent reply other threads:[~2019-05-02 5:35 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-01 15:57 [PATCH v4 00/22] soundwire: code cleanup Pierre-Louis Bossart
2019-05-01 15:57 ` [PATCH v4 01/22] soundwire: Kconfig: fix help format Pierre-Louis Bossart
2019-05-01 15:57 ` [PATCH v4 02/22] soundwire: fix SPDX license for header files Pierre-Louis Bossart
2019-05-01 16:05 ` Greg KH
2019-05-02 5:16 ` Vinod Koul
2019-05-02 6:31 ` Greg KH
2019-05-02 6:44 ` Vinod Koul
2019-05-01 15:57 ` [PATCH v4 03/22] soundwire: fix alignment issues in " Pierre-Louis Bossart
2019-05-02 5:25 ` Vinod Koul
2019-05-01 15:57 ` [PATCH v4 04/22] soundwire: bus: fix alignment issues Pierre-Louis Bossart
2019-05-01 15:57 ` [PATCH v4 05/22] soundwire: bus: fix typos in comments Pierre-Louis Bossart
2019-05-01 15:57 ` [PATCH v4 06/22] soundwire: bus: remove useless parentheses Pierre-Louis Bossart
2019-05-02 5:32 ` Vinod Koul
2019-05-01 15:57 ` [PATCH v4 07/22] soundwire: bus: fix boolean comparisons Pierre-Louis Bossart
2019-05-01 15:57 ` [PATCH v4 08/22] soundwire: bus: remove spurious newline Pierre-Louis Bossart
2019-05-01 15:57 ` [PATCH v4 09/22] soundwire: bus_type: fix alignment issues Pierre-Louis Bossart
2019-05-01 15:57 ` [PATCH v4 10/22] soundwire: mipi_disco: " Pierre-Louis Bossart
2019-05-02 5:34 ` Vinod Koul [this message]
2019-05-01 15:57 ` [PATCH v4 11/22] soundwire: mipi_disco: fix boolean comparisons Pierre-Louis Bossart
2019-05-01 15:57 ` [PATCH v4 12/22] soundwire: stream: fix alignment issues Pierre-Louis Bossart
2019-05-01 15:57 ` [PATCH v4 13/22] soundwire: slave: " Pierre-Louis Bossart
2019-05-01 15:57 ` [PATCH v4 14/22] soundwire: intel_init: " Pierre-Louis Bossart
2019-05-01 15:57 ` [PATCH v4 15/22] soundwire: intel: " Pierre-Louis Bossart
2019-05-01 15:57 ` [PATCH v4 16/22] soundwire: intel: protect macro parameters Pierre-Louis Bossart
2019-05-01 15:57 ` [PATCH v4 17/22] soundwire: intel: fix boolean comparison Pierre-Louis Bossart
2019-05-01 15:57 ` [PATCH v4 18/22] soundwire: cadence_master: fix alignment issues Pierre-Louis Bossart
2019-05-01 15:57 ` [PATCH v4 19/22] soundwire: cadence_master: balance parentheses Pierre-Louis Bossart
2019-05-01 15:57 ` [PATCH v4 20/22] soundwire: cadence_master: fix boolean comparisons Pierre-Louis Bossart
2019-05-01 15:57 ` [PATCH v4 21/22] soundwire: cadence_master: remove spurious newline Pierre-Louis Bossart
2019-05-01 15:57 ` [PATCH v4 22/22] soundwire: add missing newlines in dynamic debug logs Pierre-Louis Bossart
2019-05-02 5:37 ` Vinod Koul
2019-05-02 6:32 ` Greg KH
2019-05-02 14:55 ` [alsa-devel] " Pierre-Louis Bossart
2019-05-01 16:07 ` [PATCH v4 00/22] soundwire: code cleanup Greg KH
2019-05-02 5:39 ` Vinod Koul
2019-05-02 5:58 ` Vinod Koul
2019-05-02 7:07 ` [alsa-devel] " Vinod Koul
2019-05-02 15:19 ` Greg KH
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=20190502053451.GD3845@vkoul-mobl.Dlink \
--to=vkoul@kernel.org \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=gregkh@linuxfoundation.org \
--cc=jank@cadence.com \
--cc=joe@perches.com \
--cc=liam.r.girdwood@linux.intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=pierre-louis.bossart@linux.intel.com \
--cc=sanyog.r.kale@intel.com \
--cc=srinivas.kandagatla@linaro.org \
--cc=tiwai@suse.de \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox