linux-kernel - Re: [PATCH] dw_dmac: don't exceed AHB master number in dwc_get_data

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAHp75VegokmDnXTPHYE=kKzwAgiuD9ySg4wRxxZFGkuXdQoStw@mail.gmail.com>
Date:	Tue, 15 Jan 2013 11:08:04 +0200
From:	Andy Shevchenko <andy.shevchenko@...il.com>
To:	Viresh Kumar <viresh.kumar@...aro.org>
Cc:	Andy Shevchenko <andriy.shevchenko@...ux.intel.com>,
	Vinod Koul <vinod.koul@...el.com>,
	linux-kernel@...r.kernel.org, spear-devel <spear-devel@...t.st.com>
Subject: Re: [PATCH] dw_dmac: don't exceed AHB master number in dwc_get_data_width

On Tue, Jan 15, 2013 at 11:01 AM, Viresh Kumar <viresh.kumar@...aro.org> wrote:
> On Tue, Jan 15, 2013 at 12:35 PM, Andy Shevchenko
> <andy.shevchenko@...il.com> wrote:
>>> The details of the patch are fine, but i didn't get how do you get
>>> master 1 selected
>>> on a system where you have only 1 master?
>>
>> min_t(..., nr_masters - 1, m) defines this. m is returned value with
>> default settings, let's say 1. In that case if nr_masters is also 1
>> the min_t(..., 1-1, 1) returns 0.
>>
>> The value of nr_masters itself is come from autoconfig or platform data.
>
> :)
> You didn't get me. How this code would work is pretty easy to understand.
> What i wanted to know is, the master information is coming from slave drivers
> and they must select master zero always. How would master 1 be selected ever
> in this driver.

You forgot the mem2mem type of transfers where we have no information
about masters except default settings. Otherwise there is the case of
custom slave config absence when we supply only request line via
slave_id.

In the rest you are right.

-- 
With Best Regards,
Andy Shevchenko
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/