{"action":"created","issue":{"url":"https://api.github.com/repos/qca/open-plc-utils/issues/33","labels_url":"https://api.github.com/repos/qca/open-plc-utils/issues/33/labels{/name}","comments_url":"https://api.github.com/repos/qca/open-plc-utils/issues/33/comments","events_url":"https://api.github.com/repos/qca/open-plc-utils/issues/33/events","html_url":"https://github.com/qca/open-plc-utils/pull/33","id":29003727,"number":33,"title":"Psin qca7000","user":{"login":"ch-aurich","id":428731,"avatar_url":"https://avatars.githubusercontent.com/u/428731?v=3","gravatar_id":"","url":"https://api.github.com/users/ch-aurich","html_url":"https://github.com/ch-aurich","followers_url":"https://api.github.com/users/ch-aurich/followers","following_url":"https://api.github.com/users/ch-aurich/following{/other_user}","gists_url":"https://api.github.com/users/ch-aurich/gists{/gist_id}","starred_url":"https://api.github.com/users/ch-aurich/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ch-aurich/subscriptions","organizations_url":"https://api.github.com/users/ch-aurich/orgs","repos_url":"https://api.github.com/users/ch-aurich/repos","events_url":"https://api.github.com/users/ch-aurich/events{/privacy}","received_events_url":"https://api.github.com/users/ch-aurich/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2014-03-07T22:25:50Z","updated_at":"2015-04-21T17:32:11Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/qca/open-plc-utils/pulls/33","html_url":"https://github.com/qca/open-plc-utils/pull/33","diff_url":"https://github.com/qca/open-plc-utils/pull/33.diff","patch_url":"https://github.com/qca/open-plc-utils/pull/33.patch"},"body":"psin did not work for QCA7000 so far. This branch tries to fix that. I have no hardware here to try it out at the moment... maybe someone can have a look around and point me to possible issues. I will test it next week when I am in my office again.\r\nAt least it would be great if someone can verify that it still works works for the other chips like before ;)\r\n\r\nChristian"},"comment":{"url":"https://api.github.com/repos/qca/open-plc-utils/issues/comments/94880689","html_url":"https://github.com/qca/open-plc-utils/pull/33#issuecomment-94880689","issue_url":"https://api.github.com/repos/qca/open-plc-utils/issues/33","id":94880689,"user":{"login":"razzlefratz","id":3029262,"avatar_url":"https://avatars.githubusercontent.com/u/3029262?v=3","gravatar_id":"","url":"https://api.github.com/users/razzlefratz","html_url":"https://github.com/razzlefratz","followers_url":"https://api.github.com/users/razzlefratz/followers","following_url":"https://api.github.com/users/razzlefratz/following{/other_user}","gists_url":"https://api.github.com/users/razzlefratz/gists{/gist_id}","starred_url":"https://api.github.com/users/razzlefratz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/razzlefratz/subscriptions","organizations_url":"https://api.github.com/users/razzlefratz/orgs","repos_url":"https://api.github.com/users/razzlefratz/repos","events_url":"https://api.github.com/users/razzlefratz/events{/privacy}","received_events_url":"https://api.github.com/users/razzlefratz/received_events","type":"User","site_admin":false},"created_at":"2015-04-21T17:32:11Z","updated_at":"2015-04-21T17:32:11Z","body":"PS. I will be cherry-picking through your pull requests. You have some\ngreat stuff in there.\n\nOn Tue, Apr 21, 2015 at 10:27 AM, Charles Maier <razzlefratz@gmail.com>\nwrote:\n\n> Hi Christian,\n>\n> The SLAC tools, PEV and EVSE, are valid because the SLAC protocol has not\n> changed, to my knowledge. Since SLAC is only implemented in the QCA7000,\n> the PIB settings should not have changed.\n>\n> The format and location of the pre-scalers has changed with each new\n> chipset in an effort to conserve memory and support the increase the number\n> of carriers. Starting with the QCA6410, QCA7000 and AR7420 chipsets,\n> everything changed forcing us to write the plc* series of tools. By\n> everything, I mean PIB structure, flash memory layout and firmware upgrade\n> procedure. The pre-scaler tools (ps*) have not been upgraded to match for\n> two reasons. The most important reason is that the location of the\n> pre-scaler set within the PIB is no longer fixed and may change from one\n> major firmware release to another. The firmware teams does not see this as\n> a problem because they claim that folks should not tamper with pre-scalers.\n> All this may take time to resolve.\n>\n> Regarding your suggestion, most of the tools detect either the PLC device\n> type or PIB version. Some do not. My solution is to keep programs as simple\n> as possible by creating a new version to each tool for each chipset\n> generation. That way we do not break earlier programs. I tried one\n> implementation where programs support multiple chipset, firmware and PIB\n> versions and it quickly got ugly. There were (and still are) corner cases\n> that make those tools unusable over the network. That implementation was\n> release to one customer for production purposes only.\n>\n> The amp* tools are for the AR7400 and QCA7450 and are mature.  The plc*\n> tools are for the QCA6410, QCA7000 and AR7420 but those chipsets introduced\n> relocatable PIB parameters and the pre-scaler formats on the QCA7000 are\n> different than the QCA6410 and QR7420 so the ps* tools do not work on these\n> chipsets.\n>\n> I hope this answers your question,\n>\n> On Sat, Apr 18, 2015 at 2:21 PM, Christian Aurich <\n> notifications@github.com> wrote:\n>\n>> For some of our customers (using the QCA7000 but not having NDA with QCA)\n>> this is one of the most usefull feature. They need to modify the prescalers\n>> and enable SLAC. They usually want to do it $SOMEHOW - it does not really\n>> matter how.\n>>\n>> If I understand it correctly the tools are working correctly but you\n>> doubt that it is always used in a way that guarantees that results in a\n>> correct PIB file. Is this correct? If this is the only problem we could add\n>> a parameter where the user must specify which chip(-set) he uses.\n>>\n>> —\n>> Reply to this email directly or view it on GitHub\n>> <https://github.com/qca/open-plc-utils/pull/33#issuecomment-94201950>.\n>>\n>\n>\n"}}